@discourse/lint-configs 2.33.0 → 2.34.1

package/eslint-rules/template-tag-no-self-this.mjs

@@ -0,0 +1,62 @@
+export default {
+  meta: {
+    type: "suggestion",
+    docs: {
+      description:
+        "Avoid using `const self = this`. Use `this` directly inside template tags.",
+    },
+    fixable: "code",
+    schema: [], // no options
+  },
+
+  create(context) {
+    return {
+      VariableDeclarator(node) {
+        if (
+          node.id.type !== "Identifier" ||
+          node.init?.type !== "ThisExpression"
+        ) {
+          return;
+        }
+
+        const variable = context.sourceCode.getDeclaredVariables(node)[0];
+        const references = variable.references.filter(
+          (ref) => ref.identifier.parent !== node
+        );
+
+        const referencedOnlyInAdjacentTemplateTag = references.every((ref) => {
+          if (ref.identifier.type !== "VarHead") {
+            return false;
+          }
+
+          const hasSameVariableScope =
+            context.sourceCode.getScope(ref.identifier).variableScope ===
+            context.sourceCode.getScope(node).variableScope;
+
+          return hasSameVariableScope;
+        });
+
+        if (referencedOnlyInAdjacentTemplateTag) {
+          context.report({
+            node,
+            message:
+              "Remove `self = this` and use `this` directly inside template tags.",
+            fix(fixer) {
+              const fixes = [];
+
+              // Replace all references to `self` with `this`
+              references.forEach((ref) => {
+                fixes.push(fixer.replaceText(ref.identifier, "this"));
+              });
+
+              // Remove the original variable declaration
+              fixes.push(fixer.remove(node.parent));
+
+              return fixes;
+            },
+          });
+        }
+      },
+    };
+  },
+};

package/eslint.mjs

@@ -29,6 +29,7 @@ import noSimpleQuerySelector from "./eslint-rules/no-simple-query-selector.mjs";
 import noUnusedServices from "./eslint-rules/no-unused-services.mjs";
 import pluginApiNoVersion from "./eslint-rules/plugin-api-no-version.mjs";
 import serviceInjectImport from "./eslint-rules/service-inject-import.mjs";
+import templateTagNoSelfThis from "./eslint-rules/template-tag-no-self-this.mjs";
 import themeImports from "./eslint-rules/theme-imports.mjs";
 import truthHelpersImports from "./eslint-rules/truth-helpers-imports.mjs";
 
@@ -138,6 +139,7 @@ export default [
           "capital-components": capitalComponents,
           "no-onclick": noOnclick,
           "no-route-template": noRouteTemplate,
+          "template-tag-no-self-this": templateTagNoSelfThis,
         },
       },
     },
@@ -314,6 +316,7 @@ export default [
       "discourse/no-curly-components": ["error"],
       "discourse/capital-components": ["error"],
       "discourse/no-onclick": ["error"],
+      "discourse/template-tag-no-self-this": ["error"],
       // "discourse/no-route-template": ["error"], // Enable by default once Ember 6.6 is on stable
     },
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@discourse/lint-configs",
-  "version": "2.33.0",
+  "version": "2.34.1",
   "description": "Shareable lint configs for Discourse core, plugins, and themes",
   "author": "Discourse",
   "license": "MIT",
@@ -12,6 +12,9 @@
     "url": "git+https://github.com/discourse/lint-configs.git"
   },
   "homepage": "https://github.com/discourse/lint-configs#readme",
+  "publishConfig": {
+    "access": "public"
+  },
   "type": "module",
   "exports": {
     "./eslint": "./eslint.mjs",