Skip to content

Notify Chief of persisted role interruptions and new exit evidence - #2500

Merged
zy95-12 merged 1 commit into
ThinkFlowLab:delivery/ai-centric-development-servicefrom
zy95-12:fix/2483-restart-chief-recovery
Nov 5, 2026
Merged

zy95-12 merged 1 commit into
ThinkFlowLab:delivery/ai-centric-development-servicefrom
zy95-12:fix/2483-restart-chief-recovery

Conversation

@zy95-12

@zy95-12 zy95-12 commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

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.

zy95-12 commented Nov 5, 2026

Copy link
Copy Markdown
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
zy95-12 merged commit 7e79762 into ThinkFlowLab:delivery/ai-centric-development-service Nov 5, 2026
9 of 13 checks passed
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.

1 participant