@servicetitan/eslint-plugin 16.0.0

package/dist/index.d.ts

@@ -0,0 +1,11 @@
+export declare const rules: {
+    'use-declare-with-decorators': import("eslint").Rule.RuleModule;
+    'mobx/use-makeObservable-with-decorators': import("eslint").Rule.RuleModule;
+    'mobx/no-abstract-decorators': import("eslint").Rule.RuleModule;
+    'react/destructure-default-import': import("eslint").Rule.RuleModule;
+    'react/no-qualified-type': import("eslint").Rule.RuleModule;
+};
+export declare const processors: {
+    stub: import("eslint").Linter.LintOptions;
+};
+//# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAOA,eAAO,MAAM,KAAK;;;;;;CAMjB,CAAC;AAEF,eAAO,MAAM,UAAU;;CAAW,CAAC"}

package/dist/index.js

@@ -0,0 +1,18 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.processors = exports.rules = void 0;
+const decorators_declare_1 = require("./rules/decorators-declare");
+const use_makeObservable_with_decorators_1 = require("./rules/mobx/use-makeObservable-with-decorators");
+const no_abstract_decorators_1 = require("./rules/mobx/no-abstract-decorators");
+const destructure_default_import_1 = require("./rules/react/destructure-default-import");
+const no_qualified_type_1 = require("./rules/react/no-qualified-type");
+const stub_1 = require("./processors/stub");
+exports.rules = {
+    'use-declare-with-decorators': decorators_declare_1.useDeclareWithDecorators,
+    'mobx/use-makeObservable-with-decorators': use_makeObservable_with_decorators_1.mobxUseMakeObservableWithDecorators,
+    'mobx/no-abstract-decorators': no_abstract_decorators_1.mobxNoAbstractDecorators,
+    'react/destructure-default-import': destructure_default_import_1.reactDestructureDefaultImport,
+    'react/no-qualified-type': no_qualified_type_1.reactNoQualifiedType,
+};
+exports.processors = { stub: stub_1.stub };
+//# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;AAAA,mEAAsE;AACtE,wGAAsG;AACtG,gFAA+E;AAC/E,yFAAyF;AACzF,uEAAuE;AACvE,4CAAyC;AAE5B,QAAA,KAAK,GAAG;IACjB,6BAA6B,EAAE,6CAAwB;IACvD,yCAAyC,EAAE,wEAAmC;IAC9E,6BAA6B,EAAE,iDAAwB;IACvD,kCAAkC,EAAE,0DAA6B;IACjE,yBAAyB,EAAE,wCAAoB;CAClD,CAAC;AAEW,QAAA,UAAU,GAAG,EAAE,IAAI,EAAJ,WAAI,EAAE,CAAC"}

package/dist/processors/stub.d.ts

@@ -0,0 +1,3 @@
+import { Linter } from 'eslint';
+export declare const stub: Linter.LintOptions;
+//# sourceMappingURL=stub.d.ts.map

package/dist/processors/stub.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"stub.d.ts","sourceRoot":"","sources":["../../src/processors/stub.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAEhC,eAAO,MAAM,IAAI,EAAE,MAAM,CAAC,WAGzB,CAAC"}

package/dist/processors/stub.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.stub = void 0;
+exports.stub = {
+    preprocess: () => [''],
+    postprocess: ([messages]) => messages,
+};
+//# sourceMappingURL=stub.js.map

package/dist/processors/stub.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"stub.js","sourceRoot":"","sources":["../../src/processors/stub.ts"],"names":[],"mappings":";;;AAEa,QAAA,IAAI,GAAuB;IACpC,UAAU,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC;IACtB,WAAW,EAAE,CAAC,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC,QAAQ;CACxC,CAAC"}

package/dist/rules/decorators-declare.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from 'eslint';
+export declare const useDeclareWithDecorators: Rule.RuleModule;
+//# sourceMappingURL=decorators-declare.d.ts.map

package/dist/rules/decorators-declare.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"decorators-declare.d.ts","sourceRoot":"","sources":["../../src/rules/decorators-declare.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAI9B,eAAO,MAAM,wBAAwB,EAAE,IAAI,CAAC,UAsC3C,CAAC"}

package/dist/rules/decorators-declare.js

