build!: migrate to type: "module" (#137)

Author: toyobayashi

.eslintignore

@@ -3,6 +3,7 @@
 node_modules
 /lib/typings
 .eslintrc.js
+.eslintrc.cjs
 /example
 /test/.build
 /test/CMakeFiles
@@ -13,4 +14,4 @@ node_modules
 /packages/**/test/**/actual
 /packages/**/test/**/expected
 /packages/**/test/**/rollup/src
-/out
+out

.eslintrc.js renamed to .eslintrc.cjs

@@ -4,7 +4,7 @@ const sharedRules = {
   'no-new-func': 'off',
   'no-implied-eval': 'off',
   'no-var': 'off',
-  'camelcase': 'off'
+  camelcase: 'off'
 }
 
 module.exports = {
@@ -72,7 +72,7 @@ module.exports = {
         project: './tsconfig.json',
         tsconfigRootDir: __dirname,
         createDefaultProgram: true
-      },
+      }
     }
   ]
 }

.vscode/settings.json

@@ -1,6 +1,6 @@
 {
   "files.associations": {
-    "**/api-extractor.json": "jsonc"
+    "**/api-extractor*.json": "jsonc"
   },
   "typescript.tsdk": "./node_modules/typescript/lib",
   "cmake.sourceDirectory": "${workspaceFolder}/packages/test",
@@ -17,5 +17,9 @@
     "launch": {
       "statusBarVisibility": "hidden"
     }
+  },
+  "livePreview.httpHeaders": {
+    "Cross-Origin-Embedder-Policy": "require-corp",
+    "Cross-Origin-Opener-Policy": "same-origin"
   }
 }

CONTRIBUTING.md

@@ -55,7 +55,7 @@ This doc will explain the structure of this project and some points need to note
 
     Macros are powered by `packages/ts-transform-macro`
 
-- `packages/core` (`dependencies`)
+- `packages/core` (`dependencies`, follow emnapi version)
 
     This package is designed for using emnapi on non-Emscripten platform, it is a trasformed output of `packages/emnapi`
     by using `packages/ts-transform-emscripten-parse-tools`.
@@ -64,15 +64,25 @@ This doc will explain the structure of this project and some points need to note
     We need to manually provide imported symbols to the second parameter of `WebAssembly.instantiate`,
     so this is the use case of this package.
 
-- `packages/runtime` (`dependencies`)
+- `packages/runtime` (`dependencies`, follow emnapi version)
 
     Provide runtime implementation of `napi_value` / `napi_handle_scope` / `napi_ref` etc.
 
-- `packages/node` (`dependencies`)
+- `packages/node` (`dependencies`, follow emnapi version)
 
     Some APIs make sense on Node.js only, such as `napi_async_init`, `napi_async_destroy` and `napi_make_callback`.
     This package provide native Node.js bindings for JavaScript implementation as bridge.
 
+- `packages/rollup-plugin-emscripten-esm-library` (`devDependencies`, independent)
+
+- `packages/ts-transform-emscripten-esm-library` (`devDependencies`, independent)
+
+- `packages/ts-transform-emscripten-parse-tools` (`devDependencies`, follow emnapi version, private)
+
+- `packages/ts-transform-emscripten-parse-tools` (`devDependencies`, independent)
+
+- `packages/wasi-threads` (`dependencies`, independent)
+
 ## Environment Requirements
 
 - Node.js `>= 16`

package.json

