Skip to content

fix(status): show 'No target configured' instead of empty target label (#985) - #2077

Open
lvthillo wants to merge 1 commit into
aws:mainfrom
lvthillo:fix/status-empty-target-label
Open

fix(status): show 'No target configured' instead of empty target label (#985)#2077
lvthillo wants to merge 1 commit into
aws:mainfrom
lvthillo:fix/status-empty-target-label

Conversation

@lvthillo

Copy link
Copy Markdown

Description

agentcore status printed an empty target label: AgentCore Status (target: ) on projects with no deployment target configured (any fresh agentcore create project, since it writes an empty aws-targets.json).

Root cause: the status action always returned a targetName string, substituting an empty string when no target was configured. The render layer only shows its "No target configured" fallback when targetName is missing entirely: an empty string counts as present, so the header printed a blank label. The fix is to leave targetName unset when there is no target, letting the existing fallback take over.

Verified with the built CLI: header now renders AgentCore Status (target: No target configured).
Note: status --json now omits targetName instead of emitting "targetName": "".

Related Issue

#985

Closes #

#985

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.

@lvthillo
lvthillo requested a review from a team August 23, 2026 20:06
@github-actions github-actions Bot added the size/s PR size: S label Aug 23, 2026
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.

1 participant