fix: remove incorrect skip logic for overconfident_comment that was causing false negatives

The skip logic at lines 825-831 was inverted - it was skipping lines that START with
// (actual comments with overconfident language) instead of only skipping lines where
the pattern appeared in non-comment contexts.

This caused the detector to miss true positives like:
// This is obviously the right approach
// The code clearly handles edge cases

missed-true-positives.ts now correctly shows 17 issues (up from 9).

Author: CodeDeficient

ai-slop-detector.ts

@@ -822,13 +822,7 @@ class AISlopDetector {
             }
           }
 
-          // Skip comment-line-only patterns that only make sense in actual code comments
-          if (pattern.id === 'overconfident_comment') {
-            const trimmed = line.trim();
-            if (trimmed.startsWith('//') || trimmed.startsWith('*') || trimmed.startsWith('/*')) {
-              continue;
-            }
-          }
+
 
           // In quiet mode, skip test and mock files for all patterns except production console logs
           if (quiet && pattern.id !== 'production_console_log') {

karpeslop-bin.js

@@ -691,14 +691,6 @@ class AISlopDetector {
             }
           }
 
-          // Skip comment-line-only patterns that only make sense in actual code comments
-          if (pattern.id === 'overconfident_comment') {
-            const trimmed = line.trim();
-            if (trimmed.startsWith('//') || trimmed.startsWith('*') || trimmed.startsWith('/*')) {
-              continue;
-            }
-          }
-
           // In quiet mode, skip test and mock files for all patterns except production console logs
           if (quiet && pattern.id !== 'production_console_log') {
             const isTestFile = filePath.includes('__tests__') || filePath.includes('.test.') || filePath.includes('.spec.') || filePath.includes('__mocks__') || filePath.includes('test-');