feat(sv): allow custom package manager in testing commands

[GauBen]

Description

As of now, pnpm is hardcoded in the testing setup. This PR enables setting a custom package manager.

Checklist

• Update snapshots (if applicable)
• Add a changeset (if applicable)
• Allow maintainers to edit this PR
• I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

[pkg-svelte-dev]

Install the latest version of sv from 03097c7:

pnpm add https://pkg.svelte.dev/sv/c/03097c7789300e3ed1db1ba95f76e263ad8c7879

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1212

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

[changeset-bot]

🦋 Changeset detected

Latest commit: 03097c7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
sv	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[AdrianGonz97]

I think this change requires a bit more nuance. We chose pnpm as the default specifically because of workspaces. setupTest inserts a pnpm-workspace.yaml file into the test's root directory so that the test variants can have their dependencies installed all at once.

In order to support other package managers, we would have to add their respective workspace configurations as well to match our current pnpm-workspace.yaml (e.g. npm/yarn/bun has workspaces defined in the package.json, while deno may have it as workspace in deno.json).

[GauBen]

Indeed I missed that aspect

I fixed my original issue another way:

• file: linking in tests https://github.com/GauBen/vite-plugin-svelte-md/blob/677795d62bf2aef903f2735b41188fc6da345c62/src/sv/index.ts#L23-L25
• installing pnpm in CI https://github.com/GauBen/vite-plugin-svelte-md/blob/677795d62bf2aef903f2735b41188fc6da345c62/.github/workflows/NodeCI.yml#L38

It could probably be more elegant but it's one less thing to worry about here