Cleanup after merging

Author: l1bbcsg

src/jsdocUtils.js

@@ -197,7 +197,7 @@ const potentiallyEmptyNamepathTags = [
   'see',
 
   // GCC syntax allows typedef to be named through variable declaration rather than jsdoc name
-  'typedef'
+  'typedef',
 ];
 
 const isPotentiallyEmptyNamepathTag = (tag) => {
@@ -249,7 +249,7 @@ const potentiallyEmptyTypeTags = [
   'return', 'returns',
   'throws', 'exception',
   'yields', 'yield',
-  'package', 'private', 'protected', 'public', 'static'
+  'package', 'private', 'protected', 'public', 'static',
 ];
 
 const isPotentiallyEmptyTypeTag = (tag) => {
@@ -263,7 +263,7 @@ const isTagWithType = (tagName) => {
 const tagsWithMandatoryNamepathOrType = [
   'augments', 'extends',
   'param', 'arg',
-  'typedef'
+  'typedef',
 ];
 const isTagWithMandatoryNamepathOrType = (tagName) => {
   return tagsWithMandatoryNamepathOrType.includes(tagName);

src/rules/validTypes.js

@@ -64,7 +64,7 @@ export default iterateJsdoc(({
     const hasType = Boolean(tag.type);
     const mustHaveType = utils.isTagWithType(tag.tag) && !utils.isPotentiallyEmptyTypeTag(tag.tag);
 
-    const hasNamePath = Boolean(tag.name);
+    const hasNamePath = Boolean(tag.name) && !(tag.tag === 'see' && !checkSeesForNamepaths);
     const mustHaveNamepath = utils.isNamepathTag(tag.tag, checkSeesForNamepaths) &&
       !utils.passesEmptyNamepathCheck(tag, allowEmptyNamepaths);
 

test/rules/assertions/validTypes.js

@@ -183,7 +183,7 @@ export default {
         message: 'Tag @callback must have a namepath',
       }],
       options: [{
-        allowEmptyNamepaths: false
+        allowEmptyNamepaths: false,
       }],
     },
     {
@@ -196,9 +196,9 @@ export default {
       errors: [
         {
           line: 3,
-          message: 'Syntax error in type: str%ng'
-        }
-      ]
+          message: 'Syntax error in type: str%ng',
+        },
+      ],
     },
     {
       code: `
@@ -209,9 +209,9 @@ export default {
       errors: [
         {
           line: 3,
-          message: 'Syntax error in type: str%ng'
-        }
-      ]
+          message: 'Syntax error in type: str%ng',
+        },
+      ],
     },
     {
       code: `
@@ -222,9 +222,9 @@ export default {
       errors: [
         {
           line: 3,
-          message: 'Syntax error in type: UserStr%ng'
-        }
-      ]
+          message: 'Syntax error in type: UserStr%ng',
+        },
+      ],
     },
     {
       code: `
@@ -236,9 +236,9 @@ export default {
       errors: [
         {
           line: 3,
-          message: 'Tag @extends must have either a type or namepath'
-        }
-      ]
+          message: 'Tag @extends must have either a type or namepath',
+        },
+      ],
     },
     {
       code: `
@@ -250,10 +250,10 @@ export default {
       errors: [
         {
           line: 3,
-          message: 'Tag @type must have a type'
-        }
-      ]
-    }
+          message: 'Tag @type must have a type',
+        },
+      ],
+    },
   ],
   valid: [
     {
@@ -403,38 +403,38 @@ export default {
            * @constant {string}
            */
            const FOO = 'foo';
-      `
+      `,
     },
     {
       code: `
           /**
            * @extends Foo
            */
            class Bar {};
-      `
+      `,
     },
     {
       code: `
           /**
            * @extends {Foo<String>}
            */
            class Bar {};
-      `
+      `,
     },
     {
       code: `
           /**
            * @typedef {number|string} UserDefinedType
            */
-      `
+      `,
     },
     {
       code: `
           /**
            * @typedef {number|string}
            */
           let UserDefinedGCCType;
-      `
-    }
+      `,
+    },
   ],
 };