Add 19 template rules (removed no-index-component-invocation, fixed JSDoc) - 101/127 total (79%)

Co-authored-by: NullVoxPopuli <[email protected]>

Author: Copilot

README.md

@@ -249,7 +249,6 @@ rules in templates can be disabled with eslint directives with mustache or html
 | [template-no-forbidden-elements](docs/rules/template-no-forbidden-elements.md)                                         | disallow specific HTML elements                                        |                                             |    |    |
 | [template-no-html-comments](docs/rules/template-no-html-comments.md)                                                   | disallow HTML comments in templates                                    |                                             |    |    |
 | [template-no-implicit-this](docs/rules/template-no-implicit-this.md)                                                   | require explicit `this` in property access                             |                                             |    |    |
-| [template-no-index-component-invocation](docs/rules/template-no-index-component-invocation.md)                         | disallow invocation of index components                                | ![badge-strict-gjs][] ![badge-strict-gts][] |    |    |
 | [template-no-inline-event-handlers](docs/rules/template-no-inline-event-handlers.md)                                   | disallow DOM event handler attributes                                  | ![badge-strict-gjs][] ![badge-strict-gts][] |    |    |
 | [template-no-inline-linkto](docs/rules/template-no-inline-linkto.md)                                                   | disallow inline form of LinkTo component                               | ![badge-strict-gjs][] ![badge-strict-gts][] |    |    |
 | [template-no-inline-styles](docs/rules/template-no-inline-styles.md)                                                   | disallow inline styles                                                 |                                             |    |    |

docs/rules/template-no-index-component-invocation.md

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Disallow curly component invocation
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'suggestion',

lib/rules/template-no-curly-component-invocation.js

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Require explicit this for property access in templates
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'suggestion',

lib/rules/template-no-implicit-this.js

@@ -1,3 +1,14 @@
+/**
+ * @param {any} node
+ * @returns {boolean}
+ */
+function isLinkToComponent(node) {
+  if (node.type === 'GlimmerElementNode') {
+    return node.tag === 'LinkTo' || node.tag === 'link-to';
+  }
+  return false;
+}
+
 /** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
@@ -16,12 +27,6 @@ module.exports = {
   },
 
   create(context) {
-    function isLinkToComponent(node) {
-      if (node.type === 'GlimmerElementNode') {
-        return node.tag === 'LinkTo' || node.tag === 'link-to';
-      }
-      return false;
-    }
 
     return {
       GlimmerElementNode(node) {

lib/rules/template-no-index-component-invocation.js

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Disallow unnecessary usage of (component) helper
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'suggestion',

lib/rules/template-no-link-to-tagname.js

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Disallow unused block params
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'suggestion',

lib/rules/template-no-unnecessary-component-helper.js

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Disallow whitespace within mustache or block expression
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'layout',

lib/rules/template-no-unused-block-params.js

@@ -1,8 +1,4 @@
-/**
- * @fileoverview Require label for form input elements
- * @type {import('eslint').Rule.RuleModule}
- */
-
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
     type: 'suggestion',