chati-dev 4.2.1 → 4.3.0

package/framework/hooks/post-dev.js

@@ -0,0 +1,207 @@
+#!/usr/bin/env node
+/**
+ * Post-Dev Auto-QA Hook — PostToolUse (runs after Write tool calls)
+ *
+ * Autonomous-completion v1 (ADR-AUTO-06). When dev writes its handoff file
+ * with status=completed and score >= threshold, this hook spawns
+ * visual-qa.js detached — 6 viewports + reduced-motion pass — so the next
+ * `/chati` turn finds .chati/visual-qa/report.json already populated.
+ *
+ * The user never types `pnpm build && pnpm start`. visual-qa.js owns its
+ * own dev-server (`npx next dev -p <port>` unless --no-server); this hook
+ * just spawns it with the right arguments and exits.
+ *
+ * Opt-out: set `auto_visual_qa: false` in `chati.dev/config.yaml` (or
+ * `.chati.dev/config.yaml` for installed projects). Default is true — the
+ * zero-commands thesis.
+ *
+ * Trigger condition:
+ *   Write tool wrote `artifacts/handoffs/dev-handoff.md` with frontmatter
+ *   `status: completed` AND `score: >= 90`.
+ *
+ * Observational: always returns `{}` — never blocks the Write call. The
+ * spawn is detached with stdio ignored so it outlives this hook process.
+ */
+
+import { existsSync, readFileSync } from 'fs';
+import { join } from 'path';
+import { spawn } from 'child_process';
+
+export const DEV_SCORE_THRESHOLD = 90;
+
+// ---------------------------------------------------------------------------
+// Config loader — checks both installed (.chati.dev) and dev (chati.dev).
+// ---------------------------------------------------------------------------
+
+export function readAutoVisualQaConfig(projectDir) {
+  const candidates = [
+    join(projectDir, '.chati.dev', 'config.yaml'),
+    join(projectDir, 'chati.dev', 'config.yaml'),
+  ];
+  for (const p of candidates) {
+    if (!existsSync(p)) continue;
+    try {
+      const raw = readFileSync(p, 'utf-8');
+      const match = raw.match(/^\s*auto_visual_qa:\s*(true|false)\s*$/m);
+      if (match) return match[1] === 'true';
+    } catch { /* fall through — config unreadable, use default */ }
+  }
+  return true; // default ON per ADR-AUTO-06
+}
+
+// ---------------------------------------------------------------------------
+// Handoff frontmatter extraction.
+// ---------------------------------------------------------------------------
+
+export function extractHandoffMeta(markdown) {
+  if (!markdown || !markdown.startsWith('---')) return null;
+  const end = markdown.indexOf('\n---', 3);
+  if (end === -1) return null;
+  const fm = markdown.slice(3, end + 1);
+  const meta = {};
+  for (const line of fm.split('\n')) {
+    const m = line.match(/^([a-zA-Z_][\w-]*):\s*(.+)$/);
+    if (m) meta[m[1].trim()] = m[2].trim().replace(/^["']|["']$/g, '');
+  }
+  return meta;
+}
+
+// ---------------------------------------------------------------------------
+// Route discovery (duplicate of cli.js discoverAppRoutes — hooks cannot
+// import package modules).
+// ---------------------------------------------------------------------------
+
+import { readdirSync } from 'fs';
+
+export function discoverRoutes(projectDir) {
+  const candidates = [join(projectDir, 'app'), join(projectDir, 'src', 'app')];
+  const appDir = candidates.find(d => existsSync(d));
+  if (!appDir) return ['/'];
+  const routes = [];
+  const PAGE_RE = /^page\.(tsx|jsx|ts|js|mdx)$/;
+  function walk(dir, segments) {
+    let entries;
+    try { entries = readdirSync(dir, { withFileTypes: true }); } catch { return; }
+    for (const e of entries) {
+      if (e.isDirectory()) {
+        const name = e.name;
+        if (name.startsWith('_')) continue;
+        if (name.startsWith('[')) continue;
+        const isGroup = name.startsWith('(') && name.endsWith(')');
+        const nextSegments = isGroup ? segments : [...segments, name];
+        walk(join(dir, e.name), nextSegments);
+      } else if (e.isFile() && PAGE_RE.test(e.name)) {
+        routes.push(segments.length === 0 ? '/' : '/' + segments.join('/'));
+      }
+    }
+  }
+  walk(appDir, []);
+  const unique = [...new Set(routes)].sort();
+  return unique.length > 0 ? unique : ['/'];
+}
+
+// ---------------------------------------------------------------------------
+// Spawn decision — returns the command + args to run, or null when auto-QA
+// should not fire. Pure for testability.
+// ---------------------------------------------------------------------------
+
+/**
+ * @param {{ projectDir: string, toolName: string, filePath: string }} ctx
+ * @returns {{ command: string, args: string[] } | null}
+ */
+export function decideSpawn(ctx) {
+  if (ctx.toolName !== 'Write') return null;
+  if (!ctx.filePath) return null;
+
+  // Match only dev-handoff.md in the canonical paths.
+  const handoffMatch = ctx.filePath.match(/artifacts\/handoffs\/dev-handoff\.md$/);
+  if (!handoffMatch) return null;
+
+  // Read the handoff to confirm status=completed and score passes threshold.
+  let meta = null;
+  try {
+    const raw = readFileSync(ctx.filePath, 'utf-8');
+    meta = extractHandoffMeta(raw);
+  } catch { return null; }
+  if (!meta) return null;
+  if (meta.status !== 'completed') return null;
+  const score = parseInt(meta.score, 10);
+  if (!Number.isFinite(score) || score < DEV_SCORE_THRESHOLD) return null;
+
+  // Config opt-out wins.
+  if (!readAutoVisualQaConfig(ctx.projectDir)) return null;
+
+  // Resolve script path.
+  const scriptCandidates = [
+    join(ctx.projectDir, '.chati.dev', 'scripts', 'visual-qa.js'),
+    join(ctx.projectDir, 'chati.dev', 'scripts', 'visual-qa.js'),
+  ];
+  const script = scriptCandidates.find(existsSync);
+  if (!script) return null;
+
+  const routes = discoverRoutes(ctx.projectDir);
+  const output = join(ctx.projectDir, '.chati', 'visual-qa');
+
+  return {
+    command: 'node',
+    args: [
+      script,
+      '--pages', ...routes,
+      '--output', output,
+      '--reduced-motion', 'on', // accessibility pass; gate in P6 reads this
+    ],
+  };
+}
+
+// ---------------------------------------------------------------------------
+// Hook entry point.
+// ---------------------------------------------------------------------------
+
+async function readStdin() {
+  let data = '';
+  for await (const chunk of process.stdin) data += chunk;
+  return data;
+}
+
+async function main() {
+  let input;
+  try {
+    input = JSON.parse(await readStdin() || '{}');
+  } catch {
+    process.stdout.write('{}\n');
+    return;
+  }
+
+  const ctx = {
+    projectDir: input.cwd || process.cwd(),
+    toolName:   input.tool_name || '',
+    filePath:   (input.tool_input && input.tool_input.file_path) || '',
+  };
+
+  // Observational default — the hook never blocks the Write call.
+  process.stdout.write('{}\n');
+
+  const plan = decideSpawn(ctx);
+  if (!plan) return;
+
+  try {
+    const child = spawn(plan.command, plan.args, {
+      cwd:      ctx.projectDir,
+      detached: true,
+      stdio:    'ignore',
+    });
+    child.unref();
+  } catch (err) {
+    process.stderr.write(`[chati] post-dev spawn failed: ${err.message}\n`);
+  }
+}
+
+import { fileURLToPath } from 'url';
+import { realpathSync } from 'fs';
+if (process.argv[1]) {
+  try {
+    if (realpathSync(process.argv[1]) === realpathSync(fileURLToPath(import.meta.url))) {
+      main();
+    }
+  } catch { /* not invoked as a script */ }
+}