chore: upgrade bug-fix and bug-test to gh-aw v0.88.7 - #4604
Open
KSchlobohm wants to merge 4 commits into
Open
KSchlobohm wants to merge 4 commits into
KSchlobohm wants to merge 4 commits into
Conversation
Regenerate bug-fix and bug-test with Copilot 1.0.80. Preserve the 1 MB patch limit and editable Python test extras through the local-package validation workaround. Add focused workflow contract regressions. Accept the v0.88.7 Copilot-specific protected-file set: CLAUDE.md and GEMINI.md are no longer hard-blocked. This exception was approved by the user. Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Remove the 1 MB compatibility override and its regression assertions. Let gh-aw v0.88.7 supply its default 4 MB patch limit without adding a repository-specific restriction. Regenerate the lock with the approved compiler. Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Allow github.com for origin Git access. Require successful discovery before using the current-checkout fallback, report discovery or checkout errors as setup failures, and require original exit codes before output filtering. Include focused regressions without a standalone shell recipe. These instructions do not guarantee agent compliance. Fork tests demonstrated Git access and exact-head handoff; inconsistent exit capture and other observed runtime limitations remain documented in the review evidence. Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The intentional 4 MB patch limit is not protected by the new regression tests.
Get a fresh assessment by requesting another Copilot review.
Review tier: Balanced
Findings: 1
Open (1)
What changed in this PR
Upgrades the bug-fix and bug-test automation to gh-aw v0.88.7 and Copilot 1.0.80 while preserving workflow contracts.
Changes:
- Regenerates both workflow locks.
- Hardens bug-test checkout and exit-code handling.
- Adds regression coverage for workflow contracts and installation failures.
| File | Description |
|---|---|
.github/workflows/bug-fix.lock.yml |
Regenerates bug-fix with upgraded runtime. |
.github/workflows/bug-test.md |
Updates networking, installation, and failure guidance. |
.github/workflows/bug-test.lock.yml |
Regenerates bug-test with upgraded runtime. |
tests/test_github_workflows.py |
Adds workflow contract and regression tests. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Add assertion for maximum patch size in PR creation Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The large generated workflow changes affect privileged automation and warrant final human review despite strong regression and fork coverage.
Review tier: Balanced
Findings: None
Resolved since last review (1)
mnriem
added
the
triage-nice-to-have
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.
Description
Upgrade
bug-fixandbug-testtogether from gh-aw v0.79.8 / Copilot 1.0.60 to gh-aw v0.88.7 / Copilot 1.0.80, following #4546.--strict --validate.github.comexplicitly for origin fetches. Clarify that failed discovery or checkout is a setup failure, not evidence that no fix exists, and require original exit codes before output filtering.Source-level label gates, bot exclusions, permissions, allowed outputs, and output limits are preserved. This intentionally accepts the new compiler defaults: a 4 MB patch limit instead of 1 MB, and protected-file checks that retain Copilot instruction files but no longer include
CLAUDE.mdorGEMINI.md.The diff contains only both generated locks,
bug-test.md, andtests/test_github_workflows.py. No community workflows, shared action pins, test fixtures, or generated product fixes are included.Testing
uv run specify --helpLocal validation:
git diff --checkpassed.Applicable CI passed on the final fork upgrade and reliability PRs, including Linux, Windows, and macOS with Python 3.13 and 3.14.
Fork testing provided evidence that the upgraded workflows work together in practice—not just that they compile—and that test results reflect the proposed fix rather than the wrong checkout. The end-to-end run supports that conclusion, but also exposed instruction-following deviations; it does not establish complete workflow-contract compliance. Three broader-suite failures also reproduced on the base checkout.
AI Disclosure
GitHub Copilot (GPT-6 Astra) prepared the workflow changes, regression tests, validation, and this description on behalf of @KSchlobohm. The human contributor directed scope and approved fork publication and controlled test runs. Agent-authored commits include attribution. Fork runtime behavior was checked against logs and GitHub state.