BREAKING CHANGE: ESM, JS bindings, triple equals and general cleanup

.eslintignore

@@ -1,5 +1,4 @@
 dist/
-docs/
 lib/binaryen.js
 lib/parse/index.js
 out/

.eslintrc.js → .eslintrc.cjs

@@ -1,3 +1,5 @@
+/* global module */
+
 module.exports = {
   root: true,
   parser: "@typescript-eslint/parser",
@@ -15,9 +17,9 @@ module.exports = {
     ecmaFeatures: {}
   },
   globals: {
+    "globalThis": "readonly",
     "BigInt64Array": "readonly",
-    "BigUint64Array": "readonly",
-    "__non_webpack_require__": "readonly"
+    "BigUint64Array": "readonly"
   },
 
   // === General rules =========================================================
@@ -218,8 +220,7 @@ module.exports = {
 
     {
       files: [
-        "./index.d.ts",
-        "./index.release.d.ts",
+        "./dist/*.d.ts"
       ],
       rules: {
         // Our definitions are complicated, and all attempts to describe them

.gitattributes

@@ -1,5 +1,4 @@
 bin/* text eol=lf
 dist/* binary
 scripts/*.sh eol=lf
-lib/binaryen.js binary
 tests/compiler/std/string-encoding.ts eol=lf

.github/workflows/publish.yml

@@ -47,6 +47,7 @@ jobs:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
       run: |
+        node ./scripts/prepublish
         if [ $(node -pe "require('./package.json').version") != "0.0.0" ]; then
           npx aspublish
         fi
@@ -59,3 +60,4 @@ jobs:
           npm publish --access public
         fi
         cd ../..
+        node ./scripts/prepublish --reset

.github/workflows/test.yml

@@ -20,12 +20,13 @@ jobs:
         else
           printf "\nOK: Distributions files have not been modified.\n";
         fi
-  test:
-    name: "Compiler (Linux, node ${{ matrix.node_version }})"
-    runs-on: ubuntu-latest
+  compiler:
+    name: "Compiler (${{ matrix.os }}, node ${{ matrix.node_version }})"
+    runs-on: ${{ matrix.os }}-latest
     needs: check
     strategy:
       matrix:
+        os: ["ubuntu", "macos", "windows"]
         node_version: ["current", "lts_latest"]
     steps:
     - uses: actions/[email protected]
@@ -34,86 +35,33 @@ jobs:
         node-version: ${{ matrix.node_version }}
     - name: Install dependencies
       run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
-    - name: Check sources
-      run: npm run check
-    - name: Test sources
-      run: npm test
-    - name: Build distribution files
-      run: npm run build
-    - name: Update entry file
-      run: npm run prepare-ci
-    - name: Test distribution
-      run: npm test
-    - name: Test browser build
-      run: node tests/browser-asc
-  test-windows:
-    name: "Compiler (Windows, node current)"
-    runs-on: windows-latest
-    needs: check
-    steps:
-    - uses: actions/[email protected]
-    - uses: dcodeIO/setup-node-nvm@master
-      with:
-        node-version: current
-    - name: Install dependencies
-      run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
-    - name: Test sources
-      run: npm test
-    - name: Build distribution files
-      run: npm run build
-    - name: Update entry file
-      run: npm run prepare-ci
-    - name: Test distribution
-      run: npm test
-    - name: Test browser build
-      run: node tests/browser-asc
-  test-macos:
-    name: "Compiler (MacOS, node current)"
-    runs-on: macos-latest
-    needs: check
-    steps:
-    - uses: actions/[email protected]
-    - uses: dcodeIO/setup-node-nvm@master
-      with:
-        node-version: current
-    - name: Install dependencies
-      run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
-    - name: Test sources
-      run: npm test
-    - name: Build distribution files
+    - name: Build
       run: npm run build
-    - name: Update entry file
-      run: npm run prepare-ci
-    - name: Test distribution
+    - name: Check
+      run: npm run check
+    - name: Test
       run: npm test
-    - name: Test browser build
-      run: node tests/browser-asc
-  test-bootstrap:
-    name: "Compiler (Bootstrap)"
+  bootstrap:
+    name: "Bootstrap (${{ matrix.target }})"
     runs-on: ubuntu-latest
     needs: check
+    strategy:
+      matrix:
+        target: ["untouched", "optimized"]
     steps:
     - uses: actions/[email protected]
     - uses: dcodeIO/setup-node-nvm@master
       with:
         node-version: current
     - name: Install dependencies
       run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
-    - name: Bootstrap the compiler
-      run: npm run bootstrap
-    - name: Run compiler tests (untouched-bootstrap)
-      run: npm run test:compiler -- --wasm out/assemblyscript.untouched-bootstrap.wasm
-    - name: Run compiler tests (optimized-bootstrap)
-      run: npm run test:compiler -- --wasm out/assemblyscript.optimized-bootstrap.wasm
-  test-features:
+    - name: Build
+      run: npm run build
+    - name: Bootstrap
+      run: npm run bootstrap:${{ matrix.target }}
+    - name: Test
+      run: npm run test:compiler -- --wasm out/assemblyscript.${{ matrix.target }}-bootstrap.wasm
+  features:
     name: "Features"
     runs-on: ubuntu-latest
     needs: check
@@ -125,14 +73,14 @@ jobs:
         node-version: 18.0.0-v8-canary20211115037fd7ae8d
     - name: Install dependencies
       run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
+    - name: Build
+      run: npm run build
     - name: Test experimental features
       env:
         ASC_FEATURES: threads,reference-types,bigint-integration,gc
       run: |
         npm run test:compiler rt/flags features/js-bigint-integration features/reference-types features/threads std-wasi/process std-wasi/crypto
-  test-runtimes:
+  runtimes:
     name: "Runtimes"
     runs-on: ubuntu-latest
     needs: check
@@ -143,8 +91,8 @@ jobs:
         node-version: current
     - name: Install dependencies
       run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
+    - name: Build
+      run: npm run build
     - name: Test default allocator
       run: |
         cd tests/allocators/default
@@ -157,7 +105,7 @@ jobs:
         npm run build
         cd ..
         npm test stub
-  test-loader:
+  loader:
     name: "Loader"
     runs-on: ubuntu-latest
     needs: check
@@ -168,8 +116,8 @@ jobs:
         node-version: current
     - name: Install dependencies
       run: npm ci --no-audit
-    - name: Clean distribution files
-      run: npm run clean
+    - name: Build
+      run: npm run build
     - name: Test the loader
       run: |
         cd lib/loader

.gitignore

@@ -1,10 +1,11 @@
-npm-debug.*
-dist/
-docs/
 node_modules/
+*debug.*
+dist/
 out/
 raw/
 .history
 *.backup
 .vscode
 .idea
+cli/index.generated.js
+src/diagnosticMessages.generated.ts

README.md

@@ -29,7 +29,7 @@
 
 <h2 align="center">Thanks to our sponsors!</h2>
 
-<p align="justify">Most of the core team members and most contributors do this open source work in their free time. If you use AssemblyScript for a serious task or plan to do so, and you'd like us to invest more time on it, <a href="https://opencollective.com/assemblyscript/donate" target="_blank" rel="noopener">please donate</a> to our <a href="https://opencollective.com/assemblyscript" target="_blank" rel="noopener">OpenCollective</a>. By sponsoring this project, your logo will show up below. Thank you so much for your support!</p>
+<p align="justify">Most of the maintainers and contributors do this open source work in their free time. If you use AssemblyScript for a serious task or plan to do so, and you'd like us to invest more time on it, <a href="https://opencollective.com/assemblyscript/donate" target="_blank" rel="noopener">please donate</a> to our <a href="https://opencollective.com/assemblyscript" target="_blank" rel="noopener">OpenCollective</a>. By sponsoring this project, your logo will show up below. Thank you so much for your support!</p>
 
 <p align="center">
   <a href="https://assemblyscript.org/#sponsors"><img src="https://assemblyscript.org/sponsors.svg" alt="Sponsor logos" width="720" /></a>

bin/asc.js

@@ -0,0 +1,32 @@
+#!/usr/bin/env node
+
+import childProcess from "child_process";
+
+const [ nodePath, thisPath, ...args ] = process.argv;
+const nodeArgs = process.execArgv;
+
+const hasSourceMaps = nodeArgs.includes("--enable-source-maps");
+const hasCustomArgs = args.includes("--");
+
+if (!hasSourceMaps || hasCustomArgs) {
+  if (!hasSourceMaps) {
+    nodeArgs.push("--enable-source-maps");
+  }
+  if (hasCustomArgs) {
+    const index = args.indexOf("--");
+    nodeArgs.push(...args.slice(index + 1));
+    args.length = index;
+  }
+  childProcess.spawnSync(
+    nodePath,
+    [...nodeArgs, thisPath, ...args],
+    { stdio: "inherit" }
+  );
+} else {
+  const asc = await import("../dist/asc.js");
+  const { error } = await asc.main(process.argv.slice(2), {
+    stdout: process.stdout,
+    stderr: process.stderr
+  });
+  if (error) process.exitCode = 1;
+}