@@ -0,0 +1,44 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.useDeclareWithDecorators = void 0;
+const customDecorators = ['injectDependency', 'koObservableToMobx'];
+exports.useDeclareWithDecorators = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow custom @servicetitan class field decorators without declare',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            decoratorsDeclare: "Use declare when decorating with '@{{name}}'",
+        },
+    },
+    create(context) {
+        return {
+            ClassProperty(node) {
+                var _a;
+                if (node.declare) {
+                    return; // field is already defined with declare
+                }
+                if ((_a = node === null || node === void 0 ? void 0 : node.decorators) === null || _a === void 0 ? void 0 : _a.length) {
+                    node.decorators.forEach((decorator) => {
+                        var _a, _b;
+                        const name = (_b = (_a = decorator.expression) === null || _a === void 0 ? void 0 : _a.callee) === null || _b === void 0 ? void 0 : _b.name;
+                        if (name && customDecorators.includes(name)) {
+                            context.report({
+                                node,
+                                messageId: 'decoratorsDeclare',
+                                data: {
+                                    name,
+                                },
+                            });
+                        }
+                    });
+                }
+            },
+        };
+    },
+};
+//# sourceMappingURL=decorators-declare.js.map

package/dist/rules/decorators-declare.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"decorators-declare.js","sourceRoot":"","sources":["../../src/rules/decorators-declare.ts"],"names":[],"mappings":";;;AAEA,MAAM,gBAAgB,GAAG,CAAC,kBAAkB,EAAE,oBAAoB,CAAC,CAAC;AAEvD,QAAA,wBAAwB,GAAoB;IACrD,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,sEAAsE;YACnF,QAAQ,EAAE,iBAAiB;YAC3B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,2CAA2C;SACnD;QACD,QAAQ,EAAE;YACN,iBAAiB,EAAE,8CAA8C;SACpE;KACJ;IACD,MAAM,CAAC,OAAO;QACV,OAAO;YACH,aAAa,CAAC,IAAS;;gBACnB,IAAI,IAAI,CAAC,OAAO,EAAE;oBACd,OAAO,CAAC,wCAAwC;iBACnD;gBAED,IAAI,MAAA,IAAI,aAAJ,IAAI,uBAAJ,IAAI,CAAE,UAAU,0CAAE,MAAM,EAAE;oBAC1B,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;;wBACvC,MAAM,IAAI,GAAG,MAAA,MAAA,SAAS,CAAC,UAAU,0CAAE,MAAM,0CAAE,IAAI,CAAC;wBAEhD,IAAI,IAAI,IAAI,gBAAgB,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE;4BACzC,OAAO,CAAC,MAAM,CAAC;gCACX,IAAI;gCACJ,SAAS,EAAE,mBAAmB;gCAC9B,IAAI,EAAE;oCACF,IAAI;iCACP;6BACJ,CAAC,CAAC;yBACN;oBACL,CAAC,CAAC,CAAC;iBACN;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/mobx/no-abstract-decorators.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from 'eslint';
+export declare const mobxNoAbstractDecorators: Rule.RuleModule;
+//# sourceMappingURL=no-abstract-decorators.d.ts.map

package/dist/rules/mobx/no-abstract-decorators.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-abstract-decorators.d.ts","sourceRoot":"","sources":["../../../src/rules/mobx/no-abstract-decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAI9B,eAAO,MAAM,wBAAwB,EAAE,IAAI,CAAC,UAsC3C,CAAC"}

package/dist/rules/mobx/no-abstract-decorators.js

@@ -0,0 +1,43 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.mobxNoAbstractDecorators = void 0;
+const mobxDecorators = ['observable', 'action', 'computed', 'override'];
+exports.mobxNoAbstractDecorators = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow MobX decorators on abstract class fields',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            noAbstractMobxDecorators: "Don't decorate abstract class fields or methods with '@{{name}}'",
+        },
+    },
+    create(context) {
+        function checkFn(node) {
+            var _a;
+            if ((_a = node === null || node === void 0 ? void 0 : node.decorators) === null || _a === void 0 ? void 0 : _a.length) {
+                node.decorators.forEach((decorator) => {
+                    var _a, _b, _c;
+                    const name = ((_a = decorator === null || decorator === void 0 ? void 0 : decorator.expression) === null || _a === void 0 ? void 0 : _a.name) || ((_c = (_b = decorator === null || decorator === void 0 ? void 0 : decorator.expression) === null || _b === void 0 ? void 0 : _b.object) === null || _c === void 0 ? void 0 : _c.name);
+                    if (name && mobxDecorators.includes(name)) {
+                        context.report({
+                            node,
+                            messageId: 'noAbstractMobxDecorators',
+                            data: {
+                                name,
+                            },
+                        });
+                    }
+                });
+            }
+        }
+        return {
+            TSAbstractClassProperty: checkFn,
+            TSAbstractMethodDefinition: checkFn,
+        };
+    },
+};
+//# sourceMappingURL=no-abstract-decorators.js.map

