Skip to content

Show the new TUI#1942

Merged
dgageot merged 1 commit intodocker:mainfrom
dgageot:show-tui
Mar 6, 2026
Merged

Show the new TUI#1942
dgageot merged 1 commit intodocker:mainfrom
dgageot:show-tui

Conversation

@dgageot
Copy link
Member

@dgageot dgageot commented Mar 5, 2026

No description provided.

Signed-off-by: David Gageot <david.gageot@docker.com>
@dgageot dgageot marked this pull request as ready for review March 5, 2026 14:45
@dgageot dgageot requested a review from a team as a code owner March 5, 2026 14:45
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

🟢 Review Summary

Assessment: APPROVE

This PR updates the demo recording configuration to use larger dimensions (2000×1600 instead of 1800×1000) and simplifies the demo script from a complex multi-step test optimization workflow to a straightforward README summarization task.

Changes reviewed:

  • ✅ Screen dimension adjustments are valid
  • ✅ Simplified demo script with clear command syntax
  • ✅ Removal of Wait+Screen and approval interactions is intentional and appropriate for the new simpler demo content

The new demo focuses on a read-only summarization task that doesn't require tool approval, making the removal of the approval flow intentional rather than problematic. No issues found in the changed code.

@dgageot dgageot merged commit 1321d1a into docker:main Mar 6, 2026
11 checks passed
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.

2 participants