@memberjunction/react-linter 0.0.1 → 5.38.0

package/dist/runtime-rules/overflow-hidden-on-layout-container.d.ts

@@ -0,0 +1,30 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: overflow-hidden-on-layout-container
+ *
+ * Detects `overflow: 'hidden'` on flex content containers that also have
+ * `flex: 1`. This combination creates an invisible clipping boundary that
+ * blocks scrolling for child content (DataGrid, tables, lists, etc.).
+ *
+ * The Angular host (`mj-react-component`) already delegates scrolling to its
+ * parent container which uses `overflow: auto`. Adding `overflow: hidden` on
+ * a flex content wrapper silently clips all descendant scrollbars.
+ *
+ * Does NOT flag:
+ * - Root viewport containers (`height: '100vh'` + overflow hidden) — these
+ *   intentionally prevent page-level scrollbars
+ * - Small/fixed-size containers (e.g., `width: '200px'` with overflow hidden)
+ * - Text truncation (overflow hidden + textOverflow ellipsis)
+ * - Containers without flex: 1
+ *
+ * Severity: medium — the component renders but scrollable content is unreachable.
+ * Applies to: all components
+ */
+export declare class OverflowHiddenOnLayoutContainerRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=overflow-hidden-on-layout-container.d.ts.map

package/dist/runtime-rules/overflow-hidden-on-layout-container.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"overflow-hidden-on-layout-container.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/overflow-hidden-on-layout-container.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;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,qBACa,mCAAoC,SAAQ,YAAY;IACnE,IAAI,IAAI,WAAoD;IAC5D,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAqDtD"}

package/dist/runtime-rules/overflow-hidden-on-layout-container.js