package/dist/rules/mobx/no-abstract-decorators.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-abstract-decorators.js","sourceRoot":"","sources":["../../../src/rules/mobx/no-abstract-decorators.ts"],"names":[],"mappings":";;;AAEA,MAAM,cAAc,GAAG,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC;AAE3D,QAAA,wBAAwB,GAAoB;IACrD,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,mDAAmD;YAChE,QAAQ,EAAE,iBAAiB;YAC3B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,2CAA2C;SACnD;QACD,QAAQ,EAAE;YACN,wBAAwB,EACpB,kEAAkE;SACzE;KACJ;IACD,MAAM,CAAC,OAAY;QACf,SAAS,OAAO,CAAC,IAAS;;YACtB,IAAI,MAAA,IAAI,aAAJ,IAAI,uBAAJ,IAAI,CAAE,UAAU,0CAAE,MAAM,EAAE;gBAC1B,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;;oBACvC,MAAM,IAAI,GAAG,CAAA,MAAA,SAAS,aAAT,SAAS,uBAAT,SAAS,CAAE,UAAU,0CAAE,IAAI,MAAI,MAAA,MAAA,SAAS,aAAT,SAAS,uBAAT,SAAS,CAAE,UAAU,0CAAE,MAAM,0CAAE,IAAI,CAAA,CAAC;oBAEhF,IAAI,IAAI,IAAI,cAAc,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE;wBACvC,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI;4BACJ,SAAS,EAAE,0BAA0B;4BACrC,IAAI,EAAE;gCACF,IAAI;6BACP;yBACJ,CAAC,CAAC;qBACN;gBACL,CAAC,CAAC,CAAC;aACN;QACL,CAAC;QAED,OAAO;YACH,uBAAuB,EAAE,OAAO;YAChC,0BAA0B,EAAE,OAAO;SACtC,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/mobx/use-makeObservable-with-decorators.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from 'eslint';
+export declare const mobxUseMakeObservableWithDecorators: Rule.RuleModule;
+//# sourceMappingURL=use-makeObservable-with-decorators.d.ts.map

package/dist/rules/mobx/use-makeObservable-with-decorators.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"use-makeObservable-with-decorators.d.ts","sourceRoot":"","sources":["../../../src/rules/mobx/use-makeObservable-with-decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAK9B,eAAO,MAAM,mCAAmC,EAAE,IAAI,CAAC,UA8DtD,CAAC"}

package/dist/rules/mobx/use-makeObservable-with-decorators.js

@@ -0,0 +1,63 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.mobxUseMakeObservableWithDecorators = void 0;
+// NOTE: "action" also covers "action.bound"
+const mobxDecorators = ['observable', 'action', 'computed', 'override'];
+exports.mobxUseMakeObservableWithDecorators = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Testing makeObservable used with decorators',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            makeObservable: 'Use makeObservable in the constructor with MobX decorators',
+            constructor: 'Add constructor with makeObservable when using MobX decorators',
+        },
+    },
+    create(context) {
+        return {
+            ClassDeclaration(node) {
+                let hasMobxDecorator = false;
+                let hasConstructor = false;
+                let hasMakeObservable = false;
+                node.body.body.forEach((methodOrProperty) => {
+                    var _a, _b;
+                    if (methodOrProperty.decorators) {
+                        methodOrProperty.decorators.forEach((decorator) => {
+                            var _a, _b, _c;
+                            if (mobxDecorators.includes(((_a = decorator === null || decorator === void 0 ? void 0 : decorator.expression) === null || _a === void 0 ? void 0 : _a.name) ||
+                                ((_c = (_b = decorator === null || decorator === void 0 ? void 0 : decorator.expression) === null || _b === void 0 ? void 0 : _b.object) === null || _c === void 0 ? void 0 : _c.name))) {
+                                hasMobxDecorator = true;
+                            }
+                        });
+                    }
+                    if (methodOrProperty.kind === 'constructor') {
+                        hasConstructor = true;
+                        (_b = (_a = methodOrProperty.value) === null || _a === void 0 ? void 0 : _a.body) === null || _b === void 0 ? void 0 : _b.body.forEach((statement) => {
+                            var _a, _b;
+                            if (((_b = (_a = statement === null || statement === void 0 ? void 0 : statement.expression) === null || _a === void 0 ? void 0 : _a.callee) === null || _b === void 0 ? void 0 : _b.name) === 'makeObservable') {
+                                hasMakeObservable = true;
+                            }
+                        });
+                    }
+                });
+                if (hasMobxDecorator && !hasMakeObservable && hasConstructor) {
+                    context.report({
+                        node,
+                        messageId: 'makeObservable',
+                    });
+                }
+                if (hasMobxDecorator && !hasMakeObservable && !hasConstructor) {
+                    context.report({
+                        node,
+                        messageId: 'constructor',
+                    });
+                }
+            },
+        };
+    },
+};
+//# sourceMappingURL=use-makeObservable-with-decorators.js.map

