feat: auto-detect package.json skills#464
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
2789c94 to
f4c3cd8
Compare
13cdba9 to
88d5b72
Compare
668f4e4 to
cab4ddd
Compare
b6c2e78 to
85c313b
Compare
|
I also thought about put it in the sidebar, on the right. I think I like it more there and also removes the issue with CLS. regarding to the last suggestion, I agree with you, I will implement it I will also change the icon from robot to the agent spec svg |
|
@danielroe ready! |
|
@onmax did you forget to push? |
ab42f80 to
2fe608b
Compare
| const { | ||
| data: fileContent, | ||
| status: fileStatus, | ||
| execute: fetchFileContent, | ||
| } = useFetch<PackageFileContentResponse>(() => fileContentUrl.value!, { immediate: false }) | ||
|
|
||
| watch( | ||
| fileContentUrl, | ||
| url => { | ||
| if (url) fetchFileContent() | ||
| }, | ||
| { immediate: true }, | ||
| ) |
There was a problem hiding this comment.
yes, let me test if this has been fixed in main. Otherwise opening another pr now
danielroe
left a comment
There was a problem hiding this comment.
made a few refactors to align with playgrounds display - I'd be up for updating the style of both though in future! and I think we need to think about how to reduce CLS when doing client-side loading for this kind of info....
package.json skills

Add tabbed UI to switch between skills-npm and skills CLI installation methods.
Sidebar trigger
Modal tabs
Changes
Cons
Fragmented skill repositories
Not many module authors store skills in the same repository as the source code. For example, the repositories for VueUse are vueuse/vueuse and vueuse/skills, which makes it hard for us to link the source code with the skills.
I like how @antfu with skills-npm (and npm-agentskills) tries to solve this by "forcing" module authors to add the skills to the same repository as the source code (the docs usually live in the same repo already!), as these should not be in different repositories.
Non-standard .well-known path
RFC 8615 specifies
.well-knownURIs must be at the domain root (/.well-known/...). This implementation uses/{package}/.well-known/skills/...(or/@org/pkg/.well-known/skills/...for scoped packages).Tradeoff: per-package namespacing works better with the skills CLI (
npx skills add <url>/{package}), but technically doesn't follow the RFC.Open questions