code-warden 3.3.0 → 3.3.1

package/tools/hooks/claude/install-hooks.js

@@ -1,112 +1,112 @@
-#!/usr/bin/env node
-/**
- * install-hooks.js
- * Merges code-warden PreToolUse hook entries into ~/.claude/settings.json.
- *
- * Idempotent: removes any existing code-warden entries by description marker,
- * then inserts current entries. Replace, not skip — ensures paths stay current
- * after reinstalls or version moves.
- *
- * Requires the skill to already be installed at skillDir before writing
- * settings — avoids dangling settings pointing at a missing skill directory.
- */
-
-'use strict';
-
-const fs   = require('fs');
-const path = require('path');
-const os   = require('os');
-
-const MARKER_PREFIX  = 'code-warden:';
-const SETTINGS_PATH  = path.join(os.homedir(), '.claude', 'settings.json');
-
-// ---------------------------------------------------------------------------
-// Settings I/O
-// ---------------------------------------------------------------------------
-
-function readSettings() {
-  if (!fs.existsSync(SETTINGS_PATH)) return {};
-  try {
-    return JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
-  } catch (err) {
-    throw new Error(`settings.json exists but could not be parsed (${SETTINGS_PATH}): ${err.message}`);
-  }
-}
-
-function writeSettings(settings) {
-  const tmp = `${SETTINGS_PATH}.tmp`;
-  fs.mkdirSync(path.dirname(SETTINGS_PATH), { recursive: true });
-  fs.writeFileSync(tmp, JSON.stringify(settings, null, 2) + '\n', 'utf8');
-  fs.renameSync(tmp, SETTINGS_PATH);
-}
-
-// ---------------------------------------------------------------------------
-// Hook entry helpers
-// ---------------------------------------------------------------------------
-
-function stripCodeWardenHooks(preToolUse) {
-  return preToolUse
-    .map(matcher => ({
-      ...matcher,
-      hooks: (matcher.hooks || []).filter(
-        h => !String(h.description || '').startsWith(MARKER_PREFIX)
-      ),
-    }))
-    .filter(matcher => (matcher.hooks || []).length > 0);
-}
-
-function buildEntries(skillDir) {
-  return [
-    {
-      type:        'command',
-      command:     'node',
-      args:        [path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-lint-hook.js')],
-      description: 'code-warden: file length gate',
-      timeout:     30,
-    },
-    {
-      type:        'command',
-      command:     'node',
-      args:        [path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-secrets-hook.js')],
-      description: 'code-warden: zero-trust secrets gate',
-      timeout:     30,
-    },
-  ];
-}
-
-// ---------------------------------------------------------------------------
-// Install
-// ---------------------------------------------------------------------------
-
-function installHooks(skillDir) {
-  // Guard: skill must be installed before settings are written
-  const required = [
-    path.join(skillDir, 'SKILL.md'),
-    path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-lint-hook.js'),
-    path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-secrets-hook.js'),
-  ];
-  for (const p of required) {
-    if (!fs.existsSync(p)) {
-      console.error('[CodeWarden] Hooks require an installed Claude target.');
-      console.error(`[CodeWarden] Missing: ${p}`);
-      console.error('[CodeWarden] Run: node install.js --target=claude --all');
-      process.exit(1);
-    }
-  }
-
-  const settings  = readSettings();
-  settings.hooks  = settings.hooks || {};
-  const existing  = settings.hooks.PreToolUse || [];
-
-  // Remove stale code-warden entries, then append fresh block
-  const cleaned   = stripCodeWardenHooks(existing);
-  settings.hooks.PreToolUse = [
-    ...cleaned,
-    { matcher: 'Write|Edit', hooks: buildEntries(skillDir) },
-  ];
-
-  writeSettings(settings);
-  return SETTINGS_PATH;
-}
-
-module.exports = { installHooks };
+#!/usr/bin/env node
+/**
+ * install-hooks.js
+ * Merges code-warden PreToolUse hook entries into ~/.claude/settings.json.
+ *
+ * Idempotent: removes any existing code-warden entries by description marker,
+ * then inserts current entries. Replace, not skip — ensures paths stay current
+ * after reinstalls or version moves.
+ *
+ * Requires the skill to already be installed at skillDir before writing
+ * settings — avoids dangling settings pointing at a missing skill directory.
+ */
+
+'use strict';
+
+const fs   = require('fs');
+const path = require('path');
+const os   = require('os');
+
+const MARKER_PREFIX  = 'code-warden:';
+const SETTINGS_PATH  = path.join(os.homedir(), '.claude', 'settings.json');
+
+// ---------------------------------------------------------------------------
+// Settings I/O
+// ---------------------------------------------------------------------------
+
+function readSettings() {
+  if (!fs.existsSync(SETTINGS_PATH)) return {};
+  try {
+    return JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
+  } catch (err) {
+    throw new Error(`settings.json exists but could not be parsed (${SETTINGS_PATH}): ${err.message}`);
+  }
+}
+
+function writeSettings(settings) {
+  const tmp = `${SETTINGS_PATH}.tmp`;
+  fs.mkdirSync(path.dirname(SETTINGS_PATH), { recursive: true });
+  fs.writeFileSync(tmp, JSON.stringify(settings, null, 2) + '\n', 'utf8');
+  fs.renameSync(tmp, SETTINGS_PATH);
+}
+
+// ---------------------------------------------------------------------------
+// Hook entry helpers
+// ---------------------------------------------------------------------------
+
+function stripCodeWardenHooks(preToolUse) {
+  return preToolUse
+    .map(matcher => ({
+      ...matcher,
+      hooks: (matcher.hooks || []).filter(
+        h => !String(h.description || '').startsWith(MARKER_PREFIX)
+      ),
+    }))
+    .filter(matcher => (matcher.hooks || []).length > 0);
+}
+
+function buildEntries(skillDir) {
+  return [
+    {
+      type:        'command',
+      command:     'node',
+      args:        [path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-lint-hook.js')],
+      description: 'code-warden: file length gate',
+      timeout:     30,
+    },
+    {
+      type:        'command',
+      command:     'node',
+      args:        [path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-secrets-hook.js')],
+      description: 'code-warden: zero-trust secrets gate',
+      timeout:     30,
+    },
+  ];
+}
+
+// ---------------------------------------------------------------------------
+// Install
+// ---------------------------------------------------------------------------
+
+function installHooks(skillDir) {
+  // Guard: skill must be installed before settings are written
+  const required = [
+    path.join(skillDir, 'SKILL.md'),
+    path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-lint-hook.js'),
+    path.join(skillDir, 'tools', 'hooks', 'claude', 'warden-secrets-hook.js'),
+  ];
+  for (const p of required) {
+    if (!fs.existsSync(p)) {
+      console.error('[CodeWarden] Hooks require an installed Claude target.');
+      console.error(`[CodeWarden] Missing: ${p}`);
+      console.error('[CodeWarden] Run: node install.js --target=claude --all');
+      process.exit(1);
+    }
+  }
+
+  const settings  = readSettings();
+  settings.hooks  = settings.hooks || {};
+  const existing  = settings.hooks.PreToolUse || [];
+
+  // Remove stale code-warden entries, then append fresh block
+  const cleaned   = stripCodeWardenHooks(existing);
+  settings.hooks.PreToolUse = [
+    ...cleaned,
+    { matcher: 'Write|Edit', hooks: buildEntries(skillDir) },
+  ];
+
+  writeSettings(settings);
+  return SETTINGS_PATH;
+}
+
+module.exports = { installHooks };

package/tools/hooks/claude/uninstall-hooks.js

@@ -1,75 +1,75 @@
-#!/usr/bin/env node
-/**
- * uninstall-hooks.js
- * Removes all code-warden hook entries from ~/.claude/settings.json.
- * Identified by description prefix "code-warden:".
- * Cleans up empty PreToolUse arrays and empty hooks objects after removal.
- */
-
-'use strict';
-
-const fs   = require('fs');
-const path = require('path');
-const os   = require('os');
-
-const MARKER_PREFIX = 'code-warden:';
-const SETTINGS_PATH = path.join(os.homedir(), '.claude', 'settings.json');
-
-function readSettings() {
-  if (!fs.existsSync(SETTINGS_PATH)) return null;
-  try {
-    return JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
-  } catch (err) {
-    throw new Error(`settings.json could not be parsed (${SETTINGS_PATH}): ${err.message}`);
-  }
-}
-
-function writeSettings(settings) {
-  const tmp = `${SETTINGS_PATH}.tmp`;
-  fs.writeFileSync(tmp, JSON.stringify(settings, null, 2) + '\n', 'utf8');
-  fs.renameSync(tmp, SETTINGS_PATH);
-}
-
-function uninstallHooks() {
-  const settings = readSettings();
-
-  if (!settings) {
-    console.log('[CodeWarden]   No settings.json found — nothing to remove.');
-    return false;
-  }
-
-  if (!settings.hooks || !settings.hooks.PreToolUse) {
-    console.log('[CodeWarden]   No PreToolUse hooks in settings.json — nothing to remove.');
-    return false;
-  }
-
-  const before  = settings.hooks.PreToolUse;
-  const cleaned = before
-    .map(matcher => ({
-      ...matcher,
-      hooks: (matcher.hooks || []).filter(
-        h => !String(h.description || '').startsWith(MARKER_PREFIX)
-      ),
-    }))
-    .filter(matcher => (matcher.hooks || []).length > 0);
-
-  const removedMatchers = before.length - cleaned.length;
-  const removedHooks    = before.flatMap(m => m.hooks || []).filter(
-    h => String(h.description || '').startsWith(MARKER_PREFIX)
-  ).length;
-
-  if (removedHooks === 0) {
-    console.log('[CodeWarden]   No code-warden hook entries found — nothing to remove.');
-    return false;
-  }
-
-  settings.hooks.PreToolUse = cleaned;
-  if (cleaned.length === 0)                      delete settings.hooks.PreToolUse;
-  if (Object.keys(settings.hooks).length === 0)  delete settings.hooks;
-
-  writeSettings(settings);
-  console.log(`[CodeWarden]   Removed ${removedHooks} hook entry(ies) from settings.json.`);
-  return true;
-}
-
-module.exports = { uninstallHooks };
+#!/usr/bin/env node
+/**
+ * uninstall-hooks.js
+ * Removes all code-warden hook entries from ~/.claude/settings.json.
+ * Identified by description prefix "code-warden:".
+ * Cleans up empty PreToolUse arrays and empty hooks objects after removal.
+ */
+
+'use strict';
+
+const fs   = require('fs');
+const path = require('path');
+const os   = require('os');
+
+const MARKER_PREFIX = 'code-warden:';
+const SETTINGS_PATH = path.join(os.homedir(), '.claude', 'settings.json');
+
+function readSettings() {
+  if (!fs.existsSync(SETTINGS_PATH)) return null;
+  try {
+    return JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf8'));
+  } catch (err) {
+    throw new Error(`settings.json could not be parsed (${SETTINGS_PATH}): ${err.message}`);
+  }
+}
+
+function writeSettings(settings) {
+  const tmp = `${SETTINGS_PATH}.tmp`;
+  fs.writeFileSync(tmp, JSON.stringify(settings, null, 2) + '\n', 'utf8');
+  fs.renameSync(tmp, SETTINGS_PATH);
+}
+
+function uninstallHooks() {
+  const settings = readSettings();
+
+  if (!settings) {
+    console.log('[CodeWarden]   No settings.json found — nothing to remove.');
+    return false;
+  }
+
+  if (!settings.hooks || !settings.hooks.PreToolUse) {
+    console.log('[CodeWarden]   No PreToolUse hooks in settings.json — nothing to remove.');
+    return false;
+  }
+
+  const before  = settings.hooks.PreToolUse;
+  const cleaned = before
+    .map(matcher => ({
+      ...matcher,
+      hooks: (matcher.hooks || []).filter(
+        h => !String(h.description || '').startsWith(MARKER_PREFIX)
+      ),
+    }))
+    .filter(matcher => (matcher.hooks || []).length > 0);
+
+  const removedMatchers = before.length - cleaned.length;
+  const removedHooks    = before.flatMap(m => m.hooks || []).filter(
+    h => String(h.description || '').startsWith(MARKER_PREFIX)
+  ).length;
+
+  if (removedHooks === 0) {
+    console.log('[CodeWarden]   No code-warden hook entries found — nothing to remove.');
+    return false;
+  }
+
+  settings.hooks.PreToolUse = cleaned;
+  if (cleaned.length === 0)                      delete settings.hooks.PreToolUse;
+  if (Object.keys(settings.hooks).length === 0)  delete settings.hooks;
+
+  writeSettings(settings);
+  console.log(`[CodeWarden]   Removed ${removedHooks} hook entry(ies) from settings.json.`);
+  return true;
+}
+
+module.exports = { uninstallHooks };

package/tools/hooks/claude/warden-lint-hook.js

@@ -1,106 +1,106 @@
-#!/usr/bin/env node
-/**
- * warden-lint-hook.js
- * PreToolUse Claude Code hook: blocks Write/Edit if the resulting file would
- * exceed the configured line limit.
- *
- * Payload (stdin JSON):  { tool_name, tool_input: { file_path, content|new_string, ... } }
- * On violation: exit 2 + JSON deny response to stdout.
- * On pass:      exit 0 (no output).
- */
-
-'use strict';
-
-const fs   = require('fs');
-const path = require('path');
-const { countLines } = require('../../lib/line-count');
-const { loadConfig } = require('../../lib/config');
-
-// ---------------------------------------------------------------------------
-// Config — loaded via shared module; falls back to 400 if missing
-// ---------------------------------------------------------------------------
-
-const { maxFileLength: MAX_LINES } = loadConfig();
-
-// ---------------------------------------------------------------------------
-// Skip list — file types where line counting is meaningless
-// ---------------------------------------------------------------------------
-
-const SKIP_EXTS = new Set([
-  '.png', '.jpg', '.jpeg', '.gif', '.bmp', '.ico', '.svg',
-  '.woff', '.woff2', '.ttf', '.eot', '.otf',
-  '.zip', '.tar', '.gz', '.rar', '.7z',
-  '.pdf', '.doc', '.docx', '.xls', '.xlsx',
-  '.mp3', '.mp4', '.avi', '.mov', '.wav',
-  '.map', '.lock',
-]);
-
-const SKIP_NAMES = new Set(['package-lock.json', 'yarn.lock', 'pnpm-lock.yaml']);
-
-function shouldSkip(filePath) {
-  if (!filePath) return true;
-  if (SKIP_NAMES.has(path.basename(filePath))) return true;
-  return SKIP_EXTS.has(path.extname(filePath).toLowerCase());
-}
-
-// ---------------------------------------------------------------------------
-// Response helpers
-// ---------------------------------------------------------------------------
-
-function deny(reason) {
-  process.stdout.write(JSON.stringify({
-    hookSpecificOutput: {
-      hookEventName: 'PreToolUse',
-      permissionDecision: 'deny',
-      permissionDecisionReason: reason,
-    },
-  }));
-  process.exit(2);
-}
-
-const allow = () => process.exit(0);
-
-// ---------------------------------------------------------------------------
-// Main
-// ---------------------------------------------------------------------------
-
-async function main() {
-  let payload;
-  try {
-    const chunks = [];
-    for await (const chunk of process.stdin) chunks.push(chunk);
-    payload = JSON.parse(Buffer.concat(chunks).toString('utf8'));
-  } catch {
-    allow(); // parse failure is non-blocking
-  }
-
-  const { tool_name, tool_input = {} } = payload;
-  const { file_path, content, old_string, new_string, replace_all } = tool_input;
-
-  if (tool_name === 'Write') {
-    if (shouldSkip(file_path)) allow();
-    const lines = countLines(content || '');
-    if (lines > MAX_LINES) {
-      deny(`[CodeWarden] File length gate: ${path.basename(file_path)} would be ${lines} lines (limit ${MAX_LINES}). Split into modules before writing.`);
-    }
-    allow();
-  }
-
-  if (tool_name === 'Edit') {
-    if (shouldSkip(file_path)) allow();
-    if (!fs.existsSync(file_path)) allow(); // new file — Write hook will catch it
-    const current = fs.readFileSync(file_path, 'utf8');
-    const patched  = replace_all
-      ? current.split(old_string || '').join(new_string || '')
-      : current.replace(old_string || '', new_string || '');
-    const lines = countLines(patched);
-    if (lines > MAX_LINES) {
-      deny(`[CodeWarden] File length gate: ${path.basename(file_path)} would be ${lines} lines after edit (limit ${MAX_LINES}). Split into modules before editing.`);
-    }
-    allow();
-  }
-
-  allow(); // unrecognised tool — pass through
-}
-
-main().catch(() => allow());
+#!/usr/bin/env node
+/**
+ * warden-lint-hook.js
+ * PreToolUse Claude Code hook: blocks Write/Edit if the resulting file would
+ * exceed the configured line limit.
+ *
+ * Payload (stdin JSON):  { tool_name, tool_input: { file_path, content|new_string, ... } }
+ * On violation: exit 2 + JSON deny response to stdout.
+ * On pass:      exit 0 (no output).
+ */
+
+'use strict';
+
+const fs   = require('fs');
+const path = require('path');
+const { countLines } = require('../../lib/line-count');
+const { loadConfig } = require('../../lib/config');
+
+// ---------------------------------------------------------------------------
+// Config — loaded via shared module; falls back to 400 if missing
+// ---------------------------------------------------------------------------
+
+const { maxFileLength: MAX_LINES } = loadConfig();
+
+// ---------------------------------------------------------------------------
+// Skip list — file types where line counting is meaningless
+// ---------------------------------------------------------------------------
+
+const SKIP_EXTS = new Set([
+  '.png', '.jpg', '.jpeg', '.gif', '.bmp', '.ico', '.svg',
+  '.woff', '.woff2', '.ttf', '.eot', '.otf',
+  '.zip', '.tar', '.gz', '.rar', '.7z',
+  '.pdf', '.doc', '.docx', '.xls', '.xlsx',
+  '.mp3', '.mp4', '.avi', '.mov', '.wav',
+  '.map', '.lock',
+]);
+
+const SKIP_NAMES = new Set(['package-lock.json', 'yarn.lock', 'pnpm-lock.yaml']);
+
+function shouldSkip(filePath) {
+  if (!filePath) return true;
+  if (SKIP_NAMES.has(path.basename(filePath))) return true;
+  return SKIP_EXTS.has(path.extname(filePath).toLowerCase());
+}
+
+// ---------------------------------------------------------------------------
+// Response helpers
+// ---------------------------------------------------------------------------
+
+function deny(reason) {
+  process.stdout.write(JSON.stringify({
+    hookSpecificOutput: {
+      hookEventName: 'PreToolUse',
+      permissionDecision: 'deny',
+      permissionDecisionReason: reason,
+    },
+  }));
+  process.exit(2);
+}
+
+const allow = () => process.exit(0);
+
+// ---------------------------------------------------------------------------
+// Main
+// ---------------------------------------------------------------------------
+
+async function main() {
+  let payload;
+  try {
+    const chunks = [];
+    for await (const chunk of process.stdin) chunks.push(chunk);
+    payload = JSON.parse(Buffer.concat(chunks).toString('utf8'));
+  } catch {
+    allow(); // parse failure is non-blocking
+  }
+
+  const { tool_name, tool_input = {} } = payload;
+  const { file_path, content, old_string, new_string, replace_all } = tool_input;
+
+  if (tool_name === 'Write') {
+    if (shouldSkip(file_path)) allow();
+    const lines = countLines(content || '');
+    if (lines > MAX_LINES) {
+      deny(`[CodeWarden] File length gate: ${path.basename(file_path)} would be ${lines} lines (limit ${MAX_LINES}). Split into modules before writing.`);
+    }
+    allow();
+  }
+
+  if (tool_name === 'Edit') {
+    if (shouldSkip(file_path)) allow();
+    if (!fs.existsSync(file_path)) allow(); // new file — Write hook will catch it
+    const current = fs.readFileSync(file_path, 'utf8');
+    const patched  = replace_all
+      ? current.split(old_string || '').join(new_string || '')
+      : current.replace(old_string || '', new_string || '');
+    const lines = countLines(patched);
+    if (lines > MAX_LINES) {
+      deny(`[CodeWarden] File length gate: ${path.basename(file_path)} would be ${lines} lines after edit (limit ${MAX_LINES}). Split into modules before editing.`);
+    }
+    allow();
+  }
+
+  allow(); // unrecognised tool — pass through
+}
+
+main().catch(() => allow());