@memberjunction/react-linter 0.0.1 → 5.38.0

package/dist/runtime-rules/no-return-component.js

@@ -0,0 +1,106 @@
+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-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
+ */
+let NoReturnComponentRule = class NoReturnComponentRule extends BaseLintRule {
+    get Name() { return 'no-return-component'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        // Check for return statements at the program/top level
+        if (ast.program && ast.program.body) {
+            for (const statement of ast.program.body) {
+                // Check for return statement returning the component
+                if (t.isReturnStatement(statement)) {
+                    const argument = statement.argument;
+                    // Check if it's returning the component identifier or any identifier
+                    if (argument && t.isIdentifier(argument)) {
+                        // If it's returning the component name or any identifier at top level
+                        violations.push(createViolation('no-return-component', 'critical', statement, `Do not return the component at the end of the file. The component function should stand alone.`, `return ${argument.name};`, {
+                            text: 'Remove the return statement at the end of the file. The component function should stand alone.',
+                            example: `// ❌ WRONG - Returning the component:
+function MyComponent({ utilities, styles, components }) {
+  const [state, setState] = useState('');
+
+  return <div>{state}</div>;
+}
+
+return MyComponent; // <-- Remove this!
+
+// ❌ ALSO WRONG - Component reference at end:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+MyComponent; // <-- Remove this!
+
+// ✅ CORRECT - Just the function declaration:
+function MyComponent({ utilities, styles, components }) {
+  const [state, setState] = useState('');
+
+  return <div>{state}</div>;
+}
+// Nothing after the function - file ends here
+
+// The runtime will find and execute your component
+// by its function name. No need to return or reference it!`,
+                        }));
+                    }
+                }
+                // Also check for expression statements that might be standalone identifiers
+                if (t.isExpressionStatement(statement) && t.isIdentifier(statement.expression) && statement.expression.name === componentName) {
+                    violations.push(createViolation('no-return-component', 'critical', statement, `Do not reference the component "${componentName}" at the end of the file. The component function should stand alone.`, statement.expression.name, {
+                        text: 'Remove the return statement at the end of the file. The component function should stand alone.',
+                        example: `// ❌ WRONG - Returning the component:
+function MyComponent({ utilities, styles, components }) {
+  const [state, setState] = useState('');
+
+  return <div>{state}</div>;
+}
+
+return MyComponent; // <-- Remove this!
+
+// ❌ ALSO WRONG - Component reference at end:
+function MyComponent({ utilities, styles, components }) {
+  return <div>Hello</div>;
+}
+
+MyComponent; // <-- Remove this!
+
+// ✅ CORRECT - Just the function declaration:
+function MyComponent({ utilities, styles, components }) {
+  const [state, setState] = useState('');
+
+  return <div>{state}</div>;
+}
+// Nothing after the function - file ends here
+
+// The runtime will find and execute your component
+// by its function name. No need to return or reference it!`,
+                    }));
+                }
+            }
+        }
+        return violations;
+    }
+};
+NoReturnComponentRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-return-component')
+], NoReturnComponentRule);
+export { NoReturnComponentRule };
+//# sourceMappingURL=no-return-component.js.map

package/dist/runtime-rules/no-return-component.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-return-component.js","sourceRoot":"","sources":["../../src/runtime-rules/no-return-component.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,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,aAAqB;QACrC,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,uDAAuD;QACvD,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,qDAAqD;gBACrD,IAAI,CAAC,CAAC,iBAAiB,CAAC,SAAS,CAAC,EAAE,CAAC;oBACnC,MAAM,QAAQ,GAAG,SAAS,CAAC,QAAQ,CAAC;oBAEpC,qEAAqE;oBACrE,IAAI,QAAQ,IAAI,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,EAAE,CAAC;wBACzC,sEAAsE;wBACtE,UAAU,CAAC,IAAI,CACb,eAAe,CACb,qBAAqB,EACrB,UAAU,EACV,SAAS,EACT,gGAAgG,EAChG,UAAU,QAAQ,CAAC,IAAI,GAAG,EAC1B;4BACE,IAAI,EAAE,gGAAgG;4BACtG,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;4DAyBiC;yBAC3C,CACF,CACF,CAAC;oBACJ,CAAC;gBACH,CAAC;gBAED,4EAA4E;gBAC5E,IAAI,CAAC,CAAC,qBAAqB,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,SAAS,CAAC,UAAU,CAAC,IAAI,SAAS,CAAC,UAAU,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;oBAC9H,UAAU,CAAC,IAAI,CACb,eAAe,CACb,qBAAqB,EACrB,UAAU,EACV,SAAS,EACT,mCAAmC,aAAa,sEAAsE,EACtH,SAAS,CAAC,UAAU,CAAC,IAAI,EACzB;wBACE,IAAI,EAAE,gGAAgG;wBACtG,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;4DAyBmC;qBAC7C,CACF,CACF,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAxGY,qBAAqB;IADjC,aAAa,CAAC,YAAY,EAAE,qBAAqB,CAAC;GACtC,qBAAqB,CAwGjC"}

