Skip to content

fix(tui): highlight session config values only#484

Merged
CSRessel merged 3 commits into
mainfrom
history-cell-value-highlight
May 18, 2026
Merged

fix(tui): highlight session config values only#484
CSRessel merged 3 commits into
mainfrom
history-cell-value-highlight

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

@CSRessel CSRessel commented May 18, 2026

Summary

🤖 Generated with Nori

  • Render ACP session-config history assignments as Name= plus a separately highlighted value span.
  • Covers startup, update, and option-set history cells so Mode=Default and Model=... leave the name/separator plain.
  • Remove the earlier unrelated MCP argument styling change from this PR.

Test Plan

  • RUSTC_WRAPPER= cargo test -p nori-tui nori::session_config_history::tests::history_cells_highlight_values_not_names
  • RUSTC_WRAPPER= cargo test -p nori-tui
  • RUSTC_WRAPPER= cargo build --bin nori
  • RUSTC_WRAPPER= cargo test -p tui-pty-e2e
  • just fmt
  • RUSTC_WRAPPER= just fix -p nori-tui
  • RUSTC_WRAPPER= cargo insta pending-snapshots --manifest-path tui/Cargo.toml
  • git diff --check
  • TUI smoke via tmux with ElizACP config

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

@CSRessel CSRessel changed the title fix(tui): highlight MCP history argument values fix(tui): highlight session config values only May 18, 2026
@CSRessel CSRessel merged commit 9b65729 into main May 18, 2026
3 checks passed
@CSRessel CSRessel deleted the history-cell-value-highlight branch May 18, 2026 21:55
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