hardhat 3.2.0 → 3.4.0

package/src/internal/builtin-plugins/node/json-rpc/handler.ts

@@ -21,6 +21,7 @@ import {
   InvalidRequestError,
   ProviderError,
 } from "../../network-manager/provider-errors.js";
+import { REVERT_ERROR_CODE } from "../../network-manager/revert-error-code.js";
 
 export class JsonRpcHandler {
   readonly #provider: EthereumProvider;
@@ -256,8 +257,15 @@ const _handleError = (error: Error): JsonRpcResponse => {
   const txHash = extractTxHash(error);
   const returnData = extractReturnData(error);
 
-  // In case of non-hardhat error, treat it as internal and associate the appropriate error code.
-  if (!ProviderError.isProviderError(error)) {
+  // Check if this is a revert error (code 3) matching the geth/anvil convention.
+  const isRevertError =
+    "code" in error &&
+    typeof error.code === "number" &&
+    error.code === REVERT_ERROR_CODE;
+
+  // In case of non-hardhat error that is not a revert, treat it as internal
+  // and associate the appropriate error code.
+  if (!ProviderError.isProviderError(error) && !isRevertError) {
     error = new InternalError(undefined, error);
   }
 

package/src/internal/builtin-plugins/node/task-action.ts

@@ -106,10 +106,10 @@ const nodeAction: NewTaskActionFunction<NodeActionArguments> = async (
     );
   }
 
-  // NOTE: This is where we initialize the network; the connect method returns
+  // NOTE: This is where we initialize the network; the create method returns
   // a fully resolved networkConfig object which might be useful for display
   const { networkConfig, provider } =
-    await hre.network.connect(connectionParams);
+    await hre.network.create(connectionParams);
 
   assertHardhatInvariant(
     provider instanceof EdrProvider,

package/src/internal/builtin-plugins/run/task-action.ts

@@ -25,7 +25,8 @@ const runScriptWithHardhat: NewTaskActionFunction<RunActionArguments> = async (
   }
 
   if (!noCompile) {
-    await hre.tasks.getTask("build").run({ quiet: true, noTests: true });
+    const noTests = hre.config.solidity.splitTestsCompilation;
+    await hre.tasks.getTask("build").run({ quiet: true, noTests });
     console.log();
   }
 

package/src/internal/builtin-plugins/solidity/build-system/artifacts.ts

@@ -1,4 +1,4 @@
-import type { Artifact, BuildInfo } from "../../../../types/artifacts.js";
+import type { Artifact } from "../../../../types/artifacts.js";
 import type { CompilationJob } from "../../../../types/solidity/compilation-job.js";
 import type {
   CompilerOutput,
@@ -107,11 +107,19 @@ declare module "hardhat/types/artifacts" {
 export async function getBuildInfo(
   compilationJob: CompilationJob,
 ): Promise<SolidityBuildInfo> {
-  const buildInfo: Required<BuildInfo> = {
+  // Defaulting to "solc" is safe here: if it's already "solc" or undefined,
+  // this doesn't alter the build info id.
+  const compilerType = compilationJob.solcConfig.type ?? "solc";
+
+  const buildInfo: SolidityBuildInfo = {
     _format: "hh3-sol-build-info-1",
     id: await compilationJob.getBuildId(),
     solcVersion: compilationJob.solcConfig.version,
     solcLongVersion: compilationJob.solcLongVersion,
+    compilerType,
+    ...(compilationJob.toolVersions !== undefined
+      ? { toolVersions: compilationJob.toolVersions }
+      : {}),
     userSourceNameMap:
       compilationJob.dependencyGraph.getRootsUserSourceNameMap(),
     input: await compilationJob.getSolcInput(),