package/dist/runtime-rules/no-unwrap-utility-libs.d.ts

@@ -0,0 +1,20 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+/**
+ * Rule: no-unwrap-utility-libs
+ *
+ * Flags unwrapLibraryComponents() calls where the first argument is a known utility
+ * library global (`_`, `dayjs`, `d3`, `moment`). These libraries expose their API as
+ * methods on the global variable — wrapping them with unwrapLibraryComponents() makes
+ * the extracted functions non-callable, causing runtime crashes.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoUnwrapUtilityLibsRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-unwrap-utility-libs.d.ts.map

package/dist/runtime-rules/no-unwrap-utility-libs.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unwrap-utility-libs.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-unwrap-utility-libs.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;AAkBhD;;;;;;;;;;GAUG;AACH,qBACa,uBAAwB,SAAQ,YAAY;IACvD,IAAI,IAAI,WAAuC;IAC/C,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CA+CtD"}

package/dist/runtime-rules/no-unwrap-utility-libs.js

@@ -0,0 +1,75 @@
+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';
+/**
+ * Known utility libraries whose globals must NOT be passed to unwrapLibraryComponents().
+ * These libraries expose their API as methods on the global variable and should be
+ * called directly (e.g., `_.debounce(fn, 500)`, `dayjs(date).format('YYYY-MM-DD')`).
+ *
+ * unwrapLibraryComponents() wraps the value in a way that makes utility functions
+ * non-callable, causing runtime crashes like:
+ *   Uncaught TypeError: D is not a function
+ */
+const UTILITY_LIBS = new Map([
+    ['_', { library: 'lodash', example: '_.debounce(myFunction, 500)' }],
+    ['dayjs', { library: 'Day.js', example: "dayjs(date).format('YYYY-MM-DD')" }],
+    ['d3', { library: 'D3.js', example: "d3.select('#chart')" }],
+    ['moment', { library: 'Moment.js', example: "moment().format('YYYY-MM-DD')" }],
+]);
+/**
+ * Rule: no-unwrap-utility-libs
+ *
+ * Flags unwrapLibraryComponents() calls where the first argument is a known utility
+ * library global (`_`, `dayjs`, `d3`, `moment`). These libraries expose their API as
+ * methods on the global variable — wrapping them with unwrapLibraryComponents() makes
+ * the extracted functions non-callable, causing runtime crashes.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoUnwrapUtilityLibsRule = class NoUnwrapUtilityLibsRule extends BaseLintRule {
+    get Name() { return 'no-unwrap-utility-libs'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                if (!t.isIdentifier(path.node.callee) || path.node.callee.name !== 'unwrapLibraryComponents') {
+                    return;
+                }
+                const firstArg = path.node.arguments[0];
+                if (!firstArg || !t.isIdentifier(firstArg)) {
+                    return;
+                }
+                const globalName = firstArg.name;
+                const libInfo = UTILITY_LIBS.get(globalName);
+                if (!libInfo) {
+                    return;
+                }
+                const message = `Do not use unwrapLibraryComponents() with utility library "${globalName}" (${libInfo.library}). ` +
+                    `Utility libraries expose their API as methods on the global variable — call them directly ` +
+                    `(e.g., ${libInfo.example}). unwrapLibraryComponents() is only for UI component libraries like antd.`;
+                violations.push(createViolation('no-unwrap-utility-libs', 'critical', path.node, message, truncateCode(path.toString()), {
+                    text: 'Remove the unwrapLibraryComponents() call and use the library global directly.',
+                    example: '// Instead of:\n' +
+                        `const { debounce } = unwrapLibraryComponents(${globalName}, 'debounce');\n` +
+                        '// Use:\n' +
+                        `const debouncedFn = ${libInfo.example};`,
+                }));
+            },
+        });
+        return violations;
+    }
+};
+NoUnwrapUtilityLibsRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-unwrap-utility-libs')
+], NoUnwrapUtilityLibsRule);
+export { NoUnwrapUtilityLibsRule };
+//# sourceMappingURL=no-unwrap-utility-libs.js.map

package/dist/runtime-rules/no-unwrap-utility-libs.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unwrap-utility-libs.js","sourceRoot":"","sources":["../../src/runtime-rules/no-unwrap-utility-libs.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;;;;;;;;GAQG;AACH,MAAM,YAAY,GAA8D,IAAI,GAAG,CAAC;IACtF,CAAC,GAAG,EAAO,EAAE,OAAO,EAAE,QAAQ,EAAK,OAAO,EAAE,6BAA6B,EAAE,CAAC;IAC5E,CAAC,OAAO,EAAG,EAAE,OAAO,EAAE,QAAQ,EAAK,OAAO,EAAE,kCAAkC,EAAE,CAAC;IACjF,CAAC,IAAI,EAAM,EAAE,OAAO,EAAE,OAAO,EAAM,OAAO,EAAE,qBAAqB,EAAE,CAAC;IACpE,CAAC,QAAQ,EAAE,EAAE,OAAO,EAAE,WAAW,EAAE,OAAO,EAAE,+BAA+B,EAAE,CAAC;CAC/E,CAAC,CAAC;AAEH;;;;;;;;;;GAUG;AAEI,IAAM,uBAAuB,GAA7B,MAAM,uBAAwB,SAAQ,YAAY;IACvD,IAAI,IAAI,KAAK,OAAO,wBAAwB,CAAC,CAAC,CAAC;IAC/C,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,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,yBAAyB,EAAE,CAAC;oBAC7F,OAAO;gBACT,CAAC;gBAED,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;gBACxC,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,EAAE,CAAC;oBAC3C,OAAO;gBACT,CAAC;gBAED,MAAM,UAAU,GAAG,QAAQ,CAAC,IAAI,CAAC;gBACjC,MAAM,OAAO,GAAG,YAAY,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;gBAC7C,IAAI,CAAC,OAAO,EAAE,CAAC;oBACb,OAAO;gBACT,CAAC;gBAED,MAAM,OAAO,GACX,8DAA8D,UAAU,MAAM,OAAO,CAAC,OAAO,KAAK;oBAClG,4FAA4F;oBAC5F,UAAU,OAAO,CAAC,OAAO,4EAA4E,CAAC;gBAExG,UAAU,CAAC,IAAI,CACb,eAAe,CACb,wBAAwB,EACxB,UAAU,EACV,IAAI,CAAC,IAAI,EACT,OAAO,EACP,YAAY,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,EAC7B;oBACE,IAAI,EAAE,gFAAgF;oBACtF,OAAO,EACL,kBAAkB;wBAClB,gDAAgD,UAAU,kBAAkB;wBAC5E,WAAW;wBACX,uBAAuB,OAAO,CAAC,OAAO,GAAG;iBAC5C,CACF,CACF,CAAC;YACJ,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAnDY,uBAAuB;IADnC,aAAa,CAAC,YAAY,EAAE,wBAAwB,CAAC;GACzC,uBAAuB,CAmDnC"}

package/dist/runtime-rules/no-use-reducer.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: no-use-reducer
+ *
+ * Prevents usage of useReducer hook in interactive components.
+ * Components should manage state with useState and persist important
+ * settings with onSaveUserSettings instead.
+ *
+ * Severity: high (pattern violation, not a functional issue)
+ * Applies to: all components
+ */
+export declare class NoUseReducerRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string): Violation[];
+}
+//# sourceMappingURL=no-use-reducer.d.ts.map

