Skip to content

Fix MCP registry OIDC audience#384

Merged
autholykos merged 1 commit intomainfrom
session/mcp-publisher-oidc-audience
May 1, 2026
Merged

Fix MCP registry OIDC audience#384
autholykos merged 1 commit intomainfrom
session/mcp-publisher-oidc-audience

Conversation

@autholykos
Copy link
Copy Markdown
Member

Summary

  • Upgrade the pinned MCP publisher used by release automation from v1.5.0 to v1.7.6.
  • Pass the production MCP registry URL explicitly during GitHub OIDC login so the requested audience matches the registry.

Backlog / Issue

  • Release follow-up for the failed v1.0.0-beta.9 MCP registry publish step.

Core Trust Surface

  • Does this PR touch a core write path or authoritative-state contract? No.

Validation

  • I ran make ci
  • I added or updated tests where needed

Validation run:

  • make fmt
  • git diff --check -- .github/workflows/release.yml
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/release.yml"); puts "valid yaml"'

Notes

  • Go tests were not run because this is release workflow-only.

Copilot AI review requested due to automatic review settings May 1, 2026 15:07
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the release automation workflow to ensure MCP registry publishing succeeds with GitHub OIDC by aligning the requested OIDC audience with the production registry.

Changes:

  • Bump pinned mcp-publisher used in the release workflow from v1.5.0 to v1.7.6 (with updated SHA256).
  • Pass the production MCP registry URL explicitly to mcp-publisher login github-oidc via --registry.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@autholykos autholykos merged commit 871232c into main May 1, 2026
9 checks passed
@autholykos autholykos deleted the session/mcp-publisher-oidc-audience branch May 1, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants