fix: properly log error when dependency install fails

[AdrianGonz97]

closes #227

[changeset-bot]

🦋 Changeset detected

Latest commit: d69760c

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

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/sveltejs/cli/sv@228

commit: d69760c

[manuel3108]

The code looks good, and probably it's just my testing that is flawed.

I created a project, edit one package inside package.json to a non existing version and executed this pr. It's still not printing anything.

Executing pnpm install by hand brings the following output with exit code 1

Output

ERR_PNPM_NO_MATCHING_VERSION  No matching version found for @sveltejs/adapter-auto@^30.0.0

This error happened while installing a direct dependency of D:\dev\web\svelte-cli-temp

The latest release of @sveltejs/adapter-auto is "3.3.1".

Other releases are:
  * next: 1.0.0-next.91

If you need the full list of all 113 published versions run "$ pnpm view @sveltejs/adapter-auto versions".
Progress: resolved 12, reused 12, downloaded 0, added 0

I'm assuming this is technically a pnpm error for not printing to stderr, but we probably should care about both.

[AdrianGonz97]

I'm assuming this is technically a pnpm error for not printing to stderr, but we probably should care about both.

Yea, I think this is a pnpm bug. Tried testing out using the silent flag: pnpm i --silent and the process never ended. I think what we have now should be fine for the time being

[manuel3108]

Just tested this again with npm and everything works as expected there.