@@ -0,0 +1,220 @@
+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, createViolation, truncateCode } from '../lint-utils.js';
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: overflow-hidden-on-layout-container
+ *
+ * Detects `overflow: 'hidden'` on flex content containers that also have
+ * `flex: 1`. This combination creates an invisible clipping boundary that
+ * blocks scrolling for child content (DataGrid, tables, lists, etc.).
+ *
+ * The Angular host (`mj-react-component`) already delegates scrolling to its
+ * parent container which uses `overflow: auto`. Adding `overflow: hidden` on
+ * a flex content wrapper silently clips all descendant scrollbars.
+ *
+ * Does NOT flag:
+ * - Root viewport containers (`height: '100vh'` + overflow hidden) — these
+ *   intentionally prevent page-level scrollbars
+ * - Small/fixed-size containers (e.g., `width: '200px'` with overflow hidden)
+ * - Text truncation (overflow hidden + textOverflow ellipsis)
+ * - Containers without flex: 1
+ *
+ * Severity: medium — the component renders but scrollable content is unreachable.
+ * Applies to: all components
+ */
+let OverflowHiddenOnLayoutContainerRule = class OverflowHiddenOnLayoutContainerRule extends BaseLintRule {
+    get Name() { return 'overflow-hidden-on-layout-container'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        traverse(ast, {
+            ObjectExpression(path) {
+                if (!isInlineStyleObject(path)) {
+                    return;
+                }
+                const props = extractStyleProperties(path.node);
+                if (!props) {
+                    return;
+                }
+                if (!hasOverflowHidden(props)) {
+                    return;
+                }
+                // Text truncation pattern — intentional, not a layout bug
+                if (hasTextTruncation(props)) {
+                    return;
+                }
+                // Only flag if this looks like a layout container
+                if (!isLayoutContainer(props)) {
+                    return;
+                }
+                violations.push(createViolation('overflow-hidden-on-layout-container', 'medium', path.node, `Component "${componentName}" uses overflow: 'hidden' on a layout container ` +
+                    `(with ${describeLayoutIndicator(props)}). This creates an invisible clipping ` +
+                    `boundary that blocks scrolling for child content like DataGrid, tables, and ` +
+                    `lists. The Angular host already manages scrolling via overflow: auto.`, truncateCode(formatStyleSnippet(props), 100), {
+                    text: `Use overflow: 'auto' to allow child scrolling, or remove the overflow property entirely.`,
+                    example: `// Before (blocks scrolling):\n` +
+                        `style={{ flex: 1, overflow: 'hidden' }}\n\n` +
+                        `// After (allows child scrolling):\n` +
+                        `style={{ flex: 1, overflow: 'auto' }}\n` +
+                        `// or simply:\n` +
+                        `style={{ flex: 1 }}`,
+                }));
+            },
+        });
+        return violations;
+    }
+};
+OverflowHiddenOnLayoutContainerRule = __decorate([
+    RegisterClass(BaseLintRule, 'overflow-hidden-on-layout-container')
+], OverflowHiddenOnLayoutContainerRule);
+export { OverflowHiddenOnLayoutContainerRule };
+/**
+ * Determines if this ObjectExpression is a JSX inline style object.
+ * Matches both `style={{ ... }}` and `style={someVar}` where someVar
+ * is assigned an object literal.
+ */
+function isInlineStyleObject(path) {
+    const parent = path.parent;
+    // Direct: style={{ ... }}
+    if (t.isJSXExpressionContainer(parent)) {
+        const grandparent = path.parentPath?.parent;
+        if (t.isJSXAttribute(grandparent) &&
+            t.isJSXIdentifier(grandparent.name) &&
+            grandparent.name.name === 'style') {
+            return true;
+        }
+    }
+    // Variable: const styles = { ... }  used as style={styles}
+    // We also catch nested objects within style objects (e.g., containerStyle)
+    if (t.isVariableDeclarator(parent)) {
+        const id = parent.id;
+        if (t.isIdentifier(id) && isStyleVariableName(id.name)) {
+            return true;
+        }
+    }
+    // Property in a styles object: { container: { overflow: 'hidden', flex: 1 } }
+    if (t.isObjectProperty(parent) && t.isIdentifier(parent.key)) {
+        const propName = parent.key.name.toLowerCase();
+        if (propName.includes('style') || propName.includes('container') ||
+            propName.includes('wrapper') || propName.includes('root') ||
+            propName.includes('layout') || propName.includes('content')) {
+            return true;
+        }
+    }
+    return false;
+}
+function isStyleVariableName(name) {
+    const lower = name.toLowerCase();
+    return lower.includes('style') || lower.includes('container') ||
+        lower.includes('wrapper') || lower.includes('root') ||
+        lower.includes('layout');
+}
+/**
+ * Extracts known style properties from an ObjectExpression as simple values.
+ */
+function extractStyleProperties(node) {
+    const result = {
+        overflow: null,
+        overflowX: null,
+        overflowY: null,
+        textOverflow: null,
+        flex: null,
+        height: null,
+        node,
+    };
+    for (const prop of node.properties) {
+        if (!t.isObjectProperty(prop) || !t.isIdentifier(prop.key)) {
+            continue;
+        }
+        const key = prop.key.name;
+        const val = prop.value;
+        switch (key) {
+            case 'overflow':
+                result.overflow = extractStringOrNumberValue(val);
+                break;
+            case 'overflowX':
+                result.overflowX = extractStringValue(val);
+                break;
+            case 'overflowY':
+                result.overflowY = extractStringValue(val);
+                break;
+            case 'textOverflow':
+                result.textOverflow = extractStringValue(val);
+                break;
+            case 'flex':
+                result.flex = extractFlexValue(val);
+                break;
+            case 'height':
+                result.height = extractStringOrNumberValue(val);
+                break;
+        }
+    }
+    return result;
+}
+function extractStringValue(node) {
+    if (t.isStringLiteral(node))
+        return node.value;
+    return null;
+}
+function extractStringOrNumberValue(node) {
+    if (t.isStringLiteral(node))
+        return node.value;
+    if (t.isNumericLiteral(node))
+        return String(node.value);
+    return null;
+}
+function extractFlexValue(node) {
+    if (t.isNumericLiteral(node))
+        return node.value;
+    if (t.isStringLiteral(node))
+        return node.value;
+    return null;
+}
+function hasOverflowHidden(props) {
+    return props.overflow === 'hidden' ||
+        (props.overflowX === 'hidden' && props.overflowY === 'hidden');
+}
+function hasTextTruncation(props) {
+    return props.textOverflow === 'ellipsis';
+}
+/**
+ * A flex content container is identified by having flex: 1 (or similar).
+ * These are inner content wrappers where overflow: hidden blocks child scrolling.
+ *
+ * Root viewport containers (height: '100vh') intentionally use overflow: hidden
+ * to prevent page-level scrollbars — that pattern is NOT flagged.
+ */
+function isLayoutContainer(props) {
+    // flex: 1 or flex: '1' or flex: '1 1 0%' etc.
+    if (props.flex !== null) {
+        const flexStr = String(props.flex);
+        if (flexStr === '1' || flexStr.startsWith('1 ')) {
+            return true;
+        }
+    }
+    return false;
+}
+function describeLayoutIndicator(props) {
+    if (props.flex !== null) {
+        return `flex: ${typeof props.flex === 'string' ? `'${props.flex}'` : props.flex}`;
+    }
+    return 'flex layout';
+}
+function formatStyleSnippet(props) {
+    const parts = [];
+    if (props.overflow)
+        parts.push(`overflow: '${props.overflow}'`);
+    if (props.flex !== null)
+        parts.push(`flex: ${typeof props.flex === 'string' ? `'${props.flex}'` : props.flex}`);
+    if (props.height)
+        parts.push(`height: '${props.height}'`);
+    return `{ ${parts.join(', ')} }`;
+}
+//# sourceMappingURL=overflow-hidden-on-layout-container.js.map