package/dist/runtime-rules/no-use-reducer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-use-reducer.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-use-reducer.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,gBAAiB,SAAQ,YAAY;IAChD,IAAI,IAAI,WAA+B;IACvC,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,GAAG,SAAS,EAAE;CAsDtD"}

package/dist/runtime-rules/no-use-reducer.js

@@ -0,0 +1,78 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import { traverse } from '../lint-utils.js';
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: no-use-reducer
+ *
+ * Prevents usage of useReducer hook in interactive components.
+ * Components should manage state with useState and persist important
+ * settings with onSaveUserSettings instead.
+ *
+ * Severity: high (pattern violation, not a functional issue)
+ * Applies to: all components
+ */
+let NoUseReducerRule = class NoUseReducerRule extends BaseLintRule {
+    get Name() { return 'no-use-reducer'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                const callee = path.node.callee;
+                if ((t.isIdentifier(callee) && callee.name === 'useReducer') ||
+                    (t.isMemberExpression(callee) &&
+                        t.isIdentifier(callee.object) &&
+                        callee.object.name === 'React' &&
+                        t.isIdentifier(callee.property) &&
+                        callee.property.name === 'useReducer')) {
+                    violations.push({
+                        rule: 'no-use-reducer',
+                        severity: 'high', // High but not critical - it's a pattern violation
+                        line: path.node.loc?.start.line || 0,
+                        column: path.node.loc?.start.column || 0,
+                        message: `Component "${componentName}" uses useReducer at line ${path.node.loc?.start.line}. Components should manage state with useState and persist important settings with onSaveUserSettings.`,
+                        code: path.toString(),
+                        suggestion: {
+                            text: 'Use useState for state management, not useReducer',
+                            example: `// Instead of:
+const [state, dispatch] = useReducer(reducer, initialState);
+
+// Use useState:
+function Component({ savedUserSettings, onSaveUserSettings }) {
+  const [selectedId, setSelectedId] = useState(
+    savedUserSettings?.selectedId
+  );
+  const [filters, setFilters] = useState(
+    savedUserSettings?.filters || {}
+  );
+
+  // Handle actions directly
+  const handleAction = (action) => {
+    switch(action.type) {
+      case 'SELECT':
+        setSelectedId(action.payload);
+        onSaveUserSettings?.({ ...savedUserSettings, selectedId: action.payload });
+        break;
+    }
+  };
+}`,
+                        },
+                    });
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoUseReducerRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-use-reducer')
+], NoUseReducerRule);
+export { NoUseReducerRule };
+//# sourceMappingURL=no-use-reducer.js.map

package/dist/runtime-rules/no-use-reducer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-use-reducer.js","sourceRoot":"","sources":["../../src/runtime-rules/no-use-reducer.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,gBAAgB,GAAtB,MAAM,gBAAiB,SAAQ,YAAY;IAChD,IAAI,IAAI,KAAK,OAAO,gBAAgB,CAAC,CAAC,CAAC;IACvC,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,IACE,CAAC,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,CAAC;oBACxD,CAAC,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC;wBAC3B,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,MAAM,CAAC;wBAC7B,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,OAAO;wBAC9B,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,QAAQ,CAAC;wBAC/B,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY,CAAC,EACxC,CAAC;oBACD,UAAU,CAAC,IAAI,CAAC;wBACd,IAAI,EAAE,gBAAgB;wBACtB,QAAQ,EAAE,MAAM,EAAE,mDAAmD;wBACrE,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,cAAc,aAAa,6BAA6B,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,wGAAwG;wBAClM,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE;wBACrB,UAAU,EAAE;4BACV,IAAI,EAAE,mDAAmD;4BACzD,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;EAqBrB;yBACW;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AA1DY,gBAAgB;IAD5B,aAAa,CAAC,YAAY,EAAE,gBAAgB,CAAC;GACjC,gBAAgB,CA0D5B"}

package/dist/runtime-rules/no-window-access.d.ts

@@ -0,0 +1,23 @@
+import * as t from '@babel/types';
+import { BaseLintRule } from '../lint-rule.js';
+import { Violation } from '../component-linter.js';
+import { ComponentSpec } from '@memberjunction/interactive-component-types';
+/**
+ * Rule: no-window-access
+ *
+ * Prevents components from accessing the window object. Libraries should be accessed
+ * through unwrapComponents, not through window. Interactive components must be
+ * self-contained and not rely on global state.
+ *
+ * Allows export handler patterns (window.ComponentNameExport) for intentional
+ * component export functionality with proper cleanup.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoWindowAccessRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, componentName: string, componentSpec?: ComponentSpec): Violation[];
+}
+//# sourceMappingURL=no-window-access.d.ts.map

package/dist/runtime-rules/no-window-access.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-window-access.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/no-window-access.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,SAAS,EAAE,MAAM,qBAAqB,CAAC;AAChD,OAAO,EAAE,aAAa,EAAE,MAAM,6CAA6C,CAAC;AAE5E;;;;;;;;;;;;GAYG;AACH,qBACa,kBAAmB,SAAQ,YAAY;IAClD,IAAI,IAAI,WAAiC;IACzC,IAAI,SAAS,IAAI,KAAK,GAAG,OAAO,GAAG,MAAM,CAAkB;IAE3D,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,EAAE,aAAa,EAAE,MAAM,EAAE,aAAa,CAAC,EAAE,aAAa,GAAG,SAAS,EAAE;CA8GrF"}

package/dist/runtime-rules/no-window-access.js

@@ -0,0 +1,136 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+import { traverse } from '../lint-utils.js';
+import * as t from '@babel/types';
+import { RegisterClass } from '@memberjunction/global';
+import { BaseLintRule } from '../lint-rule.js';
+/**
+ * Rule: no-window-access
+ *
+ * Prevents components from accessing the window object. Libraries should be accessed
+ * through unwrapComponents, not through window. Interactive components must be
+ * self-contained and not rely on global state.
+ *
+ * Allows export handler patterns (window.ComponentNameExport) for intentional
+ * component export functionality with proper cleanup.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoWindowAccessRule = class NoWindowAccessRule extends BaseLintRule {
+    get Name() { return 'no-window-access'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, componentName, componentSpec) {
+        const violations = [];
+        // Build a map of library names to their global variables from the component spec
+        const libraryMap = new Map();
+        if (componentSpec?.libraries) {
+            for (const lib of componentSpec.libraries) {
+                // Skip empty library objects or those without required fields
+                if (lib.globalVariable && lib.name) {
+                    // Store both the library name and globalVariable for lookup
+                    libraryMap.set(lib.name.toLowerCase(), lib.globalVariable);
+                    libraryMap.set(lib.globalVariable.toLowerCase(), lib.globalVariable);
+                }
+            }
+        }
+        traverse(ast, {
+            MemberExpression(path) {
+                // Check if accessing window object
+                if (t.isIdentifier(path.node.object) && path.node.object.name === 'window') {
+                    // Check what property is being accessed from window
+                    let propertyName = '';
+                    let isDestructuring = false;
+                    if (t.isIdentifier(path.node.property)) {
+                        propertyName = path.node.property.name;
+                    }
+                    else if (t.isMemberExpression(path.node.property)) {
+                        // Handle chained access like window.Recharts.ResponsiveContainer
+                        const firstProp = path.node.property;
+                        if (t.isIdentifier(firstProp.object)) {
+                            propertyName = firstProp.object.name;
+                        }
+                    }
+                    // Check if this is part of a destructuring assignment
+                    let currentPath = path.parentPath;
+                    while (currentPath) {
+                        if (t.isVariableDeclarator(currentPath.node) && t.isObjectPattern(currentPath.node.id)) {
+                            isDestructuring = true;
+                            break;
+                        }
+                        currentPath = currentPath.parentPath;
+                    }
+                    // Check if the property matches a known library
+                    const matchedLibrary = libraryMap.get(propertyName.toLowerCase());
+                    // Allow export handler patterns: window.ComponentNameExport
+                    // These are intentional patterns for export functionality with proper cleanup
+                    const isExportPattern = propertyName.endsWith('Export') || propertyName.endsWith('Handler') || propertyName.endsWith('Callback');
+                    if (isExportPattern) {
+                        // Skip - this is an intentional pattern for component export functionality
+                        return;
+                    }
+                    if (matchedLibrary) {
+                        // Specific guidance for library access
+                        violations.push({
+                            rule: 'no-window-access',
+                            severity: 'critical',
+                            line: path.node.loc?.start.line || 0,
+                            column: path.node.loc?.start.column || 0,
+                            message: `Component "${componentName}" should not access window.${propertyName}. Use "${matchedLibrary}" directly - it's already available in the component's closure scope. Change "window.${propertyName}" to just "${matchedLibrary}".`,
+                            code: path.toString().substring(0, 100),
+                        });
+                    }
+                    else if (isDestructuring) {
+                        // Likely trying to destructure from an unknown library
+                        violations.push({
+                            rule: 'no-window-access',
+                            severity: 'critical',
+                            line: path.node.loc?.start.line || 0,
+                            column: path.node.loc?.start.column || 0,
+                            message: `Component "${componentName}" is trying to access window.${propertyName}. Libraries must be accessed using unwrapComponents, not through the window object. If this library is in your spec, use: const { ... } = unwrapComponents(${propertyName}, [...]); If it's not in your spec, you cannot use it.`,
+                            code: path.toString().substring(0, 100),
+                        });
+                    }
+                    else {
+                        // General window access
+                        violations.push({
+                            rule: 'no-window-access',
+                            severity: 'critical',
+                            line: path.node.loc?.start.line || 0,
+                            column: path.node.loc?.start.column || 0,
+                            message: `Component "${componentName}" must not access the window object. Interactive components should be self-contained and not rely on global state.`,
+                            code: path.toString().substring(0, 100),
+                        });
+                    }
+                }
+            },
+            Identifier(path) {
+                // Also check for direct window references (less common but possible)
+                if (path.node.name === 'window' && path.isReferencedIdentifier()) {
+                    // Make sure it's not part of a member expression we already caught
+                    const parent = path.parent;
+                    if (!t.isMemberExpression(parent) || parent.object !== path.node) {
+                        violations.push({
+                            rule: 'no-window-access',
+                            severity: 'critical',
+                            line: path.node.loc?.start.line || 0,
+                            column: path.node.loc?.start.column || 0,
+                            message: `Component "${componentName}" must not reference the window object directly. Interactive components should be self-contained.`,
+                            code: path.toString().substring(0, 100),
+                        });
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoWindowAccessRule = __decorate([
+    RegisterClass(BaseLintRule, 'no-window-access')
+], NoWindowAccessRule);
+export { NoWindowAccessRule };
+//# sourceMappingURL=no-window-access.js.map

package/dist/runtime-rules/no-window-access.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-window-access.js","sourceRoot":"","sources":["../../src/runtime-rules/no-window-access.ts"],"names":[],"mappings":";;;;;;AAAA,OAAO,EAAE,QAAQ,EAAY,MAAM,eAAe,CAAC;AACnD,OAAO,KAAK,CAAC,MAAM,cAAc,CAAC;AAClC,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAI5C;;;;;;;;;;;;GAYG;AAEI,IAAM,kBAAkB,GAAxB,MAAM,kBAAmB,SAAQ,YAAY;IAClD,IAAI,IAAI,KAAK,OAAO,kBAAkB,CAAC,CAAC,CAAC;IACzC,IAAI,SAAS,KAA+B,OAAO,KAAK,CAAC,CAAC,CAAC;IAE3D,IAAI,CAAC,GAAW,EAAE,aAAqB,EAAE,aAA6B;QACpE,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,iFAAiF;QACjF,MAAM,UAAU,GAAG,IAAI,GAAG,EAAkB,CAAC;QAC7C,IAAI,aAAa,EAAE,SAAS,EAAE,CAAC;YAC7B,KAAK,MAAM,GAAG,IAAI,aAAa,CAAC,SAAS,EAAE,CAAC;gBAC1C,8DAA8D;gBAC9D,IAAI,GAAG,CAAC,cAAc,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;oBACnC,4DAA4D;oBAC5D,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,EAAE,GAAG,CAAC,cAAc,CAAC,CAAC;oBAC3D,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,cAAc,CAAC,WAAW,EAAE,EAAE,GAAG,CAAC,cAAc,CAAC,CAAC;gBACvE,CAAC;YACH,CAAC;QACH,CAAC;QAED,QAAQ,CAAC,GAAG,EAAE;YACZ,gBAAgB,CAAC,IAAkC;gBACjD,mCAAmC;gBACnC,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;oBAC3E,oDAAoD;oBACpD,IAAI,YAAY,GAAG,EAAE,CAAC;oBACtB,IAAI,eAAe,GAAG,KAAK,CAAC;oBAE5B,IAAI,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;wBACvC,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC;oBACzC,CAAC;yBAAM,IAAI,CAAC,CAAC,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;wBACpD,iEAAiE;wBACjE,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC;wBACrC,IAAI,CAAC,CAAC,YAAY,CAAC,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC;4BACrC,YAAY,GAAG,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC;wBACvC,CAAC;oBACH,CAAC;oBAED,sDAAsD;oBACtD,IAAI,WAAW,GAA4B,IAAI,CAAC,UAAU,CAAC;oBAC3D,OAAO,WAAW,EAAE,CAAC;wBACnB,IAAI,CAAC,CAAC,oBAAoB,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,eAAe,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAC;4BACvF,eAAe,GAAG,IAAI,CAAC;4BACvB,MAAM;wBACR,CAAC;wBACD,WAAW,GAAG,WAAW,CAAC,UAAU,CAAC;oBACvC,CAAC;oBAED,gDAAgD;oBAChD,MAAM,cAAc,GAAG,UAAU,CAAC,GAAG,CAAC,YAAY,CAAC,WAAW,EAAE,CAAC,CAAC;oBAElE,4DAA4D;oBAC5D,8EAA8E;oBAC9E,MAAM,eAAe,GAAG,YAAY,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,YAAY,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,YAAY,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC;oBAEjI,IAAI,eAAe,EAAE,CAAC;wBACpB,2EAA2E;wBAC3E,OAAO;oBACT,CAAC;oBAED,IAAI,cAAc,EAAE,CAAC;wBACnB,uCAAuC;wBACvC,UAAU,CAAC,IAAI,CAAC;4BACd,IAAI,EAAE,kBAAkB;4BACxB,QAAQ,EAAE,UAAU;4BACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;4BACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;4BACxC,OAAO,EAAE,cAAc,aAAa,8BAA8B,YAAY,UAAU,cAAc,wFAAwF,YAAY,cAAc,cAAc,IAAI;4BAC1O,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,GAAG,CAAC;yBACxC,CAAC,CAAC;oBACL,CAAC;yBAAM,IAAI,eAAe,EAAE,CAAC;wBAC3B,uDAAuD;wBACvD,UAAU,CAAC,IAAI,CAAC;4BACd,IAAI,EAAE,kBAAkB;4BACxB,QAAQ,EAAE,UAAU;4BACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;4BACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;4BACxC,OAAO,EAAE,cAAc,aAAa,gCAAgC,YAAY,8JAA8J,YAAY,wDAAwD;4BAClT,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,GAAG,CAAC;yBACxC,CAAC,CAAC;oBACL,CAAC;yBAAM,CAAC;wBACN,wBAAwB;wBACxB,UAAU,CAAC,IAAI,CAAC;4BACd,IAAI,EAAE,kBAAkB;4BACxB,QAAQ,EAAE,UAAU;4BACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;4BACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;4BACxC,OAAO,EAAE,cAAc,aAAa,oHAAoH;4BACxJ,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,GAAG,CAAC;yBACxC,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;YACD,UAAU,CAAC,IAA4B;gBACrC,qEAAqE;gBACrE,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ,IAAI,IAAI,CAAC,sBAAsB,EAAE,EAAE,CAAC;oBACjE,mEAAmE;oBACnE,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;oBAC3B,IAAI,CAAC,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,MAAM,KAAK,IAAI,CAAC,IAAI,EAAE,CAAC;wBACjE,UAAU,CAAC,IAAI,CAAC;4BACd,IAAI,EAAE,kBAAkB;4BACxB,QAAQ,EAAE,UAAU;4BACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;4BACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;4BACxC,OAAO,EAAE,cAAc,aAAa,mGAAmG;4BACvI,IAAI,EAAE,IAAI,CAAC,QAAQ,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,GAAG,CAAC;yBACxC,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAlHY,kBAAkB;IAD9B,aAAa,CAAC,YAAY,EAAE,kBAAkB,CAAC;GACnC,kBAAkB,CAkH9B"}

package/dist/runtime-rules/noisy-settings-updates.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: noisy-settings-updates
+ *
+ * Prevents saving settings on every change/keystroke. Settings should only be
+ * saved on blur, submit, or after debouncing to avoid excessive updates.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+export declare class NoisySettingsUpdatesRule extends BaseLintRule {
+    get Name(): string;
+    get AppliesTo(): 'all' | 'child' | 'root';
+    Test(ast: t.File, _componentName: string): Violation[];
+}
+//# sourceMappingURL=noisy-settings-updates.d.ts.map

package/dist/runtime-rules/noisy-settings-updates.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"noisy-settings-updates.d.ts","sourceRoot":"","sources":["../../src/runtime-rules/noisy-settings-updates.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;AAgChD;;;;;;;;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,EAAE,cAAc,EAAE,MAAM,GAAG,SAAS,EAAE;CA4DvD"}

package/dist/runtime-rules/noisy-settings-updates.js

@@ -0,0 +1,110 @@
+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';
+/**
+ * Helper function to extract function name from a NodePath
+ */
+function getFunctionName(path) {
+    const node = path.node;
+    // Check for named function
+    if (t.isFunctionDeclaration(node) && node.id) {
+        return node.id.name;
+    }
+    // Check for arrow function assigned to variable
+    if (t.isArrowFunctionExpression(node) || t.isFunctionExpression(node)) {
+        const parent = path.parent;
+        if (t.isVariableDeclarator(parent) && t.isIdentifier(parent.id)) {
+            return parent.id.name;
+        }
+    }
+    // Check for function assigned as property
+    if (t.isArrowFunctionExpression(node) || t.isFunctionExpression(node)) {
+        const parent = path.parent;
+        if (t.isObjectProperty(parent) && t.isIdentifier(parent.key)) {
+            return parent.key.name;
+        }
+    }
+    return null;
+}
+/**
+ * Rule: noisy-settings-updates
+ *
+ * Prevents saving settings on every change/keystroke. Settings should only be
+ * saved on blur, submit, or after debouncing to avoid excessive updates.
+ *
+ * Severity: critical
+ * Applies to: all components
+ */
+let NoisySettingsUpdatesRule = class NoisySettingsUpdatesRule extends BaseLintRule {
+    get Name() { return 'noisy-settings-updates'; }
+    get AppliesTo() { return 'all'; }
+    Test(ast, _componentName) {
+        const violations = [];
+        traverse(ast, {
+            CallExpression(path) {
+                // Check for onSaveUserSettings calls
+                if (t.isOptionalCallExpression(path.node) || t.isCallExpression(path.node)) {
+                    const callee = path.node.callee;
+                    if (t.isIdentifier(callee) && callee.name === 'onSaveUserSettings') {
+                        // Check if this is inside an onChange/onInput handler
+                        let parent = path.getFunctionParent();
+                        if (parent) {
+                            const funcName = getFunctionName(parent);
+                            if (funcName && (funcName.includes('Change') || funcName.includes('Input'))) {
+                                // Check if it's not debounced or on blur
+                                const parentBody = parent.node.body;
+                                const hasDebounce = parentBody && parentBody.toString().includes('debounce');
+                                const hasTimeout = parentBody && parentBody.toString().includes('setTimeout');
+                                if (!hasDebounce && !hasTimeout) {
+                                    violations.push({
+                                        rule: 'noisy-settings-updates',
+                                        severity: 'critical',
+                                        line: path.node.loc?.start.line || 0,
+                                        column: path.node.loc?.start.column || 0,
+                                        message: `Saving settings on every change/keystroke. Save on blur, submit, or after debouncing.`,
+                                        suggestion: {
+                                            text: 'Save settings sparingly - only on meaningful user actions',
+                                            example: `// ❌ WRONG - Saving on every keystroke:
+const handleSearchChange = (e) => {
+  setSearchTerm(e.target.value);
+  onSaveUserSettings?.({ searchTerm: e.target.value }); // TOO NOISY!
+};
+
+// ✅ CORRECT - Save on blur or debounced:
+const handleSearchBlur = () => {
+  if (searchTerm !== savedUserSettings?.searchTerm) {
+    onSaveUserSettings?.({ ...savedUserSettings, searchTerm });
+  }
+};
+
+// ✅ CORRECT - Debounced save:
+const saveSearchTerm = useMemo(() =>
+  debounce((term) => {
+    onSaveUserSettings?.({ ...savedUserSettings, searchTerm: term });
+  }, 500),
+  [savedUserSettings]
+);`,
+                                        },
+                                    });
+                                }
+                            }
+                        }
+                    }
+                }
+            },
+        });
+        return violations;
+    }
+};
+NoisySettingsUpdatesRule = __decorate([
+    RegisterClass(BaseLintRule, 'noisy-settings-updates')
+], NoisySettingsUpdatesRule);
+export { NoisySettingsUpdatesRule };
+//# sourceMappingURL=noisy-settings-updates.js.map

package/dist/runtime-rules/noisy-settings-updates.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"noisy-settings-updates.js","sourceRoot":"","sources":["../../src/runtime-rules/noisy-settings-updates.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;;GAEG;AACH,SAAS,eAAe,CAAC,IAAc;IACrC,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC;IAEvB,2BAA2B;IAC3B,IAAI,CAAC,CAAC,qBAAqB,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,EAAE,EAAE,CAAC;QAC7C,OAAO,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC;IACtB,CAAC;IAED,gDAAgD;IAChD,IAAI,CAAC,CAAC,yBAAyB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,IAAI,CAAC,EAAE,CAAC;QACtE,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;QAC3B,IAAI,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC;YAChE,OAAO,MAAM,CAAC,EAAE,CAAC,IAAI,CAAC;QACxB,CAAC;IACH,CAAC;IAED,0CAA0C;IAC1C,IAAI,CAAC,CAAC,yBAAyB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,IAAI,CAAC,EAAE,CAAC;QACtE,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;QAC3B,IAAI,CAAC,CAAC,gBAAgB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;YAC7D,OAAO,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC;QACzB,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;;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,EAAE,cAAsB;QACtC,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,QAAQ,CAAC,GAAG,EAAE;YACZ,cAAc,CAAC,IAAgC;gBAC7C,qCAAqC;gBACrC,IAAI,CAAC,CAAC,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC3E,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC;oBAChC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,oBAAoB,EAAE,CAAC;wBACnE,sDAAsD;wBACtD,IAAI,MAAM,GAAG,IAAI,CAAC,iBAAiB,EAAE,CAAC;wBACtC,IAAI,MAAM,EAAE,CAAC;4BACX,MAAM,QAAQ,GAAG,eAAe,CAAC,MAAM,CAAC,CAAC;4BACzC,IAAI,QAAQ,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,EAAE,CAAC;gCAC5E,yCAAyC;gCACzC,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC;gCACpC,MAAM,WAAW,GAAG,UAAU,IAAI,UAAU,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC;gCAC7E,MAAM,UAAU,GAAG,UAAU,IAAI,UAAU,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC;gCAE9E,IAAI,CAAC,WAAW,IAAI,CAAC,UAAU,EAAE,CAAC;oCAChC,UAAU,CAAC,IAAI,CAAC;wCACd,IAAI,EAAE,wBAAwB;wCAC9B,QAAQ,EAAE,UAAU;wCACpB,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC;wCACpC,MAAM,EAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,MAAM,IAAI,CAAC;wCACxC,OAAO,EAAE,uFAAuF;wCAChG,UAAU,EAAE;4CACV,IAAI,EAAE,2DAA2D;4CACjE,OAAO,EAAE;;;;;;;;;;;;;;;;;;;GAmB5B;yCACkB;qCACF,CAAC,CAAC;gCACL,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC,CAAC;QAEH,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAA;AAhEY,wBAAwB;IADpC,aAAa,CAAC,YAAY,EAAE,wBAAwB,CAAC;GACzC,wBAAwB,CAgEpC"}