Add missing MCP scenario patterns - #8342
Merged
Merged
Conversation
Keep the manually maintained catalogue in step with the eight scenario patterns published on primer.style.
🦋 Changeset detectedLatest commit: 416d312 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the @primer/mcp server’s pattern catalogue to match the scenario patterns now published on primer.style, and reduces duplication in tool metadata so the catalogue is less likely to drift.
Changes:
- Add Create, Customize, Delegate, Edit, and View to the MCP scenario patterns catalogue used by
list_patterns/get_pattern. - Add coverage to ensure the new scenario patterns are listed and that
get_patternfetches the correct published URLs. - Simplify
list_patterns/get_patterntool descriptions to avoid embedding a hardcoded example list.
Show a summary per file
| File | Description |
|---|---|
| packages/mcp/src/server.ts | Removes the duplicated “(copy, delete, filter, search)” examples from the pattern tool descriptions. |
| packages/mcp/src/server.test.ts | Adds tests for scenario pattern listing and correct URL resolution for newly added patterns. |
| packages/mcp/src/primer.ts | Extends the scenario pattern catalogue with Create/Customize/Delegate/Edit/View. |
| .changeset/update-mcp-scenario-patterns.md | Adds a minor changeset documenting the new scenario patterns. |
Review details
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 4/4 changed files
- Comments generated: 0
- Review effort level: Lite
joshblack
approved these changes
Aug 27, 2026
joshblack
added
the
integration-tests: skipped manually
joshblack
enabled auto-merge
August 27, 2026 14:03
TylerJDev
removed this pull request from the merge queue due to the queue being cleared
Aug 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This brings the MCP pattern catalogue back in line with primer.style. The catalogue is manually maintained and had fallen behind the docs that have now been published as part of https://github.com/github/core-ux/issues/2845.
It adds Create, Customize, Delegate, Edit and View to
list_patternsandget_pattern. I’ve also removed the duplicated pattern list from the tool descriptions, so there’s one less place for the catalogue to drift.Closes github/core-ux#3479
Changelog
New
Changed
Removed
Rollout strategy
No special rollout is needed.
Testing & Reviewing
npm test -w @primer/mcpnpm run type-check -w @primer/mcpnpm run build -w @primer/mcp200