package/dist/runtime-rules/overflow-hidden-on-layout-container.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"overflow-hidden-on-layout-container.js","sourceRoot":"","sources":["../../src/runtime-rules/overflow-hidden-on-layout-container.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,EAAE,QAAQ,EAAY,eAAe,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAClF,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAClC,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAG5C;;;;;;;;;;;;;;;;;;;;GAoBG;AAEI,IAAM,mCAAmC,GAAzC,MAAM,mCAAoC,SAAQ,YAAY;IACnE,IAAI,IAAI,KAAK,OAAO,qCAAqC,CAAC,CAAC,CAAC;IAC5D,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,gBAAgB,CAAC,IAAkC;gBACjD,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC/B,OAAO;gBACT,CAAC;gBAED,MAAM,KAAK,GAAG,sBAAsB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAChD,IAAI,CAAC,KAAK,EAAE,CAAC;oBACX,OAAO;gBACT,CAAC;gBAED,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC9B,OAAO;gBACT,CAAC;gBAED,0DAA0D;gBAC1D,IAAI,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC7B,OAAO;gBACT,CAAC;gBAED,kDAAkD;gBAClD,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;oBAC9B,OAAO;gBACT,CAAC;gBAED,UAAU,CAAC,IAAI,CAAC,eAAe,CAC7B,qCAAqC,EACrC,QAAQ,EACR,IAAI,CAAC,IAAI,EACT,cAAc,aAAa,kDAAkD;oBAC7E,SAAS,uBAAuB,CAAC,KAAK,CAAC,wCAAwC;oBAC/E,8EAA8E;oBAC9E,uEAAuE,EACvE,YAAY,CAAC,kBAAkB,CAAC,KAAK,CAAC,EAAE,GAAG,CAAC,EAC5C;oBACE,IAAI,EAAE,0FAA0F;oBAChG,OAAO,EACL,iCAAiC;wBACjC,6CAA6C;wBAC7C,sCAAsC;wBACtC,yCAAyC;wBACzC,iBAAiB;wBACjB,qBAAqB;iBACxB,CACF,CAAC,CAAC;YACL,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAzDY,mCAAmC;IAD/C,aAAa,CAAC,YAAY,EAAE,qCAAqC,CAAC;GACtD,mCAAmC,CAyD/C;;AAcD;;;;GAIG;AACH,SAAS,mBAAmB,CAAC,IAAkC;IAC7D,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;IAE3B,0BAA0B;IAC1B,IAAI,CAAC,CAAC,wBAAwB,CAAC,MAAM,CAAC,EAAE,CAAC;QACvC,MAAM,WAAW,GAAG,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC;QAC5C,IAAI,CAAC,CAAC,cAAc,CAAC,WAAW,CAAC;YAC7B,CAAC,CAAC,eAAe,CAAC,WAAW,CAAC,IAAI,CAAC;YACnC,WAAW,CAAC,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;YACtC,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,2DAA2D;IAC3D,2EAA2E;IAC3E,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,EAAE,CAAC;QACnC,MAAM,EAAE,GAAG,MAAM,CAAC,EAAE,CAAC;QACrB,IAAI,CAAC,CAAC,YAAY,CAAC,EAAE,CAAC,IAAI,mBAAmB,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACvD,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,8EAA8E;IAC9E,IAAI,CAAC,CAAC,gBAAgB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;QAC7D,MAAM,QAAQ,GAAG,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;QAC/C,IAAI,QAAQ,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,WAAW,CAAC;YAC5D,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC;YACzD,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,CAAC;YAChE,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,mBAAmB,CAAC,IAAY;IACvC,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IACjC,OAAO,KAAK,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,WAAW,CAAC;QACtD,KAAK,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC;QACnD,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;AAClC,CAAC;AAED;;GAEG;AACH,SAAS,sBAAsB,CAAC,IAAwB;IACtD,MAAM,MAAM,GAAe;QACzB,QAAQ,EAAE,IAAI;QACd,SAAS,EAAE,IAAI;QACf,SAAS,EAAE,IAAI;QACf,YAAY,EAAE,IAAI;QAClB,IAAI,EAAE,IAAI;QACV,MAAM,EAAE,IAAI;QACZ,IAAI;KACL,CAAC;IAEF,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;QACnC,IAAI,CAAC,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC;YAC3D,SAAS;QACX,CAAC;QACD,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC;QAC1B,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC;QAEvB,QAAQ,GAAG,EAAE,CAAC;YACZ,KAAK,UAAU;gBACb,MAAM,CAAC,QAAQ,GAAG,0BAA0B,CAAC,GAAG,CAAC,CAAC;gBAClD,MAAM;YACR,KAAK,WAAW;gBACd,MAAM,CAAC,SAAS,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC;gBAC3C,MAAM;YACR,KAAK,WAAW;gBACd,MAAM,CAAC,SAAS,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC;gBAC3C,MAAM;YACR,KAAK,cAAc;gBACjB,MAAM,CAAC,YAAY,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC;gBAC9C,MAAM;YACR,KAAK,MAAM;gBACT,MAAM,CAAC,IAAI,GAAG,gBAAgB,CAAC,GAAG,CAAC,CAAC;gBACpC,MAAM;YACR,KAAK,QAAQ;gBACX,MAAM,CAAC,MAAM,GAAG,0BAA0B,CAAC,GAAG,CAAC,CAAC;gBAChD,MAAM;QACV,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY;IACtC,IAAI,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC,KAAK,CAAC;IAC/C,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,0BAA0B,CAAC,IAAY;IAC9C,IAAI,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC,KAAK,CAAC;IAC/C,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC;QAAE,OAAO,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACxD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,gBAAgB,CAAC,IAAY;IACpC,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC,KAAK,CAAC;IAChD,IAAI,CAAC,CAAC,eAAe,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC,KAAK,CAAC;IAC/C,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAiB;IAC1C,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ;QAC3B,CAAC,KAAK,CAAC,SAAS,KAAK,QAAQ,IAAI,KAAK,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC;AACxE,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAiB;IAC1C,OAAO,KAAK,CAAC,YAAY,KAAK,UAAU,CAAC;AAC3C,CAAC;AAED;;;;;;GAMG;AACH,SAAS,iBAAiB,CAAC,KAAiB;IAC1C,8CAA8C;IAC9C,IAAI,KAAK,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;QACxB,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QACnC,IAAI,OAAO,KAAK,GAAG,IAAI,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;YAChD,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,uBAAuB,CAAC,KAAiB;IAChD,IAAI,KAAK,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;QACxB,OAAO,SAAS,OAAO,KAAK,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;IACpF,CAAC;IACD,OAAO,aAAa,CAAC;AACvB,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAiB;IAC3C,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,KAAK,CAAC,QAAQ;QAAE,KAAK,CAAC,IAAI,CAAC,cAAc,KAAK,CAAC,QAAQ,GAAG,CAAC,CAAC;IAChE,IAAI,KAAK,CAAC,IAAI,KAAK,IAAI;QAAE,KAAK,CAAC,IAAI,CAAC,SAAS,OAAO,KAAK,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC;IAChH,IAAI,KAAK,CAAC,MAAM;QAAE,KAAK,CAAC,IAAI,CAAC,YAAY,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC;IAC1D,OAAO,KAAK,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;AACnC,CAAC"}

