feat: no-deprecated-api support removed api

eslint-community · Apr 11, 2024 · fa74be9 · fa74be9
1 parent 539da1e
commit fa74be9
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 13 deletions.
diff --git a/lib/rules/no-deprecated-api.js b/lib/rules/no-deprecated-api.js
@@ -18,8 +18,9 @@ const unprefixNodeColon = require("../util/unprefix-node-colon")
 
 /**
  * @typedef DeprecatedInfo
- * @property {string} since
- * @property {string|{ name: string, supported: string }[]|null} replacedBy
+ * @property {string} since the version when the API was deprecated.
+ * @property {string|{ name: string, supported: string }[]|null} replacedBy the text of substitute way.
+ * @property {string} [removed] the version when the API was removed.
  */
 /**
  * @typedef ParsedOptions
@@ -626,7 +627,7 @@ const globals = {
     },
     Intl: {
         v8BreakIterator: {
-            [READ]: { since: "7.0.0", replacedBy: null },
+            [READ]: { since: "7.0.0", replacedBy: null, removed: "9.0.0" },
         },
     },
     require: {
@@ -745,6 +746,8 @@ module.exports = {
         messages: {
             deprecated:
                 "{{name}} was deprecated since v{{version}}{{replace}}.",
+            removed:
+                "{{name}} was deprecated since v{{version}}, and removed in v{{removed}}.",
         },
     },
     create(context) {
@@ -760,17 +763,21 @@ module.exports = {
          * @returns {void}
          */
         function reportItem(node, name, info) {
+            const messageId = info.removed ? "removed" : "deprecated"
+            const data = {
+                name,
+                version: info.since,
+                replace: toReplaceMessage(info.replacedBy, version),
+            }
+            if (info.removed) Object.assign(data, { removed: info.removed })
+
             context.report({
                 node,
                 loc: /** @type {NonNullable<import('estree').Node["loc"]>} */ (
                     node.loc
                 ),
-                messageId: "deprecated",
-                data: {
-                    name,
-                    version: info.since,
-                    replace: toReplaceMessage(info.replacedBy, version),
-                },
+                messageId,
+                data,
             })
         }
 

diff --git a/lib/unsupported-features/types.js b/lib/unsupported-features/types.js
@@ -17,8 +17,9 @@
 
 /**
  * @typedef DeprecatedInfo
- * @property {string} since
- * @property {string|{ name: string, supported: string }[]|null} replacedBy
+ * @property {string} since the version when the API was deprecated.
+ * @property {string|{ name: string, supported: string }[]|null} replacedBy the text of substitute way.
+ * @property {string} [removed] the version when the API was removed.
  */
 
 /** @typedef {import('@eslint-community/eslint-utils').TraceMap<DeprecatedInfo>} DeprecatedInfoTraceMap */

diff --git a/package.json b/package.json
@@ -20,7 +20,7 @@
         "enhanced-resolve": "^5.15.0",
         "eslint-plugin-es-x": "^7.5.0",
         "get-tsconfig": "^4.7.0",
-        "globals": "^14.0.0",
+        "globals": "^15.0.0",
         "ignore": "^5.2.4",
         "minimatch": "^9.0.0",
         "semver": "^7.5.3"

diff --git a/tests/lib/rules/no-deprecated-api.js b/tests/lib/rules/no-deprecated-api.js
@@ -791,7 +791,16 @@ ruleTester.run("no-deprecated-api", rule, {
         {
             code: "Intl.v8BreakIterator;",
             options: [{ version: "7.0.0" }],
-            errors: ["'Intl.v8BreakIterator' was deprecated since v7.0.0."],
+            errors: [
+                {
+                    messageId: "removed",
+                    data: {
+                        name: "'Intl.v8BreakIterator'",
+                        version: "7.0.0",
+                        removed: "9.0.0",
+                    },
+                },
+            ],
         },
         {
             code: "require.extensions;",