@@ -1,6 +1,7 @@
 {
   "private": true,
   "version": "0.0.0",
+  "type": "module",
   "scripts": {
     "prepare": "npm run build",
     "build": "npm run build --workspaces --if-present",
@@ -53,6 +54,7 @@
     "typescript": "~5.7.2"
   },
   "workspaces": [
+    "packages/shared",
     "packages/ts-transform-macro",
     "packages/ts-transform-emscripten-esm-library",
     "packages/ts-transform-emscripten-parse-tools",

packages/core/.npmignore

@@ -7,3 +7,4 @@ node_modules
 .npmignore
 api-extractor.json
 tsconfig.json
+/dist/types

packages/core/api-extractor.json

@@ -1,115 +1,3 @@
 {
-  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
-
-  // "extends": "./shared/api-extractor-base.json"
-  // "extends": "my-package/include/api-extractor-base.json"
-
-  "projectFolder": ".",
-
-  "mainEntryPointFilePath": "<projectFolder>/lib/typings/index.d.ts",
-
-  "bundledPackages": [],
-
-  "compiler": {
-    "tsconfigFilePath": "<projectFolder>/tsconfig.json"
-
-    // "overrideTsconfig": {
-    //   . . .
-    // }
-
-    // "skipLibCheck": true,
-  },
-
-  "apiReport": {
-    "enabled": false
-
-    // "reportFileName": "<unscopedPackageName>.api.md",
-
-    // "reportFolder": "<projectFolder>/etc/",
-
-    // "reportTempFolder": "<projectFolder>/api/temp/"
-  },
-
-  "docModel": {
-    "enabled": false
-
-    // "apiJsonFilePath": "<projectFolder>/temp/<unscopedPackageName>.api.json"
-  },
-
-  "dtsRollup": {
-    "enabled": true,
-
-    "untrimmedFilePath": "",
-
-    // "betaTrimmedFilePath": "<projectFolder>/dist/<unscopedPackageName>-beta.d.ts",
-
-    "publicTrimmedFilePath": "<projectFolder>/dist/emnapi-core.d.ts"
-
-    // "omitTrimmingComments": true
-  },
-
-  "tsdocMetadata": {
-    "enabled": false,
-
-    "tsdocMetadataFilePath": "<projectFolder>/dist/tsdoc-metadata.json"
-  },
-
-  // "newlineKind": "crlf",
-
-  "messages": {
-    /**
-     * Configures handling of diagnostic messages reported by the TypeScript compiler engine while analyzing
-     * the input .d.ts files.
-     *
-     * TypeScript message identifiers start with "TS" followed by an integer.  For example: "TS2551"
-     *
-     * DEFAULT VALUE:  A single "default" entry with logLevel=warning.
-     */
-    "compilerMessageReporting": {
-      "default": {
-        "logLevel": "warning"
-
-        // "addToApiReportFile": false
-      }
-
-      // "TS2551": {
-      //   "logLevel": "warning",
-      //   "addToApiReportFile": true
-      // },
-      //
-      // . . .
-    },
-
-    "extractorMessageReporting": {
-      "default": {
-        "logLevel": "warning"
-        // "addToApiReportFile": false
-      },
-      "ae-missing-release-tag": {
-        "logLevel": "none",
-        "addToApiReportFile": false
-      }
-
-      // "ae-extra-release-tag": {
-      //   "logLevel": "warning",
-      //   "addToApiReportFile": true
-      // },
-      //
-      // . . .
-    },
-
-    "tsdocMessageReporting": {
-      "default": {
-        "logLevel": "warning"
-        // "addToApiReportFile": false
-      }
-
-      // "tsdoc-link-tag-unescaped-text": {
-      //   "logLevel": "warning",
-      //   "addToApiReportFile": true
-      // },
-      //
-      // . . .
-    }
-  }
+  "extends": "../shared/api-extractor-base.json"
 }

packages/core/index.js

@@ -2,36 +2,32 @@
   "name": "@emnapi/core",
   "version": "1.4.0",
   "description": "emnapi core",
-  "main": "index.js",
-  "module": "./dist/emnapi-core.esm-bundler.js",
-  "types": "./dist/emnapi-core.d.ts",
+  "type": "module",
+  "main": "./dist/emnapi-core.cjs",
+  "module": "./dist/emnapi-core.js",
+  "types": "./dist/emnapi-core.d.cts",
   "sideEffects": false,
   "exports": {
     ".": {
       "types": {
         "module": "./dist/emnapi-core.d.ts",
-        "import": "./dist/emnapi-core.d.mts",
-        "default": "./dist/emnapi-core.d.ts"
+        "module-sync": "./dist/emnapi-core.d.ts",
+        "import": "./dist/emnapi-core.d.cts",
+        "require": "./dist/emnapi-core.d.cts",
+        "default": "./dist/emnapi-core.d.cts"
       },
-      "module": "./dist/emnapi-core.esm-bundler.js",
-      "import": "./dist/emnapi-core.mjs",
-      "default": "./index.js"
+      "module": "./dist/emnapi-core.js",
+      "module-sync": "./dist/emnapi-core.js",
+      "default": "./dist/emnapi-core.cjs"
     },
-    "./dist/emnapi-core.cjs.min": {
-      "types": "./dist/emnapi-core.d.ts",
-      "default": "./dist/emnapi-core.cjs.min.js"
-    },
-    "./dist/emnapi-core.min.mjs": {
-      "types": "./dist/emnapi-core.d.mts",
-      "default": "./dist/emnapi-core.min.mjs"
-    }
+    "./package.json": "./package.json"
   },
   "dependencies": {
     "@emnapi/wasi-threads": "1.0.1",
     "tslib": "^2.4.0"
   },
   "scripts": {
-    "build": "node ./script/build.js"
+    "build": "node ./script/build.js && rollup -c"
   },
   "repository": {
     "type": "git",

packages/core/package.json

@@ -0,0 +1,10 @@
+import { defineConfig } from '@emnapi/shared'
+import pkg from './package.json' with { type: "json" }
+
+export default defineConfig({
+  outputName: 'emnapiCore',
+  outputFile: 'emnapi-core',
+  defines: {
+    __VERSION__: JSON.stringify(pkg.version)
+  }
+})