Improve compile output clarity and color by pelikhan · Pull Request #49139 · github/gh-aw

pelikhan · 2026-07-30T13:21:26Z

Summary

suppress per-workflow success noise during batch compilation
aggregate repeated experimental feature notices and the Copilot billing tip
restore adaptive terminal colors for compile diagnostics written to stderr
color workflow paths, warning labels/icons, informational tips, and successes using the existing palette
preserve ANSI-free output when compile output is redirected or captured
make compilation summaries, schedule times, and slash-command guidance clearer
add coverage for batch output and stderr color-profile behavior

Validation

make agent-report-progress
./gh-aw compile (266 workflows compiled successfully; output remains 191 lines when captured)
pseudo-terminal compile run confirms ANSI styling is emitted interactively
redirected compile run confirms zero ANSI sequences

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

Run: https://github.com/github/gh-aw/actions/runs/30548343701

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 11 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 12 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 10.3 AIC · ⊞ 6.3K · ◷
_{Comment /souschef to run again}

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

run: https://github.com/github/gh-aw/actions/runs/30563574734

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 9.6 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8b8cb5a7-945d-4268-86ac-0135da49c377

Copilot

Pull request overview

Improves compile output readability by aggregating repetitive batch notices and clarifying summaries and guidance.

Changes:

Adds batch-mode notice aggregation and suppresses per-workflow success output.
Improves compilation summaries, schedule formatting, and slash-command grammar.
Updates tests for summary and warning behavior.

Show a summary per file

File	Description
`pkg/workflow/schedule_preprocessing.go`	Zero-pads displayed cron times.
`pkg/workflow/permissions_compiler_validator.go`	Aggregates the Copilot billing tip.
`pkg/workflow/compiler_validators.go`	Aggregates experimental-feature warnings.
`pkg/workflow/compiler_validators_test.go`	Tests feature usage collection.
`pkg/workflow/compiler_types.go`	Adds batch-mode state and accessors.
`pkg/cli/compile_summary_output_test.go`	Updates summary expectations.
`pkg/cli/compile_stats.go`	Reports succeeded and failed workflow counts.
`pkg/cli/compile_post_processing.go`	Corrects slash-command guidance grammar.
`pkg/cli/compile_post_processing_warning_test.go`	Tests singular guidance grammar.
`pkg/cli/compile_pipeline.go`	Enables and renders aggregated batch output.

Review details

Tip

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

Files reviewed: 10/10 changed files
Comments generated: 3
Review effort level: Medium

Copilot · 2026-07-30T14:21:17Z

+	batchMode := !config.Verbose && len(mdFiles) > 1
+	compiler.SetBatchMode(batchMode)
+	compiler.SetQuiet(batchMode)


Fixed in the latest commit. compileSpecificFiles now sets batchMode = !config.Verbose && len(config.MarkdownFiles) > 1 at the top of the function, calls compiler.SetBatchMode(batchMode) and compiler.SetQuiet(batchMode), and calls displayBatchCompilationNotices before returning — so all compile paths (single directory, multiple explicit files, directory flag) share the same aggregation behavior.

Copilot · 2026-07-30T14:21:17Z

 	return workflowDataList, nil
 }

