@evandu/super-pi 0.1.146 → 0.1.147

package/dist/server/task-tool.js

@@ -384,7 +384,8 @@ Usage notes:
 2. The initial task should provide clear context and goals since the new session starts with a fresh conversation history.
 3. Use session_status with the returned conversationId to check progress, or session_dispatch to send further messages.
 4. The new session reports completion to its DIRECT parent automatically. 'parentConversationId' optionally selects that parent; by default it is the calling session.
-5. The new session always belongs to the calling session's workspace. Pass 'cwd' as a workspace-relative subdirectory (for example, 'packages/a') when it should work below the workspace root; absolute paths and traversal are rejected.`;
+5. The new session always belongs to the calling session's workspace. Pass 'cwd' as a workspace-relative subdirectory (for example, 'packages/a') when it should work below the workspace root; absolute paths and traversal are rejected.
+6. Optionally pass 'model' to choose which model the NEW session runs on: {"provider":"...","modelId":"..."} - the same "provider/modelId" pair the model picker shows, and BOTH halves are required. Leave 'model' out and the new session starts on the workspace default model.`;
 /** Model-facing description for session_status. */
 export const SESSION_STATUS_DESCRIPTION = `Inspect the progress, run state, and latest activity of sibling conversations created by the orchestrator.
 

package/dist/server/turn-failure.d.ts

@@ -19,4 +19,30 @@
  * needs no server, socket or Pi process.
  */
 export declare function formatTurnFailure(error: unknown): string;
+/**
+ * THE MODEL FILLED ITS OUTPUT BUDGET AND STOPPED MID-ANSWER.
+ *
+ * A length stop is NOT a crash. The provider answered, streamed real text, and stopped because it
+ * ran out of `max_tokens` - which is what happens every time an answer is longer than one output
+ * budget. Treating it as a failure is what made an orchestrated child session die: the run service
+ * stamped the conversation `failed`, the orchestrator reported a failure to the parent, and a task
+ * that was one sentence from done was thrown away.
+ *
+ * It carries a CLASS so the layers above can tell it apart from a provider error by identity rather
+ * than by matching a Chinese substring - the message is written for a reader, and a reader-facing
+ * sentence is not a reliable machine contract.
+ */
+export declare class LengthExceededError extends Error {
+    readonly stopReason: "length";
+    constructor(message?: string);
+}
+/**
+ * Whether this failure is a length stop rather than a real failure.
+ *
+ * The class is the answer; the message match is the BACKSTOP for a throw site that never got to
+ * construct the class - an error that crossed a process boundary, or one that was rebuilt from its
+ * text somewhere along the way. Both are checked because a length stop that loses its class on the
+ * way up is exactly the case that used to kill a child session.
+ */
+export declare function isLengthExceededError(error: unknown): boolean;
 export declare function controllerAborted(error: unknown): boolean;

package/dist/server/turn-failure.js

@@ -82,11 +82,62 @@ export function formatTurnFailure(error) {
     if (lower.includes("不支持图片输入")) {
         return raw;
     }
+    // LENGTH LIMIT - the model stopped because it filled the output budget, NOT because anything
+    // broke. The answer is incomplete, but everything that was said is still on screen, so the
+    // sentence has to offer the way out (say "continue") instead of calling it a failure.
+    //
+    // THE SENTENCE IS BUILT, NOT TAKEN FROM THE ERROR. The error's own message says the fact
+    // ("模型输出达到本轮长度上限，回答未完成") and stops there - which is a statement, not an action, and
+    // a reader who is told a thing stopped has no idea what to do next. What they need is the way
+    // out, so the sentence always carries it. Pi's own words are kept only when they add something
+    // the sentence does not already say.
+    if (isLengthExceededError(error)) {
+        const detail = raw && !/长度上限/.test(raw) ? `（${raw}）` : "";
+        return `模型单次输出达到长度上限，回答没有写完${detail}。可以点「继续写完」或者自己发一句「继续」，模型会从被截断的地方接着写。`;
+    }
     // GENERIC - the literal message is more useful than a paraphrase, but it has to be the LAST
     // branch and the message has to be non-empty (a thrown `new Error()` is empty and renders
     // nothing, which is exactly the silence this whole change exists to break).
     return raw || "Pi 未能完成本轮回答，已标记为失败。可以重发任务继续。";
 }
