Repository navigation
exec: v1.4 notify only where kept jobs survive idling; crash-duplicate fix split out - #398
Merged
Merged
Conversation
…icate fix is its own change
Advertising notify is now an endpoint's promise that a kept job survives
its sandbox going idle, decided by the endpoint's deployment (off unless
turned on). An endpoint without native jobs advertises notify for the
baseline by answering the job probe with {"notify": "v1.4"} and no "jobs";
the baseline offers notify only then.
The job ledger leaves v1.4: notify promises no duplicate or loss window
beyond today's. The crash window in which a sandbox job's end can be
reported twice exists today for held streams too, and ARCHITECTURE.md now
describes it, and its fix, as a separate known gap.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JimHGjsfk2Ktm5GxyZJKKk
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Two changes to the v1.4 "Sandbox job notify" spec. As before, the spec, ARCHITECTURE.md, the fixtures and the coverage test change together. Nothing is implemented yet.
Notify only where kept jobs survive idling. Releasing a job removes the heartbeat that kept its sandbox busy, so a sandbox that stops or kills idle processes (a vendor's auto-stop) would lose the job because of v1.4.
{"jobs": "v1.3", "notify": "v1.4"}.{"notify": "v1.4"}and nojobs. A client that predates v1.4 reads that as no native jobs, as it does today.curland the endpoint honoursstdin_base64.baseline_notifyexpectation, and a conformance check,a kept job survives idling.The job ledger leaves v1.4. v1.4's own guarantee is now that notify adds no window in which an end is reported twice or not at all, beyond those today's code has (N34).
collect_garbageremoves the record.exec_notify_vectorspasses 10/10. The previous test fails on the new probe fixture (2 of 10).🤖 Generated with Claude Code
https://claude.ai/code/session_01JimHGjsfk2Ktm5GxyZJKKk