@devrik-tools/claude-gates 0.3.1 → 0.6.0

package/plugins/tasks/hooks/register-requests.mjs

@@ -1,145 +1,145 @@
-// register-requests — UserPromptSubmit hook. Runs on every user message, BEFORE the
-// assistant responds. It is the deterministic half of the task system: it never decides
-// what is a task (that judgment is the assistant's — the hook only asks it to judge), it
-// only makes sure the judgment is asked and open tasks are not forgotten. Two jobs:
-//   1. On every message, ask the assistant to classify: if this message is a new task,
-//      register it via `claude-gates task add` (persistence is deterministic — the store —
-//      but the classification itself is the model's judgment, never a regex here).
-//   2. Every `remindEveryMessages` messages (counted via the store's counter), also recite
-//      the currently active tasks so a long or compacted conversation never loses the
-//      thread of what is still pending.
-//
-// A UserPromptSubmit hook's stdout is added to the assistant's context as plain text, so
-// both are simply written to stdout. Self-contained: Node built-ins + the task-store lib
-// only, no npm at runtime.
-//
-// justification: no existing tool covers this. The task store and the audit were done when
-// task-store.mjs was written (see its header); this file is the hook wiring on top.
-
-import { existsSync, readFileSync } from 'node:fs';
-import { homedir } from 'node:os';
-import { dirname, join } from 'node:path';
-import { openTaskStore } from './lib/task-store.mjs';
-
-const STDIN_FILE_DESCRIPTOR = 0;
-const CONFIG_KEY = 'remindOpenTasks';
-const DEFAULT_REMIND_EVERY_MESSAGES = 5;
-const MAX_TASKS_SHOWN = 12;
-const DUMP_ENV = 'CLAUDE_GATES_DUMP_DEFAULTS';
-
-// Config lookup mirrors the gates' config.mjs (project → global), kept local so this plugin
-// stays independent of the gates plugin.
-const PROJECT_ROOT_MARKERS = ['.git', '.ai'];
-const PROJECT_CONFIG = join('.ai', 'config.json');
-const GLOBAL_CONFIG = join('.claude', 'claude-gates', 'config.json');
-
-function readJson(path) {
-  try {
-    return JSON.parse(readFileSync(path, 'utf8'));
-  } catch {
-    return null;
-  }
-}
-
-function projectRootOf(startDirectory) {
-  let current = startDirectory;
-  while (true) {
-    if (
-      PROJECT_ROOT_MARKERS.some((marker) => existsSync(join(current, marker)))
-    ) {
-      return current;
-    }
-    const parent = dirname(current);
-    if (parent === current) return null;
-    current = parent;
-  }
-}
-
-/** The tasks gate config entry (project overrides global), or null. */
-function taskConfig(startDirectory) {
-  const root = projectRootOf(startDirectory);
-  const projectData = root ? readJson(join(root, PROJECT_CONFIG)) : null;
-  const globalData = readJson(join(homedir(), GLOBAL_CONFIG));
-  const layer = projectData?.gates ?? globalData?.gates ?? {};
-  const entry = layer[CONFIG_KEY];
-  if (typeof entry === 'boolean') return { enabled: entry };
-  if (entry && typeof entry === 'object') return entry;
-  return null;
-}
-
-function readPayload() {
-  try {
-    return JSON.parse(readFileSync(STDIN_FILE_DESCRIPTOR, 'utf8'));
-  } catch {
-    return {};
-  }
-}
-
-// Asked on every message. The hook does not classify — it asks the model to, and the
-// model is the one that persists (via the CLI, which enforces the store's own rules,
-// e.g. evidence on close). This is legitimate: the hook is not depending on the model to
-// make something OCCUR deterministically; it is depending on the model's judgment for a
-// judgment call (whether this message describes a new task), same shape as ask-adoption.
-const CLASSIFY_PROMPT =
-  '[tasks] MANDATORY before you respond: judge whether the message you just received describes ' +
-  'a new actionable task (not a question, not small talk, not a follow-up on an already-registered ' +
-  'task). If it does, you MUST register it now, before writing your reply — run ' +
-  '`claude-gates task add "<title>" [--description <text>] [--size <size>]` from the project root ' +
-  '(or the CLI\'s absolute path if `claude-gates` is not on PATH). Do not defer this, do not decide ' +
-  'to register it "later", do not silently skip it because the answer seems obvious. If it is not a ' +
-  'new task, skip registration — do not fabricate one.';
-
-function formatReminder(tasks) {
-  const shown = tasks.slice(0, MAX_TASKS_SHOWN);
-  const lines = shown.map(
-    (task) => `  - [${task.status}] ${task.id}: ${task.title}`,
-  );
-  const extra = tasks.length - shown.length;
-  if (extra > 0) lines.push(`  - …and ${extra} more`);
-  return (
-    '[tasks] OPEN TASKS — recite these pending tasks to the user and confirm whether any ' +
-    `should be dropped:\n${lines.join('\n')}`
-  );
-}
-
-function main() {
-  // Same defaults-dump protocol as the gates plugin's runGate (see hook-io.mjs): the CLI's
-  // materialize.mjs spawns this script with the env var set to read its built-in params as
-  // the single source of truth, before touching stdin.
-  if (process.env[DUMP_ENV]) {
-    process.stdout.write(
-      JSON.stringify({
-        id: 'remind-open-tasks',
-        configKey: CONFIG_KEY,
-        enabledByDefault: true,
-        defaultParams: { remindEveryMessages: DEFAULT_REMIND_EVERY_MESSAGES },
-      }),
-    );
-    return;
-  }
-
-  const payload = readPayload();
-  const cwd = payload.cwd || process.cwd();
-
-  const config = taskConfig(cwd);
-  if (config && config.enabled === false) return; // gate turned off for this project
-
-  const store = openTaskStore(cwd);
-  if (!store) return; // no project: nowhere to track
-
-  const lines = [CLASSIFY_PROMPT];
-
-  const tasks = store.active();
-  if (tasks.length > 0) {
-    const remindEveryMessages =
-      Number(config?.remindEveryMessages) || DEFAULT_REMIND_EVERY_MESSAGES;
-    const next = store.counter() + 1;
-    const forced = next >= remindEveryMessages;
-    store.setCounter(forced ? 0 : next);
-    if (forced) lines.push(formatReminder(tasks));
-  }
-
-  process.stdout.write(`${lines.join('\n')}\n`);
-}
-
-main();
+// register-requests — UserPromptSubmit hook. Runs on every user message, BEFORE the
+// assistant responds. It is the deterministic half of the task system: it never decides
+// what is a task (that judgment is the assistant's — the hook only asks it to judge), it
+// only makes sure the judgment is asked and open tasks are not forgotten. Two jobs:
+//   1. On every message, ask the assistant to classify: if this message is a new task,
+//      register it via `claude-gates task add` (persistence is deterministic — the store —
+//      but the classification itself is the model's judgment, never a regex here).
+//   2. Every `remindEveryMessages` messages (counted via the store's counter), also recite
+//      the currently active tasks so a long or compacted conversation never loses the
+//      thread of what is still pending.
+//
+// A UserPromptSubmit hook's stdout is added to the assistant's context as plain text, so
+// both are simply written to stdout. Self-contained: Node built-ins + the task-store lib
+// only, no npm at runtime.
+//
+// justification: no existing tool covers this. The task store and the audit were done when
+// task-store.mjs was written (see its header); this file is the hook wiring on top.
+
+import { existsSync, readFileSync } from 'node:fs';
+import { homedir } from 'node:os';
+import { dirname, join } from 'node:path';
+import { openTaskStore } from './lib/task-store.mjs';
+
+const STDIN_FILE_DESCRIPTOR = 0;
+const CONFIG_KEY = 'remindOpenTasks';
+const DEFAULT_REMIND_EVERY_MESSAGES = 5;
+const MAX_TASKS_SHOWN = 12;
+const DUMP_ENV = 'CLAUDE_GATES_DUMP_DEFAULTS';
+
+// Config lookup mirrors the gates' config.mjs (project → global), kept local so this plugin
+// stays independent of the gates plugin.
+const PROJECT_ROOT_MARKERS = ['.git', '.ai'];
+const PROJECT_CONFIG = join('.ai', 'config.json');
+const GLOBAL_CONFIG = join('.claude', 'claude-gates', 'config.json');
+
+function readJson(path) {
+  try {
+    return JSON.parse(readFileSync(path, 'utf8'));
+  } catch {
+    return null;
+  }
+}
+
+function projectRootOf(startDirectory) {
+  let current = startDirectory;
+  while (true) {
+    if (
+      PROJECT_ROOT_MARKERS.some((marker) => existsSync(join(current, marker)))
+    ) {
+      return current;
+    }
+    const parent = dirname(current);
+    if (parent === current) return null;
+    current = parent;
+  }
+}
+
+/** The tasks gate config entry (project overrides global), or null. */
+function taskConfig(startDirectory) {
+  const root = projectRootOf(startDirectory);
+  const projectData = root ? readJson(join(root, PROJECT_CONFIG)) : null;
+  const globalData = readJson(join(homedir(), GLOBAL_CONFIG));
+  const layer = projectData?.gates ?? globalData?.gates ?? {};
+  const entry = layer[CONFIG_KEY];
+  if (typeof entry === 'boolean') return { enabled: entry };
+  if (entry && typeof entry === 'object') return entry;
+  return null;
+}
+
+function readPayload() {
+  try {
+    return JSON.parse(readFileSync(STDIN_FILE_DESCRIPTOR, 'utf8'));
+  } catch {
+    return {};
+  }
+}
+
+// Asked on every message. The hook does not classify — it asks the model to, and the
+// model is the one that persists (via the CLI, which enforces the store's own rules,
+// e.g. evidence on close). This is legitimate: the hook is not depending on the model to
+// make something OCCUR deterministically; it is depending on the model's judgment for a
+// judgment call (whether this message describes a new task), same shape as ask-adoption.
+const CLASSIFY_PROMPT =
+  '[tasks] MANDATORY before you respond: judge whether the message you just received describes ' +
+  'a new actionable task (not a question, not small talk, not a follow-up on an already-registered ' +
+  'task). If it does, you MUST register it now, before writing your reply — run ' +
+  '`claude-gates task add "<title>" [--description <text>] [--size <size>]` from the project root ' +
+  "(or the CLI's absolute path if `claude-gates` is not on PATH). Do not defer this, do not decide " +
+  'to register it "later", do not silently skip it because the answer seems obvious. If it is not a ' +
+  'new task, skip registration — do not fabricate one.';
+
+function formatReminder(tasks) {
+  const shown = tasks.slice(0, MAX_TASKS_SHOWN);
+  const lines = shown.map(
+    (task) => `  - [${task.status}] ${task.id}: ${task.title}`,
+  );
+  const extra = tasks.length - shown.length;
+  if (extra > 0) lines.push(`  - …and ${extra} more`);
+  return (
+    '[tasks] OPEN TASKS — recite these pending tasks to the user and confirm whether any ' +
+    `should be dropped:\n${lines.join('\n')}`
+  );
+}
+
+function main() {
+  // Same defaults-dump protocol as the gates plugin's runGate (see hook-io.mjs): the CLI's
+  // materialize.mjs spawns this script with the env var set to read its built-in params as
+  // the single source of truth, before touching stdin.
+  if (process.env[DUMP_ENV]) {
+    process.stdout.write(
+      JSON.stringify({
+        id: 'remind-open-tasks',
+        configKey: CONFIG_KEY,
+        enabledByDefault: true,
+        defaultParams: { remindEveryMessages: DEFAULT_REMIND_EVERY_MESSAGES },
+      }),
+    );
+    return;
+  }
+
+  const payload = readPayload();
+  const cwd = payload.cwd || process.cwd();
+
+  const config = taskConfig(cwd);
+  if (config && config.enabled === false) return; // gate turned off for this project
+
+  const store = openTaskStore(cwd);
+  if (!store) return; // no project: nowhere to track
+
+  const lines = [CLASSIFY_PROMPT];
+
+  const tasks = store.active();
+  if (tasks.length > 0) {
+    const remindEveryMessages =
+      Number(config?.remindEveryMessages) || DEFAULT_REMIND_EVERY_MESSAGES;
+    const next = store.counter() + 1;
+    const forced = next >= remindEveryMessages;
+    store.setCounter(forced ? 0 : next);
+    if (forced) lines.push(formatReminder(tasks));
+  }
+
+  process.stdout.write(`${lines.join('\n')}\n`);
+}
+
+main();

