fix(aw-compat): gh aw fix returns exit code 2 and correct summary when guided-error codemod fires by Copilot · Pull Request #43169 · github/gh-aw

Copilot · 2026-07-03T10:26:29Z

gh aw fix was printing ✓ No fixes needed (exit 0) even when a guided-error codemod (e.g. top-level-env-secrets-guided-error) fired, because processing errors caused the file to be skipped before the fix counter was incremented — leaving the summary with zero fixes and a false success signal.

Changes

GuidedError sentinel type (fix_codemods.go) — new error type with Unwrap support; distinguishes manual-fix conditions from hard processing failures.
Guided bool field on Codemod — marks codemods whose Apply errors are advisory (not auto-correctable). getTopLevelEnvSecretsGuidedErrorCodemod is the first consumer.
processWorkflowFileWithInfo — wraps errors from Guided: true codemods as &GuidedError{Cause: ...} so the caller can identify them via errors.As.
runFixCommand summary logic:
- Tracks totalGuidedErrors separately from auto-fix counts.
- Suppresses ✓ No fixes needed when guided errors are present.
- Prints ✗ N file(s) need a manual fix.
- Returns &ExitCodeError{Code: 2} — distinct from the crash exit code (1) so automation can branch on this condition.

Before / After

# Before
✗ Error processing smoke.md: codemod top-level-env-secrets-guided-error failed: ...Manual fix required...
i ✓ No fixes needed
$ echo $?
0
# After
✗ Error processing smoke.md: codemod top-level-env-secrets-guided-error failed: ...Manual fix required...
✗ 1 file needs a manual fix
$ echo $?
2

Generated by 👨‍🍳 PR Sous Chef · 7.05 AIC · ⌖ 5.18 AIC · ⊞ 6.6K · ◷
_{Comment /souschef to run again}

… message Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot

Pull request overview

This PR improves gh aw fix’s behavior when a “guided/manual-fix” codemod fires (e.g., top-level env: secrets): instead of reporting “No fixes needed” with exit code 0, it now emits a manual-fix summary and returns an ExitCodeError{Code: 2} so automation can branch on “manual intervention required”.

Changes:

Introduces a GuidedError sentinel (with Unwrap) and a Codemod.Guided flag to classify advisory/manual-fix codemod failures.
Updates processWorkflowFileWithInfo to wrap Apply errors from Guided: true codemods as GuidedError.
Updates runFixCommand to count guided errors, suppress “✓ No fixes needed” when present, print a manual-fix summary line, and return exit code 2.
Adds tests covering exit code 2, suppression of “No fixes needed”, pluralization, and GuidedError wrapping.

Show a summary per file

File	Description
pkg/cli/fix_command.go	Tracks guided/manual-fix codemod errors, adjusts summary output, and returns exit code 2 when manual fixes are required.
pkg/cli/fix_command_test.go	Adds regression tests for guided-error exit code + output behavior and for GuidedError wrapping.
pkg/cli/fix_codemods.go	Adds `Codemod.Guided` and the `GuidedError` sentinel type.
pkg/cli/codemod_engine_env_secrets.go	Marks the top-level env secrets codemod as `Guided: true`.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Files reviewed: 4/4 changed files
Comments generated: 2
Review effort level: Low

 fmt.Fprintf(os.Stderr, "%s\n", console.FormatErrorMessage(fmt.Sprintf("Error processing %s: %v", filepath.Base(file), err)))
+ var guidedErr *GuidedError
+ if errors.As(err, &guidedErr) {
+ totalGuidedErrors++
+ }


 if totalFixed > 0 {
 fmt.Fprintf(os.Stderr, "%s\n", console.FormatSuccessMessage(fmt.Sprintf("✓ Fixed %d of %d workflow files", totalFixed, totalFiles)))
- } else {
+ } else if totalGuidedErrors == 0 {
 fmt.Fprintf(os.Stderr, "%s\n", console.FormatInfoMessage("✓ No fixes needed"))
 }


github-actions · 2026-07-03T13:02:22Z

🤖 PR Triage

