Fix cli package resolution for @astrojs/db

[BryceRussell]

Changes

Fixes bug where every time a astro db command is used it prompts the user to add @astrojs/db even if it is already installed

Testing

No tests, it is a pretty small fix for an error that doesn't get surfaced because it is inside a try/catch

Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

Docs

No docs needed

[changeset-bot]

⚠️ No Changeset found

Latest commit: 984b50c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

[BryceRussell]

This was mentioned in Discord (as a reply to a triage/PR message for this issue):

We are addressing this! Let me guess, you’re on windows and use an Astro.config.ts instead of mjs?

Not sure if this relates to this issue or another one, maybe the fix for this is more complicated? I will close this if I hear anything else about the issue being fixed

[bluwy]

That is being fixed at #10342. It looks like this is a bit different than that though.

I was waiting for someone more familiar with db before merging this, just in case, but seems like this is a simple fix enough. EDIT: nvm Matthew just approved.

[BryceRussell]

Now that this has been approved, is this waiting on me to merge it? Sorry I haven't made a PR in a long time and I just want to make sure I am not blocking this

[bluwy]

You can merge but no I didn't expect that and it wasn't blocking the PR. I saw Matthew approved and I thought he was gonna merge it. I'm gonna merge this now 😄

[BryceRussell]

Awesome thanks!