Skip to content

Update stacklok/toolhive to v0.24.0#788

Merged
rdimitrov merged 8 commits intomainfrom
renovate/stacklok-toolhive-0.x
Apr 22, 2026
Merged

Update stacklok/toolhive to v0.24.0#788
rdimitrov merged 8 commits intomainfrom
renovate/stacklok-toolhive-0.x

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Apr 22, 2026 •
edited by github-actions Bot
Loading

This PR contains the following updates:

Package Update Change
stacklok/toolhive minor v0.23.1 → v0.24.0

After this PR opens, .github/workflows/upstream-release-docs.yml adds source-verified content edits for the new release. For stacklok/toolhive, the same workflow also syncs reference assets (CLI help, Swagger) and regenerates the CRD MDX pages.

Release Notes

stacklok/toolhive (stacklok/toolhive)

v0.24.0

Compare Source

What's Changed

Full Changelog: stacklok/toolhive@v0.23.1...v0.24.0

Configuration

📅 Schedule: (in timezone America/New_York)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Never, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Docs update for toolhive v0.24.0

At a glance
Upstream stacklok/toolhive v0.23.1 → v0.24.0
Hand-written changes 2 commit(s)
Reference assets refreshed (separate commit)
Gaps 0
Release contributors 4 auto-assigned (see sidebar)
Action required Spot-check skill-authored prose for accuracy

Summary of changes

Summary of changes

  • Added docs/toolhive/guides-vmcp/local-cli.mdx: a new guide for running vMCP
    locally via thv vmcp serve/validate/init, covering quick mode vs.
    config-file mode, optimizer tiers 0-3, --enable-audit, full flag tables, and
    a comparison to the Kubernetes VirtualMCPServer CRD path.
  • Updated docs/toolhive/guides-vmcp/intro.mdx and
    docs/toolhive/guides-vmcp/index.mdx so the entry points surface the new
    local CLI option alongside the existing Kubernetes deployment.
  • Updated docs/toolhive/concepts/vmcp.mdx so the concept page acknowledges
    both delivery modes (operator CRD and CLI) and links to the local guide.
  • Updated sidebars.ts to list the new local-cli guide under Virtual MCP
    Server    , immediately after the Kubernetes quickstart.

Run cost

Session Turns Cost (USD)
Generation 62 $3.8214
Editorial review 33 $2.0296
Total 95 $5.8510
How this PR was built

Two Claude Opus sessions run per release: a generation pass
(upstream-release-docs skill, 6 phases) followed by a fresh-
context editorial pass (docs-review). Prettier/ESLint
auto-fixes are applied after.

Auto-synced paths — do not hand-edit these in review:

  • docs/toolhive/reference/cli/
  • docs/toolhive/reference/crds/
  • static/api-specs/

If a "Gaps needing human context" section is present above,
each entry includes a paste-ready Helper prompt for local
Claude a reviewer can use to resolve the gap.

@renovate renovate Bot added the dependencies Pull requests that update a dependency file label Apr 22, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment Apr 22, 2026 5:21pm

Request Review

@claude

This comment was marked as outdated.

