feat(cli): sessions table gains LLM column; headers say what they mean - #112
Merged
Conversation
Column shuffle per operator review of the act-2 demo: - NAME becomes AGENT NAME and AGENT becomes RUNTIME. The old AGENT header violated elem-runtime-names-harness (the column renders Runtime.Name, which names the harness, never the agent). - New LLM column shows the model as the metering producer named it: the stream accountant's raw id for headless sessions, the statusline feed's display name for interactive ones. The heartbeat RPC gains an optional model param and ctx-forward now sends the payload's display_name; a reporter that sends "" leaves any prior reading standing. - Watch-mode sort keys: l=llm, r=runtime, R=role, h=health (health was never sortable before), help moves to ? (htop convention). RSS keeps its name (it names exactly what procstat samples) and its m keybind. - render_test's column helper now splits on 2+ spaces so the two-word AGENT NAME header stays one column. Verified live against the running act-2 demo: analyst-p LLM claude-fable-5 via stream beside analyst-t LLM "Sonnet 4.6 (1M context)" via statusline, after a daemon swap-in-place (adopt path). Refs: aae-orc-7hzb, finding-011
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.
Operator review of the act-2 demo surfaced that the sessions table misnames two columns and hides a signal it already has. The AGENT header renders Runtime.Name — which names the harness, never the agent, per the ratified elem-runtime-names-harness ruling — and the model each session runs was already tracked (SessionContext.ContextModel) but never shown.
modelparam;ctx-forwardsends the payload's display_name; empty means "keep the prior reading" so the simulator changes nothing.l=llm,r=runtime,R=role,h=health — health was never sortable before — and help moves to?(htop convention). RSS keeps its name (it names exactly what procstat samples: resident set over the pid subtree) and itsmkeybind.Verified live against the running act-2 demo, including a daemon swap-in-place through the adopt path:
analyst-pLLMclaude-fable-5via stream besideanalyst-tLLM "Sonnet 4.6 (1M context)" via statusline.Additive; the only breaking surface is the internal store method signature (
UpdateSessionHeartbeatgains a model arg) and watch keybinds (aremoved, help on?).Refs: aae-orc-7hzb, finding-011