Skip to content

feat(tui): Make agent option history user-facing#480

Merged
CSRessel merged 3 commits into
mainfrom
agent-options-history
May 18, 2026
Merged

feat(tui): Make agent option history user-facing#480
CSRessel merged 3 commits into
mainfrom
agent-options-history

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

Summary

🤖 Generated with Nori

  • Replace raw ACP config history messages with agent-named, changed-only option history.
  • Remove noisy in-progress session config history lines and suppress redundant current-value selections.
  • Seed initial config snapshots silently so the first real backend update is still shown.

Test Plan

  • env RUSTC_WRAPPER= cargo test -p nori-protocol
  • env RUSTC_WRAPPER= cargo test -p nori-acp
  • env RUSTC_WRAPPER= cargo test -p nori-tui
  • env RUSTC_WRAPPER= cargo build --bin nori
  • env RUSTC_WRAPPER= cargo test -p tui-pty-e2e
  • TUI smoke test with ElizACP via tmux

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

@CSRessel CSRessel marked this pull request as ready for review May 15, 2026 20:59
@CSRessel CSRessel changed the title Make agent option history user-facing feat(tui): Make agent option history user-facing May 17, 2026
@CSRessel CSRessel merged commit dd06fc9 into main May 18, 2026
3 checks passed
@CSRessel CSRessel deleted the agent-options-history branch May 18, 2026 19:24
CSRessel added a commit that referenced this pull request May 18, 2026
)

## Summary
🤖 Generated with [Nori](https://noriagentic.com/)

- Show an explicit banner the first time the agent announces its session
config options at startup:
\`• Claude Code options: Mode=Default, Model=Opus 4.6, Effort=High
(/config to change)\`
- Replaces the previous baseline-only behavior (#480) which silently
swallowed the initial \`SessionConfigUpdate\` and left no on-screen
indication of the agent's options or how to change them.
- Subsequent updates continue to render only changed values (no behavior
change after the banner).

Pairs with #480 (config-options diff rendering) and #481 (mode-changed
cell) — finishes the cleanup of the original \`ACP config options
updated: …\` jargon line.

## Test Plan
- [x] New unit test
\`session_config_update_history_renders_startup_banner_on_first_snapshot\`
(snapshot in \`session_config_update_startup_banner.snap\`)
- [x] Existing
\`session_config_update_history_shows_only_changed_values_after_baseline\`
updated to drain the new banner and still snapshot just the diff
- [x] \`cargo test -p nori-tui\` — 1304 passed
- [x] \`just fmt\` + \`just fix -p nori-tui\` — clean

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

Co-authored-by: Nori <contact@tilework.tech>
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