Skip to content

Comments

feat: indicate which skills failed to update, fix #362, code cleanup#387

Open
EDM115 wants to merge 12 commits intovercel-labs:mainfrom
EDM115:edm115-failed-update-details
Open

feat: indicate which skills failed to update, fix #362, code cleanup#387
EDM115 wants to merge 12 commits intovercel-labs:mainfrom
EDM115:edm115-failed-update-details

Conversation

@EDM115
Copy link

@EDM115 EDM115 commented Feb 18, 2026

  • skills which failed to update/check for update are now properly listed so we can check if the skill still exists or if it have been renamed for instance. example output :
    $ skills check -g
    Checking for skill updates...
    
    Checking 66 skill(s) for updates...
    
    2 update(s) available:
    
      ↑ make-repo-contribution
        source: github/awesome-copilot
      ↑ vueuse
        source: onmax/nuxt-skills
    
    Run npx skills update to update all skills
    
    Could not check 1 skill(s) (may need reinstall)
    
      ✗ vue-development-guides
        source: vuejs-ai/skills
    
    
  • when the skills CLI have been installed globally using PNPM for example, update won't work since it calls a nested npx for a package that haven't been retrieved before. skills update now just reuse its own CLI entrypoint to run that. fix [Bug]: The update command can fail when skills is not installed with npm #362
  • remove unused consts/functions/types/imports
  • fixed an issue with a potentially undefined variable
  • refresh the AI instructions
  • fixed the link to supported agents in the readme
  • format MD files
  • prettier scripts now also run fine on Windows

- remove unused imports and consts/functions. some have been left, ex unused term colors, as they might be useful in the future
- fixed the agents link in the readme
- make sure prettier commands also run on windows
- reformat md files
- refresh the AGENTS instructions
- fixed an undefined var issue
@vercel
Copy link

vercel bot commented Feb 18, 2026

@EDM115 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@quuu
Copy link
Collaborator

quuu commented Feb 18, 2026

please resolve merge conflicts, then i'll merge

EDM115 and others added 5 commits February 18, 2026 15:05
- remove unused imports and consts/functions. some have been left, ex unused term colors, as they might be useful in the future
- fixed the agents link in the readme
- make sure prettier commands also run on windows
- reformat md files
- refresh the AGENTS instructions
- fixed an undefined var issue
@EDM115
Copy link
Author

EDM115 commented Feb 18, 2026

good catch for the entrypoint existence 🫡
i'll resolve the conflicts now

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.

[Bug]: The update command can fail when skills is not installed with npm

2 participants