Skip to content

feat(skills): surface source repository and tighten registry metadata#2064

Merged
samuv merged 3 commits intomainfrom
skill-registry-metadata
Apr 22, 2026
Merged

feat(skills): surface source repository and tighten registry metadata#2064
samuv merged 3 commits intomainfrom
skill-registry-metadata

Conversation

@samuv
Copy link
Copy Markdown
Collaborator

@samuv samuv commented Apr 21, 2026

Following #2045, this PR brings the skills registry closer to parity with MCP servers: the upstream repo is one click away from every view, the SKILL.md preview drops the metadata we already render as badges, and the detail page stops resizing between skills.

The registry response distinguishes between namespace (packager, e.g. io.github.stacklok) and repository.url (upstream source, e.g. github.com/trailofbits/skills). Today only namespace is visible — it's not obvious the skill was authored elsewhere and packaged downstream. MCP servers already solve this via repository_url; this PR lines the two up.

Changes

  • Card / table / detail — link skill.repository.url with the same GitHub-icon pattern used for MCP servers: ghost icon on the card footer, an icon-only trailing column on the registry table, and an outline GitHub button next to Install on the detail page. All three render nothing when repository is absent, and their click handlers stop propagation so card/row navigation isn't triggered.
  • Table column renameAuthorRegistry. The column holds the reverse-DNS namespace (the packager/registry), not the author.
  • SKILL.md preview — enable stripFrontmatter on the registry detail. SkillMarkdown already supported it; only the build detail was passing it. Preview now starts at the first heading instead of the YAML block duplicating name / description / license / version.
  • Detail layout — the right panel was flex-1 without min-w-0, so wide SKILL.md content (code blocks, tables) pushed the column past its 7/12 allotment and varied between skills. Adding min-w-0 lets the flex child shrink; overflow now scrolls inside the card. Applies to both registry and build detail pages (shared layout).

How to validate

  1. Registry tab — card footer has a GitHub icon; opens the upstream repo in a new tab, card stays in place.
  2. Table view — Registry header (previously Author); GitHub icon sits between About and Install.
  3. Skill detail (e.g. io.github.stacklok/agentic-actions-auditor) — GitHub button next to Install; SKILL.md starts at the first heading; card width is identical across skills.

samuv added 2 commits April 21, 2026 19:21
Link skill.repository.url with the same GitHub-icon pattern used for MCP servers — ghost icon on the card footer, a trailing column on the registry table, and an outline 'GitHub' button next to Install on the detail page. Rename the table's 'Author' column to 'Registry' to match the reverse-DNS namespace it actually holds, and strip the YAML frontmatter from the registry SKILL.md preview so it starts at the first real heading.
The right panel was flex-1 without min-w-0, so wide SKILL.md content (long code blocks, tables) pushed the column past its intended 7/12. Adding min-w-0 lets the flex child shrink to its allotment; overflow now scrolls inside the card instead of stretching the column across skills.
@samuv samuv self-assigned this Apr 21, 2026
Copilot AI review requested due to automatic review settings April 21, 2026 17:23
@samuv samuv changed the title Skill registry metadata feat(skills): surface source repository and tighten registry metadata Apr 21, 2026
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

This PR improves the skills registry UX to better match the MCP servers experience by exposing upstream repository metadata, reducing duplicated SKILL.md metadata rendering, and stabilizing the skill detail layout width.

Changes:

  • Add upstream repository (GitHub) links to registry skill card, table, and detail views when skill.repository.url is present.
  • Rename the table column label from Author to Registry to reflect the namespace/packager value.
  • Strip YAML frontmatter from SKILL.md previews and prevent the detail right panel from expanding layout width (min-w-0).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
renderer/src/features/skills/components/table-registry-skills.tsx Adds repository icon column/link and renames “Author” → “Registry”.
renderer/src/features/skills/components/skill-detail-page.tsx Adds GitHub button/link and enables stripFrontmatter for SKILL.md rendering.
renderer/src/features/skills/components/skill-detail-layout.tsx Adds min-w-0 to stabilize right panel sizing/overflow behavior.
renderer/src/features/skills/components/card-registry-skill.tsx Adds GitHub icon link in the card footer next to Install.
renderer/src/features/skills/components/tests/table-registry-skills.test.tsx Updates column header assertion and adds coverage for repository link behavior.
renderer/src/features/skills/components/tests/card-registry-skill.test.tsx Adds coverage for repository link presence and click not triggering navigation.

Comment thread renderer/src/features/skills/components/skill-detail-page.tsx Outdated
Comment thread renderer/src/features/skills/components/table-registry-skills.tsx
Comment thread renderer/src/features/skills/components/card-registry-skill.tsx
Comment thread renderer/src/features/skills/components/skill-detail-page.tsx Outdated
Copy link
Copy Markdown
Collaborator

@peppescg peppescg left a comment

Choose a reason for hiding this comment

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

🚢

@samuv samuv merged commit be08937 into main Apr 22, 2026
17 checks passed
@samuv samuv deleted the skill-registry-metadata branch April 22, 2026 08:32
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