feat(server): coordinate replay-safe runner turns - #4873
Conversation
Signed-off-by: Chris Peterson <chris.peterson@stablejack.com>
Signed-off-by: Chris Peterson <chris.peterson@stablejack.com>
|
@chrispeterson Thanks for the PR! It doesn't reference an issue yet. We require an issue for every PR, so the work can be prioritized before it's reviewed. Add one to the description:
No issue exists for this yet? Open one first, then reference it. That's how we track what's worth doing, and it's usually quicker than it sounds. Note a reference has to point at an issue: naming another PR doesn't count. The only exceptions are changes with no user-visible behaviour: pure Refactor / chore, Docs, or Test / CI work. If that's genuinely what this is, check that box under Type of change. Anything that fixes a bug, adds a feature, or changes the UI needs an issue, even when it also touches docs or tests. See CONTRIBUTING.md for the full policy. No action is taken beyond this comment. |
Signed-off-by: Chris Peterson <chris.peterson@stablejack.com>
Signed-off-by: Chris Peterson <chris.peterson@stablejack.com>
cebddf5 to
43d7492
Compare
Stack and merge order
This PR is stacked on #4868 and its runtime path depends on #4871. After #4868 merges, rebase this branch so the review diff contains only the coordinator commits.
Summary
dispatch_unknowntimed_outwithout redispatchValidation
43d7492fce14d01ef781fdee49b3674fe0768aaftest_launch_cancelled_midspawn_does_not_leak_untracked_runnerCI failure passed 20 consecutive isolated reruns locallysucceededBoundary
This tranche remains unwired from the public session route. #4875 supplies the authenticated principal, required idempotency key, deterministic cross-store input persistence, and typed operation status surface.
Relates to #4861.