pin esbuild

Author: kuhe

package.json

@@ -39,7 +39,7 @@
     "@typescript-eslint/parser": "4.30.0",
     "chai": "^4.2.0",
     "chai-as-promised": "^7.1.1",
-    "esbuild": "^0.19.9",
+    "esbuild": "0.19.11",
     "eslint": "7.32.0",
     "eslint-config-prettier": "8.3.0",
     "eslint-plugin-prettier": "3.4.1",

scripts/compilation/Inliner.js

@@ -138,18 +138,9 @@ module.exports = class Inliner {
       allowOverwrite: true,
       entryPoints: [path.join(root, this.subfolder, this.package, "src", "index.ts")],
       outfile: this.outfile,
-      external: [
-        "tslib",
-        "@aws-crypto/*",
-        "@smithy/*",
-        "@aws-sdk/*",
-        "typescript",
-        "vscode-oniguruma",
-        "pnpapi",
-        "fast-xml-parser",
-        "node_modules/*",
-        ...this.variantExternalsForEsBuild,
-      ],
+      keepNames: true,
+      packages: "external",
+      external: [...this.variantExternalsForEsBuild],
     });
     return this;
   }
@@ -161,6 +152,12 @@ module.exports = class Inliner {
   async rewriteStubs() {
     for await (const file of walk(path.join(this.packageDirectory, "dist-cjs"))) {
       const relativePath = file.replace(path.join(this.packageDirectory, "dist-cjs"), "").slice(1);
+
+      if (!file.endsWith(".js")) {
+        console.log("Skipping", path.basename(file), "file extension is not .js.");
+        continue;
+      }
+
       if (relativePath === "index.js") {
         console.log("Skipping index.js");
         continue;
@@ -198,12 +195,12 @@ module.exports = class Inliner {
       const basename = path.basename(variant).replace(/.js$/, "");
       const dirname = path.dirname(variant);
 
-      const find = new RegExp(`require\\("./(.*?)/${basename}"\\)`);
+      const find = new RegExp(`require\\("\\.(.*?)/${basename}"\\)`);
       const replace = `require("./${dirname}/${basename}")`;
 
       this.indexContents = this.indexContents.replace(find, replace);
 
-      console.log("replacing", find, "with", replace);
+      console.log("Replacing", find, "with", replace);
     }
 
     fs.writeFileSync(this.outfile, this.indexContents, "utf-8");
@@ -235,7 +232,7 @@ module.exports = class Inliner {
           const redundantVariable = `import_${variableSuffix}${redundancyIndex}`;
 
           if (this.indexContents.match(new RegExp(redundantRequire))) {
-            console.log("replacing", redundantVariable);
+            console.log("Replacing var", redundantVariable);
             this.indexContents = this.indexContents
               .replace(new RegExp(redundantRequire, "g"), "")
               .replace(new RegExp(redundantVariable, "g"), `import_${variableSuffix}`);

yarn.lock

@@ -5430,7 +5430,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"esbuild@npm:^0.19.9":
+"esbuild@npm:0.19.11":
   version: 0.19.11
   resolution: "esbuild@npm:0.19.11"
   dependencies:
@@ -9966,7 +9966,7 @@ __metadata:
     "@typescript-eslint/parser": 4.30.0
     chai: ^4.2.0
     chai-as-promised: ^7.1.1
-    esbuild: ^0.19.9
+    esbuild: 0.19.11
     eslint: 7.32.0
     eslint-config-prettier: 8.3.0
     eslint-plugin-prettier: 3.4.1