Field	Value
Category	`bug`
Risk	🟢 Low
Score	60/100 (Impact 28 + Urgency 18 + Quality 14)
Action	⚡ `fast_track`
Batch	`pr-batch:safe-output-fixes` (with #43168)

Score Breakdown

Impact (28/50): Fixes gh aw fix false success signal — important CLI correctness fix
Urgency (18/30): CI checks all skipped; no reviews; surgical 4-file change
Quality (14/20): Precise root-cause description, minimal scope, targeted fix

Batch candidate with #43168 (safe-output ingestion hardening). Both are small, clean bug fixes.

Generated by 🔧 PR Triage Agent · 92.3 AIC · ⌖ 6.73 AIC · ⊞ 5.5K · ◷

gh-aw-bot · 2026-07-03T14:33:10Z

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Generated by 👨‍🍳 PR Sous Chef · 7.05 AIC · ⌖ 5.18 AIC · ⊞ 6.6K · ◷
_{Comment /souschef to run again}

…nd return exit 1 on hard processing errors Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot · 2026-07-03T14:41:46Z

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.
...

Done — addressed both unresolved review threads in commit fix: count guided-error files in totalFiles; suppress false success and return exit 1 on hard processing errors:

totalFiles denominator — guided-error files are now counted in totalFiles (incremented before the continue), so the "Fixed X of Y" summary is accurate when a mix of clean, guided-error, and hard-error files is processed.
Hard processing errors — a new totalProcessingErrors counter tracks non-guided failures (read/parse/write errors). When non-zero, "✓ No fixes needed" is suppressed and the function returns ExitCodeError{Code: 1}, giving automation a clear non-zero signal distinct from the guided-error code 2.

Three new tests were added: exit code 1 for hard errors, suppression of "No fixes needed" for hard errors, and a mixed-file test verifying totalFiles includes guided-error files. Local make fmt and make test-unit are green.

github-actions · 2026-07-07T10:09:04Z

🎉 This pull request is included in a new release.

Release: v0.82.3

Initial plan
158a18e

CopilotAI assigned Copilot and pelikhan Jul 3, 2026

Copilotstarted work on behalf of pelikhan July 3, 2026 10:26 View session

CopilotAI linked an issue Jul 3, 2026 that may be closed by this pull request
[aw-compat] gh aw fix: reports "✓ No fixes needed" (exit 0) while a guided-error codemod says "Manual fix required" #43163
Closed

CopilotAIand others added 2 commits July 3, 2026 10:40

fix: guided-error codemods now return exit code 2 and correct summary…
99830b5
… message Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

review: rename noun to pluralSuffix, add singular form assertion in test
4a04da8
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

CopilotAI changed the title ~~[WIP] Fix gh aw reporting manual fixes needed incorrectly~~fix(aw-compat): gh aw fix returns exit code 2 and correct summary when guided-error codemod firesJul 3, 2026

Copilotfinished work on behalf of pelikhan July 3, 2026 10:44

CopilotAI requested a review from pelikhan July 3, 2026 10:44

github-actionsBot mentioned this pull request Jul 3, 2026
[aw] No-Op Runs #39849
Closed

pelikhan marked this pull request as ready for review July 3, 2026 12:50

CopilotAI review requested due to automatic review settings July 3, 2026 12:50

Copilotstarted reviewing on behalf of pelikhan July 3, 2026 12:51 View session

CopilotAI reviewed Jul 3, 2026
View reviewed changes

github-actionsBot added pr-action:fast_track pr-agent:copilot-swe-agent pr-priority:medium pr-risk:low pr-type:bug pr-batch:safe-output-fixes labels Jul 3, 2026

This was referenced Jul 3, 2026
Harden issue-intent confidence handling in safe-output ingestion #43168
Closed
[PR Triage Report] 🤖 PR Triage Report — 2026-07-03 Run #28661615516 #43190
Closed

Merge branch 'main' into copilot/aw-compat-fix-gh-aw-reporting
e94fc4c

Copilotstarted work on behalf of gh-aw-bot July 3, 2026 14:33 View session

fix: count guided-error files in totalFiles; suppress false success a…
a1d3709
…nd return exit 1 on hard processing errors Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilotfinished work on behalf of gh-aw-bot July 3, 2026 14:42

CopilotAI requested a review from gh-aw-bot July 3, 2026 14:42

This was referenced Jul 3, 2026
[PR Triage Report] PR Triage Report — 2026-07-03 (Run #28677894072) #43257
Closed
[PR Triage Report] PR Triage Report — 2026-07-04 (Run #28689979672) #43275
Closed

pelikhan merged commit 068e5ab into mainJul 4, 2026
29 checks passed

pelikhan deleted the copilot/aw-compat-fix-gh-aw-reporting branch July 4, 2026 03:04

Navigation Menu

Uh oh!

fix(aw-compat): gh aw fix returns exit code 2 and correct summary when guided-error codemod fires - #43169

fix(aw-compat): gh aw fix returns exit code 2 and correct summary when guided-error codemod fires#43169
pelikhan merged 5 commits into
mainfrom
copilot/aw-compat-fix-gh-aw-reporting

CopilotAI commented Jul 3, 2026•
edited by github-actions Bot
Loading

Uh oh!

CopilotAI left a comment

Uh oh!

github-actionsBot commented Jul 3, 2026

Uh oh!

gh-aw-bot commented Jul 3, 2026

Uh oh!

CopilotAI commented Jul 3, 2026

Uh oh!

Uh oh!

github-actionsBot commented Jul 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

CopilotAI commented Jul 3, 2026• edited by github-actions BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Before / After

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

github-actionsBot commented Jul 3, 2026

🤖 PR Triage

Score Breakdown

Uh oh!

gh-aw-bot commented Jul 3, 2026

Uh oh!

CopilotAI commented Jul 3, 2026

Uh oh!

Uh oh!

github-actionsBot commented Jul 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

CopilotAI commented Jul 3, 2026•
edited by github-actions Bot
Loading