@memberjunction/react-linter 0.0.1 → 5.38.0

package/dist/runtime-rules/no-iife-wrapper.js

@@ -0,0 +1,217 @@
+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 { createViolation } from '../lint-utils.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
+ */
+let NoIifeWrapperRule = class NoIifeWrapperRule extends BaseLintRule {
+    get Name() { return 'no-iife-wrapper'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast) {
+        const violations = [];
+        // Check if the entire code is wrapped in an IIFE
+        if (ast.program && ast.program.body) {
+            for (const statement of ast.program.body) {
+                // Check for IIFE pattern: (function() { ... })() or (function() { ... }())
+                if (t.isExpressionStatement(statement)) {
+                    const expr = statement.expression;
+                    // Pattern 1: (function() { ... })()
+                    if (t.isCallExpression(expr)) {
+                        const callee = expr.callee;
+                        // Check if calling a function expression wrapped in parentheses
+                        if (t.isParenthesizedExpression && t.isParenthesizedExpression(callee)) {
+                            const inner = callee.expression;
+                            if (t.isFunctionExpression(inner) || t.isArrowFunctionExpression(inner)) {
+                                violations.push(createViolation('no-iife-wrapper', 'critical', statement, `Component code must not be wrapped in an IIFE (Immediately Invoked Function Expression). Define the component function directly.`, statement.toString().substring(0, 50) + '...', {
+                                    text: 'Remove the IIFE wrapper. Component code should be plain functions, not wrapped in immediately invoked functions.',
+                                    example: `// ❌ WRONG - IIFE wrapper patterns:
+(function() {
+  function MyComponent({ utilities, styles, components }) {
+    return <div>Hello</div>;
+  }
+  return MyComponent;
+})();
+
+// Also wrong:
+(function() {
+  const MyComponent = ({ utilities }) => {
+    return <div>Hello</div>;
+  };
+})();
+
+// Also wrong - arrow function IIFE:
+(() => {
+  function MyComponent({ utilities }) {
+    return <div>Hello</div>;
+  }
+})();
+
+// ✅ CORRECT - Direct function declaration:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+// Why no IIFE?
+// 1. Components run in their own scope already
+// 2. The runtime handles isolation
+// 3. IIFEs prevent proper component discovery
+// 4. Makes debugging harder
+// 5. Unnecessary complexity`,
+                                }));
+                            }
+                        }
+                        // Also check without ParenthesizedExpression (some parsers handle it differently)
+                        if (t.isFunctionExpression(callee) || t.isArrowFunctionExpression(callee)) {
+                            violations.push(createViolation('no-iife-wrapper', 'critical', statement, `Component code must not be wrapped in an IIFE. Define the component function directly.`, statement.toString().substring(0, 50) + '...', {
+                                text: 'Remove the IIFE wrapper. Component code should be plain functions, not wrapped in immediately invoked functions.',
+                                example: `// ❌ WRONG - IIFE wrapper patterns:
+(function() {
+  function MyComponent({ utilities, styles, components }) {
+    return <div>Hello</div>;
+  }
+  return MyComponent;
+})();
+
+// Also wrong:
+(function() {
+  const MyComponent = ({ utilities }) => {
+    return <div>Hello</div>;
+  };
+})();
+
+// Also wrong - arrow function IIFE:
+(() => {
+  function MyComponent({ utilities }) {
+    return <div>Hello</div>;
+  }
+})();
+
+// ✅ CORRECT - Direct function declaration:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+// Why no IIFE?
+// 1. Components run in their own scope already
+// 2. The runtime handles isolation
+// 3. IIFEs prevent proper component discovery
+// 4. Makes debugging harder
+// 5. Unnecessary complexity`,
+                            }));
+                        }
+                    }
+                    // Pattern 2: (function() { ... }())
+                    if (t.isParenthesizedExpression && t.isParenthesizedExpression(expr)) {
+                        const inner = expr.expression;
+                        if (t.isCallExpression(inner)) {
+                            const callee = inner.callee;
+                            if (t.isFunctionExpression(callee) || t.isArrowFunctionExpression(callee)) {
+                                violations.push(createViolation('no-iife-wrapper', 'critical', statement, `Component code must not be wrapped in an IIFE. Define the component function directly.`, statement.toString().substring(0, 50) + '...', {
+                                    text: 'Remove the IIFE wrapper. Component code should be plain functions, not wrapped in immediately invoked functions.',
+                                    example: `// ❌ WRONG - IIFE wrapper patterns:
+(function() {
+  function MyComponent({ utilities, styles, components }) {
+    return <div>Hello</div>;
+  }
+  return MyComponent;
+})();
+
+// Also wrong:
+(function() {
+  const MyComponent = ({ utilities }) => {
+    return <div>Hello</div>;
+  };
+})();
+
+// Also wrong - arrow function IIFE:
+(() => {
+  function MyComponent({ utilities }) {
+    return <div>Hello</div>;
+  }
+})();
+
+// ✅ CORRECT - Direct function declaration:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+// Why no IIFE?
+// 1. Components run in their own scope already
+// 2. The runtime handles isolation
+// 3. IIFEs prevent proper component discovery
+// 4. Makes debugging harder
+// 5. Unnecessary complexity`,
+                                }));
+                            }
+                        }
+                    }
+                }
+                // Also check for variable assignment with IIFE
+                if (t.isVariableDeclaration(statement)) {
+                    for (const decl of statement.declarations) {
+                        if (decl.init && t.isCallExpression(decl.init)) {
+                            const callee = decl.init.callee;
+                            if (t.isFunctionExpression(callee) || t.isArrowFunctionExpression(callee)) {
+                                violations.push(createViolation('no-iife-wrapper', 'critical', decl, `Do not use IIFE pattern for component initialization. Define components as plain functions.`, decl.toString().substring(0, 50) + '...', {
+                                    text: 'Remove the IIFE wrapper. Component code should be plain functions, not wrapped in immediately invoked functions.',
+                                    example: `// ❌ WRONG - IIFE wrapper patterns:
+(function() {
+  function MyComponent({ utilities, styles, components }) {
+    return <div>Hello</div>;
+  }
+  return MyComponent;
+})();
+
+// Also wrong:
+(function() {
+  const MyComponent = ({ utilities }) => {
+    return <div>Hello</div>;
+  };
+})();
+
+// Also wrong - arrow function IIFE:
+(() => {
+  function MyComponent({ utilities }) {
+    return <div>Hello</div>;
+  }
+})();
+
+// ✅ CORRECT - Direct function declaration:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+// Why no IIFE?
+// 1. Components run in their own scope already
+// 2. The runtime handles isolation
+// 3. IIFEs prevent proper component discovery
+// 4. Makes debugging harder
+// 5. Unnecessary complexity`,
+                                }));
+                            }
+                        }
+                    }
+                }
+            }
+        }
+        return violations;
+    }
+};
+NoIifeWrapperRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-iife-wrapper')
+], NoIifeWrapperRule);
+export { NoIifeWrapperRule };
+//# sourceMappingURL=no-iife-wrapper.js.map

