@tiny-fish/cli 0.19.1-next.180 → 0.20.1

package/dist/commands/connect.js

@@ -35,6 +35,7 @@ async function connectNativeMcpClient(client, options) {
     await runGuarded(state, telemetry, () => {
         telemetry.track("started");
         const { optionalSupported } = requireCommandSupport(client);
+        state.harnessDegraded = !!client.loginArgs && !optionalSupported;
         telemetry.track("checkpoint", { phase: "prerequisite_ok" });
         state.stage = "registration_cleanup";
         for (const removal of client.removals)
@@ -47,7 +48,7 @@ async function connectNativeMcpClient(client, options) {
         const storedKey = validatedApiKey(options.apiKey);
         const useKeyAuth = client.headerAuthSupported === true && !!storedKey;
         // Undefining `loginArgs` reuses the deferred-auth path codex/hermes already take.
-        const loginDegraded = !!client.loginArgs && !optionalSupported;
+        const loginDegraded = state.harnessDegraded === true;
         const loginArgs = loginDegraded ? undefined : client.loginArgs;
         // A probed client authenticates inside `mcp add` too, so a failure there is either.
         state.stage =
@@ -98,9 +99,15 @@ async function connectNativeMcpClient(client, options) {
             }
             telemetry.track("checkpoint", { phase: "oauth_done" });
         }
+        const signInDeferred = !useKeyAuth && !addSignedIn && !pendingLogin;
+        const authMode = useKeyAuth
+            ? "api-key"
+            : signInDeferred
+                ? "deferred"
+                : "oauth";
         // Registration/OAuth make MCP work; later steps are cosmetic and must not fail the attempt.
-        settle(state, telemetry, "completed");
-        runPostInstallSteps(client, options, state, telemetry, !useKeyAuth && !addSignedIn && !pendingLogin);
+        settle(state, telemetry, "completed", { authMode });
+        runPostInstallSteps(client, options, state, telemetry, signInDeferred);
     });
 }
 function connectedLine(client, signInDeferred) {
@@ -163,7 +170,12 @@ export async function connectOpencode(options) {
 }
 export async function connectOpenClaw(options) {
     const telemetry = createConnectTelemetry(options.mcpUrl, "openclaw", options);
-    const state = { stage: "prerequisite_check", settled: false };
+    // Skill install, no MCP server — never degraded, and false keeps it inside a `= false` filter.
+    const state = {
+        stage: "prerequisite_check",
+        settled: false,
+        harnessDegraded: false,
+    };
     await runGuarded(state, telemetry, () => {
         telemetry.track("started");
         requireCommandSupport(OPENCLAW);
@@ -230,7 +242,13 @@ export async function connectOpenClaw(options) {
 /** Writes mcp.json directly — no `cursor mcp add` exists. */
 export async function connectCursor(options) {
     const telemetry = createConnectTelemetry(options.mcpUrl, "cursor", options);
-    const state = { stage: "prerequisite_check", settled: false };
+    // Never degraded: no sign-in command to be missing. False, not absent, so a `= false`
+    // filter still catches Cursor.
+    const state = {
+        stage: "prerequisite_check",
+        settled: false,
+        harnessDegraded: false,
+    };
     await runGuarded(state, telemetry, async () => {
         telemetry.track("started");
         telemetry.track("checkpoint", { phase: "prerequisite_ok" });
@@ -279,7 +297,7 @@ export async function connectCursor(options) {
         else {
             errLine("TinyFish is connected. Reload the Cursor window, then approve/sign in under Settings → MCP.");
         }
-        settle(state, telemetry, "completed");
+        settle(state, telemetry, "completed", { authMode: resolvedKey ? "api-key" : "deferred" });
     });
 }
 /** Best-effort: false when no handler/open fails — caller falls back to reload copy. */

package/dist/commands/doctor.js

@@ -93,7 +93,7 @@ function checkCredential() {
         id: "cli-credential",
         title: "CLI credential",
         status: "key" in resolved ? "pass" : "fail",
-        detail: "key" in resolved ? `valid key from ${resolved.source}` : resolved.error,
+        detail: "key" in resolved ? `key present from ${resolved.source}` : resolved.error,
         harness: null,
     };
     return "key" in resolved ? { check, key: resolved.key } : { check };

package/dist/lib/connect-all.js

@@ -210,46 +210,53 @@ function computeExitCode(results) {
 }
 export async function runConnectAll(opts) {
     const isTTY = detectHumanInitiated();
-    const rl = isTTY
-        ? readline.createInterface({ input: process.stdin, output: process.stderr })
-        : undefined;
-    let prompt;
-    if (rl) {
-        // Without a close-linked signal, question() never settles on Ctrl+D.
-        const closed = new AbortController();
-        rl.on("close", () => closed.abort());
-        prompt = (question) => rl.question(question, { signal: closed.signal });
-    }
-    const detections = detectInstalledHarnesses();
-    const results = [];
-    try {
-        for (const [index, detection] of detections.entries()) {
-            const { harness } = detection;
-            errLine(`step ${index + 1} of ${detections.length} — ${DISPLAY_NAMES[harness]}`);
-            let result;
+    let stdinClosed = false;
+    // Open interface holds stdin raw; hermes' input() then wedges.
+    const prompt = isTTY
+        ? async (question) => {
+            if (stdinClosed)
+                throw new Error("stdin is closed");
+            const rl = readline.createInterface({ input: process.stdin, output: process.stderr });
+            // Without a close-linked signal, question() never settles on Ctrl+D.
+            const closed = new AbortController();
+            rl.on("close", () => closed.abort());
             try {
-                result = await processHarness(detection, opts, prompt);
+                return await rl.question(question, { signal: closed.signal });
             }
             catch (e) {
-                // Isolation: one harness throwing never stops the others.
-                result = {
-                    harness,
-                    detected: detection.detected,
-                    configPath: detection.configPath,
-                    installed: false,
-                    outcome: "failed",
-                    errorMessage: e instanceof Error ? e.message : String(e),
-                    fixCommand: `tinyfish connect ${harness}`,
-                };
+                stdinClosed = true;
+                throw e;
+            }
+            finally {
+                rl.close();
             }
-            results.push(result);
-            // Ctrl+C means stop, not continue to the next harness.
-            if (result.outcome === "interrupted")
-                break;
         }
-    }
-    finally {
-        rl?.close();
+        : undefined;
+    const detections = detectInstalledHarnesses();
+    const results = [];
+    for (const [index, detection] of detections.entries()) {
+        const { harness } = detection;
+        errLine(`step ${index + 1} of ${detections.length} — ${DISPLAY_NAMES[harness]}`);
+        let result;
+        try {
+            result = await processHarness(detection, opts, prompt);
+        }
+        catch (e) {
+            // Isolation: one harness throwing never stops the others.
+            result = {
+                harness,
+                detected: detection.detected,
+                configPath: detection.configPath,
+                installed: false,
+                outcome: "failed",
+                errorMessage: e instanceof Error ? e.message : String(e),
+                fixCommand: `tinyfish connect ${harness}`,
+            };
+        }
+        results.push(result);
+        // Ctrl+C means stop, not continue to the next harness.
+        if (result.outcome === "interrupted")
+            break;
     }
     // The interrupt break truncates `results`; pad so the 5-entry telemetry schema still parses.
     for (const detection of detections) {

package/dist/lib/connect-runtime.d.ts

@@ -1,8 +1,10 @@
+import { type AuthMode } from "./registration-detect.js";
 export declare const NON_INTERACTIVE_TIMEOUT_MS = 10000;
 export type AgentClient = "claude-code" | "codex" | "cursor" | "hermes" | "openclaw" | "opencode";
 type ConnectStage = "started" | "checkpoint" | "completed" | "failed" | "aborted" | "post_install_failed";
 type ConnectFailureStage = "prerequisite_check" | "registration_cleanup" | "registration" | "registration_or_authentication" | "client_oauth" | "cli_install" | "skill_install" | "authentication" | "walkthrough_launch";
 type ConnectFailureReason = "harness_not_installed" | "harness_command_unsupported" | "harness_too_old";
+export type ConnectAuthMode = Extract<AuthMode, "oauth" | "api-key"> | "deferred";
 type ConnectCheckpoint = "prerequisite_ok" | "registered" | "oauth_done" | "cli_installed" | "skill_installed" | "authenticated";
 /** Ctrl+C/SIGTERM killed a setup child — abandonment, not error. */
 export declare class ConnectInterruptedError extends Error {
@@ -15,12 +17,17 @@ export declare function commandNotFound(error: unknown): boolean;
 export interface ConnectRunState {
     stage: ConnectFailureStage;
     settled: boolean;
+    /** On state, not per-call: otherwise degraded installs never appear to fail. */
+    harnessDegraded?: boolean;
 }
 interface ConnectStageDetail {
     failedStage?: ConnectFailureStage;
     phase?: ConnectCheckpoint;
     failureReason?: ConnectFailureReason;
     harnessVersion?: string;
+    authMode?: ConnectAuthMode;
+    /** Orthogonal to `authMode`: a stored key wins over `mcp login` on a healthy client too. */
+    harnessDegraded?: boolean;
 }
 export interface ConnectTelemetry {
     attemptId: string;

package/dist/lib/connect-runtime.js

@@ -36,7 +36,7 @@ export function settle(state, telemetry, stage, detail) {
     if (state.settled)
         return;
     state.settled = true;
-    telemetry.track(stage, detail);
+    telemetry.track(stage, { harnessDegraded: state.harnessDegraded, ...detail });
 }
 // Raw-mode children still surface via throwIfInterrupted; cooked-mode stages
 // (npm/npx/OAuth waits) land here.
@@ -165,6 +165,8 @@ export function createConnectTelemetry(mcpUrl, client, opts) {
                 phase: detail?.phase,
                 failure_reason: detail?.failureReason,
                 harness_version: detail?.harnessVersion,
+                auth_mode: detail?.authMode,
+                harness_degraded: detail?.harnessDegraded,
                 runtime_platform: process.platform,
                 node_version: process.version,
                 cli_version: CLI_VERSION,

package/dist/lib/registration-detect.js

@@ -6,8 +6,8 @@ import { loadConfig } from "./auth.js";
 import { errLine } from "./output.js";
 import { readCursorTinyfishEntry } from "./cursor-config.js";
 import { detectInstalledHarnesses, harnessConfigPath, harnessDisplayPath, } from "./harness-detect.js";
-// 2s tripped on a cold `hermes mcp list` (0.3s warm), reporting `unknown` on a healthy install.
-const PROBE_TIMEOUT_MS = 6_000;
+// 6s tripped on a cold `hermes mcp list`, measured at 6.34s.
+const PROBE_TIMEOUT_MS = 15_000;
 // A list, verified against codex 0.146. A name-keyed map was accepted here too, but every
 // field is optional, so that arm parsed *any* object-of-objects: a wrapper like
 // `{"servers": {...}}` decoded as one entry named `servers` and reported TinyFish absent.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiny-fish/cli",
-  "version": "0.19.1-next.180",
+  "version": "0.20.1",
   "description": "TinyFish CLI — run web automations from your terminal",
   "type": "module",
   "bin": {