@ouro.bot/cli 0.1.0-alpha.72 → 0.1.0-alpha.74

package/changelog.json

@@ -1,6 +1,20 @@
 {
   "_note": "This changelog is maintained as part of the PR/version-bump workflow. Agent-curated, not auto-generated. Agents read this file directly via read_file to understand what changed between versions.",
   "versions": [
+    {
+      "version": "0.1.0-alpha.74",
+      "changes": [
+        "Fix: `ouro config model` now pings the model with a tiny API call before switching, rejecting the change if the model returns an error (403, not supported, etc.)."
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.73",
+      "changes": [
+        "New `ouro config models --agent <name>` command: list available models for the current provider. For github-copilot, queries the models API; other providers show a static message.",
+        "Fix: `ouro config model` now validates model availability for github-copilot before writing, showing available models if the requested one isn't found.",
+        "Fix: system prompt now tells the agent that model/provider changes take effect on the next turn automatically (no restart needed)."
+      ]
+    },
     {
       "version": "0.1.0-alpha.72",
       "changes": [

package/dist/heart/daemon/daemon-cli.js

@@ -34,6 +34,8 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ensureDaemonRunning = ensureDaemonRunning;
+exports.listGithubCopilotModels = listGithubCopilotModels;
+exports.pingGithubCopilotModel = pingGithubCopilotModel;
 exports.parseOuroCommand = parseOuroCommand;
 exports.discoverExistingCredentials = discoverExistingCredentials;
 exports.createDefaultOuroCliDeps = createDefaultOuroCliDeps;
@@ -270,6 +272,7 @@ function usage() {
         "  ouro stop|down|status|logs|hatch",
         "  ouro -v|--version",
         "  ouro config model --agent <name> <model-name>",
+        "  ouro config models --agent <name>",
         "  ouro auth --agent <name> [--provider <provider>]",
         "  ouro auth verify --agent <name> [--provider <provider>]",
         "  ouro auth switch --agent <name> --provider <provider>",
@@ -488,7 +491,73 @@ async function verifyProviderCredentials(provider, providers, fetchImpl = fetch)
         return "failed (no api key)";
     return "ok";
 }
-/* v8 ignore stop */
+async function listGithubCopilotModels(baseUrl, token, fetchImpl = fetch) {
+    const url = `${baseUrl.replace(/\/+$/, "")}/models`;
+    const response = await fetchImpl(url, {
+        headers: { Authorization: `Bearer ${token}` },
+    });
+    if (!response.ok) {
+        throw new Error(`model listing failed (HTTP ${response.status})`);
+    }
+    const body = await response.json();
+    /* v8 ignore start -- response shape handling: tested via config-models.test.ts @preserve */
+    const items = Array.isArray(body) ? body : (body?.data ?? []);
+    return items.map((item) => {
+        const rec = item;
+        const capabilities = Array.isArray(rec.capabilities)
+            ? rec.capabilities.filter((c) => typeof c === "string")
+            : undefined;
+        return {
+            id: String(rec.id ?? rec.name ?? ""),
+            name: String(rec.name ?? rec.id ?? ""),
+            ...(capabilities ? { capabilities } : {}),
+        };
+    });
+    /* v8 ignore stop */
+}
+async function pingGithubCopilotModel(baseUrl, token, model, fetchImpl = fetch) {
+    const base = baseUrl.replace(/\/+$/, "");
+    const isClaude = model.startsWith("claude");
+    const url = isClaude ? `${base}/chat/completions` : `${base}/responses`;
+    const body = isClaude
+        ? JSON.stringify({ model, messages: [{ role: "user", content: "ping" }], max_tokens: 1 })
+        : JSON.stringify({ model, input: "ping", max_output_tokens: 16 });
+    try {
+        const response = await fetchImpl(url, {
+            method: "POST",
+            headers: {
+                Authorization: `Bearer ${token}`,
+                "Content-Type": "application/json",
+            },
+            body,
+        });
+        if (response.ok)
+            return { ok: true };
+        let detail = `HTTP ${response.status}`;
+        try {
+            const json = await response.json();
+            /* v8 ignore start -- error format parsing: all branches tested via config-models.test.ts @preserve */
+            if (typeof json.error === "string")
+                detail = json.error;
+            else if (typeof json.error === "object" && json.error !== null) {
+                const errObj = json.error;
+                if (typeof errObj.message === "string")
+                    detail = errObj.message;
+            }
+            else if (typeof json.message === "string")
+                detail = json.message;
+            /* v8 ignore stop */
+        }
+        catch {
+            // response body not JSON — keep HTTP status
+        }
+        return { ok: false, error: detail };
+    }
+    catch (err) {
+        /* v8 ignore next -- defensive: fetch errors are always Error instances @preserve */
+        return { ok: false, error: err instanceof Error ? err.message : String(err) };
+    }
+}
 function parseHatchCommand(args) {
     let agentName;
     let humanName;
@@ -805,6 +874,11 @@ function parseConfigCommand(args) {
             throw new Error(`Usage: ouro config model --agent <name> <model-name>`);
         return { kind: "config.model", agent, modelName };
     }
+    if (sub === "models") {
+        if (!agent)
+            throw new Error("--agent is required for config models");
+        return { kind: "config.models", agent };
+    }
     throw new Error(`Usage\n${usage()}`);
 }
 function parseMcpCommand(args) {
@@ -1796,9 +1870,69 @@ async function runOuroCli(args, deps = createDefaultOuroCliDeps()) {
         return message;
     }
     /* v8 ignore stop */
+    // ── config models (local, no daemon socket needed) ──
+    /* v8 ignore start -- config models: tested via daemon-cli.test.ts @preserve */
+    if (command.kind === "config.models") {
+        const { config } = (0, auth_flow_1.readAgentConfigForAgent)(command.agent);
+        const provider = config.provider;
+        if (provider !== "github-copilot") {
+            const message = `model listing not available for ${provider} — check provider documentation.`;
+            deps.writeStdout(message);
+            return message;
+        }
+        const { secrets } = (0, auth_flow_1.loadAgentSecrets)(command.agent);
+        const ghConfig = secrets.providers["github-copilot"];
+        if (!ghConfig.githubToken || !ghConfig.baseUrl) {
+            throw new Error(`github-copilot credentials not configured. Run \`ouro auth --agent ${command.agent} --provider github-copilot\` first.`);
+        }
+        const fetchFn = deps.fetchImpl ?? fetch;
+        const models = await listGithubCopilotModels(ghConfig.baseUrl, ghConfig.githubToken, fetchFn);
+        if (models.length === 0) {
+            const message = "no models found";
+            deps.writeStdout(message);
+            return message;
+        }
+        const lines = ["available models:"];
+        for (const m of models) {
+            const caps = m.capabilities?.length ? ` (${m.capabilities.join(", ")})` : "";
+            lines.push(`  ${m.id}${caps}`);
+        }
+        const message = lines.join("\n");
+        deps.writeStdout(message);
+        return message;
+    }
+    /* v8 ignore stop */
     // ── config model (local, no daemon socket needed) ──
     /* v8 ignore start -- config model: tested via daemon-cli.test.ts @preserve */
     if (command.kind === "config.model") {
+        // Validate model availability for github-copilot before writing
+        const { config } = (0, auth_flow_1.readAgentConfigForAgent)(command.agent);
+        if (config.provider === "github-copilot") {
+            const { secrets } = (0, auth_flow_1.loadAgentSecrets)(command.agent);
+            const ghConfig = secrets.providers["github-copilot"];
+            if (ghConfig.githubToken && ghConfig.baseUrl) {
+                const fetchFn = deps.fetchImpl ?? fetch;
+                try {
+                    const models = await listGithubCopilotModels(ghConfig.baseUrl, ghConfig.githubToken, fetchFn);
+                    const available = models.map((m) => m.id);
+                    if (available.length > 0 && !available.includes(command.modelName)) {
+                        const message = `model '${command.modelName}' not found. available models:\n${available.map((id) => `  ${id}`).join("\n")}`;
+                        deps.writeStdout(message);
+                        return message;
+                    }
+                }
+                catch {
+                    // Catalog validation failed — fall through to ping test
+                }
+                // Ping test: verify the model actually works before switching
+                const pingResult = await pingGithubCopilotModel(ghConfig.baseUrl, ghConfig.githubToken, command.modelName, fetchFn);
+                if (!pingResult.ok) {
+                    const message = `model '${command.modelName}' ping failed: ${pingResult.error}\nrun \`ouro config models --agent ${command.agent}\` to see available models.`;
+                    deps.writeStdout(message);
+                    return message;
+                }
+            }
+        }
         const { provider, previousModel } = (0, auth_flow_1.writeAgentModel)(command.agent, command.modelName);
         const message = previousModel
             ? `updated ${command.agent} model on ${provider}: ${previousModel} → ${command.modelName}`

package/dist/mind/prompt.js

@@ -182,12 +182,15 @@ my bones give me the \`ouro\` cli. always pass \`--agent ${agentName}\`:
   ouro session list --agent ${agentName}
   ouro reminder create --agent ${agentName} <title> --body <body>
   ouro config model --agent ${agentName} <model-name>
+  ouro config models --agent ${agentName}
   ouro auth --agent ${agentName} --provider <provider>
   ouro auth verify --agent ${agentName} [--provider <provider>]
   ouro auth switch --agent ${agentName} --provider <provider>
   ouro mcp list --agent ${agentName}
   ouro mcp call --agent ${agentName} <server> <tool> --args '{...}'
-  ouro --help`;
+  ouro --help
+
+provider/model changes via \`ouro config model\` or \`ouro auth switch\` take effect on the next turn automatically — no restart needed.`;
 }
 function mcpToolsSection(mcpManager) {
     if (!mcpManager)

package/dist/nerves/index.js

@@ -4,6 +4,7 @@ exports.createTraceId = createTraceId;
 exports.ensureTraceId = ensureTraceId;
 exports.createFanoutSink = createFanoutSink;
 exports.formatTerminalEntry = formatTerminalEntry;
+exports.registerSpinnerHooks = registerSpinnerHooks;
 exports.createTerminalSink = createTerminalSink;
 exports.createStderrSink = createStderrSink;
 exports.createNdjsonFileSink = createNdjsonFileSink;
@@ -73,15 +74,26 @@ function formatTerminalEntry(entry) {
     const level = entry.level.toUpperCase();
     return `${formatTerminalTime(entry.ts)} ${level} [${entry.component}] ${entry.message}${formatTerminalMeta(entry.meta)}`;
 }
+// Spinner coordination: the CLI sense registers these so log output
+// doesn't interleave with the active spinner animation.
+let _pauseSpinner = null;
+let _resumeSpinner = null;
+function registerSpinnerHooks(pause, resume) {
+    _pauseSpinner = pause;
+    _resumeSpinner = resume;
+}
 function createTerminalSink(write = (chunk) => process.stderr.write(chunk), colorize = true) {
     return (entry) => {
+        _pauseSpinner?.();
         const line = formatTerminalEntry(entry);
         if (!colorize) {
             write(`${line}\n`);
+            _resumeSpinner?.();
             return;
         }
         const prefix = LEVEL_COLORS[entry.level];
         write(`${prefix}${line}\x1b[0m\n`);
+        _resumeSpinner?.();
     };
 }
 function createStderrSink(write = (chunk) => process.stderr.write(chunk)) {

package/dist/repertoire/guardrails.js

@@ -168,6 +168,7 @@ exports.OURO_CLI_TRUST_MANIFEST = {
     "friend update": "family",
     "reminder create": "friend",
     "config model": "friend",
+    "config models": "friend",
     "mcp list": "acquaintance",
     "mcp call": "friend",
     auth: "family",

package/dist/senses/cli.js

@@ -36,6 +36,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.MarkdownStreamer = exports.InputController = exports.Spinner = exports.wrapCliText = exports.formatEchoedInputSummary = void 0;
 exports.formatPendingPrefix = formatPendingPrefix;
 exports.getCliContinuityIngressTexts = getCliContinuityIngressTexts;
+exports.pauseActiveSpinner = pauseActiveSpinner;
+exports.resumeActiveSpinner = resumeActiveSpinner;
+exports.setActiveSpinner = setActiveSpinner;
 exports.handleSigint = handleSigint;
 exports.addHistory = addHistory;
 exports.renderMarkdown = renderMarkdown;
@@ -90,6 +93,14 @@ function getCliContinuityIngressTexts(input) {
     const trimmed = input.trim();
     return trimmed ? [trimmed] : [];
 }
+// Module-level active spinner for log coordination.
+// The terminal log sink calls these to avoid interleaving with spinner output.
+let _activeSpinner = null;
+/* v8 ignore start -- spinner coordination: exercised at runtime, not unit-testable without real terminal @preserve */
+function pauseActiveSpinner() { _activeSpinner?.pause(); }
+function resumeActiveSpinner() { _activeSpinner?.resume(); }
+/* v8 ignore stop */
+function setActiveSpinner(s) { _activeSpinner = s; }
 // spinner that only touches stderr, cleans up after itself
 // exported for direct testability (stop-without-start branch)
 class Spinner {
@@ -131,6 +142,20 @@ class Spinner {
         this.lastPhrase = next;
         this.msg = next;
     }
+    /* v8 ignore start -- pause/resume: exercised at runtime via log sink coordination @preserve */
+    /** Clear the spinner line temporarily so other output can print cleanly. */
+    pause() {
+        if (this.stopped)
+            return;
+        process.stderr.write("\r\x1b[K");
+    }
+    /** Restore the spinner line after a pause. */
+    resume() {
+        if (this.stopped)
+            return;
+        this.spin();
+    }
+    /* v8 ignore stop */
     stop(ok) {
         this.stopped = true;
         if (this.iv) {
@@ -319,6 +344,7 @@ function createCliCallbacks() {
         meta: {},
     });
     let currentSpinner = null;
+    function setSpinner(s) { currentSpinner = s; setActiveSpinner(s); }
     let hadReasoning = false;
     let hadToolRun = false;
     let textDirty = false; // true when text/reasoning was written without a trailing newline
@@ -326,14 +352,14 @@ function createCliCallbacks() {
     return {
         onModelStart: () => {
             currentSpinner?.stop();
-            currentSpinner = null;
+            setSpinner(null);
             hadReasoning = false;
             textDirty = false;
             streamer.reset();
             const phrases = (0, phrases_1.getPhrases)();
             const pool = hadToolRun ? phrases.followup : phrases.thinking;
             const first = (0, phrases_1.pickPhrase)(pool);
-            currentSpinner = new Spinner(first, pool);
+            setSpinner(new Spinner(first, pool));
             currentSpinner.start();
         },
         onModelStreamStart: () => {
@@ -350,7 +376,7 @@ function createCliCallbacks() {
             // otherwise keep running (and its \r writes overwrite response text).
             if (currentSpinner) {
                 currentSpinner.stop();
-                currentSpinner = null;
+                setSpinner(null);
             }
             if (hadReasoning) {
                 // Single newline to separate reasoning from reply — reasoning
@@ -366,7 +392,7 @@ function createCliCallbacks() {
         onReasoningChunk: (text) => {
             if (currentSpinner) {
                 currentSpinner.stop();
-                currentSpinner = null;
+                setSpinner(null);
             }
             hadReasoning = true;
             process.stdout.write(`\x1b[2m${text}\x1b[0m`);
@@ -385,13 +411,13 @@ function createCliCallbacks() {
             }
             const toolPhrases = (0, phrases_1.getPhrases)().tool;
             const first = (0, phrases_1.pickPhrase)(toolPhrases);
-            currentSpinner = new Spinner(first, toolPhrases);
+            setSpinner(new Spinner(first, toolPhrases));
             currentSpinner.start();
             hadToolRun = true;
         },
         onToolEnd: (name, argSummary, success) => {
             currentSpinner?.stop();
-            currentSpinner = null;
+            setSpinner(null);
             const msg = (0, format_1.formatToolResult)(name, argSummary, success);
             const color = success ? "\x1b[32m" : "\x1b[31m";
             process.stderr.write(`${color}${msg}\x1b[0m\n`);
@@ -399,17 +425,17 @@ function createCliCallbacks() {
         onError: (error, severity) => {
             if (severity === "transient") {
                 currentSpinner?.fail(error.message);
-                currentSpinner = null;
+                setSpinner(null);
             }
             else {
                 currentSpinner?.stop();
-                currentSpinner = null;
+                setSpinner(null);
                 process.stderr.write(`\x1b[31m${(0, format_1.formatError)(error)}\x1b[0m\n`);
             }
         },
         onKick: () => {
             currentSpinner?.stop();
-            currentSpinner = null;
+            setSpinner(null);
             if (textDirty) {
                 process.stdout.write("\n");
                 textDirty = false;
@@ -418,7 +444,7 @@ function createCliCallbacks() {
         },
         flushMarkdown: () => {
             currentSpinner?.stop();
-            currentSpinner = null;
+            setSpinner(null);
             const remaining = streamer.flush();
             if (remaining)
                 process.stdout.write(remaining);
@@ -694,6 +720,8 @@ async function main(agentName, options) {
     if (agentName)
         (0, identity_1.setAgentName)(agentName);
     const pasteDebounceMs = options?.pasteDebounceMs ?? 50;
+    // Register spinner hooks so log output clears the spinner before printing
+    (0, nerves_1.registerSpinnerHooks)(pauseActiveSpinner, resumeActiveSpinner);
     // Fallback: apply pending updates for daemon-less direct CLI usage
     (0, update_hooks_1.registerUpdateHook)(bundle_meta_1.bundleMetaHook);
     await (0, update_hooks_1.applyPendingUpdates)((0, identity_1.getAgentBundlesRoot)(), (0, bundle_manifest_1.getPackageVersion)());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouro.bot/cli",
-  "version": "0.1.0-alpha.72",
+  "version": "0.1.0-alpha.74",
   "main": "dist/heart/daemon/ouro-entry.js",
   "bin": {
     "cli": "dist/heart/daemon/ouro-bot-entry.js",