package/plugins/tasks/hooks/session-tasks.mjs

@@ -1,108 +1,108 @@
-// session-tasks — SessionStart hook. Runs once when a session opens. Lists this project's
-// ACTIVE tasks (open/blocked/in_forge) so a fresh session (or one that just compacted) does
-// not lose track of what was already registered. Silent when there is nothing active: this
-// hook only speaks when it has something worth saying.
-//
-// Self-contained: Node built-ins + the task-store lib only, no npm at runtime.
-//
-// justification: no existing tool covers this. task-store.mjs (and its audit) already
-// exists; this is the SessionStart wiring on top of it, mirroring register-requests.mjs's
-// UserPromptSubmit wiring for the same store.
-
-import { existsSync, readFileSync } from 'node:fs';
-import { homedir } from 'node:os';
-import { dirname, join } from 'node:path';
-import { openTaskStore, STATUS } from './lib/task-store.mjs';
-
-const STDIN_FILE_DESCRIPTOR = 0;
-const SESSION_START_EVENT = 'SessionStart';
-const CONFIG_KEY = 'listTasksOnSessionStart';
-const MAX_TASKS_SHOWN = 20;
-
-// Same config lookup as register-requests.mjs: project overrides global. Kept local so this
-// plugin stays independent of the gates plugin.
-const PROJECT_ROOT_MARKERS = ['.git', '.ai'];
-const PROJECT_CONFIG = join('.ai', 'config.json');
-const GLOBAL_CONFIG = join('.claude', 'claude-gates', 'config.json');
-
-function readJson(path) {
-  try {
-    return JSON.parse(readFileSync(path, 'utf8'));
-  } catch {
-    return null;
-  }
-}
-
-function projectRootOf(startDirectory) {
-  let current = startDirectory;
-  while (true) {
-    if (
-      PROJECT_ROOT_MARKERS.some((marker) => existsSync(join(current, marker)))
-    ) {
-      return current;
-    }
-    const parent = dirname(current);
-    if (parent === current) return null;
-    current = parent;
-  }
-}
-
-/** The tasks gate config entry (project overrides global), or null. */
-function sessionTasksConfig(startDirectory) {
-  const root = projectRootOf(startDirectory);
-  const projectData = root ? readJson(join(root, PROJECT_CONFIG)) : null;
-  const globalData = readJson(join(homedir(), GLOBAL_CONFIG));
-  const layer = projectData?.gates ?? globalData?.gates ?? {};
-  const entry = layer[CONFIG_KEY];
-  if (typeof entry === 'boolean') return { enabled: entry };
-  if (entry && typeof entry === 'object') return entry;
-  return null;
-}
-
-function readPayload() {
-  try {
-    return JSON.parse(readFileSync(STDIN_FILE_DESCRIPTOR, 'utf8'));
-  } catch {
-    return {};
-  }
-}
-
-function formatActiveTasks(tasks) {
-  const shown = tasks.slice(0, MAX_TASKS_SHOWN);
-  const lines = shown.map((task) => {
-    const marker = task.status === STATUS.IN_FORGE ? ' (in_forge)' : '';
-    return `  - ${task.id}: ${task.title} [${task.status}]${marker}`;
-  });
-  const extra = tasks.length - shown.length;
-  if (extra > 0) lines.push(`  - …and ${extra} more`);
-  return `[tasks] Active tasks for this project:\n${lines.join('\n')}`;
-}
-
-function speak(context) {
-  process.stdout.write(
-    JSON.stringify({
-      hookSpecificOutput: {
-        hookEventName: SESSION_START_EVENT,
-        additionalContext: context,
-      },
-    }),
-  );
-}
-
-function main() {
-  const payload = readPayload();
-  const cwd = payload.cwd || process.cwd();
-
-  const config = sessionTasksConfig(cwd);
-  if (config && config.enabled === false) return; // gate turned off for this project
-
-  const store = openTaskStore(cwd);
-  if (!store) return; // no project: nowhere to track
-
-  const tasks = store.active();
-  if (tasks.length === 0) return; // nothing active: stay silent
-
-  speak(formatActiveTasks(tasks));
-}
-
-main();
+// session-tasks — SessionStart hook. Runs once when a session opens. Lists this project's
+// ACTIVE tasks (open/blocked/in_forge) so a fresh session (or one that just compacted) does
+// not lose track of what was already registered. Silent when there is nothing active: this
+// hook only speaks when it has something worth saying.
+//
+// Self-contained: Node built-ins + the task-store lib only, no npm at runtime.
+//
+// justification: no existing tool covers this. task-store.mjs (and its audit) already
+// exists; this is the SessionStart wiring on top of it, mirroring register-requests.mjs's
+// UserPromptSubmit wiring for the same store.
+
+import { existsSync, readFileSync } from 'node:fs';
+import { homedir } from 'node:os';
+import { dirname, join } from 'node:path';
+import { openTaskStore, STATUS } from './lib/task-store.mjs';
+
+const STDIN_FILE_DESCRIPTOR = 0;
+const SESSION_START_EVENT = 'SessionStart';
+const CONFIG_KEY = 'listTasksOnSessionStart';
+const MAX_TASKS_SHOWN = 20;
+
+// Same config lookup as register-requests.mjs: project overrides global. Kept local so this
+// plugin stays independent of the gates plugin.
+const PROJECT_ROOT_MARKERS = ['.git', '.ai'];
+const PROJECT_CONFIG = join('.ai', 'config.json');
+const GLOBAL_CONFIG = join('.claude', 'claude-gates', 'config.json');
+
+function readJson(path) {
+  try {
+    return JSON.parse(readFileSync(path, 'utf8'));
+  } catch {
+    return null;
+  }
+}
+
+function projectRootOf(startDirectory) {
+  let current = startDirectory;
+  while (true) {
+    if (
+      PROJECT_ROOT_MARKERS.some((marker) => existsSync(join(current, marker)))
+    ) {
+      return current;
+    }
+    const parent = dirname(current);
+    if (parent === current) return null;
+    current = parent;
+  }
+}
+
+/** The tasks gate config entry (project overrides global), or null. */
+function sessionTasksConfig(startDirectory) {
+  const root = projectRootOf(startDirectory);
+  const projectData = root ? readJson(join(root, PROJECT_CONFIG)) : null;
+  const globalData = readJson(join(homedir(), GLOBAL_CONFIG));
+  const layer = projectData?.gates ?? globalData?.gates ?? {};
+  const entry = layer[CONFIG_KEY];
+  if (typeof entry === 'boolean') return { enabled: entry };
+  if (entry && typeof entry === 'object') return entry;
+  return null;
+}
+
+function readPayload() {
+  try {
+    return JSON.parse(readFileSync(STDIN_FILE_DESCRIPTOR, 'utf8'));
+  } catch {
+    return {};
+  }
+}
+
+function formatActiveTasks(tasks) {
+  const shown = tasks.slice(0, MAX_TASKS_SHOWN);
+  const lines = shown.map((task) => {
+    const marker = task.status === STATUS.IN_FORGE ? ' (in_forge)' : '';
+    return `  - ${task.id}: ${task.title} [${task.status}]${marker}`;
+  });
+  const extra = tasks.length - shown.length;
+  if (extra > 0) lines.push(`  - …and ${extra} more`);
+  return `[tasks] Active tasks for this project:\n${lines.join('\n')}`;
+}
+
+function speak(context) {
+  process.stdout.write(
+    JSON.stringify({
+      hookSpecificOutput: {
+        hookEventName: SESSION_START_EVENT,
+        additionalContext: context,
+      },
+    }),
+  );
+}
+
+function main() {
+  const payload = readPayload();
+  const cwd = payload.cwd || process.cwd();
+
+  const config = sessionTasksConfig(cwd);
+  if (config && config.enabled === false) return; // gate turned off for this project
+
+  const store = openTaskStore(cwd);
+  if (!store) return; // no project: nowhere to track
+
+  const tasks = store.active();
+  if (tasks.length === 0) return; // nothing active: stay silent
+
+  speak(formatActiveTasks(tasks));
+}
+
+main();