Skip to content

fix: conditionally show invoke in deploy next steps only when agents exist#508

Merged
notgitika merged 1 commit intoaws:mainfrom
notgitika:notgitika/fix-next-steps-invoke
Mar 6, 2026
Merged

fix: conditionally show invoke in deploy next steps only when agents exist#508
notgitika merged 1 commit intoaws:mainfrom
notgitika:notgitika/fix-next-steps-invoke

Conversation

@notgitika
Copy link
Contributor

Description

The TUI deploy screen always showed invoke as a next step after deployment, even when no agents were configured (e.g., memory-only projects). This was misleading since there's nothing to invoke without an agent.

Changed DEPLOY_NEXT_STEPS from a static constant to a function getDeployNextSteps(hasAgents) that checks context.projectSpec.agents to determine the appropriate next steps. When no agents exist, it now shows "Add an agent" instead of "invoke". This aligns TUI behavior with the CLI deploy path (actions.ts) which already handled this correctly.

Related Issue

Closes #507

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.

@notgitika notgitika requested a review from a team March 6, 2026 18:45
Copy link
Contributor

@jesseturner21 jesseturner21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

niceeeee

@github-actions github-actions bot added the size/xs PR size: XS label Mar 6, 2026
@notgitika notgitika merged commit baae06b into aws:main Mar 6, 2026
16 of 17 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.

Deploy next steps show 'invoke' even when no agents exist

2 participants