package/dist/runtime-rules/no-iife-wrapper.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-iife-wrapper.js","sourceRoot":"","sources":["../../src/runtime-rules/no-iife-wrapper.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,eAAe,EAAE,MAAM,eAAe,CAAC;AAEhD;;;;;;;;GAQG;AAEI,IAAM,iBAAiB,GAAvB,MAAM,iBAAkB,SAAQ,YAAY;IACjD,IAAI,IAAI,KAAK,OAAO,iBAAiB,CAAC,CAAC,CAAC;IACxC,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW;QACd,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,iDAAiD;QACjD,IAAI,GAAG,CAAC,OAAO,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC;YACpC,KAAK,MAAM,SAAS,IAAI,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC;gBACzC,2EAA2E;gBAC3E,IAAI,CAAC,CAAC,qBAAqB,CAAC,SAAS,CAAC,EAAE,CAAC;oBACvC,MAAM,IAAI,GAAG,SAAS,CAAC,UAAU,CAAC;oBAElC,oCAAoC;oBACpC,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,EAAE,CAAC;wBAC7B,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;wBAE3B,gEAAgE;wBAChE,IAAI,CAAC,CAAC,yBAAyB,IAAI,CAAC,CAAC,yBAAyB,CAAC,MAAM,CAAC,EAAE,CAAC;4BACvE,MAAM,KAAK,GAAG,MAAM,CAAC,UAAU,CAAC;4BAChC,IAAI,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,yBAAyB,CAAC,KAAK,CAAC,EAAE,CAAC;gCACxE,UAAU,CAAC,IAAI,CACb,eAAe,CACb,iBAAiB,EACjB,UAAU,EACV,SAAS,EACT,kIAAkI,EAClI,SAAS,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,EAC7C;oCACE,IAAI,EAAE,kHAAkH;oCACxH,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;6BAgCF;iCACR,CACF,CACF,CAAC;4BACJ,CAAC;wBACH,CAAC;wBAED,kFAAkF;wBAClF,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,yBAAyB,CAAC,MAAM,CAAC,EAAE,CAAC;4BAC1E,UAAU,CAAC,IAAI,CACb,eAAe,CACb,iBAAiB,EACjB,UAAU,EACV,SAAS,EACT,wFAAwF,EACxF,SAAS,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,EAC7C;gCACE,IAAI,EAAE,kHAAkH;gCACxH,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;6BAgCA;6BACV,CACF,CACF,CAAC;wBACJ,CAAC;oBACH,CAAC;oBAED,oCAAoC;oBACpC,IAAI,CAAC,CAAC,yBAAyB,IAAI,CAAC,CAAC,yBAAyB,CAAC,IAAI,CAAC,EAAE,CAAC;wBACrE,MAAM,KAAK,GAAG,IAAI,CAAC,UAAU,CAAC;wBAC9B,IAAI,CAAC,CAAC,gBAAgB,CAAC,KAAK,CAAC,EAAE,CAAC;4BAC9B,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC;4BAC5B,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,yBAAyB,CAAC,MAAM,CAAC,EAAE,CAAC;gCAC1E,UAAU,CAAC,IAAI,CACb,eAAe,CACb,iBAAiB,EACjB,UAAU,EACV,SAAS,EACT,wFAAwF,EACxF,SAAS,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,EAC7C;oCACE,IAAI,EAAE,kHAAkH;oCACxH,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;6BAgCF;iCACR,CACF,CACF,CAAC;4BACJ,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;gBAED,+CAA+C;gBAC/C,IAAI,CAAC,CAAC,qBAAqB,CAAC,SAAS,CAAC,EAAE,CAAC;oBACvC,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,YAAY,EAAE,CAAC;wBAC1C,IAAI,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;4BAC/C,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC;4BAChC,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,yBAAyB,CAAC,MAAM,CAAC,EAAE,CAAC;gCAC1E,UAAU,CAAC,IAAI,CACb,eAAe,CACb,iBAAiB,EACjB,UAAU,EACV,IAAI,EACJ,6FAA6F,EAC7F,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,EACxC;oCACE,IAAI,EAAE,kHAAkH;oCACxH,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;6BAgCF;iCACR,CACF,CACF,CAAC;4BACJ,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AA5OY,iBAAiB;IAD7B,aAAa,CAAC,YAAY,EAAE,iBAAiB,CAAC;GAClC,iBAAiB,CA4O7B"}

