refactor(linter): store ExternalRuleId in OxlintOverrides not raw names

Author: camc314

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

@@ -364,6 +364,13 @@ Found 0 warnings and 6 errors.
 Finished in Xms on 2 files using X threads."
 `;
 
+exports[`oxlint2 CLI > should report an error if a a rule is not found within a custom plugin (via overrides) 1`] = `
+"Failed to build configuration.
+
+  x Rule 'missing' not found in plugin 'basic-custom-plugin'
+"
+`;
+
 exports[`oxlint2 CLI > should report an error if a custom plugin cannot be loaded 1`] = `
 "Failed to parse configuration file.
 

napi/oxlint2/test/e2e.test.ts

@@ -84,6 +84,15 @@ describe('oxlint2 CLI', () => {
     expect(normalizeOutput(stdout)).toMatchSnapshot();
   });
 
+  it('should report an error if a a rule is not found within a custom plugin (via overrides)', async () => {
+    const { stdout, exitCode } = await runOxlint(
+      'test/fixtures/custom_plugin_via_overrides_missing_rule',
+    );
+
+    expect(exitCode).toBe(1);
+    expect(normalizeOutput(stdout)).toMatchSnapshot();
+  });
+
   it('should report the correct severity when using a custom plugin', async () => {
     const { stdout, exitCode } = await runOxlint(
       'test/fixtures/basic_custom_plugin_warn_severity',

napi/oxlint2/test/fixtures/custom_plugin_via_overrides_missing_rule/.oxlintrc.json

@@ -0,0 +1,15 @@
+{
+  "rules": {
+    "no-debugger": "off",
+  },
+  "overrides": [
+    {
+      "files": ["*.js"],
+      "plugins": ["./test_plugin"],
+      "rules": {
+        "basic-custom-plugin/missing": "error"
+      }
+    }
+  ],
+  "ignorePatterns": ["test_plugin"]
+}

napi/oxlint2/test/fixtures/custom_plugin_via_overrides_missing_rule/index.js

@@ -0,0 +1 @@
+debugger;

napi/oxlint2/test/fixtures/custom_plugin_via_overrides_missing_rule/test_plugin/index.js

@@ -0,0 +1,19 @@
+export default {
+  meta: {
+    name: "basic-custom-plugin",
+  },
+  rules: {
+    "no-debugger": {
+      create(context) {
+        return {
+          DebuggerStatement(debuggerStatement) {
+            context.report({
+              message: "Unexpected Debugger Statement",
+              node: debuggerStatement,
+            });
+          },
+        };
+      },
+    },
+  },
+};