@evandu/super-pi 0.1.125 → 0.1.127

package/dist/client/index.html

@@ -8,8 +8,8 @@
     <link rel="alternate icon" href="/favicon.ico" />
     <link rel="apple-touch-icon" href="/favicon.svg" />
     <title>Super Pi</title>
-    <script type="module" crossorigin src="/assets/index-qTPZ6eno.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-CBPWeTvl.css">
+    <script type="module" crossorigin src="/assets/index-DvxaTOG8.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-BAaymrG2.css">
   </head>
   <body>
     <div id="root"></div>

package/dist/server/application/catalog-service.d.ts

@@ -1,5 +1,5 @@
 import type { AgentRuntime } from "../runtime.js";
-import type { AgentCatalog, ModelCatalog, ModelSelection } from "../types.js";
+import type { AgentCatalog, ModelCatalog, ModelSelection, PromptCatalog } from "../types.js";
 import type { EventBroadcaster } from "../server-context.js";
 import type { Store } from "../store.js";
 /**
@@ -12,6 +12,7 @@ export declare class CatalogService {
     private readonly rootDir?;
     private readonly onDirsChanged?;
     private agentDirs;
+    private readonly promptCatalogs;
     constructor(runtime: AgentRuntime, broadcaster: EventBroadcaster, agentsDir: string | readonly string[], store?: Store | undefined, rootDir?: string | undefined, onDirsChanged?: ((dirs: string[]) => void) | undefined);
     /**
      * Returns current active agent discovery directories.
@@ -67,6 +68,8 @@ export declare class CatalogService {
      * Broadcasts subagents to connected clients.
      */
     broadcastSubagents(): Promise<void>;
+    promptCatalog(workspaceId: string): Promise<PromptCatalog>;
+    broadcastPromptCatalog(workspaceId: string): Promise<void>;
     /**
      * Sets the app-wide default model, or clears it back to "Pi's own default".
      *

package/dist/server/application/catalog-service.js

@@ -1,5 +1,6 @@
 import { assertModelAllowed, conversationWorkspace } from "../model-filter.js";
 import { readAgents, resolveAgentDirectories } from "../agents.js";
+import { loadPromptTemplateSummaries } from "../pi-session.js";
 function summarize(agent) {
     const { prompt, toolFlags, ...summary } = agent;
     const toolSummary = toolFlags.includes("--no-tools")
@@ -21,6 +22,7 @@ export class CatalogService {
     rootDir;
     onDirsChanged;
     agentDirs;
+    promptCatalogs = new Map();
     constructor(runtime, broadcaster, agentsDir, store, rootDir, onDirsChanged) {
         this.runtime = runtime;
         this.broadcaster = broadcaster;
@@ -176,6 +178,25 @@ export class CatalogService {
     async broadcastSubagents() {
         this.broadcaster.broadcast({ type: "subagent.catalog", catalog: await this.subagentCatalog() });
     }
+    async promptCatalog(workspaceId) {
+        const cached = this.promptCatalogs.get(workspaceId);
+        if (cached && cached.expiresAt > Date.now())
+            return cached.catalog;
+        const workspace = this.store?.snapshot().workspaces.find((item) => item.id === workspaceId);
+        if (!workspace)
+            return { source: "unavailable", prompts: [], workspaceId, fetchedAt: new Date().toISOString(), error: "workspace not found" };
+        try {
+            const catalog = { source: "sdk", prompts: await loadPromptTemplateSummaries(workspace.path), workspaceId, fetchedAt: new Date().toISOString() };
+            this.promptCatalogs.set(workspaceId, { expiresAt: Date.now() + 1_000, catalog });
+            return catalog;
+        }
+        catch (error) {
+            return { source: "unavailable", prompts: [], workspaceId, fetchedAt: new Date().toISOString(), error: error instanceof Error ? error.message : String(error) };
+        }
+    }
+    async broadcastPromptCatalog(workspaceId) {
+        this.broadcaster.broadcast({ type: "prompt.catalog", catalog: await this.promptCatalog(workspaceId) });
+    }
     /**
      * Sets the app-wide default model, or clears it back to "Pi's own default".
      *

package/dist/server/application/provider-service.d.ts

@@ -56,6 +56,20 @@ export declare class ProviderService {
      */
     disable(providerId: string): Promise<PublicProvider[]>;
     enable(providerId: string): Promise<PublicProvider[]>;