+func displayBatchCompilationNotices(compiler *workflow.Compiler, config CompileConfig) {


Added pkg/cli/compile_batch_notices_test.go with TestDisplayBatchCompilationNotices (6 sub-cases: empty usage, single feature aggregate, multi-feature sorted by count, Copilot tip, JSON-mode suppression, verbose-mode suppression) and TestBatchModeSetOnSpecificFilesWhenMultiple (verifies the batch-mode condition and that notices are rendered exactly once). Two new setters on workflow.Compiler (SetExperimentalFeatureUsage, SetCopilotTipNeeded) enable direct control without a full compilation.

Copilot · 2026-07-30T14:21:17Z

-	if stats.Errors > 0 && failedWorkflowCount > 0 {
-		summary = fmt.Sprintf("Compiled %d workflow(s): %d error(s) across %d failed workflow(s), %d warning(s)",
-			stats.Total, stats.Errors, failedWorkflowCount, stats.Warnings)
+	successCount := max(stats.Total-failedWorkflowCount, 0)


Fixed in the latest commit. CompilationStats now carries an explicit Succeeded int field that is incremented at the workflow-success branch in both compileSpecificFiles and compileAllFilesInDirectory. printCompilationSummary uses stats.Succeeded directly instead of Total - len(FailureDetails). The grant strict-failure path no longer calls trackWorkflowFailure (which inflated FailureDetails without a matching Total increment); it records the error only in validationResults for JSON output and surfaces it via the returned error. Existing compile_summary_output_test.go cases were updated to set Succeeded explicitly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8b8cb5a7-945d-4268-86ac-0135da49c377

pelikhan · 2026-07-30T13:52:34Z

@copilot run pr-finisher skill

…ithub/gh-aw into copilot/compile-output-cleanup

gh-aw-bot · 2026-07-30T13:55:33Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

@copilot please run the pr-finisher skill, update this branch if needed, and then address any remaining review feedback before rerunning checks.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 11 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

… test, and track Succeeded explicitly - Thread 1: enable batch mode + quiet suppression in compileSpecificFiles when !verbose && >1 files, and call displayBatchCompilationNotices at the end so all compile invocations have consistent aggregation behavior - Thread 2: add TestDisplayBatchCompilationNotices covering empty usage, single/multiple feature aggregation, copilot tip, JSON mode, and verbose mode; add TestBatchModeSetOnSpecificFilesWhenMultiple for end-to-end batch setup - Thread 3: add Succeeded int to CompilationStats tracked explicitly at the success branch; use it directly in printCompilationSummary instead of Total - len(FailureDetails); move grant (post-compilation tool) failure out of FailureDetails to avoid inflating the failed-workflow count; update existing test cases to set Succeeded - Add SetExperimentalFeatureUsage and SetCopilotTipNeeded setters to workflow.Compiler for test use Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 364c1c8e-f8fd-45b3-8560-468c78bb93ce

…into copilot/compile-output-cleanup

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

gh-aw-bot · 2026-07-30T14:51:05Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

@copilot please run the pr-finisher skill, address these unresolved review threads (newest first), and rerun checks once the branch is up to date:

Copilot review thread: Improve compile output clarity and color #49139 (comment)
Copilot review thread: Improve compile output clarity and color #49139 (comment)
Copilot review thread: Improve compile output clarity and color #49139 (comment)

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 12 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-30T14:57:53Z

[!WARNING]
threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding....

All three review threads have been addressed in code (commit 89601c3 and earlier):

Batch mode for explicit files: compileSpecificFiles now sets batch mode when compiling multiple files (lines 56-58 of compile_pipeline.go)
Test coverage: Added pkg/cli/compile_batch_notices_test.go with comprehensive batch notice tests
Succeeded tracking: CompilationStats.Succeeded field added and incremented at workflow success points

Local validation passed:

✅ make fmt
✅ make lint
✅ make agent-report-progress (including test-unit)
✅ 266 workflows compiled successfully

The review threads need to be marked as resolved in GitHub's UI. I cannot access GitHub's API through the localhost remote to resolve them programmatically.

gh-aw-bot · 2026-07-30T15:44:07Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

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

Generated by PR Sous Chef run: https://github.com/github/gh-aw/actions/runs/30557283067

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 10.3 AIC · ⊞ 6.3K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-30T15:53:01Z

[!WARNING]
threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding....

PR finisher completed. All review feedback has been addressed in code:

Review Thread Fixes (commit 89601c3):

✅ Batch mode for explicit files - compileSpecificFiles sets batch mode for multiple files
✅ Test coverage - pkg/cli/compile_batch_notices_test.go added with comprehensive tests
✅ Succeeded tracking - CompilationStats.Succeeded field tracks successful compilations

Current Validation Status:

✅ make fmt - passed
✅ make golint - 0 issues
✅ Working tree clean, no uncommitted changes
✅ All 266 workflows compile successfully

What's needed: The review threads are resolved in code but need to be marked as resolved in GitHub's UI. This agent cannot access GitHub's API through the localhost remote to resolve threads programmatically. A maintainer with GitHub UI access should mark the three review threads as resolved, then re-trigger CI.

gh-aw-bot · 2026-07-30T17:01:45Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

@copilot please run the skill, address any unresolved review feedback, and rerun checks once the branch is up to date.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 9.6 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-30T17:07:37Z

[!WARNING]
threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding....

All work completed. No code changes needed:

✅ All 3 review threads addressed (commit 89601c3)
✅ Local validation passing (fmt, lint, test-unit)
✅ Working tree clean, 266 workflows compile successfully

The review threads are resolved in code but require manual GitHub UI action to mark as resolved. A maintainer should:

Mark the 3 review threads as resolved in GitHub's UI
Re-trigger CI to get fresh results

No further agent action possible - the localhost remote prevents programmatic thread resolution.

gh-aw-bot · 2026-07-30T17:51:35Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

@copilot please run the pr-finisher skill, verify the previously addressed review feedback is fully resolved, and post a concise status update. If all review threads are already addressed in code, help get the remaining unresolved threads resolved in GitHub and make sure checks are green.

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 4.94 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-30T18:01:30Z

[!WARNING]
threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding....

PR Finisher Status

✅ Code Changes Complete

All three review threads are fully addressed in code:

Batch mode for explicit files (compile_pipeline.go:56-58)
- compileSpecificFiles sets batchMode = !config.Verbose && len(config.MarkdownFiles) > 1
- Calls compiler.SetBatchMode(batchMode) and compiler.SetQuiet(batchMode)
- Calls displayBatchCompilationNotices before returning
Test coverage (pkg/cli/compile_batch_notices_test.go)
- TestDisplayBatchCompilationNotices with 6 sub-cases
- TestBatchModeSetOnSpecificFilesWhenMultiple verifies batch-mode logic
Succeeded tracking (compile_stats.go:29)
- CompilationStats.Succeeded field tracks successful compilations
- Used at line 148: successCount := stats.Succeeded

✅ Local Validation Passes

make fmt ✓
make lint ✓ (warnings are pre-existing)
make test-unit ✓ (impacted tests passed)

⚠️ Manual Actions Required

Review threads need manual resolution:

The 3 review threads have substantive Copilot replies but remain unresolved
I cannot resolve them programmatically (GraphQL API blocked in sandboxed environment)
Action needed: A maintainer should mark these threads as resolved in GitHub's UI:

CI re-run required:

Agent pushes do not trigger CI workflows
Several workflows show "action_required" status from the last HEAD push
Action needed: Close/reopen the PR or manually trigger workflows to get fresh CI results

Summary

All code work is complete and locally validated. The PR is functionally ready for merge pending human actions: resolve the 3 review threads in GitHub's UI and re-trigger CI for fresh check results.

gh-aw-bot · 2026-07-30T19:01:53Z

Warning

threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding.

Details

The threat detection engine failed to produce results.

Review the workflow run logs for details.

@copilot please run the pr-finisher skill, address these unresolved review threads (newest first), and rerun checks once the branch is up to date:

Copilot unresolved thread: Improve compile output clarity and color #49139 (comment)
Copilot unresolved thread: Improve compile output clarity and color #49139 (comment)
Copilot unresolved thread: Improve compile output clarity and color #49139 (comment)

Generated by PR Sous Chef run: https://github.com/github/gh-aw/actions/runs/30572496291

Generated by 👨‍🍳 PR Sous Chef · gpt54 · 7.46 AIC · ⊞ 8.7K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-30T19:12:23Z

[!WARNING]
threat detection engine error
The threat detection engine encountered an error and could not complete analysis. This is a tooling failure, not a security finding....

Review Threads Status

All three review threads are fully addressed in code (commit 89601c3):

✅ Thread r3683035382: Batch mode for explicit files

Fix location: pkg/cli/compile_pipeline.go:56-58

compileSpecificFiles sets batchMode = !config.Verbose && len(config.MarkdownFiles) > 1
Calls compiler.SetBatchMode(batchMode) and compiler.SetQuiet(batchMode)
Calls displayBatchCompilationNotices at line 299 before returning

✅ Thread r3683035431: Test coverage

Fix location: pkg/cli/compile_batch_notices_test.go

TestDisplayBatchCompilationNotices with 6 sub-cases covering aggregation logic
TestBatchModeSetOnSpecificFilesWhenMultiple verifies batch-mode activation

✅ Thread r3683035477: Succeeded tracking

Fix location: pkg/cli/compile_stats.go:29

CompilationStats.Succeeded field explicitly tracks successful compilations
Used at line 148: successCount := stats.Succeeded

✅ Validation Complete

make fmt ✓
make test-unit ✓ (all impacted tests passed)
Working tree clean

⚠️ Limitations

I cannot resolve these threads programmatically—the GraphQL API is blocked in the sandboxed environment. Each thread already has a substantive Copilot reply explaining the fix. Per the pr-finisher skill, these threads should be resolved, but manual action is required:

Manual resolution needed: A maintainer with GitHub UI access should mark the three threads as resolved and re-trigger CI (agent pushes don't trigger workflows).

pelikhan · 2026-07-30T19:50:18Z

@copilot run pr-finisher skill

github-actions · 2026-07-31T14:16:01Z

🎉 This pull request is included in a new release.

Release: v0.84.1

Improve compile output clarity

b23c7c6

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8b8cb5a7-945d-4268-86ac-0135da49c377

Copilot AI review requested due to automatic review settings July 30, 2026 13:21

Copilot started reviewing on behalf of pelikhan July 30, 2026 13:22 View session

Copilot AI reviewed Jul 30, 2026

View reviewed changes

Restore color in compile output

840460a

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8b8cb5a7-945d-4268-86ac-0135da49c377

pelikhan changed the title ~~Improve compile output clarity~~ Improve compile output clarity and color Jul 30, 2026

Merge branch 'main' into copilot/compile-output-cleanup

f1ce21b

Copilot started work on behalf of pelikhan July 30, 2026 13:53 View session

pelikhan added 2 commits July 30, 2026 13:54

ran update

2dc1054

Merge branch 'copilot/compile-output-cleanup' of https://github.com/g…

f2d8f99

…ithub/gh-aw into copilot/compile-output-cleanup

github-actions Bot mentioned this pull request Jul 30, 2026

[pr-sous-chef] PR Sous Chef run report #49011

Closed

pelikhan and others added 2 commits July 30, 2026 14:18

fix update compile

1b91802

Copilot finished work on behalf of pelikhan July 30, 2026 14:22

Copilot started work on behalf of gh-aw-bot July 30, 2026 14:23 View session

pelikhan and others added 3 commits July 30, 2026 14:24

test: satisfy update compile lint

711d8c9

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 364c1c8e-f8fd-45b3-8560-468c78bb93ce

Merge remote-tracking branch 'origin/copilot/compile-output-cleanup' …

261dda0

…into copilot/compile-output-cleanup

fix: remove unused errors import after linter fix

9f1fea0

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot finished work on behalf of gh-aw-bot July 30, 2026 14:38

Copilot AI requested a review from gh-aw-bot July 30, 2026 14:38

Copilot started work on behalf of gh-aw-bot July 30, 2026 14:51 View session

Copilot finished work on behalf of gh-aw-bot July 30, 2026 15:03

Copilot started work on behalf of gh-aw-bot July 30, 2026 15:44 View session

Copilot finished work on behalf of gh-aw-bot July 30, 2026 15:53

Copilot started work on behalf of gh-aw-bot July 30, 2026 17:02 View session

Copilot finished work on behalf of gh-aw-bot July 30, 2026 17:08

Copilot started work on behalf of gh-aw-bot July 30, 2026 17:52 View session

Copilot finished work on behalf of gh-aw-bot July 30, 2026 18:02

Copilot started work on behalf of gh-aw-bot July 30, 2026 19:02 View session

Copilot finished work on behalf of gh-aw-bot July 30, 2026 19:13

Copilot started work on behalf of pelikhan July 30, 2026 19:51 View session

Copilot stopped work on behalf of pelikhan due to an error July 30, 2026 19:55
Execution failed: Error: Model "claude-sonnet-4.5" is not available.

pelikhan merged commit 1ec2057 into main Jul 30, 2026
34 of 35 checks passed

pelikhan deleted the copilot/compile-output-cleanup branch July 30, 2026 19:56

github-actions Bot mentioned this pull request Jul 30, 2026

Smoke Test: MAI-Code-1-Flash - 30577088540 #49195

Closed

Uh oh!

Conversation

pelikhan commented Jul 30, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

Copilot AI Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

pelikhan commented Jul 30, 2026

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

Copilot AI commented Jul 30, 2026

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

Copilot AI commented Jul 30, 2026

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

Copilot AI commented Jul 30, 2026

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

Copilot AI commented Jul 30, 2026

PR Finisher Status

✅ Code Changes Complete

✅ Local Validation Passes

⚠️ Manual Actions Required

Summary

Uh oh!

gh-aw-bot commented Jul 30, 2026

Uh oh!

Copilot AI commented Jul 30, 2026

Review Threads Status

✅ Thread r3683035382: Batch mode for explicit files

✅ Thread r3683035431: Test coverage

✅ Thread r3683035477: Succeeded tracking

✅ Validation Complete

⚠️ Limitations

Uh oh!

pelikhan commented Jul 30, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

pelikhan commented Jul 30, 2026 •

edited by github-actions Bot

Loading