Skip to content

fix(tui): list project first in the root menu; rename 'scaffolded agent code' to 'agent code' - #2220

Merged
tejaskash merged 1 commit into
refactorfrom
fix/tui-menu-order-and-agent-code-label
Sep 4, 2026
Merged

fix(tui): list project first in the root menu; rename 'scaffolded agent code' to 'agent code'#2220
tejaskash merged 1 commit into
refactorfrom
fix/tui-menu-order-and-agent-code-label

Conversation

@tejaskash

Copy link
Copy Markdown
Contributor

Summary

Two small UX tweaks to the interactive TUI:

  • Root menu: project is now the first entry (it was last among the screen-backed commands). Menu and --help order follow handler registration order, so createProjectHandler is registered first in src/handlers/index.tsx.
  • project create → type step: the second option now reads agent code instead of scaffolded agent code.

Tests that pinned the old ordering/label are updated accordingly. No behavioral changes beyond ordering and the label.

Before / after

Root menu:

❯ harness     manage AgentCore harnesses          ❯ project     manage an AgentCore project
  identity    ...                                   harness     manage AgentCore harnesses
  ...                                               identity    ...
  project     manage an AgentCore project           ...

project create type step:

○ harness (recommended)          ○ harness (recommended)
● scaffolded agent code    →     ● agent code

Verification

  • bun test — 2900 pass, 0 fail
  • bun run typecheck, bun run lint:check, bun run format:check — clean

@github-actions github-actions Bot added the size/s PR size: S label Sep 4, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added claude-security-reviewing Claude Code /security-review in progress agentcore-harness-reviewing AgentCore Harness review in progress labels Sep 4, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Sep 4, 2026

@agentcore-devx-automation agentcore-devx-automation Bot 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.

AgentCore Harness Review

Verdict: Looks good

Small, focused change that:

  • Reorders the root TUI menu so project is the first entry (in both supportedTuiCommands and handler registration).
  • Renames the "scaffolded agent code" wizard option to "agent code".
  • Updates all affected tests (RouterScreen, CliOnlyScreen, root, project create wizard) to match the new order and label.

I checked for stragglers and everything is consistent:

  • No remaining references to "scaffolded agent code" anywhere in src/.
  • Handler registration order (index.tsx) matches supportedTuiCommands(...) and matches the expected list in root.test.tsx.
  • Other menu-related tests (e.g. project.screen.test.tsx) filter by typing, so they're not affected by the reorder.
  • The ❯ harness example in src/testing/setup.ts is just an illustrative comment about ANSI styling and doesn't need to change.

Nothing to block on. LGTM.

@agentcore-devx-automation agentcore-devx-automation Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Sep 4, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.04%. Comparing base (da28031) to head (9051572).

Additional details and impacted files
@@            Coverage Diff            @@
##           refactor    #2220   +/-   ##
=========================================
  Coverage     97.04%   97.04%           
=========================================
  Files           544      544           
  Lines         37536    37536           
=========================================
  Hits          36428    36428           
  Misses         1108     1108           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tejaskash
tejaskash merged commit ae2503d into refactor Sep 4, 2026
24 of 25 checks passed
@tejaskash
tejaskash deleted the fix/tui-menu-order-and-agent-code-label branch September 4, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants