assembly live: richer, better-spaced tool-call lines#258
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…widget Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Keeps both files under the 500-line file-length gate. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
alexkroman
pushed a commit
that referenced
this pull request
Jun 23, 2026
… 500-line gate The filler + planning-discard work and the #258 merge pushed engine.py and two test files over the 500-line file-length gate. Extract the Renderer/Player protocols and CascadeDeps into agent_cascade/_io.py (re-exported from engine), and consolidate the spoken-filler + planning-discard tests into test_agent_cascade_filler.py. Also drop the stale test_live_tui_launch.py (duplicate of this branch's test_live_tui_wiring.py) and retarget CascadeDeps.real patches at _io. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.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
2 participants
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.
What & why
Improves the tool-call UX in
assembly live(the agent-cascade voice TUI), per the screenshot feedback that the tool lines were cramped and detail-free.Before:
Searching the web…/Using read_file…— bare verbs, packed flush against the prompt and each other.After: the call's identifying argument is shown, and the block gets a blank line above it (consecutive calls stay tight):
How
agent_cascade/brain.py): a new_tool_affordance(name, args)composes the friendly label with the one identifying arg via the existingcode_agent.summarize.describe_argshelper (the same oneassembly codeuses). TheRenderer.tool_call(label)protocol is unchanged — only the string gets richer, so the non-TUIAgentRendererbenefits for free. The trailing…is still added by the renderers.code_agent/messages.py+agent_cascade/tui.py): a dedicated dimToolAffordancetranscript widget (alongside the sibling widgets, reusing_DIM/Text).show_tool_callmounts it with a top margin for the first call of a turn and a-tightclass (margin 0) when the previous line was already a tool call.run_agent_cascadewiring tests intotests/test_live_tui_launch.pyso both files stay under the 500-line file-length gate.Out of scope (deliberately light touch): no tool results, spinners, or completion states;
_tool_label/_TOOL_LABELSand theRendererprotocol are untouched.Testing
Full
./scripts/check.shgreen: 3561 tests pass, 99.65% coverage, Textual surfaces (tui.py,messages.py) at 100%, 100% patch coverage on changed lines, mutation gate clean, no new escape hatches, build + twine PASSED. Design/plan docs included underdocs/superpowers/.🤖 Generated with Claude Code