@tiny-fish/cli 0.50.1-next.399 → 0.50.1-next.401

package/dist/lib/cli-presence.d.ts

@@ -0,0 +1,4 @@
+/** Connect installs the CLI for the skills; only a missing or older PATH copy needs one. */
+export declare function cliInstallNeeded(): boolean;
+/** SemVer order: a prerelease sorts before its release (`0.51.0-next.3` < `0.51.0`). */
+export declare function isOlder(installed: string, current: string): boolean;

package/dist/lib/cli-presence.js

@@ -0,0 +1,54 @@
+import * as path from 'node:path';
+import spawn from 'cross-spawn';
+import which from 'which';
+import { CLI_VERSION } from './constants.js';
+import { installRoot } from './install-root.js';
+const VERSION_PROBE_TIMEOUT_MS = 10_000;
+const SEMVER = /^\d+\.\d+\.\d+(?:-\S+)?$/;
+/** Connect installs the CLI for the skills; only a missing or older PATH copy needs one. */
+export function cliInstallNeeded() {
+    // Running from the global prefix means PATH already has this exact version.
+    if (installRoot())
+        return false;
+    // npx puts its own cache bin first on PATH, and that copy vanishes after the run.
+    const candidates = (which.sync('tinyfish', { all: true, nothrow: true }) ?? []).filter((candidate) => !candidate.split(path.sep).includes('_npx') && dirOnPath(candidate));
+    if (candidates.length === 0)
+        return true;
+    const installed = pathCliVersion(candidates[0]);
+    if (!installed)
+        return true;
+    return isOlder(installed, CLI_VERSION);
+}
+/** SemVer order: a prerelease sorts before its release (`0.51.0-next.3` < `0.51.0`). */
+export function isOlder(installed, current) {
+    const [installedCore, installedPre] = installed.split(/-(.*)/);
+    const [currentCore, currentPre] = current.split(/-(.*)/);
+    const order = installedCore.localeCompare(currentCore, 'en', { numeric: true });
+    if (order !== 0)
+        return order < 0;
+    if (!installedPre || !currentPre)
+        return Boolean(installedPre) && !currentPre;
+    return installedPre.localeCompare(currentPre, 'en', { numeric: true }) < 0;
+}
+// which searches the cwd first on Windows; a skill run elsewhere would miss that copy.
+function dirOnPath(candidate) {
+    const key = (dir) => {
+        const resolved = path.resolve(dir);
+        return process.platform === 'win32' ? resolved.toLowerCase() : resolved;
+    };
+    const dirs = (process.env['PATH'] ?? '').split(path.delimiter).filter(Boolean).map(key);
+    return dirs.includes(key(path.dirname(candidate)));
+}
+function pathCliVersion(binary) {
+    const result = spawn.sync(binary, ['--version'], {
+        encoding: 'utf8',
+        timeout: VERSION_PROBE_TIMEOUT_MS,
+    });
+    if (result.error || result.status !== 0)
+        return undefined;
+    // Unparseable output stays undefined, which installs: today's behaviour.
+    return result.stdout
+        ?.split(/\r?\n/)
+        .map((line) => line.trim())
+        .find((line) => SEMVER.test(line));
+}

package/dist/lib/connect-all.js

@@ -4,6 +4,7 @@ import spawn from 'cross-spawn';
 import { connectHarness } from './connect-harness.js';
 import { loadConfig, validatedApiKey } from './auth.js';
 import { installTinyFishCli, } from './cli-install.js';
+import { cliInstallNeeded } from './cli-presence.js';
 import { authGate, mapConnectError, OAUTH_SIGN_IN_TIMEOUT_MS } from './connect-all-auth.js';
 import { actionable, computeExitCode, reloadHint, renderSummary } from './connect-all-summary.js';
 import { planText, uninstallHarness, uninstallPlanText } from './connect-all-uninstall.js';
@@ -167,8 +168,12 @@ function hoistCliIfNeeded(detections, opts, willAttempt) {
             willAttempt(d.harness));
     if (!cliNeeded)
         return undefined;
-    errLine('First: the TinyFish CLI, shared by every harness.');
-    const outcome = runHoistedCliInstall(opts.verbose ?? false);
+    // A skip still reports cli_installed; the route rejects unknown phases.
+    let outcome = { ok: true, durationMs: 0 };
+    if (cliInstallNeeded()) {
+        errLine('First: the TinyFish CLI, shared by every harness.');
+        outcome = runHoistedCliInstall(opts.verbose ?? false);
+    }
     if (!outcome)
         return 'interrupted';
     let taken = false;

package/dist/lib/connect-fallback.js

@@ -2,6 +2,7 @@ import * as readline from 'node:readline/promises';
 import spawn from 'cross-spawn';
 import { resolvedApiKey, validateKeyFormat } from './auth.js';
 import { installTinyFishCli } from './cli-install.js';
+import { cliInstallNeeded } from './cli-presence.js';
 import { ensureCliAuthenticated, TINYFISH_CLI_NOT_FOUND_MESSAGE } from './connect-auth.js';
 import { TINYFISH_API_KEY_VAR } from './constants.js';
 import { commandNotFound, ConnectInterruptedError, ConnectStepError, throwIfInterrupted, } from './connect-runtime.js';
@@ -50,6 +51,8 @@ export function createStdinPrompt() {
     };
 }
 function installStep(opts) {
+    if (!cliInstallNeeded())
+        return null;
     try {
         installTinyFishCli({ verbose: opts.verbose, announce: true });
         return null;

package/dist/lib/connect-steps.js

@@ -1,5 +1,6 @@
 import { verifyMcpHealth } from './verify.js';
 import { installTinyFishCli } from './cli-install.js';
+import { cliInstallNeeded } from './cli-presence.js';
 import { createStdinPrompt, runCliFallback } from './connect-fallback.js';
 import { detectHumanInitiated } from './harness.js';
 import { errLine } from './output.js';
@@ -77,7 +78,8 @@ export function runCliInstallStep(options, telemetry) {
         takeHoistedCliInstall(options.hoistedCliInstall, telemetry);
         return;
     }
-    installTinyFishCli({ verbose: options.verbose ?? false, announce: true });
+    if (cliInstallNeeded())
+        installTinyFishCli({ verbose: options.verbose ?? false, announce: true });
     telemetry.track('checkpoint', { phase: 'cli_installed' });
 }
 export function walkthroughPhase(outcome) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiny-fish/cli",
-  "version": "0.50.1-next.399",
+  "version": "0.50.1-next.401",
   "description": "TinyFish CLI — run web automations from your terminal",
   "type": "module",
   "bin": {
@@ -32,7 +32,8 @@
     "type-check": "tsc --noEmit --project tsconfig.all.json",
     "prepublishOnly": "npm run build",
     "lint:budget": "bash ../../.github/scripts/oxlint-budget.sh sdk/cli",
-    "format:check": "oxfmt --check ."
+    "format:check": "oxfmt --check .",
+    "test:e2e": "vitest --run --config vitest.e2e.config.ts"
   },
   "dependencies": {
     "@tiny-fish/sdk": "^0.8.0",