chore: add Windows to the unit tests CI

Author: Varixo

.github/workflows/ci.yml

@@ -592,7 +592,13 @@ jobs:
   test-unit:
     name: Unit Tests
     if: always() && needs.changes.outputs.build-unit == 'true'
-    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        settings:
+          - host: ubuntu-latest
+          - host: windows-latest
+
+    runs-on: ${{ matrix.settings.host }}
     needs:
       - changes
       - build-other-packages

packages/insights/package.json

@@ -22,7 +22,7 @@
     "@types/density-clustering": "1.3.3",
     "@types/node": "24.0.4",
     "autoprefixer": "10.4.19",
-    "better-sqlite3": "9.6.0",
+    "better-sqlite3": "12.2.0",
     "eslint": "9.25.1",
     "eslint-plugin-qwik": "workspace:*",
     "globals": "16.0.0",

packages/qwik/src/optimizer/src/plugins/plugin.ts

@@ -242,10 +242,10 @@ export function createQwikPlugin(optimizerOptions: OptimizerOptions = {}) {
       // we only provide inputs if none were provided by the user
       if (opts.target === 'ssr') {
         // this is for dev mode, prod will have own setting
-        opts.input = [path.resolve(srcDir, 'entry.ssr')];
+        opts.input = [normalizePath(path.resolve(srcDir, 'entry.ssr'))];
       } else if (opts.target === 'client') {
         // not really an entry, just a starting point
-        opts.input = [path.resolve(srcDir, 'root')];
+        opts.input = [normalizePath(path.resolve(srcDir, 'root'))];
       } else {
         // others including lib should be ok already
         opts.input = undefined!;
@@ -390,6 +390,10 @@ export function createQwikPlugin(optimizerOptions: OptimizerOptions = {}) {
     if (opts.input && typeof opts.input === 'string') {
       opts.input = [opts.input];
     }
+    // Normalize input paths to ensure consistent path separators across platforms
+    if (opts.input && Array.isArray(opts.input)) {
+      opts.input = opts.input.map((input) => normalizePath(input));
+    }
     return out;
   };
 

packages/qwik/src/optimizer/src/plugins/rollup.ts

@@ -170,7 +170,9 @@ const getChunkFileName = (
       // The chunk name can often be a path. We sanitize it to use dashes instead of slashes, to keep the same folder structure as without debug:true.
       // Besides, Rollup doesn't accept absolute or relative paths as inputs for the [name] placeholder for the same reason.
       const relativePath = optimizer.sys.path.relative(optimizer.sys.cwd(), chunkInfo.name);
-      const sanitized = relativePath
+      // Normalize path separators to forward slashes before sanitizing
+      const normalizedPath = relativePath.replace(/\\/g, '/');
+      const sanitized = normalizedPath
         .replace(/^(\.\.\/)+/, '')
         .replace(/^\/+/, '')
         .replace(/\//g, '-');

packages/qwik/src/optimizer/src/plugins/vite.ts

@@ -183,6 +183,16 @@ export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
       const opts = await qwikPlugin.normalizeOptions(pluginOpts);
       input ||= opts.input;
 
+      // Normalize input paths to ensure consistent path separators across platforms
+      if (input && Array.isArray(input)) {
+        input = input.map((inputPath) => {
+          // Preserve relative path prefixes (./) while normalizing separators
+          const isRelative = inputPath.startsWith('./');
+          const normalized = qwikPlugin.normalizePath(inputPath);
+          return isRelative && !normalized.startsWith('./') ? `./${normalized}` : normalized;
+        });
+      }
+
       manifestInput = opts.manifestInput;
       srcDir = opts.srcDir;
       rootDir = opts.rootDir;