[WIP prerequisite] preserve cached snapshot semantics for #850 - #1195
Draft
sethkarten wants to merge 13 commits into
Draft
[WIP prerequisite] preserve cached snapshot semantics for #850#1195sethkarten wants to merge 13 commits into
sethkarten wants to merge 13 commits into
Conversation
… workers A worker with a durable stop intent is now reported as "stopping" and a disconnected worker is never reported as "ready". Stopping workers are excluded from live session lists and daemon-wide fan-out commands.
…y checks The list response feeds shutdownStaleDaemonIfNotBusy and probeRunningDaemonSessions, so hiding stopping workers made a tombstoned-but-running worker look idle and let a stale-daemon replace terminate it silently. Stopping workers stay listed with an honest "stopping" workerState; command fan-outs still skip them.
…ates
Revision 14 records the workerState wire-semantics change ("stopping"
state; disconnected workers no longer report "ready") so version probes
can tell old and new daemons apart. The field stays optional and
backward-tolerant, so no capability gate is needed.
… workers A worker with a durable stop intent is now reported as "stopping" and a disconnected worker is never reported as "ready". Stopping workers are excluded from live session lists and daemon-wide fan-out commands.
…y checks The list response feeds shutdownStaleDaemonIfNotBusy and probeRunningDaemonSessions, so hiding stopping workers made a tombstoned-but-running worker look idle and let a stale-daemon replace terminate it silently. Stopping workers stay listed with an honest "stopping" workerState; command fan-outs still skip them.
…ates
Revision 14 records the workerState wire-semantics change ("stopping"
state; disconnected workers no longer report "ready") so version probes
can tell old and new daemons apart. The field stays optional and
backward-tolerant, so no capability gate is needed.
sethkarten
changed the base branch from
snimu/worker-lifecycle-accuracy
to
main
August 11, 2026 03:33
7 tasks
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
3 participants
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.
Purpose
Draft prerequisite validation of the contributor #850 semantics plus the cached-snapshot correction, based on
mainsolely so the repository CI workflow runs. This is a CI-validation and collaboration surface, not a v0.8 stack deliverable and not ready for human merge review.Exact candidate
dba83904c43f0f32be4e62c74e8678d9a0bd7abca6db0246b52cea74f2f675fea95ac43c9842d39a(one-line test-type fixture repair atop the ancestry/CI commits; production tree remains revieweddba83904...; contributor head is a parent)0026c8adc13d29fd9efcb52b28fe2adbb9dbb7a0749831dd9dd1a0b35708513390080030878d9e4fCorrections
SessionSummary.workerStatewire shape.Evidence and hold
Focused daemon protocol, supervisor monitor, ENG-4602, and ENG-4677 coverage: 69/69 passed locally with direct pinned dependencies. Independent review found no remaining P0/P1 in this delta. Local full-shard evidence was rejected as environment-contaminated (
FORCE_COLORplus live HOME/daemon state), so this Draft exists to obtain clean GitHub CI evidence.Do not merge or restack #851/#852 from this head until CI is green and the prerequisite is accepted. No contributor branch was force-pushed or mutated.
First remote CI attempt
Run
31455798294classified the product candidate: all production/process/kernel/shards except one unrelated 60-second IPython bootstrap outlier passed. Build/check found a test-only TS2502 self-referential parameter name introduced by the added regression fixture. Heada6db0246...changes only that parameter name; direct monitor 51/51 and direct tsgo passed. Failed-run evidence is retained with SHA-256e262fcd750b9c54a6248d01ca68a86a81da25f04bc7cda674118c7b12e9133ab.