Skip to content

feat(ui): add install command snippet and copy functionality to Skill…#588

Open
moonrailgun wants to merge 1 commit intoopenclaw:mainfrom
moonrailgun:feat/add-install-command
Open

feat(ui): add install command snippet and copy functionality to Skill…#588
moonrailgun wants to merge 1 commit intoopenclaw:mainfrom
moonrailgun:feat/add-install-command

Conversation

@moonrailgun
Copy link

…DetailPage

add this section which help people quick install

image

@vercel
Copy link
Contributor

vercel bot commented Mar 3, 2026

@moonrailgun is attempting to deploy a commit to the Amantus Machina Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 3, 2026

Greptile Summary

This PR adds a clean "Install via clawhub" card to the SkillDetailPage that displays the CLI install command with a one-click clipboard copy button. The implementation correctly:

  • Hides the card for malware-blocked or removed skills via the modInfo checks (line 387)
  • Leverages existing loading/null guards to ensure skill.slug is always safe to access
  • Uses simple, accessible React patterns with proper ARIA labels
  • Follows the existing design system with CSS variables, hover/focus-visible styles, and smooth transitions

The feature is straightforward and ready for merge.

Confidence Score: 5/5

  • Safe to merge - purely additive feature with correct moderation guards and no logic issues
  • This PR adds a new UI feature that is well-implemented and properly integrated. The InstallViaClawhubCard component correctly checks modInfo to hide itself for malware-blocked or removed skills. The clipboard copy functionality includes proper error handling, and the CSS styling follows existing design patterns. No functional issues or regressions identified.
  • No files require special attention

Last reviewed commit: 5483774

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