Skip to content

fix(update): update all installed instances of a skill#78

Open
maxpetrusenko wants to merge 1 commit intonumman-ali:mainfrom
maxpetrusenko:fix/75-update-all-instances
Open

fix(update): update all installed instances of a skill#78
maxpetrusenko wants to merge 1 commit intonumman-ali:mainfrom
maxpetrusenko:fix/75-update-all-instances

Conversation

@maxpetrusenko
Copy link

Summary

  • update command now scans all installed skill instances (no name dedupe), so duplicates across .agent/.claude and project/global are all candidates
  • keep existing dedupe behavior for other commands by making dedupe optional in findAllSkills
  • add regression test proving update demo updates both project and global instances of demo

Why

Closes #75.

Verification

  • npm run typecheck
  • npm run build
  • npx vitest run tests/commands/update.test.ts tests/utils/skills.test.ts

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] Update command doesn't update all paths

1 participant