@massu/core 1.10.8 → 1.12.0

package/src/hooks/post-tool-use.ts

@@ -1,413 +0,0 @@
-#!/usr/bin/env node
-// Copyright (c) 2026 Massu. All rights reserved.
-// Licensed under BSL 1.1 - see LICENSE file for details.
-
-// ============================================================
-// P3-002: PostToolUse Observation Hook
-// Captures tool usage as observations (lightweight, no AI needed).
-// Must complete in <500ms.
-// ============================================================
-
-import { getMemoryDb, addObservation, createSession, deduplicateFailedAttempt, addSummary } from '../memory-db.ts';
-import { classifyRealTimeToolCall, detectPlanProgress } from '../observation-extractor.ts';
-import { logAuditEntry } from '../audit-trail.ts';
-import { trackModification, recordTestResult } from '../regression-detector.ts';
-import { validateFile, storeValidationResult } from '../validation-engine.ts';
-import { scoreFileSecurity, storeSecurityScore } from '../security-scorer.ts';
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { parse as parseYaml } from 'yaml';
-import { ingestMemoryFile } from '../memory-file-ingest.ts';
-
-interface HookInput {
-  session_id: string;
-  transcript_path: string;
-  cwd: string;
-  hook_event_name: string;
-  tool_name: string;
-  tool_input: Record<string, unknown>;
-  tool_response: string;
-}
-
-// In-memory dedup for Read calls within this session
-const seenReads = new Set<string>();
-let currentSessionId: string | null = null;
-
-async function main(): Promise<void> {
-  try {
-    const input = await readStdin();
-    const hookInput = JSON.parse(input) as HookInput;
-    const { session_id, tool_name, tool_input, tool_response } = hookInput;
-
-    // Reset seen reads if session changed
-    if (currentSessionId !== session_id) {
-      seenReads.clear();
-      currentSessionId = session_id;
-    }
-
-    const db = getMemoryDb();
-    try {
-      // Ensure session exists
-      createSession(db, session_id);
-
-      // Classify and filter
-      const observation = classifyRealTimeToolCall(tool_name, tool_input, tool_response, seenReads);
-      if (!observation) {
-        process.exit(0);
-        return;
-      }
-
-      // Deduplicate failed attempts
-      if (observation.type === 'failed_attempt') {
-        deduplicateFailedAttempt(db, session_id, observation.title, observation.detail, observation.opts);
-      } else {
-        addObservation(db, session_id, observation.type, observation.title, observation.detail, observation.opts);
-      }
-
-      // Auto-detect plan progress
-      if (tool_response) {
-        const progress = detectPlanProgress(tool_response);
-        if (progress.length > 0) {
-          // Update plan_progress in session summary
-          updatePlanProgress(db, session_id, progress);
-        }
-      }
-
-      // Audit trail logging for file changes
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          logAuditEntry(db, {
-            sessionId: session_id,
-            eventType: 'code_change',
-            actor: 'ai',
-            filePath,
-            changeType: tool_name === 'Write' ? 'create' : 'edit',
-          });
-
-          // Track modification for regression detection
-          if (filePath) {
-            const featureMatch = filePath.match(/(?:routers|components|app\/\(([^)]+)\))\/([^/.]+)/);
-            if (featureMatch) {
-              const featureKey = featureMatch[1] ?? featureMatch[2];
-              trackModification(db, featureKey);
-            }
-          }
-        }
-      } catch (_auditErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // Real-time validation for Edit/Write
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          if (filePath && (filePath.endsWith('.ts') || filePath.endsWith('.tsx'))) {
-            const projectRoot = hookInput.cwd;
-            const checks = validateFile(filePath, projectRoot);
-            const violations = checks.filter(c => c.severity === 'error' || c.severity === 'critical');
-            if (violations.length > 0) {
-              storeValidationResult(db, filePath, checks, session_id);
-            }
-          }
-        }
-      } catch (_validationErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // Auto-security scoring for router/API files
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          if (filePath && (filePath.includes('routers/') || filePath.includes('api/'))) {
-            const projectRoot = hookInput.cwd;
-            const { riskScore, findings } = scoreFileSecurity(filePath, projectRoot);
-            if (findings.length > 0) {
-              storeSecurityScore(db, session_id, filePath, riskScore, findings);
-            }
-          }
-        }
-      } catch (_securityErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // P-H029 (plan-stage-c-high-batch): wire recordTestResult() into the
-      // post-tool-use hook so `feature_health` dashboard reflects real test
-      // deltas. Pre-fix: `trackModification` fired but `recordTestResult` was
-      // unit-tested yet never called; dashboard showed tests_passing=0 for
-      // every feature.
-      //
-      // Strategy: when a Bash tool call runs a test runner AND the output
-      // includes parseable pass/fail counts, call recordTestResult for every
-      // feature with `modifications_since_test > 0`. This resets their counter
-      // and updates pass/fail tallies based on the run.
-      try {
-        if (tool_name === 'Bash') {
-          const command = (tool_input.command as string) ?? '';
-          if (isTestRunnerCommand(command)) {
-            const counts = parseTestRunOutput(tool_response ?? '');
-            if (counts) {
-              const modifiedFeatures = db
-                .prepare(
-                  'SELECT feature_key FROM feature_health WHERE modifications_since_test > 0',
-                )
-                .all() as Array<{ feature_key: string }>;
-              for (const row of modifiedFeatures) {
-                recordTestResult(db, row.feature_key, counts.passing, counts.failing);
-              }
-              // Also record a session-level aggregate so the dashboard has at
-              // least one row even when no features were modified.
-              if (modifiedFeatures.length === 0) {
-                recordTestResult(db, '_session_test_run', counts.passing, counts.failing);
-              }
-            }
-          }
-        }
-      } catch (_testResultErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // MEMORY.md integrity check on write
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          if (filePath && filePath.endsWith('MEMORY.md') && filePath.includes('/memory/')) {
-            const issues = checkMemoryFileIntegrity(filePath);
-            if (issues.length > 0) {
-              addObservation(db, session_id, 'incident_near_miss',
-                'MEMORY.md integrity issue detected',
-                issues.join('; '),
-                { importance: 4 }
-              );
-            }
-          }
-        }
-      } catch (_memoryErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // Memory file auto-ingest: when Claude writes a memory/*.md file,
-      // parse frontmatter and ingest into observations table
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          if (filePath && filePath.includes('/memory/') && filePath.endsWith('.md')) {
-            const basename = filePath.split('/').pop() ?? '';
-            if (basename !== 'MEMORY.md') {
-              ingestMemoryFile(db, session_id, filePath);
-            }
-          }
-        }
-      } catch (_memoryIngestErr) {
-        // Best-effort: never block post-tool-use
-      }
-
-      // Knowledge index staleness check on knowledge file edits
-      try {
-        if (tool_name === 'Edit' || tool_name === 'Write') {
-          const filePath = (tool_input.file_path as string) ?? '';
-          if (filePath && isKnowledgeSourceFile(filePath)) {
-            addObservation(db, session_id, 'discovery',
-              'Knowledge source file modified - index may be stale',
-              `Edited ${filePath.split('/').pop() ?? filePath}. Run knowledge re-index to update.`,
-              { importance: 3 }
-            );
-          }
-        }
-      } catch (_knowledgeErr) {
-        // Best-effort: never block post-tool-use
-      }
-    } finally {
-      db.close();
-    }
-  } catch (_e) {
-    // Best-effort: never block Claude Code
-  }
-  process.exit(0);
-}
-
-function updatePlanProgress(db: import('better-sqlite3').Database, sessionId: string, progress: Array<{ planItem: string; status: string }>): void {
-  // Get or create latest summary's plan_progress
-  const existing = db.prepare(
-    'SELECT id, plan_progress FROM session_summaries WHERE session_id = ? ORDER BY created_at_epoch DESC LIMIT 1'
-  ).get(sessionId) as { id: number; plan_progress: string } | undefined;
-
-  if (existing) {
-    try {
-      const currentProgress = JSON.parse(existing.plan_progress) as Record<string, string>;
-      for (const p of progress) {
-        currentProgress[p.planItem] = p.status;
-      }
-      db.prepare('UPDATE session_summaries SET plan_progress = ? WHERE id = ?')
-        .run(JSON.stringify(currentProgress), existing.id);
-    } catch (_e) {
-      // Skip if JSON parse fails
-    }
-  } else {
-    // Create a minimal summary with plan progress
-    const progressMap: Record<string, string> = {};
-    for (const p of progress) {
-      progressMap[p.planItem] = p.status;
-    }
-    addSummary(db, sessionId, { planProgress: progressMap });
-  }
-}
-
-/**
- * P-H029: Detect test-runner commands. Conservative match — only commands that
- * START with a recognized test runner so a build script invoking `npm run test`
- * is included but a script that merely mentions "test" in a filename is not.
- */
-function isTestRunnerCommand(command: string): boolean {
-  const trimmed = command.trim().toLowerCase();
-  // Strip leading `cd <dir> && ` or `(cd <dir> && ...)` prefix so we match
-  // the actual test command.
-  const stripped = trimmed
-    .replace(/^cd\s+\S+\s*(&&|;)\s*/, '')
-    .replace(/^\(\s*cd\s+\S+\s*(&&|;)\s*/, '');
-  const testRunnerPrefixes = [
-    'npm test', 'npm run test', 'npx vitest', 'npx jest', 'vitest', 'jest',
-    'pnpm test', 'pnpm run test', 'yarn test', 'pytest', 'go test', 'cargo test',
-  ];
-  return testRunnerPrefixes.some((prefix) => stripped.startsWith(prefix));
-}
-
-/**
- * P-H029: Parse test-run output for pass/fail counts. Supports vitest
- * (`Tests  N passed (N)`, `Tests  X failed | Y passed (Z)`), jest
- * (`Tests: X failed, Y passed, Z total`), and pytest (`X passed, Y failed`).
- * Returns null if no parseable summary line found.
- */
-function parseTestRunOutput(output: string): { passing: number; failing: number } | null {
-  // vitest: " Tests  439 passed (439)" or " Tests  3 failed | 436 passed (439)"
-  const vitestSplit = output.match(/Tests?\s+(?:(\d+)\s+failed\s+\|\s+)?(\d+)\s+passed/);
-  if (vitestSplit) {
-    return {
-      passing: parseInt(vitestSplit[2], 10),
-      failing: vitestSplit[1] ? parseInt(vitestSplit[1], 10) : 0,
-    };
-  }
-  // jest: "Tests:       1 failed, 5 passed, 6 total"
-  const jest = output.match(/Tests?:\s+(?:(\d+)\s+failed,\s+)?(\d+)\s+passed/);
-  if (jest) {
-    return {
-      passing: parseInt(jest[2], 10),
-      failing: jest[1] ? parseInt(jest[1], 10) : 0,
-    };
-  }
-  // pytest: "5 passed, 2 failed in 1.23s" or "5 passed in 1.23s"
-  const pytestPassed = output.match(/(\d+)\s+passed/);
-  const pytestFailed = output.match(/(\d+)\s+failed/);
-  if (pytestPassed) {
-    return {
-      passing: parseInt(pytestPassed[1], 10),
-      failing: pytestFailed ? parseInt(pytestFailed[1], 10) : 0,
-    };
-  }
-  return null;
-}
-
-function readStdin(): Promise<string> {
-  return new Promise((resolve) => {
-    let data = '';
-    process.stdin.setEncoding('utf-8');
-    process.stdin.on('data', (chunk: string) => { data += chunk; });
-    process.stdin.on('end', () => resolve(data));
-    setTimeout(() => resolve(data), 3000);
-  });
-}
-
-/**
- * Read the conventions section from massu.config.yaml directly.
- * Hooks are compiled with esbuild and cannot use getConfig() from config.ts.
- * Falls back to sensible defaults if the config file is not found.
- */
-function readConventions(cwd?: string): {
-  knowledgeSourceFiles: string[];
-  claudeDirName: string;
-} {
-  const defaults = {
-    knowledgeSourceFiles: ['CLAUDE.md', 'MEMORY.md', 'corrections.md'],
-    claudeDirName: '.claude',
-  };
-  try {
-    const projectRoot = cwd ?? process.cwd();
-    const configPath = join(projectRoot, 'massu.config.yaml');
-    if (!existsSync(configPath)) return defaults;
-    const content = readFileSync(configPath, 'utf-8');
-    // pattern-scanner-allow: yaml-parse — reason: compiled standalone hook (esbuild bundle). Per P2-023a, hooks cannot import getConfig() — they run in the Claude Code subprocess context with no module resolution path back to packages/core. Direct YAML parse is the only available access pattern.
-    const parsed = parseYaml(content) as Record<string, unknown> | null;
-    if (!parsed || typeof parsed !== 'object') return defaults;
-    const conventions = parsed.conventions as Record<string, unknown> | undefined;
-    if (!conventions || typeof conventions !== 'object') return defaults;
-    return {
-      knowledgeSourceFiles: Array.isArray(conventions.knowledgeSourceFiles)
-        ? conventions.knowledgeSourceFiles as string[]
-        : defaults.knowledgeSourceFiles,
-      claudeDirName: typeof conventions.claudeDirName === 'string'
-        ? conventions.claudeDirName
-        : defaults.claudeDirName,
-    };
-  } catch {
-    return defaults;
-  }
-}
-
-/**
- * Check if a file path is a knowledge source file (CLAUDE.md, corrections.md,
- * memory files, or knowledge system source files).
- * When these are edited, the knowledge index may become stale.
- */
-function isKnowledgeSourceFile(filePath: string): boolean {
-  const basename = filePath.split('/').pop() ?? '';
-  const conventions = readConventions();
-  const knowledgeSourcePatterns = [
-    ...conventions.knowledgeSourceFiles,
-    'file-index.md',
-    'knowledge-db.ts',
-    'knowledge-indexer.ts',
-    'knowledge-tools.ts',
-  ];
-  return knowledgeSourcePatterns.some(p => basename === p) ||
-    filePath.includes('/memory/') ||
-    filePath.includes(conventions.claudeDirName + '/');
-}
-
-/**
- * Check MEMORY.md file integrity after a write.
- * Verifies: file exists, has expected structure, and is under line limit.
- * Returns array of issue descriptions (empty = all good).
- */
-function checkMemoryFileIntegrity(filePath: string): string[] {
-  const issues: string[] = [];
-
-  try {
-    if (!existsSync(filePath)) {
-      issues.push('MEMORY.md file does not exist after write');
-      return issues;
-    }
-
-    const content = readFileSync(filePath, 'utf-8');
-    const lines = content.split('\n');
-
-    // Check line count (CLAUDE.md truncates after ~200 lines)
-    const MAX_LINES = 200;
-    if (lines.length > MAX_LINES) {
-      issues.push(`MEMORY.md exceeds ${MAX_LINES} lines (currently ${lines.length}). Consider archiving old entries.`);
-    }
-
-    // Check required structure sections
-    const requiredSections = ['# Massu Memory', '## Key Learnings', '## Common Gotchas'];
-    for (const section of requiredSections) {
-      if (!content.includes(section)) {
-        issues.push(`Missing required section: "${section}"`);
-      }
-    }
-  } catch (_e) {
-    // Graceful degradation: don't report issues if we can't check
-  }
-
-  return issues;
-}
-
-main();

package/src/hooks/pre-compact.ts

@@ -1,168 +0,0 @@
-#!/usr/bin/env node
-// Copyright (c) 2026 Massu. All rights reserved.
-// Licensed under BSL 1.1 - see LICENSE file for details.
-
-// ============================================================
-// P3-006: PreCompact State Snapshot Hook
-// Captures current session state into DB before compaction destroys context.
-// ============================================================
-
-import { getMemoryDb, addSummary, createSession, addObservation } from '../memory-db.ts';
-import { logAuditEntry } from '../audit-trail.ts';
-import type { SessionSummary } from '../memory-db.ts';
-
-interface HookInput {
-  session_id: string;
-  transcript_path: string;
-  cwd: string;
-  hook_event_name: string;
-}
-
-async function main(): Promise<void> {
-  try {
-    const input = await readStdin();
-    const hookInput = JSON.parse(input) as HookInput;
-    const { session_id } = hookInput;
-
-    const db = getMemoryDb();
-    try {
-      // Ensure session exists
-      createSession(db, session_id);
-
-      // 1. Get all observations for this session
-      const observations = db.prepare(
-        'SELECT * FROM observations WHERE session_id = ? ORDER BY created_at_epoch ASC'
-      ).all(session_id) as Array<Record<string, unknown>>;
-
-      // 2. Get user prompts
-      const prompts = db.prepare(
-        'SELECT prompt_text FROM user_prompts WHERE session_id = ? ORDER BY prompt_number ASC'
-      ).all(session_id) as Array<{ prompt_text: string }>;
-
-      // 3. Generate mid-session summary snapshot
-      const summary = buildSnapshotSummary(observations, prompts);
-
-      // 4. Store with pre_compact marker in plan_progress
-      addSummary(db, session_id, summary);
-
-      // 5. Preserve knowledge system state for post-compaction context
-      try {
-        const knowledgeObs = observations.filter(
-          o => (o.title as string)?.includes('knowledge') ||
-               (o.title as string)?.includes('Knowledge') ||
-               (o.detail as string)?.includes('knowledge')
-        );
-        if (knowledgeObs.length > 0) {
-          const knowledgeContext = knowledgeObs
-            .map(o => `[${o.type}] ${o.title}`)
-            .join('; ');
-          // Store as a high-importance observation so session-start picks it up
-          addObservation(db, session_id, 'discovery',
-            'Knowledge context preserved before compaction',
-            knowledgeContext,
-            { importance: 4 }
-          );
-        }
-      } catch (_knowledgeErr) {
-        // Best-effort: never block compaction
-      }
-
-      // Log compaction event for audit trail continuity
-      try {
-        logAuditEntry(db, {
-          sessionId: session_id,
-          eventType: 'compaction',
-          actor: 'hook',
-          metadata: { observations_count: observations.length, prompts_count: prompts.length },
-        });
-      } catch (_auditErr) {
-        // Best-effort: never block compaction
-      }
-    } finally {
-      db.close();
-    }
-  } catch (_e) {
-    // Best-effort: never block Claude Code
-  }
-  process.exit(0);
-}
-
-function buildSnapshotSummary(
-  observations: Array<Record<string, unknown>>,
-  prompts: Array<{ prompt_text: string }>
-): SessionSummary {
-  const request = prompts[0]?.prompt_text?.slice(0, 500) ?? undefined;
-
-  const completed = observations
-    .filter(o => ['feature', 'bugfix', 'refactor'].includes(o.type as string))
-    .map(o => `- ${o.title}`)
-    .join('\n');
-
-  const failedAttempts = observations
-    .filter(o => o.type === 'failed_attempt')
-    .map(o => `- ${o.title}`)
-    .join('\n');
-
-  const decisions = observations
-    .filter(o => o.type === 'decision')
-    .map(o => `- ${o.title}`)
-    .join('\n');
-
-  // Collect file changes
-  const filesCreated: string[] = [];
-  const filesModified: string[] = [];
-  for (const o of observations) {
-    if (o.type !== 'file_change') continue;
-    const files = safeParseJson(o.files_involved as string, []) as string[];
-    const title = o.title as string;
-    if (title.startsWith('Created')) filesCreated.push(...files);
-    else if (title.startsWith('Edited')) filesModified.push(...files);
-  }
-
-  // Collect plan progress
-  const planProgress: Record<string, string> = { snapshot_type: 'pre_compact' };
-  for (const o of observations) {
-    if (!o.plan_item) continue;
-    planProgress[o.plan_item as string] = 'in_progress';
-  }
-
-  // Verification results
-  const verificationResults: Record<string, string> = {};
-  for (const o of observations) {
-    if (o.type !== 'vr_check') continue;
-    const vrType = o.vr_type as string;
-    const passed = (o.title as string).includes('PASS');
-    if (vrType) verificationResults[vrType] = passed ? 'PASS' : 'FAIL';
-  }
-
-  return {
-    request,
-    completed: completed || undefined,
-    failedAttempts: failedAttempts || undefined,
-    decisions: decisions || undefined,
-    filesCreated: [...new Set(filesCreated)],
-    filesModified: [...new Set(filesModified)],
-    verificationResults,
-    planProgress,
-  };
-}
-
-function safeParseJson(json: string, fallback: unknown): unknown {
-  try {
-    return JSON.parse(json);
-  } catch (_e) {
-    return fallback;
-  }
-}
-
-function readStdin(): Promise<string> {
-  return new Promise((resolve) => {
-    let data = '';
-    process.stdin.setEncoding('utf-8');
-    process.stdin.on('data', (chunk: string) => { data += chunk; });
-    process.stdin.on('end', () => resolve(data));
-    setTimeout(() => resolve(data), 3000);
-  });
-}
-
-main();