package/dist/rules/mobx/use-makeObservable-with-decorators.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"use-makeObservable-with-decorators.js","sourceRoot":"","sources":["../../../src/rules/mobx/use-makeObservable-with-decorators.ts"],"names":[],"mappings":";;;AAEA,4CAA4C;AAC5C,MAAM,cAAc,GAAG,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC;AAE3D,QAAA,mCAAmC,GAAoB;IAChE,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,6CAA6C;YAC1D,QAAQ,EAAE,iBAAiB;YAC3B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,2CAA2C;SACnD;QACD,QAAQ,EAAE;YACN,cAAc,EAAE,4DAA4D;YAC5E,WAAW,EAAE,gEAAgE;SAChF;KACJ;IACD,MAAM,CAAC,OAAO;QACV,OAAO;YACH,gBAAgB,CAAC,IAAI;gBACjB,IAAI,gBAAgB,GAAG,KAAK,CAAC;gBAC7B,IAAI,cAAc,GAAG,KAAK,CAAC;gBAC3B,IAAI,iBAAiB,GAAG,KAAK,CAAC;gBAE9B,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,gBAAqB,EAAE,EAAE;;oBAC7C,IAAI,gBAAgB,CAAC,UAAU,EAAE;wBAC7B,gBAAgB,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;;4BACnD,IACI,cAAc,CAAC,QAAQ,CACnB,CAAA,MAAA,SAAS,aAAT,SAAS,uBAAT,SAAS,CAAE,UAAU,0CAAE,IAAI;iCACvB,MAAA,MAAA,SAAS,aAAT,SAAS,uBAAT,SAAS,CAAE,UAAU,0CAAE,MAAM,0CAAE,IAAI,CAAA,CAC1C,EACH;gCACE,gBAAgB,GAAG,IAAI,CAAC;6BAC3B;wBACL,CAAC,CAAC,CAAC;qBACN;oBAED,IAAI,gBAAgB,CAAC,IAAI,KAAK,aAAa,EAAE;wBACzC,cAAc,GAAG,IAAI,CAAC;wBAEtB,MAAA,MAAA,gBAAgB,CAAC,KAAK,0CAAE,IAAI,0CAAE,IAAI,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;;4BAC1D,IAAI,CAAA,MAAA,MAAA,SAAS,aAAT,SAAS,uBAAT,SAAS,CAAE,UAAU,0CAAE,MAAM,0CAAE,IAAI,MAAK,gBAAgB,EAAE;gCAC1D,iBAAiB,GAAG,IAAI,CAAC;6BAC5B;wBACL,CAAC,CAAC,CAAC;qBACN;gBACL,CAAC,CAAC,CAAC;gBAEH,IAAI,gBAAgB,IAAI,CAAC,iBAAiB,IAAI,cAAc,EAAE;oBAC1D,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI;wBACJ,SAAS,EAAE,gBAAgB;qBAC9B,CAAC,CAAC;iBACN;gBAED,IAAI,gBAAgB,IAAI,CAAC,iBAAiB,IAAI,CAAC,cAAc,EAAE;oBAC3D,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI;wBACJ,SAAS,EAAE,aAAa;qBAC3B,CAAC,CAAC;iBACN;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/react/destructure-default-import.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from 'eslint';
+export declare const reactDestructureDefaultImport: Rule.RuleModule;
+//# sourceMappingURL=destructure-default-import.d.ts.map

