@odla-ai/cli 0.25.9 → 0.25.10

package/dist/index.cjs

@@ -6623,7 +6623,7 @@ Usage:
   odla-ai pm handoff --app <id> [--json]
   odla-ai discuss groups [--json]
   odla-ai discuss list [--app <id>] [--q <text>] [--state open|resolved|all] [--json]
-  odla-ai discuss read <topic> [--json]
+  odla-ai discuss read <topic> [--limit <n> --offset <n>] [--json]
   odla-ai discuss post --app <id> --subject "..." --body "..." [--markup "... @[Label](kind/id)"] [--mutation-id <id>]
   odla-ai discuss reply <topic> --body "..." [--markup "..."] [--mutation-id <id>]
   odla-ai discuss resolve <topic> [--reopen] [--mutation-id <id>]
@@ -7469,22 +7469,59 @@ async function discussList(ctx, parsed) {
     }
   });
 }
-async function discussRead(ctx, id) {
-  const result = await request(
-    ctx,
-    "GET",
-    `/topics/${encodeURIComponent(id)}`
-  );
-  emit(ctx, result, () => {
-    ctx.out.log(`${result.topic.subject}  [${state(result.topic)}]  ${result.topic.appId ?? ""}`);
-    for (const post of result.posts) {
-      const who = post.authorKind === "bot" ? `${post.authorId} (agent)` : post.authorId;
-      ctx.out.log(`
-\u2014 ${who}`);
-      ctx.out.log(bodyWithRefs(post));
-      for (const file of post.attachments ?? []) ctx.out.log(`  [attachment] ${file.name} (${file.size} bytes)`);
+async function discussRead(ctx, id, parsed) {
+  const requestedLimit = stringOpt(parsed.options.limit);
+  const requestedOffset = stringOpt(parsed.options.offset);
+  if (requestedLimit !== void 0 || requestedOffset !== void 0) {
+    const query = new URLSearchParams({
+      limit: requestedLimit ?? "200",
+      offset: requestedOffset ?? "0"
+    });
+    const page = await request(ctx, "GET", `/topics/${encodeURIComponent(id)}?${query}`);
+    emit(ctx, page, () => renderRead(ctx, page.topic, page.posts));
+    return;
+  }
+  for (let scan = 0; scan < 3; scan++) {
+    const posts = /* @__PURE__ */ new Map();
+    let topic = null;
+    let offset = 0;
+    for (; ; ) {
+      const page = await request(
+        ctx,
+        "GET",
+        `/topics/${encodeURIComponent(id)}?limit=200&offset=${offset}`
+      );
+      topic = page.topic;
+      for (const post of page.posts) posts.set(post.id, post);
+      if (posts.size > 1e4) throw new Error("discuss read failed: conversation exceeds 10000 posts");
+      if (!page.page?.hasMore) break;
+      if (page.page.nextOffset === null || page.page.nextOffset <= offset) {
+        throw new Error("discuss read failed: registry returned a non-advancing post page");
+      }
+      offset = page.page.nextOffset;
     }
-  });
+    const ordered = [...posts.values()].sort(
+      (a, b) => a.createdAt - b.createdAt || (a.id < b.id ? -1 : a.id > b.id ? 1 : 0)
+    );
+    const expected = Number.isInteger(topic.replyCount) ? topic.replyCount + 1 : ordered.length;
+    if (ordered.length === expected) {
+      const result = { topic, posts: ordered };
+      emit(ctx, result, () => renderRead(ctx, result.topic, result.posts));
+      return;
+    }
+    if (offset === 0) throw new Error("discuss read failed: registry did not provide forward post pages");
+  }
+  throw new Error("discuss read failed: conversation changed during every complete-read attempt");
+}
+function renderRead(ctx, topic, posts) {
+  ctx.out.log(`${topic.subject}  [${state(topic)}]  ${topic.appId ?? ""}`);
+  for (const post of posts) {
+    const who = post.authorKind === "bot" ? `${post.authorId} (agent)` : post.authorId;
+    ctx.out.log(`
+\u2014 ${who}`);
+    ctx.out.log(bodyWithRefs(post));
+    for (const file of post.attachments ?? []) ctx.out.log(`  [attachment] ${file.name} (${file.size} bytes)`);
+  }
 }
 async function discussPost(ctx, parsed) {
   const appId = stringOpt(parsed.options.app);
@@ -7811,7 +7848,7 @@ async function discussCommand(parsed, deps = {}) {
     case "topics":
       return discussList(ctx, parsed);
     case "read":
-      return discussRead(ctx, requireId(id, "read"));
+      return discussRead(ctx, requireId(id, "read"), parsed);
     case "post":
       return discussPost(ctx, parsed);
     case "reply":