@fluid-topics/ft-eslint 1.4.4 → 2.0.0

package/build/external-rules.js

@@ -31,4 +31,5 @@ export const externalRules = {
         },
     ],
     "@typescript-eslint/no-explicit-any": ["warn"],
+    "@typescript-eslint/consistent-type-assertions": ["warn", { assertionStyle: "never" }],
 };

package/build/plugin.d.ts

@@ -10,6 +10,8 @@ declare const plugin: {
         "number-property": import("eslint").Rule.RuleModule;
         "lower-case-ft-css-variables": import("eslint").Rule.RuleModule;
         "ft-css-variable-names": import("eslint").Rule.RuleModule;
+        "custom-elements": import("eslint").Rule.RuleModule;
+        "index-define-imports": import("eslint").Rule.RuleModule;
     };
     configs: {
         recommended: {};

package/build/plugin.js

@@ -4,6 +4,8 @@ import { interfaceNoCommaRule } from "./rules/interface-no-commas";
 import { numberPropertyRule } from "./rules/number-property";
 import { lowerCaseFtCssVariables } from "./rules/lower-case-ft-css-variables";
 import { ftCssVariableNames } from "./rules/ft-css-variable-names";
+import { customElementsRule } from "./rules/custom-elements";
+import { indexDefineImportRule } from "./rules/index-define-import";
 const plugin = {
     meta: {
         name: "eslint-plugin-fluid-topics",
@@ -16,6 +18,8 @@ const plugin = {
         "number-property": numberPropertyRule,
         "lower-case-ft-css-variables": lowerCaseFtCssVariables,
         "ft-css-variable-names": ftCssVariableNames,
+        "custom-elements": customElementsRule,
+        "index-define-imports": indexDefineImportRule,
     },
     configs: {
         recommended: {},

package/build/rules/custom-elements.d.ts

@@ -0,0 +1,2 @@
+import { Rule } from "eslint";
+export declare const customElementsRule: Rule.RuleModule;

package/build/rules/custom-elements.js

@@ -0,0 +1,50 @@
+export const customElementsRule = {
+    meta: {
+        docs: {
+            description: "Enforces that customElement definitions are in define.ts",
+            recommended: false,
+        },
+        fixable: "code",
+        hasSuggestions: true,
+        schema: [],
+        messages: {
+            forbiddenDecorator: "@customElement decorator is not allowed, define the element in a define.ts file.",
+            wrongFilename: "customElement calls must be in a define.ts file.",
+        },
+    },
+    create(context) {
+        return {
+            "ClassDeclaration": (node) => {
+                var _a;
+                const tsClass = node;
+                for (const decorator of (_a = tsClass.decorators) !== null && _a !== void 0 ? _a : []) {
+                    if (decorator.expression.type !== "CallExpression") {
+                        continue;
+                    }
+                    const expression = decorator.expression;
+                    if (expression.callee.type !== "Identifier") {
+                        continue;
+                    }
+                    const callee = expression.callee;
+                    if (callee.name === "customElement") {
+                        context.report({
+                            loc: decorator.loc,
+                            messageId: "forbiddenDecorator",
+                        });
+                    }
+                }
+            },
+            "CallExpression[callee.name='customElement']": (node) => {
+                if (node.parent.type === "Decorator") {
+                    return; // handled by above case
+                }
+                if (!context.filename.endsWith("define.ts")) {
+                    context.report({
+                        loc: node.loc,
+                        messageId: "wrongFilename",
+                    });
+                }
+            },
+        };
+    },
+};

package/build/rules/index-define-import.d.ts

@@ -0,0 +1,2 @@
+import { Rule } from "eslint";
+export declare const indexDefineImportRule: Rule.RuleModule;

package/build/rules/index-define-import.js

@@ -0,0 +1,33 @@
+export const indexDefineImportRule = {
+    meta: {
+        docs: {
+            description: "Enforces full imports in the form \"@fluid-topics/ft-component/build/define\"",
+            recommended: false,
+        },
+        fixable: "code",
+        hasSuggestions: true,
+        schema: [],
+        messages: {
+            forbiddenImport: "Using imports in the form `import \"@fluid-topics/ft-component\"` does not work, you need to import define file.",
+        },
+    },
+    create(context) {
+        return {
+            "ImportDeclaration": (node) => {
+                if (node.specifiers.length !== 0) {
+                    return;
+                }
+                const packageName = node.source.value;
+                console.log(packageName);
+                if (packageName.startsWith("@fluid-topics/ft") && !packageName.endsWith("/build/define")) {
+                    const cleanName = packageName.replace(/\/$/, "");
+                    context.report({
+                        loc: node.loc,
+                        messageId: "forbiddenImport",
+                        fix: (fixer) => fixer.replaceTextRange(node.source.range, `"${cleanName}/build/define"`),
+                    });
+                }
+            },
+        };
+    },
+};

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@fluid-topics/ft-eslint",
-    "version": "1.4.4",
+    "version": "2.0.0",
     "description": "ESlint rules for web components",
     "keywords": [
         "Lit"
@@ -21,5 +21,5 @@
     "dependencies": {
         "lit": "3.1.0"
     },
-    "gitHead": "fdd626e6a7b6cfef613b7060c1ac2ece1073a1da"
+    "gitHead": "139801842483cdd25e6c0d850431dfb2b930f394"
 }