Fix a bug related to parsing the same bytecode for two different co… (#…

…64) * Fixed a bug related to parsing the same bytecode for two different contracts * Updated CHANGELOG.md and versions * Clarified comment
dl-solarity · Dec 15, 2023 · 0f5330b · 0f5330b
1 parent d4e2b93
commit 0f5330b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Version 2.0.0-alpha.21
+
+* Fix a bug related to parsing the same bytecode for two different contracts
+
 ## Version 2.0.0-alpha.20
 
 * Remove redundant initializers in a verifier task

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@solarity/hardhat-migrate",
-  "version": "2.0.0-alpha.20",
+  "version": "2.0.0-alpha.21",
   "description": "Automatic deployment and verification of smart contracts",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",

diff --git a/src/tools/storage/ArtifactProcessor.ts b/src/tools/storage/ArtifactProcessor.ts
@@ -25,7 +25,10 @@ export class ArtifactProcessor {
       }
 
       ArtifactStorage.set(name, contract);
-      ArtifactStorage.set(bytecodeHash(artifact.bytecode), contract);
+
+      // Forcing the overwriting of the existing bytecode value in the Artifacts storage is necessary to prevent
+      // failure in edge cases, such as with internal libraries or disabled bytecode metadata hash generation.
+      ArtifactStorage.set(bytecodeHash(artifact.bytecode), contract, artifact.abi.length === 0);
     }
   }