+    /**
+     * Repairs Pi's model configuration: the empty fields Pi's schema refuses are dropped, and nothing
+     * else in the file is touched.
+     *
+     * WHY IT IS WORTH A BUTTON. Pi validates the WHOLE `models.json`, so one entry with an empty
+     * `baseUrl` takes every custom provider out of every picker - and nothing on screen says so,
+     * because the settings panel reads the file leniently and lists them as if they worked. The repair
+     * restores the file Pi will accept, which is the difference between "my provider is gone" and
+     * "your file had an empty field".
+     */
+    repair(): Promise<{
+        fixed: string[];
+        providers: PublicProvider[];
+    }>;
     /**
      * Adds a provider, or adds a model to one, or updates one.
      *

package/dist/server/application/provider-service.js

@@ -135,6 +135,22 @@ export class ProviderService {
         await this.afterChange();
         return this.list();
     }
+    /**
+     * Repairs Pi's model configuration: the empty fields Pi's schema refuses are dropped, and nothing
+     * else in the file is touched.
+     *
+     * WHY IT IS WORTH A BUTTON. Pi validates the WHOLE `models.json`, so one entry with an empty
+     * `baseUrl` takes every custom provider out of every picker - and nothing on screen says so,
+     * because the settings panel reads the file leniently and lists them as if they worked. The repair
+     * restores the file Pi will accept, which is the difference between "my provider is gone" and
+     * "your file had an empty field".
+     */
+    async repair() {
+        const result = await this.registry.repairModelsConfig();
+        if (result.changed)
+            await this.afterChange();
+        return { fixed: result.fixed, providers: await this.list() };
+    }
     /**
      * Adds a provider, or adds a model to one, or updates one.
      *

package/dist/server/command-handler.js

@@ -122,6 +122,12 @@ export class CommandHandler {
             await this.providerService.enable(command.providerId);
             return;
         }
+        if (command.type === "providers.repair") {
+            if (!this.providerService)
+                throw new Error("供应商功能不可用");
+            await this.providerService.repair();
+            return;
+        }
         if (command.type === "providers.fetch") {
             if (!this.providerService)
                 throw new Error("供应商功能不可用");
@@ -326,6 +332,12 @@ export class CommandHandler {
                 sendToSocket({ type: "model.catalog.all", catalog: await this.catalogService.getAllCatalog() });
             return;
         }
+        if (command.type === "prompt.catalog.fetch") {
+            const catalog = await this.catalogService.promptCatalog(command.workspaceId);
+            if (sendToSocket)
+                sendToSocket({ type: "prompt.catalog", catalog });
+            return;
+        }
         if (command.type === "agent.catalog.fetch") {
             await this.catalogService.broadcastAgents();
             return;

package/dist/server/pi-session.d.ts

@@ -77,6 +77,15 @@ export interface SdkCatalog {
         provider: string;
         modelId: string;
     }): Promise<string[] | undefined>;
+    /**
+     * WHAT PI SAYS IS WRONG WITH ITS OWN MODEL CONFIGURATION, verbatim.
+     *
+     * Pi validates the WHOLE `models.json` against a schema, and one entry it refuses takes every
+     * custom provider out of every picker - while a lenient reader of the same file still lists them
+     * all. Pi's own words are the only ones that name the offending field, so they are what the
+     * settings panel shows instead of a guess.
+     */
+    configError(): string | undefined;
 }
 /**
  * Builds a zero-process catalog reader.
@@ -361,6 +370,21 @@ export interface SessionCreation {
  */
 export type PiSessionFactory = (creation: SessionCreation) => Promise<PiSessionLike>;
 export declare function createPiSession(creation: SessionCreation): Promise<PiSessionLike>;
