feat: split hevy-mcp into runtime-boundary packages - #715
Merged
Conversation
Contributor
|
Important Review skippedToo many files! This PR contains 157 files, which is 57 over the limit of 100. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (208)
📒 Files selected for processing (172)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Contributor
MCP tool token costMeasured with
Change from baseline
Per-tool changes
Per-tool breakdown
Per-tool counts encode each complete tool object independently. The total encodes the complete |
Contributor
Unit Test Results 1 files 52 suites 4s ⏱️ Results for commit 29399e3. ♻️ This comment has been updated with latest results. |
![charliecreates[bot]](https://avatars.githubusercontent.com/in/1140780?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
No actionable findings.
I reviewed the runtime-boundary package split across the generated client, core server, Node stdio/telemetry path, Worker HTTP/OAuth path, package manifests and export maps, release/Docker configuration, compatibility facades, and the associated CI coverage for this head. The completed checks provide coverage for the Node 24/26 builds and tests, boundary/export/release checks, Worker HTTP and preview behavior, stdio/contract baselines, packaging, Docker smoke, CodeQL, and security scans.
chrisdoc
marked this pull request as ready for review
July 22, 2026 17:33
Contributor
|
Tick the box to add this pull request to the merge queue (same as
|
Contributor
There was a problem hiding this comment.
No actionable findings.
Reviewed head b84707b72ca9c67efed7eed2480c6d53eb4da003 across the runtime-boundary package split, generated client relocation, core/Node/Worker entrypoints and export maps, Docker/release/changeset wiring, OAuth/security/privacy paths, and boundary/packaging checks. GitHub reports successful Node 24/26 build and test checks, Worker preview, Docker smoke, CodeQL, GitGuardian, and unit-test results (615 tests, 619 runs).
PR Summary by QodoSplit hevy-mcp into runtime-boundary workspaces (core / hevy-client / node / worker)
AI Description
Diagram
High-Level Assessment
Files changed (28)
|
Code Review by Qodo
Context used✅ Compliance rules (platform):
65 rules 1.
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
chrisdoc
changed the title
Fix/node prepack
![gitstream-cm[bot]](https://avatars.githubusercontent.com/in/230441?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
✨ PR Review
LGTM
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how
Merged
Bundle ReportChanges will increase total bundle size by 983 bytes (0.57%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: hevy-mcp-esmAssets Changed:
Files in
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #715 +/- ##
==========================================
- Coverage 94.52% 85.76% -8.77%
==========================================
Files 50 51 +1
Lines 2266 2452 +186
Branches 639 689 +50
==========================================
- Hits 2142 2103 -39
- Misses 51 184 +133
- Partials 73 165 +92 ☔ View full report in Codecov by Harness. |
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
1 participant
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.
Primary changes
@hevy-mcp/hevy-client@hevy-mcp/corehevy-mcp(public Node package)@hevy-mcp/workercreateNodeMcpServerandrunStdioServer.Reviewer walkthrough
packages/hevy-clientfor generated API client ownership and the package export surface.packages/corefor runtime-neutral MCP construction, tools, resources, prompts, and shared utilities.packages/nodefor the public Node package,createNodeMcpServer,runStdioServer, telemetry, and stdio wiring.packages/workerfor the Worker HTTP and OAuth entrypoints, then finish with the root workspace orchestration, compatibility facade, and release/package checks.Correctness and invariants
@hevy-mcp/hevy-client, while runtime-neutral MCP construction remains in@hevy-mcp/core.hevy-mcp; Worker HTTP/OAuth code stays in@hevy-mcp/worker.Testing and QA
npm run checknpm run check:workspacesnpm run check:boundariesnpm run check:exportsnpm run check:typesnpm run test:unitnpm run test:mcpnpm run test:contractnpm run test:stdionpm run test:worker-httpnpm run test:packnpm run worker:dry-run✨ PR Description
Purpose: Reorganize hevy-mcp monorepo into four runtime-boundary packages with explicit dependency graph and published Node distribution.
Main changes:
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how