Fix "Cannot read properties of null" in cli/telemetry

[the-dijkstra]

Changes

Latest version of Astro is throwing the following error on CI:

error   Cannot read properties of null (reading 'name')
    File:
      node_modules/astro/dist/cli/telemetry/index.js:5:57
    Code:
      4 | async function notify() {
      > 5 |   const packageManager = (await whichPm(process.cwd())).name ?? "npm";
          |                                                         ^
        6 |   await telemetry.notify(() => {
        7 |     console.log(msg.telemetryNotice(packageManager) + "\n");
        8 |     return true;
    Stacktrace:
  TypeError: Cannot read properties of null (reading 'name')
      at notify (node_modules/astro/dist/cli/telemetry/index.js:5:57)
      at async runCommand (node_modules/astro/dist/cli/index.js:97:3)
      at async cli (node_modules/astro/dist/cli/index.js:144:5)

I will add changsets if this gets approved

Testing

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 8c3ca89

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

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

[jacobdalamb]

Oops, my mistake for forgetting to add the ? to it.
d2f2a11

[Princesseuh]

Thanks, this looks great! Surprised this wasn't a type error 🤔