package/dist/rules/react/destructure-default-import.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"destructure-default-import.d.ts","sourceRoot":"","sources":["../../../src/rules/react/destructure-default-import.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAE9B,eAAO,MAAM,6BAA6B,EAAE,IAAI,CAAC,UAsChD,CAAC"}

package/dist/rules/react/destructure-default-import.js

@@ -0,0 +1,40 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.reactDestructureDefaultImport = void 0;
+exports.reactDestructureDefaultImport = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: "Disallow importing React's default export",
+            category: 'Best Practices',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            importDefault: 'Avoid importing React default export',
+        },
+    },
+    create(context) {
+        return {
+            ImportDeclaration(node) {
+                const reactImportSpecifiers = new Set();
+                if (node.source.value !== 'react') {
+                    return;
+                }
+                node.specifiers.forEach(specifier => {
+                    if (specifier.type === 'ImportDefaultSpecifier' ||
+                        specifier.type === 'ImportNamespaceSpecifier') {
+                        context.report({
+                            node,
+                            messageId: 'importDefault',
+                        });
+                    }
+                    else {
+                        reactImportSpecifiers.add(specifier.imported);
+                    }
+                });
+            },
+        };
+    },
+};
+//# sourceMappingURL=destructure-default-import.js.map

package/dist/rules/react/destructure-default-import.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"destructure-default-import.js","sourceRoot":"","sources":["../../../src/rules/react/destructure-default-import.ts"],"names":[],"mappings":";;;AAEa,QAAA,6BAA6B,GAAoB;IAC1D,IAAI,EAAE;QACF,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACF,WAAW,EAAE,2CAA2C;YACxD,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,2CAA2C;SACnD;QACD,QAAQ,EAAE;YACN,aAAa,EAAE,sCAAsC;SACxD;KACJ;IACD,MAAM,CAAC,OAAO;QACV,OAAO;YACH,iBAAiB,CAAC,IAAI;gBAClB,MAAM,qBAAqB,GAAG,IAAI,GAAG,EAAE,CAAC;gBAExC,IAAI,IAAI,CAAC,MAAM,CAAC,KAAK,KAAK,OAAO,EAAE;oBAC/B,OAAO;iBACV;gBAED,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE;oBAChC,IACI,SAAS,CAAC,IAAI,KAAK,wBAAwB;wBAC3C,SAAS,CAAC,IAAI,KAAK,0BAA0B,EAC/C;wBACE,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI;4BACJ,SAAS,EAAE,eAAe;yBAC7B,CAAC,CAAC;qBACN;yBAAM;wBACH,qBAAqB,CAAC,GAAG,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;qBACjD;gBACL,CAAC,CAAC,CAAC;YACP,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/react/no-qualified-type.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from 'eslint';
+export declare const reactNoQualifiedType: Rule.RuleModule;
+//# sourceMappingURL=no-qualified-type.d.ts.map

package/dist/rules/react/no-qualified-type.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-qualified-type.d.ts","sourceRoot":"","sources":["../../../src/rules/react/no-qualified-type.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAE9B,eAAO,MAAM,oBAAoB,EAAE,IAAI,CAAC,UA4BvC,CAAC"}

package/dist/rules/react/no-qualified-type.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.reactNoQualifiedType = void 0;
+exports.reactNoQualifiedType = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow using React types from default export',
+            category: 'Best Practices',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            importDefault: "Import {{right}} from 'react' instead of using React.{{right}}",
+        },
+    },
+    create(context) {
+        return {
+            TSQualifiedName(node) {
+                if (node.left && node.left.name === 'React') {
+                    context.report({
+                        node,
+                        messageId: 'importDefault',
+                        data: {
+                            right: node.right.name,
+                        },
+                    });
+                }
+            },
+        };
+    },
+};
+//# sourceMappingURL=no-qualified-type.js.map

