Update doc

Author: ota-meshi

README.md

@@ -285,22 +285,22 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | [@ota-meshi/svelte/shorthand-attribute](https://ota-meshi.github.io/eslint-plugin-svelte/rules/shorthand-attribute.html) | enforce use of shorthand syntax in attribute | :wrench: |
 | [@ota-meshi/svelte/spaced-html-comment](https://ota-meshi.github.io/eslint-plugin-svelte/rules/spaced-html-comment.html) | enforce consistent spacing after the `<!--` and before the `-->` in a HTML comment | :wrench: |
 
-## System
+## Extension Rules
 
-These rules relate to this plugin works:
+These rules extend the rules provided by ESLint itself to work well in Svelte:
 
 | Rule ID | Description |    |
 |:--------|:------------|:---|
-| [@ota-meshi/svelte/comment-directive](https://ota-meshi.github.io/eslint-plugin-svelte/rules/comment-directive.html) | support comment-directives in HTML template | :star: |
-| [@ota-meshi/svelte/system](https://ota-meshi.github.io/eslint-plugin-svelte/rules/system.html) | system rule for working this plugin | :star: |
+| [@ota-meshi/svelte/no-inner-declarations](https://ota-meshi.github.io/eslint-plugin-svelte/rules/no-inner-declarations.html) | disallow variable or `function` declarations in nested blocks | :star: |
 
-## Extension Rules
+## System
 
-These rules extend the rules provided by ESLint itself to work well in Svelte:
+These rules relate to this plugin works:
 
 | Rule ID | Description |    |
 |:--------|:------------|:---|
-| [@ota-meshi/svelte/no-inner-declarations](https://ota-meshi.github.io/eslint-plugin-svelte/rules/no-inner-declarations.html) | disallow variable or `function` declarations in nested blocks | :star: |
+| [@ota-meshi/svelte/comment-directive](https://ota-meshi.github.io/eslint-plugin-svelte/rules/comment-directive.html) | support comment-directives in HTML template | :star: |
+| [@ota-meshi/svelte/system](https://ota-meshi.github.io/eslint-plugin-svelte/rules/system.html) | system rule for working this plugin | :star: |
 
 <!--RULES_TABLE_END-->
 <!--RULES_SECTION_END-->

docs/.vuepress/components/rules/index.js

@@ -24,8 +24,8 @@ const CATEGORY_INDEX = {
   "Security Vulnerability": 2,
   "Best Practices": 3,
   "Stylistic Issues": 4,
-  System: 5,
-  "Extension Rules": 5.5,
+  "Extension Rules": 5,
+  System: 5.5,
   "eslint-core-rules@Possible Errors": 6,
   "eslint-core-rules@Best Practices": 7,
   "eslint-core-rules@Strict Mode": 8,
@@ -39,8 +39,8 @@ const CATEGORY_CLASSES = {
   "Security Vulnerability": "eslint-plugin-svelte__category",
   "Best Practices": "eslint-plugin-svelte__category",
   "Stylistic Issues": "eslint-plugin-svelte__category",
-  System: "eslint-plugin-svelte__category",
   "Extension Rules": "eslint-plugin-svelte__category",
+  System: "eslint-plugin-svelte__category",
 }
 
 const allRules = []
@@ -51,9 +51,7 @@ for (const k of Object.keys(plugin.rules)) {
     continue
   }
 
-  const category = rule.meta.docs.extensionRule
-    ? "Extension Rules"
-    : rule.meta.docs.category
+  const category = rule.meta.docs.category
 
   allRules.push({
     classes: "eslint-plugin-svelte__rule",

docs/.vuepress/config.js

@@ -2,15 +2,14 @@ const path = require("path")
 // eslint-disable-next-line node/no-missing-require, node/no-unpublished-require -- no build
 const { rules } = require("../../lib/utils/rules")
 
-const svelteRules = rules.filter(
-  (rule) => !rule.meta.docs.extensionRule && !rule.meta.deprecated,
-)
+const svelteRules = rules.filter((rule) => !rule.meta.deprecated)
 
 const categories = [
   "Possible Errors",
   "Security Vulnerability",
   "Best Practices",
   "Stylistic Issues",
+  "Extension Rules",
   "System",
 ]
 svelteRules.forEach((rule) => {
@@ -25,10 +24,6 @@ const categoryRules = categories.map((cat) => {
   }
 })
 
-const extensionRules = rules.filter(
-  (rule) => rule.meta.docs.extensionRule && !rule.meta.deprecated,
-)
-
 function ruleToLink({
   meta: {
     docs: { ruleId, ruleName },
@@ -84,15 +79,6 @@ module.exports = {
             children: cat.rules.map(ruleToLink),
           }
         }),
-        ...(extensionRules.length
-          ? [
-              {
-                title: "Extension Rules",
-                collapsable: false,
-                children: extensionRules.map(ruleToLink),
-              },
-            ]
-          : []),
         // Rules in no category.
         ...(rules.some((rule) => rule.meta.deprecated)
           ? [

docs/rules/README.md

@@ -53,19 +53,19 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | [@ota-meshi/svelte/shorthand-attribute](./shorthand-attribute.md) | enforce use of shorthand syntax in attribute | :wrench: |
 | [@ota-meshi/svelte/spaced-html-comment](./spaced-html-comment.md) | enforce consistent spacing after the `<!--` and before the `-->` in a HTML comment | :wrench: |
 
-## System
+## Extension Rules
 
-These rules relate to this plugin works:
+These rules extend the rules provided by ESLint itself to work well in Svelte:
 
 | Rule ID | Description |    |
 |:--------|:------------|:---|
-| [@ota-meshi/svelte/comment-directive](./comment-directive.md) | support comment-directives in HTML template | :star: |
-| [@ota-meshi/svelte/system](./system.md) | system rule for working this plugin | :star: |
+| [@ota-meshi/svelte/no-inner-declarations](./no-inner-declarations.md) | disallow variable or `function` declarations in nested blocks | :star: |
 
-## Extension Rules
+## System
 
-These rules extend the rules provided by ESLint itself to work well in Svelte:
+These rules relate to this plugin works:
 
 | Rule ID | Description |    |
 |:--------|:------------|:---|
-| [@ota-meshi/svelte/no-inner-declarations](./no-inner-declarations.md) | disallow variable or `function` declarations in nested blocks | :star: |
+| [@ota-meshi/svelte/comment-directive](./comment-directive.md) | support comment-directives in HTML template | :star: |
+| [@ota-meshi/svelte/system](./system.md) | system rule for working this plugin | :star: |

src/rules/no-inner-declarations.ts

@@ -13,7 +13,7 @@ export default createRule("no-inner-declarations", {
     docs: {
       description:
         "disallow variable or `function` declarations in nested blocks",
-      category: "Possible Errors",
+      category: "Extension Rules",
       recommended: true,
       extensionRule: "no-inner-declarations",
     },

src/types.ts

@@ -60,6 +60,7 @@ export type RuleCategory =
   | "Security Vulnerability"
   | "Best Practices"
   | "Stylistic Issues"
+  | "Extension Rules"
   | "System"
 
 export interface RuleMetaData {

tools/render-rules.ts

@@ -6,6 +6,7 @@ const categories = [
   "Security Vulnerability",
   "Best Practices",
   "Stylistic Issues",
+  "Extension Rules",
   "System",
 ] as const
 
@@ -18,14 +19,13 @@ const descriptions: Record<typeof categories[number], string> = {
     "These rules relate to better ways of doing things to help you avoid problems:",
   "Stylistic Issues":
     "These rules relate to style guidelines, and are therefore quite subjective:",
+  "Extension Rules":
+    "These rules extend the rules provided by ESLint itself to work well in Svelte:",
   System: "These rules relate to this plugin works:",
 }
 
 const activeRules = rules.filter((rule) => !rule.meta.deprecated)
-const svelteRules = activeRules.filter((rule) => !rule.meta.docs.extensionRule)
-const extensionRules = activeRules.filter(
-  (rule) => rule.meta.docs.extensionRule,
-)
+const svelteRules = activeRules
 const deprecatedRules = rules.filter((rule) => rule.meta.deprecated)
 
 activeRules.forEach((rule) => {
@@ -87,17 +87,6 @@ ${cat.rules.map(toRuleRow).join("\n")}
 `
     })
     .join("")
-  if (extensionRules.length >= 1) {
-    rulesTableContent += `
-## Extension Rules
-
-These rules extend the rules provided by ESLint itself to work well in Svelte:
-
-| Rule ID | Description |    |
-|:--------|:------------|:---|
-${extensionRules.map(toRuleRow).join("\n")}
-`
-  }
 
   // -----------------------------------------------------------------------------
   if (deprecatedRules.length >= 1) {