@memberjunction/react-linter 0.0.1 → 5.38.0

package/dist/runtime-rules/library-variable-names.js

@@ -0,0 +1,88 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import { traverse } from '../lint-utils.js';
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: library-variable-names
+ *
+ * Ensures correct usage of library global variables. Libraries must be accessed
+ * using unwrapComponents with the correct global variable name as specified in
+ * the component spec.
+ *
+ * Prevents:
+ * - Using incorrect library global variable names
+ * - Self-assignment of library globals (const chroma = chroma)
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let LibraryVariableNamesRule = class LibraryVariableNamesRule extends BaseLintRule {
+    get Name() { return 'library-variable-names'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName, componentSpec) {
+        const violations = [];
+        // Build a map of library names to their globalVariables
+        const libraryGlobals = new Map();
+        if (componentSpec?.libraries) {
+            for (const lib of componentSpec.libraries) {
+                // Skip empty library objects or those without required fields
+                if (lib.name && lib.globalVariable) {
+                    // Store both the exact name and lowercase for comparison
+                    libraryGlobals.set(lib.name.toLowerCase(), lib.globalVariable);
+                }
+            }
+        }
+        traverse(ast, {
+            VariableDeclarator(path) {
+                // Check for destructuring from a variable (library global)
+                if (t.isObjectPattern(path.node.id) && t.isIdentifier(path.node.init)) {
+                    const sourceVar = path.node.init.name;
+                    // Check if this looks like a library name (case-insensitive match)
+                    const matchedLib = Array.from(libraryGlobals.entries()).find(([libName, globalVar]) => sourceVar.toLowerCase() === libName || sourceVar.toLowerCase() === globalVar.toLowerCase());
+                    if (matchedLib) {
+                        const [, correctGlobal] = matchedLib;
+                        if (sourceVar !== correctGlobal) {
+                            violations.push({
+                                rule: 'library-variable-names',
+                                severity: 'critical',
+                                line: path.node.loc?.start.line || 0,
+                                column: path.node.loc?.start.column || 0,
+                                message: `Incorrect library global variable "${sourceVar}". Use unwrapComponents with the correct global: "const { ... } = unwrapComponents(${correctGlobal}, [...]);"`,
+                            });
+                        }
+                    }
+                }
+                // Check for self-assignment (const chroma = chroma)
+                if (t.isIdentifier(path.node.id) && t.isIdentifier(path.node.init)) {
+                    const idName = path.node.id.name;
+                    const initName = path.node.init.name;
+                    if (idName === initName) {
+                        // Check if this is a library global
+                        const isLibraryGlobal = Array.from(libraryGlobals.values()).some((global) => global === idName);
+                        if (isLibraryGlobal) {
+                            violations.push({
+                                rule: 'library-variable-names',
+                                severity: 'critical',
+                                line: path.node.loc?.start.line || 0,
+                                column: path.node.loc?.start.column || 0,
+                                message: `Self-assignment of library global "${idName}". This variable is already available as a global from the library. Remove this line entirely - the library global is already accessible.`,
+                            });
+                        }
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+LibraryVariableNamesRule = __decorate([
+    RegisterClass(BaseLintRule, 'library-variable-names')
+], LibraryVariableNamesRule);
+export { LibraryVariableNamesRule };
+//# sourceMappingURL=library-variable-names.js.map

package/dist/runtime-rules/library-variable-names.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"library-variable-names.js","sourceRoot":"","sources":["../../src/runtime-rules/library-variable-names.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,EAAE,QAAQ,EAAY,MAAM,eAAe,CAAC;AACnD,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAClC,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAI5C;;;;;;;;;;;;;GAaG;AAEI,IAAM,wBAAwB,GAA9B,MAAM,wBAAyB,SAAQ,YAAY;IACxD,IAAI,IAAI,KAAK,OAAO,wBAAwB,CAAC,CAAC,CAAC;IAC/C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,cAAsB,EAAE,aAA6B;QACrE,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,wDAAwD;QACxD,MAAM,cAAc,GAAG,IAAI,GAAG,EAAkB,CAAC;QACjD,IAAI,aAAa,EAAE,SAAS,EAAE,CAAC;YAC7B,KAAK,MAAM,GAAG,IAAI,aAAa,CAAC,SAAS,EAAE,CAAC;gBAC1C,8DAA8D;gBAC9D,IAAI,GAAG,CAAC,IAAI,IAAI,GAAG,CAAC,cAAc,EAAE,CAAC;oBACnC,yDAAyD;oBACzD,cAAc,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,EAAE,GAAG,CAAC,cAAc,CAAC,CAAC;gBACjE,CAAC;YACH,CAAC;QACH,CAAC;QAED,QAAQ,CAAC,GAAG,EAAE;YACZ,kBAAkB,CAAC,IAAoC;gBACrD,2DAA2D;gBAC3D,IAAI,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBACtE,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;oBAEtC,mEAAmE;oBACnE,MAAM,UAAU,GAAG,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,OAAO,EAAE,CAAC,CAAC,IAAI,CAC1D,CAAC,CAAC,OAAO,EAAE,SAAS,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,WAAW,EAAE,KAAK,OAAO,IAAI,SAAS,CAAC,WAAW,EAAE,KAAK,SAAS,CAAC,WAAW,EAAE,CACrH,CAAC;oBAEF,IAAI,UAAU,EAAE,CAAC;wBACf,MAAM,CAAC,EAAE,aAAa,CAAC,GAAG,UAAU,CAAC;wBACrC,IAAI,SAAS,KAAK,aAAa,EAAE,CAAC;4BAChC,UAAU,CAAC,IAAI,CAAC;gCACd,IAAI,EAAE,wBAAwB;gCAC9B,QAAQ,EAAE,UAAU;gCACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;gCACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;gCACxC,OAAO,EAAE,sCAAsC,SAAS,sFAAsF,aAAa,YAAY;6BACxK,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;gBAED,oDAAoD;gBACpD,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBACnE,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC;oBACjC,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;oBAErC,IAAI,MAAM,KAAK,QAAQ,EAAE,CAAC;wBACxB,oCAAoC;wBACpC,MAAM,eAAe,GAAG,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC;wBAEhG,IAAI,eAAe,EAAE,CAAC;4BACpB,UAAU,CAAC,IAAI,CAAC;gCACd,IAAI,EAAE,wBAAwB;gCAC9B,QAAQ,EAAE,UAAU;gCACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;gCACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;gCACxC,OAAO,EAAE,sCAAsC,MAAM,2IAA2I;6BACjM,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AArEY,wBAAwB;IADpC,aAAa,CAAC,YAAY,EAAE,wBAAwB,CAAC;GACzC,wBAAwB,CAqEpC"}

package/dist/runtime-rules/no-child-implementation.d.ts

@@ -0,0 +1,18 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: no-child-implementation
+ *
+ * Ensures that root component files do not contain child component implementations.
+ * Root components should only reference child components, not implement them.
+ *
+ * Severity: critical
+ * Applies to: root components only
+ */
+export declare class NoChildImplementationRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-child-implementation.d.ts.map

package/dist/runtime-rules/no-child-implementation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-child-implementation.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-child-implementation.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAEhD;;;;;;;;GAQG;AACH,qBACa,yBAA0B,SAAQ,YAAY;IACzD,IAAI,IAAI,WAAwC;IAChD,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAmB;IAE5D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAkCtD"}

package/dist/runtime-rules/no-child-implementation.js

@@ -0,0 +1,57 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import { traverse } from '../lint-utils.js';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: no-child-implementation
+ *
+ * Ensures that root component files do not contain child component implementations.
+ * Root components should only reference child components, not implement them.
+ *
+ * Severity: critical
+ * Applies to: root components only
+ */
+let NoChildImplementationRule = class NoChildImplementationRule extends BaseLintRule {
+    get Name() { return 'no-child-implementation'; }
+    get AppliesTo() { return 'root'; }
+    Test(ast, componentName) {
+        const violations = [];
+        const rootFunctionName = componentName;
+        const declaredFunctions = [];
+        // First pass: collect all function declarations
+        traverse(ast, {
+            FunctionDeclaration(path) {
+                if (path.node.id) {
+                    declaredFunctions.push(path.node.id.name);
+                }
+            },
+        });
+        // If there are multiple function declarations and they look like components
+        // (start with capital letter), it's likely implementing children
+        const componentFunctions = declaredFunctions.filter((name) => name !== rootFunctionName && /^[A-Z]/.test(name));
+        if (componentFunctions.length > 0) {
+            violations.push({
+                rule: 'no-child-implementation',
+                severity: 'critical',
+                line: 1,
+                column: 0,
+                message: `Root component file contains child component implementations: ${componentFunctions.join(', ')}. Root should only reference child components, not implement them.`,
+                suggestion: {
+                    text: 'Remove child component implementations. Only the root component function should be in this file',
+                    example: 'Move child component functions to separate generation requests',
+                },
+            });
+        }
+        return violations;
+    }
+};
+NoChildImplementationRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-child-implementation')
+], NoChildImplementationRule);
+export { NoChildImplementationRule };
+//# sourceMappingURL=no-child-implementation.js.map

package/dist/runtime-rules/no-child-implementation.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-child-implementation.js","sourceRoot":"","sources":["../../src/runtime-rules/no-child-implementation.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,EAAE,QAAQ,EAAY,MAAM,eAAe,CAAC;AAEnD,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAG5C;;;;;;;;GAQG;AAEI,IAAM,yBAAyB,GAA/B,MAAM,yBAA0B,SAAQ,YAAY;IACzD,IAAI,IAAI,KAAK,OAAO,yBAAyB,CAAC,CAAC,CAAC;IAChD,IAAI,SAAS,KAA+B,OAAO,MAAM,CAAC,CAAC,CAAC;IAE5D,IAAI,CAAC,GAAW,EAAE,aAAqB;QACrC,MAAM,UAAU,GAAgB,EAAE,CAAC;QACnC,MAAM,gBAAgB,GAAG,aAAa,CAAC;QACvC,MAAM,iBAAiB,GAAa,EAAE,CAAC;QAEvC,gDAAgD;QAChD,QAAQ,CAAC,GAAG,EAAE;YACZ,mBAAmB,CAAC,IAAqC;gBACvD,IAAI,IAAI,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC;oBACjB,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC;gBAC5C,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,4EAA4E;QAC5E,iEAAiE;QACjE,MAAM,kBAAkB,GAAG,iBAAiB,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,KAAK,gBAAgB,IAAI,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAEhH,IAAI,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClC,UAAU,CAAC,IAAI,CAAC;gBACd,IAAI,EAAE,yBAAyB;gBAC/B,QAAQ,EAAE,UAAU;gBACpB,IAAI,EAAE,CAAC;gBACP,MAAM,EAAE,CAAC;gBACT,OAAO,EAAE,iEAAiE,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,oEAAoE;gBAC3K,UAAU,EAAE;oBACV,IAAI,EAAE,iGAAiG;oBACvG,OAAO,EAAE,gEAAgE;iBAC1E;aACF,CAAC,CAAC;QACL,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAtCY,yBAAyB;IADrC,aAAa,CAAC,YAAY,EAAE,yBAAyB,CAAC;GAC1C,yBAAyB,CAsCrC"}

package/dist/runtime-rules/no-data-prop.d.ts

@@ -0,0 +1,22 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+import { ComponentSpec } from '@memberjunction/interactive-component-types';
+/**
+ * Rule: no-data-prop
+ *
+ * Discourages the use of generic 'data' prop names in favor of more specific
+ * names like 'items', 'customers', etc. This improves code clarity and makes
+ * the component's purpose more explicit.
+ *
+ * Skips chart/visualization components that legitimately need generic data props.
+ *
+ * Severity: low (opinion-based style preference, not a functional issue)
+ * Applies to: all components
+ */
+export declare class NoDataPropRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string, componentSpec?: ComponentSpec): Violation[];
+}
+//# sourceMappingURL=no-data-prop.d.ts.map

package/dist/runtime-rules/no-data-prop.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-data-prop.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-data-prop.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAChD,OAAO,EAAE,aAAa,EAAE,MAAM,6CAA6C,CAAC;AAE5E;;;;;;;;;;;GAWG;AACH,qBACa,cAAe,SAAQ,YAAY;IAC9C,IAAI,IAAI,WAA6B;IACrC,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,EAAE,aAAa,CAAC,EAAE,aAAa,GAAG,SAAS,EAAE;CAsFrF"}

package/dist/runtime-rules/no-data-prop.js

@@ -0,0 +1,111 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import { traverse } from '../lint-utils.js';
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: no-data-prop
+ *
+ * Discourages the use of generic 'data' prop names in favor of more specific
+ * names like 'items', 'customers', etc. This improves code clarity and makes
+ * the component's purpose more explicit.
+ *
+ * Skips chart/visualization components that legitimately need generic data props.
+ *
+ * Severity: low (opinion-based style preference, not a functional issue)
+ * Applies to: all components
+ */
+let NoDataPropRule = class NoDataPropRule extends BaseLintRule {
+    get Name() { return 'no-data-prop'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName, componentSpec) {
+        const violations = [];
+        // Skip this rule for chart/visualization components that legitimately need generic data props
+        // These are reusable components designed to work with any entity type
+        const isChartComponent = componentSpec?.type === 'chart' ||
+            componentName.toLowerCase().includes('chart') ||
+            componentName.toLowerCase().includes('graph') ||
+            componentName.toLowerCase().includes('visualization') ||
+            componentName.toLowerCase().includes('grid') ||
+            componentName.toLowerCase().includes('table');
+        if (isChartComponent) {
+            return violations; // Skip - generic data prop is expected for chart components
+        }
+        traverse(ast, {
+            // Check function parameters for 'data' prop
+            FunctionDeclaration(path) {
+                if (path.node.id && path.node.id.name === componentName && path.node.params[0]) {
+                    const param = path.node.params[0];
+                    if (t.isObjectPattern(param)) {
+                        for (const prop of param.properties) {
+                            if (t.isObjectProperty(prop) && t.isIdentifier(prop.key) && prop.key.name === 'data') {
+                                violations.push({
+                                    rule: 'no-data-prop',
+                                    severity: 'low', // Opinion-based style preference, not a functional issue
+                                    line: prop.loc?.start.line || 0,
+                                    column: prop.loc?.start.column || 0,
+                                    message: `Component "${componentName}" accepts generic 'data' prop. Consider using more specific prop names like 'items', 'customers', etc. for clarity.`,
+                                    code: 'data prop in component signature',
+                                    suggestion: {
+                                        text: 'Replace generic data prop with specific named props',
+                                        example: `// Instead of:
+function Component({ data, savedUserSettings, onSaveUserSettings }) {
+  return <div>{data.items.map(...)}</div>;
+}
+
+// Use specific props:
+function Component({ items, customers, savedUserSettings, onSaveUserSettings }) {
+  // Component owns its state
+  const [selectedItemId, setSelectedItemId] = useState(
+    savedUserSettings?.selectedItemId
+  );
+
+  return <div>{items.map(...)}</div>;
+}
+
+// Load data using utilities:
+const result = await utilities.rv.RunView({ entityName: 'Items' });`,
+                                    },
+                                });
+                            }
+                        }
+                    }
+                }
+            },
+            // Also check arrow functions
+            VariableDeclarator(path) {
+                if (t.isIdentifier(path.node.id) && path.node.id.name === componentName) {
+                    const init = path.node.init;
+                    if (t.isArrowFunctionExpression(init) && init.params[0]) {
+                        const param = init.params[0];
+                        if (t.isObjectPattern(param)) {
+                            for (const prop of param.properties) {
+                                if (t.isObjectProperty(prop) && t.isIdentifier(prop.key) && prop.key.name === 'data') {
+                                    violations.push({
+                                        rule: 'no-data-prop',
+                                        severity: 'low', // Opinion-based style preference, not a functional issue
+                                        line: prop.loc?.start.line || 0,
+                                        column: prop.loc?.start.column || 0,
+                                        message: `Component "${componentName}" accepts generic 'data' prop. Consider using more specific prop names like 'items', 'customers', etc. for clarity.`,
+                                        code: 'data prop in component signature',
+                                    });
+                                }
+                            }
+                        }
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoDataPropRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-data-prop')
+], NoDataPropRule);
+export { NoDataPropRule };
+//# sourceMappingURL=no-data-prop.js.map

package/dist/runtime-rules/no-data-prop.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-data-prop.js","sourceRoot":"","sources":["../../src/runtime-rules/no-data-prop.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,EAAE,QAAQ,EAAY,MAAM,eAAe,CAAC;AACnD,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAClC,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAI5C;;;;;;;;;;;GAWG;AAEI,IAAM,cAAc,GAApB,MAAM,cAAe,SAAQ,YAAY;IAC9C,IAAI,IAAI,KAAK,OAAO,cAAc,CAAC,CAAC,CAAC;IACrC,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,aAAqB,EAAE,aAA6B;QACpE,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,8FAA8F;QAC9F,sEAAsE;QACtE,MAAM,gBAAgB,GACpB,aAAa,EAAE,IAAI,KAAK,OAAO;YAC/B,aAAa,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC;YAC7C,aAAa,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC;YAC7C,aAAa,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,eAAe,CAAC;YACrD,aAAa,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,MAAM,CAAC;YAC5C,aAAa,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;QAEhD,IAAI,gBAAgB,EAAE,CAAC;YACrB,OAAO,UAAU,CAAC,CAAC,4DAA4D;QACjF,CAAC;QAED,QAAQ,CAAC,GAAG,EAAE;YACZ,4CAA4C;YAC5C,mBAAmB,CAAC,IAAqC;gBACvD,IAAI,IAAI,CAAC,IAAI,CAAC,EAAE,IAAI,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,KAAK,aAAa,IAAI,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;oBAC/E,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;oBAClC,IAAI,CAAC,CAAC,eAAe,CAAC,KAAK,CAAC,EAAE,CAAC;wBAC7B,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,UAAU,EAAE,CAAC;4BACpC,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,KAAK,MAAM,EAAE,CAAC;gCACrF,UAAU,CAAC,IAAI,CAAC;oCACd,IAAI,EAAE,cAAc;oCACpB,QAAQ,EAAE,KAAK,EAAE,yDAAyD;oCAC1E,IAAI,EAAE,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;oCAC/B,MAAM,EAAE,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;oCACnC,OAAO,EAAE,cAAc,aAAa,qHAAqH;oCACzJ,IAAI,EAAE,kCAAkC;oCACxC,UAAU,EAAE;wCACV,IAAI,EAAE,qDAAqD;wCAC3D,OAAO,EAAE;;;;;;;;;;;;;;;;oEAgBuC;qCACjD;iCACF,CAAC,CAAC;4BACL,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAED,6BAA6B;YAC7B,kBAAkB,CAAC,IAAoC;gBACrD,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;oBACxE,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;oBAC5B,IAAI,CAAC,CAAC,yBAAyB,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;wBACxD,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;wBAC7B,IAAI,CAAC,CAAC,eAAe,CAAC,KAAK,CAAC,EAAE,CAAC;4BAC7B,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,UAAU,EAAE,CAAC;gCACpC,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,KAAK,MAAM,EAAE,CAAC;oCACrF,UAAU,CAAC,IAAI,CAAC;wCACd,IAAI,EAAE,cAAc;wCACpB,QAAQ,EAAE,KAAK,EAAE,yDAAyD;wCAC1E,IAAI,EAAE,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;wCAC/B,MAAM,EAAE,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;wCACnC,OAAO,EAAE,cAAc,aAAa,qHAAqH;wCACzJ,IAAI,EAAE,kCAAkC;qCACzC,CAAC,CAAC;gCACL,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AA1FY,cAAc;IAD1B,aAAa,CAAC,YAAY,EAAE,cAAc,CAAC;GAC/B,cAAc,CA0F1B"}

package/dist/runtime-rules/no-export-statements.d.ts

@@ -0,0 +1,18 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: no-export-statements
+ *
+ * Ensures that interactive components do not contain export statements.
+ * Components are self-contained and should not export values.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoExportStatementsRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-export-statements.d.ts.map

package/dist/runtime-rules/no-export-statements.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-export-statements.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-export-statements.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAGhD;;;;;;;;GAQG;AACH,qBACa,sBAAuB,SAAQ,YAAY;IACtD,IAAI,IAAI,WAAqC;IAC7C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAkJtD"}

package/dist/runtime-rules/no-export-statements.js

@@ -0,0 +1,143 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+import { traverse, createViolation, truncateCode } from '../lint-utils.js';
+/**
+ * Rule: no-export-statements
+ *
+ * Ensures that interactive components do not contain export statements.
+ * Components are self-contained and should not export values.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoExportStatementsRule = class NoExportStatementsRule extends BaseLintRule {
+    get Name() { return 'no-export-statements'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        // Track if we're inside the main function and where it ends
+        let mainFunctionEnd = 0;
+        // First pass: find the main component function
+        traverse(ast, {
+            FunctionDeclaration(path) {
+                if (path.node.id?.name === componentName) {
+                    mainFunctionEnd = path.node.loc?.end.line || 0;
+                    path.stop(); // Stop traversing once we find it
+                }
+            },
+            FunctionExpression(path) {
+                // Check for function expressions assigned to const/let/var
+                const parent = path.parent;
+                if (t.isVariableDeclarator(parent) && t.isIdentifier(parent.id) && parent.id.name === componentName) {
+                    mainFunctionEnd = path.node.loc?.end.line || 0;
+                    path.stop();
+                }
+            },
+            ArrowFunctionExpression(path) {
+                // Check for arrow functions assigned to const/let/var
+                const parent = path.parent;
+                if (t.isVariableDeclarator(parent) && t.isIdentifier(parent.id) && parent.id.name === componentName) {
+                    mainFunctionEnd = path.node.loc?.end.line || 0;
+                    path.stop();
+                }
+            },
+        });
+        // Second pass: check for export statements
+        traverse(ast, {
+            ExportNamedDeclaration(path) {
+                const line = path.node.loc?.start.line || 0;
+                violations.push(createViolation('no-export-statements', 'critical', path.node, `Component "${componentName}" contains an export statement${mainFunctionEnd > 0 && line > mainFunctionEnd ? ' after the component function' : ''}. Interactive components are self-contained and cannot export values.`, truncateCode(path.toString()), {
+                    text: 'Remove all export statements. The component function should be the only code, not exported.',
+                    example: `// ❌ WRONG - Using export:
+export function MyComponent({ utilities }) {
+  return <div>Hello</div>;
+}
+
+export const helper = () => {};
+export default MyComponent;
+
+// ✅ CORRECT - Just the function, no exports:
+function MyComponent({ utilities, styles, components }) {
+  // Helper functions defined inside if needed
+  const helper = () => {
+    // ...
+  };
+
+  return <div>Hello</div>;
+}
+
+// The component is self-contained.
+// No exports needed - the host environment
+// will execute the function directly.`,
+                }));
+            },
+            ExportDefaultDeclaration(path) {
+                const line = path.node.loc?.start.line || 0;
+                violations.push(createViolation('no-export-statements', 'critical', path.node, `Component "${componentName}" contains an export default statement${mainFunctionEnd > 0 && line > mainFunctionEnd ? ' after the component function' : ''}. Interactive components are self-contained and cannot export values.`, truncateCode(path.toString()), {
+                    text: 'Remove all export statements. The component function should be the only code, not exported.',
+                    example: `// ❌ WRONG - Using export:
+export function MyComponent({ utilities }) {
+  return <div>Hello</div>;
+}
+
+export const helper = () => {};
+export default MyComponent;
+
+// ✅ CORRECT - Just the function, no exports:
+function MyComponent({ utilities, styles, components }) {
+  // Helper functions defined inside if needed
+  const helper = () => {
+    // ...
+  };
+
+  return <div>Hello</div>;
+}
+
+// The component is self-contained.
+// No exports needed - the host environment
+// will execute the function directly.`,
+                }));
+            },
+            ExportAllDeclaration(path) {
+                const line = path.node.loc?.start.line || 0;
+                violations.push(createViolation('no-export-statements', 'critical', path.node, `Component "${componentName}" contains an export * statement${mainFunctionEnd > 0 && line > mainFunctionEnd ? ' after the component function' : ''}. Interactive components are self-contained and cannot export values.`, truncateCode(path.toString()), {
+                    text: 'Remove all export statements. The component function should be the only code, not exported.',
+                    example: `// ❌ WRONG - Using export:
+export function MyComponent({ utilities }) {
+  return <div>Hello</div>;
+}
+
+export const helper = () => {};
+export default MyComponent;
+
+// ✅ CORRECT - Just the function, no exports:
+function MyComponent({ utilities, styles, components }) {
+  // Helper functions defined inside if needed
+  const helper = () => {
+    // ...
+  };
+
+  return <div>Hello</div>;
+}
+
+// The component is self-contained.
+// No exports needed - the host environment
+// will execute the function directly.`,
+                }));
+            },
+        });
+        return violations;
+    }
+};
+NoExportStatementsRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-export-statements')
+], NoExportStatementsRule);
+export { NoExportStatementsRule };
+//# sourceMappingURL=no-export-statements.js.map

package/dist/runtime-rules/no-export-statements.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-export-statements.js","sourceRoot":"","sources":["../../src/runtime-rules/no-export-statements.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAClC,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAE5C,OAAO,EAAE,QAAQ,EAAY,eAAe,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAElF;;;;;;;;GAQG;AAEI,IAAM,sBAAsB,GAA5B,MAAM,sBAAuB,SAAQ,YAAY;IACtD,IAAI,IAAI,KAAK,OAAO,sBAAsB,CAAC,CAAC,CAAC;IAC7C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,aAAqB;QACrC,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,4DAA4D;QAC5D,IAAI,eAAe,GAAG,CAAC,CAAC;QAExB,+CAA+C;QAC/C,QAAQ,CAAC,GAAG,EAAE;YACZ,mBAAmB,CAAC,IAAqC;gBACvD,IAAI,IAAI,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,KAAK,aAAa,EAAE,CAAC;oBACzC,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,IAAI,IAAI,CAAC,CAAC;oBAC/C,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC,kCAAkC;gBACjD,CAAC;YACH,CAAC;YACD,kBAAkB,CAAC,IAAoC;gBACrD,2DAA2D;gBAC3D,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;gBAC3B,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC,IAAI,MAAM,CAAC,EAAE,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;oBACpG,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,IAAI,IAAI,CAAC,CAAC;oBAC/C,IAAI,CAAC,IAAI,EAAE,CAAC;gBACd,CAAC;YACH,CAAC;YACD,uBAAuB,CAAC,IAAyC;gBAC/D,sDAAsD;gBACtD,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;gBAC3B,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC,IAAI,MAAM,CAAC,EAAE,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;oBACpG,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,IAAI,IAAI,CAAC,CAAC;oBAC/C,IAAI,CAAC,IAAI,EAAE,CAAC;gBACd,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,2CAA2C;QAC3C,QAAQ,CAAC,GAAG,EAAE;YACZ,sBAAsB,CAAC,IAAwC;gBAC7D,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;gBAC5C,UAAU,CAAC,IAAI,CACb,eAAe,CACb,sBAAsB,EACtB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,iCAAiC,eAAe,GAAG,CAAC,IAAI,IAAI,GAAG,eAAe,CAAC,CAAC,CAAC,+BAA+B,CAAC,CAAC,CAAC,EAAE,uEAAuE,EACvN,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;oBACE,IAAI,EAAE,6FAA6F;oBACnG,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;uCAoBgB;iBAC1B,CACF,CACF,CAAC;YACJ,CAAC;YACD,wBAAwB,CAAC,IAA0C;gBACjE,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;gBAC5C,UAAU,CAAC,IAAI,CACb,eAAe,CACb,sBAAsB,EACtB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,yCAAyC,eAAe,GAAG,CAAC,IAAI,IAAI,GAAG,eAAe,CAAC,CAAC,CAAC,+BAA+B,CAAC,CAAC,CAAC,EAAE,uEAAuE,EAC/N,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;oBACE,IAAI,EAAE,6FAA6F;oBACnG,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;uCAoBgB;iBAC1B,CACF,CACF,CAAC;YACJ,CAAC;YACD,oBAAoB,CAAC,IAAsC;gBACzD,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;gBAC5C,UAAU,CAAC,IAAI,CACb,eAAe,CACb,sBAAsB,EACtB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,mCAAmC,eAAe,GAAG,CAAC,IAAI,IAAI,GAAG,eAAe,CAAC,CAAC,CAAC,+BAA+B,CAAC,CAAC,CAAC,EAAE,uEAAuE,EACzN,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;oBACE,IAAI,EAAE,6FAA6F;oBACnG,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;uCAoBgB;iBAC1B,CACF,CACF,CAAC;YACJ,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAtJY,sBAAsB;IADlC,aAAa,CAAC,YAAY,EAAE,sBAAsB,CAAC;GACvC,sBAAsB,CAsJlC"}

package/dist/runtime-rules/no-iife-wrapper.d.ts

@@ -0,0 +1,18 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: no-iife-wrapper
+ *
+ * Ensures that component code is not wrapped in an IIFE (Immediately Invoked Function Expression).
+ * Components should be defined as plain functions.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoIifeWrapperRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File): Violation[];
+}
+//# sourceMappingURL=no-iife-wrapper.d.ts.map

package/dist/runtime-rules/no-iife-wrapper.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-iife-wrapper.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-iife-wrapper.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAGhD;;;;;;;;GAQG;AACH,qBACa,iBAAkB,SAAQ,YAAY;IACjD,IAAI,IAAI,WAAgC;IACxC,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,GAAG,SAAS,EAAE;CAwO/B"}