Conversation
Cover the new local vMCP CLI mode, ToolHive Desktop UI v0.32.2 Skills installation flow (install into Claude Code/Cursor, connect a custom registry, package your own skills), and Registry Server version metadata in ListRegistryEntries responses. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@claude perform an editorial review of this PR; keep in mind it's an update blog post not a docs page. |
|
Claude finished @danbarr's task in 1m 17s —— View job Editorial review
Overall this is a solid, well-structured update post. The intro paragraph does a good job summarizing all three topics, the {/* truncate */} is placed correctly, and the post avoids em dashes and smart quotes. A few things worth addressing before merge: Issues to fix1. UI element not bolded —
Per the style guide, UI elements should be in bold. Should read: Skills tab. 2. Title/content order mismatch The title is "Discover and install skills in your AI client, and run vMCP locally" — skills first, vMCP second — but the post covers vMCP first and skills second. Readers scanning the title will expect to hit skills content first and may feel disoriented. Either reorder the title to "Run vMCP locally and install skills into your AI client" to match the post body, or swap the section order. The vMCP-first ordering makes sense given it's the bigger structural change. 3. Awkward phrasing — "turning catalog pages into an N+1 query pattern" (line 58)
"Catalog pages" is an odd framing — it's more natural to say something like "...turning every catalog view into an N+1 query pattern." The N+1 terminology is fine for the technical audience, but "catalog pages" reads as a mixed metaphor (pages is a UI concept, N+1 is an API concept). Suggestions (optional)4. Tone shift in the third section The first two sections speak directly to what users can do ("you can take the MCP servers you already run," "Pick a skill and install it"). The registry version metadata section shifts to describing system behavior ("UIs and registry clients can render staleness signals...") without framing what the reader can now do as a result. Consider adding a lead sentence from the user's perspective, e.g.: "If you build on top of the Registry Server, or use a client that does, you'll notice a speed improvement and richer catalog views." 5. Brief definition of "skills" would help new readers "Skills" is mentioned six times in the second section but never defined in the post itself. The link to 6. Consider "we" voice per blog post conventions The style guide explicitly permits first-person voice in update posts. The current post uses impersonal framing throughout ("Virtual MCP Server is now supported locally," "ToolHive Desktop UI v0.32.2 makes the Skills tab..."). Switching some lines to "we've" or "we're" would bring it closer to the voice of a team-written update and differentiate it more clearly from docs. This is low-priority but worth a pass if the post feels too formal. |
There was a problem hiding this comment.
Pull request overview
Adds the weekly ToolHive updates post for 2026-04-27, summarizing new local vMCP CLI support, Desktop UI skill installation, and Registry Server list responses including version metadata.
Changes:
- Adds a new weekly update MDX post for Apr 27 covering vMCP local mode, skills installation in Desktop UI v0.32.2, and registry version metadata in list responses.
- Links readers to the relevant ToolHive guides and release pages for follow-up details.
| This week, Virtual MCP Server (vMCP) is now supported locally, the ToolHive | ||
| Desktop UI installs skills directly into the AI clients you already use, and the | ||
| Registry Server surfaces version information in list views. |
There was a problem hiding this comment.
The lead sentence is a comma splice (multiple independent clauses joined by commas), which reads as a run-on. Consider splitting into two sentences or using a semicolon/em-dash after “locally” to improve grammar and readability.
| This week, Virtual MCP Server (vMCP) is now supported locally, the ToolHive | |
| Desktop UI installs skills directly into the AI clients you already use, and the | |
| Registry Server surfaces version information in list views. | |
| This week, Virtual MCP Server (vMCP) is now supported locally, and the ToolHive | |
| Desktop UI installs skills directly into the AI clients you already use. The | |
| Registry Server also surfaces version information in list views. |
| response. This applies to both MCP server and skill entries served through the | ||
| registry-scoped browse endpoints. |
There was a problem hiding this comment.
“This applies to both MCP server and skill entries …” is slightly ambiguous/uneven phrasing (singular vs plural) and can read like it applies to an MCP server plus multiple skills. Consider rewording to “both MCP server entries and skill entries” (or similar) for clarity.
| response. This applies to both MCP server and skill entries served through the | |
| registry-scoped browse endpoints. | |
| response. This applies to both MCP server entries and skill entries served | |
| through the registry-scoped browse endpoints. |
Reference the dedicated Skills section, Registry tab, Local Builds, and Installed tab pages from PR #813 so the weekly post points readers at the new desktop guides instead of generic concept and registry links. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Lead with the Desktop UI skills story (the headline product update), followed by local vMCP and registry version metadata. Bump bundled Desktop UI version to v0.33.0, rewrite the description to match the new ordering, and clean up duplicated wording in the skills bullets. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Split the comma-spliced lede into two sentences, add a one-line skills definition, bold the Skills tab names, lead the registry section with a user-facing benefit, replace "catalog pages" with "list view" framing, and disambiguate "MCP server and skill entries". Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Addressed in From @copilot-pull-request-reviewer:
From the @claude editorial review:
Heads-up on broken-link CI: The four new |
|
Claude finished @Derek2Tu's task in 1m 0s —— View job Follow-up review
All reported fixes check out. Here's the verification: From @copilot-pull-request-reviewer:
From the previous @claude editorial review:
No new issues found. The post reads cleanly and is ready to merge once #813 lands to resolve the broken-link check on the |
Description
Adds the 2026-04-27 weekly update post covering:
thv vmcpguide and frames who the local mode is for (evaluation, local dev against several MCP servers, lighter tool lists via the optimizer).ListRegistryEntriesnow embeds version metadata in list responses, replacing an N+1 fetch pattern.Drops items already covered in the 2026-04-20 post (claims endpoint, dropped legacy format, single managed source, DB password, claims required on publish) and skips API cleanup, fixes, and security/under-the-hood items per editorial scope for this week.
Type of change
Submitter checklist
Content and formatting
🤖 Generated with Claude Code