Skip to content

Read the commander HUD from native owners and delete the private board - #2670

Closed
trissim wants to merge 3 commits into
next/control-plane-ownedfrom
next/commander-hud
Closed

trissim wants to merge 3 commits into
next/control-plane-ownedfrom
next/commander-hud

Conversation

@trissim

@trissim trissim commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

comms_hud now reads the commander's native records in one frame: open orders grouped by holder, due review rounds, live strikes, unjudged edit counts, recorded open PRs separated by GitHub's draft flag, declared plan phases, and the existing overload decision. No board file, status store, background poll or extra cache is created.

Delete CommanderContext.board_text, BoardVerb, and the two board_cells implementations used only by that renderer. Existing callers use the HUD tool. rotate.py and phase_state.py are already absent on this branch; this replaces the combined order/phase/PR polling those scripts and the fleet backseat scripts perform, not source watching or steering an active unit.

Each HUD field reads its owner: TodoStore/Todo for orders and strikes; TodoStore.timers_up_in for review rounds; OrderEditCountRead for edit counts; PullRequests for linked GitHub PRs and their draft flag; PlanPhase with a single PlanReading for done/ready/holders; OverloadDegradation for load and overload. The PR reader fills its existing per-read family views from one GitHub response rather than fetching the same rows again for each state kind.

The HUD is implemented at the native read boundary. The heartbeat does not import its view; doing that would introduce a standing-rule/heartbeat/overload import cycle.

Stacked on next/control-plane-owned (#2654). No tests, debt scans or censuses were run under Tristan's stop order. The integrator owns affected commander, Todo/PR lookup, tool declaration/serialization and import checks once on integration.

Assumed: PR ownership comes from the orders' recorded links, not guessed branch/title matches; native records remain authoritative; the integrator owns runtime/installed qualification. No Toad page or installed UI performance result is claimed by this Core HUD delivery.

Final HUD head: be31180. Against its declared parent: 54 lines deleted, 201 added. Board renderer/verb and both display-only board_cells implementations are deleted; copied-bus callers now read the HUD. No new status text was inserted into the plan.

HUD field Native owner at this head
Open orders and holder identity src/agent_comms/todos.py:3873 TodoStore.history_for_commander; :3378 Todo.holder and held assignment identity
Review timers up src/agent_comms/todos.py:3846 TodoStore.timers_up_in; :3355 Todo.timer_up
Live strikes src/agent_comms/todos.py:3398 Todo.live_strikes
Unjudged edits src/agent_comms/turn_digests.py:207 OrderEditCountRead
Ready and draft PRs src/agent_comms/todos.py:615 PullRequests.read, preserving GitHub isDraft
Recorded work progress src/agent_comms/todos.py:1039 OrderProgress.read
Plan done/ready/holders src/agent_comms/plan_phases.py:101, :105, :124 PlanPhase methods on one PlanReading
Load and overload decision src/agent_comms/commander/vibe_check.py:199, :208 OverloadDegradation

CommanderHud.read (commander_hud.py:92) joins those views; the typed cards do not persist them. This makes the order/phase/PR board polling in legacy rotate/phase-state/fleet backseat helpers redundant. It does not replace source-level conversation watching, intent diagnosis or steering; those are not HUD fields.

@trissim
trissim marked this pull request as ready for review October 23, 2026 08:05
Copilot AI balanced review requested due to automatic review settings October 23, 2026 08:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@trissim

trissim commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

Landed on main through batch #2684 (merge f966a53): head be31180 is an ancestor of origin/main. GitHub can't mark it merged because its base is a stacked branch, and it can't be retargeted because main already contains every commit.

@trissim trissim closed this Oct 23, 2026
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.

2 participants