fix(log-view): address Seer and Bugbot review findings

- Cap per_page at API_MAX_PER_PAGE (100) in getLogs to prevent silent
  truncation when >100 log IDs are requested
- Fix parsePositionalArgs docstring: remove false claim about multi-ID
  auto-detect pattern (2+ args always treats first as target)
- Fix trace/logs.ts callers to use normalized return value from
  validateTraceId instead of discarding it
- Update trace/logs test for lowercase normalization

Author: BYK

src/commands/log/view.ts

@@ -52,10 +52,12 @@ function splitLogIds(arg: string): string[] {
  * Parse positional arguments for log view.
  * Handles:
  * - `<log-id>` — single log ID (auto-detect org/project)
- * - `<log-id1> <log-id2> ...` — multiple log IDs (auto-detect)
  * - `<target> <log-id> [<log-id>...]` — explicit target + one or more log IDs
  * - `<org>/<project>/<log-id>` — single slash-separated arg
  *
+ * When two or more args are provided, the first is always treated as the
+ * target (org/project specifier) and the rest as log IDs.
+ *
  * Arguments containing newlines are split into multiple IDs.
  *
  * @param args - Positional arguments from CLI

src/commands/trace/logs.ts

@@ -89,13 +89,11 @@ export function parsePositionalArgs(args: string[]): {
         throw new ContextError("Trace ID", USAGE_HINT);
       }
 
-      validateTraceId(traceId);
-      return { traceId, orgArg };
+      return { traceId: validateTraceId(traceId), orgArg };
     }
 
     // Plain trace ID — org will be auto-detected
-    validateTraceId(first);
-    return { traceId: first, orgArg: undefined };
+    return { traceId: validateTraceId(first), orgArg: undefined };
   }
 
   // Two or more args — first is org, second is trace ID
@@ -106,8 +104,7 @@ export function parsePositionalArgs(args: string[]): {
     throw new ContextError("Trace ID", USAGE_HINT);
   }
 
-  validateTraceId(traceId);
-  return { traceId, orgArg };
+  return { traceId: validateTraceId(traceId), orgArg };
 }
 
 export const logsCommand = buildCommand({

src/lib/api-client.ts

@@ -1802,8 +1802,8 @@ export async function getLogs(
       dataset: "logs",
       field: DETAILED_LOG_FIELDS,
       query,
-      // per_page must match the number of IDs so the API returns all matches
-      per_page: logIds.length,
+      // Cap at API_MAX_PER_PAGE — the API silently truncates larger values
+      per_page: Math.min(logIds.length, API_MAX_PER_PAGE),
       statsPeriod: "90d",
     },
   });

test/commands/trace/logs.test.ts

@@ -57,10 +57,10 @@ describe("parsePositionalArgs", () => {
       expect(result.orgArg).toBeUndefined();
     });
 
-    test("accepts mixed-case hex trace ID", () => {
+    test("normalizes mixed-case hex trace ID to lowercase", () => {
       const mixedCase = "AAAA1111bbbb2222CCCC3333dddd4444";
       const result = parsePositionalArgs([mixedCase]);
-      expect(result.traceId).toBe(mixedCase);
+      expect(result.traceId).toBe(mixedCase.toLowerCase());
       expect(result.orgArg).toBeUndefined();
     });
   });