Sign in to view
@rdimitrov rdimitrov mentioned this pull request Apr 22, 2026
Merged
rdimitrov added a commit that referenced this pull request Apr 22, 2026
@rdimitrov @claude
Bump skill turn caps after hitting review limit on v0.24.0 (#789)
41520bc 
PR #788 (Update stacklok/toolhive to v0.24.0) failed the run
when skill_review hit the 30-turn cap mid-edit. Details from run
24786157448:

- skill_gen: 67 turns / $4.72 (under 500 cap) -- succeeded,
  committed `42267da Document local vMCP CLI mode`
- skill_review: 31 turns / $1.86 -- exited with subtype
  `error_max_turns`, cascading workflow to `failure`, which
  skipped autofix and PR body augmentation

My initial cap of 30 for review was sized against silent/no-
changes releases (4-6 turns baseline). For a real multi-file
content review the editorial pass walks each edited file and
makes tightening edits; ~30-100 turns is legitimate working
range.

Changes:
  - skill_review: 30 -> 200 (2x-6x headroom over working range)
  - skill_gen:    500 -> 1000 (defensive doubling; 500 was
    never hit in production, but 1000 keeps us well clear of
    the 397-turn v3-test anomaly)

Hitting either cap still produces a loud failure; raise
deliberately if a release genuinely needs more.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rdimitrov rdimitrov mentioned this pull request Apr 22, 2026
Merged
rdimitrov added a commit that referenced this pull request Apr 22, 2026
@rdimitrov @claude
Gitignore SUMMARY.md signal file (#790)
2b9a32b 
PR #788's skill run committed SUMMARY.md to the branch alongside
its content commit (42267da) because /SUMMARY.md isn't in
.gitignore -- `git add -A` during claude-code-action's auto-commit
swept it up. The workflow's signals step reads and removes
SUMMARY.md, but only from the working tree; the commit that added
it sticks around.

/GAPS.md and /NO_CHANGES.md are already gitignored for exactly
this reason; SUMMARY.md was added in #785 but I missed adding the
matching gitignore entry.

With this in place, claude-code-action's `git add -A` will skip
SUMMARY.md, the skill still writes it to disk, the signals step
still reads + deletes it -- no commit involvement at all, which
is how GAPS.md and NO_CHANGES.md already work.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@renovate @rdimitrov
Update stacklok/toolhive to v0.24.0
70ed1d8 
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@rdimitrov rdimitrov force-pushed the renovate/stacklok-toolhive-0.x branch from fa0212b to 70ed1d8 Compare April 22, 2026 15:41
@stacklok stacklok deleted a comment from github-actions Bot Apr 22, 2026
@stacklok stacklok deleted a comment from github-actions Bot Apr 22, 2026
@rdimitrov rdimitrov closed this Apr 22, 2026
@rdimitrov rdimitrov reopened this Apr 22, 2026
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 22, 2026 •
edited
Loading

Claude finished @rdimitrov's task in 5m 26s —— View job

Upstream release docs: toolhive v0.24.0

Done. All six phases completed; pushed as commit ffea8d5.

  • Phase 1: Discovery
  • Phase 2: Deep dive (PR context fetch)
  • Phase 3: Docs audit
  • Phase 4: Source-verified implementation
  • Phase 5: Validation (deferred to CI - local sandbox blocks npm)
  • Phase 6: SUMMARY.md written, committed, pushed

What shipped

The release's one documentation-impacting change was PR #5001 - thv vmcp serve/validate/init were un-hidden from thv --help and are now a supported user surface. The auto-generated CLI reference pages already landed via the reference-refresh step (commit 89629bf); this commit adds the conceptual/guide scaffolding around them.

  • Added docs/toolhive/guides-vmcp/local-cli.mdx - full guide for the local vMCP CLI mode: quick mode vs. config-file mode (init → validate → serve), optimizer tier table (0/1/2/3), --enable-audit, loopback-only quick-mode constraint, and a comparison to the VirtualMCPServer CRD path. All flags, defaults, and behavior verified against cmd/thv/app/vmcp.go and pkg/vmcp/cli/serve.go at tag v0.24.0.
  • Updated docs/toolhive/guides-vmcp/index.mdx and intro.mdx to point readers at the local CLI option alongside the existing Kubernetes content.
  • Updated docs/toolhive/concepts/vmcp.mdx so the opening paragraph acknowledges both forms (operator CRD and CLI) and links to the local guide.
  • Updated sidebars.ts to list the new guide under Virtual MCP Server, immediately after the Kubernetes quickstart.

Skipped

  • PR #5000 (OCI-annotation for local skill builds) - the existing thv skill builds docs already describe the feature as "OCI skill artifacts built locally with thv skill build", which matches the v0.24.0 behavior (the fix aligned the code to what the docs already said). No edit needed.
  • PR #4914 (MCPServer spec patch) - PR author explicitly flagged no user-facing change.
  • CI / infra / dependency / test-only PRs - no docs impact.

Notes

Local npm run build and npm run eslint are blocked by the sandbox; the existing PR CI will run these checks. No GAPS.md was written - every feature's rationale was derivable from the linked PR body, issue #4890, and the upstream docs/arch/vmcp-local.md arch doc that shipped in the same PR.
· Branch

@github-actions
Add local vMCP CLI guide for toolhive v0.24.0
ffea8d5 
Document thv vmcp serve/validate/init, which was unhidden in
upstream PR #5001. Add a new guide covering quick mode,
config-file mode, optimizer tiers, and a comparison to the
Kubernetes VirtualMCPServer path. Update vMCP intro, index,
and concept pages to surface the local CLI alongside the
operator-based deployment.
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 22, 2026 •
edited
Loading

Claude finished @rdimitrov's task in 3m 58s —— View job

Editorial review: toolhive v0.24.0 docs

Done. Pushed as commit 56a121b.

  • Identify files changed in the previous commit
  • Read each changed doc file (excluding reference/CLI, CRDs, signal files)
  • Run /docs-review editorial pass
  • Apply fixes per skill guidance
  • Re-verify factual concerns against the auto-generated CLI reference
  • Commit and push

Findings and fixes

Two primary issues, both in docs/toolhive/guides-vmcp/local-cli.mdx:

  1. Wrong cross-link target (line 29): The "For production and multi-tenant deployments, use the Kubernetes VirtualMCPServer resource instead" sentence linked to ../guides-k8s/quickstart.mdx (the general operator quickstart) rather than the vMCP-specific quickstart at ./quickstart.mdx. Fixed to link to ./quickstart.mdx.

  2. Imprecise "hybrid search" claim (line ~179): The intro to the Optimizer tiers table said "tiers 1 through 3 ... perform hybrid search against the aggregated tool set." The table itself shows Tier 1 is FTS5 keyword only - only Tiers 2 and 3 add semantic embeddings. Rewrote to: "tiers 1 through 3 ... search the aggregated tool set. Tier 1 uses FTS5 keyword search only; tiers 2 and 3 add semantic embeddings on top for hybrid search."

Not flagged

  • No em/en dashes in any new or modified content.
  • Front-matter descriptions are within length and front-loaded.
  • Flag tables reconcile against the auto-generated CLI reference for thv vmcp serve/init/validate.
  • "Next steps" → "Related information" ordering correct on all touched pages.
  • New local-cli.mdx has solid inbound coverage (intro, index, concepts, sidebar).
  • Sidebar placement (under Virtual MCP Server, after the Kubernetes quickstart) matches information-architecture guidance.

Third-person phrasing in concepts/vmcp.mdx ("Engineers manage...", "An engineering team...") predates this PR and was left alone per the scoped editorial remit.

· Branch ·

github-actions Bot and others added 2 commits April 22, 2026 15:52
@github-actions
Fix vMCP local CLI guide link and tier description
56a121b 
Point the "production" cross-link at the vMCP-specific quickstart
instead of the general operator quickstart. Also tighten the
optimizer-tier intro so Tier 1 (FTS5 keyword only) is not described
as hybrid search.

Co-authored-by: Radoslav Dimitrov <undefined@users.noreply.github.com>
@github-actions
Apply prettier and eslint fixups to skill output
4aaa63f
@stacklok stacklok deleted a comment from renovate Bot Apr 22, 2026
@stacklok stacklok deleted a comment from github-actions Bot Apr 22, 2026
@stacklok stacklok deleted a comment from claude Bot Apr 22, 2026
@danbarr
Copy link
Copy Markdown
Collaborator

danbarr commented Apr 22, 2026

@claude the local mode of vMCP is a secondary workflow for individual users and local testing; Kubernetes based deployment should be considered primary. The current positioning above k8s in the intro isn't ideal, and the new guide's position in the sidebar right after the quickstart feels like it interrupts the primary user journey.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 22, 2026 •
edited
Loading

Claude finished @danbarr's task in 1m 40s —— View job

Repositioning local vMCP CLI as secondary workflow

  • Gather context - read intro.mdx, index.mdx, sidebars.ts
  • Reorder index.mdx "Where to start" to present K8s quickstart before local CLI
  • Reframe intro.mdx to remove "workstation first" implication
  • Move local-cli sidebar entry from post-quickstart to end of vMCP section
  • Commit and push

Changes made

docs/toolhive/guides-vmcp/index.mdx — "Where to start" previously listed local CLI before the K8s quickstart, giving the impression it was the natural first stop. Reordered so the K8s deployment path comes second (after evaluating), and local testing is last.

docs/toolhive/guides-vmcp/intro.mdx — The phrase "switch to the local CLI guide if you want to try vMCP on your workstation first" implied local CLI is a stepping stone before Kubernetes. Replaced with "For individual development or local testing without a cluster, see..." which frames it as a secondary/optional path.

sidebars.ts — Moved local-cli from position 3 (immediately after quickstart) to the end of the vMCP section, after scaling-and-performance. The primary K8s journey (quickstart → configuration → backends → auth → …) now flows uninterrupted.
· Branch

@github-actions @danbarr
Reposition local vMCP CLI as secondary workflow
753dd0a 
Move local-cli sidebar entry to end of vMCP section; reorder
index.mdx "Where to start" so K8s quickstart comes before local
CLI; reframe intro.mdx to remove implication that local is a
first step toward K8s.

Co-authored-by: Dan Barr <danbarr@users.noreply.github.com>
@danbarr
Fix formatting
017a53e 
Signed-off-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
@danbarr danbarr added autogen-docs Pull requests generated by automated release workflows and removed dependencies Pull requests that update a dependency file labels Apr 22, 2026
@rdimitrov rdimitrov merged commit f0bd4b5 into main Apr 22, 2026
4 checks passed
@rdimitrov rdimitrov deleted the renovate/stacklok-toolhive-0.x branch April 22, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jhrozek jhrozek jhrozek approved these changes

@ChrisJBurns ChrisJBurns Awaiting requested review from ChrisJBurns

@samuv samuv Awaiting requested review from samuv

@yrobla yrobla Awaiting requested review from yrobla

Assignees

No one assigned

Labels

autogen-docs Pull requests generated by automated release workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@danbarr @jhrozek @rdimitrov