From 63cb425a57de1cf1f7e42ec9de70d44212dd027c Mon Sep 17 00:00:00 2001 From: Jascha Wanger Date: Thu, 23 Apr 2026 12:33:18 -0700 Subject: [PATCH] =?UTF-8?q?ci:=20rename=20rollup.config.js=20=E2=86=92=20r?= =?UTF-8?q?ollup.config.mjs=20for=20Node=2018=20compatibility?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CI on Node 18 fails because each packages/*/rollup.config.js uses ESM import syntax but no package.json declares "type": "module" and the file extension is .js. Node 20+ transparently detects the ESM and reparses as a module (the behavior we were relying on locally); Node 18 throws SyntaxError: Cannot use import statement outside a module. Rename all 9 rollup configs from .js to .mjs, which is the canonical way to mark a config file as ESM without forcing the whole package to be type:module. No package.json or source changes needed. Verified locally: `npm run build --workspaces` still builds all 9 packages, and `npx vitest run` still reports 1082/1082 tests passing. --- packages/agent/{rollup.config.js => rollup.config.mjs} | 0 packages/cli/{rollup.config.js => rollup.config.mjs} | 0 packages/core/{rollup.config.js => rollup.config.mjs} | 0 packages/mcp/{rollup.config.js => rollup.config.mjs} | 0 packages/policy/{rollup.config.js => rollup.config.mjs} | 0 packages/secrets/{rollup.config.js => rollup.config.mjs} | 0 packages/testing/{rollup.config.js => rollup.config.mjs} | 0 packages/tool-review/{rollup.config.js => rollup.config.mjs} | 0 packages/types/{rollup.config.js => rollup.config.mjs} | 0 9 files changed, 0 insertions(+), 0 deletions(-) rename packages/agent/{rollup.config.js => rollup.config.mjs} (100%) rename packages/cli/{rollup.config.js => rollup.config.mjs} (100%) rename packages/core/{rollup.config.js => rollup.config.mjs} (100%) rename packages/mcp/{rollup.config.js => rollup.config.mjs} (100%) rename packages/policy/{rollup.config.js => rollup.config.mjs} (100%) rename packages/secrets/{rollup.config.js => rollup.config.mjs} (100%) rename packages/testing/{rollup.config.js => rollup.config.mjs} (100%) rename packages/tool-review/{rollup.config.js => rollup.config.mjs} (100%) rename packages/types/{rollup.config.js => rollup.config.mjs} (100%) diff --git a/packages/agent/rollup.config.js b/packages/agent/rollup.config.mjs similarity index 100% rename from packages/agent/rollup.config.js rename to packages/agent/rollup.config.mjs diff --git a/packages/cli/rollup.config.js b/packages/cli/rollup.config.mjs similarity index 100% rename from packages/cli/rollup.config.js rename to packages/cli/rollup.config.mjs diff --git a/packages/core/rollup.config.js b/packages/core/rollup.config.mjs similarity index 100% rename from packages/core/rollup.config.js rename to packages/core/rollup.config.mjs diff --git a/packages/mcp/rollup.config.js b/packages/mcp/rollup.config.mjs similarity index 100% rename from packages/mcp/rollup.config.js rename to packages/mcp/rollup.config.mjs diff --git a/packages/policy/rollup.config.js b/packages/policy/rollup.config.mjs similarity index 100% rename from packages/policy/rollup.config.js rename to packages/policy/rollup.config.mjs diff --git a/packages/secrets/rollup.config.js b/packages/secrets/rollup.config.mjs similarity index 100% rename from packages/secrets/rollup.config.js rename to packages/secrets/rollup.config.mjs diff --git a/packages/testing/rollup.config.js b/packages/testing/rollup.config.mjs similarity index 100% rename from packages/testing/rollup.config.js rename to packages/testing/rollup.config.mjs diff --git a/packages/tool-review/rollup.config.js b/packages/tool-review/rollup.config.mjs similarity index 100% rename from packages/tool-review/rollup.config.js rename to packages/tool-review/rollup.config.mjs diff --git a/packages/types/rollup.config.js b/packages/types/rollup.config.mjs similarity index 100% rename from packages/types/rollup.config.js rename to packages/types/rollup.config.mjs