Add support for transpiling ESM code (and convert this repo itself to…

… ESM)

.eslintrc.cjs

@@ -0,0 +1,48 @@
+module.exports = {
+  extends: "@gadgetinc/eslint-config",
+  parserOptions: {
+    tsconfigRootDir: __dirname,
+    project: ["./tsconfig.test.json"],
+  },
+  ignorePatterns: [".eslintrc.cjs", "spec/fixtures/**/*"],
+  rules: {
+    "lodash/import-scope": "off",
+    // Disable all Jest-related ESLint rules
+    "jest/no-alias-methods": "off",
+    "jest/no-commented-out-tests": "off",
+    "jest/no-conditional-expect": "off",
+    "jest/no-deprecated-functions": "off",
+    "jest/no-disabled-tests": "off",
+    "jest/no-done-callback": "off",
+    "jest/no-duplicate-hooks": "off",
+    "jest/no-export": "off",
+    "jest/no-focused-tests": "off",
+    "jest/no-hooks": "off",
+    "jest/no-identical-title": "off",
+    "jest/no-if": "off",
+    "jest/no-interpolation-in-snapshots": "off",
+    "jest/no-jasmine-globals": "off",
+    "jest/no-jest-import": "off",
+    "jest/no-large-snapshots": "off",
+    "jest/no-mocks-import": "off",
+    "jest/no-restricted-matchers": "off",
+    "jest/no-standalone-expect": "off",
+    "jest/no-test-prefixes": "off",
+    "jest/no-test-return-statement": "off",
+    "jest/prefer-called-with": "off",
+    "jest/prefer-expect-assertions": "off",
+    "jest/prefer-hooks-on-top": "off",
+    "jest/prefer-spy-on": "off",
+    "jest/prefer-strict-equal": "off",
+    "jest/prefer-to-be": "off",
+    "jest/prefer-to-contain": "off",
+    "jest/prefer-to-have-length": "off",
+    "jest/prefer-todo": "off",
+    "jest/require-to-throw-message": "off",
+    "jest/require-top-level-describe": "off",
+    "jest/valid-describe-callback": "off",
+    "jest/valid-expect": "off",
+    "jest/valid-expect-in-promise": "off",
+    "jest/valid-title": "off"
+  },
+};

.github/workflows/test.yml

@@ -13,8 +13,8 @@ jobs:
       - uses: actions/checkout@v2
       - uses: ./.github/actions/setup-test-env
       - run: pnpm build
-      - run: integration-test/test.sh
-      - run: pnpm jest
+      - run: pnpm zx integration-test/test.js
+      - run: pnpm test
 
   lint:
     runs-on: ubuntu-latest

.swcrc

@@ -9,7 +9,6 @@
     "target": "esnext"
   },
   "module": {
-    "type": "commonjs",
-    "lazy": true
+    "type": "es6"
   }
 }

flake.nix

@@ -21,7 +21,7 @@
           packages =
             rec {
               bash = pkgs.bash;
-              nodejs = pkgs.nodejs_20;
+              nodejs = pkgs.nodejs_22;
               corepack = pkgs.corepack;
             };
 

gitpkg.config.js

@@ -1,6 +1,6 @@
-const { execSync } = require('child_process')
+import { execSync } from "child_process";
 
