feat: support RegExp in ignore - #474

src/utils/module-visitor.ts (1)

18-32: ⚠️ Potential issue | 🟠 Major

Harden ignore normalization and make regex matching deterministic.

Now that ignore accepts real RegExp values, using .test() on shared /g or /y regexes is stateful and can miss matches across imports. Also, with item schema loosened, non-string/RegExp values can be silently coerced by new RegExp(...).

🔧 Proposed fix

 export function moduleVisitor(visitor: Visitor, options?: ModuleOptions) {
   const ignore = options?.ignore
@@
-  const ignoreRegExps = ignore == null ? [] : ignore.map(p => new RegExp(p))
+  const ignoreRegExps =
+    ignore == null
+      ? []
+      : ignore.map(pattern => {
+          if (pattern instanceof RegExp) {
+            return pattern
+          }
+          if (typeof pattern === 'string') {
+            return new RegExp(pattern)
+          }
+          throw new TypeError('`ignore` must contain only string or RegExp values')
+        })
@@
-    if (ignoreRegExps.some(re => re.test(String(source.value)))) {
+    if (
+      ignoreRegExps.some(re => {
+        re.lastIndex = 0
+        return re.test(String(source.value))
+      })
+    ) {
       return
     }

Applies to: lines 18-32, 49-50, 203-207

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/utils/module-visitor.ts` around lines 18 - 32, Normalize the
options.ignore array in moduleVisitor by validating each item: accept RegExp
instances and strings only, throw or skip invalid types; for RegExp items create
a fresh RegExp using the original pattern and flags but strip stateful flags
('g' and 'y') so .test is deterministic, and for string items create a RegExp
from the string (escaping or intended pattern per existing behavior) rather than
passing through to new RegExp blindly; replace the current ignore.map ->
ignoreRegExps logic to produce these clean, stateless regexes (used where
ignoreRegExps and related matching at the places referenced) so matching no
longer depends on shared regex state.