chore: remove enableTransientStorage from EDR config

kanej · kanej · commit 8cb80da40c25 · 2025-07-24T10:46:47.000+01:00
This property was needed in Hardhat 2 but is no longer used. Resolves #6182.
diff --git a/v-next/hardhat/src/internal/builtin-plugins/network-manager/config-resolution.ts b/v-next/hardhat/src/internal/builtin-plugins/network-manager/config-resolution.ts
@@ -86,17 +86,13 @@ export function resolveEdrNetwork(
       networkConfig.allowUnlimitedContractSize ?? false,
     blockGasLimit: BigInt(networkConfig.blockGasLimit ?? 30_000_000n),
     coinbase: resolveCoinbase(networkConfig.coinbase),
-    enableTransientStorage: networkConfig.enableTransientStorage ?? false,
+
     forking: resolveForkingConfig(
       networkConfig.forking,
       cachePath,
       resolveConfigurationVariable,
     ),
-    hardfork: resolveHardfork(
-      networkConfig.hardfork,
-      networkConfig.chainType,
-      networkConfig.enableTransientStorage,
-    ),
+    hardfork: resolveHardfork(networkConfig.hardfork, networkConfig.chainType),
     initialBaseFeePerGas: resolveInitialBaseFeePerGas(
       networkConfig.initialBaseFeePerGas,
     ),
@@ -275,7 +271,6 @@ export async function resolveChainDescriptors(
 export function resolveHardfork(
   hardfork: string | undefined,
   chainType: ChainType | undefined = GENERIC_CHAIN_TYPE,
-  _enableTransientStorage: boolean | undefined,
 ): string {
   if (hardfork !== undefined) {
     return hardfork;
diff --git a/v-next/hardhat/src/internal/builtin-plugins/network-manager/type-extensions/config.ts b/v-next/hardhat/src/internal/builtin-plugins/network-manager/type-extensions/config.ts
@@ -108,7 +108,6 @@ declare module "../../../../types/config.js" {
     allowUnlimitedContractSize?: boolean;
     blockGasLimit?: number | bigint;
     coinbase?: string;
-    enableTransientStorage?: boolean;
     forking?: EdrNetworkForkingUserConfig;
     hardfork?: string;
     initialBaseFeePerGas?: number | bigint;
@@ -248,7 +247,6 @@ declare module "../../../../types/config.js" {
     allowUnlimitedContractSize: boolean;
     blockGasLimit: bigint;
     coinbase: Uint8Array;
-    enableTransientStorage: boolean;
     forking?: EdrNetworkForkingConfig;
     hardfork: string;
     initialBaseFeePerGas?: bigint;
diff --git a/v-next/hardhat/src/internal/builtin-plugins/network-manager/type-validation.ts b/v-next/hardhat/src/internal/builtin-plugins/network-manager/type-validation.ts
@@ -304,7 +304,6 @@ const edrNetworkUserConfigSchema = z.object({
   allowUnlimitedContractSize: z.optional(z.boolean()),
   blockGasLimit: z.optional(gasUnitUserConfigSchema),
   coinbase: z.optional(z.string()),
-  enableTransientStorage: z.optional(z.boolean()),
   forking: z.optional(edrNetworkForkingUserConfigSchema),
   hardfork: z.optional(z.string()),
   initialBaseFeePerGas: z.optional(gasUnitUserConfigSchema),
@@ -334,7 +333,6 @@ function refineEdrNetworkUserConfig(
       hardfork,
       minGasPrice,
       initialBaseFeePerGas,
-      enableTransientStorage,
     } = networkConfig;
 
     if (hardfork !== undefined && !isValidHardforkName(hardfork, chainType)) {
@@ -366,25 +364,6 @@ function refineEdrNetworkUserConfig(
           });
         }
       }
-
-      if (
-        !hardforkGte(resolvedHardfork, L1HardforkName.CANCUN, chainType) &&
-        enableTransientStorage === true
-      ) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          message: `'enableTransientStorage' is not supported for hardforks before 'cancun'. Please use a hardfork from 'cancun' onwards to enable this feature.`,
-        });
-      }
-      if (
-        hardforkGte(resolvedHardfork, L1HardforkName.CANCUN, chainType) &&
-        enableTransientStorage === false
-      ) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          message: `'enableTransientStorage' must be enabled for hardforks 'cancun' or later. To disable this feature, use a hardfork before 'cancun'.`,
-        });
-      }
     }
 
     if (
diff --git a/v-next/hardhat/test/internal/builtin-plugins/network-manager/config-resolution.ts b/v-next/hardhat/test/internal/builtin-plugins/network-manager/config-resolution.ts
@@ -123,7 +123,6 @@ describe("config-resolution", () => {
         allowBlocksWithSameTimestamp: true,
         allowUnlimitedContractSize: true,
         blockGasLimit: 20_000_000,
-        enableTransientStorage: true,
         initialDate: new Date(),
         loggingEnabled: true,
         minGasPrice: 10,
@@ -157,10 +156,6 @@ describe("config-resolution", () => {
         // eslint-disable-next-line @typescript-eslint/consistent-type-assertions -- cast for testing
         BigInt(userConfig.blockGasLimit as number),
       );
-      assert.equal(
-        edrNetworkConfig.enableTransientStorage,
-        userConfig.enableTransientStorage,
-      );
       assert.equal(edrNetworkConfig.initialDate, userConfig.initialDate);
       assert.equal(edrNetworkConfig.loggingEnabled, userConfig.loggingEnabled);
       assert.equal(
@@ -200,7 +195,6 @@ describe("config-resolution", () => {
       assert.equal(edrNetworkConfig.allowBlocksWithSameTimestamp, false);
       assert.equal(edrNetworkConfig.allowUnlimitedContractSize, false);
       assert.equal(edrNetworkConfig.blockGasLimit, 30_000_000n);
-      assert.equal(edrNetworkConfig.enableTransientStorage, false);
       const initialDate = new Date(edrNetworkConfig.initialDate);
       assert.ok(
         Math.abs(initialDate.getTime() - now.getTime()) < 1000,
@@ -770,25 +764,21 @@ describe("config-resolution", () => {
 
   describe("resolveHardfork", () => {
     it("should return the hardfork if it is provided", () => {
-      let hardfork = resolveHardfork(
-        L1HardforkName.LONDON,
-        L1_CHAIN_TYPE,
-        true,
-      );
+      let hardfork = resolveHardfork(L1HardforkName.LONDON, L1_CHAIN_TYPE);
       assert.equal(hardfork, L1HardforkName.LONDON);
 
-      hardfork = resolveHardfork(L1HardforkName.LONDON, L1_CHAIN_TYPE, false);
+      hardfork = resolveHardfork(L1HardforkName.LONDON, L1_CHAIN_TYPE);
       assert.equal(hardfork, L1HardforkName.LONDON);
     });
 
     it("should return the current hardfork if no hardfork is provided", () => {
-      let hardfork = resolveHardfork(undefined, L1_CHAIN_TYPE, true);
+      let hardfork = resolveHardfork(undefined, L1_CHAIN_TYPE);
       assert.equal(hardfork, getCurrentHardfork(L1_CHAIN_TYPE));
 
-      hardfork = resolveHardfork(undefined, undefined, true);
+      hardfork = resolveHardfork(undefined, undefined);
       assert.equal(hardfork, getCurrentHardfork(L1_CHAIN_TYPE));
 
-      hardfork = resolveHardfork(undefined, OPTIMISM_CHAIN_TYPE, true);
+      hardfork = resolveHardfork(undefined, OPTIMISM_CHAIN_TYPE);
       assert.equal(hardfork, getCurrentHardfork(OPTIMISM_CHAIN_TYPE));
     });
   });
diff --git a/v-next/hardhat/test/internal/builtin-plugins/network-manager/network-manager.ts b/v-next/hardhat/test/internal/builtin-plugins/network-manager/network-manager.ts
@@ -1878,72 +1878,6 @@ describe("NetworkManagerImplementation", () => {
       });
     });
 
-    describe("enableTransientStorage", () => {
-      describe("edr config", () => {
-        it("should validate a valid network config", async () => {
-          let validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: true }),
-          );
-
-          assertValidationErrors(validationErrors, []);
-
-          validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: false, hardfork: "berlin" }),
-          );
-
-          assertValidationErrors(validationErrors, []);
-        });
-
-        it("should not validate an invalid network config", async () => {
-          let validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: true, hardfork: "berlin" }),
-          );
-
-          assertValidationErrors(validationErrors, [
-            {
-              path: ["networks", "hardhat"],
-              message:
-                "'enableTransientStorage' is not supported for hardforks before 'cancun'. Please use a hardfork from 'cancun' onwards to enable this feature.",
-            },
-          ]);
-
-          validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: false, hardfork: "cancun" }),
-          );
-
-          assertValidationErrors(validationErrors, [
-            {
-              path: ["networks", "hardhat"],
-              message:
-                "'enableTransientStorage' must be enabled for hardforks 'cancun' or later. To disable this feature, use a hardfork before 'cancun'.",
-            },
-          ]);
-
-          validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: "incorrect" }),
-          );
-
-          assertValidationErrors(validationErrors, [
-            {
-              path: ["networks", "hardhat", "enableTransientStorage"],
-              message: "Expected boolean, received string",
-            },
-          ]);
-
-          validationErrors = await validateNetworkUserConfig(
-            edrConfig({ enableTransientStorage: "incorrect" }),
-          );
-
-          assertValidationErrors(validationErrors, [
-            {
-              path: ["networks", "hardhat", "enableTransientStorage"],
-              message: "Expected boolean, received string",
-            },
-          ]);
-        });
-      });
-    });
-
     describe("forking", () => {
       describe("edr config", () => {
         it("should validate a valid network config", async () => {
