0xray 4.0.13 → 4.0.14

package/dist/inference/semantic-patterns.js

@@ -0,0 +1,310 @@
+import { execSync } from "child_process";
+export function analyzeStructuralPatterns(fromRef, toRef) {
+    const changes = getFileChanges(fromRef, toRef);
+    if (changes.length === 0)
+        return [];
+    const patterns = [];
+    const extractMethod = detectExtractMethod(changes, fromRef, toRef);
+    if (extractMethod)
+        patterns.push(extractMethod);
+    const registry = detectRegistryPattern(fromRef, toRef);
+    if (registry)
+        patterns.push(registry);
+    const facade = detectFacadePattern(changes, fromRef, toRef);
+    if (facade)
+        patterns.push(facade);
+    const convention = detectConventionOverConfig(changes, fromRef, toRef);
+    if (convention)
+        patterns.push(convention);
+    const deadCode = detectDeadCodeRemoval(changes);
+    if (deadCode)
+        patterns.push(deadCode);
+    const testExpansion = detectTestExpansion(changes);
+    if (testExpansion)
+        patterns.push(testExpansion);
+    const di = detectDependencyInjection(fromRef, toRef);
+    if (di)
+        patterns.push(di);
+    const stabilitySprint = detectStabilitySprint(changes);
+    if (stabilitySprint)
+        patterns.push(stabilitySprint);
+    return patterns.sort((a, b) => b.confidence - a.confidence);
+}
+function getFileChanges(fromRef, toRef) {
+    try {
+        const output = execSync(`git diff --numstat ${fromRef}..${toRef} --no-renames`, { encoding: "utf-8", stdio: "pipe", timeout: 5000 });
+        return output
+            .split("\n")
+            .filter(Boolean)
+            .map((line) => {
+            const [ins, dels, filePath] = line.split("\t");
+            if (!filePath)
+                return null;
+            const status = getChangeStatus(filePath, fromRef, toRef);
+            const parts = filePath.split("/");
+            return {
+                path: filePath,
+                status,
+                insertions: parseInt(ins || "0", 10) || 0,
+                deletions: parseInt(dels || "0", 10) || 0,
+                dir: parts.length > 1 ? parts.slice(0, -1).join("/") : ".",
+                basename: parts[parts.length - 1],
+            };
+        })
+            .filter((c) => c !== null);
+    }
+    catch {
+        return [];
+    }
+}
+function getChangeStatus(filePath, fromRef, toRef) {
+    try {
+        const status = execSync(`git diff --name-status ${fromRef}..${toRef} -- ${filePath}`, { encoding: "utf-8", stdio: "pipe", timeout: 3000 }).trim();
+        const statusChar = status.charAt(0);
+        if (statusChar === "A" || statusChar === "M" || statusChar === "D" || statusChar === "R" || statusChar === "C") {
+            return statusChar;
+        }
+        return "M";
+    }
+    catch {
+        return "M";
+    }
+}
+function detectExtractMethod(changes, _fromRef, _toRef) {
+    const added = changes.filter((c) => c.status === "A" && c.path.endsWith(".ts"));
+    const modified = changes.filter((c) => c.status === "M");
+    if (added.length < 2)
+        return null;
+    const addedDirs = new Set(added.map((c) => c.dir));
+    const modifiedDirs = new Set(modified.map((c) => c.dir));
+    const sharedDirs = [...addedDirs].filter((d) => modifiedDirs.has(d));
+    if (sharedDirs.length === 0)
+        return null;
+    const evidence = [];
+    for (const dir of sharedDirs) {
+        const addedInDir = added.filter((c) => c.dir === dir);
+        const modInDir = modified.filter((c) => c.dir === dir);
+        evidence.push(`${addedInDir.length} new files + ${modInDir.length} modified in ${dir}/`);
+        for (const f of addedInDir.slice(0, 3)) {
+            evidence.push(`  + ${f.basename}`);
+        }
+        for (const f of modInDir.slice(0, 3)) {
+            evidence.push(`  ~ ${f.basename} (-${f.deletions} lines)`);
+        }
+    }
+    const confidence = Math.min(0.95, 0.6 + added.length * 0.05 + sharedDirs.length * 0.1);
+    return {
+        name: "Extract Method",
+        confidence,
+        evidence,
+        description: "Methods extracted from monolithic modules into dedicated files in the same directory. Classic decomposition pattern: one large file sheds responsibilities into focused modules.",
+    };
+}
+function detectRegistryPattern(fromRef, toRef) {
+    try {
+        const diff = execSync(`git diff ${fromRef}..${toRef} --no-renames`, {
+            encoding: "utf-8",
+            stdio: "pipe",
+            timeout: 8000,
+        });
+        const evidence = [];
+        let switchRemovals = 0;
+        let mapAdditions = 0;
+        const switchLines = diff.split("\n");
+        for (const line of switchLines) {
+            if (line.startsWith("-") &&
+                (line.includes("switch") || line.includes("case "))) {
+                switchRemovals++;
+            }
+            if (line.startsWith("+") && line.includes("new Map")) {
+                mapAdditions++;
+            }
+            if (line.startsWith("+") && line.includes(".set(")) {
+                mapAdditions++;
+            }
+        }
+        if (switchRemovals > 0 && mapAdditions > 0) {
+            evidence.push(`Removed ${switchRemovals} switch/case lines, added ${mapAdditions} Map operations`);
+        }
+        else if (mapAdditions >= 2) {
+            evidence.push(`${mapAdditions} Map operations added`);
+        }
+        else {
+            return null;
+        }
+        const factoryPattern = diff.includes("registerProcessor") || diff.includes("register(");
+        if (factoryPattern) {
+            evidence.push("Factory/registry registration pattern detected");
+        }
+        return {
+            name: "Registry Pattern",
+            confidence: switchRemovals > 0 ? 0.9 : 0.7,
+            evidence,
+            description: "Switch statement replaced with Map-based registry lookup. O(1) dispatch replaces O(n) case matching. New entries require only registration, not code changes to the dispatcher.",
+        };
+    }
+    catch {
+        return null;
+    }
+}
+function detectFacadePattern(changes, fromRef, toRef) {
+    const modified = changes.filter((c) => c.status === "M" &&
+        c.deletions > c.insertions * 2 &&
+        c.deletions > 50);
+    if (modified.length === 0)
+        return null;
+    const added = changes.filter((c) => c.status === "A" && c.path.endsWith(".ts"));
+    if (added.length === 0)
+        return null;
+    try {
+        const diff = execSync(`git diff ${fromRef}..${toRef} -- ${modified.map((c) => c.path).join(" ")} --no-renames`, { encoding: "utf-8", stdio: "pipe", timeout: 5000 });
+        const removedFunctionLines = diff
+            .split("\n")
+            .filter((l) => l.startsWith("-") &&
+            (l.includes("async ") || l.includes("function ") || l.includes("private "))).length;
+        if (removedFunctionLines < 3)
+            return null;
+        const evidence = [];
+        for (const f of modified) {
+            evidence.push(`${f.path}: -${f.deletions}/+${f.insertions} lines, ${removedFunctionLines} function signatures removed`);
+        }
+        evidence.push(`${added.length} new files created to absorb extracted logic`);
+        return {
+            name: "Facade Pattern",
+            confidence: 0.85,
+            evidence,
+            description: "Monolithic file stripped down to a thin facade while logic moves to dedicated modules. The original becomes a coordinator, not an implementer.",
+        };
+    }
+    catch {
+        return null;
+    }
+}
+function detectConventionOverConfig(changes, fromRef, toRef) {
+    const hasImplementations = changes.some((c) => c.status === "A" && c.path.includes("implementations/"));
+    if (!hasImplementations)
+        return null;
+    try {
+        const diff = execSync(`git diff ${fromRef}..${toRef} --no-renames`, {
+            encoding: "utf-8",
+            stdio: "pipe",
+            timeout: 8000,
+        });
+        const hasDiscovery = diff.includes("discover") || diff.includes("auto-register") || diff.includes("scan");
+        const hasReflection = diff.includes("reflect") || diff.includes("metadata");
+        const hasDynamicImport = diff.includes("import(") || diff.includes("require(");
+        if (!hasDiscovery && !hasDynamicImport)
+            return null;
+        const addedImpl = changes.filter((c) => c.status === "A" && c.path.includes("implementations/"));
+        const evidence = [
+            `${addedImpl.length} new implementations auto-discovered`,
+        ];
+        if (hasDiscovery)
+            evidence.push("Discovery/scan logic present");
+        if (hasDynamicImport)
+            evidence.push("Dynamic imports for runtime loading");
+        if (hasReflection)
+            evidence.push("Reflection/metadata introspection");
+        return {
+            name: "Convention over Configuration",
+            confidence: 0.88,
+            evidence,
+            description: "New modules are auto-discovered at runtime instead of manually registered. Drop a file in the right directory and the system finds it. Eliminates registration boilerplate and prevents orphaned modules.",
+        };
+    }
+    catch {
+        return null;
+    }
+}
+function detectDeadCodeRemoval(changes) {
+    const deleted = changes.filter((c) => c.status === "D");
+    if (deleted.length < 2)
+        return null;
+    const evidence = [];
+    for (const f of deleted.slice(0, 5)) {
+        evidence.push(`- ${f.path}`);
+    }
+    if (deleted.length > 5) {
+        evidence.push(`... and ${deleted.length - 5} more`);
+    }
+    return {
+        name: "Dead Code Removal",
+        confidence: 0.9,
+        evidence,
+        description: `${deleted.length} files deleted — unused code eliminated. Reduces maintenance burden, removes misleading signals for new readers.`,
+    };
+}
+function detectTestExpansion(changes) {
+    const addedTests = changes.filter((c) => c.status === "A" && c.path.includes("__tests__"));
+    if (addedTests.length < 2)
+        return null;
+    const evidence = [];
+    for (const f of addedTests.slice(0, 5)) {
+        evidence.push(`+ ${f.path}`);
+    }
+    return {
+        name: "Test Coverage Expansion",
+        confidence: 0.95,
+        evidence,
+        description: `${addedTests.length} new test files added. Test-first or test-alongside development — covering new code as it ships.`,
+    };
+}
+function detectDependencyInjection(fromRef, toRef) {
+    try {
+        const diff = execSync(`git diff ${fromRef}..${toRef} --no-renames`, {
+            encoding: "utf-8",
+            stdio: "pipe",
+            timeout: 8000,
+        });
+        const evidence = [];
+        const hasStaticDeps = diff.includes("+  static readonly dependencies") ||
+            diff.includes("+  static dependencies");
+        const hasInject = diff.includes("inject") || diff.includes("resolve(");
+        const hasConstructorDI = diff.includes("constructor(") &&
+            (diff.includes("private ") || diff.includes("readonly "));
+        if (!hasStaticDeps && !hasInject)
+            return null;
+        if (hasStaticDeps)
+            evidence.push("Static dependency declarations added");
+        if (hasInject)
+            evidence.push("DI container / injection logic");
+        if (hasConstructorDI)
+            evidence.push("Constructor-based DI signatures");
+        return {
+            name: "Dependency Injection",
+            confidence: 0.82,
+            evidence,
+            description: "Explicit dependency declarations replacing hardcoded imports. Enables testing with mocks, runtime configuration, and auto-wiring.",
+        };
+    }
+    catch {
+        return null;
+    }
+}
+function detectStabilitySprint(changes) {
+    const fixChanges = changes.filter((c) => c.path.toLowerCase().includes("fix") ||
+        c.path.toLowerCase().includes("patch"));
+    const totalModified = changes.filter((c) => c.status === "M" || c.status === "A").length;
+    const deleted = changes.filter((c) => c.status === "D");
+    if (deleted.length < 3 && fixChanges.length < 3)
+        return null;
+    const netDeletions = changes.reduce((s, c) => s + c.deletions - c.insertions, 0);
+    if (netDeletions < 100 && deleted.length < 3)
+        return null;
+    const evidence = [];
+    if (deleted.length >= 3) {
+        evidence.push(`${deleted.length} files deleted`);
+    }
+    if (netDeletions > 0) {
+        evidence.push(`Net ${netDeletions} lines removed`);
+    }
+    if (fixChanges.length >= 3) {
+        evidence.push(`${fixChanges.length} fix-related changes`);
+    }
+    return {
+        name: "Stability Sprint",
+        confidence: 0.75,
+        evidence,
+        description: "Focus on removal and fixing over feature addition. Technical debt being paid down — a sign of mature development practice.",
+    };
+}

