@psnext/slingcli 2.5.20260724-1 → 2.5.20260727-1

package/node_modules/@earendil-works/pi-coding-agent/dist/core/compaction/compaction.js

@@ -4,7 +4,7 @@
  * Pure functions for compaction logic. The session manager handles I/O,
  * and after compaction the session is reloaded.
  */
-import { contentText, retryAssistantCall } from "@earendil-works/pi-ai";
+import { contentText, retryAssistantCall, uuidv7 } from "@earendil-works/pi-ai";
 import { completeSimple } from "@earendil-works/pi-ai/compat";
 import { convertToLlm } from "../messages.js";
 import { buildSessionContext, sessionEntryToContextMessages, } from "../session-manager.js";
@@ -438,8 +438,16 @@ function createSummarizationOptions(model, maxTokens, apiKey, headers, env, sign
  * immediately (see {@link retryAssistantCall}).
  */
 export async function completeSummarization(model, context, options, streamFn, retry, callbacks) {
-    const produce = async () => streamFn ? (await streamFn(model, context, options)).result() : completeSimple(model, context, options);
-    return retryAssistantCall(produce, retry, options.signal, callbacks);
+    // Summaries are standalone requests, so isolate routing and avoid cache writes that cannot be reused.
+    const requestOptions = {
+        ...options,
+        cacheRetention: "none",
+        sessionId: uuidv7(),
+    };
+    const produce = async () => streamFn
+        ? (await streamFn(model, context, requestOptions)).result()
+        : completeSimple(model, context, requestOptions);
+    return retryAssistantCall(produce, retry, requestOptions.signal, callbacks);
 }
 /**
  * Generate a summary of the conversation using the LLM.

package/node_modules/@earendil-works/pi-coding-agent/dist/core/extensions/runner.js

@@ -483,6 +483,10 @@ export class ExtensionRunner {
                 runner.assertActive();
                 return getModel();
             },
+            get thinkingLevel() {
+                runner.assertActive();
+                return runner.runtime.getThinkingLevel();
+            },
             isIdle: () => {
                 runner.assertActive();
                 return runner.isIdleFn();

package/node_modules/@earendil-works/pi-coding-agent/dist/core/model-config.js

@@ -83,6 +83,7 @@ const OpenAICompletionsCompatSchema = Type.Object({
     cacheControlFormat: Type.Optional(Type.Literal("anthropic")),
     openRouterRouting: Type.Optional(OpenRouterRoutingSchema),
     vercelGatewayRouting: Type.Optional(VercelGatewayRoutingSchema),
+    supportsOpenAIGrammarTools: Type.Optional(Type.Boolean()),
     supportsStrictMode: Type.Optional(Type.Boolean()),
     sendSessionAffinityHeaders: Type.Optional(Type.Boolean()),
     deferredToolsMode: Type.Optional(Type.Literal("kimi")),
@@ -93,6 +94,8 @@ const OpenAIResponsesCompatSchema = Type.Object({
     supportsDeveloperRole: Type.Optional(Type.Boolean()),
     sessionAffinityFormat: Type.Optional(Type.Union([Type.Literal("openai"), Type.Literal("openai-nosession"), Type.Literal("openrouter")])),
     supportsLongCacheRetention: Type.Optional(Type.Boolean()),
+    supportsStrictMode: Type.Optional(Type.Boolean()),
+    supportsOpenAIGrammarTools: Type.Optional(Type.Boolean()),
     supportsToolSearch: Type.Optional(Type.Boolean()),
 });
 const AnthropicMessagesCompatSchema = Type.Object({
@@ -100,7 +103,10 @@ const AnthropicMessagesCompatSchema = Type.Object({
     supportsLongCacheRetention: Type.Optional(Type.Boolean()),
     sendSessionAffinityHeaders: Type.Optional(Type.Boolean()),
     supportsCacheControlOnTools: Type.Optional(Type.Boolean()),
+    supportsTemperature: Type.Optional(Type.Boolean()),
     forceAdaptiveThinking: Type.Optional(Type.Boolean()),
+    allowEmptySignature: Type.Optional(Type.Boolean()),
+    supportsStrictTools: Type.Optional(Type.Boolean()),
     supportsToolReferences: Type.Optional(Type.Boolean()),
 });
 const ProviderCompatSchema = Type.Union([

package/node_modules/@earendil-works/pi-coding-agent/dist/core/model-registry.js

@@ -9,8 +9,8 @@ export class ModelRegistry {
         this.runtime = runtime;
     }
     /** Reload models.json asynchronously. Await before making synchronous registry reads. */
-    refresh() {
-        return this.runtime.reloadConfig();
+    async refresh() {
+        await this.runtime.refresh();
     }
     getError() {
         return this.runtime.getError();

package/node_modules/@earendil-works/pi-coding-agent/dist/core/model-resolver.js

@@ -216,6 +216,13 @@ export async function resolveModelScopeWithDiagnostics(patterns, modelRuntime) {
                     globPattern = pattern.substring(0, colonIdx);
                 }
             }
+            const exactMatch = findExactModelReferenceMatch(globPattern, availableModels);
+            if (exactMatch) {
+                if (!scopedModels.find((sm) => modelsAreEqual(sm.model, exactMatch))) {
+                    scopedModels.push({ model: exactMatch, thinkingLevel });
+                }
+                continue;
+            }
             // Match against "provider/modelId" format OR just model ID
             // This allows "*sonnet*" to match without requiring "anthropic/*sonnet*"
             const matchingModels = availableModels.filter((m) => {
@@ -223,7 +230,12 @@ export async function resolveModelScopeWithDiagnostics(patterns, modelRuntime) {
                 return minimatch(fullId, globPattern, { nocase: true }) || minimatch(m.id, globPattern, { nocase: true });
             });
             if (matchingModels.length === 0) {
-                diagnostics.push({ type: "warning", message: `No models match pattern "${pattern}"`, pattern });
+                diagnostics.push({
+                    type: "warning",
+                    code: "no-match",
+                    message: `No models match pattern "${pattern}"`,
+                    pattern,
+                });
                 continue;
             }
             for (const model of matchingModels) {
@@ -235,10 +247,15 @@ export async function resolveModelScopeWithDiagnostics(patterns, modelRuntime) {
         }
         const { model, thinkingLevel, warning } = parseModelPattern(pattern, availableModels);
         if (warning) {
-            diagnostics.push({ type: "warning", message: warning, pattern });
+            diagnostics.push({ type: "warning", code: "invalid-thinking-level", message: warning, pattern });
         }
         if (!model) {
-            diagnostics.push({ type: "warning", message: `No models match pattern "${pattern}"`, pattern });
+            diagnostics.push({
+                type: "warning",
+                code: "no-match",
+                message: `No models match pattern "${pattern}"`,
+                pattern,
+            });
             continue;
         }
         // Avoid duplicates

package/node_modules/@earendil-works/pi-coding-agent/dist/core/model-runtime.js

@@ -62,11 +62,12 @@ export class ModelRuntime {
             (modelsPath
                 ? new FileModelsStore(options.modelsStorePath ?? join(dirname(modelsPath), "models-store.json"))
                 : new InMemoryCodingAgentModelsStore());
+        const builtinModelDataGeneratedAt = builtinProviderCatalog.getBuiltinModelDataGeneratedAt();
         const providers = builtinProviderCatalog
             .builtinProviders()
             .map((provider) => provider.id === "radius"
             ? provider
-            : withRemoteCatalog(provider, options.catalogBaseUrl, builtinProviderCatalog.getBuiltinModelDataUrl(provider.id)));
+            : withRemoteCatalog(provider, options.catalogBaseUrl, builtinModelDataGeneratedAt));
         const runtime = new ModelRuntime(credentials, config, modelsPath, modelsStore, providers, process.env.PI_OFFLINE === undefined);
         runtime.configureRadiusProviders();
         runtime.rebuildProviders();
@@ -359,13 +360,10 @@ export class ModelRuntime {
         this.recomposeProvider(providerId);
         await this.refresh({ allowNetwork: this.modelNetworkEnabled });
     }
-    async reloadConfig() {
+    async refresh(options = {}) {
         this.config = await ModelConfig.load(this.modelsPath);
         this.configureRadiusProviders();
         this.rebuildProviders();
-        await this.refresh({ allowNetwork: this.modelNetworkEnabled });
-    }
-    async refresh(options = {}) {
         const refreshOptions = {
             ...options,
             allowNetwork: options.allowNetwork ?? this.modelNetworkEnabled,

package/node_modules/@earendil-works/pi-coding-agent/dist/core/remote-catalog-provider.js

@@ -1,4 +1,3 @@
-import { stat } from "node:fs/promises";
 import { VERSION } from "../config.js";
 import { getPiUserAgent } from "../utils/pi-user-agent.js";
 const DEFAULT_CATALOG_BASE_URL = "https://pi.dev";
@@ -28,17 +27,16 @@ function parseCatalog(providerId, value) {
         .filter((entry) => typeof entry === "object" && entry !== null && "id" in entry)
         .map((model) => ({ ...model, provider: providerId }));
 }
-function remoteModels(entry, localLastModified) {
+function remoteModels(entry, localGeneratedAt) {
     if (!entry)
         return [];
-    if (localLastModified !== undefined &&
-        (entry.lastModified === undefined || entry.lastModified <= localLastModified)) {
+    if (localGeneratedAt !== undefined && (entry.lastModified === undefined || entry.lastModified <= localGeneratedAt)) {
         return [];
     }
     return entry.models;
 }
 /** Add a persisted pi.dev catalog overlay to a static built-in provider. */
-export function withRemoteCatalog(provider, catalogBaseUrl = DEFAULT_CATALOG_BASE_URL, localCatalogUrl) {
+export function withRemoteCatalog(provider, catalogBaseUrl = DEFAULT_CATALOG_BASE_URL, localGeneratedAt) {
     let dynamicModels = [];
     let inflightRefresh;
     return {
@@ -47,11 +45,8 @@ export function withRemoteCatalog(provider, catalogBaseUrl = DEFAULT_CATALOG_BAS
         refreshModels: (context) => {
             inflightRefresh ??= (async () => {
                 try {
-                    const localLastModified = localCatalogUrl
-                        ? await stat(localCatalogUrl).then((value) => value.mtimeMs, () => undefined)
-                        : undefined;
                     const stored = await context.store.read();
-                    dynamicModels = remoteModels(stored, localLastModified).filter((model) => model.provider === provider.id);
+                    dynamicModels = remoteModels(stored, localGeneratedAt).filter((model) => model.provider === provider.id);
                     if (!context.allowNetwork || context.signal?.aborted)
                         return;
                     if (!context.force &&
@@ -60,22 +55,39 @@ export function withRemoteCatalog(provider, catalogBaseUrl = DEFAULT_CATALOG_BAS
                         Date.now() - stored.checkedAt < REMOTE_CATALOG_REFRESH_INTERVAL_MS) {
                         return;
                     }
+                    // Only revalidate when a cached body backs the validator, so a 304 can never
+                    // leave the overlay empty.
+                    const validator = stored?.models.length ? stored.etag : undefined;
                     const url = new URL(`/api/models/providers/${encodeURIComponent(provider.id)}`, catalogBaseUrl);
                     const response = await fetch(url, {
                         headers: {
                             accept: "application/json",
                             "User-Agent": getPiUserAgent(VERSION),
+                            ...(validator ? { "if-none-match": validator } : {}),
                         },
                         signal: context.signal,
                     });
                     if (context.signal?.aborted)
                         return;
                     const checkedAt = Date.now();
+                    // Unchanged: dynamicModels already holds the stored overlay, so only the
+                    // freshness window moves.
+                    if (response.status === 304 && stored) {
+                        await context.store.write({ ...stored, checkedAt });
+                        return;
+                    }
                     if (response.status === 404 || response.status === 501) {
-                        await context.store.write({ ...(stored ?? { models: [] }), checkedAt, lastModified: 0 });
+                        await context.store.write({
+                            ...(stored ?? { models: [] }),
+                            checkedAt,
+                            lastModified: 0,
+                            etag: undefined,
+                        });
                         return;
                     }
                     if (!response.ok) {
+                        // Transient failure: the cached body and its validator stay valid, so keep the
+                        // etag and let the next refresh revalidate instead of downloading the catalog.
                         await context.store.write({ ...(stored ?? { models: [] }), checkedAt });
                         throw new Error(`Model catalog request failed for ${provider.id}: ${response.status}`);
                     }
@@ -87,8 +99,9 @@ export function withRemoteCatalog(provider, catalogBaseUrl = DEFAULT_CATALOG_BAS
                         models: refreshed,
                         checkedAt,
                         lastModified: Number.isNaN(lastModified) ? 0 : lastModified,
+                        etag: response.headers.get("etag") ?? undefined,
                     };
-                    dynamicModels = remoteModels(entry, localLastModified);
+                    dynamicModels = remoteModels(entry, localGeneratedAt);
                     await context.store.write(entry);
                 }
                 finally {

package/node_modules/@earendil-works/pi-coding-agent/dist/core/resource-loader.js

@@ -33,6 +33,9 @@ function loadContextFileFromDir(dir) {
         const filePath = join(dir, filename);
         if (existsSync(filePath)) {
             try {
+                if (!statSync(filePath).isFile()) {
+                    continue;
+                }
                 return {
                     path: filePath,
                     content: readFileSync(filePath, "utf-8"),

package/node_modules/@earendil-works/pi-coding-agent/dist/core/system-prompt.js

@@ -85,7 +85,7 @@ Sling documentation (read only when the user asks about sling itself, its SDK, e
 - Additional docs: ${docsPath}
 - Examples: ${examplesPath} (extensions, custom tools, SDK)
 - When reading sling docs or examples, resolve docs/... under Additional docs and examples/... under Examples, not the current working directory
-- When asked about: extensions (docs/extensions.md, examples/extensions/), themes (docs/themes.md), skills (docs/skills.md), prompt templates (docs/prompt-templates.md), TUI components (docs/tui.md), keybindings (docs/keybindings.md), SDK integrations (docs/sdk.md), custom providers (docs/custom-provider.md), adding models (docs/models.md), sling packages (docs/packages.md)
+- When asked about: extensions (docs/extensions.md, examples/extensions/), themes (docs/themes.md), skills (docs/skills.md), prompt templates (docs/prompt-templates.md), TUI components (docs/tui.md), keybindings (docs/keybindings.md), SDK integrations (docs/sdk.md), custom providers (docs/custom-provider.md), adding models (docs/models.md), sling packages (docs/packages.md), environment variables (docs/environment-variables.md)
 - When working on sling topics, read the docs and examples, and follow .md cross-references before implementing
 - Always read sling .md files completely and follow links to related docs (e.g., tui.md for TUI API details)`;
     if (appendSection) {

package/node_modules/@earendil-works/pi-coding-agent/dist/core/tools/bash.js

@@ -111,8 +111,27 @@ export function createLocalBashOperations(options) {
         },
     };
 }
-function resolveSpawnContext(command, cwd, spawnHook) {
-    const baseContext = { command, cwd, env: { ...getShellEnv() } };
+function resolveSpawnContext(command, cwd, spawnHook, exposeSessionEnvironment, ctx) {
+    const env = { ...getShellEnv() };
+    delete env.PI_SESSION_ID;
+    delete env.PI_SESSION_FILE;
+    delete env.PI_PROVIDER;
+    delete env.PI_MODEL;
+    delete env.PI_REASONING_LEVEL;
+    if (exposeSessionEnvironment && ctx) {
+        const model = ctx.model;
+        env.PI_SESSION_ID = ctx.sessionManager.getSessionId();
+        const sessionFile = ctx.sessionManager.getSessionFile();
+        if (sessionFile)
+            env.PI_SESSION_FILE = sessionFile;
+        if (model) {
+            env.PI_PROVIDER = model.provider;
+            env.PI_MODEL = model.id;
+        }
+        if (ctx.thinkingLevel)
+            env.PI_REASONING_LEVEL = ctx.thinkingLevel;
+    }
+    const baseContext = { command, cwd, env };
     return spawnHook ? spawnHook(baseContext) : baseContext;
 }
 const BASH_PREVIEW_LINES = 5;
@@ -202,16 +221,20 @@ function rebuildBashResultRenderComponent(component, result, options, showImages
 export function createBashToolDefinition(cwd, options) {
     const ops = options?.operations ?? createLocalBashOperations({ shellPath: options?.shellPath });
     const commandPrefix = options?.commandPrefix;
+    const exposeSessionEnvironment = options?.exposeSessionEnvironment ?? true;
     const spawnHook = options?.spawnHook;
     return {
         name: "bash",
         label: "bash",
         description: `Execute a bash command in the current working directory. Returns stdout and stderr. Output is truncated to last ${DEFAULT_MAX_LINES} lines or ${DEFAULT_MAX_BYTES / 1024}KB (whichever is hit first). If truncated, full output is saved to a temp file. Optionally provide a timeout in seconds.`,
         promptSnippet: "Execute bash commands (ls, grep, find, etc.)",
+        promptGuidelines: exposeSessionEnvironment
+            ? ["Inspect PI_* environment variables for current model and session details."]
+            : undefined,
         parameters: bashSchema,
-        async execute(_toolCallId, { command, timeout }, signal, onUpdate, _ctx) {
+        async execute(_toolCallId, { command, timeout }, signal, onUpdate, ctx) {
             const resolvedCommand = commandPrefix ? `${commandPrefix}\n${command}` : command;
-            const spawnContext = resolveSpawnContext(resolvedCommand, cwd, spawnHook);
+            const spawnContext = resolveSpawnContext(resolvedCommand, cwd, spawnHook, exposeSessionEnvironment, ctx);
             const output = new OutputAccumulator({ tempFilePrefix: "pi-bash" });
             let acceptingOutput = true;
             let updateTimer;
@@ -356,6 +379,12 @@ export function createBashToolDefinition(cwd, options) {
     };
 }
 export function createBashTool(cwd, options) {
-    return wrapToolDefinition(createBashToolDefinition(cwd, options));
+    const definition = createBashToolDefinition(cwd, options);
+    const tool = wrapToolDefinition(definition);
+    Object.assign(tool, {
+        promptSnippet: definition.promptSnippet,
+        promptGuidelines: definition.promptGuidelines,
+    });
+    return tool;
 }
 //# sourceMappingURL=bash.js.map

package/node_modules/@earendil-works/pi-coding-agent/dist/core/tools/tool-definition-wrapper.js

@@ -5,9 +5,10 @@ export function wrapToolDefinition(definition, ctxFactory) {
         label: definition.label,
         description: definition.description,
         parameters: definition.parameters,
+        constrainedSampling: definition.constrainedSampling,
         prepareArguments: definition.prepareArguments,
         executionMode: definition.executionMode,
-        execute: (toolCallId, params, signal, onUpdate) => definition.execute(toolCallId, params, signal, onUpdate, ctxFactory?.()),
+        execute: (toolCallId, params, signal, onUpdate, ctx) => definition.execute(toolCallId, params, signal, onUpdate, ctx ?? ctxFactory?.()),
     };
 }
 /** Wrap multiple ToolDefinitions into AgentTools for the core runtime. */
@@ -26,6 +27,7 @@ export function createToolDefinitionFromAgentTool(tool) {
         label: tool.label,
         description: tool.description,
         parameters: tool.parameters,
+        constrainedSampling: tool.constrainedSampling,
         prepareArguments: tool.prepareArguments,
         executionMode: tool.executionMode,
         execute: async (toolCallId, params, signal, onUpdate) => tool.execute(toolCallId, params, signal, onUpdate),

package/node_modules/@earendil-works/pi-coding-agent/dist/extensions/llama/provider.js

@@ -2,7 +2,6 @@ import { stream, streamSimple } from "@earendil-works/pi-ai/compat";
 import { LlamaClient, llamaInferenceUrl, normalizeLlamaServerUrl } from "./client.js";
 export const LLAMA_PROVIDER_ID = "llama.cpp";
 export const DEFAULT_LLAMA_SERVER_URL = "http://127.0.0.1:8080";
-const DEFAULT_MAX_TOKENS = 16384;
 function credentialServerUrl(credential) {
     const value = credential?.env?.LLAMA_BASE_URL;
     return typeof value === "string" && value.trim() ? normalizeLlamaServerUrl(value) : undefined;
@@ -24,7 +23,7 @@ function toPiModel(model, serverUrl) {
         input: model.architecture?.input_modalities?.includes("image") ? ["text", "image"] : ["text"],
         cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
         contextWindow,
-        maxTokens: Math.min(DEFAULT_MAX_TOKENS, contextWindow),
+        maxTokens: contextWindow,
         compat: {
             supportsStore: false,
             supportsDeveloperRole: false,
@@ -86,6 +85,10 @@ export function createLlamaProvider() {
         },
         getModels: () => models,
         refreshModels: async (context) => {
+            const stored = await context.store.read();
+            if (stored) {
+                models = stored.models.filter((model) => model.provider === LLAMA_PROVIDER_ID && model.api === "openai-completions");
+            }
             if (!context.allowNetwork || context.signal?.aborted || context.credential?.type !== "api_key")
                 return;
             const serverUrl = credentialServerUrl(context.credential);
@@ -93,6 +96,8 @@ export function createLlamaProvider() {
                 return;
             const catalog = await new LlamaClient(serverUrl, context.credential.key).list({ signal: context.signal });
             setCatalog(catalog, serverUrl);
+            if (!context.signal?.aborted)
+                await context.store.write({ models, checkedAt: Date.now() });
         },
         stream: (model, context, options) => stream(model, context, options),
         streamSimple: (model, context, options) => streamSimple(model, context, options),

package/node_modules/@earendil-works/pi-coding-agent/dist/modes/interactive/components/custom-message.js

@@ -11,11 +11,13 @@ export class CustomMessageComponent extends Container {
     customComponent;
     markdownTheme;
     _expanded = false;
-    constructor(message, customRenderer, markdownTheme = getMarkdownTheme()) {
+    outputPad;
+    constructor(message, customRenderer, markdownTheme = getMarkdownTheme(), outputPad = 1) {
         super();
         this.message = message;
         this.customRenderer = customRenderer;
         this.markdownTheme = markdownTheme;
+        this.outputPad = outputPad;
         this.addChild(new Spacer(1));
         // Create box with purple background (used for default rendering)
         this.box = new Box(1, 1, (t) => theme.bg("customMessageBg", t));
@@ -27,6 +29,12 @@ export class CustomMessageComponent extends Container {
             this.rebuild();
         }
     }
+    setOutputPad(outputPad) {
+        if (this.outputPad !== outputPad) {
+            this.outputPad = outputPad;
+            this.rebuild();
+        }
+    }
     invalidate() {
         super.invalidate();
         this.rebuild();
@@ -41,7 +49,7 @@ export class CustomMessageComponent extends Container {
         // Try custom renderer first - it handles its own styling
         if (this.customRenderer) {
             try {
-                const component = this.customRenderer(this.message, { expanded: this._expanded }, theme);
+                const component = this.customRenderer(this.message, { expanded: this._expanded, outputPad: this.outputPad }, theme);
                 if (component) {
                     // Custom renderer provides its own styled component
                     this.customComponent = component;

package/node_modules/@earendil-works/pi-coding-agent/dist/modes/interactive/components/extension-editor.js

@@ -2,11 +2,8 @@
  * Multi-line editor component for extensions.
  * Supports Ctrl+G for external editor.
  */
-import { spawn } from "node:child_process";
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
 import { Container, Editor, getKeybindings, Spacer, Text, } from "@earendil-works/pi-tui";
+import { editInExternalEditor } from "../external-editor.js";
 import { getEditorTheme, theme } from "../theme/theme.js";
 import { DynamicBorder } from "./dynamic-border.js";
 import { keyHint } from "./keybinding-hints.js";
@@ -29,7 +26,11 @@ export class ExtensionEditorComponent extends Container {
         super();
         this.tui = tui;
         this.keybindings = keybindings;
-        this.externalEditorCommand = externalEditorCommand;
+        this.externalEditorCommand =
+            externalEditorCommand ||
+                process.env.VISUAL ||
+                process.env.EDITOR ||
+                (process.platform === "win32" ? "notepad" : "nano");
         this.onSubmitCallback = onSubmit;
         this.onCancelCallback = onCancel;
         // Add top border
@@ -50,13 +51,12 @@ export class ExtensionEditorComponent extends Container {
         this.addChild(this.editor);
         this.addChild(new Spacer(1));
         // Add hint
-        const hasExternalEditor = !!this.getExternalEditorCommand();
         const hint = keyHint("tui.select.confirm", "submit") +
             "  " +
             keyHint("tui.input.newLine", "newline") +
             "  " +
             keyHint("tui.select.cancel", "cancel") +
-            (hasExternalEditor ? `  ${keyHint("app.editor.external", "external editor")}` : "");
+            `  ${keyHint("app.editor.external", "external editor")}`;
         this.addChild(new Text(hint, 1, 0));
         this.addChild(new Spacer(1));
         // Add bottom border
@@ -71,56 +71,26 @@ export class ExtensionEditorComponent extends Container {
         }
         // External editor (app keybinding)
         if (this.keybindings.matches(keyData, "app.editor.external")) {
-            this.openExternalEditor();
+            void this.handleOpenExternalEditor();
             return;
         }
         // Forward to editor
         this.editor.handleInput(keyData);
     }
-    getExternalEditorCommand() {
-        const editorCmd = this.externalEditorCommand || process.env.VISUAL || process.env.EDITOR;
-        if (editorCmd) {
-            return editorCmd;
-        }
-        return process.platform === "win32" ? "notepad" : "nano";
-    }
-    async openExternalEditor() {
-        const editorCmd = this.getExternalEditorCommand();
-        if (!editorCmd) {
-            return;
-        }
-        const currentText = this.editor.getText();
-        const tmpFile = path.join(os.tmpdir(), `pi-extension-editor-${Date.now()}.md`);
+    async handleOpenExternalEditor() {
+        const content = this.editor.getText();
+        this.tui.stop();
         try {
-            fs.writeFileSync(tmpFile, currentText, "utf-8");
-            this.tui.stop();
-            const [editor, ...editorArgs] = editorCmd.split(" ");
-            process.stdout.write(`Launching external editor: ${editorCmd}\nPi will resume when the editor exits.\n`);
-            // Do not use spawnSync here. On Windows, synchronous child_process calls can keep
-            // Node/libuv's console input read active after tui.stop() pauses stdin, racing
-            // vim/nvim for the console input buffer until Ctrl+C cancels the pending read.
-            const status = await new Promise((resolve) => {
-                const child = spawn(editor, [...editorArgs, tmpFile], {
-                    stdio: "inherit",
-                    shell: process.platform === "win32",
-                });
-                child.on("error", () => resolve(null));
-                child.on("close", (code) => resolve(code));
+            const result = await editInExternalEditor({
+                command: this.externalEditorCommand,
+                content,
             });
-            if (status === 0) {
-                const newContent = fs.readFileSync(tmpFile, "utf-8").replace(/\n$/, "");
-                this.editor.setText(newContent);
+            if (result.status === "complete") {
+                this.editor.setText(result.content);
             }
         }
         finally {
-            try {
-                fs.unlinkSync(tmpFile);
-            }
-            catch {
-                // Ignore cleanup errors
-            }
             this.tui.start();
-            // Force full re-render since external editor uses alternate screen
             this.tui.requestRender(true);
         }
     }

package/node_modules/@earendil-works/pi-coding-agent/dist/modes/interactive/components/scoped-models-selector.js

@@ -23,7 +23,7 @@ function enableAll(enabledIds, allIds, targetIds) {
         if (!result.includes(id))
             result.push(id);
     }
-    return result.length === allIds.length ? null : result;
+    return result.length === allIds.length && result.every((id) => allIds.includes(id)) ? null : result;
 }
 function clearAll(enabledIds, allIds, targetIds) {
     if (enabledIds === null) {
@@ -108,19 +108,19 @@ export class ScopedModelsSelectorComponent extends Container {
         this.updateList();
     }
     buildItems() {
-        // Filter out IDs that no longer have a corresponding model (e.g., after logout)
-        return getSortedIds(this.enabledIds, this.allIds)
-            .filter((id) => this.modelsById.has(id))
-            .map((id) => ({
+        return getSortedIds(this.enabledIds, this.allIds).map((id) => ({
             fullId: id,
             model: this.modelsById.get(id),
             enabled: isEnabled(this.enabledIds, id),
         }));
     }
     getFooterText() {
-        const enabledCount = this.enabledIds?.length ?? this.allIds.length;
+        const enabledCount = this.enabledIds?.filter((id) => this.modelsById.has(id)).length ?? this.allIds.length;
+        const unavailableCount = this.enabledIds?.filter((id) => !this.modelsById.has(id)).length ?? 0;
         const allEnabled = this.enabledIds === null;
-        const countText = allEnabled ? "all enabled" : `${enabledCount}/${this.allIds.length} enabled`;
+        const countText = allEnabled
+            ? "all enabled"
+            : `${enabledCount}/${this.allIds.length} enabled${unavailableCount ? ` · ${unavailableCount} unavailable` : ""}`;
         const parts = [
             `${keyText("tui.select.confirm")} toggle`,
             `${keyText("app.models.enableAll")} all`,
@@ -138,7 +138,9 @@ export class ScopedModelsSelectorComponent extends Container {
         const query = this.searchInput.getValue();
         const items = this.buildItems();
         this.filteredItems = query
-            ? fuzzyFilter(items, query, (i) => getModelSearchText({ id: i.model.id, provider: i.model.provider, name: i.model.name }))
+            ? fuzzyFilter(items, query, (item) => item.model
+                ? getModelSearchText({ id: item.model.id, provider: item.model.provider, name: item.model.name })
+                : item.fullId)
             : items;
         this.selectedIndex = Math.min(this.selectedIndex, Math.max(0, this.filteredItems.length - 1));
         this.updateList();
@@ -160,9 +162,16 @@ export class ScopedModelsSelectorComponent extends Container {
             const item = this.filteredItems[i];
             const isSelected = i === this.selectedIndex;
             const prefix = isSelected ? theme.fg("accent", "→ ") : "  ";
-            const modelText = isSelected ? theme.fg("accent", item.model.id) : item.model.id;
-            const providerBadge = theme.fg("muted", ` [${item.model.provider}]`);
-            const status = allEnabled ? "" : item.enabled ? theme.fg("success", " ✓") : theme.fg("dim", " ✗");
+            const id = item.model?.id ?? item.fullId;
+            const modelText = isSelected ? theme.fg("accent", id) : id;
+            const providerBadge = theme.fg("muted", item.model ? ` [${item.model.provider}]` : " [unavailable]");
+            const status = item.model
+                ? allEnabled
+                    ? ""
+                    : item.enabled
+                        ? theme.fg("success", " ✓")
+                        : theme.fg("dim", " ✗")
+                : theme.fg("dim", " ✗");
             this.listContainer.addChild(new Text(`${prefix}${modelText}${providerBadge}${status}`, 0, 0));
         }
         // Add scroll indicator if needed
@@ -172,7 +181,7 @@ export class ScopedModelsSelectorComponent extends Container {
         if (this.filteredItems.length > 0) {
             const selected = this.filteredItems[this.selectedIndex];
             this.listContainer.addChild(new Spacer(1));
-            this.listContainer.addChild(new Text(theme.fg("muted", `  Model Name: ${selected.model.name}`), 0, 0));
+            this.listContainer.addChild(new Text(theme.fg("muted", `  ${selected.model ? `Model Name: ${selected.model.name}` : "Model unavailable"}`), 0, 0));
         }
     }
     handleInput(data) {
@@ -246,7 +255,7 @@ export class ScopedModelsSelectorComponent extends Container {
         // Toggle provider of current item
         if (kb.matches(data, "app.models.toggleProvider")) {
             const item = this.filteredItems[this.selectedIndex];
-            if (item) {
+            if (item?.model) {
                 const provider = item.model.provider;
                 const providerIds = this.allIds.filter((id) => this.modelsById.get(id).provider === provider);
                 const allEnabled = providerIds.every((id) => isEnabled(this.enabledIds, id));