Fix build and run on Windows (+ additional benchmarks CI target platform).

.github/workflows/benchmarks.yml

@@ -15,7 +15,7 @@ jobs:
       fail-fast: true
       matrix:
         node: [20]
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout repository
@@ -37,6 +37,7 @@ jobs:
           git submodule update --init --recursive
           npm ci
           npm run build
-          bash run-ci-benchmarks.sh
+          echo 'Running o1js benchmarks.'
+          node --enable-source-maps --stack-trace-limit=1000 src/build/run.js benchmark/runners/with-cloud-history.ts --bundle >>benchmarks.log 2>&1
           cat benchmarks.log >> $GITHUB_STEP_SUMMARY
         shell: bash

benchmark/benchmark.ts

@@ -112,7 +112,7 @@ function logResult(
   result: BenchmarkResult,
   previousResult?: BenchmarkResult
 ): void {
-  console.log(result.label + `\n`);
+  console.log(result.label);
   console.log(`time: ${resultToString(result)}`);
 
   if (previousResult === undefined) return;

src/build/build-example.js

@@ -2,14 +2,19 @@ import fs from 'fs/promises';
 import path from 'path';
 import ts from 'typescript';
 import esbuild from 'esbuild';
+import { platform } from 'node:process';
 
 export { buildAndImport, build, buildOne };
 
 async function buildAndImport(srcPath, { keepFile = false }) {
   let absPath = await build(srcPath);
   let importedModule;
   try {
-    importedModule = await import(absPath);
+    let absPathForImport = absPath
+    if (platform === 'win32') {
+      absPathForImport = 'file:///' + absPathForImport;
+    }
+    importedModule = await import(absPathForImport);
   } finally {
     if (!keepFile) await fs.unlink(absPath);
   }

src/build/build-node.js

@@ -1,4 +1,5 @@
 import path from 'node:path';
+import { platform } from 'node:process';
 import { fileURLToPath } from 'node:url';
 import esbuild from 'esbuild';
 import minimist from 'minimist';
@@ -46,13 +47,13 @@ async function buildNode({ production }) {
 }
 
 function makeNodeModulesExternal() {
-  let isNodeModule = /^[^./]|^\.[^./]|^\.\.[^/]/;
+  let isNodeModule = /^[^./\\]|^\.[^./\\]|^\.\.[^/\\]/;
   return {
     name: 'plugin-external',
     setup(build) {
       build.onResolve({ filter: isNodeModule }, ({ path }) => ({
         path,
-        external: true,
+        external: !(platform === 'win32' && path.endsWith('index.js')),
       }));
     },
   };

src/build/run.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 import minimist from 'minimist';
+import { platform } from 'node:process';
 import { buildAndImport, buildOne } from './build-example.js';
 
 let {
@@ -18,6 +19,9 @@ npx snarky-run [file]`);
 if (!bundle) {
   let absPath = await buildOne(filePath);
   console.log(`running ${absPath}`);
+  if (platform === 'win32') {
+    absPath = 'file:///' + absPath;
+  }
   let module = await import(absPath);
   if (main) await module.main();
   if (runDefault) await module.default();