+/**
+ * Pi's own global configuration directory, when it can be worked out without importing the SDK.
+ *
+ * The SDK's `getAgentDir()` is the authority, but asking it would mean importing the SDK - which this
+ * file goes out of its way not to do for a path it can derive. The derivation is Pi's own default:
+ * `~/.pi/agent`, with the home directory taken from the environment rather than from `os.homedir()`,
+ * because on Windows the two disagree often enough to matter.
+ *
+ * `PI_CODING_AGENT_DIR` WINS, exactly as it wins in Pi (`core/config.ts:getAgentDir`). A person who
+ * set it expects Pi to read its whole configuration from there, and this function used to disagree
+ * with Pi about it: Pi's model runtime read `models.json` from that directory while the session's
+ * resource loader read the skills from `~/.pi/agent`. Two halves of one Pi pointed at two
+ * directories, and only one of them was the one the person had configured.
+ */
+export declare function defaultAgentDir(): string;
 /**
  * The Super Pi extension files, as absolute paths.
  *
@@ -369,4 +393,11 @@ export declare function createPiSession(creation: SessionCreation): Promise<PiSe
  * rather than against the server's directory, so a relative path would find a different file for every
  * workspace. The SDK takes an absolute path, and this keeps the two paths of the same file.
  */
+export interface PromptTemplateSummary {
+    name: string;
+    description: string;
+    argumentHint?: string;
+    scope: "user" | "project";
+}
+export declare function loadPromptTemplateSummaries(cwd: string): Promise<PromptTemplateSummary[]>;
 export declare function superPiExtensionPaths(repositoryRoot: string): string[];

package/dist/server/pi-session.js

@@ -24,7 +24,8 @@
  * would notice the difference.
  */
 import { existsSync, statSync } from "node:fs";
