@tiny-fish/cli 0.18.1-next.170 → 0.18.1-next.172

package/dist/commands/upgrade.js

@@ -1,6 +1,7 @@
 import spawn from "cross-spawn";
 import { resolvedApiKey } from "../lib/auth.js";
 import { CLI_VERSION, TINYFISH_CLI_PACKAGE } from "../lib/constants.js";
+import { suppressNotice } from "../lib/notice.js";
 import { errLine } from "../lib/output.js";
 import { sendUpgradeCompleted, telemetryDisabled, } from "../lib/setup-telemetry.js";
 import { installSignalGuard, SIGNAL_EXIT_CODES } from "../lib/signals.js";
@@ -55,6 +56,8 @@ function installedCliVersion() {
 }
 /** `runUpgrade` plus its outcome telemetry, which must never change what the upgrade does. */
 export async function runUpgradeCommand() {
+    // This command is the upgrade, so a trailing "an update is available" would be noise.
+    suppressNotice();
     let reported = false;
     // Read on the failed path too: a failed skill refresh still leaves an upgraded CLI, so
     // `outcome` reports whether every step ran and `to_version` whether the binary moved.

package/dist/index.js

@@ -1,55 +1,6 @@
 #!/usr/bin/env node
-import { createRequire } from "module";
-import { Command } from "commander";
-import { registerAuth } from "./commands/auth.js";
-import { registerBatch } from "./commands/batch.js";
-import { registerFetch } from "./commands/fetch.js";
-import { registerBrowser } from "./commands/browser.js";
-import { registerProfile } from "./commands/profile.js";
-import { registerVault } from "./commands/vault.js";
-import { registerWallet } from "./commands/wallet.js";
-import { registerRun } from "./commands/run.js";
-import { registerRuns } from "./commands/runs.js";
-import { registerConfigureClaude } from "./commands/config-claude.js";
-import { registerSearch } from "./commands/search.js";
-import { registerConnect } from "./commands/connect.js";
-import { registerDoctor } from "./commands/doctor.js";
-import { registerUpgrade } from "./commands/upgrade.js";
-import { registerOnboard } from "./commands/onboard.js";
-const { version } = createRequire(import.meta.url)("../package.json");
-const program = new Command();
-program
-    .name("tinyfish")
-    .description("TinyFish CLI — run web automations from your terminal or agent.")
-    .version(version, "-V, --version", "Show version")
-    .helpOption("-h, --help", "Show help")
-    .addHelpCommand(false)
-    .enablePositionalOptions()
-    .option("--debug", "Print HTTP requests and responses to stderr (or set TINYFISH_DEBUG=1)")
-    .hook("preAction", (cmd) => {
-    if (cmd.opts().debug) {
-        process.env["TINYFISH_DEBUG"] = "1";
-    }
-});
-registerAuth(program);
-registerOnboard(program);
-registerConnect(program);
-registerDoctor(program);
-registerUpgrade(program);
-registerConfigureClaude(program);
-registerBrowser(program);
-registerProfile(program);
-registerVault(program);
-registerWallet(program);
-const agentCmd = program
-    .command("agent")
-    .description("Agent automation commands")
-    .enablePositionalOptions();
-const runCmd = registerRun(agentCmd);
-registerRuns(runCmd);
-registerBatch(agentCmd);
-registerSearch(program);
-registerFetch(program);
+import { buildProgram } from "./program.js";
+const { program } = buildProgram();
 // Await parseAsync so async command handlers complete before the process exits
 program.parseAsync(process.argv).catch((e) => {
     process.stderr.write(JSON.stringify({ error: e instanceof Error ? e.message : String(e) }) + "\n");

package/dist/lib/client.js

@@ -2,7 +2,7 @@ import { APIStatusError, agentRunAsyncResponseSchema, agentRunResponseSchema, ag
 import { CONNECT_SOURCE, loadConfig } from "./auth.js";
 import { API_URL_OVERRIDE, CLI_VERSION } from "./constants.js";
 import { detectHarness, detectHumanInitiated } from "./harness.js";
-import { surfaceNotice } from "./notice.js";
+import { captureNotice } from "./notice.js";
 import { ApiError } from "./output.js";
 import { z } from "zod";
 // Install-level connect context (PF-3169). A known caller is credited only to its own
@@ -73,7 +73,7 @@ class TinyFishCliClient extends TinyFish {
             method: "DELETE",
             headers: this._buildHeaders(),
         });
-        surfaceNotice(response.headers);
+        captureNotice(response.headers);
         if (!response.ok) {
             let message = response.statusText;
             try {
@@ -96,7 +96,7 @@ function createSdkClient(apiKey, timeout) {
         ...(API_URL_OVERRIDE !== undefined ? { baseURL: API_URL_OVERRIDE } : {}),
         maxRetries: 0,
         // Fires per HTTP attempt, but maxRetries is 0 and the notice latches once per invocation.
-        onResponse: ({ headers }) => surfaceNotice(headers),
+        onResponse: ({ headers }) => captureNotice(headers),
         ...(timeout !== undefined ? { timeout } : {}),
     });
 }

package/dist/lib/notice.d.ts

@@ -1,8 +1,11 @@
+import type { Command } from "commander";
 /** Test-only: the capture slot and the once-per-invocation latch are module state. */
 export declare function resetNoticeState(): void;
 /** Records the first notice seen; printing is deferred so it lands at the end of a command. */
 export declare function captureNotice(headers: Headers): void;
 /** Advisory only, never changes stdout, the exit code, or control flow. */
 export declare function emitNotice(): void;
-/** For callers where the response is the end of the command, such as one-shot API calls. */
-export declare function surfaceNotice(headers: Headers): void;
+/** For commands that are the fix themselves, such as `upgrade`, where the nudge is noise. */
+export declare function suppressNotice(): void;
+/** postAction covers clean completions, the exit hook covers the process.exit and rejection paths, the latch keeps it to one. */
+export declare function installNoticeEmit(program: Command): () => void;

package/dist/lib/notice.js

@@ -1,7 +1,6 @@
-// Surfaces the advisory X-TF-Notice header the server attaches to outdated clients (PF-3301).
 import { CLI_VERSION } from "./constants.js";
 import { detectHumanInitiated } from "./harness.js";
-import { err, errLine, sanitizeLine } from "./output.js";
+import { err, sanitizeLine, warnLine } from "./output.js";
 const NOTICE_HEADER = "x-tf-notice";
 const LEVEL_HEADER = "x-tf-notice-level";
 const MAX_LENGTH = 300;
@@ -41,13 +40,20 @@ export function emitNotice() {
     printed = true;
     const { message, level } = captured;
     if (detectHumanInitiated()) {
-        errLine(message);
+        warnLine(message);
         return;
     }
     err({ notice: agentMessage(level), ...(level ? { level } : {}) });
 }
-/** For callers where the response is the end of the command, such as one-shot API calls. */
-export function surfaceNotice(headers) {
-    captureNotice(headers);
-    emitNotice();
+/** For commands that are the fix themselves, such as `upgrade`, where the nudge is noise. */
+export function suppressNotice() {
+    captured = undefined;
+    printed = true;
+}
+/** postAction covers clean completions, the exit hook covers the process.exit and rejection paths, the latch keeps it to one. */
+export function installNoticeEmit(program) {
+    const backstop = () => emitNotice();
+    program.hook("postAction", () => emitNotice());
+    process.once("exit", backstop);
+    return () => process.removeListener("exit", backstop);
 }

package/dist/lib/output.d.ts

@@ -8,5 +8,7 @@ export declare function out(data: unknown): void;
 export declare function sanitizeLine(text: string): string;
 export declare function outLine(line: string): void;
 export declare function errLine(line: string): void;
+/** Advisory stderr line, yellow on a terminal and plain everywhere else so piped output stays clean. */
+export declare function warnLine(line: string): void;
 export declare function err(data: unknown): void;
 export declare function handleApiError(e: unknown): never;

package/dist/lib/output.js

@@ -23,6 +23,14 @@ export function outLine(line) {
 export function errLine(line) {
     process.stderr.write(sanitizeLine(line) + "\n");
 }
+const WARN_ON = "\x1b[33m";
+const WARN_OFF = "\x1b[39m";
+/** Advisory stderr line, yellow on a terminal and plain everywhere else so piped output stays clean. */
+export function warnLine(line) {
+    const text = sanitizeLine(line);
+    const colour = Boolean(process.stderr.isTTY) && !process.env["NO_COLOR"];
+    process.stderr.write(colour ? `${WARN_ON}${text}${WARN_OFF}\n` : `${text}\n`);
+}
 export function err(data) {
     const payload = typeof data === "string" ? { error: data } : data;
     const serialized = JSON.stringify(payload) ?? "undefined";

package/dist/program.d.ts

@@ -0,0 +1,6 @@
+import { Command } from "commander";
+/** Split from index.ts so tests can drive the real program instead of hand-assembling one. */
+export declare function buildProgram(): {
+    program: Command;
+    uninstallNotice: () => void;
+};

package/dist/program.js

@@ -0,0 +1,57 @@
+import { createRequire } from "module";
+import { Command } from "commander";
+import { registerAuth } from "./commands/auth.js";
+import { registerBatch } from "./commands/batch.js";
+import { registerFetch } from "./commands/fetch.js";
+import { registerBrowser } from "./commands/browser.js";
+import { registerProfile } from "./commands/profile.js";
+import { registerVault } from "./commands/vault.js";
+import { registerWallet } from "./commands/wallet.js";
+import { registerRun } from "./commands/run.js";
+import { registerRuns } from "./commands/runs.js";
+import { registerConfigureClaude } from "./commands/config-claude.js";
+import { registerSearch } from "./commands/search.js";
+import { registerConnect } from "./commands/connect.js";
+import { registerDoctor } from "./commands/doctor.js";
+import { registerUpgrade } from "./commands/upgrade.js";
+import { registerOnboard } from "./commands/onboard.js";
+import { installNoticeEmit } from "./lib/notice.js";
+/** Split from index.ts so tests can drive the real program instead of hand-assembling one. */
+export function buildProgram() {
+    const { version } = createRequire(import.meta.url)("../package.json");
+    const program = new Command();
+    program
+        .name("tinyfish")
+        .description("TinyFish CLI — run web automations from your terminal or agent.")
+        .version(version, "-V, --version", "Show version")
+        .helpOption("-h, --help", "Show help")
+        .addHelpCommand(false)
+        .enablePositionalOptions()
+        .option("--debug", "Print HTTP requests and responses to stderr (or set TINYFISH_DEBUG=1)")
+        .hook("preAction", (cmd) => {
+        if (cmd.opts().debug) {
+            process.env["TINYFISH_DEBUG"] = "1";
+        }
+    });
+    registerAuth(program);
+    registerOnboard(program);
+    registerConnect(program);
+    registerDoctor(program);
+    registerUpgrade(program);
+    registerConfigureClaude(program);
+    registerBrowser(program);
+    registerProfile(program);
+    registerVault(program);
+    registerWallet(program);
+    const agentCmd = program
+        .command("agent")
+        .description("Agent automation commands")
+        .enablePositionalOptions();
+    const runCmd = registerRun(agentCmd);
+    registerRuns(runCmd);
+    registerBatch(agentCmd);
+    registerSearch(program);
+    registerFetch(program);
+    const uninstallNotice = installNoticeEmit(program);
+    return { program, uninstallNotice };
+}

package/package.json

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