Skip to content

Resolve runtime payloads in service-operation workflow commands - #144

Merged
rmcdaniel merged 2 commits into
mainfrom
fix/service-operation-runtime-payload
Sep 8, 2026
Merged

rmcdaniel merged 2 commits into
mainfrom
fix/service-operation-runtime-payload

Conversation

@rmcdaniel

@rmcdaniel rmcdaniel commented Sep 8, 2026

Copy link
Copy Markdown
Member

Addresses #143; paired with durable-workflow/sdk-php#63.

Resolve workflow-command request_payload through the existing namespace-authenticated runtime reference middleware. No codec, provider adapter, request-limit change, or admission bypass.

All three regressions fail on the original source and pass with the fix. Focused tests: 72 / 651 assertions. Full Feature suite: 2,062 tests / 41,663 assertions, three existing skips and six PHPUnit deprecations. Pint and all public CI passed, including rolling upgrade, replay/query HTTP, bounded-growth smoke and chart installation.

Prepares Server 2.3.3 and Helm 0.1.82. Qualification and transport-boundary review are in the comments. Issue closure follows publication.

@rmcdaniel

Copy link
Copy Markdown
Member Author

Full local Feature suite: 2,062 tests and 41,663 assertions passed (three skips, six PHPUnit deprecations). The focused pair passed 72 tests/651 assertions. The new three-case regression failed on the target before the one-line production fix. Public CI is running.

Review: this reuses the existing authenticated namespace claim/integrity path; it does not weaken size, lease, service authorization, or cross-namespace rules. Next: include the patch release identity and publish after checks pass.

@rmcdaniel
rmcdaniel marked this pull request as ready for review September 8, 2026 10:38
@rmcdaniel
rmcdaniel merged commit a1bcc67 into main Sep 8, 2026
14 checks passed
@rmcdaniel

Copy link
Copy Markdown
Member Author

Merged source is a1bcc67, tree-identical to the qualified PR head. Immutable tag 2.3.3 starts the repository-owned publication run: https://github.com/durable-workflow/server/actions/runs/34216699544. Watching this exact run; #143 stays open until image/chart publication is verified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants