Merge pull request #3406 from NomicFoundation/fix-solcjs-in-bundled-h…

…ardhat Make solcjs compilation work when Hardhat is bundled
NomicFoundation · Dec 8, 2022 · 42d38ea · 42d38ea
2 parents 41f5e7b + 7154371
commit 42d38ea
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/.changeset/orange-jobs-hug.md b/.changeset/orange-jobs-hug.md
@@ -0,0 +1,5 @@
+---
+"hardhat": patch
+---
+
+Fixed an issue that caused compilation with solcjs to not work when Hardhat is bundled
diff --git a/packages/hardhat-core/src/internal/solidity/compiler/index.ts b/packages/hardhat-core/src/internal/solidity/compiler/index.ts
@@ -41,6 +41,14 @@ export class Compiler implements ICompiler {
    */
   private _loadCompilerSources(compilerPath: string) {
     const Module = module.constructor as any;
+
+    // if Hardhat is bundled (for example, in the vscode extension), then
+    // Module._extenions might be undefined. In that case, we just use a plain
+    // require.
+    if (Module._extensions === undefined) {
+      return require(compilerPath);
+    }
+
     const previousHook = Module._extensions[".js"];
 
     Module._extensions[".js"] = function (