package/dist/inference/session-capture.d.ts

@@ -0,0 +1,32 @@
+import { StructuralPattern } from "./semantic-patterns.js";
+export interface SessionInference {
+    sessionId: string;
+    timestamp: string;
+    span: {
+        from: string;
+        to: string;
+    };
+    problems: string[];
+    approaches: string[];
+    wrongTurns: string[];
+    solutions: string[];
+    reasoningChain: ReasoningLink[];
+    patterns: StructuralPattern[];
+    metrics: SessionMetrics;
+}
+export interface ReasoningLink {
+    from: string;
+    to: string;
+    reasoning: string;
+}
+export interface SessionMetrics {
+    commits: number;
+    filesChanged: number;
+    insertions: number;
+    deletions: number;
+    filesAdded: number;
+    filesDeleted: number;
+    uniqueDirs: number;
+}
+export declare function captureSessionInference(fromRef: string, toRef: string): SessionInference | null;
+export declare function saveSessionInference(inference: SessionInference, outputDir?: string): string;

package/dist/inference/session-capture.js

@@ -0,0 +1,308 @@
+import * as fs from "fs";
+import * as path from "path";
+import { execSync } from "child_process";
+import { analyzeStructuralPatterns } from "./semantic-patterns.js";
+export function captureSessionInference(fromRef, toRef) {
+    const commits = getCommitData(fromRef, toRef);
+    if (commits.length === 0)
+        return null;
+    const metrics = computeMetrics(fromRef, toRef);
+    const patterns = analyzeStructuralPatterns(fromRef, toRef);
+    const problems = extractProblems(commits, patterns);
+    const approaches = extractApproaches(commits, patterns);
+    const wrongTurns = extractWrongTurns(commits, fromRef, toRef);
+    const solutions = extractSolutions(commits, patterns);
+    const reasoningChain = buildReasoningChain(problems, approaches, wrongTurns, solutions);
+    return {
+        sessionId: `session-${new Date().toISOString().slice(0, 10)}`,
+        timestamp: new Date().toISOString(),
+        span: { from: fromRef, to: toRef },
+        problems,
+        approaches,
+        wrongTurns,
+        solutions,
+        reasoningChain,
+        patterns,
+        metrics,
+    };
+}
+export function saveSessionInference(inference, outputDir) {
+    const dir = outputDir || path.join(process.cwd(), "docs", "inference");
+    if (!fs.existsSync(dir)) {
+        fs.mkdirSync(dir, { recursive: true });
+    }
+    const filename = `session-${inference.sessionId.replace(/^session-/, "")}-${Date.now()}.json`;
+    const filePath = path.join(dir, filename);
+    fs.writeFileSync(filePath, JSON.stringify(inference, null, 2));
+    const latestPath = path.join(dir, "latest-session.json");
+    fs.writeFileSync(latestPath, JSON.stringify(inference, null, 2));
+    return filePath;
+}
+function getCommitData(fromRef, toRef) {
+    try {
+        const log = execSync(`git log ${fromRef}..${toRef} --format="%H||%s" --shortstat --no-merges`, { encoding: "utf-8", stdio: "pipe", timeout: 5000 });
+        if (!log.trim())
+            return [];
+        return log
+            .split(/\n{2,}/)
+            .filter((b) => b.trim().length > 0 && b.includes("||"))
+            .map((block) => {
+            const lines = block.trim().split("\n");
+            const headerLine = lines.find((l) => l.includes("||")) || "";
+            const [hash, message] = headerLine.split("||");
+            const statsLine = lines.find((l) => l.includes("file") &&
+                (l.includes("insertion") || l.includes("deletion"))) || "";
+            return {
+                hash: (hash || "").trim().slice(0, 7),
+                message: (message || "").trim(),
+                insertions: parseInt(statsLine.match(/(\d+) insertion/)?.[1] || "0", 10),
+                deletions: parseInt(statsLine.match(/(\d+) deletion/)?.[1] || "0", 10),
+                filesChanged: parseInt(statsLine.match(/(\d+) files? changed/)?.[1] || "0", 10),
+            };
+        });
+    }
+    catch {
+        return [];
+    }
+}
+function computeMetrics(fromRef, toRef) {
+    try {
+        const stat = execSync(`git diff --stat ${fromRef}..${toRef} --no-renames`, {
+            encoding: "utf-8",
+            stdio: "pipe",
+            timeout: 30000,
+            maxBuffer: 50 * 1024 * 1024,
+        });
+        const finalLine = stat.trim().split("\n").pop() || "";
+        const filesChanged = parseInt(finalLine.match(/(\d+) files? changed/)?.[1] || "0", 10);
+        const insertions = parseInt(finalLine.match(/(\d+) insertion/)?.[1] || "0", 10);
+        const deletions = parseInt(finalLine.match(/(\d+) deletion/)?.[1] || "0", 10);
+        const nameStatus = execSync(`git diff --name-status ${fromRef}..${toRef} --no-renames`, { encoding: "utf-8", stdio: "pipe", timeout: 30000, maxBuffer: 50 * 1024 * 1024 });
+        const lines = nameStatus.split("\n").filter(Boolean);
+        const filesAdded = lines.filter((l) => l.startsWith("A")).length;
+        const filesDeleted = lines.filter((l) => l.startsWith("D")).length;
+        const uniqueDirs = new Set(lines.map((l) => {
+            const parts = l.split("\t")[1].split("/");
+            return parts.length > 1 ? parts.slice(0, -1).join("/") : ".";
+        })).size;
+        const commitCount = getCommitData(fromRef, toRef).length;
+        return {
+            commits: commitCount,
+            filesChanged,
+            insertions,
+            deletions,
+            filesAdded,
+            filesDeleted,
+            uniqueDirs,
+        };
+    }
+    catch {
+        return {
+            commits: 0,
+            filesChanged: 0,
+            insertions: 0,
+            deletions: 0,
+            filesAdded: 0,
+            filesDeleted: 0,
+            uniqueDirs: 0,
+        };
+    }
+}
+function extractProblems(commits, patterns) {
+    const problems = [];
+    for (const c of commits) {
+        const msg = c.message;
+        const msgLower = msg.toLowerCase();
+        const isFixCommit = /^[a-f0-9]+:? ?fix[\s:(/]/i.test(msg) || msgLower.startsWith("fix") || msgLower.includes("bug fix") || msgLower.includes("bugfix");
+        if (isFixCommit) {
+            problems.push(`Bug: ${msg} (${c.hash})`);
+        }
+        if (msgLower.includes("remove") && (msgLower.includes("circular") || msgLower.includes("dead code"))) {
+            problems.push(`Code health: ${msg} (${c.hash})`);
+        }
+        if (msgLower.includes("stub") || msgLower.includes("placeholder")) {
+            problems.push(`Incomplete implementation: ${msg} (${c.hash})`);
+        }
+    }
+    for (const p of patterns) {
+        if (p.name === "Dead Code Removal") {
+            problems.push(`Accumulated dead code: ${p.evidence[0]}`);
+        }
+        if (p.name === "Stability Sprint") {
+            problems.push(`Technical debt requiring stability focus`);
+        }
+    }
+    if (problems.length === 0 && commits.length > 3) {
+        problems.push(`General development session: ${commits.length} commits`);
+    }
+    return [...new Set(problems)];
+}
+function extractApproaches(commits, patterns) {
+    const approaches = [];
+    for (const p of patterns) {
+        switch (p.name) {
+            case "Extract Method":
+                approaches.push(`Extract methods into dedicated files to reduce monolith size`);
+                break;
+            case "Registry Pattern":
+                approaches.push(`Replace switch statements with Map-based registry for O(1) dispatch`);
+                break;
+            case "Facade Pattern":
+                approaches.push(`Thin facade + extracted modules: coordinator delegates, doesn't implement`);
+                break;
+            case "Convention over Configuration":
+                approaches.push(`Auto-discover implementations at runtime to eliminate manual registration`);
+                break;
+            case "Dependency Injection":
+                approaches.push(`Declare dependencies explicitly for testability and auto-wiring`);
+                break;
+            case "Dead Code Removal":
+                approaches.push(`Delete unused code to reduce maintenance burden`);
+                break;
+            case "Test Coverage Expansion":
+                approaches.push(`Add tests alongside new implementations`);
+                break;
+        }
+    }
+    for (const c of commits) {
+        const msg = c.message.toLowerCase();
+        if (msg.includes("sync") && msg.includes("version")) {
+            approaches.push(`Automate version synchronization across config files`);
+        }
+        if (msg.includes("append") && (msg.includes("doc") || msg.includes("agent"))) {
+            approaches.push(`Append-only doc updates to prevent data loss`);
+        }
+        if (msg.includes("guard")) {
+            approaches.push(`Structural guard to enforce invariants`);
+        }
+    }
+    return [...new Set(approaches)];
+}
+function extractWrongTurns(commits, fromRef, toRef) {
+    const wrongTurns = [];
+    try {
+        const log = execSync(`git log ${fromRef}..${toRef} --format="%s" --no-merges`, { encoding: "utf-8", stdio: "pipe", timeout: 5000 });
+        const messages = log.split("\n").filter(Boolean).map((m) => m.toLowerCase());
+        if (messages.some((m) => m.includes("revert"))) {
+            wrongTurns.push("Approach was reverted — initial direction was wrong");
+        }
+        const fixAfterFeature = findFixAfterFeature(messages);
+        if (fixAfterFeature) {
+            wrongTurns.push(fixAfterFeature);
+        }
+    }
+    catch {
+        // ignore
+    }
+    for (const c of commits) {
+        const msg = c.message.toLowerCase();
+        if (msg.includes("wrong") || msg.includes("incorrect") || msg.includes("oops")) {
+            wrongTurns.push(`Self-corrected: ${c.message} (${c.hash})`);
+        }
+    }
+    const allMessages = commits.map((c) => c.message.toLowerCase());
+    const hasPathFix = allMessages.filter((m) => m.includes("path") && (m.includes("fix") || m.includes("bogus"))).length;
+    if (hasPathFix > 0) {
+        wrongTurns.push("Path handling bug: absolute vs relative path confusion");
+    }
+    const hasFlakyFix = allMessages.filter((m) => m.includes("timeout") || m.includes("flaky")).length;
+    if (hasFlakyFix > 0) {
+        wrongTurns.push(`Timing issue: test assumptions violated in parallel execution`);
+    }
+    return [...new Set(wrongTurns)];
+}
+function findFixAfterFeature(messages) {
+    let lastFeatureIdx = -1;
+    let firstFixAfter = -1;
+    for (let i = 0; i < messages.length; i++) {
+        if (messages[i].includes("feat") ||
+            messages[i].includes("add") ||
+            messages[i].includes("implement")) {
+            lastFeatureIdx = i;
+        }
+        if (lastFeatureIdx >= 0 &&
+            messages[i].includes("fix") &&
+            firstFixAfter < 0) {
+            firstFixAfter = i;
+        }
+    }
+    if (lastFeatureIdx >= 0 && firstFixAfter > lastFeatureIdx) {
+        return `Feature commit followed by fix — incomplete initial implementation`;
+    }
+    return null;
+}
+function extractSolutions(commits, patterns) {
+    const solutions = [];
+    for (const p of patterns) {
+        solutions.push(`Applied ${p.name} pattern (confidence: ${Math.round(p.confidence * 100)}%)`);
+    }
+    for (const c of commits) {
+        const msg = c.message.toLowerCase();
+        if (msg.includes("create") || msg.includes("add script")) {
+            solutions.push(`New tooling: ${c.message} (${c.hash})`);
+        }
+        if (msg.includes("wire") || msg.includes("integrate")) {
+            solutions.push(`Integration: ${c.message} (${c.hash})`);
+        }
+        if (msg.includes("consolidat")) {
+            solutions.push(`Consolidation: ${c.message} (${c.hash})`);
+        }
+    }
+    if (solutions.length === 0 && commits.length > 0) {
+        solutions.push(`${commits.length} commits shipped successfully`);
+    }
+    return [...new Set(solutions)];
+}
+function buildReasoningChain(problems, approaches, wrongTurns, solutions) {
+    const chain = [];
+    for (const approach of approaches) {
+        const matchingSolution = solutions.find((s) => approaches.indexOf(approach) === solutions.indexOf(s) ||
+            s.toLowerCase().includes(approachKeyword(approach)));
+        if (matchingSolution) {
+            chain.push({
+                from: "approach",
+                to: "solution",
+                reasoning: `${approach} → ${matchingSolution}`,
+            });
+        }
+        else {
+            chain.push({
+                from: "problem",
+                to: "approach",
+                reasoning: approach,
+            });
+        }
+    }
+    for (const wt of wrongTurns) {
+        chain.push({
+            from: "approach",
+            to: "wrong_turn",
+            reasoning: wt,
+        });
+    }
+    return chain;
+}
+function approachKeyword(approach) {
+    const lower = approach.toLowerCase();
+    if (lower.includes("extract"))
+        return "extract";
+    if (lower.includes("registry") || lower.includes("map"))
+        return "registry";
+    if (lower.includes("facade") || lower.includes("thin"))
+        return "facade";
+    if (lower.includes("convention") || lower.includes("discover"))
+        return "convention";
+    if (lower.includes("dependen") || lower.includes("inject"))
+        return "dependenc";
+    if (lower.includes("dead code") || lower.includes("delete"))
+        return "dead code";
+    if (lower.includes("test"))
+        return "test";
+    if (lower.includes("version") || lower.includes("sync"))
+        return "version";
+    if (lower.includes("append"))
+        return "append";
+    if (lower.includes("guard"))
+        return "guard";
+    return "";
+}