@tiny-fish/cli 0.45.1-next.352 → 0.45.1-next.353

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

@@ -2,6 +2,8 @@ import { z } from 'zod';
 import { AuthMode, type Harness, Registered } from './harness-detect.js';
 /** Bumped whenever a consumer could misread the payload; the cookbook skill releases separately. */
 export declare const DOCTOR_SCHEMA_VERSION = 3;
+/** Closed: three consumers match these ids by literal. */
+export declare const CHECK_IDS: readonly ["cli-version", "mcp-connectivity", "harness-registration", "cli-credential", "cli-auth-call", "pi-adapter", "hermes-plugin"];
 declare const checkStatusSchema: z.ZodEnum<{
     pass: "pass";
     fail: "fail";
@@ -14,7 +16,15 @@ declare const checkScopeSchema: z.ZodEnum<{
     info: "info";
 }>;
 declare const doctorCheckSchema: z.ZodObject<{
-    id: z.ZodString;
+    id: z.ZodEnum<{
+        "cli-version": "cli-version";
+        "mcp-connectivity": "mcp-connectivity";
+        "harness-registration": "harness-registration";
+        "cli-credential": "cli-credential";
+        "cli-auth-call": "cli-auth-call";
+        "pi-adapter": "pi-adapter";
+        "hermes-plugin": "hermes-plugin";
+    }>;
     title: z.ZodString;
     status: z.ZodEnum<{
         pass: "pass";
@@ -92,7 +102,15 @@ export declare const doctorReportSchema: z.ZodObject<{
     ok_harnesses: z.ZodBoolean;
     ok_cli: z.ZodBoolean;
     checks: z.ZodArray<z.ZodObject<{
-        id: z.ZodString;
+        id: z.ZodEnum<{
+            "cli-version": "cli-version";
+            "mcp-connectivity": "mcp-connectivity";
+            "harness-registration": "harness-registration";
+            "cli-credential": "cli-credential";
+            "cli-auth-call": "cli-auth-call";
+            "pi-adapter": "pi-adapter";
+            "hermes-plugin": "hermes-plugin";
+        }>;
         title: z.ZodString;
         status: z.ZodEnum<{
             pass: "pass";

package/dist/lib/doctor-report.js

@@ -2,12 +2,22 @@ import { z } from 'zod';
 import { ALL_HARNESSES, AuthMode, Registered } from './harness-detect.js';
 /** Bumped whenever a consumer could misread the payload; the cookbook skill releases separately. */
 export const DOCTOR_SCHEMA_VERSION = 3;
+/** Closed: three consumers match these ids by literal. */
+export const CHECK_IDS = [
+    'cli-version',
+    'mcp-connectivity',
+    'harness-registration',
+    'cli-credential',
+    'cli-auth-call',
+    'pi-adapter',
+    'hermes-plugin',
+];
 const checkStatusSchema = z.enum(['pass', 'fail', 'warn', 'skip']);
 const harnessSchema = z.enum(ALL_HARNESSES);
 // Not derived from `harness`: connectivity is harness-null yet harness-scoped.
 const checkScopeSchema = z.enum(['harness', 'cli', 'info']);
 const doctorCheckSchema = z.object({
-    id: z.string(),
+    id: z.enum(CHECK_IDS),
     title: z.string(),
     status: checkStatusSchema,
     detail: z.string(),

package/dist/lib/setup-telemetry.js

@@ -95,6 +95,10 @@ export async function postConnectEvent(options) {
         }
     }
 }
+/** Route 400s a bad id, and a 4xx is never retried. */
+function stampedAttemptId(id) {
+    return id && z.uuid().safeParse(id).success ? id : undefined;
+}
 const harnessResultSchema = z.object({
     harness: z.enum(ALL_HARNESSES),
     detected: z.boolean(),
@@ -106,8 +110,7 @@ const harnessResultSchema = z.object({
 export async function sendSetupStarted(mcpUrl, attemptId, apiKey) {
     if (telemetryDisabled())
         return;
-    // Route 400s a bad id and postConnectEvent never retries 4xx, losing the whole event.
-    const stamped = attemptId && z.uuid().safeParse(attemptId).success ? attemptId : undefined;
+    const stamped = stampedAttemptId(attemptId);
     await postConnectEvent({
         endpoint: new URL('/api/cli/connect-event', mcpUrl).toString(),
         body: JSON.stringify({
@@ -127,7 +130,7 @@ export async function sendSetupStarted(mcpUrl, attemptId, apiKey) {
 export async function sendSetupBlocked(mcpUrl, detail, apiKey) {
     if (telemetryDisabled())
         return;
-    const stamped = detail.attemptId && z.uuid().safeParse(detail.attemptId).success ? detail.attemptId : undefined;
+    const stamped = stampedAttemptId(detail.attemptId);
     await postConnectEvent({
         endpoint: new URL('/api/cli/connect-event', mcpUrl).toString(),
         body: JSON.stringify({
@@ -161,6 +164,8 @@ export const setupCompletedPayloadSchema = z.object({
 });
 /** `toVersion` is null when the run was interrupted or the installed version could not be read. */
 export async function sendUpgradeCompleted(fromVersion, toVersion, outcome, scope, apiKey) {
+    if (telemetryDisabled())
+        return;
     await postConnectEvent({
         endpoint: new URL('/api/cli/connect-event', BASE_URL).toString(),
         body: JSON.stringify({
@@ -181,6 +186,7 @@ export async function sendSetupCompleted(mcpUrl, harnesses, apiKey, mode, attemp
     // Ahead of the parse so an opt-out run prints no telemetry debug.
     if (telemetryDisabled())
         return;
+    const stamped = stampedAttemptId(attemptId);
     let endpoint;
     let body;
     try {
@@ -189,10 +195,7 @@ export async function sendSetupCompleted(mcpUrl, harnesses, apiKey, mode, attemp
             harnesses,
             cli_version: CLI_VERSION,
             ...(mode ? { mode } : {}),
-            // Guarded here, not only at the route: a parse throw drops the whole event, not one field.
-            ...(attemptId && z.uuid().safeParse(attemptId).success
-                ? { connect_attempt_id: attemptId }
-                : {}),
+            ...(stamped ? { connect_attempt_id: stamped } : {}),
         });
         endpoint = new URL('/api/cli/connect-event', mcpUrl).toString();
         body = JSON.stringify({

package/package.json

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