Repository navigation
Bump MCPG to v0.4.37 - #71219
Bump MCPG to v0.4.37#71219
Conversation
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
|
Thanks
|
|
✅ PR Code Quality Reviewer completed the code quality review.
|
|
✅ Test Quality Sentinel completed test quality analysis. Test Quality SentinelOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅
|
|
✅ Ponytail Reviewer completed successfully! Ponytail ReviewerOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
|
✅ Design Decision Gate 🏗️ completed the design decision gate check. See the comment below for the result and any generated ADR draft. Design Decision Gate 🏗️Outcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
There was a problem hiding this comment.
🔵 Needs a closer look
The supplied diffs cover only part of the reported 346-lock regeneration, and the published gateway digest remains independently unverified.
No open issues 🎉
What changed in this PR
Upgrades gh-aw’s default MCP gateway to v0.4.37 for #71217, aligning compiler defaults, image pins, generated workflows, and regression assertions.
Changes:
- Updates the canonical gateway version and synchronized container digest data.
- Propagates the new image pin through generated workflow locks.
- Refreshes version/digest assertions and the daily PR review prompt’s gateway reference without changing its instructions.
| File | Description |
|---|---|
scripts/test_daily_pr_review_integrity.py |
Updates the expected prompt version. |
pkg/workflow/data/action_pins.json |
Refreshes embedded gateway pin data. |
pkg/constants/version_constants.go |
Sets the default gateway to v0.4.37. |
pkg/constants/version_constants_test.go |
Updates the default-version assertion. |
pkg/actionpins/data/action_pins.json |
Refreshes embedded gateway pin data. |
pkg/actionpins/actionpins_internal_test.go |
Updates the expected gateway digest. |
.github/workflows/smoke-pi-routed.lock.yml |
Updates the routed Pi gateway pin. |
.github/workflows/smoke-pi-auto.lock.yml |
Updates the Pi auto gateway pin. |
.github/workflows/smoke-github-codex.lock.yml |
Updates the GitHub Codex gateway pin. |
.github/workflows/smoke-copilot-sdk-routed.lock.yml |
Updates the routed SDK gateway pin. |
.github/workflows/smoke-copilot-routed-steering.lock.yml |
Updates the steering smoke-test gateway pin. |
.github/workflows/smoke-codex-bare-auto.lock.yml |
Updates the bare Codex gateway pin. |
.github/workflows/smoke-codex-auto.lock.yml |
Updates the Codex auto gateway pin. |
.github/workflows/smoke-claude-dynamic.lock.yml |
Updates the dynamic Claude gateway pin. |
.github/workflows/smoke-claude-copilot.lock.yml |
Updates the Claude Copilot gateway pin. |
.github/workflows/smoke-claude-copilot-auto.lock.yml |
Updates the Claude Copilot auto gateway pin. |
.github/workflows/smoke-claude-auto.lock.yml |
Updates the Claude auto gateway pin. |
.github/workflows/smoke-azure-pi.lock.yml |
Updates the Azure Pi gateway pin. |
.github/workflows/smoke-azure-codex.lock.yml |
Updates the Azure Codex gateway pin. |
.github/workflows/schema-feature-coverage.lock.yml |
Updates the coverage checker’s gateway pin. |
.github/workflows/github-remote-mcp-auth-test.lock.yml |
Updates the authentication test’s gateway pin. |
.github/workflows/dev.lock.yml |
Updates the development smoke-test gateway pin. |
.github/workflows/daily-pr-review-cursor.md |
Refreshes the gateway version reference. |
.github/workflows/daily-github-docs-seo-optimizer.lock.yml |
Updates the optimizer’s gateway pin. |
.github/workflows/copilot-pr-merged-report.lock.yml |
Updates the reporting workflow’s gateway pin. |
.github/workflows/codex-github-remote-mcp-test.lock.yml |
Updates the remote MCP test’s gateway pin. |
.github/aw/actions-lock.json |
Updates the repository gateway image pin. |
Review effort: Max
There was a problem hiding this comment.
Skills-Based Review 🧠
Applied /codebase-design and /tdd (via pr-triage fallback classification: mechanical dependency bump). No actionable issues found.
📋 Verification notes
- Version bump
v0.4.36→v0.4.37is applied consistently across all canonical sources:pkg/constants/version_constants.go,pkg/actionpins/data/action_pins.json,pkg/workflow/data/action_pins.json, and.github/aw/actions-lock.json. - New digest
sha256:9c7f242f...matches identically across all 346 regenerated.lock.ymlworkflow files — no drift detected. - Accompanying tests (
version_constants_test.go,actionpins_internal_test.go,test_daily_pr_review_integrity.py) and thedaily-pr-review-cursor.mdprompt reference were updated in lockstep with the version bump. - Pre-existing doc references to "v0.4.36 or later" as a minimum-version requirement (not a default-version claim) remain semantically correct and don't need updating.
No deepening opportunities, missing abstractions, or test gaps identified for this change shape — it is a clean, low-risk, fully-synchronized dependency bump.
🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · copilot · sonnet50 · ~30 AIC · ⌖ ~15 AIC · ⊞ 10.2K
Comment /matt to run again
There was a problem hiding this comment.
Review summary
No actionable blocking issues found in the changed lines. The MCP Gateway version, digest pins, embedded test expectation, and integrity-policy documentation are updated consistently. Static review only; tests were not run.
🔎 Code quality review by PR Code Quality Reviewer · copilot · gpt50 · ~3 AIC · ⌖ ~1 AIC · ⊞ 1.6K
Comment /review to run again
|
🎉 This pull request is included in a new release. Release: |
Upgrade the default MCP gateway from v0.4.36 to v0.4.37 with the verified multi-platform image digest.
Validation
Focused tests, linting, lock synchronization, and secret scanning passed. Fallback review found no issues; CodeQL timed out.