adw: clens-013-list-reverse-flag build (run clens-013-list-reverse-flag-1785085459819) (#21)

Co-authored-by: adw factory <adw-factory@e2b.local>

Author: silouone

packages/cli/src/cli.ts

@@ -78,6 +78,7 @@ const commands: Readonly<Record<string, CommandDef>> = {
 				projectDir: ctx.projectDir,
 				json: ctx.flags.json,
 				global: ctx.flags.global,
+				reverse: ctx.flags.reverse,
 			});
 		},
 	},
@@ -329,7 +330,7 @@ const GLOBAL_FLAGS = new Set(["--help", "-h", "--version", "-v"]);
 
 const VALID_FLAGS_BY_COMMAND: Readonly<Record<string, ReadonlySet<string>>> = {
 	init: new Set(["--remove", "--status", "--dev", "--global", "--legacy"]),
-	list: new Set(["--json", "--global"]),
+	list: new Set(["--json", "--global", "--reverse"]),
 	distill: new Set(["--last", "--all", "--global", "--force", "--deep", "--json", "--pricing"]),
 	report: new Set(["--last", "--json", "--detail", "--full", "--intent"]),
 	agents: new Set(["--last", "--json", "--comms"]),
@@ -397,6 +398,7 @@ ${bold("Setup:")}
 ${bold("Sessions:")}
   ${cyan("list")}              List captured sessions
   ${cyan("list --global")}     List sessions across all registered projects
+  ${cyan("list --reverse")}    List sessions oldest-first
   ${cyan("name")}              Set/clear a session's label & color flag
   ${cyan("distill")}           Extract insights from session data
   ${cyan("distill --global")}  Distill every session across all registered projects
@@ -488,6 +490,7 @@ const flags: Flags = {
 	comms: args.includes("--comms"),
 	global: args.includes("--global"),
 	legacy: args.includes("--legacy"),
+	reverse: args.includes("--reverse"),
 	...(pricingValue ? { pricing: pricingValue } : {}),
 };
 

packages/cli/src/commands/list.ts

@@ -12,8 +12,9 @@ export const listCommand = async (args: {
 	projectDir: string;
 	json: boolean;
 	global: boolean;
+	reverse: boolean;
 }): Promise<void> => {
-	const sessions: readonly SessionSummary[] = args.global
+	const fetchedSessions: readonly SessionSummary[] = args.global
 		? await (async () => {
 				const { listGlobalSessions } = await import("../session/global-read");
 				return listGlobalSessions();
@@ -24,6 +25,10 @@ export const listCommand = async (args: {
 				return enrichSessionSummaries(raw, args.projectDir);
 			})();
 
+	const sessions: readonly SessionSummary[] = args.reverse
+		? [...fetchedSessions].reverse()
+		: fetchedSessions;
+
 	if (args.json) {
 		console.log(JSON.stringify(sessions, null, 2));
 		return;

packages/cli/src/commands/shared.ts

@@ -17,6 +17,7 @@ export type Flags = {
 	readonly comms: boolean;
 	readonly global: boolean;
 	readonly legacy: boolean;
+	readonly reverse: boolean;
 	readonly pricing?: string;
 };
 

packages/cli/test/cli.test.ts

@@ -70,6 +70,11 @@ describe("cli --help", () => {
 		expect(stdout).toContain("Usage:");
 		expect(stdout).toContain("Analysis:");
 	});
+
+	test("documents list --reverse", async () => {
+		const { stdout } = await runCli("--help");
+		expect(stdout).toContain("list --reverse");
+	});
 });
 
 describe("cli init", () => {
@@ -156,6 +161,127 @@ describe("cli list", () => {
 		expect(stdout).toContain("Total:");
 		expect(stdout).toContain("session(s)");
 	});
+
+	test("--reverse reorders the table oldest-first, footer count unchanged", async () => {
+		const idA = "aaaa1111-0000-0000-0000-000000000000";
+		const idB = "bbbb2222-0000-0000-0000-000000000000";
+		writeSession(idA, [
+			makeStoredEvent({ t: 1000, event: "SessionStart", sid: idA }),
+			makeStoredEvent({ t: 2000, event: "SessionEnd", sid: idA }),
+		]);
+		writeSession(idB, [
+			makeStoredEvent({ t: 50000, event: "SessionStart", sid: idB }),
+			makeStoredEvent({ t: 60000, event: "SessionEnd", sid: idB }),
+		]);
+
+		const reversed = await runCli("list", "--reverse");
+		expect(reversed.exitCode).toBe(0);
+		expect(reversed.stdout.indexOf("aaaa1111")).toBeGreaterThanOrEqual(0);
+		expect(reversed.stdout.indexOf("bbbb2222")).toBeGreaterThanOrEqual(0);
+		expect(reversed.stdout.indexOf("aaaa1111")).toBeLessThan(reversed.stdout.indexOf("bbbb2222"));
+		expect(reversed.stdout).toContain("Total:");
+		expect(reversed.stdout).toContain("across 2 session(s)");
+
+		// Mirror image: without --reverse, newest (B) comes first.
+		const forward = await runCli("list");
+		expect(forward.stdout.indexOf("bbbb2222")).toBeLessThan(forward.stdout.indexOf("aaaa1111"));
+	});
+
+	test("--json --reverse reverses the JSON array (oldest first)", async () => {
+		const idA = "aaaa1111-0000-0000-0000-000000000000";
+		const idB = "bbbb2222-0000-0000-0000-000000000000";
+		writeSession(idA, [
+			makeStoredEvent({ t: 1000, event: "SessionStart", sid: idA }),
+			makeStoredEvent({ t: 2000, event: "SessionEnd", sid: idA }),
+		]);
+		writeSession(idB, [
+			makeStoredEvent({ t: 50000, event: "SessionStart", sid: idB }),
+			makeStoredEvent({ t: 60000, event: "SessionEnd", sid: idB }),
+		]);
+
+		const { exitCode, stdout } = await runCli("list", "--json", "--reverse");
+		expect(exitCode).toBe(0);
+		const parsed = JSON.parse(stdout);
+		expect(parsed).toHaveLength(2);
+		expect(parsed[0].session_id).toBe(idA);
+		expect(parsed[parsed.length - 1].session_id).toBe(idB);
+
+		// Cross-check: exact reverse of the plain --json order.
+		const forward = await runCli("list", "--json");
+		const forwardParsed = JSON.parse(forward.stdout);
+		expect(parsed).toEqual([...forwardParsed].reverse());
+	});
+
+	test("no --reverse flag is byte-identical to today (newest-first)", async () => {
+		const idA = "aaaa1111-0000-0000-0000-000000000000";
+		const idB = "bbbb2222-0000-0000-0000-000000000000";
+		writeSession(idA, [
+			makeStoredEvent({ t: 1000, event: "SessionStart", sid: idA }),
+			makeStoredEvent({ t: 2000, event: "SessionEnd", sid: idA }),
+		]);
+		writeSession(idB, [
+			makeStoredEvent({ t: 50000, event: "SessionStart", sid: idB }),
+			makeStoredEvent({ t: 60000, event: "SessionEnd", sid: idB }),
+		]);
+
+		const { exitCode, stdout } = await runCli("list");
+		expect(exitCode).toBe(0);
+		expect(stdout.indexOf("bbbb2222")).toBeLessThan(stdout.indexOf("aaaa1111"));
+	});
+
+	test("empty session list with --reverse still prints 'No sessions found.'", async () => {
+		const { exitCode, stdout } = await runCli("list", "--reverse");
+		expect(exitCode).toBe(0);
+		expect(stdout.trim()).toBe("No sessions found.");
+	});
+
+	test("empty session list with --json --reverse prints an empty JSON array", async () => {
+		const { exitCode, stdout } = await runCli("list", "--json", "--reverse");
+		expect(exitCode).toBe(0);
+		expect(JSON.parse(stdout)).toEqual([]);
+	});
+
+	test("--reverse with a single session is a no-op (same output as forward)", async () => {
+		const id = "cccc3333-0000-0000-0000-000000000000";
+		writeSession(id, [
+			makeStoredEvent({ t: 1000, event: "SessionStart", sid: id }),
+			makeStoredEvent({ t: 2000, event: "SessionEnd", sid: id }),
+		]);
+
+		const forward = await runCli("list");
+		const reversed = await runCli("list", "--reverse");
+		expect(reversed.exitCode).toBe(0);
+		expect(reversed.stdout).toBe(forward.stdout);
+	});
+
+	test("--reverse fully reverses three sessions, not just the endpoints", async () => {
+		const idOldest = "aaaa1111-0000-0000-0000-000000000000";
+		const idMiddle = "bbbb2222-0000-0000-0000-000000000000";
+		const idNewest = "cccc3333-0000-0000-0000-000000000000";
+		writeSession(idOldest, [
+			makeStoredEvent({ t: 1000, event: "SessionStart", sid: idOldest }),
+			makeStoredEvent({ t: 2000, event: "SessionEnd", sid: idOldest }),
+		]);
+		writeSession(idMiddle, [
+			makeStoredEvent({ t: 30000, event: "SessionStart", sid: idMiddle }),
+			makeStoredEvent({ t: 40000, event: "SessionEnd", sid: idMiddle }),
+		]);
+		writeSession(idNewest, [
+			makeStoredEvent({ t: 50000, event: "SessionStart", sid: idNewest }),
+			makeStoredEvent({ t: 60000, event: "SessionEnd", sid: idNewest }),
+		]);
+
+		const { exitCode, stdout } = await runCli("list", "--json", "--reverse");
+		expect(exitCode).toBe(0);
+		const parsed = JSON.parse(stdout) as Array<{ session_id: string }>;
+		expect(parsed.map((s) => s.session_id)).toEqual([idOldest, idMiddle, idNewest]);
+	});
+
+	test("--reverse with an unknown flag still rejects the unknown flag", async () => {
+		const { exitCode, stderr } = await runCli("list", "--reverse", "--bogus");
+		expect(exitCode).toBe(1);
+		expect(stderr).toContain("Unknown flag --bogus");
+	});
 });
 
 describe("cli session resolution", () => {