devez-vibe 1.3.27 → 1.3.30

package/bridge/claude-agent-sdk-bridge.mjs

@@ -1006,93 +1006,10 @@ function emitOpeningNotice(session) {
   session.turn.openingNoticeEmitted = true;
 }
 
-// Windows rounds larger timer delays up to the next scheduler slice. Ten
-// milliseconds lands near one terminal frame instead of visibly stepping at ~30ms.
-const SMOOTH_TEXT_INTERVAL_MS = 10;
-const SMOOTH_TEXT_TARGET_FRAMES = 10;
-const SMOOTH_TEXT_MAX_GRAPHEMES = 24;
-const graphemeSegmenter = typeof Intl.Segmenter === "function"
-  ? new Intl.Segmenter(undefined, { granularity: "grapheme" })
-  : null;
-
-function splitGraphemes(text) {
-  return graphemeSegmenter
-    ? Array.from(graphemeSegmenter.segment(text), ({ segment }) => segment)
-    : Array.from(text);
-}
-
-// Claude can deliver a whole phrase in one SDK event. Drain roughly one visual
-// frame's share at a time, catching up quickly when a large backlog arrives.
-function takeSmoothTextChunk(text) {
-  const graphemes = splitGraphemes(text);
-  const size = Math.min(
-    SMOOTH_TEXT_MAX_GRAPHEMES,
-    Math.max(1, Math.ceil(graphemes.length / SMOOTH_TEXT_TARGET_FRAMES)),
-  );
-  return {
-    chunk: graphemes.slice(0, size).join(""),
-    rest: graphemes.slice(size).join(""),
-  };
-}
-
-class SmoothTextStream {
-  constructor(emit, intervalMs = SMOOTH_TEXT_INTERVAL_MS) {
-    this.emit = emit;
-    this.intervalMs = intervalMs;
-    this.pending = "";
-    this.timer = null;
-    this.waiters = [];
-  }
-
-  push(text) {
-    this.pending += text;
-    this.schedule();
-  }
-
-  schedule() {
-    if (this.timer != null) return;
-    // Wait one visual frame before the first drain. Claude often sends several
-    // tiny deltas back-to-back; batching them removes the uneven one-character
-    // jumps while keeping added latency below one frame.
-    this.timer = setTimeout(() => {
-      this.timer = null;
-      this.drain();
-    }, this.intervalMs);
-  }
-
-  drain() {
-    if (!this.pending) {
-      this.timer = null;
-      for (const resolve of this.waiters.splice(0)) resolve();
-      return;
-    }
-    const { chunk, rest } = takeSmoothTextChunk(this.pending);
-    this.pending = rest;
-    this.emit(chunk);
-    if (this.pending) {
-      this.schedule();
-    } else {
-      for (const resolve of this.waiters.splice(0)) resolve();
-    }
-  }
-
-  finish() {
-    if (!this.pending && this.timer == null) return Promise.resolve();
-    return new Promise((resolve) => this.waiters.push(resolve));
-  }
-
-  flush() {
-    if (this.timer != null) clearTimeout(this.timer);
-    this.timer = null;
-    if (this.pending) this.emit(this.pending);
-    this.pending = "";
-    for (const resolve of this.waiters.splice(0)) resolve();
-  }
-}
-
-function flushSmoothStreams(session) {
-  for (const block of session.streamBlocks.values()) block.smooth?.flush();
-}
+// Text pacing lives in the host renderer, not here. A timer in this process
+// runs on Node's scheduler, which cannot line up with the terminal's redraw
+// rhythm — the mismatch was itself the visible stutter. Forward each delta as
+// it arrives and let the renderer reveal it on its own frames.
 
 function tokenBreakdown(usage) {
   if (!usage) return null;
@@ -1150,7 +1067,6 @@ async function processStreamEvent(session, message) {
   if (!session.turn || message.parent_tool_use_id) return;
   const event = message.event || {};
   if (event.type === "message_start") {
-    flushSmoothStreams(session);
     session.streamBlocks.clear();
   }
   if (event.type === "content_block_start") {
@@ -1160,9 +1076,10 @@ async function processStreamEvent(session, message) {
     const item = block.type === "text"
       ? { id, type: "agentMessage", text: "", provider: "Claude" }
       : { id, type: "reasoning", summary: [] };
-    const smooth = block.type === "text"
-      ? new SmoothTextStream((delta) => emitDelta(session, "item/agentMessage/delta", id, delta))
-      : null;
+    const method = block.type === "text"
+      ? "item/agentMessage/delta"
+      : "item/reasoning/summaryTextDelta";
+    const emit = (delta) => emitDelta(session, method, id, delta);
     // A held English line can end up dropped entirely, and an item announced
     // before that decision would stay on screen as an empty bubble. Hold the
     // start too, and emit it with the first text that survives.
@@ -1171,7 +1088,7 @@ async function processStreamEvent(session, message) {
       id,
       type: block.type,
       text: "",
-      smooth,
+      emit,
       languagePending: held ? "" : null,
       holdEnglishProgress: false,
       pendingStart: held ? item : null,
@@ -1188,10 +1105,6 @@ async function processStreamEvent(session, message) {
     // Held text may still be dropped, and counting it as visible would suppress
     // the opening notice in its place — leaving the turn with nothing to show.
     if (current.type === "text" && current.languagePending == null) session.turn.sawVisibleText = true;
-    if (!current.smooth) {
-      emitDelta(session, "item/reasoning/summaryTextDelta", current.id, delta);
-      return;
-    }
     if (current.languagePending != null) {
       current.languagePending += delta;
       const probe = current.languagePending.trimStart();
@@ -1203,11 +1116,11 @@ async function processStreamEvent(session, message) {
       }
       emitHeldStart(session, current);
       session.turn.sawVisibleText = true;
-      current.smooth.push(current.languagePending);
+      current.emit(current.languagePending);
       current.languagePending = null;
       return;
     }
-    current.smooth.push(delta);
+    current.emit(delta);
     return;
   }
   if (event.type === "content_block_stop") {
@@ -1223,10 +1136,9 @@ async function processStreamEvent(session, message) {
       }
       emitHeldStart(session, current);
       if (visible.trim()) session.turn.sawVisibleText = true;
-      current.smooth?.push(visible);
+      current.emit(visible);
     }
     emitHeldStart(session, current);
-    await current.smooth?.finish();
     const item = current.type === "text"
       ? { id: current.id, type: "agentMessage", text: current.text, provider: "Claude" }
       : { id: current.id, type: "reasoning", summary: [current.text] };
@@ -1912,7 +1824,6 @@ async function runPendingPrompt(session) {
 function finishTurn(session, error, durationMs) {
   if (!session.turn) return;
   flushPendingPlan(session);
-  flushSmoothStreams(session);
   clearForegroundSubagents(session);
   const turn = { id: session.turn.id, status: error ? "failed" : "completed" };
   if (error) turn.error = { message: error instanceof Error ? error.message : error.message || String(error) };
@@ -2122,22 +2033,22 @@ function historyState(messages) {
     const userText = message.type === "user"
       ? stripInternalTags(stripHandoff(blocks.filter((block) => block.type === "text").map((block) => block.text || "").join("\n")))
       : "";
-    if (userText
-      && !blocks.some((block) => block.type === "tool_result")
-      && !isInternalHistoryText(message, userText)) {
-      turn = {
-        id: `claude-turn-${message.uuid}`,
-        status: "completed",
-        items: [{ id: `claude-user-${message.uuid}`, type: "userMessage", content: [{ type: "text", text: userText }] }],
-      };
-      const startedAt = messageTime(message);
-      if (startedAt != null) turn.startedAt = Math.floor(startedAt / 1000);
-      turns.push(turn);
-    }
-    if (!turn) continue;
-    const completedAt = messageTime(message);
-    if (completedAt != null) turn.completedAt = Math.floor(completedAt / 1000);
-    if (message.type === "assistant") {
+    if (userText
+      && !blocks.some((block) => block.type === "tool_result")
+      && !isInternalHistoryText(message, userText)) {
+      turn = {
+        id: `claude-turn-${message.uuid}`,
+        status: "completed",
+        items: [{ id: `claude-user-${message.uuid}`, type: "userMessage", content: [{ type: "text", text: userText }] }],
+      };
+      const startedAt = messageTime(message);
+      if (startedAt != null) turn.startedAt = Math.floor(startedAt / 1000);
+      turns.push(turn);
+    }
+    if (!turn) continue;
+    const completedAt = messageTime(message);
+    if (completedAt != null) turn.completedAt = Math.floor(completedAt / 1000);
+    if (message.type === "assistant") {
       if (message.message?.model === "<synthetic>") {
         turn.synthetic = true;
         continue;
@@ -2374,6 +2285,14 @@ async function dispatch(method, params = {}) {
       ...(rejection ? { rejection } : {}),
     };
   }
+  // The host clears its `Working` row on `turn/completed` alone, so a single
+  // dropped notification would strand the spinner forever. This answers "is the
+  // turn you are still waiting on actually running?" from the bridge's own state.
+  if (method === "session/turnStatus") {
+    const session = lookupSession(params.sessionId);
+    if (!session) return { known: false, running: false, turnId: null };
+    return { known: true, running: Boolean(session.turn), turnId: session.turn?.id ?? null };
+  }
   if (method === "session/start") {
     const { session, account, usage } = await createSession(params);
     return {
@@ -3019,25 +2938,6 @@ async function runSelfTest() {
   if (keptStarted !== 0 || keptCompleted?.params?.item?.text !== "타일 보기 로직을 고쳤습니다.") {
     throw new Error(`Claude held Korean text self-test failed: ${JSON.stringify(keptEvents)}`);
   }
-  const smoothText = "Claude가 👨‍👩‍👧‍👦 한 문장을 한꺼번에 보내도 부드럽게 표시합니다.";
-  const emitted = [];
-  const smooth = new SmoothTextStream((chunk) => emitted.push(chunk), 0);
-  smooth.push(smoothText);
-  if (emitted.length !== 0) {
-    throw new Error(`Claude smooth stream did not batch its first frame: ${JSON.stringify(emitted)}`);
-  }
-  await smooth.finish();
-  if (emitted.length < 2 || emitted.join("") !== smoothText) {
-    throw new Error(`Claude smooth stream self-test failed: ${JSON.stringify(emitted)}`);
-  }
-  const flushed = [];
-  const interrupted = new SmoothTextStream((chunk) => flushed.push(chunk), 1000);
-  interrupted.push(smoothText);
-  interrupted.flush();
-  await interrupted.finish();
-  if (flushed.join("") !== smoothText) {
-    throw new Error(`Claude smooth stream flush self-test failed: ${JSON.stringify(flushed)}`);
-  }
   process.stdout.write("Claude bridge self-test passed\n");
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devez-vibe",
-  "version": "1.3.27",
+  "version": "1.3.30",
   "description": "Stable terminal UI for Codex and Claude Agent SDK",
   "keywords": [
     "codex",