fix(lint): report physical JSONL line numbers (blank lines no longer shift them)

Author: Andy00L

src/commands/test.test.ts

@@ -2775,10 +2775,12 @@ describe('runLint', () => {
     expect(report).toMatchObject({ checked: 1, valid: 1, issues: [] });
   });
 
-  it('a JSONL file reports each bad line with its line number', async () => {
+  it('a JSONL file reports each bad line with its PHYSICAL line number (blank lines skipped, not renumbered)', async () => {
     const dir = mkdtempSync(join(tmpdir(), 'cli-lint-jsonl-'));
     const file = join(dir, 'plans.jsonl');
-    writeFileSync(file, `${VALID_PLAN}\nnot json at all\n${INVALID_PLAN}\n`, 'utf8');
+    // A blank separator line sits between entries: line 1 valid, line 2 blank,
+    // line 3 bad JSON, line 4 invalid plan. Reported numbers must be 3 and 4.
+    writeFileSync(file, `${VALID_PLAN}\n\nnot json at all\n${INVALID_PLAN}\n`, 'utf8');
     const out: string[] = [];
     const rejection = await runLint(
       { profile: 'default', output: 'json', debug: false, plans: file },
@@ -2787,8 +2789,10 @@ describe('runLint', () => {
     expect(rejection).toMatchObject({ exitCode: 5 });
     const report = JSON.parse(out.join('')) as { checked: number; issues: Array<{ file: string }> };
     expect(report.checked).toBe(3);
-    expect(report.issues.some(issue => issue.file.endsWith(':2'))).toBe(true);
     expect(report.issues.some(issue => issue.file.endsWith(':3'))).toBe(true);
+    expect(report.issues.some(issue => issue.file.endsWith(':4'))).toBe(true);
+    // The blank line itself is not an entry and never reports.
+    expect(report.issues.some(issue => issue.file.endsWith(':2'))).toBe(false);
   });
 
   it('requires exactly one input source', async () => {

src/commands/test.ts

@@ -3728,27 +3728,30 @@ export async function runLint(opts: LintOptions, deps: TestDeps = {}): Promise<C
     } catch {
       throw localValidationError('plans', `cannot read file: ${absolute}`);
     }
-    const lines = content
+    // Index lines BEFORE dropping blanks so every reported `file:N` points at
+    // the PHYSICAL line in the file (a blank separator line must not shift all
+    // subsequent line numbers).
+    const numberedLines = content
       .split('\n')
-      .map(line => line.trim())
-      .filter(line => line.length > 0);
-    if (lines.length === 0) throw localValidationError('plans', 'contains no plan lines');
-    lines.forEach((line, index) => {
-      collect(`${opts.plans}:${index + 1}`, () => {
+      .map((rawLine, physicalIndex) => ({ line: rawLine.trim(), lineNo: physicalIndex + 1 }))
+      .filter(entry => entry.line.length > 0);
+    if (numberedLines.length === 0) throw localValidationError('plans', 'contains no plan lines');
+    for (const { line, lineNo } of numberedLines) {
+      collect(`${opts.plans}:${lineNo}`, () => {
         let parsed: unknown;
         try {
           parsed = JSON.parse(line);
         } catch {
           throw localValidationError(
             'plans',
-            `line ${index + 1} is not valid JSON`,
+            `line ${lineNo} is not valid JSON`,
             undefined,
             'field',
           );
         }
-        assertPlanShape(parsed, { specIndex: index });
+        assertPlanShape(parsed, { specIndex: lineNo - 1 });
       });
-    });
+    }
   }
 
   const filesWithIssues = new Set(issues.map(issue => issue.file)).size;