package/dist/runtime-rules/pass-standard-props.d.ts

@@ -0,0 +1,19 @@
+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: pass-standard-props
+ *
+ * Ensures that all dependency components receive the standard props:
+ * styles, utilities, and components.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class PassStandardPropsRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, _componentName: string, componentSpec?: ComponentSpec): Violation[];
+}
+//# sourceMappingURL=pass-standard-props.d.ts.map

package/dist/runtime-rules/pass-standard-props.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pass-standard-props.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/pass-standard-props.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;AAChD,OAAO,EAAE,aAAa,EAAE,MAAM,6CAA6C,CAAC;AAG5E;;;;;;;;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,cAAc,EAAE,MAAM,EAAE,aAAa,CAAC,EAAE,aAAa,GAAG,SAAS,EAAE;CAwEtF"}

package/dist/runtime-rules/pass-standard-props.js

@@ -0,0 +1,82 @@
+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, getJSXElementName, hasJSXAttribute } from '../lint-utils.js';
+/**
+ * Rule: pass-standard-props
+ *
+ * Ensures that all dependency components receive the standard props:
+ * styles, utilities, and components.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let PassStandardPropsRule = class PassStandardPropsRule extends BaseLintRule {
+    get Name() { return 'pass-standard-props'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName, componentSpec) {
+        const violations = [];
+        const requiredProps = ['styles', 'utilities', 'components'];
+        // ONLY check components that are explicitly in our dependencies
+        // Do NOT check library components, HTML elements, or anything else
+        const ourComponentNames = new Set();
+        // Only add components from the componentSpec.dependencies array
+        if (componentSpec?.dependencies && Array.isArray(componentSpec.dependencies)) {
+            for (const dep of componentSpec.dependencies) {
+                if (dep && dep.name) {
+                    ourComponentNames.add(dep.name);
+                }
+            }
+        }
+        // If there are no dependencies, skip this rule entirely
+        if (ourComponentNames.size === 0) {
+            return violations;
+        }
+        traverse(ast, {
+            JSXElement(path) {
+                const openingElement = path.node.openingElement;
+                const elementName = getJSXElementName(openingElement);
+                // Only check if this is one of our dependency components
+                if (elementName && ourComponentNames.has(elementName)) {
+                    // Check for required props
+                    const missingProps = [];
+                    for (const propName of requiredProps) {
+                        if (!hasJSXAttribute(openingElement, propName)) {
+                            missingProps.push(propName);
+                        }
+                    }
+                    if (missingProps.length > 0) {
+                        violations.push(createViolation('pass-standard-props', 'critical', openingElement, `Dependency component "${elementName}" is missing required props: ${missingProps.join(', ')}. Components from dependencies must receive styles, utilities, and components props.`, `<${elementName} ... />`, {
+                            text: 'Always pass standard props to all components',
+                            example: `// Always include these props when calling components:
+<ChildComponent
+  items={items}  // Data props
+
+  // Settings persistence
+  savedUserSettings={savedUserSettings?.childComponent}
+  onSaveUserSettings={handleChildSettings}
+
+  // Standard props
+  styles={styles}
+  utilities={utilities}
+  components={components}
+  callbacks={callbacks}
+/>`,
+                        }));
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+PassStandardPropsRule = __decorate([
+    RegisterClass(BaseLintRule, 'pass-standard-props')
+], PassStandardPropsRule);
+export { PassStandardPropsRule };
+//# sourceMappingURL=pass-standard-props.js.map

package/dist/runtime-rules/pass-standard-props.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pass-standard-props.js","sourceRoot":"","sources":["../../src/runtime-rules/pass-standard-props.ts"],"names":[],"mappings":";;;;;;AACA,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAG5C,OAAO,EAAE,QAAQ,EAAY,eAAe,EAAE,iBAAiB,EAAE,eAAe,EAAE,MAAM,eAAe,CAAC;AAExG;;;;;;;;GAQG;AAEI,IAAM,qBAAqB,GAA3B,MAAM,qBAAsB,SAAQ,YAAY;IACrD,IAAI,IAAI,KAAK,OAAO,qBAAqB,CAAC,CAAC,CAAC;IAC5C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,cAAsB,EAAE,aAA6B;QACrE,MAAM,UAAU,GAAgB,EAAE,CAAC;QACnC,MAAM,aAAa,GAAG,CAAC,QAAQ,EAAE,WAAW,EAAE,YAAY,CAAC,CAAC;QAE5D,gEAAgE;QAChE,mEAAmE;QACnE,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAU,CAAC;QAE5C,gEAAgE;QAChE,IAAI,aAAa,EAAE,YAAY,IAAI,KAAK,CAAC,OAAO,CAAC,aAAa,CAAC,YAAY,CAAC,EAAE,CAAC;YAC7E,KAAK,MAAM,GAAG,IAAI,aAAa,CAAC,YAAY,EAAE,CAAC;gBAC7C,IAAI,GAAG,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;oBACpB,iBAAiB,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBAClC,CAAC;YACH,CAAC;QACH,CAAC;QAED,wDAAwD;QACxD,IAAI,iBAAiB,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;YACjC,OAAO,UAAU,CAAC;QACpB,CAAC;QAED,QAAQ,CAAC,GAAG,EAAE;YACZ,UAAU,CAAC,IAA4B;gBACrC,MAAM,cAAc,GAAG,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC;gBAChD,MAAM,WAAW,GAAG,iBAAiB,CAAC,cAAc,CAAC,CAAC;gBAEtD,yDAAyD;gBACzD,IAAI,WAAW,IAAI,iBAAiB,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,CAAC;oBACtD,2BAA2B;oBAC3B,MAAM,YAAY,GAAa,EAAE,CAAC;oBAElC,KAAK,MAAM,QAAQ,IAAI,aAAa,EAAE,CAAC;wBACrC,IAAI,CAAC,eAAe,CAAC,cAAc,EAAE,QAAQ,CAAC,EAAE,CAAC;4BAC/C,YAAY,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;wBAC9B,CAAC;oBACH,CAAC;oBAED,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;wBAC5B,UAAU,CAAC,IAAI,CACb,eAAe,CACb,qBAAqB,EACrB,UAAU,EACV,cAAc,EACd,yBAAyB,WAAW,gCAAgC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,sFAAsF,EACjL,IAAI,WAAW,SAAS,EACxB;4BACE,IAAI,EAAE,8CAA8C;4BACpD,OAAO,EAAE;;;;;;;;;;;;;GAaxB;yBACc,CACF,CACF,CAAC;oBACJ,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AA5EY,qBAAqB;IADjC,aAAa,CAAC,YAAY,EAAE,qBAAqB,CAAC;GACtC,qBAAqB,CA4EjC"}

package/dist/runtime-rules/prefer-async-await.d.ts

@@ -0,0 +1,17 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: prefer-async-await
+ *
+ * Encourages the use of async/await over .then() chains for better readability.
+ *
+ * Severity: low
+ * Applies to: all components
+ */
+export declare class PreferAsyncAwaitRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, _componentName: string): Violation[];
+}
+//# sourceMappingURL=prefer-async-await.d.ts.map

