@delegance/claude-autopilot 5.0.7 → 5.0.8

package/dist/src/adapters/review-engine/parse-output.js

@@ -35,10 +35,20 @@ const FILE_REF = new RegExp(String.raw `(?:` +
     String.raw `\x60([^\x60]+\.[a-z]{1,6})\x60` +
     String.raw `|(\b[\w./\-]+\.` + CODE_EXT + String.raw `)(?::(\d+))?` +
     String.raw `)`, 'i');
+// Matches "line 42", "on line 42", "at line 42" — used as a fallback when the
+// LLM mentions a line number separately from the file ref. Critical for `fix`:
+// without a line, the fixer can't extract a code snippet, so findings without
+// `line` got silently dropped from `fix --dry-run` (the path-only finding case
+// was the most-cited demo torpedo from the 5.0.7 stress test).
+const LINE_REF = /\b(?:on |at )?line\s+(\d+)\b/i;
 function extractFileRef(text) {
     const m = text.match(FILE_REF);
-    if (!m)
-        return { file: '<unspecified>' };
+    if (!m) {
+        // No file ref at all — but maybe the body still has "line N" prose we can
+        // surface. Caller treats file `<unspecified>` as a sentinel either way.
+        const lm = text.match(LINE_REF);
+        return lm ? { file: '<unspecified>', line: parseInt(lm[1], 10) } : { file: '<unspecified>' };
+    }
     const raw = (m[1] ?? m[2]);
     // Skip version strings (v1.2.3), bare dotfile extensions with no path
     // separator, and known prose abbreviations that slipped through the regex
@@ -47,10 +57,16 @@ function extractFileRef(text) {
     if (/^v?\d/.test(raw) ||
         (!raw.includes('/') && raw.startsWith('.') && raw.split('.').length === 2) ||
         /^(?:e\.g|i\.e|etc|vs|cf|al|U\.S|U\.K)$/i.test(raw)) {
-        return { file: '<unspecified>' };
+        const lm = text.match(LINE_REF);
+        return lm ? { file: '<unspecified>', line: parseInt(lm[1], 10) } : { file: '<unspecified>' };
     }
-    const line = m[3] ? parseInt(m[3], 10) : undefined;
-    return { file: raw, line };
+    // Prefer the colon-line from the file ref (`foo.ts:42`); fall back to a
+    // separately-mentioned line ("line 42") only when the file ref didn't carry one.
+    const colonLine = m[3] ? parseInt(m[3], 10) : undefined;
+    if (colonLine !== undefined)
+        return { file: raw, line: colonLine };
+    const lm = text.match(LINE_REF);
+    return lm ? { file: raw, line: parseInt(lm[1], 10) } : { file: raw };
 }
 // Accepts any of: `### [CRITICAL] title`, `### CRITICAL title`, `### **CRITICAL** title`,
 // `### **[CRITICAL]** title`. Severity capture works across variants.

package/dist/src/cli/fix.js

@@ -38,8 +38,13 @@ export async function runFix(options = {}) {
         console.log(fmt('yellow', '[fix] No cached findings — run `guardrail scan <path>` or `guardrail run` first.'));
         return 0;
     }
-    const fixable = findings.filter(f => {
-        if (!f.line || !f.file || f.file === '<unspecified>' || f.file === '<pipeline>')
+    // Two gates:
+    //  - "actionable": has a real file path. Surfaced in dry-run so the user sees
+    //    findings even when the LLM didn't pin a line number.
+    //  - "fixable": also has a line. The LLM-fix loop needs both to extract a
+    //    code snippet around the finding location.
+    const actionable = findings.filter(f => {
+        if (!f.file || f.file === '<unspecified>' || f.file === '<pipeline>')
             return false;
         if (severityFilter === 'all')
             return true;
@@ -47,8 +52,21 @@ export async function runFix(options = {}) {
             return f.severity === 'critical';
         return f.severity === 'critical' || f.severity === 'warning';
     });
+    const fixable = actionable.filter(f => f.line && f.line > 0);
+    if (actionable.length === 0) {
+        console.log(fmt('yellow', `[fix] No actionable findings (severity=${severityFilter}, need file path).`));
+        return 0;
+    }
     if (fixable.length === 0) {
-        console.log(fmt('yellow', `[fix] No fixable findings (severity=${severityFilter}, need file+line).`));
+        const verb = actionable.length === 1 ? 'has' : 'have';
+        const noun = actionable.length === 1 ? 'finding' : 'findings';
+        console.log(fmt('yellow', `[fix] ${actionable.length} ${noun} ${verb} file but no line — model output was line-less. Re-run scan with --ask "include line numbers" or run \`claude-autopilot run\` for richer extraction.`));
+        for (const f of actionable) {
+            const sev = f.severity === 'critical' ? fmt('red', 'CRITICAL')
+                : f.severity === 'warning' ? fmt('yellow', 'WARNING ')
+                    : fmt('dim', 'NOTE    ');
+            console.log(`  [${sev}] ${fmt('dim', f.file)} ${f.message}`);
+        }
         return 0;
     }
     const modeNote = options.dryRun ? ' (dry run)' : options.yes ? '' : ' (interactive — use --yes to skip prompts)';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@delegance/claude-autopilot",
-  "version": "5.0.7",
+  "version": "5.0.8",
   "type": "module",
   "description": "Autonomous development pipeline for Claude Code: brainstorm → spec → plan → implement → migrate → validate → PR → review → merge. Multi-model, local-first, every phase a skill you can intervene in.",
   "keywords": [