[try 2] Update docs for registry server v0.6.3#605
Conversation
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
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
maxMetaSizedefault and documents a newregistry-titleannotation.
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>
ReviewMeaningfully 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 fixAPI path inconsistency in Line 39 (correctly) uses
Should be What's improved vs. prior rounds
🤖 Generated with Claude Code |
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
|
@danbarr - addressed the comment above 👍 |
Description
Type of change
Related issues/PRs
Screenshots
Submitter checklist
Content and formatting
Navigation
sidebars.ts) updated for added, deleted, reordered, or renamed filesvercel.jsonfor moved, renamed, or deleted pages (i.e., if the URL slug changed)Reviewer checklist
Content