fix(project): show the live progress list for project add on a TTY

[aidandaly24]

Description

agentcore project add <resource> now renders the same live progress list as project create, build, and deploy when run in a terminal: a spinner on the running step, a scrolling tail of its output (for example uv sync during add runtime), and a ✓ on each completed step.

Screen.Recording.2026-09-08.at.12.27.19.PM.mov

Every add sub-command runs its generator through the shared helper in src/handlers/project/add/shared.ts. Since #2218 that helper called runWithProgress with interactive: false, which forces the plain line-per-step path even on a TTY. The comment there preserved the pre-#2218 behavior, where each add handler wrote its own plain lines and never used the progress driver. This change gates the plain path on --json only, the value every other project mutation already passes.

Behavior for project add:

Situation	Before	After
Interactive terminal	plain lines	spinner task list, ✓ per step
Piped or CI (no TTY)	plain lines	plain lines, unchanged
--json	plain lines	plain lines, unchanged

TaskList: keep the glyph when a step title is wider than the terminal

The new TTY test surfaced a pre-existing rendering bug on the Windows runner. Ink boxes shrink by default, so when a step title is wider than the row (a long path with no break opportunity, such as a Windows temp path), the one-column ✓/✕ glyph and spinner frame were squeezed to zero width and disappeared:

before (80 columns):
 Reading project spec file at 'C:\Users\ContainerAdministrator\AppData\Local\Temp
\agentcore-memory-aX9hjo\TestProject\agentcore\agentcore.json'

after:
✓ Reading project spec file at 'C:\Users\ContainerAdministrator\AppData\Local\Te
 mp\agentcore-memory-aX9hjo\TestProject\agentcore\agentcore.json'

TaskList and Spinner now wrap the glyph in <Box flexShrink={0}>. The title still wraps beneath itself. The same glitch showed in project create at exactly the terminal width, so this fixes it there too.

Related Issue

No upstream issue; reported internally as "agentcore project add missing nice spinners".

Documentation PR

N/A. Output shape for --json and non-TTY callers is unchanged.

Type of Change

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

Testing

New tests:

• src/handlers/project/add/memory/index.test.ts: a TTY run asserts the rendered frames contain ✓ Reading project spec file, ✓ Updating project spec file, and the success line, with nothing on stdout. A TTY run with --json asserts no escape sequences reach stderr and the mutation result lands on stdout.
• src/components/ui/task-list/TaskList.test.tsx: at 40 columns, a title wider than the row keeps the ✓ on a done task and the spinner frame on a running task. Fails without the flexShrink change on every platform.

Manual checks against a scaffolded project:

• Pseudo-TTY (script with stty cols 200): add memory shows both steps ✓; add runtime shows the spinner with the uv sync tail under the running step, then all four steps ✓.

• 60-column terminal via the TUI harness: add runtime keeps every ✓ with the long paths wrapping beneath their titles.

• Non-TTY and --json output is byte-identical to before.

• bun test (3181 pass, 0 fail)

• I ran bun run typecheck

• I ran bun run lint:check and bun run format:check

• If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots (not applicable)

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 (no docs change needed)
• 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.

[agentcore-devx-automation]

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

[agentcore-devx-automation]

AgentCore Harness Review

Verdict: Looks good

Small, focused bug fix. addProjectResource now gates the plain progress path on --json only, letting runWithProgress's TTY detection do its job — matching the exact pattern already used in project create, build, and deploy (interactive: ctx.require(JsonKey) ? false : undefined).

Verified:

• The shared driver runWithProgress treats interactive: undefined as "fall back to io.stderr.isTTY", so the non-JSON branch correctly restores the live task list.
• JsonKey is a global flag with .default(false), so ctx.require(JsonKey) is guaranteed to return a boolean.
• Since addProjectResource is used by every project add sub-resource (memory, runtime, harness, gateway, evaluator, etc.), the fix applies uniformly.
• Tests exercise real IO via testIO({ isTTY }) and TestCoreClient — no new mocks introduced — and cover both the TTY frame path and the --json-on-a-TTY ANSI-free path.
• No telemetry gap: this is a rendering bug fix, not a new feature.

Nothing blocking. LGTM.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.04%. Comparing base (d9ab959) to head (942bbf6).
⚠️ Report is 1 commits behind head on refactor.

Additional details and impacted files

@@            Coverage Diff            @@
##           refactor    #2247   +/-   ##
=========================================
  Coverage     97.04%   97.04%           
=========================================
  Files           566      566           
  Lines         39409    39414    +5     
=========================================
+ Hits          38244    38249    +5     
  Misses         1165     1165           

☔ 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.

[agentcore-devx-automation]

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