Skip to content

Remove hardcoded invoke suggestion from azd ai agent run#7898

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/remove-incorrect-payload-suggestions
Draft

Remove hardcoded invoke suggestion from azd ai agent run#7898
Copilot wants to merge 2 commits intomainfrom
copilot/remove-incorrect-payload-suggestions

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 23, 2026

azd ai agent run was printing a static azd ai agent invoke --local "Hello!" suggestion that is incorrect for agents using the /invocations protocol — the payload shape is agent-specific and "Hello!" is not a valid invocation body in most cases, leading to confusing runtime errors.

Changes

  • runRun(): Remove the three-line "After startup, in another terminal, try: …" block; retain only the Starting agent on http://localhost:<port> (Ctrl+C to stop) line.
  • newRunCommand() Long description: Remove the trailing "Use a separate terminal to invoke the running agent: …" paragraph for the same reason.

Copilot AI changed the title [WIP] Remove incorrect payload suggestions on run command Remove hardcoded invoke suggestion from azd ai agent run Apr 23, 2026
Copilot AI requested a review from therealjohn April 23, 2026 15:15
Copy link
Copy Markdown
Member

@jongio jongio left a comment

Choose a reason for hiding this comment

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

Changes look correct - removes the hardcoded "Hello!" suggestion that doesn't match most agent protocols.

Both locations (the Long description and the runtime fmt.Printf output) are cleaned up. The remaining \n prefix on the Printf properly preserves blank-line spacing. No test impact.

Minor note: invoke.go:63 still shows azd ai agent invoke --local "Hello!" in the invoke command's own examples. That's a different context (documenting invoke's syntax vs suggesting a specific payload from run), but worth considering whether the invoke examples should also use a more protocol-appropriate placeholder.

@JeffreyCA JeffreyCA added the ext-agents azure.ai.agents extension label May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ext-agents azure.ai.agents extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove incorrect payload suggestions on run command

4 participants