claude-code-kanban 5.3.0 → 5.4.0

package/README.md

@@ -19,7 +19,7 @@ Watch the tour on YouTube: [light](https://youtu.be/QbvDBFyfC7s), [dark](https:/
 
 ## Getting started
 
-You need Node.js 20 or later, the `claude` CLI, and `jq` for the hook scripts.
+You need Node.js 20 or later and the `claude` CLI.
 
 ### 1. Install the integration (one time)
 
@@ -27,7 +27,7 @@ You need Node.js 20 or later, the `claude` CLI, and `jq` for the hook scripts.
 npx claude-code-kanban --install
 ```
 
-The installer adds a Claude Code plugin with hooks and skills, and a statusLine script for context use and cost. It asks before each change and keeps your other settings. Without the hooks, the board shows tasks only: no agent log, no live activity, no waiting prompts.
+The installer adds a Claude Code plugin with hooks, skills and a mod for context use and cost (Claude Code 2.1.287 or later). It asks before it installs and keeps your other settings. Without the hooks, the board shows tasks only: no agent log, no live activity, no waiting prompts.
 
 To remove it, run `npx claude-code-kanban --uninstall`. For another Claude config dir, pass the same `--dir=<path>` (or set `CLAUDE_CONFIG_DIR`) to `--install`, `--uninstall` and the server. See [Getting started](https://nikiforovall.blog/claude-code-kanban/getting-started/) for each install step.
 

package/cli.js

@@ -331,8 +331,8 @@ function printTopHelp() {
   console.log('  --port <n>            Port to listen on (default 3541)');
   console.log('  --dir <path>          Override Claude config dir (default ~/.claude); also targets --install/--uninstall');
   console.log('  --open                Open browser on start');
-  console.log('  --install, --uninstall    Install or remove the plugin, context spy, and statusline');
-  console.log('  --plugin-only         With --install: refresh only the plugin, skip context spy and statusline');
+  console.log('  --install, --uninstall    Install or remove the plugin');
+  console.log('  --yes                 With --install: install without a prompt');
   console.log('\nEnvironment:');
   console.log('  CCK_URL               Server base URL, e.g. http://127.0.0.1:4795 (wins over PORT)');
   console.log('  PORT                  Server port (default: the one this config dir\'s server reports, else 3541)');
@@ -431,6 +431,8 @@ class CliUnreachable extends Error { constructor() { super(unreachable()); this.
 // connects run at once. Nothing reached the server, so even a POST is safe to send again.
 const CONNECT_RETRY_MS = [250, 500, 1000, 2000];
 const isConnectTimeout = (e) => e.cause?.code === 'ETIMEDOUT' && e.cause?.syscall === 'connect';
+// A loopback connection can also be reset after it opens. A GET is safe to send again; a write may have landed.
+const isRetryable = (e, init) => isConnectTimeout(e) || (!init?.method && e.cause?.code === 'ECONNRESET');
 
 async function cliFetch(urlPath, init) {
   const base = cliBaseUrl();
@@ -439,7 +441,7 @@ async function cliFetch(urlPath, init) {
     try {
       return await fetch(`${base}${urlPath}`, init);
     } catch (e) {
-      if (isConnectTimeout(e) && i < CONNECT_RETRY_MS.length) {
+      if (isRetryable(e, init) && i < CONNECT_RETRY_MS.length) {
         await new Promise((r) => setTimeout(r, CONNECT_RETRY_MS[i]));
         continue;
       }
@@ -810,9 +812,12 @@ async function runSessionViewCli(args) {
     ].filter(Boolean).join(' · ');
     lines.push(`  Model: ${modelName}${modelExtras ? ` (${modelExtras})` : ''}`);
     if (cw.used_percentage != null) {
-      lines.push(`  Context: ${cw.used_percentage}% used · ${fmtTok(cw.total_input_tokens)} in / ${fmtTok(cw.total_output_tokens)} out · cache ${fmtTok(cw.current_usage?.cache_read_input_tokens)} read`);
+      lines.push(`  Context: ${cw.used_percentage}% used · ${fmtTok(cw.total_input_tokens)} in${cw.total_output_tokens != null ? ` / ${fmtTok(cw.total_output_tokens)} out` : ''} · cache ${fmtTok(cw.current_usage?.cache_read_input_tokens)} read`);
     }
-    lines.push(`  Cost: ${fmtCost(cost.total_cost_usd)} · ${cost.total_api_duration_ms != null ? formatAge(cost.total_api_duration_ms) : '-'} api / ${cost.total_duration_ms != null ? formatAge(cost.total_duration_ms) : '-'} total · +${cost.total_lines_added || 0}/-${cost.total_lines_removed || 0}`);
+    const timing = cost.total_duration_ms != null
+      ? ` · ${cost.total_api_duration_ms != null ? formatAge(cost.total_api_duration_ms) : '-'} api / ${formatAge(cost.total_duration_ms)} total · +${cost.total_lines_added || 0}/-${cost.total_lines_removed || 0}`
+      : '';
+    lines.push(`  Cost: ${fmtCost(cost.total_cost_usd)}${timing}`);
     if (rl.five_hour || rl.seven_day) {
       lines.push(`  Limits: 5h ${rl.five_hour?.used_percentage ?? '-'}% · 7d ${rl.seven_day?.used_percentage ?? '-'}%`);
     }

package/install.js

@@ -10,14 +10,13 @@ const { PLUGIN_ID } = require('./lib/plugin-status');
 const CLAUDE_DIR = getClaudeDir();
 const CLI_ENV = claudeCliEnv(CLAUDE_DIR);
 const CCK_DIR = path.join(CLAUDE_DIR, '.cck');
-const HOOKS_DIR = path.join(CLAUDE_DIR, 'hooks');
 const SETTINGS_PATH = path.join(CLAUDE_DIR, 'settings.json');
 const PLUGIN_SRC = path.join(__dirname, 'plugin');
 const PLUGIN_DEST = path.join(CCK_DIR, 'plugin');
+// Earlier versions piped the statusLine through this script to capture context use; the plugin's mod does it now.
 const CTX_SCRIPT_NAME = 'context-status.sh';
-const CTX_SCRIPT_SRC = path.join(PLUGIN_SRC, 'plugins', 'claude-code-kanban', 'scripts', CTX_SCRIPT_NAME);
-const CTX_SCRIPT_DEST = path.join(HOOKS_DIR, CTX_SCRIPT_NAME);
-const CTX_COMMAND = displayPath(CTX_SCRIPT_DEST);
+const CTX_SCRIPT_DEST = path.join(CLAUDE_DIR, 'hooks', CTX_SCRIPT_NAME);
+const MIN_CLAUDE_VERSION = [2, 1, 287];
 
 // ANSI helpers
 const green = s => `\x1b[32m${s}\x1b[0m`;
@@ -47,10 +46,37 @@ function runCLI(cmd, okPatterns = []) {
   }
 }
 
-function copyScript(src, dest) {
-  fs.mkdirSync(path.dirname(dest), { recursive: true });
-  fs.copyFileSync(src, dest);
-  try { fs.chmodSync(dest, 0o755); } catch {}
+function isBelowMinVersion(versionOutput) {
+  const parts = /(\d+)\.(\d+)\.(\d+)/.exec(versionOutput)?.slice(1).map(Number);
+  if (!parts) return false;
+  const i = parts.findIndex((n, k) => n !== MIN_CLAUDE_VERSION[k]);
+  return i !== -1 && parts[i] < MIN_CLAUDE_VERSION[i];
+}
+
+function removeContextSpy() {
+  if (fs.existsSync(CTX_SCRIPT_DEST)) {
+    fs.unlinkSync(CTX_SCRIPT_DEST);
+    console.log(`  Old context spy: ${green('✓')} Removed ${dim(displayPath(CTX_SCRIPT_DEST))}`);
+  }
+  if (!fs.existsSync(SETTINGS_PATH)) return;
+  let settings;
+  try {
+    settings = JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
+  } catch {
+    console.log(`  Settings: ${red('✗')} Could not parse settings.json`);
+    return;
+  }
+  const cmd = settings.statusLine?.command;
+  if (!cmd?.includes(CTX_SCRIPT_NAME)) return;
+  const stripped = cmd.replace(/\S*context-status\.sh\s*\|?\s*/, '').trim();
+  if (stripped) {
+    settings.statusLine.command = stripped;
+    console.log(`  StatusLine: ${green('✓')} Restored to "${stripped}"`);
+  } else {
+    delete settings.statusLine;
+    console.log(`  StatusLine: ${green('✓')} Removed`);
+  }
+  fs.writeFileSync(SETTINGS_PATH, `${JSON.stringify(settings, null, 2)}\n`);
 }
 
 // Deletes files but never directories: on Windows the running Claude Code process holds
@@ -80,8 +106,8 @@ function copyDirSync(src, dest) {
   }
 }
 
-async function runInstall({ pluginOnly = false } = {}) {
-  console.log(`\n  ${bold('claude-code-kanban')} — ${pluginOnly ? 'Plugin installer' : 'Plugin & StatusLine installer'}\n`);
+async function runInstall({ yes = false } = {}) {
+  console.log(`\n  ${bold('claude-code-kanban')} — Plugin installer\n`);
   console.log(`  Claude config dir: ${dim(displayPath(CLAUDE_DIR))}\n`);
   let failed = false;
 
@@ -90,23 +116,18 @@ async function runInstall({ pluginOnly = false } = {}) {
   const claude = runCLI('claude --version');
   if (claude.ok) {
     console.log(green(`✓ found (${claude.output})`));
+    if (isBelowMinVersion(claude.output)) {
+      console.log(`    ${yellow(`⚠ context use and cost tracking needs Claude Code ${MIN_CLAUDE_VERSION.join('.')} or later — run \`claude update\``)}`);
+    }
   } else {
     console.log(red('✗ claude CLI not found'));
     console.log(`    ${dim('Install Claude Code CLI first: https://docs.anthropic.com/en/docs/claude-code')}`);
     return;
   }
 
-  process.stdout.write('  Checking jq... ');
-  const jq = runCLI('jq --version');
-  if (jq.ok) {
-    console.log(green(`✓ found (${jq.output})`));
-  } else {
-    console.log(yellow('⚠ not found — hook scripts require jq for JSON parsing'));
-  }
-
   // 2. Copy plugin to stable location & register marketplace
   console.log(`\n  Plugin: ${dim(PLUGIN_DEST)}`);
-  if (pluginOnly || await prompt(`    Install claude-code-kanban plugin? [Y/n] `)) {
+  if (yes || await prompt(`    Install claude-code-kanban plugin? [Y/n] `)) {
     process.stdout.write(`    Copying plugin to ${displayPath(PLUGIN_DEST)}... `);
     try {
       clearFilesRecursive(PLUGIN_DEST);
@@ -155,75 +176,8 @@ async function runInstall({ pluginOnly = false } = {}) {
     console.log(`    ${dim('Skipped')}`);
   }
 
-  if (pluginOnly) {
-    console.log(`\n  ${dim('Context spy and statusline skipped (--plugin-only).')}`);
-    printSummary(failed);
-    return;
-  }
-
-  // 3. StatusLine setup (context-status.sh must be copied globally since statusLine is not plugin-scoped)
-  console.log(`\n  Context spy: ${dim(CTX_SCRIPT_DEST)}`);
-  let ctxInstalled = false;
-  if (fs.existsSync(CTX_SCRIPT_DEST)) {
-    const existing = fs.readFileSync(CTX_SCRIPT_DEST, 'utf8');
-    const bundled = fs.readFileSync(CTX_SCRIPT_SRC, 'utf8');
-    if (existing === bundled) {
-      console.log(`    ${green('✓')} Up to date`);
-      ctxInstalled = true;
-    } else if (await prompt(`    Different version found. Update? [Y/n] `)) {
-      copyScript(CTX_SCRIPT_SRC, CTX_SCRIPT_DEST);
-      console.log(`    ${green('✓')} Updated`);
-      ctxInstalled = true;
-    } else {
-      console.log(`    ${dim('Skipped')}`);
-    }
-  } else if (await prompt(`    Not found. Install? [Y/n] `)) {
-    copyScript(CTX_SCRIPT_SRC, CTX_SCRIPT_DEST);
-    console.log(`    ${green('✓')} Installed`);
-    ctxInstalled = true;
-  } else {
-    console.log(`    ${dim('Skipped')}`);
-  }
-
-  // 4. StatusLine config in settings.json
-  if (ctxInstalled) {
-    let settings;
-    try {
-      settings = fs.existsSync(SETTINGS_PATH)
-        ? JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'))
-        : {};
-    } catch {
-      console.log(`    ${red('✗')} Malformed JSON in settings.json — skipping statusline config`);
-      printSummary(true);
-      return;
-    }
-
-    const hasCtx = settings.statusLine?.command?.includes(CTX_SCRIPT_NAME);
-    if (hasCtx) {
-      console.log(`\n  StatusLine: ${green('✓')} Already configured`);
-    } else if (!settings.statusLine) {
-      console.log(`\n  StatusLine: ${dim('not configured')}`);
-      if (await prompt(`    Set up context tracking statusline? [Y/n] `)) {
-        settings.statusLine = { type: 'command', command: CTX_COMMAND };
-        fs.writeFileSync(SETTINGS_PATH, `${JSON.stringify(settings, null, 2)}\n`);
-        console.log(`    ${green('✓')} StatusLine configured`);
-      } else {
-        console.log(`    ${dim('Skipped')}`);
-      }
-    } else {
-      const existing = settings.statusLine.command;
-      console.log(`\n  StatusLine: ${dim(`current: ${existing}`)}`);
-      if (await prompt(`    Prepend context spy to existing statusline? [Y/n] `)) {
-        settings.statusLine.type = 'command';
-        settings.statusLine.command = `${CTX_COMMAND} | ${existing}`;
-        fs.writeFileSync(SETTINGS_PATH, `${JSON.stringify(settings, null, 2)}\n`);
-        console.log(`    ${green('✓')} StatusLine updated`);
-      } else {
-        console.log(`    ${dim('Skipped')}`);
-      }
-    }
-  }
-
+  console.log('');
+  removeContextSpy();
   printSummary(failed);
 }
 
@@ -265,41 +219,7 @@ async function runUninstall() {
     console.log(`  Plugin copy: ${dim('Not found')}`);
   }
 
-  // 4. Remove context-status.sh copy
-  if (fs.existsSync(CTX_SCRIPT_DEST)) {
-    fs.unlinkSync(CTX_SCRIPT_DEST);
-    console.log(`  Context spy: ${green('✓')} Removed`);
-  } else {
-    console.log(`  Context spy: ${dim('Not found')}`);
-  }
-
-  // 5. Clean up settings.json (statusLine)
-  if (fs.existsSync(SETTINGS_PATH)) {
-    try {
-      const settings = JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
-      let changed = false;
-
-      // Strip context-status.sh from statusLine
-      if (settings.statusLine?.command?.includes(CTX_SCRIPT_NAME)) {
-        const cmd = settings.statusLine.command;
-        const stripped = cmd.replace(new RegExp(`\\S*${CTX_SCRIPT_NAME.replace('.', '\\.')}\\s*\\|\\s*`), '').trim();
-        if (stripped && stripped !== cmd) {
-          settings.statusLine.command = stripped;
-          console.log(`  StatusLine: ${green('✓')} Restored to "${stripped}"`);
-        } else {
-          delete settings.statusLine;
-          console.log(`  StatusLine: ${green('✓')} Removed`);
-        }
-        changed = true;
-      }
-
-      if (changed) {
-        fs.writeFileSync(SETTINGS_PATH, `${JSON.stringify(settings, null, 2)}\n`);
-      }
-    } catch {
-      console.log(`  Settings: ${red('✗')} Could not parse settings.json`);
-    }
-  }
+  removeContextSpy();
 
   console.log(`\n  ${green('Uninstall complete.')}\n`);
 }

package/lib/approvals.js

@@ -1,18 +1,18 @@
-// UI-driven approvals (_plans/cck-ui-approvals/): approval-gate.sh writes the
+// UI-driven approvals (_plans/cck-ui-approvals/): the plugin's mod writes the
 // _waiting.json marker with a request id (D8) and polls for _decision-<id>.json.
 // buildDecision validates a board response against the live marker and shapes
-// the decision file; the route writes it. The hook deletes marker + decision on
-// consumption; orphans from a terminal deny (D13) are left for the sweep.
+// the decision file; the route writes it. Mods cannot delete files, so the mod
+// clears the marker and the sweep removes decision files.
 
-// Ids are hook-minted (uuidgen or a time-pid-random compound) — anything outside
-// this alphabet is either corruption or a path-traversal attempt (3.5).
+// Ids are mod-minted UUIDs — anything outside this alphabet is either
+// corruption or a path-traversal attempt (3.5).
 function sanitizeRequestId(raw) {
   return typeof raw === 'string' && /^[a-zA-Z0-9-]{1,64}$/.test(raw) ? raw : null;
 }
 
-// A decision whose shape doesn't match the ask's kind produces hook output
-// Claude Code rejects, surfacing as an opaque stall — reject at the API
-// boundary instead (D7). Returns { error, status } or { decision }.
+// A decision whose shape doesn't match the ask's kind cannot answer the call,
+// surfacing as an opaque stall — reject at the API boundary instead (D7).
+// Returns { error, status } or { decision }.
 function buildDecision(marker, body) {
   if (!marker || marker.status !== 'waiting') {
     return { error: 'No pending ask', status: 410 };
@@ -45,7 +45,6 @@ function buildDecision(marker, body) {
   const decision = { behavior: body.behavior };
   if (typeof body.message === 'string' && body.message) decision.message = body.message;
   if (body.updatedInput && typeof body.updatedInput === 'object') decision.updatedInput = body.updatedInput;
-  if (Array.isArray(body.updatedPermissions)) decision.updatedPermissions = body.updatedPermissions;
   return { decision };
 }
 
@@ -58,14 +57,14 @@ function isDecisionFile(name) {
   return name.startsWith('_decision-') && name.endsWith('.json');
 }
 
-// approval-gate.sh only polls for a decision for waitSeconds — after that the
-// ask belongs to the terminal. Default and clamp mirror the gate's own parse
-// (PERMISSION_TTL_MS hides the card at 30 min, so waiting longer than the UI
-// can show the ask is strictly worse — D11); keep them in sync with
-// approval-gate.sh.
+// The mod only polls for a decision for waitSeconds — after that the ask
+// belongs to the terminal. Default and clamp mirror the mod's own parse
+// (boardWaitMs in the plugin's hooks/activity.ts; PERMISSION_TTL_MS hides the
+// card at 30 min, so waiting longer than the UI can show the ask is strictly
+// worse — D11); keep them in sync.
 const WAIT_SECONDS_DEFAULT = 1800;
 const WAIT_SECONDS_MAX = 1800;
-// Small grace over the gate's own deadline so a race never 410s a live hook
+// Small grace over the mod's own deadline so a race never 410s a live ask
 const LAPSE_GRACE_MS = 5000;
 function waitSecondsFrom(cfg) {
   const raw = cfg?.waitSeconds;
@@ -75,7 +74,7 @@ function waitSecondsFrom(cfg) {
 
 // Normalizes the `approvals` section of <CCK_DIR>/config.json. The gate is on
 // by default: only an explicit `enabled: false` turns it off, and anything
-// unparseable falls back to defaults — the same parse approval-gate.sh does.
+// unparseable falls back to defaults — the same parse the mod does.
 function approvalsFrom(cfg) {
   const a = cfg && typeof cfg.approvals === 'object' && cfg.approvals ? cfg.approvals : {};
   return {
@@ -85,7 +84,7 @@ function approvalsFrom(cfg) {
   };
 }
 
-// Whether the hook waits for a board decision on a marker of this kind
+// Whether the mod waits for a board decision on a marker of this kind
 function isKindGated(kind, approvals) {
   if (!approvals.enabled) return false;
   if (kind === 'question') return approvals.mode === 'permission+question';

package/lib/git-branch.js

@@ -0,0 +1,47 @@
+// Read from HEAD rather than by spawning git: a sync spawn on the session-list path blocks the
+// event loop for up to half a second per cwd on Windows, and terminal input waits behind it.
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+const REF_RE = /^ref:\s*refs\/heads\/(.+)$/;
+const GITDIR_RE = /^gitdir:\s*(.+)$/;
+
+// `.git` is a directory in an ordinary checkout and a `gitdir:` pointer file in a linked
+// worktree or a submodule.
+function headFile(dir, readFile) {
+  const dotGit = path.join(dir, '.git');
+  let text;
+  try {
+    text = readFile(dotGit);
+  } catch (e) {
+    if (e.code === 'EISDIR') return path.join(dotGit, 'HEAD');
+    return null;
+  }
+  const m = GITDIR_RE.exec(text.trim());
+  return m ? path.join(path.resolve(dir, m[1]), 'HEAD') : null;
+}
+
+/**
+ * @param {string} cwd
+ * @param {(file: string) => string} [readFile]
+ * @returns {string|null} null outside a repo and on a detached HEAD
+ */
+function readGitBranch(cwd, readFile = (f) => fs.readFileSync(f, 'utf8')) {
+  for (let dir = path.resolve(cwd); ; ) {
+    const head = headFile(dir, readFile);
+    if (head) {
+      try {
+        const m = REF_RE.exec(readFile(head).trim());
+        return m ? m[1] : null;
+      } catch {
+        return null;
+      }
+    }
+    const up = path.dirname(dir);
+    if (up === dir) return null;
+    dir = up;
+  }
+}
+
+module.exports = { readGitBranch };