fix: make build and node test scripts cross-platform

laplace young · fatso83 · commit 2c975393f19b · 2026-05-05T21:55:24.000+02:00
diff --git a/build.cjs b/build.cjs
@@ -1,5 +1,5 @@
 #!/usr/bin/env node
- 
+
 const fs = require("node:fs");
 const { execFileSync } = require("node:child_process");
 const esbuild = require("esbuild");
@@ -8,7 +8,8 @@ const pkg = require("./package.json");
 
 // Step 1: Run Rollup to generate lib/ from src/
 console.log("Generating lib/ from src/ using Rollup...");
-execFileSync("npx", ["rollup", "-c", "rollup.config.mjs"], {
+const rollupCli = require.resolve("rollup/dist/bin/rollup");
+execFileSync(process.execPath, [rollupCli, "-c", "rollup.config.mjs"], {
     stdio: "inherit",
 });
 
diff --git a/package.json b/package.json
@@ -33,7 +33,7 @@
   "scripts": {
     "build-artifacts": "node ./build.cjs",
     "pretest-node": "npm run build-artifacts",
-    "test-node": "mocha --recursive -R dot 'test/{src,issues,scripts}/**/*-test.js'",
+    "test-node": "mocha --recursive -R dot \"test/{src,issues,scripts}/**/*-test.js\"",
     "pretest-headless": "npm run build-artifacts",
     "test-dev": "CHOKIDAR_USEPOLLING=1 npm run test-node -- --watch",
     "test-headless": "node ./scripts/run-browser-suite.mjs headless",
@@ -56,7 +56,7 @@
     "build": "npm run build-artifacts",
     "build-docs": "cd docs; make build",
     "serve-docs": "cd docs; make livereload",
-    "lint": "eslint --max-warnings 0 '**/*.{js,cjs,mjs}'",
+    "lint": "eslint --max-warnings 0 \"**/*.{js,cjs,mjs}\"",
     "pretest-webworker": "npm run build-artifacts",
     "prebuild": "rimraf pkg && npm run update-compatibility",
     "postbuild": "run-s test-esm-support test-esm-browser-build test-contract check-dependencies",
diff --git a/rollup.config.mjs b/rollup.config.mjs
@@ -11,7 +11,7 @@ function getAllFiles(dir, fileList = []) {
         if (fs.statSync(filePath).isDirectory()) {
             getAllFiles(filePath, fileList);
         } else if (filePath.endsWith(".js")) {
-            fileList.push(filePath);
+            fileList.push(filePath.split(path.sep).join("/"));
         }
     }
     return fileList;

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ function getAllFiles(dir, fileList = []) {`
`11`	`11`	`if (fs.statSync(filePath).isDirectory()) {`
`12`	`12`	`getAllFiles(filePath, fileList);`
`13`	`13`	`} else if (filePath.endsWith(".js")) {`
`14`		`- fileList.push(filePath);`
	`14`	`+ fileList.push(filePath.split(path.sep).join("/"));`
`15`	`15`	`}`
`16`	`16`	`}`
`17`	`17`	`return fileList;`