Skip to content

[try 2] Update docs for registry server v0.6.3#605

Open
rdimitrov wants to merge 5 commits intomainfrom
registry-docs-part-2
Open

[try 2] Update docs for registry server v0.6.3#605
rdimitrov wants to merge 5 commits intomainfrom
registry-docs-part-2

Conversation

@rdimitrov
Copy link
Member

Description

Type of change

  • New documentation
  • Documentation update
  • Bug fix (typo, broken link, etc.)
  • Navigation/structure change

Related issues/PRs

Screenshots

Submitter checklist

Content and formatting

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Navigation

  • New pages include a frontmatter section with title and description at a minimum
  • Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files
  • Redirects added to vercel.json for moved, renamed, or deleted pages (i.e., if the URL slug changed)

Reviewer checklist

Content

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
Copilot AI review requested due to automatic review settings March 11, 2026 09:52
@vercel
Copy link

vercel bot commented Mar 11, 2026

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

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment Mar 16, 2026 3:07pm

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation for ToolHive Registry Server “skills” and updates existing Registry Server docs/config references to include skills and revised defaults.

Changes:

  • Adds new “Understanding skills” concept page and “Manage skills” how-to guide.
  • Updates Registry Server overview/index pages and sidebar navigation to include skills content.
  • Updates Registry Server configuration docs to reflect a larger maxMetaSize default and documents a new registry-title annotation.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
sidebars.ts Adds Skills pages to the Registry Server and Concepts sidebar sections.
docs/toolhive/guides-registry/skills.mdx New how-to guide for publishing/listing/retrieving/deleting skills via the extensions API.
docs/toolhive/guides-registry/intro.mdx Mentions skills in the Registry Server overview and adds links to the new guide.
docs/toolhive/guides-registry/index.mdx Updates section description to include skills.
docs/toolhive/guides-registry/database.mdx Updates maxMetaSize default value in examples and tables.
docs/toolhive/guides-registry/configuration.mdx Updates maxMetaSize, adds skills link under managed registry operations, and documents registry-title annotation.
docs/toolhive/concepts/skills.mdx New concept page defining skills, structure, and versioning.

Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
@danbarr
Copy link
Collaborator

danbarr commented Mar 16, 2026

Review

Meaningfully better than both #601 and #603. The primary blocker from those rounds — the missing consumer workflow explanation — is now addressed with the "Current status and what's next" section in both the concept page and guide. The explicit "planned for a future release" language is exactly what was needed.

One issue to fix

API path inconsistency in concepts/skills.mdx line 101

Line 39 (correctly) uses /{registryName}/v0.1/x/dev.toolhive/skills, but line 101 drops the prefix:

The skills API is available as an extension endpoint on the Registry server (/v0.1/x/dev.toolhive/skills).

Should be /{registryName}/v0.1/x/dev.toolhive/skills to match line 39 and the guide.

What's improved vs. prior rounds

🤖 Generated with Claude Code

Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
@rdimitrov
Copy link
Member Author

@danbarr - addressed the comment above 👍

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.

3 participants