Skip to content

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

Open
notgitika wants to merge 6 commits intoaws:mainfrom
notgitika:feat/show-all-commands-in-tui
Open

feat: show all CLI commands in TUI with / toggle#635
notgitika wants to merge 6 commits intoaws:mainfrom
notgitika:feat/show-all-commands-in-tui

Conversation

@notgitika
Copy link
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
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?

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

2 participants