fix(prefetch): Fix the inclusion of @types/network-information

[connor-baer]

Changes

#7104 added the "files" field to all packages to limit the files that are published to npm. The @astro/prefetch package only includes the dist folder, however, the code tries to import from the @types folder, causing the following error:

Could not resolve "../@types/network-information.d.ts" from "node_modules/@astrojs/prefetch/dist/client.js"

I added the @types folder to the "files" field to fix it.

I changed the import to a triple-slash directive as suggested by @bluwy.

Testing

I did not test the change as it seems straightforward.

Docs

This fixes an expected behavior, so an update to the docs doesn't seem necessary.

[changeset-bot]

🦋 Changeset detected

Latest commit: 907990f

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

[bluwy]

Hi, thanks for catching this. I didn't expect the types file to be imported, and it doesn't seem right that it's imported by a JS file.

I think instead we can remove the import here:

astro/packages/integrations/prefetch/src/client.ts

Line 2 in 31cbf43

import '../@types/network-information.d.ts';

and add this to the top of the file instead.

/// <reference types="../@types/network-information.d.ts" />

[connor-baer]

@bluwy That makes sense! I've changed the approach as suggested.

[millette]

Are releases automated/on a schedule? Downgrading for now.

[bluwy]

I've cut a new release to fix it.