fix(linter): enforce exact matching for disable directives

Co-authored-by: overlookmotel <557937+overlookmotel@users.noreply.github.com>

Author: Copilot

crates/oxc_linter/src/disable_directives.rs

@@ -143,10 +143,10 @@ impl<'a> DisableDirectives<'a> {
                 // Check if this rule should be disabled
                 let rule_matches = match interval.val {
                     DisabledRule::All { .. } => true,
-                    // Our rule name currently does not contain the prefix.
-                    // For example, this will match `@typescript-eslint/no-var-requires` given
-                    // our rule_name is `no-var-requires`.
-                    DisabledRule::Single { rule_name: name, .. } => name.contains(rule_name),
+                    // Use exact matching instead of substring matching to require precise rule names.
+                    // For external plugin rules, this enforces the requirement that disable directives
+                    // must include the full plugin name (e.g., "plugin-name/rule-name").
+                    DisabledRule::Single { rule_name: name, .. } => name == rule_name,
                 };
 
                 if !rule_matches {

napi/oxlint2/test/__snapshots__/e2e.test.ts.snap

@@ -462,44 +462,44 @@ Finished in Xms on 1 file using X threads."
 
 exports[`oxlint2 CLI > should respect disable directives for custom plugin rules 1`] = `
 "
-  x test-plugin(no-var): Use let or const instead of var
-   ,-[index.js:1:1]
- 1 | var shouldError = 1;
-   : ^^^^^^^^^^^^^^^^^^^^
- 2 | 
-   \`----
-
-  x ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html\\eslint(no-debugger)]8;;\\: \`debugger\` statement is not allowed
-    ,-[index.js:10:1]
-  9 | // should trigger an error
- 10 | debugger;
-    : ^^^^^^^^^
- 11 | 
-    \`----
-  help: Remove the debugger statement
-
-  x test-plugin(no-var): Use let or const instead of var
-    ,-[index.js:16:1]
- 15 | /* oxlint-disable-next-line test-plugin */ // \`test-plugin\` should be \`test-plugin/no-var\`
- 16 | var incorrectlyDisabled = 4;
-    : ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
- 17 | 
-    \`----
-
-  x test-plugin(no-var): Use let or const instead of var
-    ,-[index.js:19:1]
- 18 | /* oxlint-disable-next-line no-var */ // \`no-var\` should be \`test-plugin/no-var\`
- 19 | var anotherIncorrectlyDisabled = 4;
-    : ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
- 20 | 
-    \`----
-
-  x test-plugin(no-var): Use let or const instead of var
-    ,-[index.js:22:1]
- 21 | // This var should trigger an error again
- 22 | var shouldErrorAgain = 3;
-    : ^^^^^^^^^^^^^^^^^^^^^^^^^
-    \`----
+  × eslint(no-debugger): \`debugger\` statement is not allowed
+    ╭─[index.js:10:1]
+  9 │ // should trigger an error
+ 10 │ debugger;
+    · ─────────
+ 11 │ 
+    ╰────
+  help: Remove the debugger statement
+
+  × test-plugin(no-var): Use let or const instead of var
+   ╭─[index.js:1:1]
+ 1 │ var shouldError = 1;
+   · ────────────────────
+ 2 │ 
+   ╰────
+
+  × test-plugin(no-var): Use let or const instead of var
+    ╭─[index.js:16:1]
+ 15 │ /* oxlint-disable-next-line test-plugin */ // \`test-plugin\` should be \`test-plugin/no-var\`
+ 16 │ var incorrectlyDisabled = 4;
+    · ────────────────────────────
+ 17 │ 
+    ╰────
+
+  × test-plugin(no-var): Use let or const instead of var
+    ╭─[index.js:19:1]
+ 18 │ /* oxlint-disable-next-line no-var */ // \`no-var\` should be \`test-plugin/no-var\`
+ 19 │ var anotherIncorrectlyDisabled = 4;
+    · ───────────────────────────────────
+ 20 │ 
+    ╰────
+
+  × test-plugin(no-var): Use let or const instead of var
+    ╭─[index.js:22:1]
+ 21 │ // This var should trigger an error again
+ 22 │ var shouldErrorAgain = 3;
+    · ─────────────────────────
+    ╰────
 
 Found 0 warnings and 5 errors.
 Finished in Xms on 1 file using X threads."