Skip to content

fix(dev): default the command to headless mode - #2236

Merged
tejaskash merged 1 commit into
aws:refactorfrom
Hweinstock:fix/default-headless-dev
Sep 4, 2026
Merged

fix(dev): default the command to headless mode#2236
tejaskash merged 1 commit into
aws:refactorfrom
Hweinstock:fix/default-headless-dev

Conversation

@Hweinstock

@Hweinstock Hweinstock commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Problem

  • The default for dev is currently opening agent inspector
  • the dev help shows tui, but doesn't support it.

Solution

  • change default to headless
  • update help text.
  • remove tui support.

Verification

$ agentcore project create --name DevTest --template agent-python-strands 
$ cd DevTest
$ agentcore project dev & 
[1] 1401410
OTEL collector listening on port 43819; traces persist to [..]
[agent_python_strands] Starting development server
[agent_python_strands] INFO:     Will watch for changes in these directories: [..]
[agent_python_strands] INFO:     Uvicorn running on http://127.0.0.1:8082 (Press CTRL+C to quit)
[agent_python_strands] INFO:     Started reloader process [1401469] using StatReload
[agent_python_strands] INFO:     Started server process [1401497]
[agent_python_strands] INFO:     Waiting for application startup.
[agent_python_strands] INFO:     Application startup complete.
[agent_python_strands] Agent 'agent_python_strands' is running on port 8082.
$ agentcore project invoke runtime --local --port 8082 --payload '{"prompt": "hello world"}'
[agent response]

before this would open the browser.

@github-actions github-actions Bot added the size/xs PR size: XS label Sep 4, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label 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 added agentcore-harness-reviewing AgentCore Harness review in progress and removed claude-security-reviewing Claude Code /security-review in progress labels Sep 4, 2026
@github-actions github-actions Bot added size/xs PR size: XS and removed size/xs PR size: XS labels Sep 4, 2026
@agentcore-devx-automation agentcore-devx-automation Bot removed the agentcore-harness-reviewing AgentCore Harness 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 fix. The three changes hang together: --mode enum drops tui, the always-throwing flags.mode === "tui" guard is removed, and the default flips to headless. InputValidationError is still used elsewhere in the file, so no orphaned imports. Nothing else in the repo references mode: "tui" besides a leftover in the test harness's helper type (src/handlers/project/dev/index.test.ts:169 still declares mode?: "browser" | "headless" | "tui"). That's harmless — no test actually passes "tui" — so not a blocker, but worth tightening in a follow-up.

One thing worth double-checking before merging: switching the default from browser to headless is a user-visible behavior change. The PR description notes the browser-open behavior is going away by default, which seems intentional given the help text update. Just make sure any docs/tutorials/release notes that show agentcore project dev opening the Agent Inspector are updated accordingly (I didn't find any in-repo, but external docs may exist).

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.98%. Comparing base (81f5af2) to head (3d34600).
⚠️ Report is 1 commits behind head on refactor.

Additional details and impacted files
@@            Coverage Diff            @@
##           refactor    #2236   +/-   ##
=========================================
  Coverage     96.98%   96.98%           
=========================================
  Files           564      564           
  Lines         39226    39221    -5     
=========================================
- Hits          38042    38040    -2     
+ Misses         1184     1181    -3     

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

@Hweinstock
Hweinstock marked this pull request as ready for review September 4, 2026 20:17
@tejaskash
tejaskash merged commit 535785c into aws:refactor Sep 4, 2026
23 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants