ossclip 0.1.14 → 0.1.16

package/editor-dist/index.html

@@ -4,7 +4,7 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>ossclip editor</title>
-    <script type="module" crossorigin src="/assets/index-CKpBGMBB.js"></script>
+    <script type="module" crossorigin src="/assets/index-CdwzWN3j.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-C8IPo60X.css">
   </head>
   <body>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ossclip",
-  "version": "0.1.14",
+  "version": "0.1.16",
   "description": "Local-first CLI video producer: cuts silence and fillers, word-timed captions, face-aware framing, and LLM-planned code-rendered graphics — transcription and rendering never leave your machine",
   "type": "module",
   "license": "MIT",
@@ -36,9 +36,9 @@
     "commander": "^12.1.0",
     "tsx": "^4.19.0",
     "zod": "^3.25.76",
-    "@ossclip/core": "0.1.14",
-    "@ossclip/scenes": "0.1.14",
-    "@ossclip/renderer": "0.1.14"
+    "@ossclip/core": "0.1.16",
+    "@ossclip/renderer": "0.1.16",
+    "@ossclip/scenes": "0.1.16"
   },
   "homepage": "https://github.com/AhsanAyaz/ossclip#readme",
   "bugs": {

package/src/doctor.ts

@@ -144,27 +144,36 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
         }),
   });
 
-  // Provider, in the same order auto-detection uses (gemini → claude →
-  // claude-cli). Needed for --produce only — the cut+captions path never
-  // touches an LLM — but doctor's contract is "ready for the whole thing".
-  const provider = p.env.GEMINI_API_KEY
-    ? "gemini (GEMINI_API_KEY is set)"
-    : p.env.ANTHROPIC_API_KEY
-      ? "claude (ANTHROPIC_API_KEY is set)"
-      : (await p.binRuns("claude", "--version"))
-        ? "claude-cli (logged-in Claude Code)"
-        : null;
+  // Provider, in the same order auto-detection uses (agy → claude CLI →
+  // gemini key → anthropic key): subscription CLIs beat ambient env keys
+  // since 2026-08 — a logged-in CLI is an explicit, already-paid choice
+  // (FINDINGS §132, antigravity provider). Bin overrides are honored so
+  // doctor probes the same binary produce would spawn. Needed for --produce
+  // only — the cut+captions path never touches an LLM — but doctor's
+  // contract is "ready for the whole thing".
+  const provider = (await p.binRuns(p.env.OSSCLIP_AGY_BIN ?? "agy", "--version"))
+    ? "antigravity (agy CLI on PATH)"
+    : (await p.binRuns(p.env.OSSCLIP_CLAUDE_BIN ?? "claude", "--version"))
+      ? "claude-cli (logged-in Claude Code)"
+      : p.env.GEMINI_API_KEY
+        ? "gemini (GEMINI_API_KEY is set)"
+        : p.env.ANTHROPIC_API_KEY
+          ? "claude (ANTHROPIC_API_KEY is set)"
+          : null;
   checks.push({
     name: "LLM provider",
     ok: provider !== null,
-    detail: provider ?? "no key set and no claude CLI on PATH — needed for --produce; cut+captions works without",
+    detail:
+      provider ??
+      "no agy or claude CLI on PATH and no key set — needed for --produce; cut+captions works without",
     ...(provider !== null
       ? {}
       : {
           fix:
             "run `ossclip setup` (it can save a key for you), or " +
             "export ANTHROPIC_API_KEY or GEMINI_API_KEY (a .env file works — see README), " +
-            "or install Claude Code (https://claude.com/claude-code) and log in",
+            "or install Claude Code (https://claude.com/claude-code) and log in, " +
+            "or install Google Antigravity (https://antigravity.google) and log in",
         }),
   });
 

package/src/interactive/produce-argv.ts

@@ -1,3 +1,5 @@
+import type { ProviderName } from "@ossclip/core";
+
 /**
  * Wizard answers → the argv a user could have typed.
  *
@@ -24,7 +26,10 @@ export interface ProduceExtras {
    * the positive `--captions` stays flags-only — it exists for replay
    * pinning, and emitting it here would restate the default. */
   captions?: boolean;
