fix(enrichment): restrict print() debug detection to Python paths

Avoid false positives on method calls like document.print().

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: bohdansolovie

review-enrichment/src/analyzers/debug-leftover.ts

@@ -15,11 +15,15 @@ const CONSOLE_RE = /\bconsole\s*\.\s*(?:log|debug|info|warn|error|trace|dir|tabl
 const PRINT_RE = /\bprint\s*\(/;
 
 /** Classify one added line for a debug leftover, or null. Pure. */
-export function detectDebugLeftover(line: string): DebugLeftoverFinding["kind"] | null {
+export function detectDebugLeftover(
+  line: string,
+  path?: string,
+): DebugLeftoverFinding["kind"] | null {
   const code = codeOnly(line);
   if (DEBUGGER_RE.test(code)) return "debugger";
   if (CONSOLE_RE.test(code)) return "console";
-  if (PRINT_RE.test(code)) return "print";
+  // Python-only: `\bprint` after a dot would false-positive on `document.print()` / `obj.print()`.
+  if (path && /\.pyi?$/i.test(path) && PRINT_RE.test(code)) return "print";
   return null;
 }
 
@@ -51,7 +55,7 @@ export function scanPatchForDebugLeftover(
     if (line.startsWith("+")) {
       const body = line.slice(1);
       if (body.length <= MAX_LINE_CHARS) {
-        const kind = detectDebugLeftover(body);
+        const kind = detectDebugLeftover(body, path);
         if (kind) {
           findings.push({ file: path, line: newLine, kind });
           if (findings.length >= maxFindings) return findings;

review-enrichment/test/debug-leftover.test.ts

@@ -16,7 +16,13 @@ test("detectDebugLeftover: recognizes debugger, console sinks, and print()", ()
   assert.equal(detectDebugLeftover("  debugger;"), "debugger");
   assert.equal(detectDebugLeftover("console.log('hi')"), "console");
   assert.equal(detectDebugLeftover("  console.debug(state)"), "console");
-  assert.equal(detectDebugLeftover("print('debug')"), "print");
+  assert.equal(detectDebugLeftover("print('debug')", "lib/b.py"), "print");
+});
+
+test("detectDebugLeftover: print() is Python-only and does not match method calls like document.print()", () => {
+  assert.equal(detectDebugLeftover("document.print()"), null);
+  assert.equal(detectDebugLeftover("printer.print('x')"), null);
+  assert.equal(detectDebugLeftover("print('debug')", "src/widget.ts"), null);
 });
 
 test("detectDebugLeftover: a console call inside a string literal is not flagged", () => {