Skip to content

Publish versioned MCPB release assets - #846

Merged
thisrohangupta merged 1 commit into
mainfrom
chore/publish-mcpb-release-assets
Aug 21, 2026
Merged

Publish versioned MCPB release assets#846
thisrohangupta merged 1 commit into
mainfrom
chore/publish-mcpb-release-assets

Conversation

@thisrohangupta

Copy link
Copy Markdown
Collaborator

Summary

  • publish a versioned cross-platform MCPB asset after each existing v*.. release
  • support exact-tag manual backfills without rerunning npm publication
  • package through the pinned official MCPB CLI with a relocatable server path and 512x512 icon

Verification

  • exact-tag v3.2.20 bundle validated and packed at 127 MB
  • unpacked bundle completed an MCP stdio initialize handshake as server version 3.2.20
  • pnpm test: 134 files, 3093 tests
  • pnpm standards:check: 77 tests
  • pnpm typecheck
  • pnpm docs:check
  • pnpm npm-shrinkwrap:check

@thisrohangupta
thisrohangupta marked this pull request as ready for review August 21, 2026 02:14

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automation check — no inline review comments to address

Triggered on ready_for_review for PR #846 (chore/publish-mcpb-release-assets).

Checked GitHub review data:

  • 0 pull review comments
  • 0 review threads
  • 0 submitted reviews

No inline reviewer feedback was present in the trigger payload or on the PR, so no code changes were made. This run will pick up inline comments once reviewers leave them on the diff.

Open in Web View Automation 

Sent by Cursor Automation: Autofix PR review comments

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR Risk Assessment

Risk level: Low

Decision: Approved — no additional reviewers required.

Evidence (from diff, not PR description)

Area Assessment
Runtime server (src/) No changes
Auth / security model No changes
Production logic No changes
Scope Release/packaging pipeline only

Changes reviewed

  • .github/workflows/release.yml — Adds an mcpb job (tag-triggered + workflow_dispatch backfill) that builds and uploads a versioned MCPB asset to GitHub Releases. Existing release job is gated to push events only.
  • scripts/prepare-mcpb.js — Refactors internal packaging: relocatable server/ layout, production package.json + shrinkwrap handling, pinned @anthropic-ai/mcpb@2.1.2 validate/pack/info.
  • manifest.json / mcp-directory/manifest.json — Entry point updated to server/index.js (bundle layout change only).
  • icon.png — Updated to 512×512 (distribution metadata).
  • Docs + tests — README updates, release-metadata.test.ts and release-workflow.test.ts cover manifest shape, icon size, and workflow structure.

Why Low (not Medium)

  • Isolated to internal release tooling and distribution artifacts; no shared runtime codepaths.
  • Additive CI with safeguards (semver tag validation, package-version match, MCPB CLI validation).
  • Comprehensive structural tests added.
  • No CODEOWNERS file; no domain-ownership ambiguity.

Reviewers

None assigned — Low risk PRs do not require additional reviewers per policy.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@thisrohangupta
thisrohangupta merged commit fd2890e into main Aug 21, 2026
12 checks passed
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.

1 participant