agent-office-cli 0.1.1 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-office-cli",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Run and manage AI agent sessions locally, with optional relay to agentoffice.top",
   "license": "MIT",
   "engines": {

package/src/core/config.js

@@ -3,11 +3,13 @@ const DEFAULT_LAN_HOST = "0.0.0.0";
 const DEFAULT_PORT = 8765;
 const DEFAULT_SERVER_URL = `http://${DEFAULT_HOST}:${DEFAULT_PORT}`;
 const LOG_LIMIT = 1000;
+const TERMINAL_REPLAY_LIMIT = 128 * 1024;
 
 module.exports = {
   DEFAULT_HOST,
   DEFAULT_LAN_HOST,
   DEFAULT_PORT,
   DEFAULT_SERVER_URL,
-  LOG_LIMIT
+  LOG_LIMIT,
+  TERMINAL_REPLAY_LIMIT
 };

package/src/core/providers/codex.js

@@ -1,6 +1,31 @@
 const { GenericProvider } = require("./generic");
 const { findManagedCodexSessionFile, summarizeCodexSession } = require("./codex-transcript");
 
+const APPROVAL_PATTERNS = [
+  "approval requested:",
+  "approval requested by ",
+  "tool call needs your approval",
+  "requires approval by policy",
+  "requires approval:"
+];
+
+const IDLE_PATTERNS = [
+  "conversation interrupted - tell the model what to do differently",
+  "something went wrong? hit `/feedback` to",
+  "something went wrong? hit /feedback to"
+];
+
+const ATTENTION_PATTERNS = [
+  "stream disconnected before completion",
+  "error sending request for url",
+  "network error",
+  "connection timeout",
+  "timed out",
+  "failed to send request",
+  "failed to submit",
+  "panic"
+];
+
 function activeOverlayPatch(session, nextLifecycleState) {
   if (!session || !["approval", "attention"].includes(session.displayState)) {
     return null;
@@ -48,19 +73,19 @@ class CodexProvider extends GenericProvider {
 
   classifyOutput(chunk) {
     const text = String(chunk).toLowerCase();
-    if (text.includes("approval") || text.includes("press enter") || text.includes("confirm")) {
+
+    if (IDLE_PATTERNS.some((pattern) => text.includes(pattern))) {
+      return "idle";
+    }
+
+    if (APPROVAL_PATTERNS.some((pattern) => text.includes(pattern))) {
       return "approval";
     }
-    if (
-      text.includes("network error") ||
-      text.includes("connection timeout") ||
-      text.includes("timed out") ||
-      text.includes("error") ||
-      text.includes("failed") ||
-      text.includes("panic")
-    ) {
+
+    if (ATTENTION_PATTERNS.some((pattern) => text.includes(pattern))) {
       return "attention";
     }
+
     return null;
   }
 

package/src/core/providers/codex.test.js

@@ -72,3 +72,39 @@ test("classifyOutput can raise attention for transcript-backed sessions", () =>
 
   assert.equal(nextState, "attention");
 });
+
+test("classifyOutput treats user interrupted Codex screens as idle", () => {
+  const provider = new CodexProvider();
+  const nextState = provider.classifyOutput(
+    "Conversation interrupted - tell the model what to do differently. Something went wrong? Hit `/feedback` to report the issue."
+  );
+
+  assert.equal(nextState, "idle");
+});
+
+test("classifyOutput treats stream disconnects as attention", () => {
+  const provider = new CodexProvider();
+  const nextState = provider.classifyOutput(
+    "stream disconnected before completion: error sending request for url (http://54.255.64.152:3000/openai/responses)"
+  );
+
+  assert.equal(nextState, "attention");
+});
+
+test("classifyOutput does not treat plain explanatory approval text as a real approval prompt", () => {
+  const provider = new CodexProvider();
+  const nextState = provider.classifyOutput(
+    "只有真实审批提示才归到 approval"
+  );
+
+  assert.equal(nextState, null);
+});
+
+test("classifyOutput recognizes real Codex approval prompts", () => {
+  const provider = new CodexProvider();
+  const nextState = provider.classifyOutput(
+    "Approval requested: Codex wants to edit files"
+  );
+
+  assert.equal(nextState, "approval");
+});

package/src/core/store/session-store.js

@@ -1,7 +1,7 @@
 const { EventEmitter } = require("node:events");
 const os = require("node:os");
 const crypto = require("node:crypto");
-const { LOG_LIMIT } = require("../config");
+const { LOG_LIMIT, TERMINAL_REPLAY_LIMIT } = require("../config");
 const { displayZoneFor } = require("../state");
 const { toPublicSession, toSessionSummary } = require("../session-contract");
 
@@ -53,7 +53,8 @@ function createSessionStore() {
       host: payload.host || os.hostname(),
       meta: { ...(payload.meta || {}) },
       logs: [...(payload.logs || [])],
-      events: [...(payload.events || [])]
+      events: [...(payload.events || [])],
+      terminalReplay: String(payload.terminalReplay || "")
     };
   }
 
@@ -151,6 +152,10 @@ function createSessionStore() {
     const lines = String(chunk).replace(/\r/g, "").split("\n").filter(Boolean);
     session.logs.push(...lines);
     session.logs = session.logs.slice(-LOG_LIMIT);
+    session.terminalReplay = `${session.terminalReplay || ""}${String(chunk)}`;
+    if (session.terminalReplay.length > TERMINAL_REPLAY_LIMIT) {
+      session.terminalReplay = session.terminalReplay.slice(-TERMINAL_REPLAY_LIMIT);
+    }
     session.lastOutputAt = isoNow();
     session.updatedAt = session.lastOutputAt;
     // Terminal output does not change session state — skip session:update broadcast.
@@ -189,6 +194,11 @@ function createSessionStore() {
     return session ? toSessionSummary(session) : null;
   }
 
+  function getTerminalReplay(sessionId) {
+    const session = sessions.get(sessionId);
+    return session ? session.terminalReplay || "" : "";
+  }
+
   function listSessions() {
     return [...sessions.values()]
       .sort((left, right) => right.updatedAt.localeCompare(left.updatedAt))
@@ -224,6 +234,7 @@ function createSessionStore() {
     markExit,
     getSession,
     getSessionSummary,
+    getTerminalReplay,
     listSessions,
     listSessionSummaries,
     removeSession

package/src/core/store/session-store.test.js

@@ -48,3 +48,21 @@ test("setSessionState derives displayZone from displayState override when zone i
   assert.equal(next.displayState, "attention");
   assert.equal(next.displayZone, "attention-zone");
 });
+
+test("appendOutput keeps a raw terminal replay buffer for fast reconnects", () => {
+  const store = createSessionStore();
+  store.upsertSession({
+    sessionId: "sess_3",
+    provider: "generic",
+    title: "Shell",
+    command: "bash",
+    cwd: process.cwd(),
+    state: "working",
+    status: "running"
+  });
+
+  store.appendOutput("sess_3", "line one\r\n");
+  store.appendOutput("sess_3", "line two\n");
+
+  assert.equal(store.getTerminalReplay("sess_3"), "line one\r\nline two\n");
+});

package/src/runtime/pty-manager.js

@@ -503,18 +503,25 @@ function createPtyManager({ store }) {
 
     let attachedClient = null;
     let tmuxStreamStarted = false;
+    let pendingCols = 120;
+    let pendingRows = 32;
 
     async function startTmuxStream(cols, rows) {
       if (tmuxStreamStarted || !entry || entry.transport !== "tmux") {
         return;
       }
       tmuxStreamStarted = true;
+      pendingCols = cols;
+      pendingRows = rows;
       try {
         const snapshot = await capturePane(entry.tmuxSession);
         if (snapshot && ws.readyState === 1) {
           ws.send(JSON.stringify({ type: "terminal:data", data: `${snapshot}\r\n` }));
         }
         attachedClient = attachClient(entry.tmuxSession, { cwd: entry.cwd, cols, rows });
+        if (pendingCols !== cols || pendingRows !== rows) {
+          attachedClient.resize(pendingCols, pendingRows);
+        }
         attachedClient.onData((chunk) => {
           if (ws.readyState === 1) {
             ws.send(JSON.stringify({ type: "terminal:data", data: chunk }));
@@ -532,7 +539,12 @@ function createPtyManager({ store }) {
 
     // For non-tmux transports that had immediate setup, keep original behavior
     if (entry && entry.transport === "pty") {
-      // PTY sessions stream via broadcastTerminal, no per-client attach needed
+      const replay = store.getTerminalReplay(sessionId);
+      if (replay) {
+        ws.send(JSON.stringify({ type: "terminal:data", data: replay }));
+      }
+    } else if (entry && entry.transport === "tmux") {
+      void startTmuxStream(120, 32);
     }
 
     ws.on("message", async (raw) => {
@@ -553,6 +565,8 @@ function createPtyManager({ store }) {
         if (message.type === "resize") {
           const cols = Number(message.cols || 120);
           const rows = Number(message.rows || 32);
+          pendingCols = cols;
+          pendingRows = rows;
           if (runtime.transport === "pty") {
             runtime.pty.resize(cols, rows);
           } else if (!tmuxStreamStarted) {