Update GitHub MCP server default to v1.7.0 - #47923
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot update container shas and recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in 3f680b1. Updated the container SHA pin in |
|
Great work! 🚀 This PR looks ready for review. This version bump to GitHub MCP v1.7.0 brings important improvements:
The PR is well-described, focused on a single change (version bump + lock file regeneration), and follows the project's recommended agentic development process (Copilot agent-created). The 261 lock files regenerated with double-recompile are expected for container SHA pin refresh. Ready to move forward! ✅ Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "patchdiff.githubusercontent.com"See Network Configuration for more information.
|
🤖 PR Triage
Rationale: Bumps GitHub MCP server default from v1.6.0 → v1.7.0, regenerating 260 lock files with updated SHA pins. Large-footprint chore but mechanically generated. Still DRAFT — needs undraft and CI run before merging. Next step: Undraft and run CI to validate pin regeneration.
|
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. Test Quality Sentinel skipped. |
|
✅ Design Decision Gate 🏗️ completed the design decision gate check. No ADR enforcement needed: PR #47923 does not have the 'implementation' label and has only 11 new lines of code in business logic directories (threshold: >100). |
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅ |
|
✅ PR Code Quality Reviewer completed the code quality review. |
There was a problem hiding this comment.
Pull request overview
Updates the default GitHub MCP server from v1.6.0 to v1.7.0 and refreshes generated container pins.
Changes:
- Bumps the default server version.
- Updates action-pin registries and generated workflow locks.
- Leaves 13 Wasm golden fixtures stale, causing golden-test failures.
Show a summary per file
| File | Description |
|---|---|
pkg/constants/version_constants.go |
Bumps the default version. |
pkg/workflow/data/action_pins.json |
Updates the image digest. |
pkg/actionpins/data/action_pins.json |
Updates the image digest. |
.github/aw/actions-lock.json |
Updates the shared image pin. |
.github/workflows/smoke-github-claude.lock.yml |
Refreshes generated pin. |
.github/workflows/smoke-copilot-small.lock.yml |
Refreshes generated pin/config. |
.github/workflows/smoke-copilot-mai.lock.yml |
Refreshes generated pin. |
.github/workflows/smoke-claude-on-copilot.lock.yml |
Refreshes generated pin. |
.github/workflows/smoke-antigravity.lock.yml |
Refreshes generated pin. |
.github/workflows/pr-description-caveman.lock.yml |
Refreshes generated pin. |
.github/workflows/outcome-collector.lock.yml |
Refreshes generated pin. |
.github/workflows/firewall.lock.yml |
Refreshes generated pin/config. |
.github/workflows/example-permissions-warning.lock.yml |
Refreshes generated pin/config. |
.github/workflows/example-failure-category-filter.lock.yml |
Refreshes generated pin/config. |
.github/workflows/eslint-miner.lock.yml |
Refreshes generated pin. |
.github/workflows/designer-drift-audit.lock.yml |
Refreshes generated pin. |
.github/workflows/daily-windows-terminal-integration-builder.lock.yml |
Refreshes generated pin. |
.github/workflows/daily-team-status.lock.yml |
Refreshes generated pin/config. |
.github/workflows/daily-safeoutputs-git-simulator.lock.yml |
Refreshes generated pin. |
.github/workflows/daily-max-ai-credits-test.lock.yml |
Refreshes generated pin/config. |
.github/workflows/daily-formal-spec-verifier.lock.yml |
Refreshes generated pin. |
.github/workflows/daily-credit-limit-test.lock.yml |
Refreshes generated pin/config. |
.github/workflows/daily-byok-ollama-test.lock.yml |
Refreshes generated pin/config. |
.github/workflows/daily-assign-issue-to-user.lock.yml |
Refreshes generated pin. |
.github/workflows/copilot-cli-deep-research.lock.yml |
Refreshes generated pin. |
.github/workflows/copilot-centralization-optimizer.lock.yml |
Refreshes generated pin/config. |
.github/workflows/agentic-token-optimizer.lock.yml |
Refreshes generated pin. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 112/260 changed files
- Comments generated: 1
- Review effort level: Medium
| @@ -60,7 +60,7 @@ const DefaultPiVersion Version = "0.80.10" | |||
| const DefaultOpenCodeVersion Version = "1.2.14" | |||
|
|
|||
| // DefaultGitHubMCPServerVersion is the default version of the GitHub MCP server Docker image | |||
| const DefaultGitHubMCPServerVersion Version = "v1.6.0" | |||
| const DefaultGitHubMCPServerVersion Version = "v1.7.0" | |||
There was a problem hiding this comment.
The changes are a consistent, digest-pinned version bump of ghcr.io/github/github-mcp-server from v1.6.0 to v1.7.0. All references across both changed files are updated in lockstep with the new digest. No issues found.
🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · sonnet46 · 20.8 AIC · ⌖ 5.2 AIC · ⊞ 5K
…n changes Add normalization for DefaultGitHubMCPServerVersion in both the Go wasm_golden_test.go and the JS test-wasm-golden.mjs so that future bumps to DefaultGitHubMCPServerVersion don't break golden tests. The default github-mcp-server image version is now replaced with GH_MCP_VERSION placeholder during golden comparison, mirroring the existing treatment of DefaultFirewallVersion and DefaultMCPGatewayVersion. Regenerate all 13 affected golden fixtures to use the new placeholder. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in the latest commit. Instead of just updating the golden files for v1.7.0, I've made the tests resilient to future
Future version bumps to |
|
@copilot resolve the merge conflicts on this branch. |
Both CI failures ( |
…-mcp-server # Conflicts: # .github/workflows/daily-multi-device-docs-tester.lock.yml Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
… after conflict resolution Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done. Merged |
|
/smoke-copilot |
|
📰 BREAKING: Smoke Copilot is now investigating this issue comment. Sources say the story is developing... |
|
Smoke: Update GitHub MCP server default to v1.7.0
|
Comment MemoryNote This comment is managed by comment memory.It stores persistent context for this thread in the code block at the top of this comment.
|
There was a problem hiding this comment.
Smoke review. Test marks left on diff.
📰 BREAKING: Report filed by Smoke Copilot · gpt54 · 75.7 AIC · ⌖ 8.29 AIC · ⊞ 21K
Comment /smoke-copilot to run again
Add label smoke to run again
…ieldsAreSorted Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Fixed in the latest commit. |
|
🎉 This pull request is included in a new release. Release: |
Bumps
DefaultGitHubMCPServerVersionfromv1.6.0→v1.7.0and regenerates all lock files with updated container SHA pins.What's new in v1.7.0
go-sdk v1.7.0-pre.3Changes
pkg/constants/version_constants.go: bumpDefaultGitHubMCPServerVersion.lock.ymlfiles regenerated (double recompile to refresh SHA pins)