Skip to content

feat(capabilities): Skill capability badges and declaration visibility in UI#546

Open
vincentkoc wants to merge 6 commits intomainfrom
vincentkoc-code/phase1-capabilities-registry-ui
Open

feat(capabilities): Skill capability badges and declaration visibility in UI#546
vincentkoc wants to merge 6 commits intomainfrom
vincentkoc-code/phase1-capabilities-registry-ui

Conversation

@vincentkoc
Copy link
Member

Summary

  • Problem:
    • Users/authors need capability visibility before enforcement is enabled.
  • Why it matters:
    • Visibility-first rollout reduces breakage and guides migration.
  • What changed:
    • Added capability extraction utilities for skill detail surfaces.
    • Updated install card rendering to show declared capabilities and visibility cues.
  • What did NOT change (scope boundary):
    • No browse-page refactor.
    • No report/comments/versions panel rewrites.
    • No publish-time blocking.

Testing

  • bun run lint
  • bun run test
  • bun run build

Related

@vercel
Copy link
Contributor

vercel bot commented Feb 27, 2026

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

Project Deployment Actions Updated (UTC)
clawhub Ready Ready Preview, Comment Feb 27, 2026 9:47pm

@vincentkoc
Copy link
Member Author

Refinement pass completed on UI branch.

Updates added:

  • Capability badge labels now include:
    • messaging (Message sending)
    • scheduling (Scheduling/cron)
  • Network label clarified as Web search/fetch.

Local validation:

  • bun run lint
  • bun run test
  • bun run build
    (all passed)

Note: current failing "Scan for Verified Secrets" check appears infrastructure-related (missing trufflesecurity/trufflehog action tarball), not diff-related.

@vincentkoc vincentkoc changed the title feat(ui): phase 1 capability badges and declaration visibility feat(capability): Skill capability badges and declaration visibility in UI Feb 27, 2026
@vincentkoc vincentkoc marked this pull request as ready for review February 27, 2026 19:01
@vincentkoc vincentkoc changed the title feat(capability): Skill capability badges and declaration visibility in UI feat(capabilities): Skill capability badges and declaration visibility in UI Feb 27, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 07a51ee4fd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

This PR adds capability badge visibility to the skill install card UI, displaying declared capabilities like shell, filesystem, network access with icons and labels. The implementation removes display sections for environment variables, dependencies, and links to keep scope focused.

Critical Issue:

  • The capabilities field is accessed but not defined in ClawdisSkillMetadataSchema, which will cause TypeScript compilation errors. This needs to be resolved before merging.

Key Changes:

  • Added CAPABILITY_DISPLAY mapping for 7 capability types (shell, filesystem, network, browser, sessions, messaging, scheduling)
  • Component now always renders, showing either declared capabilities or "No capabilities declared"
  • Removed envVars, dependencies, and links display sections per stated scope boundary

Confidence Score: 1/5

  • This PR has a critical type error that will prevent compilation
  • The code references clawdis?.capabilities but the ClawdisSkillMetadataSchema doesn't define this field. This will cause TypeScript compilation to fail when type checking is enabled. The schema must be updated to include capabilities before this PR can be merged. The UI implementation itself is clean and well-structured, but the missing schema definition is a blocking issue.
  • src/components/SkillInstallCard.tsx requires schema update in packages/schema/src/schemas.ts before it can function correctly

Last reviewed commit: 07a51ee

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@vincentkoc
Copy link
Member Author

Refinement pass complete on this branch.

UI updates in 464365d:

  • Restored envVar-only visibility in Runtime requirements
  • Restored Dependencies and Links panels
  • Kept capability badges and capability panel behavior
  • Preserved install specs and no-data guard behavior

Validation on branch:

  • bun run lint
  • bun run test
  • bun run build

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 464365d3f8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@vincentkoc
Copy link
Member Author

Second refinement pass pushed.

New commit:

  • d45cf3d: only render capabilities panel when parsed capabilities exist (avoid false "No capabilities declared" state during parser/schema rollout)

Validation: bun run lint && bun run test && bun run build passes.
Replied inline on the new review thread.

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.

1 participant