fix(scan): use fileURLToPath for the entrypoint scan root (Windows)

[dustinlacewell]

Problem

scanQwikEntrypoints uses config.root.pathname as the cwd for the grep spawn and as the base for resolve(). On Windows that value is /C:/project/, which is not a valid filesystem path. spawn fails with ENOENT, grepQwikFiles swallows the error and returns "", so:

• no entrypoints are detected,
• the standalone client build is skipped,
• every QRL reports a Q14 error during prerender.

Reproduction on Windows with the code from 1.1.0:

pathname: /D:/code/site/
ERR ENOENT spawn grep ENOENT

grep itself is on PATH (Git for Windows). The cwd is the failure.

Fix

Convert the root URL with fileURLToPath(config.root). That yields a native path on every platform, so both the cwd and the resolved absolute entrypoint paths are correct. One line, no behaviour change on POSIX.

This is the minimal version of #292. It does not add a Node fallback for hosts without grep; it only fixes the path.

Tests

Added scanQwikEntrypoints unit tests that scan a real temp directory (one Qwik file, one plain file, one match under node_modules) and assert the native absolute path is returned and the filter is honoured. The first test fails on the current code on Windows and passes with the fix.

Note: two pre-existing resolveQwikPaths tests already fail on Windows because they compare against /-separated strings. Left out of scope here.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 9c90d2f

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

[thejackshelton]

LGTM