Skip to content

fix: make agent preview start require preview mode flag @W-21785024@#380

Merged
shetzel merged 10 commits intomainfrom
wr/noDefaultForPreview
Mar 31, 2026
Merged

fix: make agent preview start require preview mode flag @W-21785024@#380
shetzel merged 10 commits intomainfrom
wr/noDefaultForPreview

Conversation

@WillieRuemmele
Copy link
Copy Markdown
Contributor

What does this PR do?

makes agent preview start require either --use-live-test or --simulate-actions flag

What issues does this PR fix or reference?

@W-21785024@

@WillieRuemmele WillieRuemmele requested a review from a team as a code owner March 27, 2026 20:20
@shetzel
Copy link
Copy Markdown
Contributor

shetzel commented Mar 31, 2026

QA:
✅ agent preview start -> errors as it should due to lack of info
✅ agent preview start --authoring-bundle MyAab -> errors since it needs explicit flag
✅ agent preview start --authoring-bundle foo --simulate-actions -> errors since foo isn't found
✅ agent preview start --authoring-bundle MyAab --simulate-actions -> success
✅ agent preview start --authoring-bundle MyAab --use-live-actions -> success
✅ agent preview start --api-name foo -> errors since foo isn't found
✅ agent preview start --api-name MyAab -> success
✅ agent preview start --api-name MyAab --simulate-actions -> success but warns that live actions are always used
✅ agent preview start --api-name MyAab --use-live-actions -> success but warns that live actions are always used

@shetzel shetzel merged commit 8635da4 into main Mar 31, 2026
15 checks passed
@shetzel shetzel deleted the wr/noDefaultForPreview branch March 31, 2026 21:06
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.

3 participants