package/dist/runtime-rules/prefer-async-await.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-async-await.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/prefer-async-await.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;;;;;;;GAOG;AACH,qBACa,oBAAqB,SAAQ,YAAY;IACpD,IAAI,IAAI,WAAmC;IAC3C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,cAAc,EAAE,MAAM,GAAG,SAAS,EAAE;CA6BvD"}

package/dist/runtime-rules/prefer-async-await.js

@@ -0,0 +1,52 @@
+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: prefer-async-await
+ *
+ * Encourages the use of async/await over .then() chains for better readability.
+ *
+ * Severity: low
+ * Applies to: all components
+ */
+let PreferAsyncAwaitRule = class PreferAsyncAwaitRule extends BaseLintRule {
+    get Name() { return 'prefer-async-await'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                const callee = path.node.callee;
+                // Check for .then() chains
+                if (t.isMemberExpression(callee) && t.isIdentifier(callee.property) && callee.property.name === 'then') {
+                    // Try to get the context of what's being chained
+                    let context = '';
+                    if (t.isMemberExpression(callee.object)) {
+                        context = ' Consider using async/await for cleaner code.';
+                    }
+                    violations.push({
+                        rule: 'prefer-async-await',
+                        severity: 'low',
+                        line: callee.property.loc?.start.line || 0,
+                        column: callee.property.loc?.start.column || 0,
+                        message: `Prefer async/await over .then() chains for better readability.${context}`,
+                        code: '.then(result => ...) → const result = await ...',
+                    });
+                }
+            },
+        });
+        return violations;
+    }
+};
+PreferAsyncAwaitRule = __decorate([
+    RegisterClass(BaseLintRule, 'prefer-async-await')
+], PreferAsyncAwaitRule);
+export { PreferAsyncAwaitRule };
+//# sourceMappingURL=prefer-async-await.js.map

