bugfix: ignore "security" scope (#183)

Author: Sysix

scripts/constants.ts

@@ -7,7 +7,7 @@ export const VERSION_PREFIX = 'oxlint_v';
 export const SPARSE_CLONE_DIRECTORY = 'crates/oxc_linter/src';
 
 // these are the rules that don't have a direct equivalent in the eslint rules
-export const ignoreScope = new Set(['oxc', 'deepscan']);
+export const ignoreScope = new Set(['oxc', 'deepscan', 'security']);
 
 // these are the mappings from the scope in the rules.rs to the eslint scope
 // only used for the scopes where the directory structure doesn't reflect the eslint scope

src/__snapshots__/rules-by-scope.spec.ts.snap

@@ -1334,9 +1334,6 @@ exports[`contains all the oxlint rules 1`] = `
   "rest-spread-spacing": [
     0,
   ],
-  "security/api-keys": [
-    0,
-  ],
   "semi": [
     0,
   ],

src/rules-by-category.ts

@@ -338,7 +338,6 @@ const correctnessRules = {
   'react/no-render-return-value': 'off',
   'react/no-string-refs': 'off',
   'react/void-dom-elements-no-children': 'off',
-  'security/api-keys': 'off',
   '@typescript-eslint/no-duplicate-enum-values': 'off',
   '@typescript-eslint/no-extra-non-null-assertion': 'off',
   '@typescript-eslint/no-misused-new': 'off',

src/rules-by-scope.ts

@@ -363,10 +363,6 @@ const reactPerfRules = {
   'react-perf/jsx-no-new-object-as-prop': 'off',
 } as const;
 
-const securityRules = {
-  'security/api-keys': 'off',
-} as const;
-
 const treeShakingRules = {
   'tree-shaking/no-side-effects-in-initialization': 'off',
 } as const;
@@ -482,7 +478,6 @@ export {
   promiseRules,
   reactRules,
   reactPerfRules,
-  securityRules,
   treeShakingRules,
   unicornRules,
   vitestRules,