refactor: unify sort values with trace list ('date'/'duration')

span list used 'time' while trace list used 'date' — both mapped to
the same -timestamp API sort. Standardize on 'date'/'duration' to
match trace list and issue list conventions.

Also removes 'time' from the listSpans API type since it was redundant
with 'date'.

Refs: PR #393 review feedback

Author: BYK

src/commands/span/list.ts

@@ -37,16 +37,16 @@ import { validateTraceId } from "../../lib/trace-id.js";
 type ListFlags = {
   readonly limit: number;
   readonly query?: string;
-  readonly sort: "time" | "duration";
+  readonly sort: "date" | "duration";
   readonly json: boolean;
   readonly fresh: boolean;
   readonly fields?: string[];
 };
 
-type SortValue = "time" | "duration";
+type SortValue = "date" | "duration";
 
-/** Accepted values for the --sort flag */
-const VALID_SORT_VALUES: SortValue[] = ["time", "duration"];
+/** Accepted values for the --sort flag (matches trace list) */
+const VALID_SORT_VALUES: SortValue[] = ["date", "duration"];
 
 /**
  * CLI-side upper bound for --limit.
@@ -62,7 +62,7 @@ const MAX_LIMIT = 1000;
 const DEFAULT_LIMIT = 25;
 
 /** Default sort order for span results */
-const DEFAULT_SORT: SortValue = "time";
+const DEFAULT_SORT: SortValue = "date";
 
 /** Usage hint for ContextError messages */
 const USAGE_HINT = "sentry span list [<org>/<project>/]<trace-id>";
@@ -120,7 +120,7 @@ function parseLimit(value: string): number {
 /**
  * Parse and validate sort flag value.
  *
- * @throws Error if value is not "time" or "duration"
+ * @throws Error if value is not "date" or "duration"
  */
 export function parseSort(value: string): SortValue {
   if (!VALID_SORT_VALUES.includes(value as SortValue)) {

src/lib/api/traces.ts

@@ -168,8 +168,8 @@ type ListSpansOptions = {
   query?: string;
   /** Maximum number of spans to return */
   limit?: number;
-  /** Sort order: "date"/"time" (newest first) or "duration" (slowest first) */
-  sort?: "date" | "time" | "duration";
+  /** Sort order: "date" (newest first) or "duration" (slowest first) */
+  sort?: "date" | "duration";
   /** Time period for spans (e.g., "7d", "24h") */
   statsPeriod?: string;
   /** Pagination cursor to resume from a previous page */

test/commands/span/list.test.ts

@@ -98,14 +98,18 @@ describe("parsePositionalArgs", () => {
 });
 
 describe("parseSort", () => {
-  test("accepts 'time'", () => {
-    expect(parseSort("time")).toBe("time");
+  test("accepts 'date'", () => {
+    expect(parseSort("date")).toBe("date");
   });
 
   test("accepts 'duration'", () => {
     expect(parseSort("duration")).toBe("duration");
   });
 
+  test("rejects 'time' (use 'date' instead)", () => {
+    expect(() => parseSort("time")).toThrow("Invalid sort value");
+  });
+
   test("throws for invalid value", () => {
     expect(() => parseSort("name")).toThrow("Invalid sort value");
   });
@@ -190,7 +194,7 @@ describe("listCommand.func", () => {
       context,
       {
         limit: 25,
-        sort: "time",
+        sort: "date",
         fresh: false,
       },
       VALID_TRACE_ID
@@ -219,7 +223,7 @@ describe("listCommand.func", () => {
       {
         limit: 25,
         query: "op:db",
-        sort: "time",
+        sort: "date",
         fresh: false,
       },
       VALID_TRACE_ID
@@ -243,7 +247,7 @@ describe("listCommand.func", () => {
       context,
       {
         limit: 25,
-        sort: "time",
+        sort: "date",
         fresh: false,
       },
       `my-org/my-project/${VALID_TRACE_ID}`