fix(replay): Reuse the shared replay list period default

Derive replay list hint rendering from LIST_PERIOD_FLAG.default so pagination commands cannot drift from the actual flag default. This keeps replay list behavior aligned with the shared list-command contract even if the default period changes later. Add focused coverage for next-page hints so the shared default period stays omitted unless the user explicitly overrides it. Co-Authored-By: OpenAI Codex <noreply@openai.com>
getsentry · BYK · May 2, 2026 · May 1, 2026 · May 1, 2026 · May 1, 2026
commit 8d1e0276bd630b90633b879efba9bfa9f8b56d03
diff --git a/src/commands/replay/list.ts b/src/commands/replay/list.ts
@@ -87,7 +87,7 @@ const SORT_MAP: Record<ReplaySortKey, ReplaySortValue> = {
   activity: "-activity",
 };
 
-const DEFAULT_PERIOD = "7d";
+const DEFAULT_PERIOD = LIST_PERIOD_FLAG.default;
 const DEFAULT_SORT: ReplaySortValue = SORT_MAP.date;
 const PAGINATION_KEY = "replay-list";
 const COMMAND_NAME = "replay list";

diff --git a/test/commands/replay/list.test.ts b/test/commands/replay/list.test.ts
@@ -16,6 +16,7 @@ import { listCommand, parseSort } from "../../../src/commands/replay/list.js";
 import * as apiClient from "../../../src/lib/api-client.js";
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as paginationDb from "../../../src/lib/db/pagination.js";
+import { LIST_PERIOD_FLAG } from "../../../src/lib/list-command.js";
 // biome-ignore lint/performance/noNamespaceImport: needed for spyOn mocking
 import * as resolveTarget from "../../../src/lib/resolve-target.js";
 import { parsePeriod } from "../../../src/lib/time-range.js";
@@ -183,4 +184,29 @@ describe("listCommand.func", () => {
     expect(output).toContain("Showing 1 replay.");
     expect(output).toContain("sentry replay view test-org/");
   });
+
+  test("omits --period in next-page hints for the shared default period", async () => {
+    resolveTargetSpy.mockResolvedValue({ org: "test-org", project: "cli" });
+    listReplaysSpy.mockResolvedValue({
+      data: sampleReplays,
+      nextCursor: "0:25:0",
+    });
+
+    const { context, stdoutWrite } = createMockContext();
+    const func = await listCommand.loader();
+    await func.call(
+      context,
+      {
+        limit: 25,
+        json: false,
+        period: parsePeriod(LIST_PERIOD_FLAG.default),
+        sort: "-started_at",
+      },
+      "test-org/cli"
+    );
+
+    const output = stdoutWrite.mock.calls.map((call) => call[0]).join("");
+    expect(output).toContain("sentry replay list test-org/cli -c next");
+    expect(output).not.toContain("--period");
+  });
 });