Skip to content

feat: show all CLI commands in TUI with / toggle#635

Merged
notgitika merged 7 commits intoaws:mainfrom
notgitika:feat/show-all-commands-in-tui
Mar 27, 2026
Merged

feat: show all CLI commands in TUI with / toggle#635
notgitika merged 7 commits intoaws:mainfrom
notgitika:feat/show-all-commands-in-tui

Conversation

@notgitika
Copy link
Copy Markdown
Contributor

@notgitika notgitika commented Mar 25, 2026

Description

Show all CLI commands in the TUI's HelpScreen, including CLI-only commands (logs, traces, pause, resume) that were previously hidden. These commands are grouped under a "CLI only ───" divider section with a / keyboard toggle to show/hide them.

Key changes:

  • Added cliOnly flag to CommandMeta interface to distinguish interactive vs CLI-only commands
  • Narrowed HIDDEN_FROM_TUI to only ['help']; package and update are now visible (they have TUI screens)
  • Added / toggle in HelpScreen to show/hide CLI-only commands section with visual divider
  • Search always finds CLI-only commands regardless of toggle state
  • Selecting a CLI-only command navigates to CliOnlyScreen with usage examples
  • Added CLI_ONLY_EXAMPLES constant with usage info for logs, traces, pause, resume

Related Issue

Closes #566

Documentation PR

N/A

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@notgitika notgitika requested a review from a team March 25, 2026 05:00
@github-actions github-actions bot added the size/m PR size: M label Mar 25, 2026
@notgitika notgitika changed the title feat: show all CLI commands in TUI with Ctrl+L toggle feat: show all CLI commands in TUI with / toggle Mar 26, 2026
@github-actions github-actions bot removed the size/m PR size: M label Mar 26, 2026
@github-actions github-actions bot added the size/m PR size: M label Mar 26, 2026
@github-actions github-actions bot added size/m PR size: M and removed size/m PR size: M labels Mar 26, 2026
@Hweinstock
Copy link
Copy Markdown
Contributor

Looks like a new unit test is failing.


FAIL  unit  src/cli/tui/utils/__tests__/commands.test.ts > getCommandsForUI > includes update and package as interactive commands
AssertionError: expected undefined to be defined
 ❯ src/cli/tui/utils/__tests__/commands.test.ts:46:20
     44|     const update = cmds.find(c => c.id === 'update');
     45|     const pkg = cmds.find(c => c.id === 'package');
     46|     expect(update).toBeDefined();
       |                    ^
     47|     expect(update!.cliOnly).toBe(false);
     48|     expect(pkg).toBeDefined();

Does this mean update isn't showing up?

These commands have TUI screens and should be visible. The test
expected them to be present but the implementation still filtered
them out, causing CI failure.
@github-actions github-actions bot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.3.0-preview.9.0.tgz

How to install

npm install https://github.com/aws/agentcore-cli/releases/download/pr-635-tarball/aws-agentcore-0.3.0-preview.9.0.tgz

@notgitika notgitika merged commit df80f90 into aws:main Mar 27, 2026
17 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

All commands to be shown in TUI

3 participants