Require explicit noop signal in Code Scanning Fixer to prevent false failures on API errors#10783
Closed
Require explicit noop signal in Code Scanning Fixer to prevent false failures on API errors#10783
Conversation
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
When no high severity alerts are found, the agent correctly reports success but may encounter transient Copilot API errors like "missing finish_reason". Now explicitly require using the noop tool to signal successful completion, which prevents these transient errors from being reported as workflow failures. Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Debug workflow failure in Code Scanning Fixer
Require explicit noop signal in Code Scanning Fixer to prevent false failures on API errors
Jan 20, 2026
Contributor
|
@copilot format, lint |
Contributor
Author
|
@pelikhan Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub. You can ask me to try again later by mentioning me in a new comment. If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: Sorry for the inconvenience! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Code Scanning Fixer workflow failed with
missing finish_reason for choice 0despite successfully finding no alerts to fix. This transient Copilot API error occurred after the agent completed its work, causing spurious failure tracking issues.Changes
nooptool when no alerts are foundnoop(message="...")callsnoopsignals persist even if CLI fails afterwardWhy this works
The safe output system captures
noopcalls independently of CLI completion. When the API fails mid-response, the previously recordednoopsignal indicates successful completion, preventing false failure reports.Before:
- If no unfixed high severity alerts remain, exit gracefully with message: "..."After:
The workflow now distinguishes between genuine failures (agent couldn't complete work) and transient API errors (agent completed work, CLI failed afterward).
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.