Repository navigation
Notify Chief of persisted role interruptions and new exit evidence - #2500
Merged
zy95-12 merged 1 commit intoNov 5, 2026
Conversation
zy95-12
marked this pull request as ready for review
November 5, 2026 04:26
Collaborator
Author
|
Written-by: Codex Exact head343c8e784fc1b5aa629952e8420aa6bb74ce706f: completed full local verify5,013/0, all eight actual disposable DSH scenarios; same-head Node24/tests-ok success run41967207450. Pairing refreshed, no unresolved blockers or external review findings, target basef2dd09866 unchanged and mergeable; actual delivery branch rules[]. enqueue-guard reports only missing independent Verifier; no independent PASS exists or is claimed. Applying Owner standing basic-gate scope solely to this delivery branch, with no main merge, admin bypass or rule edits. Keep umbrella #2483 open for onsite readiness/integrity/actual model acceptance. |
zy95-12
merged commit Nov 5, 2026
7e79762
into
ThinkFlowLab:delivery/ai-centric-development-service
9 of 13 checks passed
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.
Written-by: Codex
Part of #2483, Owner-authorized restart stage; keep the umbrella open.
Restart persisted role claims as blocked/interrupted, while the Chief pump discovered only arbitration. Reuse existing scoped inspection requests for interrupted responsibility and current exit/committed-retirement facts. Stable claim identity avoids repeat notifications after failed routes or reordered state rows; discard queued notices after facts progress. Chief chooses recovery through existing pools. Preserve draft/cwd/logical context, unknown-exit fences and current authority. Neutral restart wording no longer prescribes a new Attempt. No new timer, Effect kind, workflow, default role or automatic Task acceptance.
At343c8e784fc1b5aa629952e8420aa6bb74ce706f: full root verify5,013/0 including eight actual disposable DSH scenarios; same-head GitHub Node24/tests-ok success (run41967207450);54 Chief tests and18 API/docs/policy checks pass. Five persist/dispose/reload/Chief-choice/re-dispatch scenarios pass: pre-start, historical blocked, released SDK fixture, explicit retirement then another judgement, unknown worker held without hot loop. All five fail on original f2dd098 with no Chief request. Final Chief coverage95.04%lines/76.46%branches/82.18%functions; added executable lines exercised, optional combinations not exhaustive. Broader affected coverage/224-test run supplements the same-head checks; no 100% branch claim.
New evidence6649441606 independently explains Main's execution-resource wait: legacy ready lacks profile.id. Added test/documentation of existing explicit project-runtime-check: stale ready refuses admission, a disposable fixture probe saves matching identity and wakes dispatch without restart. Identity checks unchanged; no live or paid probe performed.
Updates long-term memory, owned architecture/operations docs, decision, changelog and generated patches/OpenAPI. Self pairing clear; no independent Verifier PASS. Chief/worker outputs and SDK exit records in added integration tests are fixtures; actual model choice/live OpenToday/Docker exit recovery not claimed. No deployment/service/config/profile/task changes. Standing Owner delivery-only basic gate applies; actual branch rules retained.