Missing .suspend / .resume definitions in durableClient.d.ts

[evanlouie]

Describe the bug

Looks like suspend/resume support was landed in 3.1.0 (#578) but durableClient.d.ts has not been updated to show the new methods.

Expected behavior

I should be able to see .suspend .resume on the DurableClient when coding in TypeScript

Actual behavior

Compiler cannot see .suspend and .resume because the d.ts file hasn't been updated.

[evanlouie]

Is there a reason why to use "declaration": false in the tsconfig.json? Seems like this sort of problem would be solved by having tsc automatically emit them and setting "types": "lib/src/index.d.ts" in package.json.

[davidmrdavid]

cc/ @erizba in case you have cycles.

@evanlouie I think we disabled tsc automatically emitting types to avoid the opposite problem: too many internally types being exposed, some by accident. In this case that's bitten us, but we've also been bit by the opposite behavior.

[jasonvangundy]

This is biting me right now as well. I have the perfect use case for suspending and resuming my orchestrations but am unable to use the built-in functionality at present. 🤞 this is coming soon.

[jasonvangundy]

Actually, is there any ETA on this? I was considering implementing this on my own using custom events, but I would much prefer to use the built-in functionality. Of course because it's code I don't have to write, but we also get a lot of value out of the orchestration status changing. If I implement this as a custom event, I lose the ability to inspect the state change, both on suspend and resume. If the original issue was correct, it seems like this is just a published types issue and not actually missing / new functionality required, and hopefully low-hanging fruit. Thanks!

[jasonvangundy]

A couple of other hopefully useful notes, presumably missing for the same reasons:

1. HttpManagementPayload returned from createHttpManagementPayload does not contain the definitions for the suspendPostUri nor the resumePostUri.
2. OrchestrationRuntimeStatus does not contain the statuses for Resumed and Suspended