@evandu/super-pi 0.1.116 → 0.1.118

package/dist/server/tool-registry.js

@@ -38,27 +38,27 @@ export class ToolRegistry {
         await this.load();
         const slug = input.slug.trim().toLowerCase();
         if (!SLUG.test(slug))
-            throw new Error("slug must contain lowercase letters, digits, and hyphens only");
+            throw new Error("工具名（slug）只能包含小写字母、数字和连字符");
         if (!input.name.trim() || input.name.length > 120)
-            throw new Error("tool name is invalid");
+            throw new Error("工具名称不合法（1-120 个字符）");
         if (this.items.some((x) => x.slug === slug))
-            throw new Error("tool slug already exists");
+            throw new Error("已经有同名（slug）的工具了，换一个名字再导入");
         if (this.items.length >= MAX_TOOLS)
-            throw new Error("tool limit reached");
+            throw new Error("工具数量已经达到上限");
         if (!Array.isArray(input.files) || !input.files.length || input.files.length > MAX_ASSETS)
-            throw new Error("tool must contain 1-32 files");
+            throw new Error("一个工具需要包含 1-32 个文件");
         const entrypoint = safeRelative(input.entrypoint ?? "index.html");
         if (!entrypoint.endsWith(".html"))
-            throw new Error("entrypoint must be an HTML file");
+            throw new Error("入口文件必须是 .html");
         const files = normalizeAssets(input.files);
         if (!files.some((file) => file.path === entrypoint))
-            throw new Error("entrypoint must be included in files");
+            throw new Error("入口文件必须出现在文件列表里");
         const script = input.script ? normalizeScript(input.script, files) : undefined;
         const environment = input.environment ? normalizeEnvironment(input.environment) : undefined;
         const permissions = normalizePermissions(input.permissions);
         const finalDir = join(this.toolsDir, slug), stagingDir = `${finalDir}.${crypto.randomUUID()}.staging`;
         if (await stat(finalDir).catch(() => undefined))
-            throw new Error("tool directory already exists");
+            throw new Error("工具目录已经存在");
         try {
             await writeAssets(stagingDir, files);
             await rename(stagingDir, finalDir);
@@ -84,7 +84,7 @@ export class ToolRegistry {
     async resolve(id, resource = "") { await this.load(); const tool = this.items.find((x) => x.id === id && x.enabled); if (!tool)
         return undefined; const root = join(this.toolsDir, tool.slug), file = resolve(root, resource ? safeRelative(resource) : tool.entrypoint); if (!inside(root, file))
         throw new Error("tool resource must stay inside the tool directory"); const info = await stat(file).catch(() => undefined); if (!info?.isFile())
-        throw new Error("tool file does not exist"); return { tool: copy(tool), file, root }; }
+        throw new Error("工具文件不存在"); return { tool: copy(tool), file, root }; }
     /**
      * THE EXPORT, AND THE ONLY PRODUCER OF A SHAREABLE BUNDLE.
      *
@@ -126,12 +126,12 @@ export class ToolRegistry {
     }
 }
 function normalizeAssets(files) { const paths = new Set(); let bytes = 0; return files.map((item) => { if (!item || typeof item.content !== "string")
-    throw new Error("tool file content must be text"); const path = safeRelative(item.path); if (paths.has(path))
-    throw new Error(`duplicate tool file: ${path}`); paths.add(path); bytes += Buffer.byteLength(item.content); if (bytes > MAX_ASSET_BYTES)
-    throw new Error("tool files exceed 512KB"); return { path, content: item.content }; }); }
+    throw new Error("工具文件的内容必须是文本"); const path = safeRelative(item.path); if (paths.has(path))
+    throw new Error(`工具里有重复的文件：${path}`); paths.add(path); bytes += Buffer.byteLength(item.content); if (bytes > MAX_ASSET_BYTES)
+    throw new Error("工具文件总大小超过 512KB"); return { path, content: item.content }; }); }
 function normalizeScript(script, files) { const path = safeRelative(script.path); if (!path.endsWith(".py") || !files.some((x) => x.path === path))
-    throw new Error("script must name an included .py file"); if (script.parameters && Object.values(script.parameters).some((x) => !isParam(x)))
-    throw new Error("script parameter schema is invalid"); return { path, timeoutMs: script.timeoutMs, parameters: script.parameters }; }
+    throw new Error("脚本必须指向文件列表里的一个 .py 文件"); if (script.parameters && Object.values(script.parameters).some((x) => !isParam(x)))
+    throw new Error("脚本的参数定义不合法"); return { path, timeoutMs: script.timeoutMs, parameters: script.parameters }; }
 /**
  * THE PIN IS CHECKED HERE, ON THE WAY IN, AND A RANGE IS A REFUSAL RATHER THAN A WARNING.
  * A warning would throw away exactly what the environment buys: by the time the user learned that
@@ -199,12 +199,12 @@ async function collectAssets(root, dir, files, skipped) {
 async function writeAssets(root, files) { for (const asset of files) {
     const target = resolve(root, asset.path);
     if (!inside(root, target))
-        throw new Error("tool file must stay inside its tool directory");
+        throw new Error("工具文件不能写到工具目录之外");
     await mkdir(dirname(target), { recursive: true });
     await writeFile(target, asset.content, "utf8");
 } }
 function safeRelative(value) { const p = value.replace(/\\/g, "/").trim(); if (!p || p.startsWith("/") || p.includes("\0") || p.split("/").includes(".."))
-    throw new Error("path must be relative and must not escape the tool directory"); return p; }
+    throw new Error("路径必须是相对路径，且不能跳出工具目录"); return p; }
 function inside(root, target) { const r = relative(resolve(root), resolve(target)); return r === "" || (!r.startsWith(`..${sep}`) && r !== ".." && !r.includes(":")); }
 function isParam(value) { const x = value; return !!x && typeof x === "object" && ["string", "number", "boolean", "object", "array"].includes(x.type ?? "") && (x.required === undefined || typeof x.required === "boolean") && (x.enum === undefined || Array.isArray(x.enum)); }
 function isTool(value) { const x = value; if (!x || typeof x !== "object" || typeof x.id !== "string" || typeof x.slug !== "string" || typeof x.name !== "string" || typeof x.entrypoint !== "string" || (x.source !== "user" && x.source !== "builtin"))

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

@@ -0,0 +1,22 @@
+/**
+ * Translates a thrown error from the turn loop into a sentence a person reads.
+ *
+ * The turn loop throws for very different reasons: a model-provider rate limit, a 5xx that Pi's RPC
+ * layer chose to surface as a child exit, an OOM that took the whole process down, a path that
+ * resolved to a directory the workspace contract forbids, an `imageInput: false` mismatch. The
+ * throw message carries the technical fact; what the reader needs is the action it implies - retry,
+ * change model, check the key, change workspace - and that is the level this function writes at.
+ *
+ * The reason the shape is match-on-substring rather than match-on-error-class is that the upstream
+ * error is whatever Pi wrote to its own log: a string, not an exception class, and the host has
+ * nothing to pattern-match against other than the message itself. The literals below are
+ * deliberately permissive on the LLM side ("quota", "rate limit", "usage limit", "too many requests",
+ * "5-hour", "5h") because the underlying model APIs name this limit in several ways, and the
+ * reader cares about the bucket, not the exact wording.
+ *
+ * THIS LIVES IN ITS OWN MODULE so the buckets can be tested without starting a run: the function is
+ * pure, and every branch below is a decision about what a person is told, which is worth a test that
+ * needs no server, socket or Pi process.
+ */
+export declare function formatTurnFailure(error: unknown): string;
+export declare function controllerAborted(error: unknown): boolean;

package/dist/server/turn-failure.js

@@ -0,0 +1,82 @@
+/**
+ * Translates a thrown error from the turn loop into a sentence a person reads.
+ *
+ * The turn loop throws for very different reasons: a model-provider rate limit, a 5xx that Pi's RPC
+ * layer chose to surface as a child exit, an OOM that took the whole process down, a path that
+ * resolved to a directory the workspace contract forbids, an `imageInput: false` mismatch. The
+ * throw message carries the technical fact; what the reader needs is the action it implies - retry,
+ * change model, check the key, change workspace - and that is the level this function writes at.
+ *
+ * The reason the shape is match-on-substring rather than match-on-error-class is that the upstream
+ * error is whatever Pi wrote to its own log: a string, not an exception class, and the host has
+ * nothing to pattern-match against other than the message itself. The literals below are
+ * deliberately permissive on the LLM side ("quota", "rate limit", "usage limit", "too many requests",
+ * "5-hour", "5h") because the underlying model APIs name this limit in several ways, and the
+ * reader cares about the bucket, not the exact wording.
+ *
+ * THIS LIVES IN ITS OWN MODULE so the buckets can be tested without starting a run: the function is
+ * pure, and every branch below is a decision about what a person is told, which is worth a test that
+ * needs no server, socket or Pi process.
+ */
+export function formatTurnFailure(error) {
+    if (controllerAborted(error))
+        return "本轮任务已停止。";
+    const raw = error instanceof Error ? error.message : typeof error === "string" ? error : "";
+    const lower = raw.toLowerCase();
+    // PROVIDER QUOTA / RATE LIMIT - the only reason the user is in this branch is because Pi was
+    // answering them a moment ago and now the model will not let it. The action is "wait or change
+    // model"; the wording has to make that obvious from the banner alone, without a tooltip.
+    if (/\b429\b|quota|rate limit|usage limit|too many requests|5-hour|5h|resource_exhausted|resourc exhaust|hit your limit|usage cap|usage restrict/.test(lower)) {
+        return "模型额度已用完（5 小时限额或类似上限），Pi 已停止回答。可以等待重置后重试，或在会话里换一个模型再发。";
+    }
+    // RPC CHANNEL DEAD - Pi died without producing a `settled` event. Anything in the turn is now
+    // unanswerable, but the deltas the user already saw are still on screen, so the message has to
+    // say "we noticed and stopped" rather than "your input was lost".
+    if (lower.includes("pi rpc channel closed") || lower.includes("agent rpc") || lower.includes("exited before settling")) {
+        return "Pi 进程异常退出，未完成本轮回答。已停止运行，可以直接重发任务继续。";
+    }
+    // NO PI - the launch path failed (binary missing, executable bit, Windows shim). The reader's
+    // best move is a `which pi` / reinstall, and the banner should point there.
+    if (lower.includes("could not find") && lower.includes("pi") || lower.includes("pi command") || lower.startsWith("pi ")) {
+        return `无法启动 Pi：${raw || "未知原因"}。请确认 Pi 已正确安装并可在命令行运行。`;
+    }
+    // WORKSPACE PATH - turn refused on directory the contract forbids. Refusal happens before
+    // anything is on screen, so the banner is also the only place the reason lives.
+    if (lower.includes("workspace directory") || lower.includes("does not exist or is not a directory")) {
+        return `工作区目录无效：${raw}`;
+    }
+    // PROVIDER UNREACHABLE - the request never reached the model. This is the branch a custom
+    // provider with a wrong Base URL lands in, and the person editing that URL needs to be told that
+    // the address or the network is the problem, not that "Pi failed".
+    if (/failed to fetch|fetch failed|econnrefused|econnreset|enotfound|eai_again|socket hang up|connection refused|connection reset|network\s*error/.test(lower)) {
+        return "连不上模型供应商：网络不可达或接口地址不对。请检查供应商的 Base URL 与网络连接，然后重发。";
+    }
+    // PROVIDER AUTH - the key is wrong, missing, or lacks access to this model. Pi reports this as a
+    // 401/403 from the provider; the actionable place is the supplier's key, not the conversation.
+    if (/\b401\b|\b403\b|unauthorized|forbidden|invalid api key|incorrect api key|authentication failed|insufficient permission/.test(lower)) {
+        return "模型供应商拒绝了这次请求：鉴权失败。请在设置里检查这个供应商的 API key，然后重发。";
+    }
+    // PROVIDER 5XX - their side is down. Retrying is the action, and saying so stops the reader from
+    // hunting for a configuration mistake on their own machine.
+    const status = /\bhttp\s*(?:status\s*)?[:=]?\s*(\d{3})\b|\bstatus(?:\s*code)?\s*[:=]\s*(\d{3})\b/.exec(lower);
+    const code = Number(status?.[1] ?? status?.[2] ?? 0);
+    if (code >= 500 && code <= 599 || /internal server error|bad gateway|service unavailable|gateway timeout/.test(lower)) {
+        return `模型供应商服务端出错了${code ? `（HTTP ${code}）` : ""}，通常稍后重试即可。`;
+    }
+    // IMAGE MISMATCH - explicit by the caller, no need to translate.
+    if (lower.includes("不支持图片输入")) {
+        return raw;
+    }
+    // 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 未能完成本轮回答，已标记为失败。可以重发任务继续。";
+}
+export function controllerAborted(error) {
+    if (!error)
+        return false;
+    if (error instanceof Error && error.name === "AbortError")
+        return true;
+    const msg = error instanceof Error ? error.message : typeof error === "string" ? error : "";
+    return /abort|cancel|aborted/i.test(msg);
+}

package/dist/server/types.d.ts

@@ -407,6 +407,14 @@ export type PublicProvider = {
     hasKey: boolean;
     models: PublicProviderModel[];
 };
+export type DiscoveredModel = {
+    id: string;
+    name: string;
+    reasoning: boolean;
+    imageInput: boolean;
+    contextWindow: number;
+    maxTokens: number;
+};
 export type ModelCatalog = {
     source: "pi" | "unavailable";
     models: ModelOption[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evandu/super-pi",
-  "version": "0.1.116",
+  "version": "0.1.118",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -98,6 +98,7 @@
     "scripts/bg-task-zombie.test.mjs",
     "scripts/scoped-sidebar-tabs.test.mjs",
     "scripts/send-retry.test.mjs",
+    "scripts/error-messages.test.mjs",
     "scripts/git-diff-view.test.mjs",
     "scripts/fork-server.test.mjs",
     "scripts/styles-no-dangling.test.mjs",
@@ -120,6 +121,7 @@
     "scripts/check-runner.test.mjs",
     "scripts/skills-discovery.test.mjs",
     "scripts/session-state-icon.test.mjs",
+    "scripts/provider-fetch-models.test.mjs",
     "scripts/shared-chrome.test.mjs"
   ],
   "check:browser": [
@@ -166,7 +168,8 @@
     "scripts/workspace-picker-browser.test.mjs",
     "scripts/optimistic-browser.test.mjs",
     "scripts/active-memory-browser.test.mjs",
-    "scripts/knowledge-base-browser.test.mjs"
+    "scripts/knowledge-base-browser.test.mjs",
+    "scripts/ux-polish-browser.test.mjs"
   ],
   "dependencies": {
     "@codemirror/commands": "^6.11.1",