Skip to content

Conversation

joshblack
Copy link
Member

Update our experimental mcp package to include a description and a correctly formatted "bin" field. It also synchronizes the version with what we have manually published to npm

@Copilot Copilot AI review requested due to automatic review settings August 6, 2025 21:28
@joshblack joshblack requested a review from a team as a code owner August 6, 2025 21:28
@joshblack joshblack added the skip changeset This change does not need a changelog label Aug 6, 2025
@joshblack joshblack requested a review from hectahertz August 6, 2025 21:28
Copy link

changeset-bot bot commented Aug 6, 2025

⚠️ No Changeset found

Latest commit: a37a76d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@Copilot 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 experimental MCP package configuration to align with the published npm package, including adding a description and properly formatting the binary entry point.

  • Added package description explaining its purpose as an MCP server for the Primer Design System
  • Updated version from 0.0.0 to 0.0.1 to match published package
  • Converted bin field from string format to object format with named binary

"type": "module",
"bin": "./dist/stdio.js",
"bin": {
"mcp": "dist/stdio.js"
Copy link
Preview

Copilot AI Aug 6, 2025

Choose a reason for hiding this comment

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

The bin path should include a leading './' for proper resolution. Change 'dist/stdio.js' to './dist/stdio.js' to match the previous format and ensure correct binary path resolution.

Suggested change
"mcp": "dist/stdio.js"
"mcp": "./dist/stdio.js"

Copilot uses AI. Check for mistakes.

@github-actions github-actions bot added the staff Author is a staff member label Aug 6, 2025
Copy link
Contributor

github-actions bot commented Aug 6, 2025

size-limit report 📦

Path Size
packages/react/dist/browser.esm.js 90.2 KB (0%)
packages/react/dist/browser.umd.js 90.38 KB (0%)

@github-actions github-actions bot requested a deployment to storybook-preview-6481 August 6, 2025 21:46 Abandoned
Copy link
Member

@jonrohan jonrohan left a comment

Choose a reason for hiding this comment

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

oh awesome 🤘🏻

@joshblack joshblack enabled auto-merge August 6, 2025 21:55
@github-actions github-actions bot temporarily deployed to storybook-preview-6481 August 6, 2025 21:56 Inactive
@joshblack joshblack added this pull request to the merge queue Aug 6, 2025
Merged via the queue into main with commit 28d3e46 Aug 6, 2025
43 checks passed
@joshblack joshblack deleted the chore/update-mcp-package branch August 6, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip changeset This change does not need a changelog staff Author is a staff member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants