caveat-cli 0.19.10 → 0.19.11

package/README.md

@@ -66,6 +66,9 @@ output also exposes `connectors.cursor.compatibility_status` for diagnosis,
 but callers do not duplicate Caveat's Cursor event set, command rules, or
 timeout checks. Without `--require-connector cursor`, the v1 aggregate keeps
 its existing Claude/Codex readiness semantics.
+For Grok, use `caveat factory-diagnostics --json --require-connector grok` to
+check the enabled Caveat MCP registration and executable paths. A live MCP and
+model smoke remain separate checks.
 
 With either host enabled, Caveat surfaces matching entries at three moments:
 before prompts, after failed tools, and after struggle-heavy sessions. Stop
@@ -87,6 +90,8 @@ caveat uninstall                    # reverse `caveat init` Claude integration
 caveat codex-hook diagnostics       # inspect Codex hook availability/install state
 caveat factory-diagnostics --json --require-connector cursor
                                     # machine gate for a host that requires Cursor
+caveat factory-diagnostics --json --require-connector grok
+                                    # machine gate for a host that requires Grok MCP
 ```
 
 ## Sharing: two boundaries, two commands

package/dist/index.js

@@ -458,11 +458,11 @@ var require_codegen = __commonJS({
         const rhs = this.rhs === void 0 ? "" : ` = ${this.rhs}`;
         return `${varKind} ${this.name}${rhs};` + _n;
       }
-      optimizeNames(names, constants4) {
+      optimizeNames(names, constants5) {
         if (!names[this.name.str])
           return;
         if (this.rhs)
-          this.rhs = optimizeExpr(this.rhs, names, constants4);
+          this.rhs = optimizeExpr(this.rhs, names, constants5);
         return this;
       }
       get names() {
@@ -479,10 +479,10 @@ var require_codegen = __commonJS({
       render({ _n }) {
         return `${this.lhs} = ${this.rhs};` + _n;
       }
-      optimizeNames(names, constants4) {
+      optimizeNames(names, constants5) {
         if (this.lhs instanceof code_1.Name && !names[this.lhs.str] && !this.sideEffects)
           return;
-        this.rhs = optimizeExpr(this.rhs, names, constants4);
+        this.rhs = optimizeExpr(this.rhs, names, constants5);
         return this;
       }
       get names() {
@@ -543,8 +543,8 @@ var require_codegen = __commonJS({
       optimizeNodes() {
         return `${this.code}` ? this : void 0;
       }
-      optimizeNames(names, constants4) {
-        this.code = optimizeExpr(this.code, names, constants4);
+      optimizeNames(names, constants5) {
+        this.code = optimizeExpr(this.code, names, constants5);
         return this;
       }
       get names() {
@@ -573,12 +573,12 @@ var require_codegen = __commonJS({
         }
         return nodes.length > 0 ? this : void 0;
       }
-      optimizeNames(names, constants4) {
+      optimizeNames(names, constants5) {
         const { nodes } = this;
         let i = nodes.length;
         while (i--) {
           const n = nodes[i];
-          if (n.optimizeNames(names, constants4))
+          if (n.optimizeNames(names, constants5))
             continue;
           subtractNames(names, n.names);
           nodes.splice(i, 1);
@@ -631,12 +631,12 @@ var require_codegen = __commonJS({
           return void 0;
         return this;
       }
-      optimizeNames(names, constants4) {
+      optimizeNames(names, constants5) {
         var _a3;
-        this.else = (_a3 = this.else) === null || _a3 === void 0 ? void 0 : _a3.optimizeNames(names, constants4);
-        if (!(super.optimizeNames(names, constants4) || this.else))
+        this.else = (_a3 = this.else) === null || _a3 === void 0 ? void 0 : _a3.optimizeNames(names, constants5);
+        if (!(super.optimizeNames(names, constants5) || this.else))
           return;
-        this.condition = optimizeExpr(this.condition, names, constants4);
+        this.condition = optimizeExpr(this.condition, names, constants5);
         return this;
       }
       get names() {
@@ -659,10 +659,10 @@ var require_codegen = __commonJS({
       render(opts) {
         return `for(${this.iteration})` + super.render(opts);
       }
-      optimizeNames(names, constants4) {
-        if (!super.optimizeNames(names, constants4))
+      optimizeNames(names, constants5) {
+        if (!super.optimizeNames(names, constants5))
           return;
-        this.iteration = optimizeExpr(this.iteration, names, constants4);
+        this.iteration = optimizeExpr(this.iteration, names, constants5);
         return this;
       }
       get names() {
@@ -698,10 +698,10 @@ var require_codegen = __commonJS({
       render(opts) {
         return `for(${this.varKind} ${this.name} ${this.loop} ${this.iterable})` + super.render(opts);
       }
-      optimizeNames(names, constants4) {
-        if (!super.optimizeNames(names, constants4))
+      optimizeNames(names, constants5) {
+        if (!super.optimizeNames(names, constants5))
           return;
-        this.iterable = optimizeExpr(this.iterable, names, constants4);
+        this.iterable = optimizeExpr(this.iterable, names, constants5);
         return this;
       }
       get names() {
@@ -743,11 +743,11 @@ var require_codegen = __commonJS({
         (_b = this.finally) === null || _b === void 0 ? void 0 : _b.optimizeNodes();
         return this;
       }
-      optimizeNames(names, constants4) {
+      optimizeNames(names, constants5) {
         var _a3, _b;
-        super.optimizeNames(names, constants4);
-        (_a3 = this.catch) === null || _a3 === void 0 ? void 0 : _a3.optimizeNames(names, constants4);
-        (_b = this.finally) === null || _b === void 0 ? void 0 : _b.optimizeNames(names, constants4);
+        super.optimizeNames(names, constants5);
+        (_a3 = this.catch) === null || _a3 === void 0 ? void 0 : _a3.optimizeNames(names, constants5);
+        (_b = this.finally) === null || _b === void 0 ? void 0 : _b.optimizeNames(names, constants5);
         return this;
       }
       get names() {
@@ -1048,7 +1048,7 @@ var require_codegen = __commonJS({
     function addExprNames(names, from) {
       return from instanceof code_1._CodeOrName ? addNames(names, from.names) : names;
     }
-    function optimizeExpr(expr, names, constants4) {
+    function optimizeExpr(expr, names, constants5) {
       if (expr instanceof code_1.Name)
         return replaceName(expr);
       if (!canOptimize(expr))
@@ -1063,14 +1063,14 @@ var require_codegen = __commonJS({
         return items;
       }, []));
       function replaceName(n) {
-        const c = constants4[n.str];
+        const c = constants5[n.str];
         if (c === void 0 || names[n.str] !== 1)
           return n;
         delete names[n.str];
         return c;
       }
       function canOptimize(e) {
-        return e instanceof code_1._Code && e._items.some((c) => c instanceof code_1.Name && names[c.str] === 1 && constants4[c.str] !== void 0);
+        return e instanceof code_1._Code && e._items.some((c) => c instanceof code_1.Name && names[c.str] === 1 && constants5[c.str] !== void 0);
       }
     }
     function subtractNames(names, from) {
@@ -6896,7 +6896,7 @@ import { existsSync as existsSync7, mkdirSync as mkdirSync3, readdirSync, rename
 import { dirname as dirname3, join as join8 } from "node:path";
 
 // src/claudeInstall.ts
-import { constants, existsSync as existsSync3, readFileSync as readFileSync3 } from "node:fs";
+import { constants as constants2, existsSync as existsSync3, readFileSync as readFileSync3 } from "node:fs";
 import { join as join4 } from "node:path";
 
 // src/installShared.ts
@@ -6957,9 +6957,13 @@ function claudeMcpConfigPath(claudeDir, env = process.env) {
 }
 
 // src/mcpInstall.ts
-import { existsSync as existsSync2, readFileSync as readFileSync2 } from "node:fs";
+import { constants, existsSync as existsSync2, readFileSync as readFileSync2 } from "node:fs";
 import { isDeepStrictEqual } from "node:util";
 import { parse, stringify } from "smol-toml";
+function isCaveatStdioMcpRegistration(value, nodePath, cliScriptPath) {
+  if (!object(value)) return false;
+  return isCanonicalAsset(value.command, nodePath, constants.X_OK) && Array.isArray(value.args) && value.args.length === 3 && value.args[0] === "--disable-warning=ExperimentalWarning" && isCanonicalAsset(value.args[1], cliScriptPath, constants.R_OK) && value.args[2] === "mcp-server";
+}
 function object(value) {
   return value !== null && typeof value === "object" && !Array.isArray(value);
 }
@@ -7063,12 +7067,12 @@ function isCanonicalCaveatClaudeHookCommand(actual, event, nodePath, cliScriptPa
   const [actualNodePath, actualCliScriptPath, subcommand, actualEvent] = tokens ?? [];
   if (tokens?.length !== 4 || actualNodePath === void 0 || actualCliScriptPath === void 0 || subcommand !== "hook" || actualEvent !== event) return false;
   if (actual !== hookCommand(actualNodePath, actualCliScriptPath, event)) return false;
-  return isCanonicalAsset(actualNodePath, nodePath, constants.X_OK) && isCanonicalAsset(actualCliScriptPath, cliScriptPath, constants.R_OK);
+  return isCanonicalAsset(actualNodePath, nodePath, constants2.X_OK) && isCanonicalAsset(actualCliScriptPath, cliScriptPath, constants2.R_OK);
 }
 function isCaveatClaudeMcpRegistration(value, nodePath, cliScriptPath) {
   if (!value || typeof value !== "object" || Array.isArray(value)) return false;
   const server = value;
-  return ["args", "command", "env", "type"].every((key) => Object.hasOwn(server, key)) && server.type === "stdio" && isCanonicalAsset(server.command, nodePath, constants.X_OK) && Array.isArray(server.args) && server.args.length === 3 && server.args[0] === "--disable-warning=ExperimentalWarning" && isCanonicalAsset(server.args[1], cliScriptPath, constants.R_OK) && server.args[2] === "mcp-server" && server.env !== null && typeof server.env === "object" && !Array.isArray(server.env) && Object.values(server.env).every((value2) => typeof value2 === "string");
+  return ["args", "command", "env", "type"].every((key) => Object.hasOwn(server, key)) && server.type === "stdio" && isCaveatStdioMcpRegistration(server, nodePath, cliScriptPath) && server.env !== null && typeof server.env === "object" && !Array.isArray(server.env) && Object.values(server.env).every((value2) => typeof value2 === "string");
 }
 function readSettings(path) {
   if (!existsSync3(path)) return {};
@@ -7150,7 +7154,7 @@ function uninstallClaudeIntegration(opts) {
 }
 
 // src/codexHookInstall.ts
-import { constants as constants2, existsSync as existsSync4, readFileSync as readFileSync4 } from "node:fs";
+import { constants as constants3, existsSync as existsSync4, readFileSync as readFileSync4 } from "node:fs";
 import { join as join5 } from "node:path";
 import { parse as parseToml } from "smol-toml";
 function hookCommand2(nodePath, cliScriptPath, event, platform = process.platform) {
@@ -7175,7 +7179,7 @@ function isCaveatCodexHookCommand(actual, event) {
 function isCanonicalCaveatCodexHookCommand(actual, event, nodePath, cliScriptPath) {
   const parsed = commandTokens(actual);
   const tokens = parsed?.[0] === "&" ? parsed.slice(1) : parsed;
-  return tokens?.length === 4 && isCanonicalAsset(tokens[0], nodePath, constants2.X_OK) && isCanonicalAsset(tokens[1], cliScriptPath, constants2.R_OK) && tokens[2] === "codex-hook" && tokens[3] === event;
+  return tokens?.length === 4 && isCanonicalAsset(tokens[0], nodePath, constants3.X_OK) && isCanonicalAsset(tokens[1], cliScriptPath, constants3.R_OK) && tokens[2] === "codex-hook" && tokens[3] === event;
 }
 function isCanonicalCaveatCodexHookEntry(value, event, nodePath, cliScriptPath) {
   if (!isPlainRecord(value) || typeof value.type !== "string" || typeof value.command !== "string") return false;
@@ -7534,7 +7538,7 @@ function detectCodexHookInstallation(codexHome) {
 // src/cursorInstall.ts
 import { existsSync as existsSync5, mkdirSync as mkdirSync2, readFileSync as readFileSync5 } from "node:fs";
 import { join as join6 } from "node:path";
-import { constants as constants3 } from "node:fs";
+import { constants as constants4 } from "node:fs";
 var TIMEOUT_SEC = 10;
 function hookCommand3(nodePath, cliScriptPath, event) {
   return `${quoteCommandPath(nodePath)} ${quoteCommandPath(cliScriptPath)} cursor-hook ${event}`;
@@ -7548,7 +7552,7 @@ function isCaveatCursorHookCommand(actual, event) {
 }
 function isCanonicalCaveatCursorHookCommand(actual, event, nodePath, cliScriptPath) {
   const tokens = commandTokens(actual);
-  return tokens?.length === 4 && isCanonicalAsset(tokens[0], nodePath, constants3.X_OK) && isCanonicalAsset(tokens[1], cliScriptPath, constants3.R_OK) && tokens[2] === "cursor-hook" && tokens[3] === event;
+  return tokens?.length === 4 && isCanonicalAsset(tokens[0], nodePath, constants4.X_OK) && isCanonicalAsset(tokens[1], cliScriptPath, constants4.R_OK) && tokens[2] === "cursor-hook" && tokens[3] === event;
 }
 function isCanonicalCaveatCursorHookEntry(value, event, nodePath, cliScriptPath) {
   if (value === null || typeof value !== "object" || Array.isArray(value)) return false;
@@ -34593,6 +34597,20 @@ function claudeRegistration(path, nodePath, cliScriptPath) {
     return unverified("config_unreadable");
   }
 }
+function grokRegistration(path, nodePath, cliScriptPath) {
+  if (!existsSync18(path)) return status(false, "not_registered");
+  try {
+    const config2 = parseToml2(readFileSync12(path, "utf8"));
+    if (!isRecord2(config2) || !isRecord2(config2.mcp_servers)) return status(false, "not_registered");
+    const server = config2.mcp_servers.caveat;
+    if (!isRecord2(server)) return status(false, "not_registered");
+    if (server.enabled === false) return status(false, "disabled");
+    if (server.enabled !== void 0 && server.enabled !== true) return unverified("config_invalid");
+    return status(isCaveatStdioMcpRegistration(server, nodePath, cliScriptPath), "not_registered");
+  } catch {
+    return unverified("config_unreadable");
+  }
+}
 function claudeHooks(claudeDir, nodePath, cliScriptPath) {
   try {
     const settings = JSON.parse(readFileSync12(join18(claudeDir, "settings.json"), "utf8"));
@@ -34690,7 +34708,7 @@ function sync(own) {
   }
 }
 function factoryDiagnostics(ctx, codexHome = resolveAgentConfigPaths(ctx.userHome).codexHome, requiredConnectors = []) {
-  const { claudeDir, claudeMcp, cursorDir } = resolveAgentConfigPaths(ctx.userHome);
+  const { claudeDir, claudeMcp, cursorDir, grokHome } = resolveAgentConfigPaths(ctx.userHome);
   const nodePath = process.execPath;
   const cliScriptPath = process.argv[1] ?? "";
   const db = database(ctx.paths.dbPath);
@@ -34701,13 +34719,15 @@ function factoryDiagnostics(ctx, codexHome = resolveAgentConfigPaths(ctx.userHom
   } catch {
   }
   const codexHook2 = (present) => present === void 0 ? unverified("config_unreadable") : !present ? hook(false) : feature.status === "ready" ? hook(true) : { ...feature };
-  const output = { schema: "caveat.native_factory_diagnostics.v1", product: "caveat", version: CAVEAT_VERSION, overall: { status: "unverified" }, database: db, sync: sync(ctx.paths.knowledgeRepo), connectors: { claude: { status: "unverified", mcp: claudeRegistration(claudeMcp, nodePath, cliScriptPath), hooks: claudeHooks(claudeDir, nodePath, cliScriptPath) }, codex: { status: "unverified", hooks: { user_prompt_submit: codexHook2(installedCodexHooks?.userPromptSubmit), post_tool_use: codexHook2(installedCodexHooks?.postToolUse), stop: codexHook2(installedCodexHooks?.stop) } }, cursor: diagnoseCursorHookConnector(cursorDir, nodePath, cliScriptPath) } };
+  const grokMcp = grokRegistration(join18(grokHome, "config.toml"), nodePath, cliScriptPath);
+  const output = { schema: "caveat.native_factory_diagnostics.v1", product: "caveat", version: CAVEAT_VERSION, overall: { status: "unverified" }, database: db, sync: sync(ctx.paths.knowledgeRepo), connectors: { claude: { status: "unverified", mcp: claudeRegistration(claudeMcp, nodePath, cliScriptPath), hooks: claudeHooks(claudeDir, nodePath, cliScriptPath) }, codex: { status: "unverified", hooks: { user_prompt_submit: codexHook2(installedCodexHooks?.userPromptSubmit), post_tool_use: codexHook2(installedCodexHooks?.postToolUse), stop: codexHook2(installedCodexHooks?.stop) } }, cursor: diagnoseCursorHookConnector(cursorDir, nodePath, cliScriptPath), grok: { status: grokMcp.status, mcp: grokMcp } } };
   const aggregate = (all) => all.includes("not_ready") ? "not_ready" : all.includes("unverified") ? "unverified" : "ready";
   const connectorStatus = (connector) => aggregate([...Object.values(connector.hooks), ...connector.mcp ? [connector.mcp] : []].map((v) => v.status));
   output.connectors.claude.status = connectorStatus(output.connectors.claude);
   output.connectors.codex.status = connectorStatus(output.connectors.codex);
   const statuses = [output.database.status, output.sync.status, output.connectors.claude.status, output.connectors.codex.status];
   if (requiredConnectors.includes("cursor")) statuses.push(output.connectors.cursor.compatibility_status);
+  if (requiredConnectors.includes("grok")) statuses.push(output.connectors.grok.status);
   output.overall.status = aggregate(statuses);
   return output;
 }
@@ -34773,7 +34793,7 @@ function collectRepeatable(value, previous) {
 }
 program.name("caveat").description("External spec gotcha knowledge base CLI").version(CAVEAT_VERSION);
 program.command("factory-diagnostics").description("Emit read-only factory diagnostics").requiredOption("--json", "emit the versioned JSON contract").option("--require-connector <name>", "include a connector in overall readiness (repeatable)", collectRepeatable, []).action((opts) => {
-  const unsupported = opts.requireConnector.filter((name) => name !== "cursor");
+  const unsupported = opts.requireConnector.filter((name) => name !== "cursor" && name !== "grok");
   if (unsupported.length > 0) throw new Error(`unsupported factory connector: ${unsupported.join(", ")}`);
   const requiredConnectors = [...new Set(opts.requireConnector)];
   const output = factoryDiagnostics(buildContext(stdoutLogger), void 0, requiredConnectors);