package/dist/rules/react/no-qualified-type.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-qualified-type.js","sourceRoot":"","sources":["../../../src/rules/react/no-qualified-type.ts"],"names":[],"mappings":";;;AAEa,QAAA,oBAAoB,GAAoB;IACjD,IAAI,EAAE;QACF,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACF,WAAW,EAAE,gDAAgD;YAC7D,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,2CAA2C;SACnD;QACD,QAAQ,EAAE;YACN,aAAa,EAAE,gEAAgE;SAClF;KACJ;IACD,MAAM,CAAC,OAAO;QACV,OAAO;YACH,eAAe,CAAC,IAAS;gBACrB,IAAI,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE;oBACzC,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI;wBACJ,SAAS,EAAE,eAAe;wBAC1B,IAAI,EAAE;4BACF,KAAK,EAAE,IAAI,CAAC,KAAK,CAAC,IAAI;yBACzB;qBACJ,CAAC,CAAC;iBACN;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/package.json

@@ -0,0 +1,30 @@
+{
+    "name": "@servicetitan/eslint-plugin",
+    "version": "16.0.0",
+    "description": "",
+    "repository": {
+        "type": "git",
+        "url": "https://github.com/servicetitan/uikit.git",
+        "directory": "packages/eslint-plugin"
+    },
+    "main": "./dist/index.js",
+    "typings": "./dist/index.d.ts",
+    "files": [
+        "dist",
+        "src"
+    ],
+    "devDependencies": {
+        "@types/eslint": "~7.28.0",
+        "eslint": "~7.31.0"
+    },
+    "peerDependencies": {
+        "eslint": "~7.31.0"
+    },
+    "publishConfig": {
+        "access": "public"
+    },
+    "cli": {
+        "webpack": false
+    },
+    "gitHead": "d2d6e21f4f639bcde5a9ab274a6fd47b14b26974"
+}

package/src/index.ts

@@ -0,0 +1,16 @@
+import { useDeclareWithDecorators } from './rules/decorators-declare';
+import { mobxUseMakeObservableWithDecorators } from './rules/mobx/use-makeObservable-with-decorators';
+import { mobxNoAbstractDecorators } from './rules/mobx/no-abstract-decorators';
+import { reactDestructureDefaultImport } from './rules/react/destructure-default-import';
+import { reactNoQualifiedType } from './rules/react/no-qualified-type';
+import { stub } from './processors/stub';
+
+export const rules = {
+    'use-declare-with-decorators': useDeclareWithDecorators,
+    'mobx/use-makeObservable-with-decorators': mobxUseMakeObservableWithDecorators,
+    'mobx/no-abstract-decorators': mobxNoAbstractDecorators,
+    'react/destructure-default-import': reactDestructureDefaultImport,
+    'react/no-qualified-type': reactNoQualifiedType,
+};
+
+export const processors = { stub };

package/src/processors/stub.ts

@@ -0,0 +1,6 @@
+import { Linter } from 'eslint';
+
+export const stub: Linter.LintOptions = {
+    preprocess: () => [''],
+    postprocess: ([messages]) => messages,
+};

package/src/rules/decorators-declare.ts

@@ -0,0 +1,43 @@
+import { Rule } from 'eslint';
+
+const customDecorators = ['injectDependency', 'koObservableToMobx'];
+
+export const useDeclareWithDecorators: Rule.RuleModule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow custom @servicetitan class field decorators without declare',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            decoratorsDeclare: "Use declare when decorating with '@{{name}}'",
+        },
+    },
+    create(context) {
+        return {
+            ClassProperty(node: any) {
+                if (node.declare) {
+                    return; // field is already defined with declare
+                }
+
+                if (node?.decorators?.length) {
+                    node.decorators.forEach((decorator: any) => {
+                        const name = decorator.expression?.callee?.name;
+
+                        if (name && customDecorators.includes(name)) {
+                            context.report({
+                                node,
+                                messageId: 'decoratorsDeclare',
+                                data: {
+                                    name,
+                                },
+                            });
+                        }
+                    });
+                }
+            },
+        };
+    },
+};

package/src/rules/mobx/no-abstract-decorators.ts

