Skip to content

feat: add startup banner to normal TUI - #3949

Open
rumpl wants to merge 3 commits into
docker:mainfrom
rumpl:feat/normal-tui-startup-banner
Open

feat: add startup banner to normal TUI#3949
rumpl wants to merge 3 commits into
docker:mainfrom
rumpl:feat/normal-tui-startup-banner

Conversation

@rumpl

@rumpl rumpl commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • share the Docker Agent ASCII banner between the lean and normal TUIs
  • show it as a centered empty state when an agent has no welcome message
  • keep configured welcome messages unchanged

Validation

  • task test
  • task lint
  • task build

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl
rumpl requested a review from a team as a code owner August 7, 2026 06:40

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

Lower-confidence findings (not posted inline)

  • [low] pkg/tui/page/chat/runtime_events.go:124 — AgentInfoEvent re-enabling showStartupBanner can flash banner mid-conversation during agent/model switches (confidence: weak 52/100)

Comment thread pkg/tui/page/chat/chat.go
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@aheritier aheritier added 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 Aug 7, 2026
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants