@bivy/bivy 0.15.0-staging.35 → 0.15.0-staging.37

package/dist/runtime/auth-errors.js

@@ -77,3 +77,12 @@ export function authProviderForSession(runtimeId, modelProvider) {
         return provider;
     return undefined;
 }
+/** Convert a provider failure into the safe wire shape consumed by clients.
+ * Raw provider text can contain stack frames, paths, or tokens and must never
+ * be used as the user-facing auth error. */
+export function classifyModelAuthError(raw, runtimeId, modelProvider) {
+    if (!isModelAuthError(raw))
+        return null;
+    const provider = authProviderForSession(runtimeId, modelProvider);
+    return provider ? { kind: "model_auth", provider } : null;
+}

package/dist/server.js

@@ -34,7 +34,7 @@ import { InMemoryLocationRegistry } from "./runtime/location-registry.js";
 import { ControlPlaneSessionLocationRegistry, LayeredSessionLocationRegistry } from "./runtime/control-plane-location.js";
 import { attachAdoptedSessions, classifyAttachFailure } from "./runtime/adoption.js";
 import { createCredentialStore, testProviderCredential } from "./runtime/credentials.js";
-import { isModelAuthError, authProviderForSession } from "./runtime/auth-errors.js";
+import { isModelAuthError, authProviderForSession, classifyModelAuthError } from "./runtime/auth-errors.js";
 import { createCredentialVault, migrateVaultDir } from "./runtime/credential-store.js";
 import { probeAnthropicAccess } from "./runtime/anthropic-preflight.js";
 import { createSessionNamer, fallbackSessionName } from "./session/session-namer.js";
@@ -930,6 +930,7 @@ function harnessDirFor(record) {
 function harnessBeginTurn(record) {
     const dir = harnessDirFor(record);
     record.harnessTurnReady = undefined;
+    record.harnessTurnFinished = false;
     if (!dir)
         return;
     const previous = record.workspaceState === "dirty" ? "dirty" : "clean";
@@ -954,6 +955,14 @@ function harnessBeginTurn(record) {
     })();
 }
 /** After a turn, snapshot again and broadcast the structured diff it produced. */
+function finishHarnessTurn(record) {
+    if (record.harnessTurnFinished)
+        return;
+    record.harnessTurnFinished = true;
+    void harnessEndTurn(record).finally(() => {
+        void replication.onTurnComplete(record.id);
+    });
+}
 async function harnessEndTurn(record) {
     const previous = record.workspaceState === "dirty" ? "dirty" : "clean";
     try {
@@ -1643,11 +1652,14 @@ function stampSessionEvent(payload) {
     return payload;
 }
 // Collapse the burst of assistant `message_update`s (one per agent stdout line,
-// each carrying the FULL text so far) into ~1 fan-out per tick. See
-// session-event-coalescer.ts for the rationale (kills the O(n^2) re-serialize).
+// each carrying the FULL text so far) to a human-smooth 4 fps. A 16 ms window
+// still allowed up to 62 cumulative snapshots/second, making a long answer
+// quadratic and surprisingly expensive over a phone's relay connection. The
+// latest snapshot supersedes every skipped one, and non-update events flush it
+// immediately, so 250 ms changes neither final content nor event ordering.
 // The coalescer's emit stamps the surviving (latest) update, so only ~one seq is
-// spent per tick — the ring holds turn-scale events, not every stdout line.
-const SESSION_UPDATE_COALESCE_MS = 16;
+// spent per window — the ring holds turn-scale events, not every stdout line.
+const SESSION_UPDATE_COALESCE_MS = 250;
 const sessionEvents = new SessionEventCoalescer({
     coalesceMs: SESSION_UPDATE_COALESCE_MS,
     emit: (payload) => broadcastCoalesced(stampSessionEvent(payload)),
@@ -2842,7 +2854,7 @@ const RELAY_COMMANDS = {
             // this the relay client (PWA) is stranded on "Working…" forever with
             // only a session.error toast. Clear working so a terminal state reaches it.
             clearSessionWorking(record);
-            broadcast({ type: "session.error", sessionId: record.id, error: actionableAgentError(record.runtimeId, error) });
+            broadcast({ type: "session.error", sessionId: record.id, error: sessionAgentError(record, error) });
         });
     },
     async "session.new"(msg) {
@@ -5959,6 +5971,7 @@ async function sessionListRows() {
             agent: meta?.runtimeId ?? s.agent,
             agentName: meta?.agentName ?? s.agentName,
             source: rec?.source ?? meta?.source,
+            bivyCreated: Boolean(meta),
             forkedFrom: rec?.forkedFrom ?? meta?.forkedFrom,
             branch: rec?.worktree?.branch ?? meta?.branch,
             sandbox: rec?.sandbox ?? normalizeSandboxTier(meta?.sandbox),
@@ -7175,6 +7188,11 @@ function actionableAgentError(runtimeId, error) {
     }
     return raw;
 }
+function sessionAgentError(record, error) {
+    const raw = humanizeAgentError(error instanceof Error ? error.message : String(error));
+    return classifyModelAuthError(raw, record.runtimeId, record.session.getCurrentModel()?.provider)
+        ?? actionableAgentError(record.runtimeId, error);
+}
 /**
  * A turn that ended in a *terminal* model/provider failure the runtime would
  * otherwise swallow. `agent_end` carries the turn's messages and whether the
@@ -7231,7 +7249,7 @@ function attachSessionListeners(record) {
             policy: sessionRunPolicy,
             onNotice: (n) => broadcast({ type: "session.notice", sessionId: record.id, level: n.level, message: n.message }),
             onModelChanged: () => broadcast({ type: "model.updated", sessionId: record.id, model: publicModel(record.session.getCurrentModel(), record.session.getCurrentModel()) }),
-            onFailed: (message) => broadcast({ type: "session.error", sessionId: record.id, error: message }),
+            onFailed: (message) => broadcast({ type: "session.error", sessionId: record.id, error: sessionAgentError(record, message) }),
         });
     }
     record.unsubscribe = record.session.subscribe((event) => {
@@ -7328,9 +7346,14 @@ function attachSessionListeners(record) {
             transcripts.resolveInlineImages(record);
         }
         // Durably persist the throttled sidecars at the turn boundary so a crash
-        // loses at most the in-flight turn's UI detail, not the whole turn.
-        if (event.type === "turn_end")
+        // loses at most the in-flight turn's UI detail, not the whole turn. A
+        // runtime may keep its agent process alive and never emit agent_end (Pi),
+        // so turn_end is also authoritative for the interactive working state.
+        if (event.type === "turn_end") {
             eventLog.flush(record.id);
+            clearSessionWorking(record);
+            finishHarnessTurn(record);
+        }
         // AskUserQuestion is intercepted and answered by the daemon's guardian /
         // QuestionManager (see guardianInterceptor), which broadcasts
         // session.question(.resolved) from its own listeners — runtimes no longer
@@ -7365,14 +7388,9 @@ function attachSessionListeners(record) {
             transcripts.clearLiveIntermediate(record.id);
             clearSessionWorking(record);
             void refreshSessionUsage(record);
-            // Snapshot the worktree and broadcast the structured diff this turn made —
-            // universal edit review + rewind target, for every runtime.
-            // Warm-replicate this turn to the standby AFTER the checkpoint is committed
-            // (so the shipped frame carries this turn's transcript AND its checkpoint).
-            // Gated on session sync — inert by default.
-            void harnessEndTurn(record).finally(() => {
-                void replication.onTurnComplete(record.id);
-            });
+            // Snapshot/diff at whichever completion boundary arrived first. Pi keeps
+            // its process alive after turn_end; other runtimes also emit agent_end.
+            finishHarnessTurn(record);
             // A turn that ended in a terminal model/provider error (e.g. an expired
             // credential or a 4xx from the API) otherwise vanished: working cleared,
             // no reply, no signal. Surface it as a session-scoped error so the client
@@ -7434,7 +7452,7 @@ function attachSessionListeners(record) {
                 scheduleAdvertise();
                 broadcast({ type: "session.failed", sessionId: record.id, failedAt: record.lastFailureAt });
                 if (messageError)
-                    broadcast({ type: "session.error", sessionId: record.id, error: actionableAgentError(record.runtimeId, messageError) });
+                    broadcast({ type: "session.error", sessionId: record.id, error: sessionAgentError(record, messageError) });
                 // If the terminal error is an auth failure (expired key/token → 4xx),
                 // also raise the sign-in sheet for the failing provider.
                 maybeSignalAuthRequired(record, turnError);
@@ -8058,7 +8076,7 @@ async function createSession(workspace = defaultWorkspace, sessionFile, opts = {
     void refreshSessionUsage(record);
     if (makeActive)
         active = record;
-    broadcast({ type: "session.created", sessionId, name: record.session.getName(), workspace: sessionWorkspace, sessionFile: record.sessionFile, source: record.source, branch: worktree?.branch, prUrl: record.prUrl, runtimeId: rt.id, agentName: rt.displayName, modelFallbackMessage, bivySession: bivySessionEnvelope(record), capabilities: capabilitiesWithCommands(rt.id, record.session) });
+    broadcast({ type: "session.created", sessionId, name: record.session.getName(), workspace: sessionWorkspace, sessionFile: record.sessionFile, source: record.source, bivyCreated: true, branch: worktree?.branch, prUrl: record.prUrl, runtimeId: rt.id, agentName: rt.displayName, modelFallbackMessage, bivySession: bivySessionEnvelope(record), capabilities: capabilitiesWithCommands(rt.id, record.session) });
     void maybeNotifyBivyUpdate();
     scheduleAdvertise();
     return record;
@@ -9853,6 +9871,7 @@ app.get("/api/sessions", async (_req, res, next) => {
                 agent: meta?.runtimeId ?? s.agent,
                 agentName: meta?.agentName ?? s.agentName,
                 source: rec?.source ?? meta?.source,
+                bivyCreated: Boolean(meta),
                 forkedFrom: rec?.forkedFrom ?? meta?.forkedFrom,
                 branch: rec?.worktree?.branch ?? meta?.branch,
                 prUrl: rec?.prUrl ?? meta?.prUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bivy/bivy",
-  "version": "0.15.0-staging.35",
+  "version": "0.15.0-staging.37",
   "type": "module",
   "license": "AGPL-3.0-only",
   "description": "Run coding agents on machines you own. Open-source, self-hostable agent workspace.",