feat!: Remove valid-jsdoc and require-jsdoc (#17694)

* feat!: Remove valid-jsdoc

Fixes #15820

* Update rule index page

* Remove references to valid-jsdoc

* Remove reference to valid-jsdoc

* Remove reference to plugin rules

* Remove require-jsdoc

* Add require-jsdoc to the removed list

* Add back docs and add note about removal

* Add period

* Remove test files

* Update tests/lib/eslint/flat-eslint.js

Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>

---------

Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>

conf/rule-type-list.json

@@ -23,6 +23,8 @@
         { "removed": "space-in-brackets", "replacedBy": ["object-curly-spacing", "array-bracket-spacing"] },
         { "removed": "space-return-throw-case", "replacedBy": ["keyword-spacing"] },
         { "removed": "space-unary-word-ops", "replacedBy": ["space-unary-ops"] },
-        { "removed": "spaced-line-comment", "replacedBy": ["spaced-comment"] }
+        { "removed": "spaced-line-comment", "replacedBy": ["spaced-comment"] },
+        { "removed": "valid-jsdoc", "replacedBy": [] },
+        { "removed": "require-jsdoc", "replacedBy": [] }
     ]
 }

docs/src/_data/rules.json

@@ -652,7 +652,7 @@
                 "description": "Disallow `Array` constructors",
                 "recommended": false,
                 "fixable": false,
-                "hasSuggestions": true
+                "hasSuggestions": false
             },
             {
                 "name": "no-bitwise",
@@ -680,7 +680,7 @@
                 "description": "Disallow the use of `console`",
                 "recommended": false,
                 "fixable": false,
-                "hasSuggestions": true
+                "hasSuggestions": false
             },
             {
                 "name": "no-continue",

docs/src/_data/rules_meta.json

@@ -2371,16 +2371,6 @@
             "url": "https://eslint.org/docs/latest/rules/require-await"
         }
     },
-    "require-jsdoc": {
-        "type": "suggestion",
-        "docs": {
-            "description": "Require JSDoc comments",
-            "recommended": false,
-            "url": "https://eslint.org/docs/latest/rules/require-jsdoc"
-        },
-        "deprecated": true,
-        "replacedBy": []
-    },
     "require-unicode-regexp": {
         "type": "suggestion",
         "docs": {
@@ -2602,17 +2592,6 @@
             "url": "https://eslint.org/docs/latest/rules/use-isnan"
         }
     },
-    "valid-jsdoc": {
-        "type": "suggestion",
-        "docs": {
-            "description": "Enforce valid JSDoc comments",
-            "recommended": false,
-            "url": "https://eslint.org/docs/latest/rules/valid-jsdoc"
-        },
-        "fixable": "code",
-        "deprecated": true,
-        "replacedBy": []
-    },
     "valid-typeof": {
         "type": "problem",
         "docs": {

docs/src/rules/require-jsdoc.md

@@ -5,8 +5,9 @@ related_rules:
 - valid-jsdoc
 ---
 
-
-This rule was [**deprecated**](https://eslint.org/blog/2018/11/jsdoc-end-of-life) in ESLint v5.10.0.
+:::important
+This rule was removed in ESLint v9.0.0 and replaced by the [`eslint-plugin-jsdoc`](https://github.com/gajus/eslint-plugin-jsdoc) equivalent.
+:::
 
 [JSDoc](http://usejsdoc.org) is a JavaScript API documentation generator. It uses specially-formatted comments inside of code to generate API documentation automatically. For example, this is what a JSDoc comment looks like for a function:
 

docs/src/rules/valid-jsdoc.md

@@ -7,9 +7,9 @@ further_reading:
 - https://jsdoc.app
 ---
 
-
-
-This rule was [**deprecated**](https://eslint.org/blog/2018/11/jsdoc-end-of-life) in ESLint v5.10.0.
+:::important
+This rule was removed in ESLint v9.0.0 and replaced by the [`eslint-plugin-jsdoc`](https://github.com/gajus/eslint-plugin-jsdoc) equivalent.
+:::
 
 [JSDoc](http://usejsdoc.org) generates application programming interface (API) documentation from specially-formatted comments in JavaScript code. For example, this is a JSDoc comment for a function:
 

docs/src/use/rule-deprecation.md

@@ -7,10 +7,12 @@ Balancing the trade-offs of improving a tool and the frustration these changes c
 
 The ESLint team is committed to making upgrading as easy and painless as possible. To that end, the team has agreed upon the following set of guidelines for deprecating rules in the future. The goal of these guidelines is to allow for improvements and changes to be made without breaking existing configurations.
 
-* Rules will never be removed from ESLint.
+* Rules will never be removed from ESLint unless one of the following is true:
+    * The rule has been replaced by another core rule.
+    * A plugin exists with a functionally equivalent rule.
 * Rules will be deprecated as needed, and marked as such in all documentation.
 * After a rule has been deprecated, the team will no longer do any work on it. This includes bug fixes, enhancements, and updates to the rule's documentation. Issues and pull requests related to deprecated rule will not be accepted and will be closed.
 
-Since deprecated rules will never be removed, you can continue to use them indefinitely if they are working for you. However, keep in mind that deprecated rules will effectively be unmaintained.
+You can continue to use deprecated rules indefinitely if they are working for you. However, keep in mind that deprecated rules will effectively be unmaintained and may be removed at some point.
 
 We hope that by following these guidelines we will be able to continue improving and working to make ESLint the best tool it can be while causing as little disruption to our users as possible during the process.

lib/rules/index.js

@@ -273,7 +273,6 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     radix: () => require("./radix"),
     "require-atomic-updates": () => require("./require-atomic-updates"),
     "require-await": () => require("./require-await"),
-    "require-jsdoc": () => require("./require-jsdoc"),
     "require-unicode-regexp": () => require("./require-unicode-regexp"),
     "require-yield": () => require("./require-yield"),
     "rest-spread-spacing": () => require("./rest-spread-spacing"),
@@ -296,7 +295,6 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "template-tag-spacing": () => require("./template-tag-spacing"),
     "unicode-bom": () => require("./unicode-bom"),
     "use-isnan": () => require("./use-isnan"),
-    "valid-jsdoc": () => require("./valid-jsdoc"),
     "valid-typeof": () => require("./valid-typeof"),
     "vars-on-top": () => require("./vars-on-top"),
     "wrap-iife": () => require("./wrap-iife"),