@ouro.bot/cli 0.1.0-alpha.124 → 0.1.0-alpha.126

package/changelog.json

@@ -1,6 +1,27 @@
 {
   "_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.126",
+      "changes": [
+        "Fixed Anthropic tool_choice incompatibility with thinking \u2014 uses auto instead of any when thinking is enabled.",
+        "auth verify and auth switch now use pingProvider for real API verification instead of format-only checks. auth switch verifies credentials work before switching."
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.125",
+      "changes": [
+        "Fixed Anthropic tool_choice incompatibility with thinking \u2014 uses auto instead of any when thinking is enabled.",
+        "auth verify and auth switch now use pingProvider for real API verification instead of format-only checks. auth switch verifies credentials work before switching."
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.125",
+      "changes": [
+        "Fix: Default runtime logger is now silent (no stderr sink) so nerves events emitted before logger configuration no longer interleave with the CLI spinner animation.",
+        "Fix: MCP server connect failures now include the command name, args, and a hint to check agent.json mcpServers configuration."
+      ]
+    },
     {
       "version": "0.1.0-alpha.124",
       "changes": [
@@ -67,6 +88,9 @@
     {
       "version": "0.1.0-alpha.114",
       "changes": [
+        "Fix: Default runtime logger is now silent (no stderr sink) so nerves events emitted before logger configuration no longer interleave with the CLI spinner animation.",
+        "Fix: MCP server connect failures now include the command name, args, and a hint to check agent.json mcpServers configuration. Retry-exhaustion messages also identify the failing command.",
+        "Verification: StreamingWordWrapper integration in CLI chat confirmed working \u2014 wraps at word boundaries during streaming output.",
         "When a model provider fails mid-conversation (auth error, usage limit, outage), the harness now classifies the error, pings alternative configured providers, and surfaces validated failover options to the user in-channel. Reply 'switch to <provider>' to continue on a working provider.",
         "Each provider now has a `classifyError` method that distinguishes auth failures, usage/subscription limits, rate limits, server errors, and network errors. The old auth guidance wrappers are replaced by this unified classification system.",
         "New `pingProvider` function makes a real heartbeat completion call to verify provider credentials and quota are live \u2014 no more format-only checks.",

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

@@ -452,49 +452,19 @@ function hasStoredCredentials(provider, providerSecrets) {
     return !!providerSecrets.endpoint && !!providerSecrets.apiKey;
 }
 /* v8 ignore stop */
-/* v8 ignore start -- verifyProviderCredentials: per-provider branches tested via auth verify tests @preserve */
-async function verifyProviderCredentials(provider, providers, fetchImpl = fetch) {
-    const p = providers[provider];
-    if (!p)
+/* v8 ignore start -- verifyProviderCredentials: delegates to pingProvider @preserve */
+async function verifyProviderCredentials(provider, providers) {
+    const config = providers[provider];
+    if (!config)
         return "not configured";
-    if (provider === "anthropic") {
-        const token = p.setupToken || "";
-        if (!token)
-            return "failed (no token)";
-        if (token.startsWith("sk-ant-"))
-            return "ok";
-        return "failed (invalid token format)";
-    }
-    if (provider === "openai-codex") {
-        const token = p.oauthAccessToken || "";
-        return token ? "ok" : "failed (no token)";
-    }
-    if (provider === "github-copilot") {
-        const token = p.githubToken || "";
-        if (!token)
-            return "failed (no token)";
-        try {
-            const response = await fetchImpl("https://api.github.com/copilot_internal/user", {
-                headers: { Authorization: `Bearer ${token}` },
-            });
-            return response.ok ? "ok" : `failed (HTTP ${response.status})`;
-        }
-        catch (error) {
-            return `failed (${error.message})`;
-        }
+    try {
+        const { pingProvider } = await Promise.resolve().then(() => __importStar(require("../../heart/provider-ping")));
+        const result = await pingProvider(provider, config);
+        return result.ok ? "ok" : `failed (${result.message})`;
     }
-    if (provider === "minimax") {
-        const apiKey = p.apiKey || "";
-        return apiKey ? "ok" : "failed (no api key)";
+    catch (error) {
+        return `failed (${error instanceof Error ? error.message : String(error)})`;
     }
-    // azure
-    const endpoint = p.endpoint || "";
-    const apiKey = p.apiKey || "";
-    if (!endpoint)
-        return "failed (no endpoint)";
-    if (!apiKey)
-        return "failed (no api key)";
-    return "ok";
 }
 async function listGithubCopilotModels(baseUrl, token, fetchImpl = fetch) {
     const url = `${baseUrl.replace(/\/+$/, "")}/models`;
@@ -1352,12 +1322,6 @@ function createDefaultOuroCliDeps(socketPath = socket_client_1.DEFAULT_DAEMON_SO
         promptInput: defaultPromptInput,
         runAdoptionSpecialist: defaultRunAdoptionSpecialist,
         runAuthFlow: auth_flow_1.runRuntimeAuthFlow,
-        /* v8 ignore start -- integration: real API ping @preserve */
-        pingProvider: async (provider, config) => {
-            const { pingProvider: ping } = await Promise.resolve().then(() => __importStar(require("../../heart/provider-ping")));
-            return ping(provider, config);
-        },
-        /* v8 ignore stop */
         registerOuroBundleType: ouro_uti_1.registerOuroBundleUti,
         installOuroCommand: ouro_path_installer_1.installOuroCommand,
         /* v8 ignore start -- self-healing: ensures active symlink matches running runtime version @preserve */
@@ -2074,17 +2038,11 @@ async function runOuroCli(args, deps = createDefaultOuroCliDeps()) {
         /* v8 ignore start -- integration: real API ping after auth @preserve */
         try {
             const { secrets } = (0, auth_flow_1.loadAgentSecrets)(command.agent);
-            const config = secrets.providers[provider];
-            const pingResult = await deps.pingProvider(provider, config);
-            if (pingResult.ok) {
-                deps.writeStdout(`verified: ${provider} credentials are working.`);
-            }
-            else {
-                deps.writeStdout(`warning: ${provider} credentials were saved but verification failed. you may need to re-run auth.`);
-            }
+            const status = await verifyProviderCredentials(provider, secrets.providers);
+            deps.writeStdout(`${provider}: ${status}`);
         }
         catch {
-            // Ping failure is non-blocking — credentials were saved regardless
+            // Verification failure is non-blocking — credentials were saved regardless
         }
         /* v8 ignore stop */
         return result.message;
@@ -2094,16 +2052,15 @@ async function runOuroCli(args, deps = createDefaultOuroCliDeps()) {
     if (command.kind === "auth.verify") {
         const { secrets } = (0, auth_flow_1.loadAgentSecrets)(command.agent);
         const providers = secrets.providers;
-        const fetchFn = deps.fetchImpl ?? fetch;
         if (command.provider) {
-            const status = await verifyProviderCredentials(command.provider, providers, fetchFn);
+            const status = await verifyProviderCredentials(command.provider, providers);
             const message = `${command.provider}: ${status}`;
             deps.writeStdout(message);
             return message;
         }
         const lines = [];
         for (const p of Object.keys(providers)) {
-            const status = await verifyProviderCredentials(p, providers, fetchFn);
+            const status = await verifyProviderCredentials(p, providers);
             lines.push(`${p}: ${status}`);
         }
         const message = lines.join("\n");
@@ -2119,8 +2076,15 @@ async function runOuroCli(args, deps = createDefaultOuroCliDeps()) {
             deps.writeStdout(message);
             return message;
         }
+        // Verify credentials actually work before switching
+        const status = await verifyProviderCredentials(command.provider, secrets.providers);
+        if (!status.startsWith("ok")) {
+            const message = `${command.provider}: ${status}. fix credentials with \`ouro auth --agent ${command.agent} --provider ${command.provider}\` before switching.`;
+            deps.writeStdout(message);
+            return message;
+        }
         (0, auth_flow_1.writeAgentProviderSelection)(command.agent, command.provider);
-        const message = `switched ${command.agent} to ${command.provider}`;
+        const message = `switched ${command.agent} to ${command.provider} (verified working)`;
         deps.writeStdout(message);
         return message;
     }

package/dist/heart/providers/anthropic.js

@@ -275,7 +275,9 @@ async function streamAnthropicMessages(client, model, request) {
     if (anthropicTools.length > 0)
         params.tools = anthropicTools;
     if (request.toolChoiceRequired && anthropicTools.length > 0) {
-        params.tool_choice = { type: "any" };
+        // Thinking (adaptive or enabled) only supports tool_choice "auto" or "none".
+        // "any" forces tool use which is incompatible with extended thinking.
+        params.tool_choice = params.thinking ? { type: "auto" } : /* v8 ignore next -- no-thinking path: thinking always set for 4.6 models @preserve */ { type: "any" };
     }
     let response;
     try {

package/dist/nerves/runtime.js

@@ -6,7 +6,11 @@ const index_1 = require("./index");
 let runtimeLogger = null;
 function getRuntimeLogger() {
     if (!runtimeLogger) {
-        runtimeLogger = (0, index_1.createLogger)({ level: "info" });
+        // Default logger has no sinks — events emitted before the real logger is
+        // configured (e.g. identity resolution during startup) are silently dropped.
+        // This prevents INFO lines from leaking to stderr and interleaving with
+        // the CLI spinner when the ouro subprocess hasn't configured its logger yet.
+        runtimeLogger = (0, index_1.createLogger)({ level: "info", sinks: [] });
     }
     return runtimeLogger;
 }

package/dist/repertoire/mcp-manager.js

@@ -76,14 +76,17 @@ class McpManager {
             entry.consecutiveFailures = 0;
         }
         catch (error) {
+            const reason = error instanceof Error ? error.message : String(error);
             (0, runtime_1.emitNervesEvent)({
                 level: "error",
                 event: "mcp.connect_error",
                 component: "repertoire",
-                message: `failed to connect MCP server: ${name}`,
+                message: `failed to connect MCP server "${name}" (command: ${config.command}). Check that the command exists and is properly configured. Reason: ${reason}`,
                 meta: {
                     server: name,
-                    reason: error instanceof Error ? error.message : String(error),
+                    command: config.command,
+                    args: config.args,
+                    reason,
                 },
             });
         }
@@ -99,8 +102,8 @@ class McpManager {
                 level: "error",
                 event: "mcp.connect_error",
                 component: "repertoire",
-                message: `MCP server "${name}" exceeded max restart retries (${MAX_RESTART_RETRIES})`,
-                meta: { server: name, failures: entry.consecutiveFailures },
+                message: `MCP server "${name}" exceeded max restart retries (${MAX_RESTART_RETRIES}). Giving up — check that "${entry.config.command}" exists and is properly configured in agent.json mcpServers.`,
+                meta: { server: name, command: entry.config.command, failures: entry.consecutiveFailures },
             });
             return;
         }

package/package.json

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