ECMAScript module (ESM) support

- change compiler options to NodeNext
- add .js extensions to all imports
- tweak webpack configs (remove fullySpecified: false)

Author: john-e-riordan

RELEASE.md

@@ -77,5 +77,11 @@ Instead whitelist files in `package.json`. It's better to be exact about what we
 
 ## Update release notes on GitHub
 
+Build the bundle files to upload.
+
+`npm run build-bundles`
+
 * Find 'Releases' on repository page
 * Draft and publish release notes for new tag
+* Upload bundles for new verison
+

build/grammarGenerator.cjs

@@ -52,7 +52,7 @@ const parser = pegjs.generate(grammarContents, {
   plugins: [tspegjs],
   "tspegjs": {
     "tslintIgnores": "interface-name, trailing-comma, object-literal-sort-keys, max-line-length, only-arrow-functions, one-variable-per-declaration, no-consecutive-blank-lines, align, radix, quotemark, semicolon, object-literal-shorthand, variable-name, no-var-keyword, whitespace, curly, prefer-const, object-literal-key-quotes, no-string-literal, one-line, no-unused-expression, space-before-function-paren, arrow-return-shorthand",
-    "customHeader": "import { NameAddrHeader } from \"../../name-addr-header\";\nimport { URI } from \"../../uri\";"
+    "customHeader": "import { NameAddrHeader } from \"../../name-addr-header.js\";\nimport { URI } from \"../../uri.js\";"
   },
   "returnTypes": {
     Contact: "URI | NameAddrHeader",

build/webpack.config.cjs

@@ -72,7 +72,11 @@ module.exports = function (env) {
       ]
     },
     resolve: {
-      extensions: ['.ts', '.d.ts', '.js']
+      extensions: ['.ts', '.d.ts', '.js'],
+      extensionAlias: {
+        '.js': ['.ts', '.js'],
+        '.mjs': ['.mts', '.mjs']
+      }
     },
     optimization: {
       minimizer: [

demo/demo-1.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-use-before-define */
 /* eslint-disable no-console */
-import { SimpleUser, SimpleUserDelegate, SimpleUserOptions } from "../src/platform/web";
-import { getAudio, getButton, getButtons, getInput, getSpan } from "./demo-utils";
+import { SimpleUser, SimpleUserDelegate, SimpleUserOptions } from "../lib/platform/web/index.js";
+import { getAudio, getButton, getButtons, getInput, getSpan } from "./demo-utils.js";
 
 const serverSpan = getSpan("server");
 const targetSpan = getSpan("target");

demo/demo-2.ts

@@ -2,9 +2,9 @@
 /* eslint-disable @typescript-eslint/explicit-function-return-type */
 /* eslint-disable no-console */
 /* eslint-disable @typescript-eslint/no-use-before-define */
-import { SimpleUser, SimpleUserDelegate, SimpleUserOptions } from "../src/platform/web";
-import { nameAlice, nameBob, uriAlice, uriBob, webSocketServerAlice, webSocketServerBob } from "./demo-users";
-import { getButton, getInput, getVideo } from "./demo-utils";
+import { SimpleUser, SimpleUserDelegate, SimpleUserOptions } from "../lib/platform/web/index.js";
+import { nameAlice, nameBob, uriAlice, uriBob, webSocketServerAlice, webSocketServerBob } from "./demo-users.js";
+import { getButton, getInput, getVideo } from "./demo-utils.js";
 
 const connectAlice = getButton("connectAlice");
 const connectBob = getButton("connectBob");

demo/demo-3.ts

@@ -7,9 +7,9 @@ import {
   SimpleUser,
   SimpleUserDelegate,
   SimpleUserOptions
-} from "../src/platform/web";
-import { nameAlice, nameBob, uriAlice, uriBob, webSocketServerAlice, webSocketServerBob } from "./demo-users";
-import { getButton, getDiv, getInput } from "./demo-utils";
+} from "../lib/platform/web/index.js";
+import { nameAlice, nameBob, uriAlice, uriBob, webSocketServerAlice, webSocketServerBob } from "./demo-users.js";
+import { getButton, getDiv, getInput } from "./demo-utils.js";
 
 // A class which extends SimpleUser to handle setup and use of a data channel
 class SimpleUserWithDataChannel extends SimpleUser {

demo/webpack.config.cjs

@@ -18,7 +18,11 @@ module.exports = {
     ]
   },
   resolve: {
-    extensions: [ '.tsx', '.ts', '.js' ]
+    extensions: [ '.tsx', '.ts', '.js' ],
+    extensionAlias: {
+      '.js': ['.ts', '.js'],
+      '.mjs': ['.mts', '.mjs'],
+    }
   },
   output: {
     filename: '[name].js',

karma.conf.cjs

@@ -24,11 +24,6 @@ module.exports = function(config) {
 
     webpack: {
       devtool: 'inline-source-map',
-      module: { rules: [ { test: /\.m?js/,
-        resolve: {
-	    fullySpecified: false
-	}
-      }]},
       mode: 'production'
     },
 

package.json

@@ -66,13 +66,13 @@
   },
   "scripts": {
     "prebuild": "npm run lint",
-    "build-bundles": "npm run build-reg-bundle && npm run build-min-bundle",
+    "build-bundles": "npm run build-reg-bundle && npm run build-min-bundle && npm run copy-dist-files",
     "build-reg-bundle": "webpack --progress --config build/webpack.config.cjs --env buildType=reg",
     "build-min-bundle": "webpack --progress --config build/webpack.config.cjs --env buildType=min",
-    "build-demo": "npm run generate-grammar && webpack --progress --config demo/webpack.config.cjs",
+    "build-demo": "npm run build && webpack --progress --config demo/webpack.config.cjs",
     "build-lib": "tsc -p src",
     "build-test": "tsc -p test",
-    "build": "npm run generate-grammar && npm run build-lib && npm run build-reg-bundle && npm run build-min-bundle && npm run copy-dist-files",
+    "build": "npm run generate-grammar && npm run build-lib",
     "build-and-test": "npm run build && npm run test",
     "build-and-test-browser": "npm run build && npm run test-browser",
     "build-docs-api": "npm run extract-api && npm run document-api && npm run document-copy-api",

src/api/ack.ts

@@ -1,4 +1,5 @@
-import { IncomingRequestMessage, IncomingAckRequest } from "../core";
+import { IncomingAckRequest } from "../core/messages/methods/ack.js";
+import { IncomingRequestMessage } from "../core/messages/incoming-request-message.js";
 
 /**
  * A request to confirm a {@link Session} (incoming ACK).