package/dist/runtime-rules/prefer-async-await.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-async-await.js","sourceRoot":"","sources":["../../src/runtime-rules/prefer-async-await.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;AAG5C;;;;;;;GAOG;AAEI,IAAM,oBAAoB,GAA1B,MAAM,oBAAqB,SAAQ,YAAY;IACpD,IAAI,IAAI,KAAK,OAAO,oBAAoB,CAAC,CAAC,CAAC;IAC3C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,cAAsB;QACtC,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,2BAA2B;gBAC3B,IAAI,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,MAAM,EAAE,CAAC;oBACvG,iDAAiD;oBACjD,IAAI,OAAO,GAAG,EAAE,CAAC;oBACjB,IAAI,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC;wBACxC,OAAO,GAAG,+CAA+C,CAAC;oBAC5D,CAAC;oBAED,UAAU,CAAC,IAAI,CAAC;wBACd,IAAI,EAAE,oBAAoB;wBAC1B,QAAQ,EAAE,KAAK;wBACf,IAAI,EAAE,MAAM,CAAC,QAAQ,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;wBAC1C,MAAM,EAAE,MAAM,CAAC,QAAQ,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;wBAC9C,OAAO,EAAE,iEAAiE,OAAO,EAAE;wBACnF,IAAI,EAAE,iDAAiD;qBACxD,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAjCY,oBAAoB;IADhC,aAAa,CAAC,YAAY,EAAE,oBAAoB,CAAC;GACrC,oBAAoB,CAiChC"}

