Repository navigation
feat: align chief supervision with lead semantics - #93
Merged
Merged
Conversation
Persist displaced Lead tool baselines across Chief transitions and Pi 0.86 session restoration, and add bounded staff transcript access with exact persisted session identity.\n\nUnify lifecycle and presentation semantics, preserve descendant states, and replace misleading activity aggregates while updating the supervision contract, documentation, and focused tests. Require Pi 0.86 for the native session APIs and transcript-backed tool restoration.
Restore stat-only O(1) transcript readiness so status and supervision refreshes do not parse complete JSONL files. Reuse resolved path-backed sessions without a duplicate open, while actual transcript reads retain full session-header validation.
Document that a non-empty persisted session candidate only makes transcript readiness advisory. The transcript action still validates the current session header, version, and exact Pi session ID before returning evidence.
Add a PR #93-only Windows job that runs the first missing test file independently from the validation matrix. This diagnostic is temporary and will be removed after the hang evidence is collected.
Wait for observed reconnect conditions instead of relying on a fixed delay, so the lifecycle watcher test synchronizes on subscription evidence across platforms. Track and destroy accepted sockets during unconditional cleanup, then remove the temporary PR #93 Windows diagnostic job now that the focused isolation is complete.
boadij
added a commit
that referenced
this pull request
Oct 2, 2026
## Summary - Publish Lead herd-run start time and Pi context percentage as display-only metadata through the existing Herdr path, and project it only after Lead identity validation. - Show responsive elapsed/context columns on Manager work rows, dropping elapsed before context as width tightens. - Reuse Pi's semantic theme roles for supervision rows; Chief receives theme styling only and no telemetry columns. - Keep persistence, project/work schemas, model context, staff/control surfaces, and Chief telemetry unchanged. ## Rationale This preserves ADR 0002's separation of durable intent from derived runtime state and ADR 0012's Manager-owned project resolution. It reuses the canonical herd-run timing from PR #19 and respects the boundaries established by PRs #93, #184, and #201. The implementation follows the applicable Pi 1.0.0 and Herdr 0.9.3 contracts without adding polling, persistence, or protocol changes. ## Validation - Focused suites: Herdr (67 passed), supervision (54 passed), presentation (75 passed), controller lifecycle + extension contract (97 passed) - `npm run check` (792 passed, 1 skipped) - `npm run build` - `npm run package:audit` - `npm run smoke -- manager-recovery` - `git diff --check` Closes #199 --------- Co-authored-by: Jeffrey Boadi <25706638+boadij@users.noreply.github.com>
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
1 participant
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.
Summary
staff transcriptusing exact persisted Pi session identity and the shared bounded transcript projection.Key behavior
staff, including after/treerestoration and at the model-call boundary.leadToolscheckpoint, while legitimate branch-local Lead tool state remains authoritative.staff transcript; the transcript action validates the current session header, version, and exact Pi session ID before returning evidence; internal session paths are not serialized.working,settling,starting,blocked,unknown, andloststates. Aggregate counts useactive,blocked, andtotal.◐remains blocked and◉remains delegated activity.Validation
npm test -- extension/supervision.test.ts extension/presentation.test.ts extension/commands.test.ts extension/extension-contract.test.ts extension/controller-api.test.ts extension/recovery.test.ts— 291 passed.npm run check— passed.npm run validate— passed (check, build, and package audit).git diff --check— passed.Manual smoke
No live manual smoke run was performed; this worktree did not run a live Pi 0.86 + Herdr environment.