slopbrick 0.26.0 → 0.27.0

package/dist/index.js

@@ -19,7 +19,7 @@ var VERSION;
 var init_header = __esm({
   "src/types/_header.ts"() {
     "use strict";
-    VERSION = "0.26.0";
+    VERSION = "0.27.0";
   }
 });
 
@@ -30643,589 +30643,6 @@ var init_struct_tag_inconsistency = __esm({
   }
 });
 
-// src/rules/java/arraylist-vs-linkedlist.ts
-var NEW_LINKED_LIST_REGEX, javaArraylistVsLinkedlistRule;
-var init_arraylist_vs_linkedlist = __esm({
-  "src/rules/java/arraylist-vs-linkedlist.ts"() {
-    "use strict";
-    init_rule();
-    NEW_LINKED_LIST_REGEX = /new\s+LinkedList\s*</g;
-    javaArraylistVsLinkedlistRule = createRule({
-      id: "java/arraylist-vs-linkedlist",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "new LinkedList<>() \u2014 use ArrayList (Effective Java, Item 28)",
-      create(_context) {
-        return {};
-      },
-      analyze(_context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        let m;
-        NEW_LINKED_LIST_REGEX.lastIndex = 0;
-        while ((m = NEW_LINKED_LIST_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          issues.push({
-            ruleId: "java/arraylist-vs-linkedlist",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `new LinkedList at line ${line} \u2014 use ArrayList instead`,
-            line,
-            column: 1,
-            advice: "Replace `new LinkedList<>()` with `new ArrayList<>()`. LinkedList is rarely the right choice (worse cache locality, 5x more memory per element, O(n) indexed access). Joshua Bloch (Effective Java, Item 28) recommends ArrayList unless you specifically need a Deque. AI agents default to LinkedList because of textbook examples. Reference: java/arraylist-vs-linkedlist v0.20."
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/builder-overuse.ts
-var BUILDER_ANNOTATION_REGEX, FIELD_DECL_REGEX, DEFAULT_FIELD_COUNT_CAP, javaBuilderOveruseRule;
-var init_builder_overuse = __esm({
-  "src/rules/java/builder-overuse.ts"() {
-    "use strict";
-    init_rule();
-    BUILDER_ANNOTATION_REGEX = /@Builder\b/;
-    FIELD_DECL_REGEX = /(?:private|public|protected)\s+(?:final\s+)?[\w<>,\s]+\s+(\w+)\s*[=;]/g;
-    DEFAULT_FIELD_COUNT_CAP = 3;
-    javaBuilderOveruseRule = createRule({
-      id: "java/builder-overuse",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "@Builder on a class with few fields \u2014 plain constructor is simpler",
-      create(_context) {
-        return { fieldCountCap: DEFAULT_FIELD_COUNT_CAP };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        BUILDER_ANNOTATION_REGEX.lastIndex = 0;
-        const builderMatch = BUILDER_ANNOTATION_REGEX.exec(source);
-        if (!builderMatch) return issues;
-        const firstBuilderLine = source.slice(0, builderMatch.index).split("\n").length;
-        FIELD_DECL_REGEX.lastIndex = 0;
-        let fieldCount = 0;
-        while (FIELD_DECL_REGEX.exec(source) !== null) {
-          fieldCount++;
-        }
-        if (fieldCount > context.fieldCountCap) return issues;
-        issues.push({
-          ruleId: "java/builder-overuse",
-          category: "typo",
-          severity: "low",
-          aiSpecific: true,
-          message: `@Builder on a class with ${fieldCount} field(s) \u2014 plain constructor is simpler`,
-          line: firstBuilderLine,
-          column: 1,
-          advice: "Use a plain constructor for classes with \u2264 3 fields. AI agents default to @Builder because their training data emphasizes the pattern. Builder adds Lombok dependency and a Builder inner class for every annotated class. Reference: java/builder-overuse v0.26.0."
-        });
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/empty-catch-block.ts
-var SINGLE_LINE_EMPTY_CATCH_REGEX, javaEmptyCatchBlockRule;
-var init_empty_catch_block = __esm({
-  "src/rules/java/empty-catch-block.ts"() {
-    "use strict";
-    init_rule();
-    SINGLE_LINE_EMPTY_CATCH_REGEX = /catch\s*\([^)]*\)\s*\{\s*\}/g;
-    javaEmptyCatchBlockRule = createRule({
-      id: "java/empty-catch-block",
-      category: "logic",
-      severity: "medium",
-      aiSpecific: true,
-      description: "Empty catch block \u2014 silently swallows exceptions",
-      create(_context) {
-        return {};
-      },
-      analyze(_context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        let m;
-        SINGLE_LINE_EMPTY_CATCH_REGEX.lastIndex = 0;
-        while ((m = SINGLE_LINE_EMPTY_CATCH_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          issues.push({
-            ruleId: "java/empty-catch-block",
-            category: "logic",
-            severity: "medium",
-            aiSpecific: true,
-            message: `Empty catch block at line ${line} \u2014 exception is silently swallowed`,
-            line,
-            column: 1,
-            advice: 'Log the exception (`log.error("...", e)`), re-throw it, or both. Empty catch blocks hide bugs. The pattern is common in AI-generated code that wants to look defensive. Reference: java/empty-catch-block v0.20.'
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/immutable-collection-preference.ts
-var IMMUTABLE_REGEX, MUTABLE_REGEX, DEFAULT_IMMUTABLE_THRESHOLD, DEFAULT_MUTABLE_CAP, javaImmutableCollectionPreferenceRule;
-var init_immutable_collection_preference = __esm({
-  "src/rules/java/immutable-collection-preference.ts"() {
-    "use strict";
-    init_rule();
-    IMMUTABLE_REGEX = /\b(List|Map|Set)\.of\s*\(/g;
-    MUTABLE_REGEX = /new\s+(ArrayList|HashMap|HashSet|LinkedList|TreeMap)\s*[<(]/g;
-    DEFAULT_IMMUTABLE_THRESHOLD = 5;
-    DEFAULT_MUTABLE_CAP = 1;
-    javaImmutableCollectionPreferenceRule = createRule({
-      id: "java/immutable-collection-preference",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Immutable factory method over-use \u2014 prefer mutable when collection will be modified",
-      create(_context) {
-        return {
-          immutableThreshold: DEFAULT_IMMUTABLE_THRESHOLD,
-          mutableCap: DEFAULT_MUTABLE_CAP
-        };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        IMMUTABLE_REGEX.lastIndex = 0;
-        let immutableCount = 0;
-        let firstIdx = 0;
-        let m;
-        while ((m = IMMUTABLE_REGEX.exec(source)) !== null) {
-          immutableCount++;
-          if (firstIdx === 0) firstIdx = m.index;
-        }
-        if (immutableCount < context.immutableThreshold) return issues;
-        MUTABLE_REGEX.lastIndex = 0;
-        let mutableCount = 0;
-        while ((m = MUTABLE_REGEX.exec(source)) !== null) {
-          mutableCount++;
-        }
-        if (mutableCount > context.mutableCap) return issues;
-        const line = source.slice(0, firstIdx).split("\n").length;
-        issues.push({
-          ruleId: "java/immutable-collection-preference",
-          category: "typo",
-          severity: "low",
-          aiSpecific: true,
-          message: `${immutableCount} immutable factory calls (List/Map/Set.of) with only ${mutableCount} mutable collection \u2014 likely over-preferring immutability`,
-          line,
-          column: 1,
-          advice: "Prefer mutable collections (new ArrayList<>()) when the collection will be modified later. AI agents default to List.of/Map.of/Set.of because their training data emphasizes functional-style Java. Reference: java/immutable-collection-preference v0.26.0."
-        });
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/legacy-date-api.ts
-var LEGACY_IMPORT_REGEX, LEGACY_USAGE_REGEX, CALENDAR_GET_INSTANCE_REGEX, javaLegacyDateApiRule;
-var init_legacy_date_api = __esm({
-  "src/rules/java/legacy-date-api.ts"() {
-    "use strict";
-    init_rule();
-    LEGACY_IMPORT_REGEX = /^import\s+(?:static\s+)?java\.(?:util|sql)\.(?:Date|Calendar|GregorianCalendar)\s*;/gm;
-    LEGACY_USAGE_REGEX = /\bnew\s+(?:Date|GregorianCalendar)\s*\(/g;
-    CALENDAR_GET_INSTANCE_REGEX = /Calendar\.getInstance\s*\(/g;
-    javaLegacyDateApiRule = createRule({
-      id: "java/legacy-date-api",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Legacy java.util.Date / Calendar \u2014 use java.time (JSR-310) from Java 8+",
-      create(_context) {
-        return {};
-      },
-      analyze(_context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        const flagged = /* @__PURE__ */ new Set();
-        let m;
-        LEGACY_IMPORT_REGEX.lastIndex = 0;
-        while ((m = LEGACY_IMPORT_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          flagged.add(line);
-          issues.push({
-            ruleId: "java/legacy-date-api",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `Legacy date import at line ${line} \u2014 use java.time (JSR-310) from Java 8+`,
-            line,
-            column: 1,
-            advice: "Replace `java.util.Date` / `java.util.Calendar` with `java.time` (`LocalDate`, `LocalDateTime`, `Instant`, `ZonedDateTime`). java.time is immutable, thread-safe, and has a much better API. AI agents default to the legacy API because their training data predates Java 8 (2014). Reference: java/legacy-date-api v0.20."
-          });
-        }
-        LEGACY_USAGE_REGEX.lastIndex = 0;
-        while ((m = LEGACY_USAGE_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          if (flagged.has(line)) continue;
-          issues.push({
-            ruleId: "java/legacy-date-api",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `new Date() / GregorianCalendar at line ${line} \u2014 use java.time`,
-            line,
-            column: 1,
-            advice: "Replace with `LocalDate.now()`, `Instant.now()`, or `ZonedDateTime.now()`. Reference: java/legacy-date-api v0.20."
-          });
-        }
-        CALENDAR_GET_INSTANCE_REGEX.lastIndex = 0;
-        while ((m = CALENDAR_GET_INSTANCE_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          if (flagged.has(line)) continue;
-          issues.push({
-            ruleId: "java/legacy-date-api",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `Calendar.getInstance() at line ${line} \u2014 use java.time`,
-            line,
-            column: 1,
-            advice: "Replace with `LocalDate.now()` (date-only) or `ZonedDateTime.now()`. Reference: java/legacy-date-api v0.20."
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/optional-overuse.ts
-var OR_ELSE_THROW_REGEX, NULL_CHECK_REGEX, DEFAULT_THRESHOLD, javaOptionalOveruseRule;
-var init_optional_overuse = __esm({
-  "src/rules/java/optional-overuse.ts"() {
-    "use strict";
-    init_rule();
-    OR_ELSE_THROW_REGEX = /\.orElseThrow\s*\(/g;
-    NULL_CHECK_REGEX = /Objects\.requireNonNull\s*\(|if\s*\([^)]*==\s*null\)\s*(?:throw|return)/g;
-    DEFAULT_THRESHOLD = 2;
-    javaOptionalOveruseRule = createRule({
-      id: "java/optional-overuse",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Optional chain over-use \u2014 null check or Objects.requireNonNull is faster",
-      create(_context) {
-        return { threshold: DEFAULT_THRESHOLD };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        OR_ELSE_THROW_REGEX.lastIndex = 0;
-        let orElseThrowCount = 0;
-        let firstIdx = 0;
-        let m;
-        while ((m = OR_ELSE_THROW_REGEX.exec(source)) !== null) {
-          orElseThrowCount++;
-          if (firstIdx === 0) firstIdx = m.index;
-        }
-        if (orElseThrowCount < context.threshold) return issues;
-        NULL_CHECK_REGEX.lastIndex = 0;
-        const nullCheckCount = (source.match(NULL_CHECK_REGEX) ?? []).length;
-        const optionalRatio = orElseThrowCount / Math.max(orElseThrowCount + nullCheckCount, 1);
-        if (optionalRatio < 0.6) return issues;
-        const line = source.slice(0, firstIdx).split("\n").length;
-        issues.push({
-          ruleId: "java/optional-overuse",
-          category: "typo",
-          severity: "low",
-          aiSpecific: true,
-          message: `${orElseThrowCount} .orElseThrow() calls with ${nullCheckCount} null checks \u2014 Optional chain over-use`,
-          line,
-          column: 1,
-          advice: 'Use Objects.requireNonNull(x, "msg") for null checks; reserve Optional for return values. AI agents default to Optional chains because their training data emphasizes null-safety. Real Java code uses null checks in hot paths. Reference: java/optional-overuse v0.26.0.'
-        });
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/raw-type-overuse.ts
-var RAW_TYPE_REGEX, javaRawTypeOveruseRule;
-var init_raw_type_overuse = __esm({
-  "src/rules/java/raw-type-overuse.ts"() {
-    "use strict";
-    init_rule();
-    RAW_TYPE_REGEX = /\b(List|Map|Set|Collection|Iterable)\s+(?![<A-Z])(\w+)/g;
-    javaRawTypeOveruseRule = createRule({
-      id: "java/raw-type-overuse",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Raw type usage (List, Map, Set) \u2014 use generics (Effective Java, Item 23)",
-      create(_context) {
-        return {};
-      },
-      analyze(_context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        let m;
-        RAW_TYPE_REGEX.lastIndex = 0;
-        while ((m = RAW_TYPE_REGEX.exec(source)) !== null) {
-          const typeName = m[1];
-          const line = source.slice(0, m.index).split("\n").length;
-          issues.push({
-            ruleId: "java/raw-type-overuse",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `Raw type ${typeName} at line ${line} \u2014 add type parameters`,
-            line,
-            column: 1,
-            advice: `Replace raw \`${typeName}\` with \`${typeName}<...>\`. Raw types disable generic type checking. Effective Java, Item 23: 'Don't use raw types in new code'. AI agents default to raw types when unsure of the correct generic parameters. Reference: java/raw-type-overuse v0.20.`
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/stream-overuse.ts
-var STREAM_OPS, DEFAULT_CHAIN_THRESHOLD, javaStreamOveruseRule;
-var init_stream_overuse = __esm({
-  "src/rules/java/stream-overuse.ts"() {
-    "use strict";
-    init_rule();
-    STREAM_OPS = [
-      ".map(",
-      ".filter(",
-      ".flatMap(",
-      ".collect(",
-      ".reduce(",
-      ".sorted(",
-      ".distinct(",
-      ".limit(",
-      ".skip(",
-      ".anyMatch(",
-      ".allMatch(",
-      ".noneMatch(",
-      ".findFirst(",
-      ".findAny("
-    ];
-    DEFAULT_CHAIN_THRESHOLD = 3;
-    javaStreamOveruseRule = createRule({
-      id: "java/stream-overuse",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Stream API chain over-use \u2014 for-loop is faster for simple transformations",
-      create(_context) {
-        return { chainThreshold: DEFAULT_CHAIN_THRESHOLD };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        const lines = source.split("\n");
-        const streamOpPattern = new RegExp(STREAM_OPS.map((op) => op.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|"), "g");
-        let firstOffendingLine = 0;
-        let firstOffendingCount = 0;
-        for (let i = 0; i < lines.length; i++) {
-          const line = lines[i] ?? "";
-          if (line.trim().startsWith("//") || line.trim().startsWith("*")) continue;
-          const matches = line.match(streamOpPattern);
-          if (matches && matches.length >= context.chainThreshold) {
-            if (firstOffendingLine === 0) {
-              firstOffendingLine = i + 1;
-              firstOffendingCount = matches.length;
-            }
-          }
-        }
-        if (firstOffendingLine === 0) return issues;
-        issues.push({
-          ruleId: "java/stream-overuse",
-          category: "typo",
-          severity: "low",
-          aiSpecific: true,
-          message: `${firstOffendingCount} stream operations on a single line \u2014 for-loop is faster`,
-          line: firstOffendingLine,
-          column: 1,
-          advice: "Prefer a for-loop for simple transformations. AI agents default to Stream API chains because their training data emphasizes functional-style Java. Stream chains allocate intermediate objects; for small collections, a for-loop is faster and more readable. Reference: java/stream-overuse v0.26.0."
-        });
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/string-concat-loop.ts
-var STRING_CONCAT_REGEX, javaStringConcatLoopRule;
-var init_string_concat_loop = __esm({
-  "src/rules/java/string-concat-loop.ts"() {
-    "use strict";
-    init_rule();
-    STRING_CONCAT_REGEX = /(\b\w+)\s*=\s*\1\s*\+\s*[^;]+;|(\b\w+)\s*\+=\s*['"`]/g;
-    javaStringConcatLoopRule = createRule({
-      id: "java/string-concat-loop",
-      category: "perf",
-      severity: "low",
-      aiSpecific: true,
-      description: "String concatenation in a loop \u2014 use StringBuilder",
-      create(_context) {
-        return {};
-      },
-      analyze(_context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        if (!/\b(for|while|do)\b/.test(source)) return issues;
-        let m;
-        STRING_CONCAT_REGEX.lastIndex = 0;
-        while ((m = STRING_CONCAT_REGEX.exec(source)) !== null) {
-          const line = source.slice(0, m.index).split("\n").length;
-          issues.push({
-            ruleId: "java/string-concat-loop",
-            category: "perf",
-            severity: "low",
-            aiSpecific: true,
-            message: `String concatenation in a loop at line ${line} \u2014 use StringBuilder`,
-            line,
-            column: 1,
-            advice: "Declare a `StringBuilder` outside the loop: `StringBuilder sb = new StringBuilder(); sb.append(...);` then `return sb.toString();` after the loop. String concatenation in a loop is O(n\xB2) \u2014 each iteration copies the prior string. AI agents concatenate strings in loops because of training-data examples. Reference: java/string-concat-loop v0.20."
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/system-out-println.ts
-var PRINTLN_REGEX, DEFAULT_THRESHOLD2, javaSystemOutPrintlnRule;
-var init_system_out_println = __esm({
-  "src/rules/java/system-out-println.ts"() {
-    "use strict";
-    init_rule();
-    PRINTLN_REGEX = /System\.out\.println\s*\(/g;
-    DEFAULT_THRESHOLD2 = 1;
-    javaSystemOutPrintlnRule = createRule({
-      id: "java/system-out-println",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "System.out.println in production code \u2014 use a logger (SLF4J, Log4j, etc.)",
-      create(_context) {
-        return { threshold: DEFAULT_THRESHOLD2 };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        const matches = [];
-        let m;
-        PRINTLN_REGEX.lastIndex = 0;
-        while ((m = PRINTLN_REGEX.exec(source)) !== null) {
-          matches.push(m.index);
-        }
-        if (matches.length <= context.threshold) return issues;
-        const cap = Math.min(matches.length, 10);
-        for (let i = 0; i < cap; i++) {
-          const idx = matches[i];
-          const line = source.slice(0, idx).split("\n").length;
-          issues.push({
-            ruleId: "java/system-out-println",
-            category: "typo",
-            severity: "low",
-            aiSpecific: true,
-            message: `System.out.println at line ${line} \u2014 use a logger for production output`,
-            line,
-            column: 1,
-            advice: "Replace with `private static final Logger log = LoggerFactory.getLogger(...);` then `log.info(...)`. AI agents default to println because their training data has countless textbook examples. Real Java code uses a logger. Reference: java/system-out-println v0.20."
-          });
-        }
-        return issues;
-      }
-    });
-  }
-});
-
-// src/rules/java/verbose-javadoc.ts
-var TAG_REGEX, DEFAULT_TAG_THRESHOLD, DEFAULT_BODY_LENGTH_CAP, javaVerboseJavadocRule;
-var init_verbose_javadoc = __esm({
-  "src/rules/java/verbose-javadoc.ts"() {
-    "use strict";
-    init_rule();
-    TAG_REGEX = /@(param|return|throws)\b/g;
-    DEFAULT_TAG_THRESHOLD = 3;
-    DEFAULT_BODY_LENGTH_CAP = 5;
-    javaVerboseJavadocRule = createRule({
-      id: "java/verbose-javadoc",
-      category: "typo",
-      severity: "low",
-      aiSpecific: true,
-      description: "Excessive Javadoc tags on trivial methods \u2014 over-documentation is an AI fingerprint",
-      create(_context) {
-        return {
-          tagThreshold: DEFAULT_TAG_THRESHOLD,
-          bodyLengthCap: DEFAULT_BODY_LENGTH_CAP
-        };
-      },
-      analyze(context, facts) {
-        const issues = [];
-        const source = facts.v2?._source;
-        if (!source) return issues;
-        if (!/\.java$/i.test(facts.filePath)) return issues;
-        TAG_REGEX.lastIndex = 0;
-        let tagCount = 0;
-        let firstTagLine = 0;
-        let m;
-        while ((m = TAG_REGEX.exec(source)) !== null) {
-          tagCount++;
-          if (firstTagLine === 0) {
-            firstTagLine = source.slice(0, m.index).split("\n").length;
-          }
-        }
-        if (tagCount < context.tagThreshold) return issues;
-        const lineCount = source.split("\n").length;
-        const tagDensity = tagCount / Math.max(lineCount, 1);
-        if (tagDensity < 0.05) return issues;
-        if (lineCount > 200) return issues;
-        issues.push({
-          ruleId: "java/verbose-javadoc",
-          category: "typo",
-          severity: "low",
-          aiSpecific: true,
-          message: `${tagCount} Javadoc tags in ${lineCount} lines (density ${(tagDensity * 100).toFixed(1)}%) \u2014 likely over-documented`,
-          line: firstTagLine,
-          column: 1,
-          advice: "Skip Javadoc on trivial methods (getters, setters, builders). AI agents default to over-documentation because their training data has countless textbook Javadoc examples. Real Java code limits Javadoc to public API surface. Reference: java/verbose-javadoc v0.26.0."
-        });
-        return issues;
-      }
-    });
-  }
-});
-
 // src/rules/kotlin/coroutine-global-scope.ts
 var GLOBAL_SCOPE_REGEX, kotlinCoroutineGlobalScopeRule;
 var init_coroutine_global_scope = __esm({
@@ -31390,13 +30807,13 @@ var init_object_singleton_misuse = __esm({
 });
 
 // src/rules/kotlin/println-debug.ts
-var PRINTLN_REGEX2, DEFAULT_THRESHOLD3, kotlinPrintlnDebugRule;
+var PRINTLN_REGEX, DEFAULT_THRESHOLD, kotlinPrintlnDebugRule;
 var init_println_debug = __esm({
   "src/rules/kotlin/println-debug.ts"() {
     "use strict";
     init_rule();
-    PRINTLN_REGEX2 = /^\s*println\s*\(/gm;
-    DEFAULT_THRESHOLD3 = 1;
+    PRINTLN_REGEX = /^\s*println\s*\(/gm;
+    DEFAULT_THRESHOLD = 1;
     kotlinPrintlnDebugRule = createRule({
       id: "kotlin/println-debug",
       category: "typo",
@@ -31404,7 +30821,7 @@ var init_println_debug = __esm({
       aiSpecific: true,
       description: "println(...) in production code \u2014 use Timber, android.util.Log, or an SLF4J facade",
       create(_context) {
-        return { threshold: DEFAULT_THRESHOLD3 };
+        return { threshold: DEFAULT_THRESHOLD };
       },
       analyze(context, facts) {
         const issues = [];
@@ -31413,8 +30830,8 @@ var init_println_debug = __esm({
         if (!/\.kt$/i.test(facts.filePath)) return issues;
         const matches = [];
         let m;
-        PRINTLN_REGEX2.lastIndex = 0;
-        while ((m = PRINTLN_REGEX2.exec(source)) !== null) {
+        PRINTLN_REGEX.lastIndex = 0;
+        while ((m = PRINTLN_REGEX.exec(source)) !== null) {
           matches.push(m.index);
         }
         if (matches.length <= context.threshold) return issues;
@@ -31440,12 +30857,12 @@ var init_println_debug = __esm({
 });
 
 // src/rules/kotlin/string-concat-loop.ts
-var STRING_CONCAT_REGEX2, kotlinStringConcatLoopRule;
-var init_string_concat_loop2 = __esm({
+var STRING_CONCAT_REGEX, kotlinStringConcatLoopRule;
+var init_string_concat_loop = __esm({
   "src/rules/kotlin/string-concat-loop.ts"() {
     "use strict";
     init_rule();
-    STRING_CONCAT_REGEX2 = /\b(\w+)\s*=\s*\1\s*\+\s*[^;}]+[;}\n]/g;
+    STRING_CONCAT_REGEX = /\b(\w+)\s*=\s*\1\s*\+\s*[^;}]+[;}\n]/g;
     kotlinStringConcatLoopRule = createRule({
       id: "kotlin/string-concat-loop",
       category: "perf",
@@ -31462,8 +30879,8 @@ var init_string_concat_loop2 = __esm({
         if (!/\.kt$/i.test(facts.filePath)) return issues;
         if (!/\b(?:for|while|repeat|forEach)\b/.test(source)) return issues;
         let m;
-        STRING_CONCAT_REGEX2.lastIndex = 0;
-        while ((m = STRING_CONCAT_REGEX2.exec(source)) !== null) {
+        STRING_CONCAT_REGEX.lastIndex = 0;
+        while ((m = STRING_CONCAT_REGEX.exec(source)) !== null) {
           const line = source.slice(0, m.index).split("\n").length;
           const lineText = source.slice(0, m.index).split("\n").pop() ?? "";
           if (/\.append\s*\(/.test(lineText)) continue;
@@ -41109,13 +40526,13 @@ var init_implicitly_unwrapped_optional = __esm({
 });
 
 // src/rules/swift/print-debug.ts
-var PRINT_REGEX, DEFAULT_THRESHOLD4, swiftPrintDebugRule;
+var PRINT_REGEX, DEFAULT_THRESHOLD2, swiftPrintDebugRule;
 var init_print_debug = __esm({
   "src/rules/swift/print-debug.ts"() {
     "use strict";
     init_rule();
     PRINT_REGEX = /\bprint\s*\(/g;
-    DEFAULT_THRESHOLD4 = 1;
+    DEFAULT_THRESHOLD2 = 1;
     swiftPrintDebugRule = createRule({
       id: "swift/print-debug",
       category: "typo",
@@ -41123,7 +40540,7 @@ var init_print_debug = __esm({
       aiSpecific: true,
       description: "print(...) in production Swift \u2014 use Logger (os.log) for level-controlled output",
       create(_context) {
-        return { threshold: DEFAULT_THRESHOLD4 };
+        return { threshold: DEFAULT_THRESHOLD2 };
       },
       analyze(context, facts) {
         const issues = [];
@@ -46873,22 +46290,11 @@ var init_builtins = __esm({
     init_error_wrap_without_context();
     init_nil_slice_vs_empty();
     init_struct_tag_inconsistency();
-    init_arraylist_vs_linkedlist();
-    init_builder_overuse();
-    init_empty_catch_block();
-    init_immutable_collection_preference();
-    init_legacy_date_api();
-    init_optional_overuse();
-    init_raw_type_overuse();
-    init_stream_overuse();
-    init_string_concat_loop();
-    init_system_out_println();
-    init_verbose_javadoc();
     init_coroutine_global_scope();
     init_data_class_defaults_overuse();
     init_object_singleton_misuse();
     init_println_debug();
-    init_string_concat_loop2();
+    init_string_concat_loop();
     init_gap_monopoly();
     init_math_element_uniformity();
     init_math_grid_uniformity();
@@ -47013,17 +46419,6 @@ var init_builtins = __esm({
       goErrorWrapWithoutContextRule,
       goNilSliceVsEmptyRule,
       goStructTagInconsistencyRule,
-      javaArraylistVsLinkedlistRule,
-      javaBuilderOveruseRule,
-      javaEmptyCatchBlockRule,
-      javaImmutableCollectionPreferenceRule,
-      javaLegacyDateApiRule,
-      javaOptionalOveruseRule,
-      javaRawTypeOveruseRule,
-      javaStreamOveruseRule,
-      javaStringConcatLoopRule,
-      javaSystemOutPrintlnRule,
-      javaVerboseJavadocRule,
       kotlinCoroutineGlobalScopeRule,
       kotlinDataClassDefaultsOveruseRule,
       kotlinObjectSingletonMisuseRule,
@@ -50190,224 +49585,6 @@ var init_signal_strength = __esm({
         _v8Lift: 1,
         defaultOff: true
       },
-      "java/arraylist-vs-linkedlist": {
-        recall: 35e-4,
-        fpRate: 94e-4,
-        ratio: 0.37,
-        precision: 0.0446,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files = 92196 total. v9 TP=36, FP=771, P=4.46%, FPR=0.94%, recall=0.35%, ratio=0.37, lift=P/FPR=4.74. Rule fires more on legacy enterprise Java (neg) than on AI/ML-integration Java (pos) because modern Java idiom defaults to ArrayList. v0.20: new rule (new LinkedList<>() \u2014 use ArrayList, Effective Java Item 28).",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 4.74,
-        _v9Recall: 35e-4,
-        _v9FpRate: 94e-4,
-        _v9Precision: 0.0446,
-        defaultOff: true
-      },
-      "java/empty-catch-block": {
-        recall: 8e-3,
-        fpRate: 0.1118,
-        ratio: 0.07,
-        precision: 89e-4,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files. v9 TP=82, FP=9159, P=0.89%, FPR=11.18%, recall=0.80%, ratio=0.07, lift=P/FPR=0.08. Strongly INVERTED: empty catch blocks are 14x more common in legacy enterprise Java than in AI/Java-modern pos repos. Heuristic likely needs to be tightened (e.g., require at least a comment) before it can serve as a meaningful discriminator. v0.20: new rule.",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 0.08,
-        _v9Recall: 8e-3,
-        _v9FpRate: 0.1118,
-        _v9Precision: 89e-4,
-        defaultOff: true
-      },
-      "java/legacy-date-api": {
-        recall: 0.0285,
-        fpRate: 0.0511,
-        ratio: 0.56,
-        precision: 0.0657,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files. v9 TP=294, FP=4181, P=6.57%, FPR=5.11%, recall=2.85%, ratio=0.56, lift=P/FPR=1.29. INVERTED-to-NOISY: legacy date API is more common in older enterprise Java (neg) than in modern AI/Java repos (pos). The heuristic correctly flags the legacy pattern, but the legacy pattern is over-represented in the neg baseline (older codebases). v0.20: new rule.",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1.29,
-        _v9Recall: 0.0285,
-        _v9FpRate: 0.0511,
-        _v9Precision: 0.0657,
-        defaultOff: true
-      },
-      "java/verbose-javadoc": {
-        recall: 0,
-        fpRate: 0,
-        ratio: 1,
-        precision: 0,
-        lastCalibratedAt: "2026-07-03T00:00:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v0.26.0: new positive AI-signal rule. Triggers when a small Java file (< 200 lines) has 3+ Javadoc tags with density >= 0.05. Calibration pending v9 Java corpus re-run; placeholder values until then. v0.20.0: anti-pattern design was wrong (humans write more Javadoc on trivial methods because their training data has it). v0.26.0: redesigned as positive signal (AI over-documents trivial methods). Reference: java/verbose-javadoc v0.26.0.",
-        aiSpecific: true,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1,
-        _v9Recall: 0,
-        _v9FpRate: 0,
-        _v9Precision: 0,
-        defaultOff: true
-      },
-      "java/optional-overuse": {
-        recall: 0,
-        fpRate: 0,
-        ratio: 1,
-        precision: 0,
-        lastCalibratedAt: "2026-07-03T00:00:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v0.26.0: new positive AI-signal rule. Triggers when a Java file has 2+ .orElseThrow() calls AND 0 Objects.requireNonNull()/null checks (optionalRatio > 0.6). Calibration pending v9 Java corpus re-run; placeholder values until then. v0.26.0: redesigned as positive signal (AI chains Optional.ofNullable().orElseThrow() where null checks would be cleaner). Reference: java/optional-overuse v0.26.0.",
-        aiSpecific: true,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1,
-        _v9Recall: 0,
-        _v9FpRate: 0,
-        _v9Precision: 0,
-        defaultOff: true
-      },
-      "java/immutable-collection-preference": {
-        recall: 0,
-        fpRate: 0,
-        ratio: 1,
-        precision: 0,
-        lastCalibratedAt: "2026-07-03T00:00:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v0.26.0: new positive AI-signal rule. Triggers when a Java file has 5+ List.of/Map.of/Set.of calls AND < 1 new ArrayList/HashMap/HashSet/LinkedList/TreeMap calls. Calibration pending v9 Java corpus re-run; placeholder values until then. v0.26.0: redesigned as positive signal (AI defaults to immutable factory methods). Reference: java/immutable-collection-preference v0.26.0.",
-        aiSpecific: true,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1,
-        _v9Recall: 0,
-        _v9FpRate: 0,
-        _v9Precision: 0,
-        defaultOff: true
-      },
-      "java/builder-overuse": {
-        recall: 0,
-        fpRate: 0,
-        ratio: 1,
-        precision: 0,
-        lastCalibratedAt: "2026-07-03T00:00:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v0.26.0: new positive AI-signal rule. Triggers when @Builder annotation (Lombok) is used on a class with <= 3 fields. Calibration pending v9 Java corpus re-run; placeholder values until then. v0.26.0: redesigned as positive signal (AI defaults to Builder pattern even for simple data classes). Reference: java/builder-overuse v0.26.0.",
-        aiSpecific: true,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1,
-        _v9Recall: 0,
-        _v9FpRate: 0,
-        _v9Precision: 0,
-        defaultOff: true
-      },
-      "java/stream-overuse": {
-        recall: 0,
-        fpRate: 0,
-        ratio: 1,
-        precision: 0,
-        lastCalibratedAt: "2026-07-03T00:00:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v0.26.0: new positive AI-signal rule. Triggers when a single line has 3+ Stream API operations (.map, .filter, .flatMap, .collect, .reduce, .sorted, .distinct, .limit, .skip, .anyMatch, .allMatch, .noneMatch, .findFirst, .findAny). Calibration pending v9 Java corpus re-run; placeholder values until then. v0.26.0: redesigned as positive signal (AI chains Stream API for everything). Reference: java/stream-overuse v0.26.0.",
-        aiSpecific: true,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 1,
-        _v9Recall: 0,
-        _v9FpRate: 0,
-        _v9Precision: 0,
-        defaultOff: true
-      },
-      "java/raw-type-overuse": {
-        recall: 0.1093,
-        fpRate: 0.2041,
-        ratio: 0.54,
-        precision: 0.0632,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files. v9 TP=1126, FP=16712, P=6.32%, FPR=20.41%, recall=10.93%, ratio=0.54, lift=P/FPR=0.31. Strongly INVERTED: raw types (no generics) are 2x more common in legacy enterprise Java (neg) than in modern Java repos (pos). Heuristic is correctly identifying the legacy anti-pattern, but it's an age marker more than an AI-fingerprint. v0.20: new rule.",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 0.31,
-        _v9Recall: 0.1093,
-        _v9FpRate: 0.2041,
-        _v9Precision: 0.0632,
-        defaultOff: true
-      },
-      "java/string-concat-loop": {
-        recall: 46e-4,
-        fpRate: 0.0358,
-        ratio: 0.13,
-        precision: 0.0158,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files. v9 TP=47, FP=2933, P=1.58%, FPR=3.58%, recall=0.46%, ratio=0.13, lift=P/FPR=0.44. Strongly INVERTED: string concatenation in a loop is 8x more common in legacy enterprise Java (neg) than in AI/ML/Java-modern pos repos. Modern Java training data and modern IDE warnings discourage the pattern. v0.20: new rule.",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 0.44,
-        _v9Recall: 46e-4,
-        _v9FpRate: 0.0358,
-        _v9Precision: 0.0158,
-        defaultOff: true
-      },
-      "java/system-out-println": {
-        recall: 0.1503,
-        fpRate: 0.2533,
-        ratio: 0.59,
-        precision: 0.0695,
-        lastCalibratedAt: "2026-07-02T19:34:00Z",
-        verdict: "DORMANT",
-        _calibrationNote: "v9 Java corpus calibration (v0.25.0, 2026-07-02): 81891 neg + 10305 pos Java files. v9 TP=1549, FP=20737, P=6.95%, FPR=25.32%, recall=15.03%, ratio=0.59, lift=P/FPR=0.27. INVERTED: System.out.println is more common in legacy enterprise Java (neg) than in modern AI/Java repos (pos). Modern Java defaults to SLF4J/Log4j, AI-generated Java starts with a logger import. The pattern is the opposite of the v0.20 design intent (which assumed println was the AI-fingerprint signal). v0.25.x: consider gating this rule by Java era (recent file mtime) or making it an age-fingerprint rather than AI-fingerprint. v0.20: new rule.",
-        aiSpecific: true,
-        _v7Verdict: "DORMANT",
-        _v7Lift: 1,
-        _v7Recall: 0,
-        _v7FpRate: 0,
-        _v7Precision: 0,
-        _v8Verdict: "DORMANT",
-        _v8Lift: 1,
-        _v9Verdict: "DORMANT",
-        _v9Lift: 0.27,
-        _v9Recall: 0.1503,
-        _v9FpRate: 0.2533,
-        _v9Precision: 0.0695,
-        defaultOff: true
-      },
       "layout/forced-layout": {
         recall: 0,
         fpRate: 0,
@@ -59870,18 +59047,6 @@ var RULE_HINTS = {
   "dup/near-duplicate": "Refactor to a shared helper. Type-2 near-duplicate: same identifiers + structure, different whitespace/comments. AI-iterated code regenerates the same function with cosmetic edits. (v0.23.0 \u2014 MinHash + LSH, k=5, thr 0.7.)",
   // v0.24.0 — Type-3 structural clone detector (two-stage MinHash)
   "dup/structural-clone": "Refactor to a shared helper. Type-3 clone: same shape as another file after identifier canonicalization, with renames and/or added/removed statements. AI agents and copy-paste both produce this. Extract the shared part. (v0.24.0.)",
-  // v0.20.0 — Java-specific rules
-  "java/system-out-println": "Replace with `private static final Logger log = LoggerFactory.getLogger(...);` then `log.info(...)`. AI agents default to println because of textbook examples; real Java code uses a logger.",
-  "java/empty-catch-block": 'Log the exception (`log.error("...", e)`), re-throw it, or both. Empty catch blocks hide bugs. Common in AI-generated code that wants to look defensive.',
-  "java/arraylist-vs-linkedlist": "Replace `new LinkedList<>()` with `new ArrayList<>()`. LinkedList is rarely the right choice (worse cache locality, 5x more memory per element, O(n) indexed access). Joshua Bloch recommends ArrayList unless you need a Deque.",
-  "java/legacy-date-api": "Replace `java.util.Date` / `Calendar` with `java.time` (`LocalDate`, `LocalDateTime`, `Instant`, `ZonedDateTime`). java.time is immutable and thread-safe. AI agents default to the legacy API.",
-  "java/raw-type-overuse": "Replace raw `List` / `Map` / `Set` with the parameterized form. Raw types disable generic type checking (Effective Java, Item 23). AI agents default to raw types when unsure of the correct generic parameters.",
-  "java/string-concat-loop": "Declare a `StringBuilder` outside the loop: `StringBuilder sb = new StringBuilder(); sb.append(...);` then `return sb.toString();` after. String concat in a loop is O(n\xB2).",
-  "java/verbose-javadoc": "Skip Javadoc on trivial methods (getters, setters, builders). AI over-documents; real Java limits Javadoc to public API surface. (v0.26.0 \u2014 DORMANT.)",
-  "java/optional-overuse": "Use Objects.requireNonNull(x) for null checks; reserve Optional for return values. AI chains .ofNullable().orElseThrow() where null checks suffice. (v0.26.0 \u2014 DORMANT.)",
-  "java/immutable-collection-preference": "Prefer mutable (new ArrayList<>()) when the collection will be modified. AI defaults to List.of/Map.of/Set.of; real Java uses mutable. (v0.26.0 \u2014 DORMANT.)",
-  "java/builder-overuse": "Use a plain constructor for classes with \u2264 3 fields. AI defaults to @Builder; Builder adds Lombok dep and a Builder inner class per class. (v0.26.0 \u2014 DORMANT.)",
-  "java/stream-overuse": "Prefer a for-loop for simple transformations. AI chains Stream API for everything; for small collections a for-loop is faster. (v0.26.0 \u2014 DORMANT.)",
   // v0.24.0 — Kotlin rules (DORMANT until v9 Kotlin corpus calibration)
   "kotlin/data-class-defaults-overuse": "Audit which fields are genuinely optional. Real Kotlin uses defaults sparingly (1-2 callbacks, the rest required). v0.24.0 \u2014 DORMANT until v9 calibration.",
   "kotlin/coroutine-global-scope": "Replace with `viewModelScope.launch { ... }`, `lifecycleScope.launch { ... }`, or `coroutineScope { ... }`. GlobalScope bypasses cancellation. (v0.24.0 \u2014 DORMANT.)",