-import { join } from "node:path";
+import { dirname, join } from "node:path";
+import { fileURLToPath, pathToFileURL } from "node:url";
 const traceSession = (conversationId, phase, startedAt, extra = "") => {
     if (process.env.SUPER_PI_TRACE_LATENCY !== "1")
         return;
@@ -89,6 +90,7 @@ export async function createSdkCatalog(cwd) {
     const levelsByModel = new Map();
     return {
         models,
+        configError: () => modelRuntime.getError?.(),
         ...(settingsManager.getDefaultProvider?.() && settingsManager.getDefaultModel?.()
             ? {
                 defaultModel: {
@@ -352,21 +354,30 @@ async function resolveSessionManager(sdk, creation) {
  * resource loader read the skills from `~/.pi/agent`. Two halves of one Pi pointed at two
  * directories, and only one of them was the one the person had configured.
  */
-function defaultAgentDir() {
+export function defaultAgentDir() {
     const configured = process.env.PI_CODING_AGENT_DIR?.trim();
     if (configured)
         return configured;
     const home = process.env.HOME ?? process.env.USERPROFILE ?? "";
     return home ? join(home, ".pi", "agent") : join(".pi", "agent");
 }
-/**
- * The Super Pi extension files, as absolute paths.
- *
- * Resolved from the repository root, exactly as `runtime.ts` resolves the `--extension` argument it
- * passes to a child process: Pi resolves a relative `--extension` against the CONVERSATION's workspace
- * rather than against the server's directory, so a relative path would find a different file for every
- * workspace. The SDK takes an absolute path, and this keeps the two paths of the same file.
- */
+let promptTemplateLoader;
+/** Reads Pi's own prompt-template loader rather than duplicating its discovery rules. */
+async function loadPromptTemplateLoader() {
+    if (!promptTemplateLoader) {
+        const packageRoot = dirname(fileURLToPath(await import.meta.resolve("@earendil-works/pi-coding-agent")));
+        promptTemplateLoader = import(pathToFileURL(join(packageRoot, "core", "prompt-templates.js")).href)
+            .then((module) => module.loadPromptTemplates);
+    }
+    return promptTemplateLoader;
+}
+export async function loadPromptTemplateSummaries(cwd) {
+    const loadPromptTemplates = await loadPromptTemplateLoader();
+    const templates = loadPromptTemplates({ cwd, agentDir: defaultAgentDir(), promptPaths: [], includeDefaults: true });
+    return templates
+        .filter((template) => template.sourceInfo.scope === "user" || template.sourceInfo.scope === "project")
+        .map((template) => ({ name: template.name, description: template.description, ...(template.argumentHint ? { argumentHint: template.argumentHint } : {}), scope: template.sourceInfo.scope }));
+}
 export function superPiExtensionPaths(repositoryRoot) {
     const paths = [
         join(repositoryRoot, "agents", "super-pi-task", "index.ts"),

package/dist/server/providers.d.ts

@@ -64,6 +64,16 @@ export type ProviderDefinition = {
     apiKey?: string;
     hasAuthKey?: boolean;
     models: ProviderModelDefinition[];
+    /**
+     * The fields in this provider's `models.json` entry that Pi's own schema refuses - today, an
+     * empty string where Pi asks for at least one character.
+     *
+     * WHY THE PANEL CARES. Pi validates the WHOLE file, so one bad entry takes every custom
+     * provider out of every picker, while this panel - which reads the file leniently - still lists
+     * them all. Naming the field is what turns "my provider is gone" into "your file has an empty
+     * baseUrl".
+     */
+    invalidFields?: string[];
     /**
      * The key is written into `models.json` itself rather than into `auth.json`.
      *
@@ -227,6 +237,23 @@ export declare class ProviderRegistry {
     removeCredential(providerId: string): Promise<void>;
     /** Forgets a disabled provider without restoring anything: the permanent form of "off". */
     forgetDisabled(providerId: string): Promise<void>;
+    /**
+     * Removes the empty fields Pi's schema refuses, so the file is accepted again.
+     *
+     * WHY THIS IS A REPAIR AND NOT A DELETE. An empty `baseUrl` is not a provider with no address -
+     * it is a value Pi refuses, and Pi refuses the WHOLE FILE when it sees one. The fix is to drop
+     * the empty field (leaving the provider exactly as it was otherwise), NOT to remove the provider:
+     * the person wrote it, and an override-only entry - one that only adds models to a built-in - has
+     * no address at all, which is legal as long as the field is ABSENT rather than empty.
+     *
+     * NOTHING ELSE IN THE FILE IS TOUCHED. The entries are read as raw JSON and written back with
+     * only the offending fields removed, so a comment-free copy of a hand-written file stays a
+     * hand-written file.
+     */
+    repairModelsConfig(): Promise<{
+        fixed: string[];
+        changed: boolean;
+    }>;
     /** Writes the registry. The key lives here, so the file is 0600. */
     save(providers: ProviderDefinition[]): Promise<void>;
     /**

package/dist/server/providers.js

@@ -167,6 +167,21 @@ async function writeJsonAtomic(path, value) {
         throw error;
     }
 }
+/**
+ * The fields of a `models.json` provider entry that Pi's schema refuses.
+ *
+ * PI'S RULE, NOT OURS: `baseUrl`, `name` and `api` are optional, but when they are present they
+ * need at least one character. An empty string is not "absent" - it is a value Pi rejects, and Pi
+ * rejects the WHOLE FILE when it finds one, which is why one bad entry can empty every picker.
+ */
+function invalidModelFields(entry) {
+    const fields = [];
+    for (const field of ["name", "baseUrl", "api"]) {
+        if (entry[field] === "")
+            fields.push(field);
+    }
+    return fields;
+}
 /**
  * The provider entry Super Pi writes into Pi's `models.json`, MERGED into what was already there.
  *
@@ -200,7 +215,10 @@ function toPiProvider(provider, existing) {
         return {
             ...priorModel,
             id: model.id,
-            name: model.name ?? model.id,
+            // NEVER AN EMPTY STRING. Pi's schema asks for at least one character, and one empty name
+            // invalidates the WHOLE models.json - which takes every custom provider out of every picker
+            // with nothing but "must not have fewer than 1 characters" to explain it.
+            name: model.name || model.id,
             reasoning: model.reasoning ?? false,
             input: model.imageInput ? ["text", "image"] : ["text"],
             contextWindow: model.contextWindow ?? DEFAULT_CONTEXT_WINDOW,
@@ -325,6 +343,12 @@ export class ProviderRegistry {
                 // credential - and the panel then says "Pi will not offer it", which is wrong exactly when
                 // the environment variable is set and Pi is offering it right now.
                 ...(typeof rec.apiKey === "string" && rec.apiKey ? { keyInModelsJson: true } : {}),
+                // THE FIELDS PI WILL REFUSE. Pi validates the WHOLE models.json against a schema where an
+                // empty string is not a value ("must not have fewer than 1 characters"), and one bad entry
+                // invalidates the file - which takes EVERY custom provider out of every picker while this
+                // panel, which reads the file leniently, still lists them all. Naming the offending fields
+                // is what turns "my provider vanished" into "your file has an empty baseUrl".
+                ...(invalidModelFields(rec).length ? { invalidFields: invalidModelFields(rec) } : {}),
                 models,
             });
         }
@@ -485,6 +509,63 @@ export class ProviderRegistry {
             throw new Error(`供应商「${providerId}」没有被停用`);
         await this.writeRegistry({ disabled: next });
     }
+    /**
+     * Removes the empty fields Pi's schema refuses, so the file is accepted again.
+     *
+     * WHY THIS IS A REPAIR AND NOT A DELETE. An empty `baseUrl` is not a provider with no address -
+     * it is a value Pi refuses, and Pi refuses the WHOLE FILE when it sees one. The fix is to drop
+     * the empty field (leaving the provider exactly as it was otherwise), NOT to remove the provider:
+     * the person wrote it, and an override-only entry - one that only adds models to a built-in - has
+     * no address at all, which is legal as long as the field is ABSENT rather than empty.
+     *
+     * NOTHING ELSE IN THE FILE IS TOUCHED. The entries are read as raw JSON and written back with
+     * only the offending fields removed, so a comment-free copy of a hand-written file stays a
+     * hand-written file.
+     */
+    async repairModelsConfig() {
+        const modelsPath = piModelsPath(this.piDir);
+        const raw = (await readJsonFile(modelsPath));
+        if (!raw || !raw.providers || typeof raw.providers !== "object")
+            return { fixed: [], changed: false };
+        const fixed = [];
+        let changed = false;
+        const providers = { ...raw.providers };
+        for (const [providerId, value] of Object.entries(providers)) {
+            if (!value || typeof value !== "object" || Array.isArray(value))
+                continue;
+            const entry = { ...value };
+            let touched = false;
+            for (const field of invalidModelFields(entry)) {
+                delete entry[field];
+                fixed.push(`${providerId}.${field}`);
+                touched = true;
+            }
+            if (Array.isArray(entry.models)) {
+                let modelsTouched = false;
+                const models = entry.models.map((model) => {
+                    if (!model || typeof model !== "object" || Array.isArray(model))
+                        return model;
+                    const record = { ...model };
+                    if (record.name === "") {
+                        delete record.name;
+                        fixed.push(`${providerId}.models.name`);
+                        modelsTouched = true;
+                    }
+                    return record;
+                });
+                if (modelsTouched)
+                    entry.models = models;
+                touched = touched || modelsTouched;
+            }
+            if (touched) {
+                providers[providerId] = entry;
+                changed = true;
+            }
+        }
+        if (changed)
+            await writeJsonAtomic(modelsPath, { ...raw, providers });
+        return { fixed, changed };
+    }
     /** Writes the registry. The key lives here, so the file is 0600. */
     async save(providers) {
         await this.writeRegistry({ providers });
@@ -541,6 +622,7 @@ export function publicProviders(providers) {
         api: provider.api,
         hasKey: Boolean(provider.apiKey || provider.hasAuthKey),
         ...(provider.keyInModelsJson ? { keyInModelsJson: true } : {}),
+        ...(provider.invalidFields?.length ? { invalidFields: [...provider.invalidFields] } : {}),
         ...(provider.builtin ? { builtin: true } : {}),
         ...(provider.disabled ? { disabled: true } : {}),
         models: provider.models.map(model => ({

package/dist/server/runtime.js

@@ -2445,6 +2445,11 @@ export class PiRpcRuntime {
                 source: "pi",
                 models,
                 levels: levelNames,
+                // PI'S OWN VERDICT ON ITS OWN CONFIGURATION. It belongs on the catalog because the catalog is
+                // what the picker shows, and a picker that is missing a provider without saying why is the
+                // hardest kind of missing: Pi rejects a whole models.json over one bad field, so EVERY
+                // custom provider disappears at once and the only explanation is somewhere else entirely.
+                ...(reader.configError?.() ? { configError: reader.configError() } : {}),
                 defaultModel: pick(reader.defaultModel ? describeModel({ provider: reader.defaultModel.provider, id: reader.defaultModel.modelId, name: reader.defaultModel.modelId }) : undefined),
                 defaultThinkingLevel: reader.defaultThinkingLevel ?? levelNames[0],
                 fetchedAt: new Date().toISOString(),

package/dist/server/types.d.ts

@@ -410,6 +410,12 @@ export type PublicProvider = {
      * in `auth.json`. It is a credential all the same, so the panel must not report it as missing.
      */
     keyInModelsJson?: boolean;
+    /**
+     * The fields of this provider's `models.json` entry that Pi's schema refuses (an empty string
+     * where Pi asks for at least one character). Pi rejects the WHOLE file over one of them, so every
+     * custom provider leaves the picker at once.
+     */
+    invalidFields?: string[];
     /**
      * TRUE when the provider is one of Pi's own, with no entry in `models.json`.
      *
@@ -441,6 +447,13 @@ export type ModelCatalog = {
     defaultThinkingLevel?: string;
     fetchedAt: string;
     error?: string;
+    /**
+     * What Pi says is wrong with its own model configuration, in Pi's own words.
+     *
+     * Pi validates the whole `models.json`, so one entry it refuses empties every picker of custom
+     * providers. This is the only place that says which field is at fault.
+     */
+    configError?: string;
 };
 export type Conversation = {
     id: string;
@@ -535,6 +548,19 @@ export type AgentSummary = Omit<AgentDefinition, "prompt" | "toolFlags" | "tools
     toolSummary: string;
     sourceDir?: string;
 };
+export type PromptTemplateSummary = {
+    name: string;
+    description: string;
+    argumentHint?: string;
+    scope: "user" | "project";
+};
+export type PromptCatalog = {
+    source: "sdk" | "unavailable";
+    prompts: PromptTemplateSummary[];
+    workspaceId: string;
+    fetchedAt: string;
+    error?: string;
+};
 export type AgentCatalog = {
     source: "directory" | "unavailable";
     agents: AgentSummary[];
@@ -1000,7 +1026,16 @@ export type ClientCommandBody =
     modelId: string;
 }
 /** Asks for the providers Super Pi manages. The answer carries no key. */
+/**
+ * Repairs Pi's model configuration: the empty fields Pi's schema refuses are dropped from
+ * `models.json`, and nothing else in the file is touched.
+ *
+ * Pi validates the whole file, so one refused entry takes every custom provider out of every
+ * picker. This is the way back.
+ */
  | {
+    type: "providers.repair";
+} | {
     type: "providers.fetch";
 }
 /**
@@ -1154,6 +1189,11 @@ export type ClientCommandBody =
  | {
     type: "agent.catalog.fetch";
 }
+/** Asks for prompt templates loaded by Pi for a workspace. */
+ | {
+    type: "prompt.catalog.fetch";
+    workspaceId: string;
+}
 /**
  * Sends a message to a conversation, and THAT IS ALL IT CLAIMS.
  *
@@ -1559,6 +1599,9 @@ export type ServerEvent = {
 } | {
     type: "agent.catalog";
     catalog: AgentCatalog;
+} | {
+    type: "prompt.catalog";
+    catalog: PromptCatalog;
 }
 /**
  * The `mode: subagent` half of the same directory, as its OWN event rather than an extra key on

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evandu/super-pi",
-  "version": "0.1.125",
+  "version": "0.1.127",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -123,7 +123,8 @@
     "scripts/skills-discovery.test.mjs",
     "scripts/session-state-icon.test.mjs",
     "scripts/provider-fetch-models.test.mjs",
-    "scripts/shared-chrome.test.mjs"
+    "scripts/shared-chrome.test.mjs",
+    "scripts/prompt-templates.test.mjs"
   ],
   "check:browser": [
     "scripts/interrupt-browser.test.mjs",
@@ -172,7 +173,8 @@
     "scripts/active-memory-browser.test.mjs",
     "scripts/knowledge-base-browser.test.mjs",
     "scripts/terminal-browser.test.mjs",
-    "scripts/ux-polish-browser.test.mjs"
+    "scripts/ux-polish-browser.test.mjs",
+    "scripts/prompt-templates-browser.test.mjs"
   ],
   "dependencies": {
     "@codemirror/commands": "^6.11.1",