Skip to content

Add example-remote-server repository access#116

Merged
localden merged 1 commit into
modelcontextprotocol:mainfrom
ochafik:ochafik/add-example-remote-server-access
Jun 5, 2026
Merged

Add example-remote-server repository access#116
localden merged 1 commit into
modelcontextprotocol:mainfrom
ochafik:ochafik/add-example-remote-server-access

Conversation

@ochafik
Copy link
Copy Markdown
Contributor

@ochafik ochafik commented Jun 5, 2026

What & why

example-remote-server is missing from repoAccess.ts. Since the org's defaultRepositoryPermission is none, that means no team currently has write access to it — e.g. @ochafik (a mcp-apps-sdk member, admin on ext-apps) has push: false there, which blocks shipping the lazy-auth example server PR.

This adds the repo alongside its sibling example-remote-client, mirroring the team grants used for ext-apps (the apps surface these example servers belong to):

{
  repository: 'example-remote-server',
  teams: [
    { team: 'core-maintainers', permission: 'push' },
    { team: 'moderators', permission: 'maintain' },
    { team: 'mcp-apps-wg', permission: 'push' },
    { team: 'mcp-apps-sdk', permission: 'admin' },
  ],
}

Verification

  • prettier --check passes on the file.
  • All four referenced teams (core-maintainers, moderators, mcp-apps-wg, mcp-apps-sdk) are defined in roles.ts, so the validate-config.ts team-reference check passes.
  • example-remote-server appears exactly once (no duplicate); test-config.ts uniqueness checks are unaffected.

After merge

The Pulumi deploy workflow applies it to GitHub, restoring write access so the example-remote-server lazy-auth PR can be pushed to the upstream.

cc @localden — could you take a look / approve? (CODEOWNERS → @modelcontextprotocol/core-maintainers.)

example-remote-server was missing from the repository access config, so
with the org default repository permission set to 'none', no team had
write access to it. Add it alongside its sibling example-remote-client,
mirroring the team grants used for ext-apps (the apps surface these
example servers belong to): core-maintainers push, moderators maintain,
mcp-apps-wg push, mcp-apps-sdk admin.
@ochafik ochafik requested a review from a team as a code owner June 5, 2026 17:29
@localden localden merged commit 900bad9 into modelcontextprotocol:main Jun 5, 2026
2 checks passed
@ochafik
Copy link
Copy Markdown
Contributor Author

ochafik commented Jun 5, 2026

Updated: also granting push to the repo's main individual contributors who aren't covered by the listed teams — jerome3o-anthropic (primary author), maxisbey, and felixweinberger. Basil Hosmer (bhosmer-ant) is already covered via the moderators team, and ochafik/antonpk1 via mcp-apps-sdk.

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