fix: address remaining review findings and lint errors

- Fix lint: use Number.POSITIVE/NEGATIVE_INFINITY, reorder imports
- Fix formatTimestamp treating unix seconds as milliseconds (showed 1970 dates)
- Preserve --query filter in pagination hints
- Surface truncation warning when transcript exceeds pagination limit
- Make view command org optional with resolveOrg fallback
- Register ai-conversations commands in ORG_ONLY_COMMANDS for completions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: sergical

src/commands/ai-conversations/list.ts

@@ -194,6 +194,9 @@ export const listCommand = buildListCommand("ai-conversations", {
     });
 
     const parts: string[] = [];
+    if (flags.query) {
+      parts.push(`-q "${flags.query}"`);
+    }
     appendPeriodHint(parts, flags.period, DEFAULT_PERIOD);
     const flagSuffix = parts.length > 0 ? ` ${parts.join(" ")}` : "";
 

src/commands/ai-conversations/view.ts

@@ -7,6 +7,7 @@
 import type { SentryContext } from "../../context.js";
 import { getConversationSpans } from "../../lib/api-client.js";
 import { buildCommand } from "../../lib/command.js";
+import { ContextError } from "../../lib/errors.js";
 import {
   buildTranscriptResult,
   formatTranscriptResult,
@@ -19,6 +20,7 @@ import {
   FRESH_FLAG,
 } from "../../lib/list-command.js";
 import { withProgress } from "../../lib/polling.js";
+import { resolveOrg } from "../../lib/resolve-target.js";
 
 type ViewFlags = {
   readonly json: boolean;
@@ -43,8 +45,9 @@ export const viewCommand = buildCommand({
       parameters: [
         {
           placeholder: "org",
-          brief: "Organization slug",
+          brief: "Organization slug (optional if auto-detected)",
           parse: String,
+          optional: true,
         },
         {
           placeholder: "conversation-id",
@@ -61,12 +64,31 @@ export const viewCommand = buildCommand({
   async *func(
     this: SentryContext,
     flags: ViewFlags,
-    org: string,
-    conversationId: string
+    orgOrConversationId: string,
+    maybeConversationId?: string
   ) {
     applyFreshFlag(flags);
+    const { cwd } = this;
 
-    const spans = await withProgress(
+    let org: string;
+    let conversationId: string;
+
+    if (maybeConversationId) {
+      org = orgOrConversationId;
+      conversationId = maybeConversationId;
+    } else {
+      const resolved = await resolveOrg({ cwd });
+      if (!resolved) {
+        throw new ContextError(
+          "Organization",
+          "sentry ai-conversations view <org> <conversation-id>"
+        );
+      }
+      org = resolved.org;
+      conversationId = orgOrConversationId;
+    }
+
+    const { spans, truncated } = await withProgress(
       {
         message: "Fetching conversation spans...",
         json: flags.json,
@@ -75,6 +97,7 @@ export const viewCommand = buildCommand({
     );
 
     const result = buildTranscriptResult(conversationId, org, spans);
+    result.truncated = truncated;
     yield new CommandOutput<TranscriptResult>(result);
   },
 });

src/lib/api/ai-conversations.ts

@@ -81,7 +81,7 @@ export async function getConversationSpans(
     project?: string;
     perPage?: number;
   } = {}
-): Promise<AIConversationSpan[]> {
+): Promise<{ spans: AIConversationSpan[]; truncated: boolean }> {
   const regionUrl = await resolveOrgRegion(orgSlug);
   const pageSchema = z.array(AIConversationSpanSchema);
 
@@ -115,11 +115,12 @@ export async function getConversationSpans(
     }
   }
 
-  if (cursor) {
+  const truncated = !!cursor;
+  if (truncated) {
     log.warn(
       `Pagination limit reached (${MAX_PAGINATION_PAGES} pages, ${spans.length} spans). Conversation transcript may be incomplete.`
     );
   }
 
-  return spans;
+  return { spans, truncated };
 }

src/lib/complete.ts

@@ -125,6 +125,8 @@ export const ORG_PROJECT_COMMANDS = new Set([
  * @internal Exported for testing only.
  */
 export const ORG_ONLY_COMMANDS = new Set([
+  "ai-conversations list",
+  "ai-conversations view",
   "org view",
   "release list",
   "release view",

src/lib/formatters/ai-conversations.ts

@@ -21,11 +21,11 @@ function truncate(value: string, max = 60): string {
   return `${value.slice(0, max - 1)}…`;
 }
 
-function formatTimestamp(ms: number): string {
-  if (ms === 0) {
+function formatTimestamp(epochSeconds: number): string {
+  if (epochSeconds === 0) {
     return "—";
   }
-  return new Date(ms).toLocaleString();
+  return new Date(epochSeconds * 1000).toLocaleString();
 }
 
 export function formatConversationTable(items: ConversationListItem[]): string {
@@ -338,6 +338,7 @@ export type TranscriptResult = {
   projects: string[];
   startTimestamp: number;
   endTimestamp: number;
+  truncated?: boolean;
 };
 
 export function formatTranscriptResult(result: TranscriptResult): string {
@@ -358,7 +359,13 @@ export function formatTranscriptResult(result: TranscriptResult): string {
     "",
   ];
 
-  return [...header, ...result.turns.map(formatTurnHuman)].join("\n");
+  const sections = [...header, ...result.turns.map(formatTurnHuman)];
+  if (result.truncated) {
+    sections.push(
+      "⚠ Transcript truncated — the conversation exceeds the pagination limit."
+    );
+  }
+  return sections.join("\n");
 }
 
 export function buildTranscriptResult(
@@ -381,14 +388,14 @@ export function buildTranscriptResult(
       spans.length > 0
         ? spans.reduce(
             (min, s) => Math.min(min, s["precise.start_ts"]),
-            Infinity
+            Number.POSITIVE_INFINITY
           )
         : 0,
     endTimestamp:
       spans.length > 0
         ? spans.reduce(
             (max, s) => Math.max(max, s["precise.finish_ts"]),
-            -Infinity
+            Number.NEGATIVE_INFINITY
           )
         : 0,
   };

src/types/index.ts

@@ -5,6 +5,8 @@
  * Re-exports all types from domain-specific modules.
  */
 
+// DSN types
+export type { DetectedDsn, DsnSource, ParsedDsn } from "../lib/dsn/types.js";
 // AI Conversations types
 export type {
   AIConversationSpan,
@@ -14,8 +16,6 @@ export {
   AIConversationSpanSchema,
   ConversationListItemSchema,
 } from "./ai-conversations.js";
-// DSN types
-export type { DetectedDsn, DsnSource, ParsedDsn } from "../lib/dsn/types.js";
 // Configuration types
 export type {
   CachedProject,