@caddis/cli 0.2.2 → 0.2.4

package/bundles/antigravity-plugin/warm_start_agy.py

@@ -2,7 +2,10 @@
 """caddis warm-start hook for agy (Antigravity) — PreInvocation event.
 
 Ports the relay half of claude-harness/hooks/inject_relay.py. agy has NO SessionStart event, so the
-nearest equivalent is PreInvocation with `invocationNum == 1` — the first model turn of a session.
+nearest equivalent is PreInvocation with `invocationNum == 0` — the first model turn of a session.
+**agy's invocations are 0-indexed** (confirmed via a live-fire session against the real `agy` binary
+2026-07-30 -- the original code checked `== 1`, which is agy's SECOND turn, so this hook silently
+never fired at all until the live check caught it; see .caddis/plans/agy-hooks-port.md step 5).
 On that turn only, the workspace's relay doc is injected as an ephemeral step so a fresh agy session
 resumes with zero re-discovery. Every later invocation emits nothing (re-injecting the relay on every
 turn would burn context and drown the conversation).
@@ -118,10 +121,10 @@ def main() -> None:
             return
         if not isinstance(data, dict):
             return
-        # SessionStart equivalent: the FIRST invocation only. Anything else (or an unparseable
-        # invocationNum) injects nothing.
+        # SessionStart equivalent: the FIRST invocation only (agy 0-indexes invocationNum, so the
+        # first turn is 0, not 1). Anything else (or an unparseable invocationNum) injects nothing.
         try:
-            if int(data.get("invocationNum")) != 1:
+            if int(data.get("invocationNum")) != 0:
                 return
         except Exception:
             return

package/bundles/antigravity-plugin-extras/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "caddis-extras",
-  "version": "1.3.14",
+  "version": "1.3.20",
   "description": "caddis-extras — the long-tail skills for the Antigravity (agy) caddis plugin; install alongside caddis for the full set."
 }

package/bundles/antigravity-plugin-extras/skills/cross-review/SKILL.md

@@ -28,10 +28,13 @@ churn — the preset table in `oss_review.py` is the one place to edit a rename,
 `REVIEW_BASE_URL` always win, so you can point at any new id without touching code.
 
 ## How to run
-From the repo root (the tool ships at `.github/tools/oss_review.py`):
+From the repo root. Locate the tool, in order:
+- `.github/tools/oss_review.py` — this project vendors its own copy.
+- `${CLAUDE_PLUGIN_ROOT}/scripts/oss_review.py` — the plugin-bundled copy (every Claude Code
+  install has this; use it whenever the project has no local copy).
 ```
-python .github/tools/oss_review.py                       # review the working tree (staged+unstaged)
-python .github/tools/oss_review.py --range origin/main..HEAD   # review a branch's commits
+python <resolved-path>/oss_review.py                       # review the working tree (staged+unstaged)
+python <resolved-path>/oss_review.py --range origin/main..HEAD   # review a branch's commits
 ```
 Optional flags: `--cwd <repo>`, `--base-url <url>`, `--model <id>` (override the env).
 

package/bundles/manifest.json

@@ -1,7 +1,7 @@
 {
-  "poolVersion": "1.3.41",
+  "poolVersion": "1.3.54",
   "bundles": {
-    "antigravity-plugin": "1.3.41",
-    "antigravity-plugin-extras": "1.3.14"
+    "antigravity-plugin": "1.3.54",
+    "antigravity-plugin-extras": "1.3.20"
   }
 }

package/dist/cli.js

@@ -3473,6 +3473,51 @@ var require_commander = __commonJS({
   }
 });
 
+// src/util/exec.ts
+async function run(cmd, args, opts = {}) {
+  const { execa } = await import("execa");
+  try {
+    const result = await execa(cmd, args, {
+      cwd: opts.cwd,
+      timeout: opts.timeout ?? DEFAULT_TIMEOUT_MS,
+      env: opts.env,
+      reject: false,
+      all: false,
+      stripFinalNewline: true,
+      // Windows: resolve .cmd/.ps1 shims through the shell-less path lookup
+      // execa already does, but be explicit that we are not using a shell.
+      shell: false,
+      windowsHide: true
+    });
+    return {
+      ok: result.exitCode === 0 && !result.failed,
+      code: result.exitCode ?? null,
+      stdout: String(result.stdout ?? ""),
+      stderr: String(result.stderr ?? ""),
+      failure: result.failed && result.exitCode === void 0 ? result.shortMessage : void 0
+    };
+  } catch (error) {
+    return {
+      ok: false,
+      code: null,
+      stdout: "",
+      stderr: "",
+      failure: error instanceof Error ? error.message : String(error)
+    };
+  }
+}
+function formatCommand(cmd, args) {
+  const quoted = args.map((a) => /\s/.test(a) ? `"${a}"` : a);
+  return [cmd, ...quoted].join(" ");
+}
+var DEFAULT_TIMEOUT_MS;
+var init_exec = __esm({
+  "src/util/exec.ts"() {
+    "use strict";
+    DEFAULT_TIMEOUT_MS = 12e4;
+  }
+});
+
 // node_modules/isexe/dist/commonjs/index.min.js
 var require_index_min = __commonJS({
   "node_modules/isexe/dist/commonjs/index.min.js"(exports) {
@@ -4483,6 +4528,13 @@ var init_drive = __esm({
 });
 
 // src/commands/report.ts
+async function checkCliUpdate(currentVersion) {
+  const result = await run("npm", ["view", "@caddis/cli", "version"], { timeout: 5e3 });
+  if (!result.ok) return null;
+  const latest = result.stdout.trim();
+  if (!latest || latest === currentVersion) return null;
+  return { current: currentVersion, latest };
+}
 function classify(adapter, detection, status4, poolVersion) {
   if (!detection.present) return "absent";
   if (!adapter.supported) return "unsupported";
@@ -4491,24 +4543,28 @@ function classify(adapter, detection, status4, poolVersion) {
   if (poolVersion === "unknown") return "unknown";
   return status4.version === poolVersion ? "current" : "stale";
 }
-async function gather(adapters) {
+async function gather(adapters, options = {}) {
   const manifest = bundleManifest();
   const poolVersion = manifest.poolVersion;
   const extrasVersion = manifest.bundles["antigravity-plugin-extras"];
-  const agents = await Promise.all(
-    adapters.map(async (adapter) => {
-      const detection = await adapter.detect();
-      const status4 = detection.present ? await adapter.status() : { installed: false, note: "agent not installed" };
-      return {
-        adapter,
-        detection,
-        status: status4,
-        drift: classify(adapter, detection, status4, poolVersion),
-        extrasDrift: classifyExtras(adapter, detection, status4, extrasVersion)
-      };
-    })
-  );
-  return { cliVersion: packageInfo().version, poolVersion, extrasVersion, agents };
+  const cliVersion = packageInfo().version;
+  const [agents, cliUpdate] = await Promise.all([
+    Promise.all(
+      adapters.map(async (adapter) => {
+        const detection = await adapter.detect();
+        const status4 = detection.present ? await adapter.status() : { installed: false, note: "agent not installed" };
+        return {
+          adapter,
+          detection,
+          status: status4,
+          drift: classify(adapter, detection, status4, poolVersion),
+          extrasDrift: classifyExtras(adapter, detection, status4, extrasVersion)
+        };
+      })
+    ),
+    options.checkCliUpdate ? checkCliUpdate(cliVersion) : Promise.resolve(null)
+  ]);
+  return { cliVersion, poolVersion, extrasVersion, agents, cliUpdate: cliUpdate ?? void 0 };
 }
 function classifyExtras(adapter, detection, status4, extrasVersion) {
   if (!status4.extras) return void 0;
@@ -4528,6 +4584,7 @@ function actionable(report, includeCurrent = false) {
 var init_report = __esm({
   "src/commands/report.ts"() {
     "use strict";
+    init_exec();
     init_pkg();
   }
 });
@@ -4646,7 +4703,7 @@ __export(doctor_exports, {
 import os3 from "os";
 import process2 from "process";
 async function doctor(options) {
-  const report = await gather(options.adapters);
+  const report = await gather(options.adapters, { checkCliUpdate: true });
   const findings = [];
   if (options.json) {
     line(JSON.stringify(toJson(report), null, 2));
@@ -4667,7 +4724,17 @@ function renderEnvironment(report, problems) {
     problems.push({ kind: "problem", text: `node ${process2.versions.node} is below the required ${REQUIRED_NODE_MAJOR}.${REQUIRED_NODE_MINOR}`, fix: "upgrade node" });
   }
   item("info", `platform ${process2.platform} ${process2.arch} (${os3.release()})`);
-  item("info", `@caddis/cli ${report.cliVersion}`);
+  if (report.cliUpdate) {
+    item("warn", `@caddis/cli ${report.cliVersion} ${color.dim("\u2192")} ${color.bold(report.cliUpdate.latest)} available`);
+    detail("everything below is compared against the pool THIS install carries, which is now behind the published one");
+    problems.push({
+      kind: "problem",
+      text: `@caddis/cli itself is behind: ${report.cliUpdate.current} installed, ${report.cliUpdate.latest} published`,
+      fix: "npm i -g @caddis/cli@latest"
+    });
+  } else {
+    item("ok", `@caddis/cli ${report.cliVersion}`);
+  }
   const manifest = bundleManifest();
   if (report.poolVersion === "unknown") {
     item("fail", "shipped pool version unknown");
@@ -4837,6 +4904,7 @@ function problemFor(entry, poolVersion) {
 function toJson(report) {
   return {
     cliVersion: report.cliVersion,
+    cliUpdate: report.cliUpdate ?? null,
     poolVersion: report.poolVersion,
     extrasVersion: report.extrasVersion ?? null,
     node: process2.versions.node,
@@ -4990,44 +5058,8 @@ var {
   Help
 } = import_index.default;
 
-// src/util/exec.ts
-var DEFAULT_TIMEOUT_MS = 12e4;
-async function run(cmd, args, opts = {}) {
-  const { execa } = await import("execa");
-  try {
-    const result = await execa(cmd, args, {
-      cwd: opts.cwd,
-      timeout: opts.timeout ?? DEFAULT_TIMEOUT_MS,
-      env: opts.env,
-      reject: false,
-      all: false,
-      stripFinalNewline: true,
-      // Windows: resolve .cmd/.ps1 shims through the shell-less path lookup
-      // execa already does, but be explicit that we are not using a shell.
-      shell: false,
-      windowsHide: true
-    });
-    return {
-      ok: result.exitCode === 0 && !result.failed,
-      code: result.exitCode ?? null,
-      stdout: String(result.stdout ?? ""),
-      stderr: String(result.stderr ?? ""),
-      failure: result.failed && result.exitCode === void 0 ? result.shortMessage : void 0
-    };
-  } catch (error) {
-    return {
-      ok: false,
-      code: null,
-      stdout: "",
-      stderr: "",
-      failure: error instanceof Error ? error.message : String(error)
-    };
-  }
-}
-function formatCommand(cmd, args) {
-  const quoted = args.map((a) => /\s/.test(a) ? `"${a}"` : a);
-  return [cmd, ...quoted].join(" ");
-}
+// src/agents/claude.ts
+init_exec();
 
 // src/util/which.ts
 var import_which = __toESM(require_lib(), 1);
@@ -5167,7 +5199,8 @@ var claudeAdapter = {
 };
 
 // src/agents/agy.ts
-import { readFileSync as readFileSync2, existsSync as existsSync2 } from "fs";
+init_exec();
+import { readFileSync as readFileSync2, existsSync as existsSync2, mkdtempSync, cpSync, rmSync } from "fs";
 import os from "os";
 import path2 from "path";
 init_pkg();
@@ -5239,6 +5272,25 @@ function shortenHome(target) {
   const home = os.homedir();
   return target.startsWith(home) ? path2.join("~", target.slice(home.length)) : target;
 }
+function stageForAgy(bundleDir) {
+  if (!bundleDir.includes("@")) {
+    return { installPath: bundleDir, cleanup: () => {
+    } };
+  }
+  const staged = mkdtempSync(path2.join(os.tmpdir(), "caddis-agy-"));
+  const dest = path2.join(staged, path2.basename(bundleDir));
+  cpSync(bundleDir, dest, { recursive: true });
+  return {
+    installPath: dest,
+    // Best-effort: a stray temp dir left behind is harmless clutter, not a failure.
+    cleanup: () => {
+      try {
+        rmSync(staged, { recursive: true, force: true });
+      } catch {
+      }
+    }
+  };
+}
 async function drive2(_action, options) {
   const detected = await detect2();
   if (!detected.present) {
@@ -5253,12 +5305,12 @@ async function drive2(_action, options) {
       message: `the ${BUNDLE} bundle is missing from this package \u2014 reinstall @caddis/cli`
     };
   }
-  const planned = [{ cmd: BIN2, args: ["plugin", "install", bundle] }];
+  const planned = [{ cmd: BIN2, args: ["plugin", "install", bundle], bundleDir: bundle }];
   const extrasInstalled = extrasStatusFromHome(os.homedir()).installed;
   if (options.extras === true || extrasInstalled) {
     const extrasBundle = bundlePath(EXTRAS_BUNDLE);
     if (extrasBundle) {
-      planned.push({ cmd: BIN2, args: ["plugin", "install", extrasBundle] });
+      planned.push({ cmd: BIN2, args: ["plugin", "install", extrasBundle], bundleDir: extrasBundle });
     } else if (options.extras === true) {
       return {
         ok: false,
@@ -5272,25 +5324,34 @@ async function drive2(_action, options) {
     return {
       ok: true,
       skipped: true,
+      // Show the logical bundle path, not a staged temp copy that was never created.
       steps: planned.map((s) => ({ command: formatCommand(s.cmd, s.args), ok: true, code: null })),
       message: "dry run \u2014 nothing executed"
     };
   }
-  const steps2 = [];
-  for (const step of planned) {
-    const command = formatCommand(step.cmd, step.args);
-    const result = await run(step.cmd, step.args);
-    steps2.push({
-      command,
-      ok: result.ok,
-      code: result.code,
-      output: result.ok ? void 0 : (result.stderr || result.stdout || result.failure || "").trim().split(/\r?\n/).slice(-3).join("\n")
-    });
-    if (!result.ok) {
-      return { ok: false, skipped: false, steps: steps2, message: `\`${command}\` failed` };
+  const cleanups = [];
+  try {
+    const steps2 = [];
+    for (const step of planned) {
+      const command = formatCommand(step.cmd, step.args);
+      const staged = stageForAgy(step.bundleDir);
+      cleanups.push(staged.cleanup);
+      const execArgs = [...step.args.slice(0, -1), staged.installPath];
+      const result = await run(step.cmd, execArgs);
+      steps2.push({
+        command,
+        ok: result.ok,
+        code: result.code,
+        output: result.ok ? void 0 : (result.stderr || result.stdout || result.failure || "").trim().split(/\r?\n/).slice(-3).join("\n")
+      });
+      if (!result.ok) {
+        return { ok: false, skipped: false, steps: steps2, message: `\`${command}\` failed` };
+      }
     }
+    return { ok: true, skipped: false, steps: steps2 };
+  } finally {
+    cleanups.forEach((fn) => fn());
   }
-  return { ok: true, skipped: false, steps: steps2 };
 }
 var agyAdapter = {
   id: "agy",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@caddis/cli",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "description": "Install, update and diagnose the caddis dev harness across every coding agent you use (Claude Code, agy/Antigravity, and more).",
   "keywords": [
     "caddis",