Remove no-types overlap

Author: JoshuaKGoldberg

README.md

@@ -4628,29 +4628,6 @@ let a;
 // "jsdoc/check-tag-names": ["error"|"warn", {"typed":true}]
 // Message: '@template' without a name is redundant when using a type system.
 
-/** @param {string} */
-function takesOne(param) {}
-// "jsdoc/check-tag-names": ["error"|"warn", {"typed":true}]
-// Message: '@param' without a description is redundant when using a type system.
-
-/** @param {boolean} param */
-function takesOne(param) {}
-// "jsdoc/check-tag-names": ["error"|"warn", {"typed":true}]
-// Message: '@param' without a description is redundant when using a type system.
-
-/**
- * Existing comment
- *  @param {boolean} param
- */
-function takesOne(param) {}
-// "jsdoc/check-tag-names": ["error"|"warn", {"typed":true}]
-// Message: '@param' without a description is redundant when using a type system.
-
-/** @param {boolean} param - takes description */
-function takesOne(param) {}
-// "jsdoc/check-tag-names": ["error"|"warn", {"typed":true}]
-// Message: Describing the type of '@param' is redundant when using a type system.
-
 /** @typoo {string} */
 let a;
 // Message: Invalid JSDoc tag name "typoo".

src/rules/checkTagNames.js

@@ -56,13 +56,6 @@ const typedTagsUnnecessaryOutsideDeclare = new Set([
   'this',
 ]);
 
-const typedTagsParamReturn = new Set([
-  'param',
-  'parameter',
-  'return',
-  'returns',
-]);
-
 export default iterateJsdoc(({
   sourceCode,
   jsdoc,
@@ -152,24 +145,6 @@ export default iterateJsdoc(({
     }, true);
   };
 
-  const tagRedundantlyChecksParamOrReturnTyped = (jsdocTag, tagIndex) => {
-    if (!typedTagsParamReturn.has(jsdocTag.tag)) {
-      return false;
-    }
-
-    if (!jsdocTag.description.trim()) {
-      reportWithTypeRemovalFixer(`'@${jsdocTag.tag}' without a description is redundant when using a type system.`, jsdocTag, tagIndex);
-      return true;
-    }
-
-    if (jsdocTag.type) {
-      reportWithTypeRemovalFixer(`Describing the type of '@${jsdocTag.tag}' is redundant when using a type system.`, jsdocTag, tagIndex);
-      return true;
-    }
-
-    return false;
-  };
-
   const checkTagForTypedValidity = (jsdocTag, tagIndex) => {
     if (typedTagsAlwaysUnnecessary.has(jsdocTag.tag)) {
       reportWithTypeRemovalFixer(`'@${jsdocTag.tag}' is redundant when using a type system.`, jsdocTag, tagIndex, {
@@ -189,10 +164,6 @@ export default iterateJsdoc(({
       return true;
     }
 
-    if (tagRedundantlyChecksParamOrReturnTyped(jsdocTag, tagIndex)) {
-      return true;
-    }
-
     return false;
   };
 

test/rules/assertions/checkTagNames.js

@@ -235,93 +235,6 @@ export default {
         let a;
       `,
     },
-    {
-      code: `
-        /** @param {string} */
-        function takesOne(param) {}
-      `,
-      errors: [
-        {
-          line: 2,
-          message: '\'@param\' without a description is redundant when using a type system.',
-        },
-      ],
-      options: [
-        {
-          typed: true,
-        },
-      ],
-      output: `
-        function takesOne(param) {}
-      `,
-    },
-    {
-      code: `
-        /** @param {boolean} param */
-        function takesOne(param) {}
-      `,
-      errors: [
-        {
-          line: 2,
-          message: '\'@param\' without a description is redundant when using a type system.',
-        },
-      ],
-      options: [
-        {
-          typed: true,
-        },
-      ],
-      output: `
-        function takesOne(param) {}
-      `,
-    },
-    {
-      code: `
-        /**
-         * Existing comment
-         *  @param {boolean} param
-         */
-        function takesOne(param) {}
-      `,
-      errors: [
-        {
-          line: 4,
-          message: '\'@param\' without a description is redundant when using a type system.',
-        },
-      ],
-      options: [
-        {
-          typed: true,
-        },
-      ],
-      output: `
-        /**
-         * Existing comment
-         */
-        function takesOne(param) {}
-      `,
-    },
-    {
-      code: `
-        /** @param {boolean} param - takes description */
-        function takesOne(param) {}
-      `,
-      errors: [
-        {
-          line: 2,
-          message: 'Describing the type of \'@param\' is redundant when using a type system.',
-        },
-      ],
-      options: [
-        {
-          typed: true,
-        },
-      ],
-      output: `
-        /** @param param - takes description */
-        function takesOne(param) {}
-      `,
-    },
     {
       code: `
         /** @typoo {string} */