@@ -0,0 +1,43 @@
+import { Rule } from 'eslint';
+
+const mobxDecorators = ['observable', 'action', 'computed', 'override'];
+
+export const mobxNoAbstractDecorators: Rule.RuleModule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow MobX decorators on abstract class fields',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            noAbstractMobxDecorators:
+                "Don't decorate abstract class fields or methods with '@{{name}}'",
+        },
+    },
+    create(context: any) {
+        function checkFn(node: any) {
+            if (node?.decorators?.length) {
+                node.decorators.forEach((decorator: any) => {
+                    const name = decorator?.expression?.name || decorator?.expression?.object?.name;
+
+                    if (name && mobxDecorators.includes(name)) {
+                        context.report({
+                            node,
+                            messageId: 'noAbstractMobxDecorators',
+                            data: {
+                                name,
+                            },
+                        });
+                    }
+                });
+            }
+        }
+
+        return {
+            TSAbstractClassProperty: checkFn,
+            TSAbstractMethodDefinition: checkFn,
+        };
+    },
+};

package/src/rules/mobx/use-makeObservable-with-decorators.ts

@@ -0,0 +1,68 @@
+import { Rule } from 'eslint';
+
+// NOTE: "action" also covers "action.bound"
+const mobxDecorators = ['observable', 'action', 'computed', 'override'];
+
+export const mobxUseMakeObservableWithDecorators: Rule.RuleModule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Testing makeObservable used with decorators',
+            category: 'Possible Errors',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            makeObservable: 'Use makeObservable in the constructor with MobX decorators',
+            constructor: 'Add constructor with makeObservable when using MobX decorators',
+        },
+    },
+    create(context) {
+        return {
+            ClassDeclaration(node) {
+                let hasMobxDecorator = false;
+                let hasConstructor = false;
+                let hasMakeObservable = false;
+
+                node.body.body.forEach((methodOrProperty: any) => {
+                    if (methodOrProperty.decorators) {
+                        methodOrProperty.decorators.forEach((decorator: any) => {
+                            if (
+                                mobxDecorators.includes(
+                                    decorator?.expression?.name ||
+                                        decorator?.expression?.object?.name
+                                )
+                            ) {
+                                hasMobxDecorator = true;
+                            }
+                        });
+                    }
+
+                    if (methodOrProperty.kind === 'constructor') {
+                        hasConstructor = true;
+
+                        methodOrProperty.value?.body?.body.forEach((statement: any) => {
+                            if (statement?.expression?.callee?.name === 'makeObservable') {
+                                hasMakeObservable = true;
+                            }
+                        });
+                    }
+                });
+
+                if (hasMobxDecorator && !hasMakeObservable && hasConstructor) {
+                    context.report({
+                        node,
+                        messageId: 'makeObservable',
+                    });
+                }
+
+                if (hasMobxDecorator && !hasMakeObservable && !hasConstructor) {
+                    context.report({
+                        node,
+                        messageId: 'constructor',
+                    });
+                }
+            },
+        };
+    },
+};

package/src/rules/react/destructure-default-import.ts

@@ -0,0 +1,41 @@
+import { Rule } from 'eslint';
+
+export const reactDestructureDefaultImport: Rule.RuleModule = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: "Disallow importing React's default export",
+            category: 'Best Practices',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            importDefault: 'Avoid importing React default export',
+        },
+    },
+    create(context) {
+        return {
+            ImportDeclaration(node) {
+                const reactImportSpecifiers = new Set();
+
+                if (node.source.value !== 'react') {
+                    return;
+                }
+
+                node.specifiers.forEach(specifier => {
+                    if (
+                        specifier.type === 'ImportDefaultSpecifier' ||
+                        specifier.type === 'ImportNamespaceSpecifier'
+                    ) {
+                        context.report({
+                            node,
+                            messageId: 'importDefault',
+                        });
+                    } else {
+                        reactImportSpecifiers.add(specifier.imported);
+                    }
+                });
+            },
+        };
+    },
+};

package/src/rules/react/no-qualified-type.ts

@@ -0,0 +1,31 @@
+import { Rule } from 'eslint';
+
+export const reactNoQualifiedType: Rule.RuleModule = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow using React types from default export',
+            category: 'Best Practices',
+            recommended: false,
+            url: 'https://github.com/servicetitan/uikit.git',
+        },
+        messages: {
+            importDefault: "Import {{right}} from 'react' instead of using React.{{right}}",
+        },
+    },
+    create(context) {
+        return {
+            TSQualifiedName(node: any) {
+                if (node.left && node.left.name === 'React') {
+                    context.report({
+                        node,
+                        messageId: 'importDefault',
+                        data: {
+                            right: node.right.name,
+                        },
+                    });
+                }
+            },
+        };
+    },
+};