Merge pull request #81 from foundry-rs/develop

hardhat-forge: configurable artifact output (#80)
foundry-rs · Jul 6, 2022 · db59298 · db59298
2 parents 748e467 + 2d33cef
commit db59298
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 3 deletions.
diff --git a/.changeset/seven-bees-itch.md b/.changeset/seven-bees-itch.md
@@ -0,0 +1,5 @@
+---
+"@foundry-rs/hardhat-forge": patch
+---
+
+Add config for writing artifacts to allow conditional writing of artifacts
diff --git a/packages/hardhat-forge/package.json b/packages/hardhat-forge/package.json
@@ -23,7 +23,7 @@
     "prettier": "prettier \"**/*.{js,md}\"",
     "test": "mocha --recursive \"test/**/*.ts\" --exit",
     "build": "tsc --build .",
-    "clean": "rimraf dist"
+    "clean": "rimraf dist test/fixture-projects/hardhat-project/{artifacts,cache,out}"
   },
   "files": [
     "dist/",

diff --git a/packages/hardhat-forge/src/forge/build/index.ts b/packages/hardhat-forge/src/forge/build/index.ts
@@ -18,6 +18,11 @@ registerProjectPathArgs(registerCompilerArgs(task("compile")))
     const input = { ...args, ...(hre.config.foundry || {}) };
     const buildArgs = await getCheckedArgs(input);
     await spawnBuild(buildArgs);
+
+    if (hre.config.foundry?.writeArtifacts!) {
+      (hre as any).artifacts.writeArtifactsSync();
+    }
+
     if (hre.config.foundry?.runSuper!) {
       await runSuper(args);
     }

diff --git a/packages/hardhat-forge/src/forge/types.ts b/packages/hardhat-forge/src/forge/types.ts
@@ -7,6 +7,7 @@ export interface FoundryHardhatConfig
   extends Partial<ForgeEvmArgs>,
     Partial<ForgeBuildArgs> {
   runSuper?: boolean;
+  writeArtifacts?: boolean;
 }
 
 declare module "hardhat/types/config" {

diff --git a/packages/hardhat-forge/src/index.ts b/packages/hardhat-forge/src/index.ts
@@ -29,7 +29,6 @@ extendEnvironment((hre: HardhatRuntimeEnvironment) => {
       config.build_info
     );
 
-    artifacts.writeArtifactsSync();
     return artifacts;
   });
 });
@@ -40,6 +39,7 @@ extendConfig(
       // Set default values then merge user defined values
       return {
         runSuper: false,
+        writeArtifacts: true,
         ...userConfig.foundry,
       };
     });

diff --git a/packages/hardhat-forge/test/fixture-projects/hardhat-project/hardhat.config.ts b/packages/hardhat-forge/test/fixture-projects/hardhat-project/hardhat.config.ts
@@ -7,7 +7,8 @@ const config: HardhatUserConfig = {
   solidity: "0.7.3",
   defaultNetwork: "hardhat",
   foundry: {
-    viaIr: true
+    viaIr: true,
+    writeArtifacts: true,
   },
 };