fix(check-tag-names): add record and undocumented tags in Closure source to Closure tags; fixes #306

Also removes mistaken use of anchor in documentation URL (minor), indicates Closure synonyms; and gives URL for source of undocumented `modifies`

Author: brettz9

.README/rules/check-tag-names.md

@@ -47,7 +47,6 @@ memberof
 memberof!
 mixes
 mixin
-modifies (Currently undocumented but in source)
 module
 name
 namespace
@@ -76,6 +75,8 @@ version
 yields
 ```
 
+`modifies` is also supported (see [source](https://github.com/jsdoc/jsdoc/blob/master/packages/jsdoc/lib/jsdoc/tag/dictionary/definitions.js#L594)) but is undocumented.
+
 The following synonyms are also recognized:
 
 ```
@@ -108,12 +109,12 @@ one may also use the following:
 template
 ```
 
-And for [Closure](https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler#nosideeffects-modifies-thisarguments),
+And for [Closure](https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler),
 when `settings.jsdoc.mode` is set to `closure`, one may use the following (in
 addition to the jsdoc and TypeScript tags):
 
 ```
-define
+define (synonym of `const` per jsdoc source)
 dict
 export
 externs
@@ -128,11 +129,29 @@ nosideeffects
 polymer
 polymerBehavior
 preserve
+record (synonym of `interface` per jsdoc source)
 struct
 suppress
 unrestricted
 ```
 
+...and these undocumented tags which are only in [source](https://github.com/google/closure-compiler/blob/master/src/com/google/javascript/jscomp/parsing/Annotation.java):
+
+```
+closurePrimitive
+customElement
+expose
+hidden
+idGenerator
+meaning
+mixinClass
+mixinFunction
+ngInject
+owner
+typeSummary
+wizaction
+```
+
 Note that the tags indicated as replacements in `settings.jsdoc.tagNamePreference` will automatically be considered as valid.
 
 #### Options

README.md

@@ -1427,7 +1427,6 @@ memberof
 memberof!
 mixes
 mixin
-modifies (Currently undocumented but in source)
 module
 name
 namespace
@@ -1456,6 +1455,8 @@ version
 yields
 ```
 
+`modifies` is also supported (see [source](https://github.com/jsdoc/jsdoc/blob/master/packages/jsdoc/lib/jsdoc/tag/dictionary/definitions.js#L594)) but is undocumented.
+
 The following synonyms are also recognized:
 
 ```
@@ -1488,12 +1489,12 @@ one may also use the following:
 template
 ```
 
-And for [Closure](https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler#nosideeffects-modifies-thisarguments),
+And for [Closure](https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler),
 when `settings.jsdoc.mode` is set to `closure`, one may use the following (in
 addition to the jsdoc and TypeScript tags):
 
 ```
-define
+define (synonym of `const` per jsdoc source)
 dict
 export
 externs
@@ -1508,11 +1509,29 @@ nosideeffects
 polymer
 polymerBehavior
 preserve
+record (synonym of `interface` per jsdoc source)
 struct
 suppress
 unrestricted
 ```
 
+...and these undocumented tags which are only in [source](https://github.com/google/closure-compiler/blob/master/src/com/google/javascript/jscomp/parsing/Annotation.java):
+
+```
+closurePrimitive
+customElement
+expose
+hidden
+idGenerator
+meaning
+mixinClass
+mixinFunction
+ngInject
+owner
+typeSummary
+wizaction
+```
+
 Note that the tags indicated as replacements in `settings.jsdoc.tagNamePreference` will automatically be considered as valid.
 
 <a name="eslint-plugin-jsdoc-rules-check-tag-names-options-2"></a>

src/tagNames.js

@@ -122,8 +122,26 @@ const typeScriptTags = {
   template: [],
 };
 
+const undocumentedClosureTags = {
+  // These are in Closure source but not in jsdoc source nor in the Closure
+  //  docs: https://github.com/google/closure-compiler/blob/master/src/com/google/javascript/jscomp/parsing/Annotation.java
+  closurePrimitive: [],
+  customElement: [],
+  expose: [],
+  hidden: [],
+  idGenerator: [],
+  meaning: [],
+  mixinClass: [],
+  mixinFunction: [],
+  ngInject: [],
+  owner: [],
+  typeSummary: [],
+  wizaction: [],
+};
+
 const closureTags = {
   ...typeScriptTags,
+  ...undocumentedClosureTags,
 
   // From https://github.com/google/closure-compiler/wiki/Annotating-JavaScript-for-the-Closure-Compiler
   // These are all recognized in https://github.com/jsdoc/jsdoc/blob/master/packages/jsdoc/lib/jsdoc/tag/dictionary/definitions.js
@@ -151,6 +169,10 @@ const closureTags = {
   polymer: [],
   polymerBehavior: [],
   preserve: [],
+
+  // Defined as a synonym of `interface` in jsdoc `definitions.js`
+  record: [],
+
   struct: [],
   suppress: [],