Fix false positives for disallowNeverMatch in regexp/no-dupe-disjunctions (#116)

Author: ota-meshi

lib/utils/regexp-ast/is-covered.ts

@@ -787,8 +787,8 @@ function isCoveredAltNodes(
     rightNodes: NormalizedNode[],
     options: Options,
 ): boolean {
-    const left = options.canOmitRight ? omitEnds(leftNodes) : leftNodes
-    const right = options.canOmitRight ? omitEnds(rightNodes) : rightNodes
+    const left = options.canOmitRight ? omitEnds(leftNodes) : [...leftNodes]
+    const right = options.canOmitRight ? omitEnds(rightNodes) : [...rightNodes]
     while (left.length && right.length) {
         const le = left.shift()!
         const re = right.shift()!

tests/lib/rules/no-dupe-disjunctions.ts

@@ -33,6 +33,10 @@ tester.run("no-dupe-disjunctions", rule as any, {
         `/(?:(a)|(a))/`,
         `/(?:a|ab)/`,
         `/(?:.|a|b)/`,
+        {
+            code: `/<("[^"]*"|'[^']*'|[^'">])*>/g`,
+            options: [{ disallowNeverMatch: true }],
+        },
     ],
     invalid: [
         ...[