Skip to content

Fix cswin32-update agent prompt to specify item_number for add-comment#66263

Merged
DeagleGross merged 1 commit into
dotnet:mainfrom
DeagleGross:fix/cswin32-add-comment-prompt
Apr 10, 2026
Merged

Fix cswin32-update agent prompt to specify item_number for add-comment#66263
DeagleGross merged 1 commit into
dotnet:mainfrom
DeagleGross:fix/cswin32-add-comment-prompt

Conversation

@DeagleGross
Copy link
Copy Markdown
Member

Test workflow run posted a PR, but did not manage to post a comment:

safe_outputs
✗ Message 2 (add_comment) failed: Target is "*" but no item_number/issue_number specified in add_comment item

Added a prompt instructions to include the item_number - it may help.

The agent was calling add_comment without specifying which PR to comment on,
causing the safe_outputs job to fail with:
  Target is * but no item_number/issue_number specified in add_comment item

Explicitly instruct the agent to include the created PR number as item_number.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@DeagleGross DeagleGross self-assigned this Apr 10, 2026
@DeagleGross DeagleGross requested a review from wtgodbe as a code owner April 10, 2026 09:39
Copilot AI review requested due to automatic review settings April 10, 2026 09:39
@DeagleGross DeagleGross requested a review from a team as a code owner April 10, 2026 09:39
@DeagleGross DeagleGross added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 10, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the cswin32-update agent workflow prompt to prevent failures when posting an add-comment safe output by explicitly requiring the created PR number to be provided as item_number.

Changes:

  • Clarifies add-comment usage so the agent includes item_number (the created PR number) when posting the post-PR note.

@DeagleGross DeagleGross merged commit 2e38cab into dotnet:main Apr 10, 2026
22 checks passed
@DeagleGross DeagleGross deleted the fix/cswin32-add-comment-prompt branch April 10, 2026 15:59
@dotnet-policy-service dotnet-policy-service Bot added this to the 11.0-preview4 milestone Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants