fixvalid-types): require or disallow names and types per tag; part of gajus#233

1. Make `constructs` disallow type (but optional name)
2. Require namepath and disallow type for `emits`;
3. Disallow type on `exports` except in closure/permissive modes;
4. Disallow type on `external` and `host`
5. Disallow type on `fires` but require name
6. Specify name as optional but type required on `function`
7. `listens`: should not have a type but must have a name
8. Make `miixin` name optional but disallow type
9. `requires` must have name but not have a type
10. `exports` must have a name in jsdoc mode
11. `interface`: disallow type

Author: brettz9

README.md

@@ -21275,7 +21275,7 @@ function quux() {
 
 /**
  *
- * @fires {module:namespace.SomeClass#event:ext_anevent}
+ * @fires module:namespace.SomeClass#event:ext_anevent
  */
 function quux() {
 

src/getDefaultTagStructureForMode.js

@@ -150,6 +150,11 @@ const getDefaultTagStructureForMode = (mode) => {
           'nameContents', 'namepath-defining',
         ],
 
+        // Not in use, but should be this value if using to power `empty-tags`
+        [
+          'nameAllowed', true,
+        ],
+
         [
           'typeAllowed', true,
         ],
@@ -193,6 +198,23 @@ const getDefaultTagStructureForMode = (mode) => {
       ]),
     ],
 
+    [
+      'constructs', new Map([
+        // Allows for "name"'s in signature, but indicated as optional
+        [
+          'nameContents', 'namepath-defining',
+        ],
+
+        [
+          'nameRequired', false,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
+      ]),
+    ],
+
     [
       'define', new Map([
         [
@@ -207,6 +229,14 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameContents', 'namepath-referencing',
         ],
+
+        [
+          'nameRequired', true,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -244,6 +274,7 @@ const getDefaultTagStructureForMode = (mode) => {
       ]),
     ],
 
+    // Closure
     [
       'export', new Map([
         [
@@ -252,6 +283,22 @@ const getDefaultTagStructureForMode = (mode) => {
       ]),
     ],
 
+    [
+      'exports', new Map([
+        [
+          'nameContents', 'namepath-defining',
+        ],
+
+        [
+          'nameRequired', isJsdoc,
+        ],
+
+        [
+          'typeAllowed', isClosureOrPermissive,
+        ],
+      ]),
+    ],
+
     [
       'extends', new Map([
         // Signature seems to require a "namepath" (and no counter-examples)
@@ -288,6 +335,10 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameRequired', true,
         ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -298,6 +349,14 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameContents', 'namepath-referencing',
         ],
+
+        [
+          'nameRequired', true,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -307,6 +366,14 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameContents', 'namepath-defining',
         ],
+
+        [
+          'nameRequired', false,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
     [
@@ -332,9 +399,8 @@ const getDefaultTagStructureForMode = (mode) => {
           'nameRequired', true,
         ],
 
-        // "namepath"
         [
-          'typeOrNameRequired', true,
+          'typeAllowed', false,
         ],
       ]),
     ],
@@ -346,6 +412,15 @@ const getDefaultTagStructureForMode = (mode) => {
           'nameContents',
           isJsdocTypescriptOrPermissive ? 'namepath-defining' : false,
         ],
+
+        // Not in use, but should be this value if using to power `empty-tags`
+        [
+          'nameAllowed', isClosure,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -380,6 +455,14 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameContents', 'namepath-referencing',
         ],
+
+        [
+          'nameRequired', true,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -455,6 +538,14 @@ const getDefaultTagStructureForMode = (mode) => {
         [
           'nameContents', 'namepath-defining',
         ],
+
+        [
+          'nameRequired', false,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
       ]),
     ],
 
@@ -618,6 +709,19 @@ const getDefaultTagStructureForMode = (mode) => {
       ]),
     ],
 
+    [
+      'requires', new Map([
+        // <someModuleName>
+        [
+          'nameRequired', true,
+        ],
+
+        [
+          'typeAllowed', false,
+        ],
+      ]),
+    ],
+
     [
       'returns', new Map([
         // Shows curly brackets in the signature and in the examples