chore: unify load-rules.ts and utils/rules.ts to rules/index.ts

docs-build/src/components/eslint/scripts/linter.mts

@@ -1,4 +1,4 @@
-import { rules as pluginRules } from "../../../../../src/utils/rules.js"
+import { rules as pluginRules } from "../../../../../src/rules"
 import type { Linter, Rule } from "eslint"
 import { builtinRules } from "eslint/use-at-your-own-risk"
 import * as astroEslintParser from "astro-eslint-parser"

docs-build/src/config.ts

@@ -1,4 +1,4 @@
-import { rules } from "../../src/utils/rules"
+import { rules } from "../../src/rules"
 
 const categories = [
   "Possible Errors",

src/configs/all.ts

@@ -1,5 +1,5 @@
 import recommended from "./recommended"
-import { rules } from "../utils/rules"
+import { rules } from "../rules"
 
 const all: Record<string, string> = {}
 for (const rule of rules.filter(

src/configs/flat/all.ts

@@ -1,5 +1,5 @@
 import recommended from "./recommended"
-import { rules } from "../../utils/rules"
+import { rules } from "../../rules"
 
 const all: Record<string, string> = {}
 for (const rule of rules.filter(

src/plugin.ts

@@ -1,4 +1,4 @@
-import { rules as ruleList } from "./utils/rules"
+import { rules as ruleList } from "./rules"
 import * as processorsDefines from "./processor"
 import type { Rule } from "eslint"
 import { name, version } from "./meta"

src/rules/index.ts

@@ -0,0 +1,39 @@
+import missingClientOnlyDirectiveValue from "./missing-client-only-directive-value"
+import noConflictSetDirectives from "./no-conflict-set-directives"
+import noDeprecatedAstroCanonicalUrl from "./no-deprecated-astro-canonicalurl"
+import noDeprecatedAstroFetchContent from "./no-deprecated-astro-fetchcontent"
+import noDeprecatedAstroResolve from "./no-deprecated-astro-resolve"
+import noDeprecatedGetEntryBySlug from "./no-deprecated-getentrybyslug"
+import noExportsFromComponents from "./no-exports-from-components"
+import noSetHtmlDirective from "./no-set-html-directive"
+import noSetTextDirective from "./no-set-text-directive"
+import noUnusedCssSelector from "./no-unused-css-selector"
+import noUnusedDefineVarsInStyle from "./no-unused-define-vars-in-style"
+import preferClassListDirective from "./prefer-class-list-directive"
+import preferObjectClassList from "./prefer-object-class-list"
+import preferSplitClassList from "./prefer-split-class-list"
+import semi from "./semi"
+import sortAttributes from "./sort-attributes"
+import validCompile from "./valid-compile"
+import { buildA11yRules } from "../a11y"
+
+export const originalRules = [
+  missingClientOnlyDirectiveValue,
+  noConflictSetDirectives,
+  noDeprecatedAstroCanonicalUrl,
+  noDeprecatedAstroFetchContent,
+  noDeprecatedAstroResolve,
+  noDeprecatedGetEntryBySlug,
+  noExportsFromComponents,
+  noSetHtmlDirective,
+  noSetTextDirective,
+  noUnusedCssSelector,
+  noUnusedDefineVarsInStyle,
+  preferClassListDirective,
+  preferObjectClassList,
+  preferSplitClassList,
+  semi,
+  sortAttributes,
+  validCompile,
+]
+export const rules = [...originalRules, ...buildA11yRules()]

tests/src/rules/index.test.ts

@@ -0,0 +1,39 @@
+import { readdirSync } from "fs"
+import { join } from "path"
+import assert from "assert"
+import { rules } from "../../../src/rules/index"
+import { buildA11yRules } from "src/a11y"
+
+describe("Test src/rules/index.ts", () => {
+  function getImportedRuleNames() {
+    return rules.map((rule) => rule.meta.docs.ruleName)
+  }
+
+  function assertRulesImported(rulesToCheck: string[]) {
+    const importedRules = getImportedRuleNames()
+    rulesToCheck.forEach((ruleName) => {
+      assert(
+        importedRules.includes(ruleName),
+        `Imported rules should include ${ruleName}`,
+      )
+    })
+  }
+
+  it("should import all rule files", () => {
+    const rulesDir = join(__dirname, "../../../src/rules")
+    const ruleFiles = readdirSync(rulesDir)
+      .filter(
+        (file) =>
+          file.endsWith(".ts") &&
+          file !== "index.ts" &&
+          !file.endsWith(".test.ts"),
+      )
+      .map((file) => file.replace(".ts", ""))
+    assertRulesImported(ruleFiles)
+  })
+
+  it("should import all a11y rules", () => {
+    const a11yRules = buildA11yRules().map((rule) => rule.meta.docs.ruleName)
+    assertRulesImported(a11yRules)
+  })
+})

tests/src/rules/jsx-a11y/aria-proptypes.ts

@@ -1,5 +1,5 @@
 import { RuleTester } from "../../../utils/eslint-compat"
-import { rules } from "../../../../src/utils/rules"
+import { rules } from "../../../../src/rules"
 import { loadTestCases } from "../../../utils/utils"
 
 const rule = rules.find(

tests/src/rules/jsx-a11y/heading-has-content.ts

@@ -1,5 +1,5 @@
 import { RuleTester } from "../../../utils/eslint-compat"
-import { rules } from "../../../../src/utils/rules"
+import { rules } from "../../../../src/rules"
 import { loadTestCases } from "../../../utils/utils"
 
 const rule = rules.find(

tests/src/rules/jsx-a11y/label-has-associated-control.ts

@@ -1,5 +1,5 @@
 import { RuleTester } from "../../../utils/eslint-compat"
-import { rules } from "../../../../src/utils/rules"
+import { rules } from "../../../../src/rules"
 import { loadTestCases } from "../../../utils/utils"
 
 const rule = rules.find(

tests/src/rules/jsx-a11y/no-autofocus.ts

@@ -1,5 +1,5 @@
 import { RuleTester } from "../../../utils/eslint-compat"
-import { rules } from "../../../../src/utils/rules"
+import { rules } from "../../../../src/rules"
 import { loadTestCases } from "../../../utils/utils"
 
 const rule = rules.find(

tools/render-rules-table.ts

@@ -1,7 +1,7 @@
 // eslint-disable-next-line @eslint-community/eslint-comments/disable-enable-pair -- ignore
 /* eslint-disable func-style -- Arrow functions are better when returning string */
 import type { RuleCategory, RuleModule } from "../src/types"
-import { rules } from "../src/utils/rules"
+import { rules } from "../src/rules"
 
 const categories: RuleCategory[] = [
   "Possible Errors",

tools/update-docs.ts

@@ -1,6 +1,6 @@
 import path from "path"
 import fs from "fs"
-import { rules } from "../src/utils/rules"
+import { rules } from "../src/rules"
 import type { RuleModule } from "../src/types"
 import { getNewVersion } from "./lib/changesets-util"
 import { formatAndSave } from "./lib/utils"

tools/update-rulesets.ts

@@ -1,5 +1,5 @@
 import path from "path"
-import { rules } from "./lib/load-rules"
+import { originalRules as rules } from "../src/rules"
 import { formatAndSave } from "./lib/utils"
 
 const baseRules = rules.filter(

tools/update.ts

@@ -1,4 +1,3 @@
-import "./update-rules"
 import "./update-rulesets"
 import "./update-docs"
 import "./update-readme"