@tiny-fish/cli 0.25.2-next.221 → 0.25.2-next.222

package/dist/commands/connect.js

@@ -63,11 +63,12 @@ async function connectNativeMcpClient(client, options) {
         const useKeyAuth = !!keyedAdd;
         if (options.keyAuthOnly && !useKeyAuth) {
             // No keyAuth at all is structural (Hermes); a missing flag is just an old install.
-            throw new Error(client.keyAuth
-                ? `This ${client.displayName} install cannot take a TinyFish API key, and signing in ` +
-                    `needs a terminal. Update it, then re-run: tinyfish connect ${client.connectClient}`
-                : `${client.displayName} cannot take a TinyFish API key, and signing in needs a ` +
-                    `terminal. Run: tinyfish connect ${client.connectClient}`);
+            if (client.keyAuth) {
+                throw new ConnectStepError(`This ${client.displayName} install cannot take a TinyFish API key, and signing in ` +
+                    `needs a terminal. Update it, then re-run: tinyfish connect ${client.connectClient}`, "harness_too_old");
+            }
+            throw new Error(`${client.displayName} cannot take a TinyFish API key, and signing in needs a ` +
+                `terminal. Run: tinyfish connect ${client.connectClient}`);
         }
         telemetry.track("checkpoint", { phase: "prerequisite_ok" });
         state.stage = "registration_cleanup";

package/dist/lib/connect-all.js

@@ -17,6 +17,11 @@ import { verifyMcpAuth, verifyMcpHealth } from "./verify.js";
 // `codex mcp add` succeeds unauthenticated; OAuth lands at first tool use.
 // TODO(PF-3580): connect now probes Codex sign-in, so this over-warns a signed-in install.
 const AUTH_DEFERRED_AT_INSTALL = new Set(["codex"]);
+// A probe that exits non-zero is an install so old the subcommand is missing entirely.
+const TOO_OLD_REASONS = new Set([
+    "harness_too_old",
+    "harness_command_unsupported",
+]);
 const AUTH_CHECK_CAPABLE = new Set(["cursor", "openclaw"]);
 // Descriptor-derived so a harness that cannot carry a key is skipped, not failed, when headless.
 // Cursor and OpenClaw hold the key themselves rather than handing it to a harness `mcp add`.
@@ -186,6 +191,15 @@ async function processHarness(detection, opts, prompt, hoistedCliInstall) {
         });
     }
     catch (e) {
+        // Both reasons carry the same "flag not listed" message; the upgrade nudge is in it.
+        if (e instanceof ConnectStepError && TOO_OLD_REASONS.has(e.failureReason)) {
+            return {
+                ...base,
+                outcome: "harness_too_old",
+                errorMessage: e.message,
+                fixCommand: `tinyfish connect ${harness}`,
+            };
+        }
         return {
             ...base,
             outcome: "failed",
@@ -250,6 +264,9 @@ function summaryLine(result, uninstall) {
     if (result.outcome === "interrupted") {
         return `✗ ${name} — interrupted before finishing. Fix: ${result.fixCommand}`;
     }
+    if (result.outcome === "harness_too_old") {
+        return `… ${name} — needs a newer ${name}: ${result.errorMessage} Fix, once upgraded: ${result.fixCommand}`;
+    }
     if (result.outcome === "failed") {
         return `✗ ${name} — install failed: ${result.errorMessage}. Fix: ${result.fixCommand}`;
     }
@@ -283,7 +300,10 @@ function computeExitCode(results) {
     if (detected.length === 0)
         return 1;
     // Only real install attempts count toward the success/failure denominator.
-    const attempted = detected.filter((r) => r.outcome === "installed" || r.outcome === "failed" || r.outcome === "interrupted");
+    const attempted = detected.filter((r) => r.outcome === "installed" ||
+        r.outcome === "failed" ||
+        r.outcome === "harness_too_old" ||
+        r.outcome === "interrupted");
     if (attempted.length === 0)
         return 0;
     return attempted.some((r) => r.outcome === "installed") ? 0 : 1;

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

@@ -4,7 +4,7 @@ export type AgentClient = "claude-code" | "codex" | "cursor" | "grok" | "hermes"
 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";
 export declare const CONNECT_FAILURE_REASONS: readonly ["harness_not_installed", "harness_command_unsupported", "harness_too_old", "command_not_found", "timeout", "spawn_error", "nonzero_exit", "invalid_config", "user_abandoned", "process_terminated", "unexpected_error"];
-type ConnectFailureReason = (typeof CONNECT_FAILURE_REASONS)[number];
+export type ConnectFailureReason = (typeof CONNECT_FAILURE_REASONS)[number];
 export type ConnectAuthMode = Exclude<AuthMode, AuthMode.Unknown> | "deferred";
 type ConnectCheckpoint = "prerequisite_ok" | "registered" | "oauth_done" | "cli_installed" | "skill_installed" | "authenticated";
 /** Ctrl+C killed a setup child — abandonment, not error. */

package/dist/lib/skill-install.js

@@ -8,7 +8,6 @@ import { commandNotFound, spawnStepError, throwIfInterrupted } from "./connect-r
 import { CLI_AGENT_IDENTITY } from "./constants.js";
 import { errLine } from "./output.js";
 // Supports Hermes without node:util.styleText, so the installer still runs on Node 20.11.
-// A bump must re-verify SKILL_AGENT_DISPLAY_NAMES against `skills list --json`.
 const SKILLS_CLI_PACKAGE = "skills@1.5.15";
 const TINYFISH_WEB_SKILL_SOURCE = "tinyfish-io/tinyfish-cookbook";
 const TINYFISH_WEB_SKILL = "use-tinyfish";
@@ -45,35 +44,23 @@ const SKILL_ALREADY_CURRENT_PATTERN = /All global skills are up to date/;
 // A lock entry with no recorded hash is untrackable, so `skills` reports it as skipped rather
 // than failed. Left undetected that reads as "up to date" while nothing was refreshed.
 const SKILL_UNCHECKABLE_PATTERN = /cannot be checked automatically/;
-// `skills list --json` reports display names, never slugs (verified on 1.5.15).
-const SKILL_AGENT_DISPLAY_NAMES = {
-    "claude-code": "Claude Code",
-    codex: "Codex",
-    cursor: "Cursor",
-    "hermes-agent": "Hermes Agent",
-    opencode: "OpenCode",
-};
-const skillListSchema = z.array(z.object({ name: z.string(), agents: z.array(z.string()) }));
+const skillListSchema = z.array(z.object({ name: z.string() }));
 // `add` exits 0 on per-skill failure; the piped list is the verdict.
-function failedListVerdict(skillAgents) {
+function failedListVerdict() {
     const list = spawn.sync("npx", ["-y", SKILLS_CLI_PACKAGE, "list", "--global", "--json"], {
         encoding: "utf8",
         env: skillSpawnEnv(),
         maxBuffer: STEP_MAX_BUFFER,
         timeout: SKILL_INSTALL_TIMEOUT_MS,
     });
-    const verified = !list.error && list.status === 0 && verifiedForAgents(list.stdout ?? "", skillAgents);
+    const verified = !list.error && list.status === 0 && skillListed(list.stdout ?? "");
     return verified ? null : list;
 }
-// `list` folds in every agent's dir, so name-only matches stale copies.
-function verifiedForAgents(stdout, skillAgents) {
+// Entry presence is the whole verdict: `agents` names the harnesses `skills` detects,
+// so an install for a harness with no config dir yet lists none (PF-3680).
+function skillListed(stdout) {
     const parsed = skillListSchema.safeParse(parseJson(stdout));
-    if (!parsed.success)
-        return false;
-    const agents = parsed.data
-        .filter((skill) => skill.name === TINYFISH_WEB_SKILL)
-        .flatMap((skill) => skill.agents);
-    return skillAgents.every((agent) => agents.includes(SKILL_AGENT_DISPLAY_NAMES[agent]));
+    return parsed.success && parsed.data.some((skill) => skill.name === TINYFISH_WEB_SKILL);
 }
 function parseJson(text) {
     try {
@@ -93,7 +80,7 @@ export function installWebSkill(client) {
         env: skillSpawnEnv(),
         timeout: SKILL_INSTALL_TIMEOUT_MS,
     });
-    const failed = add.error || add.status !== 0 ? add : failedListVerdict([client.skillAgent]);
+    const failed = add.error || add.status !== 0 ? add : failedListVerdict();
     if (failed) {
         throw spawnStepError(`Could not install the TinyFish web skill in ${client.displayName}`, failed);
     }
@@ -162,7 +149,7 @@ function reinstallWebSkill(skillAgents, verbose) {
     const output = capturedOutput(result);
     // Quiet capture stays (#4434); the list verdict replaced prose matching.
     const addFailed = Boolean(result.error) || result.status !== 0;
-    const failedList = addFailed ? null : failedListVerdict(skillAgents);
+    const failedList = addFailed ? null : failedListVerdict();
     const failed = addFailed || failedList !== null;
     if (failed)
         throwIfInterrupted(failedList ?? result);

package/package.json

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