fix: capture normalized eventId and update E2E tests for hex validation

Use the return value of validateHexId() so UUID-format event IDs get
normalized (dashes stripped) before downstream API calls. Update E2E
tests to use valid 32-char hex IDs and add a dedicated test for
invalid event ID format rejection.

Author: BYK

src/commands/event/view.ts

@@ -737,7 +737,7 @@ export const viewCommand = buildCommand({
     const log = logger.withTag("event.view");
 
     // Parse positional args
-    const { eventId, targetArg, warning, issueId, issueShortId } =
+    let { eventId, targetArg, warning, issueId, issueShortId } =
       parsePositionalArgs(args);
     if (warning) {
       log.warn(warning);
@@ -746,8 +746,9 @@ export const viewCommand = buildCommand({
     // Validate event ID format early (before API calls) when the ID came
     // from user input. Skip when the ID is a sentinel from issue URL/short
     // ID detection — those paths resolve the event through issue lookup.
+    // Capture the normalized return value (lowercased, UUID dashes stripped).
     if (eventId !== LATEST_EVENT_SENTINEL && !issueId && !issueShortId) {
-      validateHexId(eventId, "event ID");
+      eventId = validateHexId(eventId, "event ID");
     }
 
     const parsed = parseOrgProjectArg(targetArg);

test/e2e/event.test.ts

@@ -52,7 +52,7 @@ describe("sentry event view", () => {
       "event",
       "view",
       `${TEST_ORG}/${TEST_PROJECT}`,
-      "abc123",
+      "abc123def456abc123def456abc123de",
     ]);
 
     expect(result.exitCode).toBe(1);
@@ -62,12 +62,27 @@ describe("sentry event view", () => {
   test("requires org and project without DSN", async () => {
     await ctx.setAuthToken(TEST_TOKEN);
 
-    const result = await ctx.run(["event", "view", "abc123"]);
+    const result = await ctx.run([
+      "event",
+      "view",
+      "abc123def456abc123def456abc123de",
+    ]);
 
     expect(result.exitCode).toBe(1);
     expect(result.stderr + result.stdout).toMatch(/organization|project/i);
   });
 
+  test("rejects invalid event ID format", async () => {
+    await ctx.setAuthToken(TEST_TOKEN);
+
+    const result = await ctx.run(["event", "view", "abc123"]);
+
+    expect(result.exitCode).toBe(1);
+    expect(result.stderr + result.stdout).toMatch(
+      /invalid event id|32-character hexadecimal/i
+    );
+  });
+
   test("handles non-existent event", async () => {
     await ctx.setAuthToken(TEST_TOKEN);
 
@@ -76,7 +91,7 @@ describe("sentry event view", () => {
       "event",
       "view",
       `${TEST_ORG}/${TEST_PROJECT}`,
-      "nonexistent123",
+      "abc123def456abc123def456abc123de",
     ]);
 
     expect(result.exitCode).toBe(1);