fix(session): allow max safe event revisions

.changeset/max-safe-event-revisions.md

@@ -0,0 +1,5 @@
+---
+"hunkdiff": patch
+---
+
+Accept the full safe-integer revision range when resuming browser review event streams.

src/session/reviewEventProtocol.test.ts

@@ -64,6 +64,12 @@ describe("review event names and ids", () => {
       type: "publication",
       address: ADDRESS,
     });
+
+    const maximumAddress = { ...ADDRESS, stateRevision: Number.MAX_SAFE_INTEGER };
+    expect(parseReviewEventId(reviewEventId("publication", maximumAddress))).toEqual({
+      type: "publication",
+      address: maximumAddress,
+    });
   });
 
   // A client echoes `Last-Event-ID` back at the server, so the parser is the boundary an
@@ -75,6 +81,9 @@ describe("review event names and ids", () => {
     expect(
       parseReviewEventId(`revent:publication:generation:test:3@${"9".repeat(40)}`),
     ).toBeUndefined();
+    expect(
+      parseReviewEventId("revent:publication:generation:test:3@9007199254740992"),
+    ).toBeUndefined();
     expect(parseReviewEventId(42)).toBeUndefined();
   });
 });

src/session/reviewEventProtocol.ts

@@ -119,7 +119,7 @@ export function parseReviewEventId(
   if (typeof value !== "string" || value.length > MAX_REVIEW_EVENT_ID_BYTES) {
     return undefined;
   }
-  const match = /^revent:([a-z-]+):(.+)@(\d{1,15})$/.exec(value);
+  const match = /^revent:([a-z-]+):(.+)@(\d{1,16})$/.exec(value);
   if (!match) {
     return undefined;
   }