@braingrid/cli 0.2.41 → 0.2.43

package/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.43] - 2026-02-17
+
+### Fixed
+
+- **grep -c newline bug in verify-acceptance-criteria.sh** — `grep -c` exits non-zero on 0 matches, causing `|| echo "0"` to produce `"0\n0"` artifacts that break arithmetic expansion; replaced with `|| true` and `${VAR:-0}` default
+
+## [0.2.42] - 2026-02-17
+
+### Added
+
+- **SessionStart and TaskCompleted hook installation** — `braingrid setup claude-code` now installs two additional hook scripts: `check-stale-build-sentinel.sh` (SessionStart) for cleaning up stale build sentinels, and `task-completed-validate.sh` (TaskCompleted) for commit validation before marking tasks complete
+- **Sync script updated** — `sync-claude-code-to-braingrid.sh` now syncs 16 files (up from 14), including the two new hook scripts
+
 ## [0.2.41] - 2026-02-17
 
 ### Fixed

package/dist/cli.js

@@ -222,7 +222,7 @@ async function axiosWithRetry(config2, options) {
 
 // src/build-config.ts
 var BUILD_ENV = true ? "production" : process.env.NODE_ENV === "test" ? "development" : "production";
-var CLI_VERSION = true ? "0.2.41" : "0.0.0-test";
+var CLI_VERSION = true ? "0.2.43" : "0.0.0-test";
 var PRODUCTION_CONFIG = {
   apiUrl: "https://app.braingrid.ai",
   workosAuthUrl: "https://auth.braingrid.ai",
@@ -2658,7 +2658,7 @@ async function installStatusLineScript(scriptContent, targetPath = ".claude/stat
     throw new Error(`Failed to install status line script to ${targetPath}: ${errorMessage}`);
   }
 }
-async function installHookScript(scriptContent, targetPath = ".claude/hooks/sync-braingrid-task.sh") {
+async function installHookScript(scriptContent, targetPath) {
   try {
     const parentDir = path2.dirname(targetPath);
     await fs2.mkdir(parentDir, { recursive: true });
@@ -2679,7 +2679,64 @@ async function copyBraingridReadme(targetPath = ".braingrid/README.md") {
     return false;
   }
 }
-async function updateClaudeSettings(settingsPath = ".claude/settings.json", scriptPath2 = ".claude/statusline.sh", hookScriptPath = ".claude/hooks/sync-braingrid-task.sh", createHookScriptPath = ".claude/hooks/create-braingrid-task.sh", verifyHookScriptPath = ".claude/hooks/verify-acceptance-criteria.sh", postTaskUpdatePromptPath = ".claude/hooks/post-task-update-prompt.sh", preTaskCreatePath = ".claude/hooks/pre-task-create-naming.sh", preTaskUpdatePath = ".claude/hooks/pre-task-update-instructions.sh") {
+var HOOK_REGISTRY = [
+  {
+    hookType: "PostToolUse",
+    matcher: "TaskUpdate",
+    mergeStrategy: "upsertByMatcher",
+    commands: [
+      { script: "sync-braingrid-task.sh", timeout: 1e4 },
+      { script: "post-task-update-prompt.sh" }
+    ]
+  },
+  {
+    hookType: "PostToolUse",
+    matcher: "TaskCreate",
+    mergeStrategy: "upsertByMatcher",
+    commands: [{ script: "create-braingrid-task.sh", timeout: 1e4 }]
+  },
+  {
+    hookType: "PreToolUse",
+    matcher: "TaskCreate",
+    mergeStrategy: "upsertByMatcher",
+    commands: [{ script: "pre-task-create-naming.sh" }]
+  },
+  {
+    hookType: "PreToolUse",
+    matcher: "TaskUpdate",
+    mergeStrategy: "upsertByMatcher",
+    commands: [{ script: "pre-task-update-instructions.sh" }]
+  },
+  {
+    hookType: "Stop",
+    mergeStrategy: "appendDedup",
+    dedupSubstring: "verify-acceptance-criteria",
+    commands: [{ script: "verify-acceptance-criteria.sh" }]
+  },
+  {
+    hookType: "SessionStart",
+    mergeStrategy: "appendDedup",
+    dedupSubstring: "check-stale-build-sentinel",
+    commands: [{ script: "check-stale-build-sentinel.sh" }]
+  },
+  {
+    hookType: "TaskCompleted",
+    mergeStrategy: "appendDedup",
+    dedupSubstring: "task-completed-validate",
+    commands: [{ script: "task-completed-validate.sh" }]
+  }
+];
+function buildHookCommand(cmd) {
+  const result = {
+    type: "command",
+    command: `.claude/hooks/${cmd.script}`
+  };
+  if (cmd.timeout) {
+    result.timeout = cmd.timeout;
+  }
+  return result;
+}
+async function updateClaudeSettings(settingsPath = ".claude/settings.json", statusLineScriptPath = ".claude/statusline.sh", hookRegistry = HOOK_REGISTRY) {
   try {
     let settings = {};
     try {
@@ -2688,105 +2745,46 @@ async function updateClaudeSettings(settingsPath = ".claude/settings.json", scri
     } catch {
     }
     const existingStatusLine = settings.statusLine;
-    if (existingStatusLine?.command?.includes(scriptPath2)) {
+    if (existingStatusLine?.command?.includes(statusLineScriptPath)) {
     } else {
       settings.statusLine = {
         type: "command",
-        command: scriptPath2,
+        command: statusLineScriptPath,
         padding: 0
       };
     }
-    const ourHookEntry = {
-      matcher: "TaskUpdate",
-      hooks: [
-        {
-          type: "command",
-          command: hookScriptPath,
-          timeout: 1e4
-        },
-        {
-          type: "command",
-          command: postTaskUpdatePromptPath
-        }
-      ]
-    };
     const existingHooks = settings.hooks && typeof settings.hooks === "object" && !Array.isArray(settings.hooks) ? settings.hooks : {};
-    const existingPostToolUse = Array.isArray(existingHooks.PostToolUse) ? existingHooks.PostToolUse : [];
-    const taskUpdateIdx = existingPostToolUse.findIndex((e) => e.matcher === "TaskUpdate");
-    let mergedPostToolUse;
-    if (taskUpdateIdx >= 0) {
-      mergedPostToolUse = [...existingPostToolUse];
-      mergedPostToolUse[taskUpdateIdx] = ourHookEntry;
-    } else {
-      mergedPostToolUse = [...existingPostToolUse, ourHookEntry];
-    }
-    const ourCreateHookEntry = {
-      matcher: "TaskCreate",
-      hooks: [
-        {
-          type: "command",
-          command: createHookScriptPath,
-          timeout: 1e4
-        }
-      ]
-    };
-    const taskCreatePostIdx = mergedPostToolUse.findIndex((e) => e.matcher === "TaskCreate");
-    if (taskCreatePostIdx >= 0) {
-      mergedPostToolUse[taskCreatePostIdx] = ourCreateHookEntry;
-    } else {
-      mergedPostToolUse = [...mergedPostToolUse, ourCreateHookEntry];
-    }
-    const ourPreToolUseEntry = {
-      matcher: "TaskCreate",
-      hooks: [
-        {
-          type: "command",
-          command: preTaskCreatePath
+    const mergedByType = {};
+    for (const entry of hookRegistry) {
+      const hookType = entry.hookType;
+      if (!mergedByType[hookType]) {
+        const existing = Array.isArray(existingHooks[hookType]) ? [...existingHooks[hookType]] : [];
+        mergedByType[hookType] = existing;
+      }
+      const merged = mergedByType[hookType];
+      const hooks = entry.commands.map(buildHookCommand);
+      if (entry.mergeStrategy === "upsertByMatcher") {
+        const matcher = entry.matcher ?? "";
+        const newEntry = { matcher, hooks };
+        const idx = merged.findIndex((e) => e.matcher === matcher);
+        if (idx >= 0) {
+          merged[idx] = newEntry;
+        } else {
+          merged.push(newEntry);
         }
-      ]
-    };
-    const existingPreToolUse = Array.isArray(existingHooks.PreToolUse) ? existingHooks.PreToolUse : [];
-    const taskCreateIdx = existingPreToolUse.findIndex((e) => e.matcher === "TaskCreate");
-    let mergedPreToolUse;
-    if (taskCreateIdx >= 0) {
-      mergedPreToolUse = [...existingPreToolUse];
-      mergedPreToolUse[taskCreateIdx] = ourPreToolUseEntry;
-    } else {
-      mergedPreToolUse = [...existingPreToolUse, ourPreToolUseEntry];
-    }
-    const ourPreToolUseTaskUpdateEntry = {
-      matcher: "TaskUpdate",
-      hooks: [
-        {
-          type: "command",
-          command: preTaskUpdatePath
+      } else {
+        const substring = entry.dedupSubstring ?? "";
+        const alreadyPresent = merged.some(
+          (e) => e.hooks?.some((h) => h.command?.includes(substring))
+        );
+        if (!alreadyPresent) {
+          merged.push({ hooks });
         }
-      ]
-    };
-    const taskUpdatePreIdx = mergedPreToolUse.findIndex((e) => e.matcher === "TaskUpdate");
-    if (taskUpdatePreIdx >= 0) {
-      mergedPreToolUse[taskUpdatePreIdx] = ourPreToolUseTaskUpdateEntry;
-    } else {
-      mergedPreToolUse = [...mergedPreToolUse, ourPreToolUseTaskUpdateEntry];
+      }
     }
-    const ourStopHookEntry = {
-      hooks: [
-        {
-          type: "command",
-          command: verifyHookScriptPath
-        }
-      ]
-    };
-    const existingStop = Array.isArray(existingHooks.Stop) ? existingHooks.Stop : [];
-    const hasVerifyHook = existingStop.some(
-      (entry) => entry.hooks?.some((h) => h.command?.includes("verify-acceptance-criteria"))
-    );
-    const mergedStop = hasVerifyHook ? existingStop : [...existingStop, ourStopHookEntry];
     settings.hooks = {
       ...existingHooks,
-      PreToolUse: mergedPreToolUse,
-      PostToolUse: mergedPostToolUse,
-      Stop: mergedStop
+      ...mergedByType
     };
     const parentDir = path2.dirname(settingsPath);
     await fs2.mkdir(parentDir, { recursive: true });
@@ -3519,8 +3517,7 @@ function buildSuccessMessage(config2, installedPerDir, extras) {
 
 `) + chalk8.dim("Files installed:\n") + dirLines + extras + chalk8.dim(`  Content injected into: ${config2.injection.targetFile}
 
-`) + chalk8.dim("Next steps:\n") + chalk8.dim("  1. Review the integration files\n") + chalk8.dim(`  2. Open ${config2.name}
-`) + chalk8.dim("  3. Try the /specify or /breakdown commands\n") + chalk8.dim("  4. Learn more: ") + chalk8.cyan(config2.docsUrl);
+`) + chalk8.dim("Next, try:\n") + chalk8.dim("  /build REQ-X                    \u2192  build a requirement\n") + chalk8.dim('  /specify "add two-factor auth"  \u2192  specify a requirement\n') + chalk8.dim("  Learn more: ") + chalk8.cyan(config2.docsUrl);
 }
 function isSetupResult(result) {
   return "data" in result && result.success === true && !("message" in result);
@@ -3568,85 +3565,23 @@ async function handleSetupClaudeCode(opts) {
         error instanceof Error ? error.message : String(error)
       );
     }
-    let syncHookInstalled = false;
-    try {
-      const syncContent = await fetchFileFromGitHub("claude-code/hooks/sync-braingrid-task.sh");
-      await installHookScript(syncContent);
-      syncHookInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install sync hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
-    }
-    let createHookInstalled = false;
-    try {
-      const createContent = await fetchFileFromGitHub("claude-code/hooks/create-braingrid-task.sh");
-      await installHookScript(createContent, ".claude/hooks/create-braingrid-task.sh");
-      createHookInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install create hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
-    }
-    let verifyHookInstalled = false;
-    try {
-      const verifyContent = await fetchFileFromGitHub(
-        "claude-code/hooks/verify-acceptance-criteria.sh"
-      );
-      await installHookScript(verifyContent, ".claude/hooks/verify-acceptance-criteria.sh");
-      verifyHookInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install verify hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
-    }
-    let preTaskCreateInstalled = false;
-    try {
-      const preTaskCreateContent = await fetchFileFromGitHub(
-        "claude-code/hooks/pre-task-create-naming.sh"
-      );
-      await installHookScript(preTaskCreateContent, ".claude/hooks/pre-task-create-naming.sh");
-      preTaskCreateInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install pre-task-create hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
-    }
-    let preTaskUpdateInstalled = false;
-    try {
-      const preTaskUpdateContent = await fetchFileFromGitHub(
-        "claude-code/hooks/pre-task-update-instructions.sh"
-      );
-      await installHookScript(
-        preTaskUpdateContent,
-        ".claude/hooks/pre-task-update-instructions.sh"
-      );
-      preTaskUpdateInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install pre-task-update hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
-    }
-    let postTaskUpdateInstalled = false;
-    try {
-      const postTaskUpdateContent = await fetchFileFromGitHub(
-        "claude-code/hooks/post-task-update-prompt.sh"
-      );
-      await installHookScript(postTaskUpdateContent, ".claude/hooks/post-task-update-prompt.sh");
-      postTaskUpdateInstalled = true;
-    } catch (error) {
-      console.error(
-        chalk8.yellow("\u26A0\uFE0F  Failed to install post-task-update hook:"),
-        error instanceof Error ? error.message : String(error)
-      );
+    const allScripts = HOOK_REGISTRY.flatMap((e) => e.commands.map((c) => c.script));
+    const installedHooks = [];
+    for (const script of allScripts) {
+      try {
+        const content = await fetchFileFromGitHub(`claude-code/hooks/${script}`);
+        await installHookScript(content, `.claude/hooks/${script}`);
+        installedHooks.push(script);
+      } catch (error) {
+        console.error(
+          chalk8.yellow(`\u26A0\uFE0F  Failed to install hook ${script}:`),
+          error instanceof Error ? error.message : String(error)
+        );
+      }
     }
     const statusLineMessage = statusLineInstalled ? chalk8.dim("  Status line: .claude/statusline.sh\n") : "";
-    const hooksMessage = (syncHookInstalled ? chalk8.dim("  Hook script: .claude/hooks/sync-braingrid-task.sh\n") : "") + (createHookInstalled ? chalk8.dim("  Hook script: .claude/hooks/create-braingrid-task.sh\n") : "") + (verifyHookInstalled ? chalk8.dim("  Hook script: .claude/hooks/verify-acceptance-criteria.sh\n") : "") + (preTaskCreateInstalled ? chalk8.dim("  Hook script: .claude/hooks/pre-task-create-naming.sh\n") : "") + (preTaskUpdateInstalled ? chalk8.dim("  Hook script: .claude/hooks/pre-task-update-instructions.sh\n") : "") + (postTaskUpdateInstalled ? chalk8.dim("  Hook script: .claude/hooks/post-task-update-prompt.sh\n") : "");
+    const hooksMessage = installedHooks.map((s) => chalk8.dim(`  Hook script: .claude/hooks/${s}
+`)).join("");
     return {
       success: true,
       message: buildSuccessMessage(config2, displayPerDir, statusLineMessage + hooksMessage)