Skip to content

Fix MCP fallback tarball discovery - #203

Merged
Keesan12 merged 1 commit into
mainfrom
fix/release-pack-fallback
Aug 13, 2026
Merged

Fix MCP fallback tarball discovery#203
Keesan12 merged 1 commit into
mainfrom
fix/release-pack-fallback

Conversation

@Keesan12

@Keesan12 Keesan12 commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • discover the locally packed MCP fallback tarball from the output directory
  • remove the final dependency on npm JSON filename metadata

Verification

  • exact smoke:published:pack gate: exit 0
  • packaged MCP install/launch: passed
  • tool surface: 22 tools
  • git diff --check: exit 0

Release blocker

Trusted release run 31673182115 stopped before pack/publish at this fallback smoke. No 0.5.0 package or release was published.

Summary by CodeRabbit

  • Bug Fixes
    • Improved local package smoke testing by reliably locating the generated package archive.
    • Removed fragile manual parsing and filename validation during fallback packaging.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e23a2ce9-2d00-401d-a266-69c366d63afb

📥 Commits

Reviewing files that changed from the base of the PR and between a9735b7 and 8d6c39f.

📒 Files selected for processing (1)
  • packages/mcp/scripts/smoke-published-package.mjs

📝 Walkthrough

Walkthrough

The smoke script now imports findSingleTarball, runs local npm pack without JSON output, and uses the shared helper to locate the generated tarball.

Changes

Local tarball resolution

Layer / File(s) Summary
Pack and resolve the fallback tarball
packages/mcp/scripts/smoke-published-package.mjs
The script replaces npm pack --json parsing and custom filename validation with findSingleTarball(tempPackDir).

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: ⚪ Minimal · up to 8d6c3

This localized change updates fallback tarball discovery, and the relevant smoke, installation, launch, and packaging checks passed. No actionable merge-blocking risk remains.

Possibly related PRs

Poem

A rabbit packs a tarball neat,
No JSON trail beneath its feet.
The shared guard finds the file,
The smoke test runs in style.
Hop, hop—clean artifacts meet!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing MCP fallback tarball discovery by replacing manual JSON parsing with a dedicated function.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/release-pack-fallback

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Keesan12
Keesan12 merged commit 514f742 into main Aug 13, 2026
5 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