@tiny-fish/cli 0.47.2 → 0.47.3-next.380

package/dist/lib/config/connect-config-file.d.ts

@@ -1,8 +1,8 @@
 import { type ConnectAuthMode } from '../connect-runtime.js';
 import { cursorMcpPath, writeCursorMcpConfig } from './cursor-config.js';
 import { ompMcpPath, writeOmpMcpConfig } from './omp-config.js';
-import { piMcpPath, writePiMcpConfig } from './pi-config.js';
-declare function piPostConnectNotes(): string | undefined;
+import { piMcpAdapterState, type PiAdapterState, piMcpPath, writePiMcpConfig } from './pi-config.js';
+declare function piPostConnectNotes(adapterState?: PiAdapterState): string | undefined;
 declare const CONFIG_FILE_HARNESSES: {
     cursor: {
         skillTarget: {
@@ -44,6 +44,7 @@ declare const CONFIG_FILE_HARNESSES: {
         write: typeof writePiMcpConfig;
         configPath: typeof piMcpPath;
         tryLaunch: () => false;
+        adapterState: typeof piMcpAdapterState;
         postConnectNote: typeof piPostConnectNotes;
         copy: {
             verified: string;

package/dist/lib/config/connect-config-file.js

@@ -7,7 +7,7 @@ import { ConnectStepError, createConnectTelemetry, runGuarded, settle, } from '.
 import { cursorInstallDeeplink, cursorMcpPath, readCursorTinyfishEntry, writeCursorMcpConfig, } from './cursor-config.js';
 import { ompMcpPath, readOmpTinyfishEntry, writeOmpMcpConfig } from './omp-config.js';
 import { PI_ADAPTER_INSTALL_COMMAND, piMcpAdapterState, piSkillDirMismatch, piMcpPath, readPiTinyfishEntry, writePiMcpConfig, } from './pi-config.js';
-import { AuthMode, HARNESS_DISPLAY_NAMES } from '../harness/harness-detect.js';
+import { AuthMode, detectInstalledHarnesses, HARNESS_COMMANDS, HARNESS_DISPLAY_NAMES, isStaleConfig, } from '../harness/harness-detect.js';
 import { detectHumanInitiated } from '../harness.js';
 import { errLine } from '../output.js';
 import { verifyMcpAuth } from '../verify.js';
@@ -21,15 +21,14 @@ const PI_ADAPTER_DORMANT_NOTE = 'Note: pi has no built-in MCP support, so the en
 // Only the dormant case was spoken for, leaving a working MCP install silent.
 const PI_ADAPTER_LIVE_NOTE = 'pi-mcp-adapter is installed, so pi loads the TinyFish MCP tools as well as the skill once ' +
     'you restart it.';
-function piAdapterNote() {
-    const state = piMcpAdapterState();
+function piAdapterNote(state) {
     if (state === 'installed')
         return PI_ADAPTER_LIVE_NOTE;
     return state === 'absent' ? PI_ADAPTER_DORMANT_NOTE : undefined;
 }
 // The `skills` CLI writes to the default dir even when pi reads another one.
-function piPostConnectNotes() {
-    const notes = [piAdapterNote()];
+function piPostConnectNotes(adapterState) {
+    const notes = [piAdapterNote(adapterState)];
     const skills = piSkillDirMismatch();
     if (skills) {
         notes.push(`Note: PI_CODING_AGENT_DIR points pi at ${skills.readFrom}, but the TinyFish skill ` +
@@ -79,6 +78,7 @@ const CONFIG_FILE_HARNESSES = {
         configPath: piMcpPath,
         // No install deeplink exists; the copy carries the restart instead.
         tryLaunch: () => false,
+        adapterState: piMcpAdapterState,
         postConnectNote: piPostConnectNotes,
         copy: {
             verified: "TinyFish is connected and verified (health+auth). Restart pi and it's live.",
@@ -111,6 +111,36 @@ function reportPostInstallFailure(harness, failed, onPostInstallFailed) {
     errLine(finishSetupHint(harness));
     onPostInstallFailed?.();
 }
+// A dormant config written here reads as `completed` to every dashboard.
+function requireHarnessInstalled(harness) {
+    const detection = detectInstalledHarnesses().find((d) => d.harness === harness);
+    if (detection?.detected && !isStaleConfig(detection))
+        return;
+    throw new ConnectStepError(`${HARNESS_DISPLAY_NAMES[harness]} is not installed (no \`${HARNESS_COMMANDS[harness]}\` on PATH). ` +
+        `Install it, then re-run: tinyfish connect ${harness}`, 'harness_not_installed');
+}
+/** The copy after `completed`: what the user does next depends on the auth path. */
+async function printConnectedCopy(spec, displayName, opts) {
+    if (opts.keyless) {
+        errLine(`TinyFish keyless Search is connected in ${displayName}.`);
+    }
+    else if (opts.resolvedKey) {
+        // Deeplink would embed the key in a URL (process args, LaunchServices logs) — reload instead.
+        const verify = await verifyMcpAuth(opts.resolvedKey, apiBaseFromMcpUrl(opts.apiMcpUrl));
+        if (verify.ok) {
+            errLine(spec.copy.verified);
+        }
+        else {
+            errLine(spec.copy.authFailed(verify.reason ?? 'unknown'));
+        }
+    }
+    else if (detectHumanInitiated() && spec.tryLaunch(opts.mcpUrl.toString())) {
+        errLine(spec.copy.launched);
+    }
+    else {
+        errLine(spec.copy.reload);
+    }
+}
 /** Writes the harness's MCP config file itself; no `mcp add` exists. */
 export async function connectConfigFileHarness(harness, options) {
     const spec = CONFIG_FILE_HARNESSES[harness];
@@ -154,6 +184,7 @@ export async function connectConfigFileHarness(harness, options) {
     };
     await runGuarded(state, telemetry, async () => {
         telemetry.track('started');
+        requireHarnessInstalled(harness);
         telemetry.track('checkpoint', { phase: 'prerequisite_ok' });
         const resolveAuth = async () => {
             let resolvedKey = validatedApiKey(options.apiKey);
@@ -202,28 +233,16 @@ export async function connectConfigFileHarness(harness, options) {
             saveConnectContext(harness, telemetry.attemptId, AuthMode.Keyless);
         else
             saveConnectContext(harness, telemetry.attemptId);
-        settle(state, telemetry, 'completed', { authMode });
+        const piAdapterState = spec.adapterState?.();
+        settle(state, telemetry, 'completed', { authMode, piAdapterState });
         const postInstallFailed = installSkillPostSettle(spec, displayName, state, telemetry, options);
-        if (keyless) {
-            errLine(`TinyFish keyless Search is connected in ${displayName}.`);
-        }
-        else if (resolvedKey) {
-            // Deeplink would embed the key in a URL (process args, LaunchServices logs) — reload instead.
-            const verify = await verifyMcpAuth(resolvedKey, apiBaseFromMcpUrl(options.mcpUrl));
-            if (verify.ok) {
-                errLine(spec.copy.verified);
-            }
-            else {
-                errLine(spec.copy.authFailed(verify.reason ?? 'unknown'));
-            }
-        }
-        else if (detectHumanInitiated() && spec.tryLaunch(mcpUrl.toString())) {
-            errLine(spec.copy.launched);
-        }
-        else {
-            errLine(spec.copy.reload);
-        }
-        const note = spec.postConnectNote?.();
+        await printConnectedCopy(spec, displayName, {
+            keyless,
+            resolvedKey,
+            mcpUrl,
+            apiMcpUrl: options.mcpUrl,
+        });
+        const note = spec.postConnectNote?.(piAdapterState);
         if (note)
             errLine(note);
         reportPostInstallFailure(harness, postInstallFailed, options.onPostInstallFailed);

package/dist/lib/connect-all.js

@@ -12,7 +12,7 @@ import { KEYLESS_ONBOARDING_PROMPT } from './connect-clients.js';
 import { gateApiKey } from './connect-preflight.js';
 import { ConnectInterruptedError } from './connect-runtime.js';
 import { pickHarnesses } from './connect-picker.js';
-import { commandOnPath, detectInstalledHarnesses, AuthMode, HARNESS_COMMANDS, HARNESS_DISPLAY_NAMES as DISPLAY_NAMES, ALL_HARNESSES, } from './harness/harness-detect.js';
+import { commandOnPath, detectInstalledHarnesses, isStaleConfig, AuthMode, HARNESS_COMMANDS, HARNESS_DISPLAY_NAMES as DISPLAY_NAMES, ALL_HARNESSES, } from './harness/harness-detect.js';
 import { detectHumanInitiated } from './harness.js';
 import { emitNotice } from './notice.js';
 import { errLine, outLine, setErrIndent } from './output.js';
@@ -138,13 +138,6 @@ function noWorkOutcome(d, uninstall) {
 export function needsFirstRun(detection) {
     return detection.detected && !detection.configDirExists;
 }
-/** A config-writing connect needs no binary; every other harness spawns one. */
-function isStaleConfig(detection) {
-    const spec = harnessSpec(detection.harness);
-    // Cursor's IDE reads mcp.json binary-less; omp and pi have no second reader.
-    const binaryless = spec.cliWritesConfig && spec.readableWithoutBinary;
-    return detection.detected && !detection.binaryOnPath && !binaryless;
-}
 async function resolvePick(detections, opts, prompt) {
     let deselected = new Set();
     let mode = 'all';

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

@@ -1,5 +1,6 @@
 import { AuthMode } from './harness/harness-detect.js';
 import type { Harness, HarnessSupportCheck } from './harness/harness-spec.js';
+import type { PiAdapterState } from './config/pi-config.js';
 export declare const NON_INTERACTIVE_TIMEOUT_MS = 10000;
 export type AgentClient = Harness;
 type ConnectStage = 'started' | 'checkpoint' | 'completed' | 'failed' | 'aborted' | 'post_install_failed';
@@ -75,6 +76,8 @@ interface ConnectStageDetail {
     stageDurationMs?: number;
     /** Dropped from the wire unless `failedStage` is set (route refine). */
     failureDetail?: string;
+    /** pi only: without the adapter its MCP entry is dormant. */
+    piAdapterState?: PiAdapterState;
 }
 export interface ConnectTelemetry {
     attemptId: string;

package/dist/lib/connect-runtime.js

@@ -325,6 +325,7 @@ export function createConnectTelemetry(mcpUrl, client, opts) {
             plugin_version: detail?.pluginVersion,
             auth_mode: detail?.authMode,
             harness_degraded: detail?.harnessDegraded,
+            pi_adapter_state: detail?.piAdapterState,
             stage_duration_ms: stageDurationMs,
             runtime_platform: process.platform,
             node_version: process.version,

package/dist/lib/doctor-checks.d.ts

@@ -1,4 +1,4 @@
-import type { DoctorCheck } from './doctor-report.js';
+import { type DoctorCheck } from './doctor-report.js';
 import type { RegistrationStatus } from './registration-detect.js';
 import { type VerifyResult } from './verify.js';
 export declare function provesHarnessReach(status: RegistrationStatus, keyAuth: VerifyResult | undefined, healthOk: boolean, mcpUrl: string): boolean;

package/dist/lib/doctor-checks.js

@@ -1,5 +1,6 @@
 import { apiKeyStatus } from './auth.js';
 import { CLI_VERSION } from './constants.js';
+import { NOT_INSTALLED_DETAIL } from './doctor-report.js';
 import { AuthMode, Registered } from './harness/harness-detect.js';
 import { resolveHermesHome } from './hermes-env.js';
 import { hermesWebBackendsOurs, readHermesPluginStatus, } from './hermes-plugin.js';
@@ -100,7 +101,7 @@ function keyedVerdict(status, keyAuth, cliKeyMissing) {
 }
 function registrationVerdict(status, mcpUrl, keyAuth, cliKeyMissing) {
     if (!status.detected)
-        return { status: 'skip', detail: 'harness not installed' };
+        return { status: 'skip', detail: NOT_INSTALLED_DETAIL };
     if (status.registered === Registered.Unknown) {
         return { status: 'warn', detail: status.reason ?? 'could not determine registration' };
     }
@@ -260,7 +261,7 @@ export function checkPiAdapter(status) {
         scope: 'harness',
     };
     if (!status.detected)
-        return { ...base, status: 'skip', detail: 'harness not installed' };
+        return { ...base, status: 'skip', detail: NOT_INSTALLED_DETAIL };
     if (status.registered !== Registered.Yes) {
         const why = status.registered === Registered.Unknown
             ? 'pi registration could not be determined'
@@ -293,7 +294,7 @@ export function checkHermesPlugin(status) {
         scope: 'harness',
     };
     if (!status.detected) {
-        return { check: { ...base, status: 'skip', detail: 'harness not installed' } };
+        return { check: { ...base, status: 'skip', detail: NOT_INSTALLED_DETAIL } };
     }
     if (status.registered !== Registered.Yes) {
         return { check: { ...base, status: 'skip', detail: 'TinyFish is not registered in Hermes' } };

package/dist/lib/doctor-report.d.ts

@@ -219,5 +219,6 @@ export declare function verdictFor(checks: DoctorCheck[]): {
 };
 /** Harness scope only: the caller asked about its agent, not us. */
 export declare function exitCodeFor(checks: DoctorCheck[]): 0 | 1;
+export declare const NOT_INSTALLED_DETAIL = "harness not installed";
 export declare function renderPretty(report: DoctorReport): string;
 export {};

package/dist/lib/doctor-report.js

@@ -95,8 +95,17 @@ function footnotes(report) {
     }
     return notes;
 }
+export const NOT_INSTALLED_DETAIL = 'harness not installed';
+const isNotInstalled = (c) => c.status === 'skip' && c.detail === NOT_INSTALLED_DETAIL;
 export function renderPretty(report) {
-    const lines = report.checks.map((c) => `${GLYPHS[c.status]} ${c.title}${c.detail ? ` — ${c.detail}` : ''}`);
+    const lines = report.checks
+        .filter((c) => !isNotInstalled(c))
+        .map((c) => `${GLYPHS[c.status]} ${c.title}${c.detail ? ` — ${c.detail}` : ''}`);
+    // Per-harness skip rows read as a to-do list; agents ran `connect` on each.
+    const absent = new Set(report.checks.filter(isNotInstalled).map((c) => c.harness));
+    if (absent.size > 0) {
+        lines.push(`${GLYPHS.skip} Not installed, nothing to do: ${[...absent].join(', ')}`);
+    }
     if (report.repairs.length > 0) {
         lines.push('', 'Repairs:');
         for (const repair of report.repairs)

package/dist/lib/harness/harness-detect.d.ts

@@ -28,3 +28,5 @@ export interface HarnessDetection {
     configDirExists: boolean;
 }
 export declare function detectInstalledHarnesses(): HarnessDetection[];
+/** A config-writing connect needs no binary; every other harness spawns one. */
+export declare function isStaleConfig(detection: HarnessDetection): boolean;

package/dist/lib/harness/harness-detect.js

@@ -71,3 +71,10 @@ export function detectInstalledHarnesses() {
         };
     });
 }
+/** A config-writing connect needs no binary; every other harness spawns one. */
+export function isStaleConfig(detection) {
+    const spec = harnessSpec(detection.harness);
+    // Cursor's IDE reads mcp.json binary-less; omp and pi have no second reader.
+    const binaryless = spec.cliWritesConfig && spec.readableWithoutBinary;
+    return detection.detected && !detection.binaryOnPath && !binaryless;
+}

package/package.json

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