@tiny-fish/cli 0.29.1-next.239 → 0.29.1-next.240

package/dist/lib/connect-all.js

@@ -210,6 +210,10 @@ async function processHarness(detection, opts, prompt, oauthBudget, hoistedCliIn
         if (spentBrowserSlot && e instanceof ConnectStepError && NO_BROWSER_REASONS.has(e.failureReason)) {
             oauthBudget.spent = false;
         }
+        // Detection saw a name on PATH; the probe proved nothing is behind it.
+        if (e instanceof ConnectStepError && e.failureReason === "harness_not_installed") {
+            return { ...base, detected: false, outcome: "not_detected" };
+        }
         // 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 {
@@ -268,9 +272,6 @@ function summaryLine(result, uninstall) {
             ? `… ${name} — ${result.errorMessage}. Finish it in a terminal: ${result.fixCommand}`
             : `… ${name} — needs a browser sign-in. Run in a terminal: ${result.fixCommand}`;
     }
-    if (result.outcome === "deselected") {
-        return `… ${name} — skipped, not selected. Fix: ${result.fixCommand}`;
-    }
     if (result.outcome === "uninstall_pointer") {
         return `… ${name} — not removed automatically (config is owned by ${name}). Run: ${result.fixCommand}`;
     }
@@ -307,6 +308,20 @@ function summaryLine(result, uninstall) {
     }
     return `✓ ${name} — installed, ${verifyLabel}`;
 }
+/** Absent and unpicked harnesses are noise once something real happened. */
+function renderSummary(results, uninstall) {
+    if (!results.some(actionable))
+        return results.map((r) => summaryLine(r, uninstall));
+    // Ids, not display names: they double as the `connect` argument.
+    const skipped = results.filter((r) => r.outcome === "deselected").map((r) => r.harness);
+    const lines = results
+        .filter((r) => r.outcome !== "deselected" && r.outcome !== "not_detected")
+        .map((r) => summaryLine(r, uninstall));
+    if (skipped.length === 0)
+        return lines;
+    lines.push("", `Skipped (not selected): ${skipped.join(", ")}`);
+    return lines;
+}
 /** A config-writing connect needs no binary; every other harness spawns one. */
 function isStaleConfig(detection) {
     return (detection.detected &&
@@ -475,7 +490,9 @@ export async function runConnectAll(opts) {
     const outro = new Set();
     const results = [];
     const oauthBudget = { spent: false };
-    const stepTotal = detections.length - deselected.size;
+    // Mirrors processHarness's own short-circuits: absent has no work, stale only when removing.
+    const stepped = (d) => !deselected.has(d.harness) && d.detected && (opts.uninstall || !isStaleConfig(d));
+    const stepTotal = detections.filter(stepped).length;
     let step = 0;
     for (const detection of detections) {
         const { harness } = detection;
@@ -488,6 +505,9 @@ export async function runConnectAll(opts) {
             });
             continue;
         }
+        // The pad loop below records these; announcing them is the noise this command had.
+        if (!stepped(detection))
+            continue;
         step += 1;
         errLine(`step ${step} of ${stepTotal} — ${DISPLAY_NAMES[harness]}`);
         let result;
@@ -533,8 +553,8 @@ export async function runConnectAll(opts) {
     }
     outLine("");
     outLine("TinyFish setup summary:");
-    for (const result of results)
-        outLine(summaryLine(result, opts.uninstall));
+    for (const line of renderSummary(results, opts.uninstall))
+        outLine(line);
     // Before the launch handover: anything printed after it is never seen.
     for (const line of outro)
         errLine(line);

package/dist/lib/connect-runtime.js

@@ -137,7 +137,8 @@ export function requireCommandSupport(client) {
         env: { ...process.env, FORCE_COLOR: "0", NO_COLOR: "1" },
         timeout: NON_INTERACTIVE_TIMEOUT_MS,
     });
-    if (commandNotFound(result.error)) {
+    // 127 is the shell's not-found code; a wrapper shim with no real binary behind it exits that.
+    if (commandNotFound(result.error) || result.status === 127) {
         throw new PrerequisiteError(`${client.displayName} is not installed or not available on PATH.`, "harness_not_installed", { cause: result.error });
     }
     if (result.error || result.status !== 0) {

package/package.json

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