package/dist/runtime-rules/prefer-jsx-syntax.d.ts

@@ -0,0 +1,17 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: prefer-jsx-syntax
+ *
+ * Encourages the use of JSX syntax over React.createElement for better readability.
+ *
+ * Severity: low
+ * Applies to: all components
+ */
+export declare class PreferJsxSyntaxRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, _componentName: string): Violation[];
+}
+//# sourceMappingURL=prefer-jsx-syntax.d.ts.map

package/dist/runtime-rules/prefer-jsx-syntax.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-jsx-syntax.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/prefer-jsx-syntax.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;;;;;;;GAOG;AACH,qBACa,mBAAoB,SAAQ,YAAY;IACnD,IAAI,IAAI,WAAkC;IAC1C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,cAAc,EAAE,MAAM,GAAG,SAAS,EAAE;CA6BvD"}

package/dist/runtime-rules/prefer-jsx-syntax.js

@@ -0,0 +1,51 @@
+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: prefer-jsx-syntax
+ *
+ * Encourages the use of JSX syntax over React.createElement for better readability.
+ *
+ * Severity: low
+ * Applies to: all components
+ */
+let PreferJsxSyntaxRule = class PreferJsxSyntaxRule extends BaseLintRule {
+    get Name() { return 'prefer-jsx-syntax'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                const callee = path.node.callee;
+                // Check for React.createElement
+                if (t.isMemberExpression(callee) &&
+                    t.isIdentifier(callee.object) &&
+                    callee.object.name === 'React' &&
+                    t.isIdentifier(callee.property) &&
+                    callee.property.name === 'createElement') {
+                    violations.push({
+                        rule: 'prefer-jsx-syntax',
+                        severity: 'low',
+                        line: callee.loc?.start.line || 0,
+                        column: callee.loc?.start.column || 0,
+                        message: 'Prefer JSX syntax over React.createElement for better readability',
+                        code: 'React.createElement(...) → <ComponentName ... />',
+                    });
+                }
+            },
+        });
+        return violations;
+    }
+};
+PreferJsxSyntaxRule = __decorate([
+    RegisterClass(BaseLintRule, 'prefer-jsx-syntax')
+], PreferJsxSyntaxRule);
+export { PreferJsxSyntaxRule };
+//# sourceMappingURL=prefer-jsx-syntax.js.map

package/dist/runtime-rules/prefer-jsx-syntax.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-jsx-syntax.js","sourceRoot":"","sources":["../../src/runtime-rules/prefer-jsx-syntax.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;AAG5C;;;;;;;GAOG;AAEI,IAAM,mBAAmB,GAAzB,MAAM,mBAAoB,SAAQ,YAAY;IACnD,IAAI,IAAI,KAAK,OAAO,mBAAmB,CAAC,CAAC,CAAC;IAC1C,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,cAAsB;QACtC,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,gCAAgC;gBAChC,IACE,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC;oBAC5B,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,MAAM,CAAC;oBAC7B,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,OAAO;oBAC9B,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,QAAQ,CAAC;oBAC/B,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,eAAe,EACxC,CAAC;oBACD,UAAU,CAAC,IAAI,CAAC;wBACd,IAAI,EAAE,mBAAmB;wBACzB,QAAQ,EAAE,KAAK;wBACf,IAAI,EAAE,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;wBACjC,MAAM,EAAE,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;wBACrC,OAAO,EAAE,mEAAmE;wBAC5E,IAAI,EAAE,kDAAkD;qBACzD,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAjCY,mBAAmB;IAD/B,aAAa,CAAC,YAAY,EAAE,mBAAmB,CAAC;GACpC,mBAAmB,CAiC/B"}

package/dist/runtime-rules/prop-state-sync.d.ts

@@ -0,0 +1,19 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: prop-state-sync
+ *
+ * Detects useEffect hooks that sync props to internal state, which creates
+ * dual state management and can lead to bugs and stale state issues.
+ * Components should use props directly or manage state independently.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class PropStateSyncRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, _componentName: string): Violation[];
+}
+//# sourceMappingURL=prop-state-sync.d.ts.map