+/**
+ * THE MODEL FILLED ITS OUTPUT BUDGET AND STOPPED MID-ANSWER.
+ *
+ * A length stop is NOT a crash. The provider answered, streamed real text, and stopped because it
+ * ran out of `max_tokens` - which is what happens every time an answer is longer than one output
+ * budget. Treating it as a failure is what made an orchestrated child session die: the run service
+ * stamped the conversation `failed`, the orchestrator reported a failure to the parent, and a task
+ * that was one sentence from done was thrown away.
+ *
+ * It carries a CLASS so the layers above can tell it apart from a provider error by identity rather
+ * than by matching a Chinese substring - the message is written for a reader, and a reader-facing
+ * sentence is not a reliable machine contract.
+ */
+export class LengthExceededError extends Error {
+    stopReason = "length";
+    constructor(message) {
+        super(message || "模型输出达到本轮长度上限，回答未完成。");
+        this.name = "LengthExceededError";
+    }
+}
+/**
+ * Whether this failure is a length stop rather than a real failure.
+ *
+ * The class is the answer; the message match is the BACKSTOP for a throw site that never got to
+ * construct the class - an error that crossed a process boundary, or one that was rebuilt from its
+ * text somewhere along the way. Both are checked because a length stop that loses its class on the
+ * way up is exactly the case that used to kill a child session.
+ */
+export function isLengthExceededError(error) {
+    if (!error)
+        return false;
+    if (error instanceof LengthExceededError)
+        return true;
+    if (error?.stopReason === "length")
+        return true;
+    const msg = error instanceof Error ? error.message : typeof error === "string" ? error : "";
+    return msg.includes("模型输出达到本轮长度上限") || msg.includes("达到本轮长度上限") || msg.includes("单次输出达到长度上限");
+}
 export function controllerAborted(error) {
     if (!error)
         return false;

package/dist/server/types.d.ts

@@ -18,7 +18,7 @@ import type { ScheduledTask } from "./scheduled-tasks.js";
  * `failed` is what any unhandled throw lands (with `Conversation.error` carrying the translated reason);
  * `completed` is the clean path.
  */
-export type RunStatus = "idle" | "waiting" | "running" | "completed" | "stopped" | "failed";
+export type RunStatus = "idle" | "waiting" | "running" | "awaiting_background" | "completed" | "stopped" | "failed";
 export type MessageRole = "user" | "assistant" | "tool" | "system";
 /**
  * One message in `state.json`.
@@ -494,6 +494,20 @@ export type Conversation = {
      * would answer "what happened" with something the reader has to go and decode.
      */
     error?: string;
+    /**
+     * WHY THIS CONVERSATION'S LAST ANSWER IS INCOMPLETE - in words a person reads.
+     *
+     * A WARNING, not a failure. The model filled its single-response output budget and stopped
+     * mid-answer; everything it said is on screen, and the conversation is still usable. It is a
+     * separate field from `error` because the two mean different things to the reader: `error` says
+     * "this run broke, here is why", this says "this answer was cut short, and here is what to do
+     * about it". Collapsing them would paint a recoverable truncation as a crash - which is exactly
+     * what made an orchestrated child session die.
+     *
+     * Cleared by the next turn, the same way `error` is: it describes the turn that just ended and
+     * must never outlive it.
+     */
+    warning?: string;
     /**
      * WHY THIS CONVERSATION IS NOT ON THE MODEL THE USER SET FOR IT - in words a person reads.
      *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evandu/super-pi",
-  "version": "0.1.146",
+  "version": "0.1.147",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -51,6 +51,7 @@
     "scripts/inline-thinking.test.mjs",
     "scripts/inline-thinking-close-tag.test.mjs",
     "scripts/pi-session.test.mjs",
+    "scripts/length-continuation.test.mjs",
     "scripts/pi-session-isolation.test.mjs",
     "scripts/installed-layout.test.mjs",
     "scripts/pi-model-switch.test.mjs",
@@ -192,6 +193,7 @@
     "scripts/document-write-overwrite.test.mjs",
     "scripts/feishu.test.mjs",
     "scripts/scheduled-tasks.test.mjs",
+    "scripts/scheduled-tasks-timezone.test.mjs",
     "scripts/scheduled-task-model-clear.test.mjs",
     "scripts/scheduled-task-scope.test.mjs",
     "scripts/scheduled-task-tool.test.mjs",
@@ -203,8 +205,18 @@
     "scripts/spawn-workspace-boundary.test.mjs",
     "scripts/cwd-agent-spawn.test.mjs",
     "scripts/capabilities-prompt.test.mjs",
+    "scripts/version-fallback.test.mjs",
+    "scripts/model-image-input-session.test.mjs",
     "scripts/conversation-usage.test.mjs",
-    "scripts/orchestrator-observability.test.mjs"
+    "scripts/orchestrator-observability.test.mjs",
+    "scripts/session-tab-preview.test.mjs",
+    "scripts/knowledge-retrieval-quality.test.mjs",
+    "scripts/rail-ui-contract.test.mjs",
+    "scripts/session-tree-scope.test.mjs",
+    "scripts/session-spawn-model.test.mjs",
+    "scripts/background-result-steer.test.mjs",
+    "scripts/session-report-watchdog.test.mjs",
+    "scripts/awaiting-background-status.test.mjs"
   ],
   "check:browser": [
     "scripts/interrupt-browser.test.mjs",
@@ -243,6 +255,7 @@
     "scripts/archive-restore-race-browser.test.mjs",
     "scripts/tool-call-blocks-browser.test.mjs",
     "scripts/model-notice-browser.test.mjs",
+    "scripts/length-warning-browser.test.mjs",
     "scripts/dead-default-agent-browser.test.mjs",
     "scripts/assistant-working-indicator-browser.test.mjs",
     "scripts/thinking-first-delta-once-browser.test.mjs",
@@ -285,7 +298,10 @@
     "scripts/orchestrated-order-role-browser.test.mjs",
     "scripts/result-file-link-browser.test.mjs",
     "scripts/git-panel-confirm-browser.test.mjs",
-    "scripts/session-origin-badge-browser.test.mjs"
+    "scripts/session-origin-badge-browser.test.mjs",
+    "scripts/rail-ui-layout-browser.test.mjs",
+    "scripts/session-landing-browser.test.mjs",
+    "scripts/responsive-rail-drawer-browser.test.mjs"
   ],
   "dependencies": {
     "@codemirror/commands": "^6.11.1",