Skip to content

feat(skills): add registry tab with skill browsing and detail page#1993

Merged
samuv merged 11 commits intomainfrom
skills-registry
Apr 15, 2026
Merged

feat(skills): add registry tab with skill browsing and detail page#1993
samuv merged 11 commits intomainfrom
skills-registry

Conversation

@samuv
Copy link
Copy Markdown
Collaborator

@samuv samuv commented Apr 14, 2026

The Skills page previously only showed installed skills and local builds. This adds a read-only Registry tab that lets users browse skills from the default registry without installing them first.

  • Add a Registry tab as the first tab on the Skills page, with a debounced server-side search input and a responsive card grid showing each skill's name, namespace, description (clamped to 3 lines with a tooltip on overflow), and an Install button that pre-fills the reference
  • Persist the active tab in the URL (?tab=registry|installed|builds) so navigating away and back restores the correct tab
  • Add a skill detail page at /skills/$namespace/$skillName with a two-column layout (summary + Install on the left, Skill.md placeholder on the right), loaded via the single-skill API endpoint with 404 handling; the Back button returns to /skills?tab=registry
  • Add unit tests for CardRegistrySkill covering rendering, card navigation, and install dialog pre-fill
Kapture.2026-04-15.at.10.38.57.mp4

@samuv samuv force-pushed the skills-registry branch from 19672c0 to 18c7926 Compare April 15, 2026 08:13
@github-actions github-actions Bot added size/L and removed size/XL labels Apr 15, 2026
@samuv samuv marked this pull request as ready for review April 15, 2026 08:13
Copilot AI review requested due to automatic review settings April 15, 2026 08:13
Copy link
Copy Markdown
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 read-only registry browsing to the Skills area, including URL-persisted tab selection and a new skill detail route backed by the v0.1 registry skills endpoints.

Changes:

  • Adds a Registry tab to /skills with server-side, debounced search and a registry skills card grid.
  • Introduces a skill detail route /skills/$namespace/$skillName with not-found handling and an install CTA.
  • Adds unit tests for the new CardRegistrySkill component (rendering, navigation, install prefill).

Reviewed changes

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

Show a summary per file
File Description
renderer/src/routes/skills_.$namespace.$skillName.tsx New registry skill detail route with loader-based data fetching and 404 mapping to notFound UI
renderer/src/routes/skills.tsx Adds tab search param validation and prefetches registry skills list in the route loader
renderer/src/route-tree.gen.ts Generated route tree updates to include the new skill detail route
renderer/src/features/skills/components/skills-page.tsx Adds Registry tab UI, debounced search state, and registry list query wiring
renderer/src/features/skills/components/skill-detail-page.tsx New detail page layout with Back-to-registry behavior and install dialog prefill
renderer/src/features/skills/components/grid-cards-registry-skills.tsx New grid component for registry skill cards + empty state
renderer/src/features/skills/components/card-registry-skill.tsx New registry skill card with navigation and install action
renderer/src/features/skills/components/tests/card-registry-skill.test.tsx Tests for registry card rendering, navigation behavior, and install dialog prefill

Comment thread renderer/src/features/skills/components/card-registry-skill.tsx
Comment thread renderer/src/routes/skills_.$namespace.$skillName.tsx Outdated
Comment thread renderer/src/routes/skills_.$namespace.$skillName.tsx Outdated
Comment thread renderer/src/routes/skills_.$namespace.$skillName.tsx Outdated
Comment thread renderer/src/features/skills/components/skill-detail-page.tsx Outdated
Comment thread renderer/src/features/skills/components/skills-page.tsx Outdated
@samuv samuv merged commit 7ffa439 into main Apr 15, 2026
35 of 45 checks passed
@samuv samuv deleted the skills-registry branch April 15, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants