Skip to content

test: cover skillkit package manager discovery - #60

Open
k4its1t wants to merge 1 commit into
Railly:mainfrom
k4its1t:test/42-skillkit-package-manager-paths
Open

test: cover skillkit package manager discovery#60
k4its1t wants to merge 1 commit into
Railly:mainfrom
k4its1t:test/42-skillkit-package-manager-paths

Conversation

@k4its1t

@k4its1t k4its1t commented Aug 28, 2026

Copy link
Copy Markdown

Summary

  • add parameterized coverage for every static package-manager directory used by skillkit discovery
  • cover NVM and mise version directories
  • cover pnpm, Yarn, and npm dynamic global-bin fallbacks
  • keep the path table synchronized with getPackageManagerBinDirs

Validation

  • bun test (31 passed)
  • bun run lint
  • bun run build
  • git diff --check

Closes #42

AI assistance: Codex was used to implement and validate this change.

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

@k4its1t is attempting to deploy a commit to the Crafter Station Team on Vercel.

A member of the Team first needs to authorize it.

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.

Parametrized test covering every package manager findSkillkitBin() supports

1 participant