package/dist/runtime-rules/prop-state-sync.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prop-state-sync.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/prop-state-sync.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;;;;;;;;;GASG;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,EAAE,cAAc,EAAE,MAAM,GAAG,SAAS,EAAE;CAqDvD"}

package/dist/runtime-rules/prop-state-sync.js

@@ -0,0 +1,76 @@
+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: prop-state-sync
+ *
+ * Detects useEffect hooks that sync props to internal state, which creates
+ * dual state management and can lead to bugs and stale state issues.
+ * Components should use props directly or manage state independently.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let PropStateSyncRule = class PropStateSyncRule extends BaseLintRule {
+    get Name() { return 'prop-state-sync'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                if (t.isIdentifier(path.node.callee) && path.node.callee.name === 'useEffect') {
+                    const effectBody = path.node.arguments[0];
+                    const deps = path.node.arguments[1];
+                    if (effectBody && (t.isArrowFunctionExpression(effectBody) || t.isFunctionExpression(effectBody))) {
+                        const bodyString = effectBody.body.toString();
+                        // Check if it's setting state based on props
+                        const hasSetState = /set[A-Z]\w*\s*\(/.test(bodyString);
+                        const depsString = deps ? deps.toString() : '';
+                        // Check if deps include prop-like names
+                        const propPatterns = ['Prop', 'value', 'data', 'items'];
+                        const hasPropDeps = propPatterns.some((p) => depsString.includes(p));
+                        if (hasSetState && hasPropDeps && !bodyString.includes('async')) {
+                            violations.push({
+                                rule: 'prop-state-sync',
+                                severity: 'critical',
+                                line: path.node.loc?.start.line || 0,
+                                column: path.node.loc?.start.column || 0,
+                                message: 'Syncing props to internal state with useEffect creates dual state management',
+                                code: path.toString().substring(0, 100),
+                                suggestion: {
+                                    text: "Initialize state once, don't sync from props",
+                                    example: `// ❌ WRONG - Syncing prop to state:
+const [value, setValue] = useState(propValue);
+useEffect(() => {
+  setValue(propValue); // Creates dual state management!
+}, [propValue]);
+
+// ✅ CORRECT - Initialize once:
+const [value, setValue] = useState(
+  savedUserSettings?.value || defaultValue
+);
+
+// ✅ CORRECT - If you need prop changes, use derived state:
+const displayValue = propOverride || value;`,
+                                },
+                            });
+                        }
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+PropStateSyncRule = __decorate([
+    RegisterClass(BaseLintRule, 'prop-state-sync')
+], PropStateSyncRule);
+export { PropStateSyncRule };
+//# sourceMappingURL=prop-state-sync.js.map

package/dist/runtime-rules/prop-state-sync.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prop-state-sync.js","sourceRoot":"","sources":["../../src/runtime-rules/prop-state-sync.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;AAG5C;;;;;;;;;GASG;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,EAAE,cAAsB;QACtC,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,QAAQ,CAAC,GAAG,EAAE;YACZ,cAAc,CAAC,IAAgC;gBAC7C,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;oBAC9E,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;oBAC1C,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;oBAEpC,IAAI,UAAU,IAAI,CAAC,CAAC,CAAC,yBAAyB,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,UAAU,CAAC,CAAC,EAAE,CAAC;wBAClG,MAAM,UAAU,GAAG,UAAU,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC;wBAE9C,6CAA6C;wBAC7C,MAAM,WAAW,GAAG,kBAAkB,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;wBACxD,MAAM,UAAU,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;wBAE/C,wCAAwC;wBACxC,MAAM,YAAY,GAAG,CAAC,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC;wBACxD,MAAM,WAAW,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC;wBAErE,IAAI,WAAW,IAAI,WAAW,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;4BAChE,UAAU,CAAC,IAAI,CAAC;gCACd,IAAI,EAAE,iBAAiB;gCACvB,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,8EAA8E;gCACvF,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,GAAG,CAAC;gCACvC,UAAU,EAAE;oCACV,IAAI,EAAE,8CAA8C;oCACpD,OAAO,EAAE;;;;;;;;;;;;4CAYiB;iCAC3B;6BACF,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAzDY,iBAAiB;IAD7B,aAAa,CAAC,YAAY,EAAE,iBAAiB,CAAC;GAClC,iBAAiB,CAyD7B"}