@memberjunction/react-test-harness 5.37.0 → 5.38.0

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

@@ -1,217 +0,0 @@
-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/lib/runtime-rules/no-iife-wrapper.js.map

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

@@ -1,18 +0,0 @@
-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/lib/runtime-rules/no-import-statements.d.ts.map

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

@@ -1,65 +0,0 @@
-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/lib/runtime-rules/no-import-statements.js.map

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

@@ -1,18 +0,0 @@
-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/lib/runtime-rules/no-react-destructuring.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-react-destructuring.d.ts","sourceRoot":"","sources":["../../../src/lib/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/lib/runtime-rules/no-react-destructuring.js

@@ -1,60 +0,0 @@
-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/lib/runtime-rules/no-react-destructuring.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-react-destructuring.js","sourceRoot":"","sources":["../../../src/lib/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/lib/runtime-rules/no-require-statements.d.ts

@@ -1,18 +0,0 @@
-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/lib/runtime-rules/no-require-statements.d.ts.map

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

@@ -1,109 +0,0 @@
-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/lib/runtime-rules/no-require-statements.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-require-statements.js","sourceRoot":"","sources":["../../../src/lib/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/lib/runtime-rules/no-return-component.d.ts

@@ -1,18 +0,0 @@
-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/lib/runtime-rules/no-return-component.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-return-component.d.ts","sourceRoot":"","sources":["../../../src/lib/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"}