You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add c8ctl suspend process-instance <key> and c8ctl resume process-instance <key>
Wired through COMMAND_REGISTRY / COMMAND_DISPATCH, calling the SDK's suspendProcessInstance / resumeProcessInstance (POST /process-instances/{key}/suspension and /resumption)
README.md and docs/command-reference.md regenerated from the registry (single source of truth)
Version note: this REST endpoint only exists starting with Camunda 8.10 — 8.8/8.9 gateways 404 on it. Documented in .github/SDK_GAPS.md, called out in the command help text ("Camunda 8.10+"), and the live integration tests below are version-gated accordingly.
npm test (integration watch.test.ts failures are pre-existing, require a live local cluster, unrelated to this change)
Behavioural unit tests for suspend/resume dry-run and missing-key cases
Live CLI integration tests (deploy a user-task process, suspend pi, poll get pi --fields state for SUSPENDED, resume pi, poll back to ACTIVE) — verified locally against Camunda 8.10 and 8.9 (confirms the 404 on 8.9), gated to 8.10+ in CI
Adds live CLI integration tests exercising suspendProcessInstance and
resumeProcessInstance against a real cluster (deploy a user-task
process, suspend, poll for SUSPENDED, resume, poll for ACTIVE),
addressing Copilot review feedback on #585 that the unit tests never
executed past the dry-run branch.
Found in the process: the REST endpoint only exists starting with
Camunda 8.10 — 8.8/8.9 gateways 404 on POST
/process-instances/{key}/suspension|resumption. Documented in
.github/SDK_GAPS.md and gated the new integration tests with a
CAMUNDA_VERSION skip, mirroring the existing businessIdSkip pattern in
the same file.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Document Camunda 8.10 requirement in help and command references
.github/SDK_GAPS.md:21
The Camunda 8.10 floor is only recorded in this contributor-facing file. The generated CLI help and public command reference advertise both commands without a version qualifier even though 8.8/8.9 remain supported targets, so users on those versions will encounter an unexplained 404. Add the requirement to both registry descriptions/help texts and regenerate README/docs.
Add suspend and resume to help and completion test coverage
tests/unit/command-registry.test.ts:45
These additions cover registry membership only. The existing help and completion tests explicitly enumerate exposed verbs but still omit suspend and resume, leaving the new generated help/completion entries without regression coverage. Add both verbs to those assertions, including detailed help and the process-instance (pi) <key> shape.
The repository supports Camunda 8.8/8.9, but this command is guaranteed to return 404 there, while the generated public help currently presents it without a version constraint. The internal SDK_GAPS.md entry is not user-facing. Add “(Camunda 8.10+)” to this help description, matching the existing Business ID annotations, and regenerate the command reference.
Mark resume command as available only in Camunda 8.10+
src/framework/command-registry.ts:1157
The generated public help also needs to identify resume as Camunda 8.10+; otherwise users of the supported 8.8/8.9 versions are told this command is available even though it always fails with 404. Annotate this help description and regenerate the command reference.
Add Camunda 8.10+ requirement to command help and documentation
src/framework/command-registry.ts:1142
This command is unavailable on the supported Camunda 8.8/8.9 clusters, but neither its top-level nor detailed help says so; users discover the limitation only through a gateway 404. Include “Camunda 8.10+” in both descriptions and regenerate README/command-reference, following the existing version annotations in this registry (for example, lines 265 and 315).
This issue also appears on line 1156 of the same file.
Addressed the "Previously missed" finding from the latest Copilot review overview (Add Camunda 8.10+ requirement to command help and documentation, src/framework/command-registry.ts:1142/1156): both suspend/resume helpDescriptions now say "(Camunda 8.10+)", following the existing "(Camunda 8.9+)" convention used for the businessId flags. docs/command-reference.md regenerated. Pushed in c7cc180.
Adds `c8ctl suspend process-instance <key>` and `c8ctl resume
process-instance <key>`, wired through the command registry and
dispatch map, calling the SDK's suspendProcessInstance /
resumeProcessInstance methods (POST /process-instances/{key}/suspension
and /resumption).
The REST endpoint only exists starting with Camunda 8.10 — 8.8/8.9
gateways 404 on this path. Documented in .github/SDK_GAPS.md, noted in
the command help text ("Camunda 8.10+"), and the live integration
tests are version-gated accordingly.
Covered by:
- behavioural unit tests (dry-run + missing-key cases)
- live CLI integration tests (deploy a user-task process, suspend,
poll for SUSPENDED via `get pi`, resume, poll for ACTIVE), gated to
Camunda 8.10+
README.md and docs/command-reference.md regenerated from the registry.
Closescamunda/camunda#57996
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jwulf I'm unsure who to request review from for this. I see that c8ctl ownership was moved to clients-sdks-ai-first-tooling (?) but I don't seem to be able to add that group as reviewer.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
c8ctl suspend process-instance <key>andc8ctl resume process-instance <key>COMMAND_REGISTRY/COMMAND_DISPATCH, calling the SDK'ssuspendProcessInstance/resumeProcessInstance(POST/process-instances/{key}/suspensionand/resumption)Version note: this REST endpoint only exists starting with Camunda 8.10 — 8.8/8.9 gateways 404 on it. Documented in
.github/SDK_GAPS.md, called out in the command help text ("Camunda 8.10+"), and the live integration tests below are version-gated accordingly.Closes camunda/camunda#57996
Test plan
npm run typechecknpx biome check --fix(clean)npm run check:layeringnpm run buildnpm run test:unit(2339/2339 pass)npm test(integrationwatch.test.tsfailures are pre-existing, require a live local cluster, unrelated to this change)suspend/resumedry-run and missing-key casessuspend pi, pollget pi --fields statefor SUSPENDED,resume pi, poll back to ACTIVE) — verified locally against Camunda 8.10 and 8.9 (confirms the 404 on 8.9), gated to 8.10+ in CI🤖 Generated with Claude Code