clud-bug 0.6.34 → 0.7.0-rc.11

package/dist/core/detect.js

@@ -0,0 +1,262 @@
+import { readFile, readdir, stat } from 'node:fs/promises';
+import { join, extname } from 'node:path';
+// Lookup tables for the project-shape detectors. Each map is `as const` so
+// downstream consumers can rely on the value types narrowing to the literal
+// strings rather than `string` — `_internal.EXT_TO_LANG['.ts']` resolves to
+// `'typescript'` for IDE navigation, not just `string`.
+export const EXT_TO_LANG = {
+    '.ts': 'typescript', '.tsx': 'typescript',
+    '.js': 'javascript', '.jsx': 'javascript', '.mjs': 'javascript', '.cjs': 'javascript',
+    '.py': 'python',
+    '.go': 'go',
+    '.rs': 'rust',
+    '.rb': 'ruby',
+    '.java': 'java', '.kt': 'kotlin',
+    '.swift': 'swift',
+    '.php': 'php',
+    '.cs': 'csharp',
+    '.c': 'c', '.h': 'c',
+    '.cpp': 'cpp', '.cc': 'cpp', '.hpp': 'cpp',
+};
+// Dependency name → search term hint passed to skills.sh.
+// Only well-known frameworks; obscure packages get filtered out so the
+// skills.sh query doesn't get drowned in noise.
+export const DEP_TO_TERM = {
+    'next': 'nextjs', 'react': 'react', 'vue': 'vue', 'svelte': 'svelte',
+    '@angular/core': 'angular', 'solid-js': 'solid',
+    'express': 'express', 'fastify': 'fastify', 'koa': 'koa', 'hono': 'hono',
+    'prisma': 'prisma', '@prisma/client': 'prisma', 'drizzle-orm': 'drizzle',
+    'mongoose': 'mongodb', 'mongodb': 'mongodb',
+    'tailwindcss': 'tailwind',
+    'vitest': 'vitest', 'jest': 'jest', 'playwright': 'playwright',
+    '@playwright/test': 'playwright',
+    'typescript': 'typescript',
+};
+export const PY_DEP_TO_TERM = {
+    'django': 'django', 'flask': 'flask', 'fastapi': 'fastapi',
+    'click': 'click', 'typer': 'typer',
+    'pytest': 'pytest', 'sqlalchemy': 'sqlalchemy',
+    'pydantic': 'pydantic', 'numpy': 'numpy', 'pandas': 'pandas',
+};
+async function fileExists(path) {
+    try {
+        await stat(path);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+async function readJsonSafe(path) {
+    try {
+        return JSON.parse(await readFile(path, 'utf8'));
+    }
+    catch {
+        return null;
+    }
+}
+async function readTextSafe(path) {
+    try {
+        return await readFile(path, 'utf8');
+    }
+    catch {
+        return null;
+    }
+}
+async function detectFromPackageJson(root) {
+    const pkg = await readJsonSafe(join(root, 'package.json'));
+    if (!pkg)
+        return null;
+    const deps = { ...(pkg.dependencies || {}), ...(pkg.devDependencies || {}) };
+    const terms = new Set();
+    for (const dep of Object.keys(deps)) {
+        const term = DEP_TO_TERM[dep];
+        if (term)
+            terms.add(term);
+    }
+    return {
+        name: pkg.name ?? null,
+        description: pkg.description || null,
+        languages: ['javascript', ...(deps.typescript || pkg.devDependencies?.typescript ? ['typescript'] : [])],
+        terms: [...terms],
+    };
+}
+async function detectFromPyproject(root) {
+    const text = await readTextSafe(join(root, 'pyproject.toml'));
+    if (!text)
+        return null;
+    const terms = new Set();
+    for (const [dep, term] of Object.entries(PY_DEP_TO_TERM)) {
+        // crude but adequate match — full TOML parse would be overkill for the
+        // dependency-name lookup we actually need
+        if (new RegExp(`["']${dep}[><=~ "']`, 'i').test(text))
+            terms.add(term);
+    }
+    const nameMatch = text.match(/^\s*name\s*=\s*["']([^"']+)["']/m);
+    const descMatch = text.match(/^\s*description\s*=\s*["']([^"']+)["']/m);
+    return {
+        name: nameMatch?.[1] ?? null,
+        description: descMatch?.[1] ?? null,
+        languages: ['python'],
+        terms: [...terms],
+    };
+}
+async function detectFromRequirements(root) {
+    const text = await readTextSafe(join(root, 'requirements.txt'));
+    if (!text)
+        return null;
+    const terms = new Set();
+    for (const line of text.split('\n')) {
+        const dep = (line.split(/[<>=~ #]/)[0] ?? '').trim().toLowerCase();
+        const term = PY_DEP_TO_TERM[dep];
+        if (term)
+            terms.add(term);
+    }
+    return { name: null, description: null, languages: ['python'], terms: [...terms] };
+}
+async function detectFromGoMod(root) {
+    const text = await readTextSafe(join(root, 'go.mod'));
+    if (!text)
+        return null;
+    const moduleMatch = text.match(/^module\s+(\S+)/m);
+    return {
+        name: moduleMatch?.[1]?.split('/').pop() ?? null,
+        description: null,
+        languages: ['go'],
+        terms: [],
+    };
+}
+async function detectFromCargo(root) {
+    const text = await readTextSafe(join(root, 'Cargo.toml'));
+    if (!text)
+        return null;
+    const nameMatch = text.match(/^\s*name\s*=\s*["']([^"']+)["']/m);
+    const descMatch = text.match(/^\s*description\s*=\s*["']([^"']+)["']/m);
+    return {
+        name: nameMatch?.[1] ?? null,
+        description: descMatch?.[1] ?? null,
+        languages: ['rust'],
+        terms: [],
+    };
+}
+async function detectFromGemfile(root) {
+    const text = await readTextSafe(join(root, 'Gemfile'));
+    if (!text)
+        return null;
+    return { name: null, description: null, languages: ['ruby'], terms: [] };
+}
+async function fileHistogram(root) {
+    const counts = {};
+    async function walk(dir, depth) {
+        if (depth > 3)
+            return;
+        let entries;
+        try {
+            entries = await readdir(dir, { withFileTypes: true });
+        }
+        catch {
+            return;
+        }
+        for (const entry of entries) {
+            if (entry.name.startsWith('.') || entry.name === 'node_modules' ||
+                entry.name === 'dist' || entry.name === 'build' ||
+                entry.name === '__pycache__' || entry.name === 'target')
+                continue;
+            const full = join(dir, entry.name);
+            if (entry.isDirectory()) {
+                await walk(full, depth + 1);
+            }
+            else {
+                const lang = EXT_TO_LANG[extname(entry.name)];
+                if (lang)
+                    counts[lang] = (counts[lang] || 0) + 1;
+            }
+        }
+    }
+    await walk(root, 0);
+    return counts;
+}
+function firstParagraph(readme) {
+    if (!readme)
+        return null;
+    const lines = readme.split('\n').slice(0, 200);
+    const paragraphs = lines.join('\n').split(/\n\s*\n/);
+    for (const p of paragraphs) {
+        const cleaned = p.replace(/^#+\s*/, '').replace(/[*_`]/g, '').trim();
+        if (cleaned.length > 40)
+            return cleaned.slice(0, 500);
+    }
+    return null;
+}
+export async function detect(root) {
+    const detectors = [
+        detectFromPackageJson, detectFromPyproject, detectFromRequirements,
+        detectFromGoMod, detectFromCargo, detectFromGemfile,
+    ];
+    const results = (await Promise.all(detectors.map(d => d(root)))).filter((r) => r !== null);
+    const histogram = await fileHistogram(root);
+    const readme = await readTextSafe(join(root, 'README.md'))
+        || await readTextSafe(join(root, 'README'));
+    const languages = new Set();
+    const terms = new Set();
+    let name = null;
+    let description = null;
+    for (const r of results) {
+        for (const lang of r.languages)
+            languages.add(lang);
+        for (const term of r.terms)
+            terms.add(term);
+        if (!name && r.name)
+            name = r.name;
+        if (!description && r.description)
+            description = r.description;
+    }
+    for (const lang of Object.keys(histogram))
+        languages.add(lang);
+    if (!description)
+        description = firstParagraph(readme);
+    // Prefer the language with the most files when picking a primary
+    const sortedLangs = [...languages].sort((a, b) => (histogram[b] || 0) - (histogram[a] || 0));
+    return {
+        name,
+        description,
+        languages: sortedLangs,
+        histogram,
+        searchTerms: [...new Set([...terms, ...sortedLangs.slice(0, 2)])],
+        primaryLanguage: sortedLangs[0] ?? null,
+    };
+}
+export function buildDescriptionLine(signals) {
+    const parts = [];
+    if (signals.name)
+        parts.push(`This project is "${signals.name}".`);
+    if (signals.description) {
+        // v0.6.25 / issue #89: when signals.description comes from a
+        // README first paragraph or similar multi-paragraph source, the
+        // raw `\n` characters survive into the rendered YAML's
+        // APPEND_SYSTEM_PROMPT value. The renderer's indent-aware
+        // substitution preserves layout but a literal `\n` inside a YAML
+        // double-quoted string is interpreted as a newline, breaking
+        // the YAML. Collapse any whitespace run (including newlines + tabs)
+        // to a single space before further processing.
+        const desc = signals.description.replace(/\s+/g, ' ').trim();
+        parts.push(/[.!?]$/.test(desc) ? desc : `${desc}.`);
+    }
+    if (signals.primaryLanguage) {
+        const frameworks = [...new Set(signals.searchTerms || [])].filter((t) => !['typescript', 'javascript', 'python', 'go', 'rust', 'ruby'].includes(t));
+        const frameworkPart = frameworks.length ? ` using ${frameworks.join(', ')}` : '';
+        parts.push(`It's primarily ${signals.primaryLanguage}${frameworkPart}.`);
+    }
+    if (parts.length === 0)
+        return 'Project context unavailable — review on the merits of the diff alone.';
+    return parts.join(' ');
+}
+// Architect's anti-pattern fix (Phase 2): the JS source used a single
+// `export const _internal = { … }` namespace as a test seam. The TS port
+// promotes the table exports (EXT_TO_LANG, DEP_TO_TERM, PY_DEP_TO_TERM)
+// to direct top-level exports, and exposes the two helper functions
+// fileHistogram + firstParagraph as direct named exports too. Tests now
+// import each symbol by name. No `_internal` re-export — that pattern is
+// gone.
+export { fileHistogram, firstParagraph };
+//# sourceMappingURL=detect.js.map

package/dist/core/detect.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"detect.js","sourceRoot":"","sources":["../../src/core/detect.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAC3D,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAE1C,2EAA2E;AAC3E,4EAA4E;AAC5E,4EAA4E;AAC5E,wDAAwD;AACxD,MAAM,CAAC,MAAM,WAAW,GAAG;IACzB,KAAK,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY;IACzC,KAAK,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY;IACrF,KAAK,EAAE,QAAQ;IACf,KAAK,EAAE,IAAI;IACX,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,MAAM;IACb,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ;IAChC,QAAQ,EAAE,OAAO;IACjB,MAAM,EAAE,KAAK;IACb,KAAK,EAAE,QAAQ;IACf,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG;IACpB,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK;CACD,CAAC;AAE5C,0DAA0D;AAC1D,uEAAuE;AACvE,gDAAgD;AAChD,MAAM,CAAC,MAAM,WAAW,GAAG;IACzB,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ;IACpE,eAAe,EAAE,SAAS,EAAE,UAAU,EAAE,OAAO;IAC/C,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM;IACxE,QAAQ,EAAE,QAAQ,EAAE,gBAAgB,EAAE,QAAQ,EAAE,aAAa,EAAE,SAAS;IACxE,UAAU,EAAE,SAAS,EAAE,SAAS,EAAE,SAAS;IAC3C,aAAa,EAAE,UAAU;IACzB,QAAQ,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,YAAY;IAC9D,kBAAkB,EAAE,YAAY;IAChC,YAAY,EAAE,YAAY;CACe,CAAC;AAE5C,MAAM,CAAC,MAAM,cAAc,GAAG;IAC5B,QAAQ,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,SAAS;IAC1D,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO;IAClC,QAAQ,EAAE,QAAQ,EAAE,YAAY,EAAE,YAAY;IAC9C,UAAU,EAAE,UAAU,EAAE,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,QAAQ;CACnB,CAAC;AA+B5C,KAAK,UAAU,UAAU,CAAC,IAAY;IACpC,IAAI,CAAC;QACH,MAAM,IAAI,CAAC,IAAI,CAAC,CAAC;QACjB,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED,KAAK,UAAU,YAAY,CAAc,IAAY;IACnD,IAAI,CAAC;QACH,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,CAAM,CAAC;IACvD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,KAAK,UAAU,YAAY,CAAC,IAAY;IACtC,IAAI,CAAC;QACH,OAAO,MAAM,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;IACtC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,KAAK,UAAU,qBAAqB,CAAC,IAAY;IAC/C,MAAM,GAAG,GAAG,MAAM,YAAY,CAAc,IAAI,CAAC,IAAI,EAAE,cAAc,CAAC,CAAC,CAAC;IACxE,IAAI,CAAC,GAAG;QAAE,OAAO,IAAI,CAAC;IACtB,MAAM,IAAI,GAA2B,EAAE,GAAG,CAAC,GAAG,CAAC,YAAY,IAAI,EAAE,CAAC,EAAE,GAAG,CAAC,GAAG,CAAC,eAAe,IAAI,EAAE,CAAC,EAAE,CAAC;IACrG,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,MAAM,IAAI,GAAI,WAAsC,CAAC,GAAG,CAAC,CAAC;QAC1D,IAAI,IAAI;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO;QACL,IAAI,EAAE,GAAG,CAAC,IAAI,IAAI,IAAI;QACtB,WAAW,EAAE,GAAG,CAAC,WAAW,IAAI,IAAI;QACpC,SAAS,EAAE,CAAC,YAAY,EAAE,GAAG,CAAC,IAAI,CAAC,UAAU,IAAI,GAAG,CAAC,eAAe,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACxG,KAAK,EAAE,CAAC,GAAG,KAAK,CAAC;KAClB,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,mBAAmB,CAAC,IAAY;IAC7C,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,gBAAgB,CAAC,CAAC,CAAC;IAC9D,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,KAAK,MAAM,CAAC,GAAG,EAAE,IAAI,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,EAAE,CAAC;QACzD,uEAAuE;QACvE,0CAA0C;QAC1C,IAAI,IAAI,MAAM,CAAC,OAAO,GAAG,WAAW,EAAE,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IACzE,CAAC;IACD,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,kCAAkC,CAAC,CAAC;IACjE,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC;IACxE,OAAO;QACL,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI;QAC5B,WAAW,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI;QACnC,SAAS,EAAE,CAAC,QAAQ,CAAC;QACrB,KAAK,EAAE,CAAC,GAAG,KAAK,CAAC;KAClB,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,sBAAsB,CAAC,IAAY;IAChD,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,kBAAkB,CAAC,CAAC,CAAC;IAChE,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,MAAM,GAAG,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QACnE,MAAM,IAAI,GAAI,cAAyC,CAAC,GAAG,CAAC,CAAC;QAC7D,IAAI,IAAI;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,QAAQ,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,KAAK,CAAC,EAAE,CAAC;AACrF,CAAC;AAED,KAAK,UAAU,eAAe,CAAC,IAAY;IACzC,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,CAAC;IACtD,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,MAAM,WAAW,GAAG,IAAI,CAAC,KAAK,CAAC,kBAAkB,CAAC,CAAC;IACnD,OAAO;QACL,IAAI,EAAE,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,IAAI,IAAI;QAChD,WAAW,EAAE,IAAI;QACjB,SAAS,EAAE,CAAC,IAAI,CAAC;QACjB,KAAK,EAAE,EAAE;KACV,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,eAAe,CAAC,IAAY;IACzC,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC,CAAC;IAC1D,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,kCAAkC,CAAC,CAAC;IACjE,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC;IACxE,OAAO;QACL,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI;QAC5B,WAAW,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI;QACnC,SAAS,EAAE,CAAC,MAAM,CAAC;QACnB,KAAK,EAAE,EAAE;KACV,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,iBAAiB,CAAC,IAAY;IAC3C,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC,CAAC;IACvD,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,MAAM,CAAC,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC;AAC3E,CAAC;AAED,KAAK,UAAU,aAAa,CAAC,IAAY;IACvC,MAAM,MAAM,GAA2B,EAAE,CAAC;IAC1C,KAAK,UAAU,IAAI,CAAC,GAAW,EAAE,KAAa;QAC5C,IAAI,KAAK,GAAG,CAAC;YAAE,OAAO;QACtB,IAAI,OAAO,CAAC;QACZ,IAAI,CAAC;YAAC,OAAO,GAAG,MAAM,OAAO,CAAC,GAAG,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAC;QAAC,CAAC;QAAC,MAAM,CAAC;YAAC,OAAO;QAAC,CAAC;QAChF,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;YAC5B,IAAI,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,IAAI,KAAK,cAAc;gBAC3D,KAAK,CAAC,IAAI,KAAK,MAAM,IAAI,KAAK,CAAC,IAAI,KAAK,OAAO;gBAC/C,KAAK,CAAC,IAAI,KAAK,aAAa,IAAI,KAAK,CAAC,IAAI,KAAK,QAAQ;gBAAE,SAAS;YACtE,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;YACnC,IAAI,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC;gBACxB,MAAM,IAAI,CAAC,IAAI,EAAE,KAAK,GAAG,CAAC,CAAC,CAAC;YAC9B,CAAC;iBAAM,CAAC;gBACN,MAAM,IAAI,GAAI,WAAsC,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC;gBAC1E,IAAI,IAAI;oBAAE,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;YACnD,CAAC;QACH,CAAC;IACH,CAAC;IACD,MAAM,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IACpB,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,cAAc,CAAC,MAAqB;IAC3C,IAAI,CAAC,MAAM;QAAE,OAAO,IAAI,CAAC;IACzB,MAAM,KAAK,GAAG,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC/C,MAAM,UAAU,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;IACrD,KAAK,MAAM,CAAC,IAAI,UAAU,EAAE,CAAC;QAC3B,MAAM,OAAO,GAAG,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;QACrE,IAAI,OAAO,CAAC,MAAM,GAAG,EAAE;YAAE,OAAO,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IACxD,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,MAAM,CAAC,IAAY;IACvC,MAAM,SAAS,GAAG;QAChB,qBAAqB,EAAE,mBAAmB,EAAE,sBAAsB;QAClE,eAAe,EAAE,eAAe,EAAE,iBAAiB;KACpD,CAAC;IACF,MAAM,OAAO,GAAG,CAAC,MAAM,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CACrE,CAAC,CAAC,EAAuB,EAAE,CAAC,CAAC,KAAK,IAAI,CACvC,CAAC;IACF,MAAM,SAAS,GAAG,MAAM,aAAa,CAAC,IAAI,CAAC,CAAC;IAC5C,MAAM,MAAM,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,WAAW,CAAC,CAAC;WACrD,MAAM,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,CAAC;IAE9C,MAAM,SAAS,GAAG,IAAI,GAAG,EAAU,CAAC;IACpC,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,IAAI,IAAI,GAAkB,IAAI,CAAC;IAC/B,IAAI,WAAW,GAAkB,IAAI,CAAC;IACtC,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;QACxB,KAAK,MAAM,IAAI,IAAI,CAAC,CAAC,SAAS;YAAE,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QACpD,KAAK,MAAM,IAAI,IAAI,CAAC,CAAC,KAAK;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QAC5C,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,IAAI;YAAE,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC;QACnC,IAAI,CAAC,WAAW,IAAI,CAAC,CAAC,WAAW;YAAE,WAAW,GAAG,CAAC,CAAC,WAAW,CAAC;IACjE,CAAC;IACD,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC;QAAE,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC/D,IAAI,CAAC,WAAW;QAAE,WAAW,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC;IAEvD,iEAAiE;IACjE,MAAM,WAAW,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAE7F,OAAO;QACL,IAAI;QACJ,WAAW;QACX,SAAS,EAAE,WAAW;QACtB,SAAS;QACT,WAAW,EAAE,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,KAAK,EAAE,GAAG,WAAW,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACjE,eAAe,EAAE,WAAW,CAAC,CAAC,CAAC,IAAI,IAAI;KACxC,CAAC;AACJ,CAAC;AAYD,MAAM,UAAU,oBAAoB,CAAC,OAA+B;IAClE,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,OAAO,CAAC,IAAI;QAAE,KAAK,CAAC,IAAI,CAAC,oBAAoB,OAAO,CAAC,IAAI,IAAI,CAAC,CAAC;IACnE,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;QACxB,6DAA6D;QAC7D,gEAAgE;QAChE,uDAAuD;QACvD,0DAA0D;QAC1D,iEAAiE;QACjE,6DAA6D;QAC7D,oEAAoE;QACpE,+CAA+C;QAC/C,MAAM,IAAI,GAAG,OAAO,CAAC,WAAW,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;QAC7D,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC;IACtD,CAAC;IACD,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC;QAC5B,MAAM,UAAU,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,OAAO,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CACtE,CAAC,CAAC,YAAY,EAAE,YAAY,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC;QAC7E,MAAM,aAAa,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,UAAU,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACjF,KAAK,CAAC,IAAI,CAAC,kBAAkB,OAAO,CAAC,eAAe,GAAG,aAAa,GAAG,CAAC,CAAC;IAC3E,CAAC;IACD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,uEAAuE,CAAC;IACvG,OAAO,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACzB,CAAC;AAED,sEAAsE;AACtE,yEAAyE;AACzE,wEAAwE;AACxE,oEAAoE;AACpE,wEAAwE;AACxE,yEAAyE;AACzE,QAAQ;AACR,OAAO,EAAE,aAAa,EAAE,cAAc,EAAE,CAAC"}

package/dist/core/diff-findings.d.ts

@@ -0,0 +1,96 @@
+/** One parsed finding from a prior `docs/reviews/PR-<n>.md`. */
+export type ParsedFinding = {
+    /** Relative path. Falls back to '(unknown file)' when the prior file
+     *  itself rendered the unknown-file marker; we keep it intact for
+     *  identity stability. */
+    file: string;
+    /**
+     * 1-indexed line number. Zero when the prior file had no `:N` suffix
+     * (cross-cutting findings). Zero participates in identity as-is.
+     */
+    line: number;
+    severity: 'critical' | 'minor' | 'preexisting';
+    skillName: string;
+    summary: string;
+};
+export type ParsedReview = {
+    findings: ParsedFinding[];
+};
+/**
+ * Parse a `docs/reviews/PR-<n>.md` markdown back into structured findings.
+ *
+ * Robust to:
+ *   - `null` / `undefined` input            → returns `null`
+ *   - empty / whitespace-only markdown      → returns `null`
+ *   - missing severity sections             → those sections contribute 0
+ *   - lines that don't match the SPEC bullet shape → silently dropped
+ *   - mixed unknown-file markers (`(unknown file)` vs absent)
+ *
+ * Parsing strategy: walk top-down, switch active severity on each
+ * `### <emoji> <Label>` header (one of the three SPEC §1.8.1 buckets),
+ * then collect every line starting with `- **` until the next header
+ * (or `---` end marker). Each `- **<file>:<line>** — <skill>: <summary>`
+ * line is split into its 4 fields; `:<line>` is optional (cross-cutting).
+ *
+ * `(unknown file)` files are preserved verbatim — they participate in
+ * identity, so the same cross-cutting finding can still be diffed across
+ * rounds even when neither pass has a line anchor.
+ */
+export declare function parsePriorReviewFile(markdown: string | null | undefined): ParsedReview | null;
+/**
+ * Diff prior vs current.
+ *
+ * `resolvedFindings`: findings that appeared in `prior` but NOT in
+ * `current`. The PR author (or auto-fix) addressed them.
+ *
+ * `stillOpenFindings`: findings that appeared in BOTH `prior` and
+ * `current`. These are persistent — the PR author hasn't fixed them
+ * (or the bot still considers them findings post-fix-push).
+ *
+ * Findings unique to `current` (newly raised this round) appear in
+ * neither list — those are surfaced directly by the renderer's normal
+ * severity-bucket emission.
+ *
+ * Order is preserved from `prior` for stability across rounds.
+ */
+export declare function diffFindings(prior: ParsedReview | null, current: {
+    critical_findings?: Array<{
+        skill: string;
+        file?: string;
+        line?: number;
+        summary: string;
+    }>;
+    minor_findings?: Array<{
+        skill: string;
+        file?: string;
+        line?: number;
+        summary: string;
+    }>;
+    preexisting_findings?: Array<{
+        skill: string;
+        file?: string;
+        line?: number;
+        summary: string;
+    }>;
+}): {
+    resolvedFindings: ParsedFinding[];
+    stillOpenFindings: ParsedFinding[];
+};
+/**
+ * Stable identity for a finding. Used for diffing prior vs current
+ * across review rounds AND (by intention) shareable with the
+ * clud-bug-app inline-thread anchor hash so future cross-feature
+ * surfaces (e.g. "the auto-fix that resolved this thread also resolves
+ * this doc-file finding") align without re-computing.
+ *
+ * Exposed for tests + downstream callers that want to align their own
+ * finding storage on the same scheme.
+ */
+export declare function findingIdentity(f: {
+    file: string;
+    line: number;
+    severity: 'critical' | 'minor' | 'preexisting';
+    skillName: string;
+    summary: string;
+}): string;
+//# sourceMappingURL=diff-findings.d.ts.map

package/dist/core/diff-findings.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-findings.d.ts","sourceRoot":"","sources":["../../src/core/diff-findings.ts"],"names":[],"mappings":"AAgCA,gEAAgE;AAChE,MAAM,MAAM,aAAa,GAAG;IAC1B;;8BAE0B;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb;;;OAGG;IACH,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,UAAU,GAAG,OAAO,GAAG,aAAa,CAAC;IAC/C,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IAAE,QAAQ,EAAE,aAAa,EAAE,CAAA;CAAE,CAAC;AAEzD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,oBAAoB,CAClC,QAAQ,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAClC,YAAY,GAAG,IAAI,CA+DrB;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,YAAY,CAC1B,KAAK,EAAE,YAAY,GAAG,IAAI,EAC1B,OAAO,EAAE;IACP,iBAAiB,CAAC,EAAE,KAAK,CAAC;QACxB,KAAK,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,OAAO,EAAE,MAAM,CAAC;KACjB,CAAC,CAAC;IACH,cAAc,CAAC,EAAE,KAAK,CAAC;QACrB,KAAK,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,OAAO,EAAE,MAAM,CAAC;KACjB,CAAC,CAAC;IACH,oBAAoB,CAAC,EAAE,KAAK,CAAC;QAC3B,KAAK,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,OAAO,EAAE,MAAM,CAAC;KACjB,CAAC,CAAC;CACJ,GACA;IACD,gBAAgB,EAAE,aAAa,EAAE,CAAC;IAClC,iBAAiB,EAAE,aAAa,EAAE,CAAC;CACpC,CAgDA;AAED;;;;;;;;;GASG;AACH,wBAAgB,eAAe,CAAC,CAAC,EAAE;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,UAAU,GAAG,OAAO,GAAG,aAAa,CAAC;IAC/C,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;CACjB,GAAG,MAAM,CAST"}

package/dist/core/diff-findings.js

@@ -0,0 +1,261 @@
+// SPEC §1.8.1 multi-pass diff: prior `docs/reviews/PR-<n>.md` vs current
+// review findings. Produces the `**Resolved this round:**` /
+// `**Still open:**` lists that `renderReviewFile` consumes.
+//
+// Identity model
+// --------------
+// Each finding's identity is a stable hash of:
+//
+//   `${file}:${line}:${severity}:${skillName}:${summary.slice(0, 100)}`
+//
+// A 100-character truncation on `summary` is enough to discriminate
+// distinct findings without being so long that whitespace/punctuation
+// drift between passes breaks identity. A severity-bucket change (e.g.
+// the same skill flags the same line as 🔴 then 🟡 the next round)
+// produces a DIFFERENT identity — by design — so a severity downgrade
+// counts as one resolved finding + one new finding (the user gets
+// credit for the fix AND can see the bot still has a concern, just at
+// a lower severity).
+//
+// The same identity shape is what auto-fix / auto-resolve in
+// clud-bug-app uses for thread anchoring; keeping it identical means a
+// future per-finding cross-reference between the doc-file diff and the
+// inline-thread surface is trivial.
+import { flattenFindings, } from './review-schema-zod.js';
+import { SEVERITY_EMOJI } from './review-writeback.js';
+/**
+ * Parse a `docs/reviews/PR-<n>.md` markdown back into structured findings.
+ *
+ * Robust to:
+ *   - `null` / `undefined` input            → returns `null`
+ *   - empty / whitespace-only markdown      → returns `null`
+ *   - missing severity sections             → those sections contribute 0
+ *   - lines that don't match the SPEC bullet shape → silently dropped
+ *   - mixed unknown-file markers (`(unknown file)` vs absent)
+ *
+ * Parsing strategy: walk top-down, switch active severity on each
+ * `### <emoji> <Label>` header (one of the three SPEC §1.8.1 buckets),
+ * then collect every line starting with `- **` until the next header
+ * (or `---` end marker). Each `- **<file>:<line>** — <skill>: <summary>`
+ * line is split into its 4 fields; `:<line>` is optional (cross-cutting).
+ *
+ * `(unknown file)` files are preserved verbatim — they participate in
+ * identity, so the same cross-cutting finding can still be diffed across
+ * rounds even when neither pass has a line anchor.
+ */
+export function parsePriorReviewFile(markdown) {
+    if (markdown == null)
+        return null;
+    if (markdown.trim() === '')
+        return null;
+    const lines = markdown.split('\n');
+    const out = [];
+    let current = null;
+    for (const raw of lines) {
+        // Strip trailing whitespace; leading indent is meaningful for
+        // sub-lines (Reasoning, attribution) which we deliberately ignore.
+        const line = raw.replace(/\s+$/, '');
+        // The SPEC §1.8.1 trailing `---` separator ends the findings region.
+        // Anything after it (the [Link to PR] line) is metadata and parsed
+        // by short-circuit so we don't mistake the literal `---` for a
+        // missing header.
+        if (line === '---') {
+            current = null;
+            continue;
+        }
+        // Switch active severity on every `### <emoji> <Label>` heading.
+        // We pattern-match on the emoji codepoint (not the label text) so a
+        // future SPEC tweak to "Critical" → "Blocking" doesn't break us.
+        if (line.startsWith('### ')) {
+            if (line.includes(SEVERITY_EMOJI.critical)) {
+                current = 'critical';
+            }
+            else if (line.includes(SEVERITY_EMOJI.minor)) {
+                current = 'minor';
+            }
+            else if (line.includes(SEVERITY_EMOJI.preexisting)) {
+                current = 'preexisting';
+            }
+            else {
+                current = null;
+            }
+            continue;
+        }
+        // Resolved / Still-open blocks under SPEC §1.8.1 — these list
+        // findings from PRIOR rounds, NOT this-round findings. Skip them
+        // so a multi-round PR doesn't double-count its own history.
+        // (`**Resolved this round:**` / `**Still open:**` headings.)
+        if (line.startsWith('**Resolved this round:') || line.startsWith('**Still open:')) {
+            current = null;
+            continue;
+        }
+        if (current == null)
+            continue;
+        // SPEC §1.8.1 bullet: `- **<file>:<line>** — <skill>: <summary>`
+        //                  OR `- **<file>** — <skill>: <summary>`
+        //                  OR `- **(unknown file)** — <skill>: <summary>`
+        //
+        // The multi-pass renderer prepends `[Pass N — Role · model]` to the
+        // bullet; we accept that prefix and discard it for parsing.
+        const parsed = parseFindingBullet(line);
+        if (parsed) {
+            out.push({ ...parsed, severity: current });
+        }
+    }
+    if (out.length === 0)
+        return null;
+    return { findings: out };
+}
+/**
+ * Diff prior vs current.
+ *
+ * `resolvedFindings`: findings that appeared in `prior` but NOT in
+ * `current`. The PR author (or auto-fix) addressed them.
+ *
+ * `stillOpenFindings`: findings that appeared in BOTH `prior` and
+ * `current`. These are persistent — the PR author hasn't fixed them
+ * (or the bot still considers them findings post-fix-push).
+ *
+ * Findings unique to `current` (newly raised this round) appear in
+ * neither list — those are surfaced directly by the renderer's normal
+ * severity-bucket emission.
+ *
+ * Order is preserved from `prior` for stability across rounds.
+ */
+export function diffFindings(prior, current) {
+    if (prior === null || prior.findings.length === 0) {
+        return { resolvedFindings: [], stillOpenFindings: [] };
+    }
+    // Build identity set for the current round. We do not need the
+    // current-round ParsedFinding objects — we only need to know which
+    // identities are still present.
+    //
+    // Cast through `Review` shape — the schema's three arrays mirror our
+    // input slice exactly (skill / file / line / summary), so we can lean
+    // on `flattenFindings` to produce a tagged list.
+    const currentReview = {
+        status_header: 'clean',
+        summary_counts: {
+            critical: 0,
+            minor: 0,
+            preexisting: 0,
+            resolved_from_prior: 0,
+            still_open: 0,
+        },
+        per_skill_scan: [],
+        critical_findings: current.critical_findings ?? [],
+        minor_findings: current.minor_findings ?? [],
+        preexisting_findings: current.preexisting_findings ?? [],
+        skills_referenced: [],
+        last_reviewed_sha: '',
+    };
+    const currentFlat = flattenFindings(currentReview);
+    const currentIds = new Set(currentFlat.map((f) => findingIdentity({
+        file: f.file ?? '(unknown file)',
+        line: f.line ?? 0,
+        severity: f.severity,
+        skillName: f.skill,
+        summary: f.summary,
+    })));
+    const resolved = [];
+    const stillOpen = [];
+    for (const f of prior.findings) {
+        const id = findingIdentity(f);
+        if (currentIds.has(id)) {
+            stillOpen.push(f);
+        }
+        else {
+            resolved.push(f);
+        }
+    }
+    return { resolvedFindings: resolved, stillOpenFindings: stillOpen };
+}
+/**
+ * Stable identity for a finding. Used for diffing prior vs current
+ * across review rounds AND (by intention) shareable with the
+ * clud-bug-app inline-thread anchor hash so future cross-feature
+ * surfaces (e.g. "the auto-fix that resolved this thread also resolves
+ * this doc-file finding") align without re-computing.
+ *
+ * Exposed for tests + downstream callers that want to align their own
+ * finding storage on the same scheme.
+ */
+export function findingIdentity(f) {
+    // Truncate summary to 100 chars to absorb whitespace/punctuation
+    // drift between rounds without losing discrimination between
+    // genuinely-distinct findings (the SPEC §1.8.1 summary line is
+    // user-visible so it tends to be stable; 100 chars is enough for any
+    // realistic distinct summary while tolerating "fix the X" → "fix X"
+    // drift).
+    const summaryPart = f.summary.slice(0, 100);
+    return `${f.file}:${f.line}:${f.severity}:${f.skillName}:${summaryPart}`;
+}
+// ---------------------------------------------------------------------------
+// Parsing internals
+// ---------------------------------------------------------------------------
+/**
+ * Parse one SPEC §1.8.1 bullet line into a {file, line, skill, summary}
+ * tuple. Returns null for any line that doesn't match the SPEC shape.
+ *
+ * Accepts both shapes:
+ *   `- **<file>:<line>** — <skill>: <summary>`
+ *   `- **<file>** — <skill>: <summary>`
+ *
+ * Also tolerates the multi-pass attribution prefix (D.2.5):
+ *   `- [Pass 1 — Role · model] **<file>:<line>** — <skill>: <summary>`
+ *
+ * Em-dash recognition: the SPEC pins U+2014 EM DASH between location
+ * and skill. Some downstream tools have been observed using `--` or
+ * regular hyphens; we accept both for resilience.
+ */
+function parseFindingBullet(line) {
+    // Strip the leading `- ` bullet marker.
+    if (!line.startsWith('- '))
+        return null;
+    let rest = line.slice(2);
+    // Strip optional `[Pass N — ...] ` attribution prefix (D.2.5).
+    if (rest.startsWith('[')) {
+        const closeIdx = rest.indexOf('] ');
+        if (closeIdx === -1)
+            return null;
+        rest = rest.slice(closeIdx + 2);
+    }
+    // Expect `**<location>** ` next.
+    if (!rest.startsWith('**'))
+        return null;
+    const locEnd = rest.indexOf('**', 2);
+    if (locEnd === -1)
+        return null;
+    const location = rest.slice(2, locEnd);
+    rest = rest.slice(locEnd + 2);
+    // Strip the location separator. SPEC pins ` — ` (U+2014 surrounded by
+    // spaces). We accept hyphen-minus variants as a courtesy.
+    // The separator may be ` — `, ` -- `, or ` - `.
+    const sepMatch = rest.match(/^\s+(?:—|--|-)\s+/);
+    if (!sepMatch)
+        return null;
+    rest = rest.slice(sepMatch[0].length);
+    // `<skill>: <summary>` — split on the first `: `.
+    const sepIdx = rest.indexOf(': ');
+    if (sepIdx === -1)
+        return null;
+    const skillName = rest.slice(0, sepIdx).trim();
+    const summary = rest.slice(sepIdx + 2).trim();
+    if (skillName === '' || summary === '')
+        return null;
+    // Split location into file + optional line. Walk RIGHTWARD from the
+    // last `:` so file names containing colons (Windows-style, rare) on
+    // the LHS don't confuse us.
+    const colonIdx = location.lastIndexOf(':');
+    let file = location;
+    let lineNum = 0;
+    if (colonIdx !== -1) {
+        const tail = location.slice(colonIdx + 1);
+        if (/^\d+$/.test(tail)) {
+            file = location.slice(0, colonIdx);
+            lineNum = Number(tail);
+        }
+    }
+    return { file, line: lineNum, skillName, summary };
+}
+//# sourceMappingURL=diff-findings.js.map

package/dist/core/diff-findings.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-findings.js","sourceRoot":"","sources":["../../src/core/diff-findings.ts"],"names":[],"mappings":"AAAA,yEAAyE;AACzE,6DAA6D;AAC7D,4DAA4D;AAC5D,EAAE;AACF,iBAAiB;AACjB,iBAAiB;AACjB,+CAA+C;AAC/C,EAAE;AACF,wEAAwE;AACxE,EAAE;AACF,oEAAoE;AACpE,sEAAsE;AACtE,uEAAuE;AACvE,mEAAmE;AACnE,sEAAsE;AACtE,kEAAkE;AAClE,sEAAsE;AACtE,qBAAqB;AACrB,EAAE;AACF,6DAA6D;AAC7D,uEAAuE;AACvE,uEAAuE;AACvE,oCAAoC;AAEpC,OAAO,EACL,eAAe,GAGhB,MAAM,wBAAwB,CAAC;AAEhC,OAAO,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAoBvD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,oBAAoB,CAClC,QAAmC;IAEnC,IAAI,QAAQ,IAAI,IAAI;QAAE,OAAO,IAAI,CAAC;IAClC,IAAI,QAAQ,CAAC,IAAI,EAAE,KAAK,EAAE;QAAE,OAAO,IAAI,CAAC;IAExC,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IACnC,MAAM,GAAG,GAAoB,EAAE,CAAC;IAChC,IAAI,OAAO,GAAqC,IAAI,CAAC;IAErD,KAAK,MAAM,GAAG,IAAI,KAAK,EAAE,CAAC;QACxB,8DAA8D;QAC9D,mEAAmE;QACnE,MAAM,IAAI,GAAG,GAAG,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;QAErC,qEAAqE;QACrE,mEAAmE;QACnE,+DAA+D;QAC/D,kBAAkB;QAClB,IAAI,IAAI,KAAK,KAAK,EAAE,CAAC;YACnB,OAAO,GAAG,IAAI,CAAC;YACf,SAAS;QACX,CAAC;QAED,iEAAiE;QACjE,oEAAoE;QACpE,iEAAiE;QACjE,IAAI,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC,EAAE,CAAC;YAC5B,IAAI,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC3C,OAAO,GAAG,UAAU,CAAC;YACvB,CAAC;iBAAM,IAAI,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC/C,OAAO,GAAG,OAAO,CAAC;YACpB,CAAC;iBAAM,IAAI,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC,WAAW,CAAC,EAAE,CAAC;gBACrD,OAAO,GAAG,aAAa,CAAC;YAC1B,CAAC;iBAAM,CAAC;gBACN,OAAO,GAAG,IAAI,CAAC;YACjB,CAAC;YACD,SAAS;QACX,CAAC;QAED,8DAA8D;QAC9D,iEAAiE;QACjE,4DAA4D;QAC5D,6DAA6D;QAC7D,IAAI,IAAI,CAAC,UAAU,CAAC,wBAAwB,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,eAAe,CAAC,EAAE,CAAC;YAClF,OAAO,GAAG,IAAI,CAAC;YACf,SAAS;QACX,CAAC;QAED,IAAI,OAAO,IAAI,IAAI;YAAE,SAAS;QAE9B,iEAAiE;QACjE,0DAA0D;QAC1D,kEAAkE;QAClE,EAAE;QACF,oEAAoE;QACpE,4DAA4D;QAC5D,MAAM,MAAM,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;QACxC,IAAI,MAAM,EAAE,CAAC;YACX,GAAG,CAAC,IAAI,CAAC,EAAE,GAAG,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC,CAAC;QAC7C,CAAC;IACH,CAAC;IAED,IAAI,GAAG,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IAClC,OAAO,EAAE,QAAQ,EAAE,GAAG,EAAE,CAAC;AAC3B,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,YAAY,CAC1B,KAA0B,EAC1B,OAmBC;IAKD,IAAI,KAAK,KAAK,IAAI,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAClD,OAAO,EAAE,gBAAgB,EAAE,EAAE,EAAE,iBAAiB,EAAE,EAAE,EAAE,CAAC;IACzD,CAAC;IAED,+DAA+D;IAC/D,mEAAmE;IACnE,gCAAgC;IAChC,EAAE;IACF,qEAAqE;IACrE,sEAAsE;IACtE,iDAAiD;IACjD,MAAM,aAAa,GAAW;QAC5B,aAAa,EAAE,OAAO;QACtB,cAAc,EAAE;YACd,QAAQ,EAAE,CAAC;YACX,KAAK,EAAE,CAAC;YACR,WAAW,EAAE,CAAC;YACd,mBAAmB,EAAE,CAAC;YACtB,UAAU,EAAE,CAAC;SACd;QACD,cAAc,EAAE,EAAE;QAClB,iBAAiB,EAAE,OAAO,CAAC,iBAAiB,IAAI,EAAE;QAClD,cAAc,EAAE,OAAO,CAAC,cAAc,IAAI,EAAE;QAC5C,oBAAoB,EAAE,OAAO,CAAC,oBAAoB,IAAI,EAAE;QACxD,iBAAiB,EAAE,EAAE;QACrB,iBAAiB,EAAE,EAAE;KACtB,CAAC;IACF,MAAM,WAAW,GAAc,eAAe,CAAC,aAAa,CAAC,CAAC;IAC9D,MAAM,UAAU,GAAG,IAAI,GAAG,CAAS,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC;QACxE,IAAI,EAAE,CAAC,CAAC,IAAI,IAAI,gBAAgB;QAChC,IAAI,EAAE,CAAC,CAAC,IAAI,IAAI,CAAC;QACjB,QAAQ,EAAE,CAAC,CAAC,QAAQ;QACpB,SAAS,EAAE,CAAC,CAAC,KAAK;QAClB,OAAO,EAAE,CAAC,CAAC,OAAO;KACnB,CAAC,CAAC,CAAC,CAAC;IAEL,MAAM,QAAQ,GAAoB,EAAE,CAAC;IACrC,MAAM,SAAS,GAAoB,EAAE,CAAC;IACtC,KAAK,MAAM,CAAC,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;QAC/B,MAAM,EAAE,GAAG,eAAe,CAAC,CAAC,CAAC,CAAC;QAC9B,IAAI,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC;YACvB,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACpB,CAAC;aAAM,CAAC;YACN,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACnB,CAAC;IACH,CAAC;IACD,OAAO,EAAE,gBAAgB,EAAE,QAAQ,EAAE,iBAAiB,EAAE,SAAS,EAAE,CAAC;AACtE,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,eAAe,CAAC,CAM/B;IACC,iEAAiE;IACjE,6DAA6D;IAC7D,+DAA+D;IAC/D,qEAAqE;IACrE,oEAAoE;IACpE,UAAU;IACV,MAAM,WAAW,GAAG,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC5C,OAAO,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,SAAS,IAAI,WAAW,EAAE,CAAC;AAC3E,CAAC;AAED,8EAA8E;AAC9E,oBAAoB;AACpB,8EAA8E;AAE9E;;;;;;;;;;;;;;GAcG;AACH,SAAS,kBAAkB,CAAC,IAAY;IACtC,wCAAwC;IACxC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC;IACxC,IAAI,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;IAEzB,+DAA+D;IAC/D,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QACzB,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QACpC,IAAI,QAAQ,KAAK,CAAC,CAAC;YAAE,OAAO,IAAI,CAAC;QACjC,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,GAAG,CAAC,CAAC,CAAC;IAClC,CAAC;IAED,iCAAiC;IACjC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC;IACxC,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IACrC,IAAI,MAAM,KAAK,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IAC/B,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,CAAC;IACvC,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IAE9B,sEAAsE;IACtE,0DAA0D;IAC1D,gDAAgD;IAChD,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;IACjD,IAAI,CAAC,QAAQ;QAAE,OAAO,IAAI,CAAC;IAC3B,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAEtC,kDAAkD;IAClD,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAClC,IAAI,MAAM,KAAK,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IAC/B,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,CAAC,IAAI,EAAE,CAAC;IAC/C,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;IAC9C,IAAI,SAAS,KAAK,EAAE,IAAI,OAAO,KAAK,EAAE;QAAE,OAAO,IAAI,CAAC;IAEpD,oEAAoE;IACpE,oEAAoE;IACpE,4BAA4B;IAC5B,MAAM,QAAQ,GAAG,QAAQ,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAC3C,IAAI,IAAI,GAAG,QAAQ,CAAC;IACpB,IAAI,OAAO,GAAG,CAAC,CAAC;IAChB,IAAI,QAAQ,KAAK,CAAC,CAAC,EAAE,CAAC;QACpB,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,CAAC,QAAQ,GAAG,CAAC,CAAC,CAAC;QAC1C,IAAI,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACvB,IAAI,GAAG,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC;YACnC,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;QACzB,CAAC;IACH,CAAC;IAED,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,CAAC;AACrD,CAAC"}