Skip to content

fix(dev): skip deploy for plain agents without harnesses#1313

Merged
notgitika merged 1 commit into
aws:previewfrom
notgitika:fix/dev-no-deploy-plain-agents-v2
May 20, 2026
Merged

fix(dev): skip deploy for plain agents without harnesses#1313
notgitika merged 1 commit into
aws:previewfrom
notgitika:fix/dev-no-deploy-plain-agents-v2

Conversation

@notgitika
Copy link
Copy Markdown
Contributor

@notgitika notgitika commented May 20, 2026

Summary

  • agentcore dev was triggering a full CDK deploy even for plain agents (Strands, LangGraph, etc.) that only need a local dev server. This is a bug, it should only deploy when the project has a harness.

Changes

  • browser-mode.ts: Guard runCliDeploy() with if (hasHarnesses) in launchBrowserDev()
  • command.tsx: Guard the --logs path deploy with && hasHarnesses
  • useDevDeploy.ts: Early-return in the TUI hook when project has no harnesses

Test plan

  • All 132 existing dev/hook tests pass
  • agentcore dev with a plain Strands agent (no harness) starts local server without deploying
  • agentcore dev with a harness project still triggers deploy as before
  • agentcore dev --logs with plain agent skips deploy
  • TUI dev mode with plain agent shows "Local agent — no deploy needed"

Closes #1189

`agentcore dev` was triggering a full CDK deploy even for plain agents
(Strands, LangGraph, etc.) that only need a local dev server. This
blocked users from iterating locally without AWS credentials/targets
configured.

Now `runCliDeploy()` is only invoked when the project has harnesses
(cloud-dependent resources). Plain agents start the local server
immediately.

Closes aws#1189
@notgitika notgitika requested a review from a team May 20, 2026 04:39
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label May 20, 2026
@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label May 20, 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 May 20, 2026
@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label May 20, 2026
@notgitika notgitika merged commit c3f52fd into aws:preview May 20, 2026
19 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants