Skip to content

Update stacklok/toolhive to v0.35.0#1031

Open
renovate[bot] wants to merge 4 commits into
mainfrom
renovate/stacklok-toolhive-0.x
Open

Update stacklok/toolhive to v0.35.0#1031
renovate[bot] wants to merge 4 commits into
mainfrom
renovate/stacklok-toolhive-0.x

Conversation

@renovate

@renovate renovate Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Update Change
stacklok/toolhive minor v0.34.0v0.35.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.35.0

Compare Source

What's Changed

Full Changelog: stacklok/toolhive@v0.34.0...v0.35.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.35.0

At a glance

Upstream stacklok/toolhive v0.34.0v0.35.0
Hand-written changes 2 commit(s)
Reference assets refreshed (separate commit)
Gaps 0
Release contributors 3 auto-assigned (see sidebar)
Action required Spot-check skill-authored prose for accuracy

Summary of changes

  • Added rate limit observability section to docs/toolhive/guides-k8s/rate-limiting.mdx documenting the three new OpenTelemetry metrics (toolhive_rate_limit_decisions, toolhive_rate_limit_redis_errors, toolhive_rate_limit_check_latency), attribute sets, counting semantics, and example PromQL queries
  • Added a "Rate limit metrics" subsection to docs/toolhive/guides-vmcp/telemetry-and-metrics.mdx covering the same metrics with a cross-link back to the K8s guide
  • Added a "Rate limit metrics" cross-link section to docs/toolhive/concepts/observability.mdx so the concept-level metrics catalog now points at the rate limit contract

Run cost

Session Turns Cost (USD)
Generation 85 $8.1113
Editorial review 33 $2.2597
Total 118 $10.3710
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:

  • static/api-specs/
  • docs/toolhive/reference/cli/ (toolhive only)
  • docs/toolhive/reference/crds/

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.

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate renovate Bot added the autogen-docs Pull requests generated by automated release workflows label Jul 13, 2026
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment Jul 13, 2026 10:00pm

Request Review

@github-actions github-actions Bot marked this pull request as draft July 13, 2026 21:44
@claude

claude Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Claude finished @renovate[bot]'s task in 9m 33s —— View job


Upstream release docs: stacklok/toolhive v0.35.0

  • Phase 1: Discovery - fetched release + categorized 7 PRs
  • Phase 2: Deep dive - verified each PR against source at v0.35.0
  • Phase 3: Audit existing docs - impact map built
  • Phase 4: Implementation - edits applied
  • Phase 5: Validation - source-verified, prettier + eslint clean, docs-review pass applied
  • Phase 6: SUMMARY.md written

Categorization

Files changed

  • docs/toolhive/guides-k8s/rate-limiting.mdx - new "Observe rate limit activity" section with metric table, counting semantics, and PromQL examples; new Next steps link
  • docs/toolhive/guides-vmcp/telemetry-and-metrics.mdx - new "Rate limit metrics" subsection cross-linking back to the K8s guide
  • docs/toolhive/concepts/observability.mdx - new "Rate limit metrics" section pointing at the K8s guide

Notes

  • npm run build fails in this sandbox because the mcp-metadata-plugin requires the thv CLI binary, which is not installed here. The actual CI build environment has it.
  • One docs-review "primary issue" (line 239-243 wording implying only MCPServer uses telemetryConfigRef) was skipped because the review's premise is incorrect - VirtualMCPServer also references an MCPTelemetryConfig via telemetryConfigRef (verified in docs/toolhive/guides-vmcp/telemetry-and-metrics.mdx lines 50-66). All other actionable review fixes applied.

Commit: 7f5c8b4 pushed to this branch.

Adds the three new OpenTelemetry metrics (decisions, Redis errors, and
Redis Lua check latency) introduced in toolhive v0.35.0. Covers attribute
sets, counting semantics, and example PromQL on the rate limiting guide,
with cross-references from the vMCP telemetry guide and the observability
concept page.

Co-authored-by: Unknown <Unknown@users.noreply.github.com>
@claude

claude Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Claude finished @renovate[bot]'s task in 3m 54s —— View job


Editorial review pass: stacklok/toolhive v0.35.0

  • Identify files touched by the release-docs commit
  • Read the three changed docs pages
  • Verify claims against upstream source (pkg/ratelimit/observability.go + upstream docs/observability.md)
  • Run /docs-review across them
  • Apply actionable fixes
  • Format + lint edits
  • Commit and push

Fixes applied in b53136f

  • docs/toolhive/concepts/observability.mdx line 233 — reword "per-bucket decision, Redis error, and Redis Lua check latency metrics" so the "per-bucket" modifier doesn't spuriously attach to all three metrics (only the decisions counter is per-bucket).
  • docs/toolhive/guides-k8s/rate-limiting.mdx lines 246–248 — split the Prometheus-suffix sentence into two so the _seconds unit suffix and the standard histogram series suffixes attach unambiguously to the histogram (matches upstream docs/observability.md's phrasing).
  • docs/toolhive/guides-k8s/rate-limiting.mdx lines 271–273 — swap loose "periods where limits stopped applying" for the sharper "fail-open periods" already used on the concept page.
  • docs/toolhive/guides-k8s/rate-limiting.mdx next-steps entry — replace the trailing "for the metrics above" with "collection and export for these metrics" so it reads correctly wherever the reader lands.

Skipped

  • Restating the discovery-method list in the observability section duplicates "How rate limiting works" but keeping the section self-contained is defensible.
  • The (none) cell for the histogram's additional-attributes column reads slightly oddly but communicates useful info.
  • Info architecture (rate-limiting under guides-k8s/, cross-linked from guides-vmcp/) is correct — MCPServer and VirtualMCPServer are K8s-only.
  • No em/en dashes, first-person, or word-list violations in the new content.

npm run prettier:fix and npm run eslint:fix both clean.
· Branch

@github-actions github-actions Bot requested review from ChrisJBurns and JAORMX July 13, 2026 21:59
@github-actions github-actions Bot marked this pull request as ready for review July 13, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autogen-docs Pull requests generated by automated release workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants