fix(error-reporting): silence ResolutionError to prevent crash reports

src/lib/error-reporting.ts

@@ -3,9 +3,9 @@
  *
  * Provides two things:
  *
- * 1. **Silencing rules** — `OutputError`, expected-state `AuthError`, and
- *    401–499 `ApiError` are not sent to Sentry as issues. A
- *    `cli.error.silenced` metric preserves volume + user/org context.
+ * 1. **Silencing rules** — `OutputError`, `ResolutionError`, expected-state
+ *    `AuthError`, and 401–499 `ApiError` are not sent to Sentry as issues.
+ *    A `cli.error.silenced` metric preserves volume + user/org context.
  *
  * 2. **Grouping tags** — enriches every error event with `cli_error.*` tags
  *    that Sentry's server-side fingerprint rules use for stable grouping.
@@ -42,7 +42,11 @@
  * Reasons an error may be silenced (not sent to Sentry as an issue).
  * Exposed as the `reason` attribute on the `cli.error.silenced` metric.
  */
-type SilenceReason = "output_error" | "auth_expected" | "api_user_error";
+type SilenceReason =
+  | "output_error"
+  | "auth_expected"
+  | "api_user_error"
+  | "user_input_error";
 
 /**
  * Classify whether an error should be silenced.
@@ -54,6 +58,9 @@
   if (error instanceof OutputError) {
     return "output_error";
   }
+  if (error instanceof ResolutionError) {
+    return "user_input_error";
+  }
   if (
     error instanceof AuthError &&
     (error.reason === "not_authenticated" || error.reason === "expired")

test/lib/error-reporting.test.ts

@@ -2,7 +2,7 @@
  * Unit tests for the central error reporting helper.
  *
  * Covers:
- * - Silencing rules (OutputError / expected AuthError / 401–499 ApiError)
+ * - Silencing rules (OutputError / ResolutionError / expected AuthError / 401–499 ApiError)
  * - Grouping tag extraction (extractResourceKind)
  * - Tag enrichment in beforeSend (enrichEventWithGroupingTags)
  * - End-to-end behavior of reportCliError (metric emission + capture)
@@ -244,12 +244,16 @@ describe("classifySilenced", () => {
     expect(classifySilenced(new ApiError("x", status))).toBeNull();
   });
 
+  test("silences ResolutionError as user_input_error", () => {
+    expect(
+      classifySilenced(
+        new ResolutionError("Project 'x'", "not found", "sentry issue list")
+      )
+    ).toBe("user_input_error");
+  });
+
   test.each([
     ["ContextError", new ContextError("Organization", "sentry org view <x>")],
-    [
-      "ResolutionError",
-      new ResolutionError("Project 'x'", "not found", "sentry issue list"),
-    ],
     ["ValidationError", new ValidationError("bad")],
     ["SeerError", new SeerError("not_enabled")],
     ["ConfigError", new ConfigError("bad")],
@@ -447,14 +451,25 @@ describe("reportCliError integration", () => {
     expect(traceErr["cli_error.kind"]).not.toBe(eventErr["cli_error.kind"]);
   });
 
-  test("captures ResolutionError", () => {
-    const err = new ResolutionError(
-      "Project 'x'",
-      "not found",
-      "sentry issue list <org>/x"
+  test("silences ResolutionError and emits metric", () => {
+    reportCliError(
+      new ResolutionError(
+        "Project 'x'",
+        "not found",
+        "sentry issue list <org>/x"
+      )
+    );
+    expect(captureSpy).not.toHaveBeenCalled();
+    expect(metricSpy).toHaveBeenCalledWith(
+      "cli.error.silenced",
+      1,
+      expect.objectContaining({
+        attributes: expect.objectContaining({
+          error_class: "ResolutionError",
+          reason: "user_input_error",
+        }),
+      })
     );
-    reportCliError(err);
-    expect(captureSpy).toHaveBeenCalledWith(err);
   });
 
   test("captures SeerError (marketing dashboard)", () => {