-module.exports = () => ({
+export default () => ({
   getTagName: (pkg) =>
     `${pkg.name}-v${pkg.version}-gitpkg-${execSync(
       'git rev-parse --short HEAD',

integration-test/cjs-js-from-ts/.swcrc

@@ -0,0 +1,14 @@
+{
+  "jsc": {
+    "parser": {
+      "syntax": "typescript",
+      "tsx": true,
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "target": "esnext"
+  },
+  "module": {
+    "type": "commonjs"
+  }
+}

integration-test/cjs-js-from-ts/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "cjs-js-from-ts",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "commonjs",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/cjs-js-from-ts/utils.js

@@ -0,0 +1 @@
+module.exports.utility = (str) => str.toUpperCase();

integration-test/cjs-sourcemap/.swcrc

@@ -0,0 +1,14 @@
+{
+  "jsc": {
+    "parser": {
+      "syntax": "typescript",
+      "tsx": true,
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "target": "esnext"
+  },
+  "module": {
+    "type": "commonjs"
+  }
+}

integration-test/cjs-sourcemap/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "cjs-sourcemap",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "commonjs",
+  "license": "ISC"
+}

integration-test/sourcemap/test.sh → integration-test/cjs-sourcemap/test.sh

@@ -2,5 +2,5 @@
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
 set -ex
 
-$DIR/../../pkg/wds.bin.js $DIR/run.ts 2>&1 | grep "sourcemap/utils.ts:7"
+$DIR/../../pkg/wds.bin.js $DIR/run.ts 2>&1 | tee /dev/stderr | grep "sourcemap/utils.ts:7"
 echo "Found correct source location"

integration-test/cjs-ts-from-js/.swcrc

@@ -0,0 +1,14 @@
+{
+  "jsc": {
+    "parser": {
+      "syntax": "typescript",
+      "tsx": true,
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "target": "esnext"
+  },
+  "module": {
+    "type": "commonjs"
+  }
+}

integration-test/cjs-ts-from-js/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "esm-ts-from-js",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "commonjs",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/cjs-ts-from-js/run.js

@@ -0,0 +1,3 @@
+const { utility } = require("./utils");
+
+console.log(utility("It worked!"));

integration-test/cjs-ts-from-js/test.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $@ $DIR/run.js | grep "IT WORKED"

integration-test/cjs-ts-from-ts/.swcrc

@@ -0,0 +1,14 @@
+{
+  "jsc": {
+    "parser": {
+      "syntax": "typescript",
+      "tsx": true,
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "target": "esnext"
+  },
+  "module": {
+    "type": "commonjs"
+  }
+}

integration-test/cjs-ts-from-ts/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "simple-esm",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "commonjs",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/cjs-ts-from-ts/run.ts

@@ -0,0 +1,3 @@
+import { utility } from "./utils";
+
+console.log(utility("It worked!"));

integration-test/cjs-ts-from-ts/test.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $@ $DIR/run.ts | grep "IT WORKED"

integration-test/cjs-ts-from-ts/utils.ts

@@ -0,0 +1 @@
+export const utility = (str: string) => str.toUpperCase();

integration-test/esm-js-from-ts/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "esm-js-from-ts",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "module",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/esm-js-from-ts/run.ts

@@ -0,0 +1,3 @@
+import { utility } from "./utils.js";
+
+console.log(utility("It worked!"));

integration-test/esm-js-from-ts/test.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $@ $DIR/run.ts | grep "IT WORKED"

integration-test/esm-js-from-ts/utils.js

@@ -0,0 +1 @@
+export const utility = (str) => str.toUpperCase();

integration-test/sourcemap/package.json → integration-test/esm-sourcemap/package.json

@@ -1,7 +1,8 @@
 {
-  "name": "sourcemap",
+  "name": "esm-sourcemap",
   "version": "1.0.0",
   "description": "",
   "main": "index.js",
+  "type": "module",
   "license": "ISC"
 }

integration-test/esm-sourcemap/run.ts

@@ -0,0 +1,3 @@
+import { utility } from "./utils.js";
+
+console.log(utility("It worked!"));

integration-test/esm-sourcemap/test.sh

@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $DIR/run.ts 2>&1 | tee /dev/stderr | grep "sourcemap/utils.ts:7"
+echo "Found correct source location"

integration-test/esm-sourcemap/utils.ts

@@ -0,0 +1,8 @@
+// add some lines that move the source lines around but not the output lines
+export type Whatever = any;
+
+/** A nice util */
+export const utility = (str: string) => {
+  // this is on line 7 which we look for in the tests
+  throw new Error("error in utils");
+};

integration-test/esm-ts-from-js/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "esm-ts-from-js",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "module",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/esm-ts-from-js/run.js

@@ -0,0 +1,3 @@
+import { utility } from "./utils.js";
+
+console.log(utility("It worked!"));

integration-test/esm-ts-from-js/test.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $@ $DIR/run.js | grep "IT WORKED"

integration-test/esm-ts-from-js/utils.ts

@@ -0,0 +1 @@
+export const utility = (str: string) => str.toUpperCase();

integration-test/esm-ts-from-ts/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "simple-esm",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "type": "module",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "license": "ISC"
+}

integration-test/esm-ts-from-ts/run.ts

@@ -0,0 +1,3 @@
+import { utility } from "./utils.js";
+
+console.log(utility("It worked!"));

integration-test/esm-ts-from-ts/test.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+set -ex
+
+$DIR/../../pkg/wds.bin.js $@ $DIR/run.ts | grep "IT WORKED"

integration-test/esm-ts-from-ts/utils.ts

@@ -0,0 +1 @@
+export const utility = (str: string) => str.toUpperCase();

integration-test/oom/run.ts

@@ -1,4 +1,4 @@
 const leak = [];
 while (true) {
-  leak.push("consuming memory");
+  leak.push("consuming memory".repeat(10000));
 }

integration-test/oom/test.sh

@@ -2,6 +2,6 @@
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
 set -ex
 
-$DIR/../../pkg/wds.bin.js $@ --max-old-space-size=50 $DIR/run.ts 2>&1 | grep "ReportOOMFailure"
+$DIR/../../pkg/wds.bin.js $@ --max-old-space-size=50 $DIR/run.ts 2>&1 | grep -E "ReportOOMFailure|JavaScript heap out of memory"
 
 echo "found OOM failure"

integration-test/reload/run-scratch.ts

@@ -1,4 +1,4 @@
-const http = require("http");
+import http from "http";
 
 const requestListener = function (req, res) {
   res.writeHead(200);