@nyssance/codex-acp-v2 0.7.10 → 0.7.11

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 0.7.11 — 2026-10-12
+
+### Added
+
+- `_codex/session_list` returns all non-archived sessions across projects when called
+  without params, walking native pages and deduplicating session IDs. Requests with
+  defined params retain the filtering and pagination behavior of `session/list`.
+
+### Fixed
+
+- Suppress duplicate review results in live updates and full-turn history when the
+  review exit repeats the assistant's answer, while preserving standalone or different
+  review results regardless of native event order.
+
 ## 0.7.10 — 2026-10-11
 
 ### Fixed

package/dist/index.js

@@ -22168,7 +22168,7 @@ var package_default = {
   publishConfig: {
     access: "public"
   },
-  version: "0.7.10",
+  version: "0.7.11",
   description: "Native ACP v2 agent for the OpenAI Codex app-server",
   packageManager: "bun@1.4.0",
   engines: {
@@ -24273,6 +24273,12 @@ function itemSnapshot(item, command) {
       return [];
   }
 }
+function isDuplicateReview(item, items) {
+  if (item.type !== "exitedReviewMode")
+    return false;
+  const text = item.review.trim();
+  return text.length > 0 && Array.from(items).some((candidate) => candidate.type === "agentMessage" && candidate.phase !== "commentary" && candidate.text.trim() === text);
+}
 
 // src/bridge/turnMetadata.ts
 function withTurnId(update, turnId, startedAt) {
@@ -24314,6 +24320,8 @@ class EventBridge {
   turnPrompt = null;
   openToolCalls = new Set;
   announcedTools = new Set;
+  pendingReviews = new Map;
+  reviewMessages = new Map;
   messagePhases = new Map;
   promptMeta = new Map;
   reasoningWithDeltas = new Set;
@@ -24377,6 +24385,8 @@ class EventBridge {
     this.openToolCalls.clear();
     this.announcedTools.clear();
     this.messagePhases.clear();
+    this.reviewMessages.clear();
+    this.pendingReviews.clear();
     this.reasoningWithDeltas.clear();
     this.terminalItems.clear();
     this.imageGenerations.clear();
@@ -24399,6 +24409,8 @@ class EventBridge {
     for (const item of items) {
       if (item.type === "userMessage")
         this.unresolvedUserItems.add(item.id);
+      if (item.type === "agentMessage")
+        this.reviewMessages.set(item.id, item);
       if (!replay && itemSnapshot(item).some((update) => update.sessionUpdate === "tool_call_update")) {
         this.unannouncedItems.set(item.id, { item, turnId });
       }
@@ -24411,8 +24423,12 @@ class EventBridge {
       if (receipt)
         this.restoredCommands.set(item.id, receipt);
       this.unresolvedUserItems.delete(item.id);
-      if (!replay)
+      if (!replay || isDuplicateReview(item, this.reviewMessages.values()))
         continue;
+      if (item.type === "exitedReviewMode") {
+        this.pendingReviews.set(item.id, { item, turnId });
+        continue;
+      }
       if (item.type === "commandExecution" && item.status === "inProgress" && usesTerminal(item))
         this.terminalItems.add(item.id);
       for (const update of itemSnapshot(item, receipt ?? undefined)) {
@@ -24532,6 +24548,7 @@ class EventBridge {
     }
   }
   async finishOpenToolCalls(status) {
+    await this.flushReviews();
     this.unannouncedItems.clear();
     const ids = [...this.openToolCalls];
     const terminals = [...this.terminalItems];
@@ -24556,6 +24573,18 @@ class EventBridge {
   }
   async flush() {
     await this.flushPlans();
+    await this.flushReviews();
+  }
+  async flushReviews() {
+    const pending = [...this.pendingReviews.values()];
+    this.pendingReviews.clear();
+    for (const { item, turnId } of pending) {
+      if (isDuplicateReview(item, this.reviewMessages.values()))
+        continue;
+      for (const update of itemSnapshot(item)) {
+        await this.client.update(this.withTiming(withTurnId(update, turnId, this.turnTime?.startedAt), turnId));
+      }
+    }
   }
   dispose() {
     this.beginTurn();
@@ -24578,7 +24607,7 @@ class EventBridge {
       case "item/started":
         return this.itemStarted(notification.params.item);
       case "item/completed":
-        return this.itemCompleted(notification.params.item);
+        return this.itemCompleted(notification.params.item, notification.params.turnId);
       case "item/commandExecution/outputDelta": {
         const { itemId, delta } = notification.params;
         return this.terminalItems.has(itemId) && delta.length > 0 ? [terminalOutputChunk(itemId, delta)] : [];
@@ -24792,9 +24821,10 @@ ${details}` : summary)];
         return [];
     }
   }
-  async itemCompleted(item) {
+  async itemCompleted(item, turnId) {
     switch (item.type) {
       case "agentMessage":
+        this.reviewMessages.set(item.id, item);
         this.messagePhases.delete(item.id);
         return itemSnapshot(item);
       case "reasoning": {
@@ -24834,8 +24864,8 @@ ${details}` : summary)];
       case "plan":
         return await this.planCompleted(item);
       case "exitedReviewMode": {
-        const text = item.review.trim();
-        return text.length > 0 ? [this.agentMessage(item.id, text)] : [];
+        this.pendingReviews.set(item.id, { item, turnId });
+        return [];
       }
       case "userMessage":
       case "hookPrompt":
@@ -27959,6 +27989,8 @@ function historyUpdates(turns, commandReceipts = new Map) {
   for (const turn of turns) {
     const openingUser = turn.items.find((item) => item.type === "userMessage");
     for (const item of turn.items) {
+      if (isDuplicateReview(item, turn.items))
+        continue;
       const command = item.type === "userMessage" ? commandReceipts.get(item.id) : undefined;
       updates.push(...itemSnapshot(item, command).map((update) => {
         const replay = withTurnId(update, turn.id, turn.startedAt);
@@ -28222,7 +28254,7 @@ async function readSessionHistory(codex, raw, signal, commandReceipts) {
   if (view === "full")
     await read.call(() => assertCommandHistory(codex, start.thread, commandReceipts, signal));
   const turns = await read.call(() => Promise.all(page.data.map(async (turn) => {
-    const projections = view === "full" ? await Promise.all(turn.items.map(async (item) => projectHistoryItem(item, turn.id, turn.startedAt, item.type === "userMessage" ? await commandReceipts?.read(item.id) : null))) : [];
+    const projections = view === "full" ? await Promise.all(turn.items.filter((item) => !isDuplicateReview(item, turn.items)).map(async (item) => projectHistoryItem(item, turn.id, turn.startedAt, item.type === "userMessage" ? await commandReceipts?.read(item.id) : null))) : [];
     return {
       turnId: turn.id,
       status: turn.status,
@@ -28501,7 +28533,7 @@ class CodexAgent {
           additionalDirectories: {}
         },
         providers: {},
-        _meta: { alwith: { tools: { version: 1 }, turns: { version: 2 } }, codex: { forkAtTurn: true, sessionLineage: true, archive: true, sessionHistory: { version: 2, items: true, modes: ["browse", "export"] }, seedHistory: true, skills: true, plugins: true, providerCatalog: true, rename: true, account: true, fuzzyFileSearch: true } }
+        _meta: { alwith: { tools: { version: 1 }, turns: { version: 2 } }, codex: { forkAtTurn: true, sessionLineage: true, archive: true, sessionList: { version: 1 }, sessionHistory: { version: 2, items: true, modes: ["browse", "export"] }, seedHistory: true, skills: true, plugins: true, providerCatalog: true, rename: true, account: true, fuzzyFileSearch: true } }
       },
       authMethods: authMethods(this.capabilities, this.env)
     };
@@ -29067,16 +29099,41 @@ class CodexAgent {
   }
   async listSessions(params) {
     this.requireInitialized("session/list");
+    return this.readSessionList(params, false);
+  }
+  async sessionList(params = {}) {
+    this.requireInitialized("_codex/session_list");
+    if (Object.values(params).some((value) => value !== undefined))
+      return this.listSessions(params);
+    return this.readSessionList({}, true);
+  }
+  async readSessionList(params, all) {
     const cwd = params.cwd?.trim() || null;
     const archived = archivedFilter(params._meta);
     const list = (cursor) => this.withCodex(() => this.codex.threadList({
       cursor,
+      ...all ? { limit: 100 } : {},
       ...cwd ? { cwd } : {},
       sourceKinds: ["cli", "vscode", "exec", "appServer", "unknown"],
       modelProviders: [],
       ...archived ? { archived: true } : {}
     }));
     const response = await list(params.cursor ?? null);
+    if (all) {
+      const threads = new Map(response.data.map((thread) => [thread.id, thread]));
+      const cursors = new Set;
+      while (response.nextCursor !== null) {
+        const cursor = response.nextCursor;
+        if (cursors.has(cursor))
+          throw RequestError.internalError(undefined, "Codex thread list repeated a cursor");
+        cursors.add(cursor);
+        const page = await list(cursor);
+        for (const thread of page.data)
+          threads.set(thread.id, thread);
+        response.nextCursor = page.nextCursor;
+      }
+      response.data = [...threads.values()];
+    }
     const home = this.codexHome;
     if (home === null)
       throw RequestError.internalError(undefined, "Codex home is unavailable after initialize");
@@ -29100,16 +29157,18 @@ class CodexAgent {
       }
       response.data.push(...missing2.values());
     }
-    return {
-      sessions: await Promise.all(response.data.map(async (thread) => ({
+    const sessions = [];
+    const batchSize = all ? 8 : response.data.length;
+    for (let offset = 0;offset < response.data.length; offset += batchSize) {
+      sessions.push(...await Promise.all(response.data.slice(offset, offset + batchSize).map(async (thread) => ({
         sessionId: thread.id,
         cwd: thread.cwd,
         title: !thread.name?.trim() && /^(?:\[@|# Files mentioned by the user:|<send_user_message_question_reply>)/.test(thread.preview.trimStart()) ? historyTitle(thread, (await this.turnPage(thread.id, null)).data) : thread.name?.trim() || thread.preview.trim() || null,
         updatedAt: new Date(thread.updatedAt * 1000).toISOString(),
         _meta: { codex: { archived, ...threadLineage(thread) } }
-      }))),
-      nextCursor: response.nextCursor
-    };
+      }))));
+    }
+    return all ? { sessions } : { sessions, nextCursor: response.nextCursor };
   }
   async closeSession(params) {
     return await this.admission(() => this.closeSessionInternal(params), params.sessionId);
@@ -29960,6 +30019,20 @@ function interruptedTurn(threadId) {
     turn: { id: "", items: [], itemsView: "notLoaded", status: "interrupted", error: null, startedAt: null, completedAt: null, durationMs: null }
   };
 }
+function parseSessionListParams(raw) {
+  const params = objectParams()(raw);
+  for (const key of ["cwd", "cursor"]) {
+    const value = params[key];
+    if (value !== undefined && value !== null && typeof value !== "string") {
+      throw RequestError.invalidParams({ [key]: value }, `${key} must be a string when given`);
+    }
+  }
+  const meta = params["_meta"];
+  if (meta !== undefined && meta !== null && (typeof meta !== "object" || Array.isArray(meta))) {
+    throw RequestError.invalidParams({ _meta: meta }, "_meta must be an object when given");
+  }
+  return params;
+}
 function parseSessionRenameParams(raw) {
   const params = objectParams()(raw);
   if (typeof params.sessionId !== "string" || params.sessionId.length === 0) {
@@ -30106,7 +30179,7 @@ function createAgentApp(options) {
   };
   return agent({ name: options.info.name }).onConnect((connection) => {
     agent2 = new CodexAgent(connection.client, options);
-  }).onRequest(methods.agent.initialize, (ctx) => current().initialize(ctx.params)).onRequest(methods.agent.auth.login, (ctx) => current().login(ctx.params, ctx.requestId)).onRequest(methods.agent.auth.logout, (ctx) => current().logout(ctx.params)).onRequest(methods.agent.providers.list, (ctx) => current().listProviders(ctx.params)).onRequest(methods.agent.providers.set, (ctx) => current().setProvider(ctx.params)).onRequest(methods.agent.providers.disable, (ctx) => current().disableProvider(ctx.params)).onRequest(methods.agent.session.new, (ctx) => current().newSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.resume, (ctx) => current().resumeSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.fork, (ctx) => current().forkSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.list, (ctx) => current().listSessions(ctx.params)).onRequest(methods.agent.session.close, (ctx) => current().closeSession(ctx.params)).onRequest(methods.agent.session.delete, (ctx) => current().deleteSession(ctx.params)).onRequest("_codex/session_history", { parse: parseSessionHistoryParams }, (ctx) => current().sessionHistory(ctx.params, ctx.signal)).onRequest("_codex/session_history_items", { parse: parseSessionHistoryItemsParams }, (ctx) => current().sessionHistoryItems(ctx.params, ctx.signal)).onRequest("_codex/session_archive", { parse: parseSessionIdParams }, (ctx) => current().archiveSession(ctx.params)).onRequest("_codex/session_unarchive", { parse: parseSessionIdParams }, (ctx) => current().unarchiveSession(ctx.params)).onRequest("_codex/skills_list", { parse: objectParams() }, (ctx) => current().skillsList(ctx.params)).onRequest("_codex/skills_config_write", { parse: objectParams() }, (ctx) => current().skillsConfigWrite(ctx.params)).onRequest("_codex/plugin_list", { parse: objectParams() }, (ctx) => current().pluginList(ctx.params)).onRequest("_codex/plugin_installed", { parse: objectParams() }, (ctx) => current().pluginInstalled(ctx.params)).onRequest("_codex/plugin_install", { parse: objectParams() }, (ctx) => current().pluginInstall(ctx.params)).onRequest("_codex/plugin_uninstall", { parse: objectParams() }, (ctx) => current().pluginUninstall(ctx.params)).onRequest("_codex/plugin_read", { parse: objectParams() }, (ctx) => current().pluginRead(ctx.params)).onRequest("_codex/marketplace_add", { parse: objectParams() }, (ctx) => current().marketplaceAdd(ctx.params)).onRequest("_codex/marketplace_remove", { parse: objectParams() }, (ctx) => current().marketplaceRemove(ctx.params)).onRequest("_codex/marketplace_upgrade", { parse: objectParams() }, (ctx) => current().marketplaceUpgrade(ctx.params)).onRequest("_codex/session_rename", { parse: parseSessionRenameParams }, (ctx) => current().sessionRename(ctx.params)).onRequest("_codex/account_read", { parse: objectParams() }, () => current().accountRead()).onRequest("_codex/rate_limits", { parse: objectParams() }, () => current().rateLimits()).onRequest("_codex/fuzzy_file_search", { parse: parseFuzzyFileSearchParams }, (ctx) => current().fuzzyFileSearch(ctx.params)).onRequest(methods.agent.session.setConfigOption, (ctx) => current().setSessionConfigOption(ctx.params)).onRequest(methods.agent.session.prompt, (ctx) => current().prompt(ctx.params)).onNotification(methods.agent.session.cancel, (ctx) => current().cancel(ctx.params));
+  }).onRequest(methods.agent.initialize, (ctx) => current().initialize(ctx.params)).onRequest(methods.agent.auth.login, (ctx) => current().login(ctx.params, ctx.requestId)).onRequest(methods.agent.auth.logout, (ctx) => current().logout(ctx.params)).onRequest(methods.agent.providers.list, (ctx) => current().listProviders(ctx.params)).onRequest(methods.agent.providers.set, (ctx) => current().setProvider(ctx.params)).onRequest(methods.agent.providers.disable, (ctx) => current().disableProvider(ctx.params)).onRequest(methods.agent.session.new, (ctx) => current().newSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.resume, (ctx) => current().resumeSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.fork, (ctx) => current().forkSession(ctx.params, ctx.signal)).onRequest(methods.agent.session.list, (ctx) => current().listSessions(ctx.params)).onRequest("_codex/session_list", { parse: parseSessionListParams }, (ctx) => current().sessionList(ctx.params)).onRequest(methods.agent.session.close, (ctx) => current().closeSession(ctx.params)).onRequest(methods.agent.session.delete, (ctx) => current().deleteSession(ctx.params)).onRequest("_codex/session_history", { parse: parseSessionHistoryParams }, (ctx) => current().sessionHistory(ctx.params, ctx.signal)).onRequest("_codex/session_history_items", { parse: parseSessionHistoryItemsParams }, (ctx) => current().sessionHistoryItems(ctx.params, ctx.signal)).onRequest("_codex/session_archive", { parse: parseSessionIdParams }, (ctx) => current().archiveSession(ctx.params)).onRequest("_codex/session_unarchive", { parse: parseSessionIdParams }, (ctx) => current().unarchiveSession(ctx.params)).onRequest("_codex/skills_list", { parse: objectParams() }, (ctx) => current().skillsList(ctx.params)).onRequest("_codex/skills_config_write", { parse: objectParams() }, (ctx) => current().skillsConfigWrite(ctx.params)).onRequest("_codex/plugin_list", { parse: objectParams() }, (ctx) => current().pluginList(ctx.params)).onRequest("_codex/plugin_installed", { parse: objectParams() }, (ctx) => current().pluginInstalled(ctx.params)).onRequest("_codex/plugin_install", { parse: objectParams() }, (ctx) => current().pluginInstall(ctx.params)).onRequest("_codex/plugin_uninstall", { parse: objectParams() }, (ctx) => current().pluginUninstall(ctx.params)).onRequest("_codex/plugin_read", { parse: objectParams() }, (ctx) => current().pluginRead(ctx.params)).onRequest("_codex/marketplace_add", { parse: objectParams() }, (ctx) => current().marketplaceAdd(ctx.params)).onRequest("_codex/marketplace_remove", { parse: objectParams() }, (ctx) => current().marketplaceRemove(ctx.params)).onRequest("_codex/marketplace_upgrade", { parse: objectParams() }, (ctx) => current().marketplaceUpgrade(ctx.params)).onRequest("_codex/session_rename", { parse: parseSessionRenameParams }, (ctx) => current().sessionRename(ctx.params)).onRequest("_codex/account_read", { parse: objectParams() }, () => current().accountRead()).onRequest("_codex/rate_limits", { parse: objectParams() }, () => current().rateLimits()).onRequest("_codex/fuzzy_file_search", { parse: parseFuzzyFileSearchParams }, (ctx) => current().fuzzyFileSearch(ctx.params)).onRequest(methods.agent.session.setConfigOption, (ctx) => current().setSessionConfigOption(ctx.params)).onRequest(methods.agent.session.prompt, (ctx) => current().prompt(ctx.params)).onNotification(methods.agent.session.cancel, (ctx) => current().cancel(ctx.params));
 }
 
 // src/codex/process.ts

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.7.10",
+  "version": "0.7.11",
   "description": "Native ACP v2 agent for the OpenAI Codex app-server",
   "packageManager": "bun@1.4.0",
   "engines": {