fix(aw): Issue Arborist - revert curl /meta workaround now that mcpg v0.3.12+ ships#8550
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Issue Arborist agentic workflow to validate that the DIFC proxy /meta block is resolved in newer gh-aw-mcpg images, by moving back to the upstream gh issue list implementation and bumping the bundled gh-aw-actions/setup tooling.
Changes:
- Reverts Issue Arborist “Fetch issues data” from the curl+REST workaround back to
gh issue list. - Regenerates the compiled
issue-arborist.lock.ymlusinggh aw v0.75.4(bringing in newer AWF/mcpg/firewall versions). - Updates
.github/aw/actions-lock.jsonentries for the newgithub/gh-aw-actions/setup@v0.75.4reference.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/issue-arborist.md | Reverts issue fetching back to gh issue list (upstream behavior). |
| .github/workflows/issue-arborist.lock.yml | Regenerated compiled workflow with updated gh-aw tooling and proxy image versions. |
| .github/aw/actions-lock.json | Updates locked action entry for github/gh-aw-actions/setup to v0.75.4. |
Copilot's findings
- Files reviewed: 3/3 changed files
- Comments generated: 3
| "sha": "f889c9c3c06adeaabccefc06e29c42733ee05dff" | ||
| "version": "v0.75.4", | ||
| "sha": "9f050961da586148d135e113d8bb025185cdf2b8" | ||
| }, |
Evangelink
left a comment
There was a problem hiding this comment.
✅ 21/21 dimensions clean — no findings.
Review Summary
This PR correctly reverts the curl-based workaround introduced in #8507 and #8185, restoring the upstream gh issue list command now that the DIFC proxy /meta block is fixed in gh-aw-mcpg v0.3.18.
Verified:
- ✅
gh-aw-actions/setupupgraded fromv0.75.0→v0.75.4(SHA9f050961da586148d135e113d8bb025185cdf2b8) - ✅
gh-aw-mcpgupgraded fromv0.3.17→v0.3.18in the lock file manifest - ✅
.github/workflows/issue-arborist.mdreverted to upstreamgithubnext/agenticssource (simplified from 38-line curl + jq to 6-linegh issue list) - ✅
.github/workflows/issue-arborist.lock.ymlregenerated with 1465 lines (up from 1428 — includes updated firewall componentsv0.25.53) - ✅ Search filter equivalence:
-is:sub-issue(REST API) ≈-parent-issue:*(GraphQL via gh CLI) - ✅ Error handling change is intentional: removed
|| echo '[]'fallback so the workflow will fail-fast ifgh issue liststill breaks, allowing validation of the DIFC proxy fix
Scope discipline: Single-purpose PR with proper cross-references to #8507, #8185, and githubnext/agentics#339. All changes are mechanical (version bumps + lock file regeneration + upstream revert).
No action items.
Generated by Expert Code Review (on open) for issue #8550 · ● 1.3M
…v0.3.12+ ships Per githubnext/agentics#339, the DIFC proxy /meta block was fixed in gh-aw-mcpg v0.3.12. main already pins ghcr.io/github/gh-aw-mcpg:v0.3.17 (via gh-aw v0.75.0), which lets `gh issue list` go through the proxy again, so we no longer need the curl + REST API workaround from #8507 / #8185. Reverts the workflow source to upstream (githubnext/agentics) and regenerates the lockfile with gh-aw v0.75.0. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
0ae7473 to
f8bc779
Compare
|
Addressed by re-compiling with
Note: main's lockfile already pins mcpg |
Per githubnext/agentics#339 the DIFC proxy
/metablock was fixed ingh-aw-mcpg v0.3.12.This PR:
gh-aw-actions/setuptov0.75.4(recompiled withgh aw v0.75.4 compile --strict), which now pinsghcr.io/github/gh-aw-mcpg:v0.3.18as the DIFC proxy image..github/workflows/issue-arborist.mdto the upstreamgithubnext/agenticssource (gh issue listinstead of the curl + REST API workaround from fix(aw): Issue Arborist - re-apply curl REST API workaround for DIFC proxy /meta block #8507 / fix(aw): Issue Arborist - replace gh CLI with curl REST API to bypass DIFC proxy /meta block #8185)..github/workflows/issue-arborist.lock.ymland.github/aw/actions-lock.jsonaccordingly.The point of this PR is to validate the GH team's fix once the next scheduled run executes. If the
Fetch issues datastep succeeds, we can keep the upstream version; otherwise we'll restore the curl workaround.