-  llm?: "claude" | "claude-cli" | "gemini" | "mock";
+  /** core's ProviderName, not an inline union — a provider added there must
+   * not be silently unofferable here (the pre-§132 union had already
+   * drifted: it never listed "antigravity"). */
+  llm?: ProviderName;
 }
 
 export interface ProduceAnswers {

package/src/interactive/produce-wizard.ts

@@ -310,6 +310,9 @@ export async function produceWizard(
       await select({
         message: "LLM provider",
         options: [
+          // Listed in auto-detection's own order (FINDINGS §132): the menu
+          // teaching a different ranking than a bare run uses would be drift.
+          { value: "antigravity", label: "antigravity", hint: "your logged-in Google Antigravity (agy), no API charges" },
           { value: "claude-cli", label: "claude-cli", hint: "your logged-in Claude Code, no API charges" },
           { value: "claude", label: "claude", hint: "needs ANTHROPIC_API_KEY" },
           { value: "gemini", label: "gemini", hint: "needs GEMINI_API_KEY" },

package/src/llm-detect.ts

@@ -0,0 +1,62 @@
+import { existsSync } from "node:fs";
+import { delimiter, join } from "node:path";
+import type { ProviderName } from "@ossclip/core";
+
+/**
+ * Provider auto-detection helpers (FINDINGS §132, antigravity provider):
+ * `binOnPath` is the real `hasBin` the CLI injects into core's
+ * `defaultProviderName`, and `detectionLine` is the one place the "which
+ * provider and why" message lives — pure, total over every ProviderName, so
+ * the drift test can pin all five lines. The old inline ternary in produce.ts
+ * covered only two providers and printed the ANTHROPIC line for a
+ * gemini-detected run.
+ */
+
+/**
+ * Is this binary reachable? Sync, and a PATH scan with `existsSync` rather
+ * than a `spawnSync(bin, ["--version"])` probe: this runs on produce's
+ * startup path, and spawning agy + claude just to detect them costs
+ * ~100ms × 2 on every run. Existence-not-runnability is the same trade
+ * doctor's `binRuns` makes in the other direction — doctor is diagnostics
+ * and can afford the spawn; startup can't.
+ */
+export function binOnPath(
+  bin: string,
+  env: NodeJS.ProcessEnv = process.env,
+  platform: NodeJS.Platform = process.platform,
+): boolean {
+  // A path-ish override (OSSCLIP_AGY_BIN=/opt/agy) names a file, not a PATH
+  // entry — check it verbatim. `\` counts on win32 only; it is a legal
+  // filename character on posix.
+  if (bin.includes("/") || (platform === "win32" && bin.includes("\\"))) {
+    return existsSync(bin);
+  }
+  const pathEntries = (env.PATH ?? "").split(delimiter).filter(Boolean);
+  // On Windows a bare `agy` resolves through PATHEXT (agy.cmd, agy.exe…);
+  // the extensionless name is still tried first for shims that have none.
+  const suffixes =
+    platform === "win32"
+      ? ["", ...(env.PATHEXT ?? ".COM;.EXE;.BAT;.CMD").split(";").filter(Boolean)]
+      : [""];
+  return pathEntries.some((dir) => suffixes.some((ext) => existsSync(join(dir, bin + ext))));
+}
+
+/**
+ * The "▸ which provider and why" line for an auto-detected run. Each line
+ * names its own trigger so a surprised user can see exactly which check won
+ * — the point of the §132 order change is visible, not silent.
+ */
+export function detectionLine(name: ProviderName): string {
+  switch (name) {
+    case "antigravity":
+      return "▸ agy CLI found — using Google Antigravity (subscription auth)";
+    case "claude-cli":
+      return "▸ using the Claude Code CLI (subscription auth)";
+    case "gemini":
+      return "▸ GEMINI_API_KEY found — using the Gemini API";
+    case "claude":
+      return "▸ ANTHROPIC_API_KEY found — using the Claude API";
+    case "mock":
+      return "▸ using the mock provider (no LLM)";
+  }
+}

package/src/produce.ts

@@ -98,6 +98,7 @@ import {
   type Transcript,
 } from "@ossclip/core";
 import { recordRecentProject } from "./edit";
+import { binOnPath, detectionLine } from "./llm-detect";
 import {
   strandedOverrideSiblings,
   strandedPointerLine,
@@ -122,6 +123,16 @@ export interface ProduceResult {
   workdir: string;
   out?: string;
   rendered: boolean;
+  /**
+   * Telemetry inputs (FINDINGS §134), surfaced here so the command layer can
+   * build the event without produce() knowing telemetry exists. The duration
+   * is only ever SENT as a bucket, and the provider is the resolved NAME —
+   * never a key, never a path.
+   */
+  sourceDurationSec: number;
+  sceneCount: number;
+  /** Resolved provider name when the LLM ran; undefined without --produce. */
+  llmProvider?: string;
 }
 
 /**
@@ -848,16 +859,16 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
   // LLM ran. Everything downstream that a viewer READS — captions, scene copy,
   // the grounding check — uses the repaired transcript instead, so a
   // mishearing can't reach the screen twice in two different spellings.
-  const providerName = opts.provider ?? defaultProviderName();
+  const providerName = opts.provider ?? defaultProviderName(process.env, binOnPath);
   let provider: LlmProvider | null = null;
   const needsLlm = opts.produce === true;
   if (needsLlm) {
+    // Only when auto-detected: a typed --llm needs no explanation. The line
+    // itself lives in llm-detect.ts so a drift test covers every provider —
+    // the inline ternary this replaces printed the ANTHROPIC line for a
+    // gemini-detected run (FINDINGS §132, antigravity provider).
     if (!opts.provider) {
-      console.log(
-        providerName === "claude-cli"
-          ? "▸ no ANTHROPIC_API_KEY — using the Claude Code CLI (subscription auth)"
-          : "▸ ANTHROPIC_API_KEY found — using the Claude API",
-      );
+      console.log(detectionLine(providerName));
     }
     provider = createTieredProvider(providerName, {
       model: opts.llmModel,
@@ -2196,7 +2207,13 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
   if (!opts.render) {
     console.log(`▸ skipping render (--no-render). Props at ${join(work, "render-props.json")}`);
     console.log(editHint(work));
-    return { workdir: work, rendered: false };
+    return {
+      workdir: work,
+      rendered: false,
+      sourceDurationSec: sourceProbe.duration,
+      sceneCount: scenes.length,
+      llmProvider: provider ? providerName : undefined,
+    };
   }
 
   const outPath = resolve(opts.out ?? defaultOutPath(originalInput));
@@ -2379,5 +2396,12 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
   await recordRecentProject(work);
   console.log(`✓ done → ${outPath}`);
   console.log(editHint(work));
-  return { workdir: work, out: outPath, rendered: true };
+  return {
+    workdir: work,
+    out: outPath,
+    rendered: true,
+    sourceDurationSec: sourceProbe.duration,
+    sceneCount: scenes.length,
+    llmProvider: provider ? providerName : undefined,
+  };
 }

package/src/program.ts

@@ -8,6 +8,14 @@ import { STUDIO_ENTRY } from "@ossclip/renderer";
 import { loadEnvFiles } from "./env";
 import { produce } from "./produce";
 import { setReplayArgv } from "./replay-argv";
+import {
+  bootstrapTelemetry,
+  durationBucket,
+  loadState,
+  maybeAskRating,
+  saveState,
+  telemetryOffReason,
+} from "./telemetry";
 
 // Before anything reads a provider key (R16 §77) — including the auto-detect
 // order in `defaultProviderName`, which decides which model runs.
@@ -30,6 +38,12 @@ const envFiles = loadEnvFiles();
 export function buildProgram(): Command {
   const program = new Command();
 
+  // Before dispatch, so the one-time first-run notice precedes any command's
+  // own output. Inert in this repo's tests by construction: while POSTHOG_KEY
+  // is the placeholder, bootstrap touches no disk and sends nothing (FINDINGS
+  // §134) — which is what lets every test build the real program unmocked.
+  const telemetry = bootstrapTelemetry();
+
   program
     .name("ossclip")
     .description(
@@ -212,8 +226,16 @@ export function buildProgram(): Command {
     .option("--intent <text>", "what the video should be ('educational video about agents…')")
     .option(
       "--llm <provider>",
-      "claude | claude-cli | gemini | mock. Default: claude if ANTHROPIC_API_KEY is set, " +
-        "else claude-cli (your logged-in Claude Code — Pro/Max subscription, no API charges)",
+      // Must state `defaultProviderName`'s real order — the old text omitted
+      // the GEMINI-first branch and promised claude-first (field report
+      // 2026-08-07); the drift test in llm-help.test.ts pins the agreement.
+      // Order changed 2026-08: subscription CLIs beat ambient env keys
+      // (FINDINGS §132, antigravity provider).
+      "antigravity | claude | claude-cli | gemini | mock. Default: antigravity if the agy " +
+        "CLI is installed (your logged-in Google Antigravity, no API charges), else " +
+        "claude-cli if the claude CLI is installed (your logged-in Claude Code — Pro/Max " +
+        "subscription, no API charges), else gemini if GEMINI_API_KEY is set, else claude " +
+        "if ANTHROPIC_API_KEY is set, else claude-cli",
     )
     .option("--llm-model <id>", "override the provider's default model")
     .option(
@@ -333,7 +355,7 @@ export function buildProgram(): Command {
       if (envFiles.length > 0) console.log(`▸ env: ${envFiles.join(", ")}`);
       const cleanup = CleanupLevelSchema.parse(opts.cleanup);
       const provider = opts.llm
-        ? z.enum(["claude", "claude-cli", "gemini", "mock"]).parse(opts.llm)
+        ? z.enum(["antigravity", "claude", "claude-cli", "gemini", "mock"]).parse(opts.llm)
         : undefined;
       const forceComponent = opts.forceComponent
         ? SceneComponentIdSchema.parse(opts.forceComponent)
@@ -356,46 +378,86 @@ export function buildProgram(): Command {
         opts.whisperLanguage !== undefined
           ? z.string().trim().min(1, "--whisper-language needs a code, e.g. ur").parse(opts.whisperLanguage)
           : undefined;
-      const result = await produce(input, {
-        out: opts.out,
-        cleanup,
-        transcript: opts.transcript,
-        render: opts.render,
-        mezzanine: opts.mezzanine,
-        workdir: opts.workdir,
-        sort,
-        sortExplicit,
-        aspect: opts.aspect === "16:9" ? "16:9" : "9:16",
-        noiseDb: opts.noiseDb,
-        produce: opts.produce,
-        intent: opts.intent,
-        provider,
-        llmModel: opts.llmModel,
-        llmFastModel: opts.llmFastModel,
-        speaker: opts.speaker,
-        scenes: opts.scenes,
-        repair: opts.repair,
-        whisperModel: opts.whisperModel,
-        whisperLanguage,
-        forceComponent,
-        // commander gives `--no-cover` as cover:false and `--cover <path>` as a
-        // string on the same key.
-        sourceIsEdited: opts.sourceIsEdited === true,
-        blooperMarker: opts.blooperMarker,
-        collapseRetakes: opts.collapseRetakes,
-        sourceFit,
-        // undefined = "not typed", so produce can let the config decide.
-        watermark: opts.watermark,
-        // undefined = "not typed" here too — the default (ON) is applied at
-        // the pin site, not coerced in transit.
-        captions: opts.captions,
-        cover: opts.cover !== false,
-        coverPath: typeof opts.cover === "string" ? opts.cover : undefined,
-        clip: opts.clip,
-        clipWindow: opts.clipWindow,
-      });
-      const { offerEditor } = await import("./interactive/offer-editor");
-      await offerEditor(result, { flag: opts.openEditor, port: opts.editorPort });
+      // Wall clock around produce() only — the editor offer below can sit at
+      // an interactive prompt for as long as the user thinks, and think-time
+      // would poison the duration metric (FINDINGS §134).
+      const startedMs = Date.now();
+      try {
+        const result = await produce(input, {
+          out: opts.out,
+          cleanup,
+          transcript: opts.transcript,
+          render: opts.render,
+          mezzanine: opts.mezzanine,
+          workdir: opts.workdir,
+          sort,
+          sortExplicit,
+          aspect: opts.aspect === "16:9" ? "16:9" : "9:16",
+          noiseDb: opts.noiseDb,
+          produce: opts.produce,
+          intent: opts.intent,
+          provider,
+          llmModel: opts.llmModel,
+          llmFastModel: opts.llmFastModel,
+          speaker: opts.speaker,
+          scenes: opts.scenes,
+          repair: opts.repair,
+          whisperModel: opts.whisperModel,
+          whisperLanguage,
+          forceComponent,
+          // commander gives `--no-cover` as cover:false and `--cover <path>` as a
+          // string on the same key.
+          sourceIsEdited: opts.sourceIsEdited === true,
+          blooperMarker: opts.blooperMarker,
+          collapseRetakes: opts.collapseRetakes,
+          sourceFit,
+          // undefined = "not typed", so produce can let the config decide.
+          watermark: opts.watermark,
+          // undefined = "not typed" here too — the default (ON) is applied at
+          // the pin site, not coerced in transit.
+          captions: opts.captions,
+          cover: opts.cover !== false,
+          coverPath: typeof opts.cover === "string" ? opts.cover : undefined,
+          clip: opts.clip,
+          clipWindow: opts.clipWindow,
+        });
+        // Counts, buckets and names only — the duration crosses the wire as a
+        // bucket, and nothing here can carry a path (assertSafeProps enforces
+        // it). Inert while POSTHOG_KEY is the placeholder (FINDINGS §134).
+        telemetry.record("produce_completed", {
+          duration_ms: Date.now() - startedMs,
+          llm_provider: result.llmProvider ?? "none",
+          produced: opts.produce === true,
+          aspect: opts.aspect === "16:9" ? "16:9" : "9:16",
+          clip: opts.clip !== undefined,
+          render: opts.render !== false,
+          source_duration_bucket: durationBucket(result.sourceDurationSec),
+          scenes: result.sceneCount,
+        });
+        if (!telemetry.disabled) {
+          telemetry.state.produceCount += 1;
+          try {
+            saveState(telemetry.state);
+          } catch {
+            // A read-only home dir must never fail the produce that just
+            // succeeded — the rating gate simply advances a run later.
+          }
+        }
+        const { offerEditor } = await import("./interactive/offer-editor");
+        await offerEditor(result, { flag: opts.openEditor, port: opts.editorPort });
+        // Deliberately LAST — after the render summary and the editor offer —
+        // so the one question ossclip ever asks is the last thing on screen.
+        await maybeAskRating(telemetry);
+      } catch (err) {
+        // The constructor NAME only, never the message: error messages
+        // routinely quote the input path, the exact thing §134 forbids.
+        telemetry.record("produce_failed", {
+          error_class: err instanceof Error ? err.constructor.name : "NonError",
+        });
+        throw err;
+      } finally {
+        await telemetry.flush();
+      }
     });
 
   program
@@ -413,7 +475,7 @@ export function buildProgram(): Command {
     )
     .action(async (input: string, opts) => {
       const cleanup = CleanupLevelSchema.parse(opts.cleanup);
-      await produce(input, {
+      const result = await produce(input, {
         cleanup,
         transcript: opts.transcript,
         render: false,
@@ -427,6 +489,11 @@ export function buildProgram(): Command {
             ? z.string().trim().min(1, "--whisper-language needs a code, e.g. ur").parse(opts.whisperLanguage)
             : undefined,
       });
+      telemetry.record("transcribe_completed", {
+        cleanup_level: cleanup,
+        source_duration_bucket: durationBucket(result.sourceDurationSec),
+      });
+      await telemetry.flush();
     });
 
   program
@@ -526,6 +593,11 @@ export function buildProgram(): Command {
         const { openInBrowser } = await import("./open");
         openInBrowser(server.url);
       }
+      // Fire-and-forget on purpose: the server keeps the process alive for
+      // the request's lifetime, and awaiting here would put a metrics POST
+      // between the user and their browser opening (FINDINGS §134).
+      telemetry.record("editor_opened", {});
+      void telemetry.flush();
     });
 
   program
@@ -541,7 +613,13 @@ export function buildProgram(): Command {
     .action(async (opts) => {
       if (envFiles.length > 0) console.log(`▸ env: ${envFiles.join(", ")}`);
       const { setup } = await import("./setup/setup");
-      await setup({ model: opts.model, skipLlm: opts.skipLlm, force: opts.force, yes: opts.yes });
+      const summary = await setup({ model: opts.model, skipLlm: opts.skipLlm, force: opts.force, yes: opts.yes });
+      telemetry.record("setup_completed", {
+        steps_total: summary.stepsTotal,
+        steps_satisfied: summary.stepsSatisfied,
+        steps_failed: summary.stepsFailed,
+      });
+      await telemetry.flush();
     });
 
   program
@@ -556,8 +634,61 @@ export function buildProgram(): Command {
       const { loadConfig } = await import("@ossclip/core");
       const checks = await runDoctor(loadConfig(), realProbes(resolveEditorPageDir()));
       console.log(formatDoctor(checks));
+      const passedCount = checks.filter((c) => c.ok).length;
+      telemetry.record("doctor_run", {
+        checks_total: checks.length,
+        checks_passed: passedCount,
+        checks_failed: checks.length - passedCount,
+      });
+      await telemetry.flush();
       if (checks.some((c) => !c.ok)) process.exit(1);
     });
 
+  program
+    .command("telemetry")
+    .description("show or change anonymous usage telemetry — on | off | status")
+    .argument("[action]", "on | off | status (default: status)")
+    .action(async (action: string | undefined) => {
+      // Parsed, not coerced (§93a shape): `ossclip telemetry offf` must be a
+      // loud error naming the typo, never a silent status print.
+      const parsed = z.enum(["on", "off", "status"]).parse(action ?? "status");
+      // A fresh load, not the bootstrap instance's state: with the
+      // placeholder key bootstrap never reads the file, but an EXPLICIT
+      // on/off is the user asking for a persisted preference — it must land
+      // in ~/.ossclip/telemetry.json either way, ready for a keyed build.
+      const state = loadState();
+      if (parsed === "off") {
+        state.enabled = false;
+        saveState(state);
+        console.log("✓ telemetry off — nothing will be sent (saved in ~/.ossclip/telemetry.json)");
+        return;
+      }
+      if (parsed === "on") {
+        state.enabled = true;
+        saveState(state);
+        console.log(
+          "✓ telemetry on — anonymous usage events only; see the README's Telemetry section",
+        );
+        return;
+      }
+      const reason = telemetryOffReason(process.env, state);
+      if (reason === null) {
+        console.log("telemetry: enabled (anonymous usage events only)");
+      } else {
+        // Name the switch that WON, not just the state — three different
+        // offs (env, standard env, config) all look identical otherwise, and
+        // "why is it still off?" needs the answer.
+        const why: Record<typeof reason, string> = {
+          "placeholder-key":
+            "this build ships without a telemetry key — nothing is ever sent or stored",
+          env: `OSSCLIP_TELEMETRY=${process.env.OSSCLIP_TELEMETRY} in the environment`,
+          "do-not-track": `DO_NOT_TRACK=${process.env.DO_NOT_TRACK} in the environment`,
+          config: "`ossclip telemetry off` (saved in ~/.ossclip/telemetry.json)",
+        };
+        console.log(`telemetry: disabled — ${why[reason]}`);
+      }
+      console.log(`anonymous id: ${state.anonymousId}`);
+    });
+
   return program;
 }

package/src/setup/plan.ts

@@ -161,15 +161,19 @@ export async function planSetup(
     });
   }
 
-  // Provider, in doctor's detection order. Setup can save a key, but only
-  // ever interactively — never invented, never required (--skip-llm).
-  const provider = p.env.GEMINI_API_KEY
-    ? "gemini (GEMINI_API_KEY is set)"
-    : p.env.ANTHROPIC_API_KEY
-      ? "claude (ANTHROPIC_API_KEY is set)"
-      : (await p.binRuns("claude", "--version"))
-        ? "claude-cli (logged-in Claude Code)"
-        : null;
+  // Provider, in doctor's detection order (agy → claude CLI → gemini key →
+  // anthropic key; subscription CLIs beat ambient keys — FINDINGS §132,
+  // antigravity provider). Setup can save a key, but only ever
+  // interactively — never invented, never required (--skip-llm).
+  const provider = (await p.binRuns(p.env.OSSCLIP_AGY_BIN ?? "agy", "--version"))
+    ? "antigravity (agy CLI on PATH)"
+    : (await p.binRuns(p.env.OSSCLIP_CLAUDE_BIN ?? "claude", "--version"))
+      ? "claude-cli (logged-in Claude Code)"
+      : p.env.GEMINI_API_KEY
+        ? "gemini (GEMINI_API_KEY is set)"
+        : p.env.ANTHROPIC_API_KEY
+          ? "claude (ANTHROPIC_API_KEY is set)"
+          : null;
   if (opts.skipLlm) {
     steps.push({
       kind: "provider",

package/src/setup/provider.ts

@@ -23,12 +23,17 @@ export async function promptForProvider(io: ProviderIO, configDir: string): Prom
   io.say("  1) I have an Anthropic API key");
   io.say("  2) I have a Google Gemini API key");
   io.say("  3) I use Claude Code (already logged in — no key needed)");
+  io.say("  4) I use Google Antigravity (agy — already logged in, no key needed)");
   io.say("  Enter) skip for now");
   const choice = (await io.ask("Choice: ")).trim();
   if (choice === "3") {
     io.say("▸ nothing to save — ossclip finds the claude CLI on PATH by itself.");
     return;
   }
+  if (choice === "4") {
+    io.say("▸ nothing to save — ossclip finds the agy CLI on PATH by itself.");
+    return;
+  }
   const envKey = choice === "1" ? "ANTHROPIC_API_KEY" : choice === "2" ? "GEMINI_API_KEY" : null;
   if (!envKey) {
     io.say("▸ skipped — `ossclip doctor` will remind you what --produce needs.");

package/src/setup/setup.ts

@@ -38,7 +38,9 @@ const probeBin = (bin: string, arg: string): Promise<boolean> =>
     child.on("exit", () => resolve(true));
   });
 
-export async function setup(opts: SetupCliOptions): Promise<void> {
+export async function setup(
+  opts: SetupCliOptions,
+): Promise<{ stepsTotal: number; stepsSatisfied: number; stepsFailed: number }> {
   const cfg = loadConfig();
   const model = opts.model ?? cfg.model;
   const probes: SetupProbes = {
@@ -70,7 +72,7 @@ export async function setup(opts: SetupCliOptions): Promise<void> {
       const answer = (await rl.question("Proceed? [Y/n] ")).trim().toLowerCase();
       if (answer === "n" || answer === "no") {
         console.log("▸ nothing changed.");
-        return;
+        return { stepsTotal: steps.length, stepsSatisfied: 0, stepsFailed: 0 };
       }
     }
 
@@ -157,7 +159,8 @@ export async function setup(opts: SetupCliOptions): Promise<void> {
           } else if (step.status === "prompt") {
             console.log(
               "▸ no LLM provider configured (non-interactive run) — needed for --produce only; " +
-                "set ANTHROPIC_API_KEY or GEMINI_API_KEY when you want it.",
+                "a logged-in Claude Code or Google Antigravity (agy) CLI is picked up automatically, " +
+                "or set ANTHROPIC_API_KEY or GEMINI_API_KEY when you want it.",
             );
           }
           break;
@@ -179,6 +182,13 @@ export async function setup(opts: SetupCliOptions): Promise<void> {
   const checks = await runDoctor(loadConfig(), realProbes(resolveEditorPageDir()));
   console.log(formatDoctor(checks));
   if (failures.length > 0) process.exitCode = 1;
+  // The caller (program.ts) owns telemetry; this module stays network-free
+  // beyond its own downloads. It just reports what happened.
+  return {
+    stepsTotal: steps.length,
+    stepsSatisfied: steps.filter((s) => s.status === "satisfied").length,
+    stepsFailed: failures.length,
+  };
 }
 
 /** Download an asset (with resume) and extract it under the managed bin dir. */