ci: test against node v22 (v1)

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest]
-        node: [18, 20]
+        node: [18, 20, 22]
       fail-fast: false
 
     steps:
@@ -28,15 +28,15 @@ jobs:
           node-version: ${{ matrix.node }}
           cache: "pnpm"
       - uses: oven-sh/setup-bun@v2
-        if: ${{ matrix.os != 'windows-latest' && matrix.node == 20 }}
+        if: ${{ matrix.os != 'windows-latest' && matrix.node == 22 }}
         with:
           bun-version: latest
       - run: pnpm install
       - run: pnpm lint
-        if: ${{ matrix.os != 'windows-latest' && matrix.node == 20 }}
+        if: ${{ matrix.os != 'windows-latest' && matrix.node == 22 }}
       - run: pnpm build
       - run: pnpm vitest run --coverage
       - run: pnpm test:bun --coverage
-        if: ${{ matrix.os != 'windows-latest' && matrix.node == 20 }}
+        if: ${{ matrix.os != 'windows-latest' && matrix.node == 22 }}
       # - name: Coverage
       #   uses: codecov/codecov-action@v1

.nvmrc

@@ -0,0 +1 @@
+22

test/__snapshots__/fixtures.test.ts.snap

@@ -14,48 +14,17 @@ import.meta.env?.TEST true"
 `;
 
 exports[`fixtures > error-parse > stderr 1`] = `
-"<root>/lib/index.js:2
-  throw err; /* ↓ Check stack trace ↓ */
-  ^
-
-Error: ParseError: \`import\` can only be used in \`import 
- <cwd>/index.ts
-    at Object.<anonymous> (<root>/bin/jiti)
-    at Module._compile (internal/modules/cjs/loader)
-    at Module._extensions..js (internal/modules/cjs/loader)
-    at Module.load (internal/modules/cjs/loader)
-    at Module._load (internal/modules/cjs/loader)
-    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main)
-    at internal/main/run_main_module
-
-Node.js v<version>"
+[
+  "Error: ParseError: \`import\` can only be used in \`import ",
+]
 `;
 
 exports[`fixtures > error-parse > stdout 1`] = `""`;
 
 exports[`fixtures > error-runtime > stderr 1`] = `
-"<root>/lib/index.js:2
-  throw err; /* ↓ Check stack trace ↓ */
-  ^
-
-TypeError: The "listener" argument must be of type function. Received undefined
-    at checkListener (events)
-    at _addListener (events)
-    at process.addListener (events)
-    at <cwd>/index.ts
-    at evalModule (<root>/dist/jiti)
-    at jiti (<root>/dist/jiti)
-    at Object.<anonymous> (<root>/bin/jiti)
-    at Module._compile (internal/modules/cjs/loader)
-    at Module._extensions..js (internal/modules/cjs/loader)
-    at Module.load (internal/modules/cjs/loader)
-    at Module._load (internal/modules/cjs/loader)
-    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main)
-    at internal/main/run_main_module {
-  code: 'ERR_INVALID_ARG_TYPE'
-}
-
-Node.js v<version>"
+[
+  "TypeError: The "listener" argument must be of type function. Received undefined",
+]
 `;
 
 exports[`fixtures > error-runtime > stdout 1`] = `""`;
@@ -65,21 +34,7 @@ exports[`fixtures > esm > stdout 1`] = `
 {
   file: '<cwd>/test.js',
   dir: '<cwd>',
-  'import.meta.url': 'file://<cwd>/test.js',
-  stack: [
-    'at getStack (<cwd>/test)',
-    'at test (<cwd>/test)',
-    'at <cwd>/index.js',
-    'at evalModule (<root>/dist/jiti)',
-    'at jiti (<root>/dist/jiti)',
-    'at Object.<anonymous> (<root>/bin/jiti)',
-    'at Module._compile (internal/modules/cjs/loader)',
-    'at Module._extensions..js (internal/modules/cjs/loader)',
-    'at Module.load (internal/modules/cjs/loader)',
-    'at Module._load (internal/modules/cjs/loader)',
-    'at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main)',
-    'at internal/main/run_main_module'
-  ]
+  'import.meta.url': 'file://<cwd>/test.js'
 }"
 `;
 
@@ -99,6 +54,12 @@ Required : { test: 123 } .default: { test: 123 }
 Dynamic Imported : { test: 123 } .default: { test: 123 }"
 `;
 
+exports[`fixtures > mixed > stderr 1`] = `
+[
+  "ExperimentalWarning: CommonJS module <cwd>/index.js is loading ES Module <cwd>/esm.js using require().",
+]
+`;
+
 exports[`fixtures > mixed > stdout 1`] = `"Mixed works for: <cwd>"`;
 
 exports[`fixtures > native > stdout 1`] = `"[Module: null prototype] { default: { hasRequire: false } }"`;

test/fixtures.test.ts

@@ -3,6 +3,8 @@ import { execa } from "execa";
 import { describe, it, expect } from "vitest";
 import fg from "fast-glob";
 
+const nodeMajor = Number.parseInt(process.version.slice(1), 10);
+
 describe("fixtures", async () => {
   const jitiPath = resolve(__dirname, "../bin/jiti.js");
 
@@ -35,6 +37,14 @@ describe("fixtures", async () => {
           .trim();
       }
 
+      function extractErrors(message: string) {
+        const errors = [] as string[];
+        for (const m of message.matchAll(/\w*(Error|Warning).*:.*$/gm)) {
+          errors.push(m[0]);
+        }
+        return errors;
+      }
+
       const { stdout, stderr } = await execa("node", [jitiPath, fixtureEntry], {
         cwd,
         stdio: "pipe",
@@ -46,9 +56,12 @@ describe("fixtures", async () => {
       });
 
       if (name.includes("error")) {
-        expect(cleanUpSnap(stderr)).toMatchSnapshot("stderr");
+        expect(extractErrors(cleanUpSnap(stderr))).toMatchSnapshot("stderr");
+      } else if (name === "mixed" && nodeMajor >= 22) {
+        console.log(stderr);
+        expect(extractErrors(cleanUpSnap(stderr))).toMatchSnapshot("stderr");
       } else {
-        // expect no error
+        // Expect no error by default
         expect(stderr).toBe("");
       }
 

test/fixtures/esm/test.js

@@ -1,13 +1,9 @@
 const getStack = () => new Error("Boo").stack;
 
 export default function test() {
   return {
     file: __filename,
     dir: __dirname,
     "import.meta.url": import.meta.url,
-    stack: getStack()
-      .split("\n")
-      .splice(1)
-      .map((s) => s.trim()),
   };
 }