package/dist/runtime-rules/no-import-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-import-statements
+ *
+ * Ensures that interactive components do not contain import statements.
+ * All dependencies must be passed as props (utilities, components, styles).
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoImportStatementsRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-import-statements.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-import-statements.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-import-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;CAiDtD"}

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

@@ -0,0 +1,65 @@
+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 { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+import { traverse, createViolation, truncateCode } from '../lint-utils.js';
+/**
+ * Rule: no-import-statements
+ *
+ * Ensures that interactive components do not contain import statements.
+ * All dependencies must be passed as props (utilities, components, styles).
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoImportStatementsRule = class NoImportStatementsRule extends BaseLintRule {
+    get Name() { return 'no-import-statements'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        traverse(ast, {
+            ImportDeclaration(path) {
+                violations.push(createViolation('no-import-statements', 'critical', path.node, `Component "${componentName}" contains an import statement. Interactive components cannot use import statements - all dependencies must be passed as props.`, truncateCode(path.toString()), {
+                    text: 'Remove all import statements. Interactive components receive everything through props.',
+                    example: `// ❌ WRONG - Using import statements:
+import React from 'react';
+import { useState } from 'react';
+import { format } from 'date-fns';
+import './styles.css';
+
+function MyComponent({ utilities, styles }) {
+  // ...
+}
+
+// ✅ CORRECT - Everything passed as props:
+function MyComponent({ utilities, styles, components }) {
+  // React hooks are available globally (useState, useEffect, etc.)
+  const [value, setValue] = useState('');
+
+  // Utilities include formatting functions
+  const formatted = utilities.formatDate(new Date());
+
+  // Styles are passed as props
+  return <div style={styles.container}>...</div>;
+}
+
+// All dependencies must be:
+// 1. Passed through the 'utilities' prop (formatting, helpers)
+// 2. Passed through the 'components' prop (child components)
+// 3. Passed through the 'styles' prop (styling)
+// 4. Available globally (React hooks)`,
+                }));
+            },
+        });
+        return violations;
+    }
+};
+NoImportStatementsRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-import-statements')
+], NoImportStatementsRule);
+export { NoImportStatementsRule };
+//# sourceMappingURL=no-import-statements.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-import-statements.js","sourceRoot":"","sources":["../../src/runtime-rules/no-import-statements.ts"],"names":[],"mappings":";;;;;;AACA,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,QAAQ,CAAC,GAAG,EAAE;YACZ,iBAAiB,CAAC,IAAmC;gBACnD,UAAU,CAAC,IAAI,CACb,eAAe,CACb,sBAAsB,EACtB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,iIAAiI,EAC5J,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;oBACE,IAAI,EAAE,wFAAwF;oBAC9F,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;uCA0BgB;iBAC1B,CACF,CACF,CAAC;YACJ,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AArDY,sBAAsB;IADlC,aAAa,CAAC,YAAY,EAAE,sBAAsB,CAAC;GACvC,sBAAsB,CAqDlC"}

package/dist/runtime-rules/no-react-destructuring.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-react-destructuring
+ *
+ * Ensures that components do not destructure from React.
+ * React hooks are available globally and don't need to be imported or destructured.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoReactDestructuringRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File): Violation[];
+}
+//# sourceMappingURL=no-react-destructuring.d.ts.map

package/dist/runtime-rules/no-react-destructuring.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-react-destructuring.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-react-destructuring.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,wBAAyB,SAAQ,YAAY;IACxD,IAAI,IAAI,WAAuC;IAC/C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,GAAG,SAAS,EAAE;CAmC/B"}

package/dist/runtime-rules/no-react-destructuring.js

@@ -0,0 +1,60 @@
+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, truncateCode } from '../lint-utils.js';
+/**
+ * Rule: no-react-destructuring
+ *
+ * Ensures that components do not destructure from React.
+ * React hooks are available globally and don't need to be imported or destructured.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoReactDestructuringRule = class NoReactDestructuringRule extends BaseLintRule {
+    get Name() { return 'no-react-destructuring'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast) {
+        const violations = [];
+        traverse(ast, {
+            VariableDeclarator(path) {
+                // Check for destructuring from React
+                if (t.isObjectPattern(path.node.id) && t.isIdentifier(path.node.init) && path.node.init.name === 'React') {
+                    // Get the destructured properties
+                    const destructuredProps = path.node.id.properties
+                        .filter((prop) => t.isObjectProperty(prop) && t.isIdentifier(prop.key))
+                        .map((prop) => prop.key)
+                        .map((key) => key.name);
+                    violations.push({
+                        rule: 'no-react-destructuring',
+                        severity: 'critical',
+                        line: path.node.loc?.start.line || 0,
+                        column: path.node.loc?.start.column || 0,
+                        message: `Cannot destructure from React. The hooks (${destructuredProps.join(', ')}) are already available as global functions in the React runtime.`,
+                        code: truncateCode(path.toString()),
+                        suggestion: {
+                            text: `Remove the destructuring statement. React hooks like ${destructuredProps.join(', ')} are already available globally and don't need to be imported or destructured.`,
+                            example: `// Remove this line entirely:
+// const { ${destructuredProps.join(', ')} } = React;
+
+// Just use the hooks directly:
+const [state, setState] = useState(initialValue);`,
+                        },
+                    });
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoReactDestructuringRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-react-destructuring')
+], NoReactDestructuringRule);
+export { NoReactDestructuringRule };
+//# sourceMappingURL=no-react-destructuring.js.map

package/dist/runtime-rules/no-react-destructuring.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-react-destructuring.js","sourceRoot":"","sources":["../../src/runtime-rules/no-react-destructuring.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,YAAY,EAAE,MAAM,eAAe,CAAC;AAEjE;;;;;;;;GAQG;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;QACd,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,QAAQ,CAAC,GAAG,EAAE;YACZ,kBAAkB,CAAC,IAAoC;gBACrD,qCAAqC;gBACrC,IAAI,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;oBACzG,kCAAkC;oBAClC,MAAM,iBAAiB,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,UAAU;yBAC9C,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;yBACtE,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAE,IAAyB,CAAC,GAAmB,CAAC;yBAC7D,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;oBAE1B,UAAU,CAAC,IAAI,CAAC;wBACd,IAAI,EAAE,wBAAwB;wBAC9B,QAAQ,EAAE,UAAU;wBACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;wBACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;wBACxC,OAAO,EAAE,6CAA6C,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,mEAAmE;wBACrJ,IAAI,EAAE,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC;wBACnC,UAAU,EAAE;4BACV,IAAI,EAAE,wDAAwD,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,gFAAgF;4BAC1K,OAAO,EAAE;aACV,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC;;;kDAGS;yBACrC;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAvCY,wBAAwB;IADpC,aAAa,CAAC,YAAY,EAAE,wBAAwB,CAAC;GACzC,wBAAwB,CAuCpC"}

package/dist/runtime-rules/no-require-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-require-statements
+ *
+ * Ensures that interactive components do not use require() or dynamic import().
+ * All dependencies must be passed as props.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoRequireStatementsRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-require-statements.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-require-statements.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-require-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,uBAAwB,SAAQ,YAAY;IACvD,IAAI,IAAI,WAAsC;IAC9C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAuGtD"}

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

@@ -0,0 +1,109 @@
+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-require-statements
+ *
+ * Ensures that interactive components do not use require() or dynamic import().
+ * All dependencies must be passed as props.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoRequireStatementsRule = class NoRequireStatementsRule extends BaseLintRule {
+    get Name() { return 'no-require-statements'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                const callee = path.node.callee;
+                // Check for require() calls
+                if (t.isIdentifier(callee) && callee.name === 'require') {
+                    violations.push(createViolation('no-require-statements', 'critical', path.node, `Component "${componentName}" contains a require() statement. Interactive components cannot use require - all dependencies must be passed as props.`, truncateCode(path.toString()), {
+                        text: 'Remove all require() and dynamic import() statements. Use props instead.',
+                        example: `// ❌ WRONG - Using require or dynamic import:
+function MyComponent({ utilities }) {
+  const lodash = require('lodash');
+  const module = await import('./module');
+
+  return <div>...</div>;
+}
+
+// ✅ CORRECT - Use utilities and components props:
+function MyComponent({ utilities, styles, components }) {
+  // Use utilities for helper functions
+  const result = utilities.debounce(() => {
+    // ...
+  }, 300);
+
+  // Use components prop for child components
+  const { DataTable, FilterPanel } = components;
+
+  return (
+    <div>
+      <DataTable {...props} />
+      <FilterPanel {...props} />
+    </div>
+  );
+}
+
+// Everything the component needs must be:
+// - Passed via props (utilities, components, styles)
+// - Available globally (React hooks)
+// No module loading allowed!`,
+                    }));
+                }
+                // Also check for dynamic import() calls
+                if (t.isImport(callee)) {
+                    violations.push(createViolation('no-require-statements', 'critical', path.node, `Component "${componentName}" contains a dynamic import() statement. Interactive components cannot use dynamic imports - all dependencies must be passed as props.`, truncateCode(path.toString()), {
+                        text: 'Remove all require() and dynamic import() statements. Use props instead.',
+                        example: `// ❌ WRONG - Using require or dynamic import:
+function MyComponent({ utilities }) {
+  const lodash = require('lodash');
+  const module = await import('./module');
+
+  return <div>...</div>;
+}
+
+// ✅ CORRECT - Use utilities and components props:
+function MyComponent({ utilities, styles, components }) {
+  // Use utilities for helper functions
+  const result = utilities.debounce(() => {
+    // ...
+  }, 300);
+
+  // Use components prop for child components
+  const { DataTable, FilterPanel } = components;
+
+  return (
+    <div>
+      <DataTable {...props} />
+      <FilterPanel {...props} />
+    </div>
+  );
+}
+
+// Everything the component needs must be:
+// - Passed via props (utilities, components, styles)
+// - Available globally (React hooks)
+// No module loading allowed!`,
+                    }));
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoRequireStatementsRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-require-statements')
+], NoRequireStatementsRule);
+export { NoRequireStatementsRule };
+//# sourceMappingURL=no-require-statements.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-require-statements.js","sourceRoot":"","sources":["../../src/runtime-rules/no-require-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,uBAAuB,GAA7B,MAAM,uBAAwB,SAAQ,YAAY;IACvD,IAAI,IAAI,KAAK,OAAO,uBAAuB,CAAC,CAAC,CAAC;IAC9C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,aAAqB;QACrC,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,QAAQ,CAAC,GAAG,EAAE;YACZ,cAAc,CAAC,IAAgC;gBAC7C,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC;gBAEhC,4BAA4B;gBAC5B,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;oBACxD,UAAU,CAAC,IAAI,CACb,eAAe,CACb,uBAAuB,EACvB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,yHAAyH,EACpJ,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;wBACE,IAAI,EAAE,0EAA0E;wBAChF,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;8BA6BK;qBACf,CACF,CACF,CAAC;gBACJ,CAAC;gBAED,wCAAwC;gBACxC,IAAI,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;oBACvB,UAAU,CAAC,IAAI,CACb,eAAe,CACb,uBAAuB,EACvB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,wIAAwI,EACnK,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;wBACE,IAAI,EAAE,0EAA0E;wBAChF,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;8BA6BK;qBACf,CACF,CACF,CAAC;gBACJ,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AA3GY,uBAAuB;IADnC,aAAa,CAAC,YAAY,EAAE,uBAAuB,CAAC;GACxC,uBAAuB,CA2GnC"}

package/dist/runtime-rules/no-return-component.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-return-component
+ *
+ * Ensures that components don't have return statements or component references
+ * at the end of the file. The component function should stand alone.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoReturnComponentRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-return-component.d.ts.map

package/dist/runtime-rules/no-return-component.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-return-component.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-return-component.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,qBAAsB,SAAQ,YAAY;IACrD,IAAI,IAAI,WAAoC;IAC5C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAoGtD"}