Skip to content

feat: add settings command to lean TUI - #4484

Merged
dgageot merged 1 commit into
docker:mainfrom
rumpl:feat/lean-tui-settings
Sep 30, 2026
Merged

dgageot merged 1 commit into
docker:mainfrom
rumpl:feat/lean-tui-settings

Conversation

@rumpl

@rumpl rumpl commented Sep 29, 2026

Copy link
Copy Markdown
Member

Summary

  • Add an inline /settings panel with keyboard navigation, save/cancel, and persistence that preserves unrelated preferences.
  • Include steering/queueing, default auto-approval, snapshots, cache-stable prompts, cache-miss warnings, and lean UI defaults.
  • Add completion sounds and duration threshold, image rendering, startup banner, and split-diff controls; omit themes, sidebar layout, and tab settings.
  • Wire completion/failure sounds into the lean event loop, without chiming on cancellations or nested stream completion.
  • Document the panel and add coverage for persistence, cancellation, confirmation, sound playback, and rendering settings.

Validation

  • task build — passed
  • task lint — passed
  • go test ./pkg/leantui/... — passed
  • task test — blocked by unrelated pkg/acp failure: TestUsageBackgroundRuntimeRecordsWithoutClientIO expects cold-resume cost 0.00075 but receives 0. This also reproduces in isolation; ACP does not depend on the lean TUI.

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl
rumpl requested a review from a team as a code owner September 29, 2026 22:03
@aheritier aheritier added area/docs Documentation changes area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only. labels Sep 29, 2026
@dgageot
dgageot added this pull request to the merge queue Sep 30, 2026
Merged via the queue into docker:main with commit 9738ae2 Sep 30, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Documentation changes area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants