feat: add pr iteration loop workflow - #1821
Conversation
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🔍 PR Validation |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. OpenSSF Scorecard
Scanned Files
|
There was a problem hiding this comment.
🟡 Changes recommended
The selection logic violates its documented priority, the version bump breaks an existing governance test, and Poutine-only changes bypass validation.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds an automated PR iteration workflow that selects repository work, maintains cross-run memory, verifies outcomes, and updates gh-aw tooling.
Changes:
- Adds the PR iteration workflow and compiled lockfile.
- Upgrades gh-aw to v0.88.7 and regenerates workflow output.
- Updates scanner configuration and workflow documentation.
File summaries
| File | Description |
|---|---|
.github/workflows/pr-iteration-loop.md |
Defines triggers, selection, tools, safe outputs, and reporting rules. |
.github/workflows/pr-iteration-loop.lock.yml |
Compiled executable workflow. |
.github/workflows/focused-coverage-controller.lock.yml |
Regenerates the controller with gh-aw v0.88.7. |
.github/workflows/gh-aw-validation.yml |
Pins and validates gh-aw v0.88.7 workflows. |
.github/aw/actions-lock.json |
Pins matching gh-aw action commits. |
.poutine.yml |
Allows GitHub’s ubuntu-slim runner. |
.github/workflows/README.md |
Documents the new workflow and version. |
.github/workflows/AUDIT.md |
Updates the validation audit entry. |
docs/AGENT_CAPABILITIES_CHECKLIST.md |
Updates the documented gh-aw version. |
Review details
- Files reviewed: 7/11 changed files
- Comments generated: 3
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "github/gh-aw-actions/setup@v0.88.7": { | ||
| "repo": "github/gh-aw-actions/setup", | ||
| "version": "v0.82.14", | ||
| "sha": "b6d1443e05b8716267fa19425b99aa4f12006b4a" | ||
| "version": "v0.88.7", | ||
| "sha": "5e508589e03a7757a7e05b26e834292f5445bfb6" |
There was a problem hiding this comment.
Fixed in 41aa783. Updated governance expectations to gh-aw v0.88.7 (keys, SHAs, install pin, and workflow list) so the test now matches the lock/runtime configuration.
| generated_at: new Date().toISOString(), | ||
| }; | ||
|
|
||
| if (context.eventName === "issues") { |
There was a problem hiding this comment.
Fixed in 41aa783. Selection now always computes ranked candidates and picks by the documented priority order; issue/PR trigger payloads are retained as context only.
| @@ -0,0 +1,8 @@ | |||
| # Configure poutine security scanner | |||
There was a problem hiding this comment.
Fixed in 41aa783. Added .poutine.yml to both push and pull_request paths filters in gh-aw-validation.yml, with a regression test.
CI/Coverage failure diagnosed and fix ready (blocked by branch history, not this change)Root cause: Verified fix (passes locally, diff --git a/tests/unit/test_gh_aw_workflow_governance.py b/tests/unit/test_gh_aw_workflow_governance.py
index a548147d6..fa58ef6a3 100644
--- a/tests/unit/test_gh_aw_workflow_governance.py
+++ b/tests/unit/test_gh_aw_workflow_governance.py
@@ -164,11 +164,11 @@ def test_gh_aw_validation_pins_runtime_version() -> None:
actions_lock = json.loads((ROOT / ".github/aw/actions-lock.json").read_text())
assert workflow["name"] == "gh-aw Validation"
- entry = actions_lock["entries"]["github/gh-aw-actions/setup@v0.82.14"]
- assert entry["sha"] == "b6d1443e05b8716267fa19425b99aa4f12006b4a"
+ entry = actions_lock["entries"]["github/gh-aw-actions/setup@v0.88.7"]
+ assert entry["sha"] == "5e508589e03a7757a7e05b26e834292f5445bfb6"
step_scripts = [step.get("run", "") for step in workflow["jobs"]["validate-gh-aw"]["steps"]]
combined = "\n".join(step_scripts)
- assert "gh extension install github/gh-aw --pin v0.82.14" in combined
+ assert "gh extension install github/gh-aw --pin v0.88.7" in combined
assert "eventrelay-ci-investigator" not in combined
assert "canonical-pr-remediator" in combined
assert "focused-coverage-controller" in combinedWhy I couldn't push it directly: Please apply the patch above directly (e.g.
|
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Addressed in 41aa783. All requested fixes from this review thread are now applied and validated with targeted governance tests plus parallel validation. |
Canonical issue
Link to the canonical issue in the PR metadata.
Outcome
Adds a repository-specific
pr-iteration-loopagentic workflow for Copilot: it selects one bounded repo problem per run, iterates on one long-running draft PR, persists cross-run memory, and grades whether the run delivered operational value for EventRelay.Scope
.github/workflows/pr-iteration-loop.mdwith issue/PR/main/scheduled triggers, repo-scoped prompt, cache-memory, Playwright CLI, agentic-workflows tooling, LSP config, chart uploads, and bounded loop rules.pr-iteration-loop.lock.yml, added the workflow to validation, and updated the repo’s gh-aw pin/lock references tov0.88.7.Risk
lspandmerge-pull-requestremain experimental surfaces.pr-iteration-loopfiles and the gh-aw pin/validation updates.Verification
List exact automated and manual checks, tied to the current head SHA.
gh aw versiongh aw compile canonical-pr-remediator focused-coverage-controller pr-iteration-loop repo-assist --validate --approvegh aw compile canonical-pr-remediator focused-coverage-controller pr-iteration-loop repo-assist --actionlint --zizmor --poutine --approvegit diff --checkProduction evidence
Not applicable; this PR only adds repository automation workflow definitions and compiled lockfiles.
Agent handoff