@gem-sdk/eslint-plugin 0.0.3 → 0.0.4

package/README.md

@@ -54,8 +54,10 @@ Choose the config that matches your tech stack:
     "@gem-sdk/no-t-for-variable": "error",
     "@gem-sdk/not-use-store-to-refs": "error",
     "@gem-sdk/not-use-custom-color-class": "warn",
+    "@gem-sdk/no-export-type-from-component": "error",
     "@gem-sdk/no-export-type-from-tsx": "error",
-    "@gem-sdk/no-import-from-package-path": "error"
+    "@gem-sdk/no-import-from-package-path": "error",
+    "@gem-sdk/no-multiple-component-exports": "error"
   }
 }
 ```
@@ -72,7 +74,11 @@ Choose the config that matches your tech stack:
 | `no-t-for-variable` | error | error | error |
 | `not-use-store-to-refs` | — | error | — |
 | `not-use-custom-color-class` | — | warn | — |
+| `no-export-type-from-component` | — | error | — |
 | `no-export-type-from-tsx` | — | — | error |
 | `no-import-from-package-path` | — | — | error |
+| `no-multiple-component-exports` | — | — | error |
 
 `vue` and `react` both extend `recommended`. Vue configs require `vue-eslint-parser`.
+
+`no-export-type-from-component` and `no-export-type-from-tsx` enforce the same thing (no `export type`/`export interface` from a component file) — the former also matches `.vue` files, the latter is React-only. They're kept separate so each config only reports once per file.

package/dist/index.d.ts

@@ -5,10 +5,12 @@ export declare const rules: {
     'pure-ui-layer': import("eslint").Rule.RuleModule;
     'module-index-required': import("eslint").Rule.RuleModule;
     'no-t-for-variable': import("eslint").Rule.RuleModule;
+    'no-export-type-from-component': import("eslint").Rule.RuleModule;
     'not-use-store-to-refs': import("eslint").Rule.RuleModule;
     'not-use-custom-color-class': import("eslint").Rule.RuleModule;
     'no-export-type-from-tsx': import("eslint").Rule.RuleModule;
     'no-import-from-package-path': import("eslint").Rule.RuleModule;
+    'no-multiple-component-exports': import("eslint").Rule.RuleModule;
 };
 export declare const configs: {
     recommended: {
@@ -33,6 +35,7 @@ export declare const configs: {
             readonly '@gem-sdk/no-t-for-variable': 'error';
             readonly '@gem-sdk/not-use-store-to-refs': 'error';
             readonly '@gem-sdk/not-use-custom-color-class': 'warn';
+            readonly '@gem-sdk/no-export-type-from-component': 'error';
         };
     };
     react: {
@@ -46,6 +49,7 @@ export declare const configs: {
             readonly '@gem-sdk/no-t-for-variable': 'error';
             readonly '@gem-sdk/no-export-type-from-tsx': 'error';
             readonly '@gem-sdk/no-import-from-package-path': 'error';
+            readonly '@gem-sdk/no-multiple-component-exports': 'error';
         };
     };
 };

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAYA,eAAO,MAAM,KAAK;;;;;;;;;;;CAegB,CAAC;AAyCnC,eAAO,MAAM,OAAO;;oCAnCR,UAAU;;qBAElB,2BAA2B,EAAE,OAAO;qBACpC,2BAA2B,EAAE,OAAO;qBACpC,iCAAiC,EAAE,OAAO;qBAC1C,wBAAwB,EAAE,OAAO;qBACjC,gCAAgC,EAAE,OAAO;qBACzC,4BAA4B,EAAE,OAAO;;;;oCAQ7B,UAAU;;kDAbW,OAAO;kDACP,OAAO;wDACD,OAAO;+CAChB,OAAO;uDACC,OAAO;mDACX,OAAO;uDAWH,OAAO;4DACF,MAAM;;;;oCAQrC,UAAU;;kDAzBW,OAAO;kDACP,OAAO;wDACD,OAAO;+CAChB,OAAO;uDACC,OAAO;mDACX,OAAO;yDAuBD,OAAO;6DACH,OAAO;;;CAID,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAcA,eAAO,MAAM,KAAK;;;;;;;;;;;;;CAkBgB,CAAC;AA2CnC,eAAO,MAAM,OAAO;;oCArCR,UAAU;;qBAElB,2BAA2B,EAAE,OAAO;qBACpC,2BAA2B,EAAE,OAAO;qBACpC,iCAAiC,EAAE,OAAO;qBAC1C,wBAAwB,EAAE,OAAO;qBACjC,gCAAgC,EAAE,OAAO;qBACzC,4BAA4B,EAAE,OAAO;;;;oCAQ7B,UAAU;;kDAbW,OAAO;kDACP,OAAO;wDACD,OAAO;+CAChB,OAAO;uDACC,OAAO;mDACX,OAAO;uDAWH,OAAO;4DACF,MAAM;+DACH,OAAO;;;;oCAQzC,UAAU;;kDA1BW,OAAO;kDACP,OAAO;wDACD,OAAO;+CAChB,OAAO;uDACC,OAAO;mDACX,OAAO;yDAwBD,OAAO;6DACH,OAAO;+DACL,OAAO;;;CAIH,CAAC"}

package/dist/index.js

@@ -7,9 +7,11 @@ exports.configs = exports.rules = void 0;
 const moduleImportBoundary_1 = __importDefault(require("./rules/moduleImportBoundary"));
 const moduleIndexRequired_1 = __importDefault(require("./rules/moduleIndexRequired"));
 const moduleStructure_1 = __importDefault(require("./rules/moduleStructure"));
+const noExportTypeFromComponent_1 = __importDefault(require("./rules/noExportTypeFromComponent"));
 const noExportTypeFromTsx_1 = __importDefault(require("./rules/noExportTypeFromTsx"));
 const noImportFromPackagePath_1 = __importDefault(require("./rules/noImportFromPackagePath"));
 const noModuleBarrel_1 = __importDefault(require("./rules/noModuleBarrel"));
+const noMultipleComponentExports_1 = __importDefault(require("./rules/noMultipleComponentExports"));
 const noTForVariable_1 = __importDefault(require("./rules/noTForVariable"));
 const notUseCustomColorClass_1 = __importDefault(require("./rules/notUseCustomColorClass"));
 const notUseStoreToRefs_1 = __importDefault(require("./rules/notUseStoreToRefs"));
@@ -23,12 +25,15 @@ exports.rules = {
     'module-index-required': moduleIndexRequired_1.default,
     // i18n
     'no-t-for-variable': noTForVariable_1.default,
+    // Shared component rules — target both .tsx and .vue
+    'no-export-type-from-component': noExportTypeFromComponent_1.default,
     // Vue-specific
     'not-use-store-to-refs': notUseStoreToRefs_1.default,
     'not-use-custom-color-class': notUseCustomColorClass_1.default,
     // React/TypeScript-specific
     'no-export-type-from-tsx': noExportTypeFromTsx_1.default,
     'no-import-from-package-path': noImportFromPackagePath_1.default,
+    'no-multiple-component-exports': noMultipleComponentExports_1.default,
 };
 /**
  * Shared architecture + i18n rules for both React and Vue repos.
@@ -53,6 +58,7 @@ const vue = {
         ...recommended.rules,
         '@gem-sdk/not-use-store-to-refs': 'error',
         '@gem-sdk/not-use-custom-color-class': 'warn',
+        '@gem-sdk/no-export-type-from-component': 'error',
     },
 };
 /**
@@ -64,6 +70,7 @@ const react = {
         ...recommended.rules,
         '@gem-sdk/no-export-type-from-tsx': 'error',
         '@gem-sdk/no-import-from-package-path': 'error',
+        '@gem-sdk/no-multiple-component-exports': 'error',
     },
 };
 exports.configs = { recommended, vue, react };

package/dist/rules/noExportTypeFromComponent.d.ts

@@ -0,0 +1,8 @@
+import type { Rule } from 'eslint';
+/**
+ * Disallows exporting types and interfaces from component files (.tsx and .vue).
+ * Types should live in a dedicated .ts file.
+ */
+declare const rule: Rule.RuleModule;
+export default rule;
+//# sourceMappingURL=noExportTypeFromComponent.d.ts.map

package/dist/rules/noExportTypeFromComponent.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"noExportTypeFromComponent.d.ts","sourceRoot":"","sources":["../../src/rules/noExportTypeFromComponent.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAInC;;;GAGG;AACH,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UAwBhB,CAAC;eAEa,IAAI"}

package/dist/rules/noExportTypeFromComponent.js

@@ -0,0 +1,31 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const COMPONENT_EXTENSIONS = ['.tsx', '.vue'];
+/**
+ * Disallows exporting types and interfaces from component files (.tsx and .vue).
+ * Types should live in a dedicated .ts file.
+ */
+const rule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow export type and interface declarations from .tsx and .vue files',
+        },
+        messages: {
+            noTypeExport: 'Exporting types and interfaces from .tsx and .vue files is not allowed. Move them to a .ts file.',
+        },
+        schema: [],
+    },
+    create(context) {
+        return {
+            ExportNamedDeclaration(node) {
+                const fileName = context.getFilename();
+                if (COMPONENT_EXTENSIONS.some((ext) => fileName.endsWith(ext)) &&
+                    node.exportKind === 'type') {
+                    context.report({ node, messageId: 'noTypeExport' });
+                }
+            },
+        };
+    },
+};
+exports.default = rule;

package/dist/rules/noMultipleComponentExports.d.ts

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

package/dist/rules/noMultipleComponentExports.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"noMultipleComponentExports.d.ts","sourceRoot":"","sources":["../../src/rules/noMultipleComponentExports.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AA8EnC,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UA+BhB,CAAC;eAEa,IAAI"}

package/dist/rules/noMultipleComponentExports.js

@@ -0,0 +1,102 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const PASCAL_CASE = /^[A-Z][A-Za-z0-9]*$/;
+function isComponentInit(node) {
+    return !!node && (node.type === 'ArrowFunctionExpression' || node.type === 'FunctionExpression');
+}
+function findLocalComponent(body, name) {
+    for (const statement of body) {
+        if (statement.type === 'VariableDeclaration') {
+            for (const declarator of statement.declarations) {
+                if (declarator.id.type === 'Identifier' && declarator.id.name === name && isComponentInit(declarator.init)) {
+                    return { name, node: declarator };
+                }
+            }
+        }
+        if (statement.type === 'FunctionDeclaration' && statement.id?.name === name) {
+            return { name, node: statement };
+        }
+    }
+    return undefined;
+}
+/**
+ * Detects components exported at the top level of a file, covering `export const X = () => {}`,
+ * `export function X() {}`, `export default`, and `export { X }` re-exports of local declarations.
+ */
+function collectExportedComponents(program) {
+    const components = [];
+    for (const statement of program.body) {
+        if (statement.type === 'ExportNamedDeclaration') {
+            const declaration = statement.declaration;
+            if (declaration?.type === 'VariableDeclaration') {
+                for (const declarator of declaration.declarations) {
+                    if (declarator.id.type === 'Identifier' &&
+                        PASCAL_CASE.test(declarator.id.name) &&
+                        isComponentInit(declarator.init)) {
+                        components.push({ name: declarator.id.name, node: declarator });
+                    }
+                }
+            }
+            else if (declaration?.type === 'FunctionDeclaration' && declaration.id && PASCAL_CASE.test(declaration.id.name)) {
+                components.push({ name: declaration.id.name, node: declaration });
+            }
+            for (const specifier of statement.specifiers) {
+                if (specifier.exported.type !== 'Identifier' || specifier.local.type !== 'Identifier')
+                    continue;
+                const exportedName = specifier.exported.name;
+                if (!PASCAL_CASE.test(exportedName))
+                    continue;
+                const local = findLocalComponent(program.body, specifier.local.name);
+                if (local)
+                    components.push({ name: exportedName, node: specifier });
+            }
+        }
+        if (statement.type === 'ExportDefaultDeclaration') {
+            const declaration = statement.declaration;
+            if (declaration.type === 'FunctionDeclaration' || declaration.type === 'ArrowFunctionExpression' || declaration.type === 'FunctionExpression') {
+                const name = declaration.type === 'FunctionDeclaration' ? declaration.id?.name : undefined;
+                if (!name || PASCAL_CASE.test(name)) {
+                    components.push({ name: name ?? 'default', node: declaration });
+                }
+            }
+            else if (declaration.type === 'Identifier') {
+                const local = findLocalComponent(program.body, declaration.name);
+                if (local)
+                    components.push(local);
+            }
+        }
+    }
+    return components;
+}
+const rule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow defining and exporting more than one component from a single file',
+        },
+        messages: {
+            multipleComponents: 'Only one component should be defined and exported per file. "{{name}}" is an additional exported component — move it to its own file.',
+        },
+        schema: [],
+    },
+    create(context) {
+        return {
+            Program(node) {
+                const filename = context.getFilename();
+                if (!filename.endsWith('.tsx'))
+                    return;
+                const components = collectExportedComponents(node);
+                if (components.length <= 1)
+                    return;
+                for (const component of components.slice(1)) {
+                    context.report({
+                        node: component.node,
+                        messageId: 'multipleComponents',
+                        data: { name: component.name },
+                    });
+                }
+            },
+        };
+    },
+};
+exports.default = rule;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gem-sdk/eslint-plugin",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "Shared ESLint rules for GemPages frontend repos (React + Vue)",
   "license": "MIT",
   "main": "./dist/index.js",