fix: improve error clarity and AuthError handling in cross-org event lookup

- resolveOrgAllTarget: throws a specific ContextError naming the org
  when the event is not found, instead of returning null and letting
  the caller emit a misleading "Organization and project is required"
- findEventAcrossOrgs: two-pass approach — first return any successful
  match, then (only if none found) propagate AuthError. This avoids
  hiding auth errors when no org has the event, while not discarding
  a valid result just because another org had an auth error.

Author: BYK

src/commands/event/view.ts

@@ -209,19 +209,22 @@ export async function resolveEventTarget(
  * Resolve target when only an org is known (e.g., from a Sentry event URL).
  * Uses the eventids endpoint to find the project directly.
  *
- * Returns null if the event is not found in the given org.
- * Propagates auth/network errors — if the user provided an explicit org
- * via URL, errors should surface rather than silently resolving a different org.
+ * Throws a ContextError if the event is not found in the given org, with a
+ * message that names the org so the error is not misleading.
+ * Propagates auth/network errors from resolveEventInOrg.
  */
 /** @internal Exported for testing */
 export async function resolveOrgAllTarget(
   org: string,
   eventId: string,
   _cwd: string
-): Promise<ResolvedEventTarget | null> {
+): Promise<ResolvedEventTarget> {
   const resolved = await resolveEventInOrg(org, eventId);
   if (!resolved) {
-    return null;
+    throw new ContextError(
+      `Event ${eventId} in organization "${org}"`,
+      `sentry event view ${org}/ ${eventId}`
+    );
   }
   return {
     org: resolved.org,

src/lib/api-client.ts

@@ -1270,14 +1270,18 @@ export async function findEventAcrossOrgs(
     orgs.map((org) => resolveEventInOrg(org.slug, eventId))
   );
 
+  // First pass: return the first successful match
   for (const result of results) {
     if (result.status === "fulfilled" && result.value !== null) {
       return result.value;
     }
-    // Propagate auth errors immediately — they indicate a global problem
-    // (expired/missing token) rather than a per-org miss, so the user needs
-    // to know. Transient per-org failures (network, 5xx) are swallowed since
-    // other orgs may still succeed.
+  }
+
+  // Second pass (only reached when no org had the event): propagate
+  // AuthError since it indicates a global problem (expired/missing token).
+  // Transient per-org failures (network, 5xx) are swallowed — they are not
+  // global, and if the event existed in any accessible org it would have matched.
+  for (const result of results) {
     if (result.status === "rejected" && result.reason instanceof AuthError) {
       throw result.reason;
     }

test/commands/event/view.test.ts

@@ -544,12 +544,12 @@ describe("resolveOrgAllTarget", () => {
     expect(result?.prefetchedEvent?.eventID).toBe("abc123");
   });
 
-  test("returns null when event not found in explicit org", async () => {
+  test("throws ContextError when event not found in explicit org", async () => {
     resolveEventInOrgSpy.mockResolvedValue(null);
 
-    const result = await resolveOrgAllTarget("acme", "notfound", "/tmp");
-
-    expect(result).toBeNull();
+    await expect(
+      resolveOrgAllTarget("acme", "notfound", "/tmp")
+    ).rejects.toBeInstanceOf(ContextError);
   });
 
   test("propagates errors from resolveEventInOrg", async () => {