Merge pull request #7046 from NomicFoundation/sentry-trace

feat: report stack trace generation errors to sentry

Author: galargh

.changeset/fifty-roses-refuse.md

@@ -0,0 +1,5 @@
+---
+"hardhat": patch
+---
+
+Report stack trace generation errors to Sentry ([#7010](https://github.com/NomicFoundation/hardhat/issues/7010))

v-next/hardhat/src/internal/builtin-plugins/network-manager/edr/edr-provider.ts

@@ -44,6 +44,7 @@ import debug from "debug";
 import { hexToBytes } from "ethereum-cryptography/utils";
 import { addr } from "micro-eth-signer";
 
+import { sendErrorTelemetry } from "../../../cli/telemetry/sentry/reporter.js";
 import {
   EDR_NETWORK_REVERT_SNAPSHOT_EVENT,
   OPTIMISM_CHAIN_TYPE,
@@ -59,6 +60,7 @@ import {
   UnknownError,
 } from "../provider-errors.js";
 
+import { EdrProviderStackTraceGenerationError } from "./stack-traces/stack-trace-generation-errors.js";
 import { createSolidityErrorWithStackTrace } from "./stack-traces/stack-trace-solidity-errors.js";
 import {
   isDebugTraceResult,
@@ -319,6 +321,10 @@ export class EdrProvider extends BaseProvider {
       try {
         stackTrace = edrResponse.stackTrace();
       } catch (e) {
+        if (e instanceof Error) {
+          await sendErrorTelemetry(new EdrProviderStackTraceGenerationError(e));
+        }
+
         log("Failed to get stack trace: %O", e);
       }
 

v-next/hardhat/src/internal/builtin-plugins/network-manager/edr/stack-traces/stack-trace-generation-errors.ts

@@ -0,0 +1,18 @@
+import { CustomError } from "@nomicfoundation/hardhat-utils/error";
+
+abstract class StackTraceGenerationError extends CustomError {}
+
+export class EdrProviderStackTraceGenerationError extends StackTraceGenerationError {
+  constructor(error: Error) {
+    super("Failed to generate stack trace for the EDR provider", error);
+  }
+}
+
+export class SolidityTestStackTraceGenerationError extends StackTraceGenerationError {
+  constructor(message: string) {
+    super(
+      "Failed to generate stack trace for the Solidity test",
+      new Error(message),
+    );
+  }
+}

v-next/hardhat/src/internal/builtin-plugins/solidity-test/reporter.ts

@@ -8,6 +8,8 @@ import type { TestResult } from "@ignored/edr-optimism";
 import { bytesToHexString } from "@nomicfoundation/hardhat-utils/hex";
 import chalk from "chalk";
 
+import { sendErrorTelemetry } from "../../cli/telemetry/sentry/reporter.js";
+import { SolidityTestStackTraceGenerationError } from "../network-manager/edr/stack-traces/stack-trace-generation-errors.js";
 import { encodeStackTraceEntry } from "../network-manager/edr/stack-traces/stack-trace-solidity-errors.js";
 
 import { formatArtifactId, formatLogs, formatTraces } from "./formatters.js";
@@ -210,6 +212,9 @@ export async function* testReporter(
           }
           break;
         case "UnexpectedError":
+          await sendErrorTelemetry(
+            new SolidityTestStackTraceGenerationError(stackTrace.errorMessage),
+          );
           yield `Stack Trace Warning: ${chalk.grey(stackTrace.errorMessage)}\n`;
           break;
         case "UnsafeToReplay":