@azerothjs/eslint-plugin 0.5.0-beta.1

package/dist/ast.d.ts

@@ -0,0 +1,42 @@
+/** The slice of ESTree the rules touch, structurally typed. */
+export interface AstNode {
+    type: string;
+    parent?: AstNode;
+    [key: string]: unknown;
+}
+/** A `name` carrier (Identifier). */
+export interface IdentifierNode extends AstNode {
+    type: 'Identifier';
+    name: string;
+}
+/** A call with a callee and arguments. */
+export interface CallNode extends AstNode {
+    type: 'CallExpression';
+    callee: AstNode;
+    arguments: AstNode[];
+}
+/** True when `node` is an Identifier with the given name. */
+export declare function isIdentifier(node: AstNode | undefined, name?: string): node is IdentifierNode;
+/** True when `node` is a call of the named identifier: `name(...)`. */
+export declare function isCallTo(node: AstNode | undefined, name: string): node is CallNode;
+/** A function expression of any flavor. */
+export declare function isFunctionNode(node: AstNode | undefined): boolean;
+/**
+ * Walks ancestors from `node` upward, returning the first one matching
+ * `predicate`, stopping (exclusive) at `boundary`.
+ */
+export declare function findAncestor(node: AstNode, predicate: (ancestor: AstNode) => boolean, boundary?: AstNode): AstNode | null;
+/**
+ * Collects the getter->setter pairs declared in the file via
+ * `const [get, set] = createSignal(...)`. Keyed both ways for the rules'
+ * lookups.
+ */
+export interface SignalPairs {
+    /** setter name -> getter name */
+    getterOf: Map<string, string>;
+    /** getter names */
+    getters: Set<string>;
+}
+/** Records a `const [g, s] = createSignal(...)` declarator into `pairs`. */
+export declare function collectSignalPair(declarator: AstNode, pairs: SignalPairs): void;
+//# sourceMappingURL=ast.d.ts.map

package/dist/ast.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"ast.d.ts","sourceRoot":"","sources":["../src/ast.ts"],"names":[],"mappings":"AAOA,+DAA+D;AAC/D,MAAM,WAAW,OAAO;IAEpB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;CAC1B;AAED,qCAAqC;AACrC,MAAM,WAAW,cAAe,SAAQ,OAAO;IAE3C,IAAI,EAAE,YAAY,CAAC;IACnB,IAAI,EAAE,MAAM,CAAC;CAChB;AAED,0CAA0C;AAC1C,MAAM,WAAW,QAAS,SAAQ,OAAO;IAErC,IAAI,EAAE,gBAAgB,CAAC;IACvB,MAAM,EAAE,OAAO,CAAC;IAChB,SAAS,EAAE,OAAO,EAAE,CAAC;CACxB;AAED,6DAA6D;AAC7D,wBAAgB,YAAY,CAAC,IAAI,EAAE,OAAO,GAAG,SAAS,EAAE,IAAI,CAAC,EAAE,MAAM,GAAG,IAAI,IAAI,cAAc,CAI7F;AAED,uEAAuE;AACvE,wBAAgB,QAAQ,CAAC,IAAI,EAAE,OAAO,GAAG,SAAS,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,IAAI,QAAQ,CAIlF;AAED,2CAA2C;AAC3C,wBAAgB,cAAc,CAAC,IAAI,EAAE,OAAO,GAAG,SAAS,GAAG,OAAO,CAOjE;AAED;;;GAGG;AACH,wBAAgB,YAAY,CACxB,IAAI,EAAE,OAAO,EACb,SAAS,EAAE,CAAC,QAAQ,EAAE,OAAO,KAAK,OAAO,EACzC,QAAQ,CAAC,EAAE,OAAO,GACnB,OAAO,GAAG,IAAI,CAYhB;AAED;;;;GAIG;AACH,MAAM,WAAW,WAAW;IAExB,iCAAiC;IACjC,QAAQ,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAE9B,mBAAmB;IACnB,OAAO,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;CACxB;AAED,4EAA4E;AAC5E,wBAAgB,iBAAiB,CAAC,UAAU,EAAE,OAAO,EAAE,KAAK,EAAE,WAAW,GAAG,IAAI,CAoB/E"}

package/dist/ast.js

@@ -0,0 +1,55 @@
+// Minimal structural AST types and helpers shared by the rules. The rules
+// are deliberately SYNTACTIC: they track signals from
+// `const [x, setX] = createSignal(...)` destructuring by name, the same
+// approach eslint-plugin-solid takes. That keeps them parser-agnostic (no
+// type-services project wiring for consumers) at the cost of not seeing
+// through aliases or imports-as - the trade is documented per rule.
+/** True when `node` is an Identifier with the given name. */
+export function isIdentifier(node, name) {
+    return node !== undefined && node.type === 'Identifier'
+        && (name === undefined || node.name === name);
+}
+/** True when `node` is a call of the named identifier: `name(...)`. */
+export function isCallTo(node, name) {
+    return node !== undefined && node.type === 'CallExpression'
+        && isIdentifier(node.callee, name);
+}
+/** A function expression of any flavor. */
+export function isFunctionNode(node) {
+    return node !== undefined && (node.type === 'ArrowFunctionExpression' ||
+        node.type === 'FunctionExpression' ||
+        node.type === 'FunctionDeclaration');
+}
+/**
+ * Walks ancestors from `node` upward, returning the first one matching
+ * `predicate`, stopping (exclusive) at `boundary`.
+ */
+export function findAncestor(node, predicate, boundary) {
+    let current = node.parent;
+    while (current !== undefined && current !== boundary) {
+        if (predicate(current)) {
+            return current;
+        }
+        current = current.parent;
+    }
+    return null;
+}
+/** Records a `const [g, s] = createSignal(...)` declarator into `pairs`. */
+export function collectSignalPair(declarator, pairs) {
+    const init = declarator.init;
+    if (!isCallTo(init, 'createSignal')) {
+        return;
+    }
+    const id = declarator.id;
+    if (id === undefined || id.type !== 'ArrayPattern') {
+        return;
+    }
+    const elements = id.elements;
+    const getter = elements[0];
+    const setter = elements[1];
+    if (getter !== null && setter !== null && isIdentifier(getter ?? undefined) && isIdentifier(setter ?? undefined)) {
+        pairs.getterOf.set(setter.name, getter.name);
+        pairs.getters.add(getter.name);
+    }
+}
+//# sourceMappingURL=ast.js.map

package/dist/ast.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ast.js","sourceRoot":"","sources":["../src/ast.ts"],"names":[],"mappings":"AAAA,0EAA0E;AAC1E,sDAAsD;AACtD,wEAAwE;AACxE,0EAA0E;AAC1E,wEAAwE;AACxE,oEAAoE;AAyBpE,6DAA6D;AAC7D,MAAM,UAAU,YAAY,CAAC,IAAyB,EAAE,IAAa;IAEjE,OAAO,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,IAAI,KAAK,YAAY;WAChD,CAAC,IAAI,KAAK,SAAS,IAAK,IAAuB,CAAC,IAAI,KAAK,IAAI,CAAC,CAAC;AAC1E,CAAC;AAED,uEAAuE;AACvE,MAAM,UAAU,QAAQ,CAAC,IAAyB,EAAE,IAAY;IAE5D,OAAO,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,IAAI,KAAK,gBAAgB;WACpD,YAAY,CAAE,IAAiB,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;AACzD,CAAC;AAED,2CAA2C;AAC3C,MAAM,UAAU,cAAc,CAAC,IAAyB;IAEpD,OAAO,IAAI,KAAK,SAAS,IAAI,CACzB,IAAI,CAAC,IAAI,KAAK,yBAAyB;QACvC,IAAI,CAAC,IAAI,KAAK,oBAAoB;QAClC,IAAI,CAAC,IAAI,KAAK,qBAAqB,CACtC,CAAC;AACN,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CACxB,IAAa,EACb,SAAyC,EACzC,QAAkB;IAGlB,IAAI,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC;IAC1B,OAAO,OAAO,KAAK,SAAS,IAAI,OAAO,KAAK,QAAQ,EACpD,CAAC;QACG,IAAI,SAAS,CAAC,OAAO,CAAC,EACtB,CAAC;YACG,OAAO,OAAO,CAAC;QACnB,CAAC;QACD,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC;IAC7B,CAAC;IACD,OAAO,IAAI,CAAC;AAChB,CAAC;AAgBD,4EAA4E;AAC5E,MAAM,UAAU,iBAAiB,CAAC,UAAmB,EAAE,KAAkB;IAErE,MAAM,IAAI,GAAG,UAAU,CAAC,IAA2B,CAAC;IACpD,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,cAAc,CAAC,EACnC,CAAC;QACG,OAAO;IACX,CAAC;IACD,MAAM,EAAE,GAAG,UAAU,CAAC,EAAyB,CAAC;IAChD,IAAI,EAAE,KAAK,SAAS,IAAI,EAAE,CAAC,IAAI,KAAK,cAAc,EAClD,CAAC;QACG,OAAO;IACX,CAAC;IACD,MAAM,QAAQ,GAAG,EAAE,CAAC,QAA8B,CAAC;IACnD,MAAM,MAAM,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;IAC3B,MAAM,MAAM,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;IAC3B,IAAI,MAAM,KAAK,IAAI,IAAI,MAAM,KAAK,IAAI,IAAI,YAAY,CAAC,MAAM,IAAI,SAAS,CAAC,IAAI,YAAY,CAAC,MAAM,IAAI,SAAS,CAAC,EAChH,CAAC;QACG,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAE,MAAyB,CAAC,IAAI,EAAG,MAAyB,CAAC,IAAI,CAAC,CAAC;QACrF,KAAK,CAAC,OAAO,CAAC,GAAG,CAAE,MAAyB,CAAC,IAAI,CAAC,CAAC;IACvD,CAAC;AACL,CAAC"}

package/dist/index.d.ts

@@ -0,0 +1,10 @@
+import type { ESLint, Linter, Rule } from 'eslint';
+declare const rules: Record<string, Rule.RuleModule>;
+declare const plugin: ESLint.Plugin & {
+    configs: {
+        recommended: Linter.Config;
+    };
+};
+export default plugin;
+export { rules };
+//# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAKnD,QAAA,MAAM,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,IAAI,CAAC,UAAU,CAI1C,CAAC;AAEF,QAAA,MAAM,MAAM,EAAE,MAAM,CAAC,MAAM,GAAG;IAAE,OAAO,EAAE;QAAE,WAAW,EAAE,MAAM,CAAC,MAAM,CAAA;KAAE,CAAA;CAWtE,CAAC;AAiBF,eAAe,MAAM,CAAC;AACtB,OAAO,EAAE,KAAK,EAAE,CAAC"}

package/dist/index.js

@@ -0,0 +1,44 @@
+// @azerothjs/eslint-plugin: the reactivity foot-guns in plain .ts files
+// (markup in .azeroth files is covered by the compiler's lint, surfaced in
+// the editor and the Vite build). The rules are SYNTACTIC by design -
+// signals are tracked from `const [x, setX] = createSignal(...)`
+// destructuring by name, so consumers need no type-services project
+// wiring. The trade: aliased or re-exported signals are invisible.
+//
+// Flat-config use:
+//
+//     import azeroth from '@azerothjs/eslint-plugin';
+//     export default [azeroth.configs.recommended];
+import { noSelfWriteInEffect } from "./rules/no-self-write-in-effect.js";
+import { requireEffectDisposal } from "./rules/require-effect-disposal.js";
+import { handlerCall } from "./rules/handler-call.js";
+const rules = {
+    'no-self-write-in-effect': noSelfWriteInEffect,
+    'require-effect-disposal': requireEffectDisposal,
+    'handler-call': handlerCall
+};
+const plugin = {
+    meta: {
+        name: '@azerothjs/eslint-plugin',
+        version: '0.4.0-beta.3'
+    },
+    rules,
+    configs: {
+        recommended: {}
+    }
+};
+// The recommended config references the plugin object itself, so it is
+// attached after construction.
+plugin.configs.recommended = {
+    plugins: {
+        azeroth: plugin
+    },
+    rules: {
+        'azeroth/no-self-write-in-effect': 'warn',
+        'azeroth/require-effect-disposal': 'warn',
+        'azeroth/handler-call': 'warn'
+    }
+};
+export default plugin;
+export { rules };
+//# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,wEAAwE;AACxE,2EAA2E;AAC3E,sEAAsE;AACtE,iEAAiE;AACjE,oEAAoE;AACpE,mEAAmE;AACnE,EAAE;AACF,mBAAmB;AACnB,EAAE;AACF,sDAAsD;AACtD,oDAAoD;AAGpD,OAAO,EAAE,mBAAmB,EAAE,MAAM,oCAAoC,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,MAAM,oCAAoC,CAAC;AAC3E,OAAO,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAEtD,MAAM,KAAK,GAAoC;IAC3C,yBAAyB,EAAE,mBAAmB;IAC9C,yBAAyB,EAAE,qBAAqB;IAChD,cAAc,EAAE,WAAW;CAC9B,CAAC;AAEF,MAAM,MAAM,GAAgE;IACxE,IAAI,EACJ;QACI,IAAI,EAAE,0BAA0B;QAChC,OAAO,EAAE,cAAc;KAC1B;IACD,KAAK;IACL,OAAO,EACP;QACI,WAAW,EAAE,EAAmB;KACnC;CACJ,CAAC;AAEF,uEAAuE;AACvE,+BAA+B;AAC/B,MAAM,CAAC,OAAO,CAAC,WAAW,GAAG;IACzB,OAAO,EACP;QACI,OAAO,EAAE,MAAM;KAClB;IACD,KAAK,EACL;QACI,iCAAiC,EAAE,MAAM;QACzC,iCAAiC,EAAE,MAAM;QACzC,sBAAsB,EAAE,MAAM;KACjC;CACJ,CAAC;AAEF,eAAe,MAAM,CAAC;AACtB,OAAO,EAAE,KAAK,EAAE,CAAC"}

package/dist/rules/handler-call.d.ts

@@ -0,0 +1,3 @@
+import type { Rule } from 'eslint';
+export declare const handlerCall: Rule.RuleModule;
+//# sourceMappingURL=handler-call.d.ts.map

package/dist/rules/handler-call.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"handler-call.d.ts","sourceRoot":"","sources":["../../src/rules/handler-call.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAuBnC,eAAO,MAAM,WAAW,EAAE,IAAI,CAAC,UA8D9B,CAAC"}

package/dist/rules/handler-call.js

@@ -0,0 +1,68 @@
+// azeroth/handler-call: the hyperscript twin of the compiler's
+// azeroth/handler-call markup rule. `h('button', { onClick: save() })`
+// calls save() while BUILDING the element and passes its result as the
+// handler. Zero-argument calls of a plain reference are flagged;
+// `onClick: makeHandler(id)` (the factory idiom) stays silent.
+import { isCallTo, isIdentifier } from "../ast.js";
+/** `save` or `actions.reset` - a bare callable reference. */
+function isBareReference(node) {
+    if (node === undefined) {
+        return false;
+    }
+    if (node.type === 'Identifier') {
+        return true;
+    }
+    if (node.type === 'MemberExpression') {
+        return (node.computed !== true)
+            && isBareReference(node.object)
+            && isIdentifier(node.property);
+    }
+    return false;
+}
+export const handlerCall = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'disallow zero-argument calls passed as h() event handlers'
+        },
+        messages: {
+            called: '{{key}}: {{callee}}() runs while the element is being built and passes its RESULT as the handler. Use {{key}}: {{callee}} or {{key}}: () => {{callee}}().'
+        },
+        schema: []
+    },
+    create(context) {
+        return {
+            CallExpression(node) {
+                const call = node;
+                if (!isCallTo(call, 'h')) {
+                    return;
+                }
+                const props = call.arguments[1];
+                if (props === undefined || props.type !== 'ObjectExpression') {
+                    return;
+                }
+                for (const property of props.properties) {
+                    if (property.type !== 'Property' || property.computed === true) {
+                        continue;
+                    }
+                    const key = property.key;
+                    if (!isIdentifier(key) || !/^on[A-Z]/.test(key.name)) {
+                        continue;
+                    }
+                    const value = property.value;
+                    if (value.type === 'CallExpression' &&
+                        value.arguments.length === 0 &&
+                        isBareReference(value.callee)) {
+                        const source = context.sourceCode.getText(value.callee);
+                        context.report({
+                            node: value,
+                            messageId: 'called',
+                            data: { key: key.name, callee: source }
+                        });
+                    }
+                }
+            }
+        };
+    }
+};
+//# sourceMappingURL=handler-call.js.map

package/dist/rules/handler-call.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"handler-call.js","sourceRoot":"","sources":["../../src/rules/handler-call.ts"],"names":[],"mappings":"AAAA,+DAA+D;AAC/D,uEAAuE;AACvE,uEAAuE;AACvE,iEAAiE;AACjE,+DAA+D;AAG/D,OAAO,EAA+B,QAAQ,EAAE,YAAY,EAAE,MAAM,WAAW,CAAC;AAEhF,6DAA6D;AAC7D,SAAS,eAAe,CAAC,IAAyB;IAE9C,IAAI,IAAI,KAAK,SAAS,EACtB,CAAC;QACG,OAAO,KAAK,CAAC;IACjB,CAAC;IACD,IAAI,IAAI,CAAC,IAAI,KAAK,YAAY,EAC9B,CAAC;QACG,OAAO,IAAI,CAAC;IAChB,CAAC;IACD,IAAI,IAAI,CAAC,IAAI,KAAK,kBAAkB,EACpC,CAAC;QACG,OAAO,CAAC,IAAI,CAAC,QAAQ,KAAK,IAAI,CAAC;eACxB,eAAe,CAAC,IAAI,CAAC,MAAiB,CAAC;eACvC,YAAY,CAAC,IAAI,CAAC,QAAmB,CAAC,CAAC;IAClD,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC;AAED,MAAM,CAAC,MAAM,WAAW,GAAoB;IACxC,IAAI,EACJ;QACI,IAAI,EAAE,SAAS;QACf,IAAI,EACJ;YACI,WAAW,EAAE,2DAA2D;SAC3E;QACD,QAAQ,EACR;YACI,MAAM,EAAE,2JAA2J;SACtK;QACD,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAO;QAEV,OAAO;YACH,cAAc,CAAC,IAAI;gBAEf,MAAM,IAAI,GAAG,IAA2B,CAAC;gBACzC,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,GAAG,CAAC,EACxB,CAAC;oBACG,OAAO;gBACX,CAAC;gBACD,MAAM,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;gBAChC,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,CAAC,IAAI,KAAK,kBAAkB,EAC5D,CAAC;oBACG,OAAO;gBACX,CAAC;gBAED,KAAK,MAAM,QAAQ,IAAI,KAAK,CAAC,UAAuB,EACpD,CAAC;oBACG,IAAI,QAAQ,CAAC,IAAI,KAAK,UAAU,IAAI,QAAQ,CAAC,QAAQ,KAAK,IAAI,EAC9D,CAAC;wBACG,SAAS;oBACb,CAAC;oBACD,MAAM,GAAG,GAAG,QAAQ,CAAC,GAAc,CAAC;oBACpC,IAAI,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,EACpD,CAAC;wBACG,SAAS;oBACb,CAAC;oBACD,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAgB,CAAC;oBACxC,IACI,KAAK,CAAC,IAAI,KAAK,gBAAgB;wBAC9B,KAAkB,CAAC,SAAS,CAAC,MAAM,KAAK,CAAC;wBAC1C,eAAe,CAAE,KAAkB,CAAC,MAAM,CAAC,EAE/C,CAAC;wBACG,MAAM,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,OAAO,CACpC,KAAkB,CAAC,MAAqE,CAC5F,CAAC;wBACF,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI,EAAE,KAAK;4BACX,SAAS,EAAE,QAAQ;4BACnB,IAAI,EAAE,EAAE,GAAG,EAAE,GAAG,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE;yBACS,CAAC,CAAC;oBAC1D,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/no-self-write-in-effect.d.ts

@@ -0,0 +1,3 @@
+import type { Rule } from 'eslint';
+export declare const noSelfWriteInEffect: Rule.RuleModule;
+//# sourceMappingURL=no-self-write-in-effect.d.ts.map

package/dist/rules/no-self-write-in-effect.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-self-write-in-effect.d.ts","sourceRoot":"","sources":["../../src/rules/no-self-write-in-effect.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAwBnC,eAAO,MAAM,mBAAmB,EAAE,IAAI,CAAC,UAsFtC,CAAC"}

package/dist/rules/no-self-write-in-effect.js

@@ -0,0 +1,69 @@
+// azeroth/no-self-write-in-effect: inside one createEffect callback, calling
+// a signal's setter while also reading its getter is the synchronous
+// feedback loop - the write re-runs the effect that is currently running.
+// Wrapping the write in untrack() (or restructuring) is the fix, and an
+// untrack-wrapped write is exactly what the rule permits.
+import { collectSignalPair, findAncestor, isCallTo, isFunctionNode, isIdentifier } from "../ast.js";
+export const noSelfWriteInEffect = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'disallow writing a signal inside an effect that reads it (synchronous feedback loop)'
+        },
+        messages: {
+            selfWrite: 'This effect reads {{getter}}() and writes it with {{setter}}() - a synchronous feedback loop. Wrap the write in untrack(() => ...) or derive the value with createMemo.'
+        },
+        schema: []
+    },
+    create(context) {
+        const pairs = { getterOf: new Map(), getters: new Set() };
+        const stack = [];
+        function insideUntrack(node, boundary) {
+            return findAncestor(node, (a) => isCallTo(a, 'untrack'), boundary) !== null;
+        }
+        return {
+            VariableDeclarator(node) {
+                collectSignalPair(node, pairs);
+            },
+            CallExpression(node) {
+                const call = node;
+                // Entering an effect: the callback is argument 0.
+                if (isCallTo(call, 'createEffect') && isFunctionNode(call.arguments[0])) {
+                    stack.push({ fn: call.arguments[0], reads: new Set(), writes: [] });
+                    return;
+                }
+                const frame = stack[stack.length - 1];
+                if (frame === undefined || !isIdentifier(call.callee)) {
+                    return;
+                }
+                const name = call.callee.name;
+                if (pairs.getters.has(name) && !insideUntrack(call, frame.fn)) {
+                    frame.reads.add(name);
+                }
+                const getter = pairs.getterOf.get(name);
+                if (getter !== undefined && !insideUntrack(call, frame.fn)) {
+                    frame.writes.push({ node: call, setterName: name });
+                }
+            },
+            'CallExpression:exit'(node) {
+                const call = node;
+                const frame = stack[stack.length - 1];
+                if (frame === undefined || !isCallTo(call, 'createEffect') || call.arguments[0] !== frame.fn) {
+                    return;
+                }
+                stack.pop();
+                for (const write of frame.writes) {
+                    const getter = pairs.getterOf.get(write.setterName);
+                    if (getter !== undefined && frame.reads.has(getter)) {
+                        context.report({
+                            node: write.node,
+                            messageId: 'selfWrite',
+                            data: { getter, setter: write.setterName }
+                        });
+                    }
+                }
+            }
+        };
+    }
+};
+//# sourceMappingURL=no-self-write-in-effect.js.map

package/dist/rules/no-self-write-in-effect.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-self-write-in-effect.js","sourceRoot":"","sources":["../../src/rules/no-self-write-in-effect.ts"],"names":[],"mappings":"AAAA,6EAA6E;AAC7E,qEAAqE;AACrE,0EAA0E;AAC1E,wEAAwE;AACxE,0DAA0D;AAG1D,OAAO,EAIH,iBAAiB,EACjB,YAAY,EACZ,QAAQ,EACR,cAAc,EACd,YAAY,EACf,MAAM,WAAW,CAAC;AAcnB,MAAM,CAAC,MAAM,mBAAmB,GAAoB;IAChD,IAAI,EACJ;QACI,IAAI,EAAE,SAAS;QACf,IAAI,EACJ;YACI,WAAW,EAAE,sFAAsF;SACtG;QACD,QAAQ,EACR;YACI,SAAS,EAAE,yKAAyK;SACvL;QACD,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAO;QAEV,MAAM,KAAK,GAAgB,EAAE,QAAQ,EAAE,IAAI,GAAG,EAAE,EAAE,OAAO,EAAE,IAAI,GAAG,EAAE,EAAE,CAAC;QACvE,MAAM,KAAK,GAAkB,EAAE,CAAC;QAEhC,SAAS,aAAa,CAAC,IAAa,EAAE,QAAiB;YAEnD,OAAO,YAAY,CAAC,IAAI,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,CAAC,CAAC,EAAE,SAAS,CAAC,EAAE,QAAQ,CAAC,KAAK,IAAI,CAAC;QAChF,CAAC;QAED,OAAO;YACH,kBAAkB,CAAC,IAAI;gBAEnB,iBAAiB,CAAC,IAA0B,EAAE,KAAK,CAAC,CAAC;YACzD,CAAC;YAED,cAAc,CAAC,IAAI;gBAEf,MAAM,IAAI,GAAG,IAA2B,CAAC;gBAEzC,kDAAkD;gBAClD,IAAI,QAAQ,CAAC,IAAI,EAAE,cAAc,CAAC,IAAI,cAAc,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EACvE,CAAC;oBACG,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,GAAG,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,CAAC;oBACpE,OAAO;gBACX,CAAC;gBAED,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;gBACtC,IAAI,KAAK,KAAK,SAAS,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,MAAM,CAAC,EACrD,CAAC;oBACG,OAAO;gBACX,CAAC;gBACD,MAAM,IAAI,GAAG,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC;gBAE9B,IAAI,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,CAAC,EAC7D,CAAC;oBACG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC1B,CAAC;gBAED,MAAM,MAAM,GAAG,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;gBACxC,IAAI,MAAM,KAAK,SAAS,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,CAAC,EAC1D,CAAC;oBACG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC,CAAC;gBACxD,CAAC;YACL,CAAC;YAED,qBAAqB,CAAC,IAAI;gBAEtB,MAAM,IAAI,GAAG,IAA2B,CAAC;gBACzC,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;gBACtC,IAAI,KAAK,KAAK,SAAS,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,cAAc,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,KAAK,CAAC,EAAE,EAC5F,CAAC;oBACG,OAAO;gBACX,CAAC;gBACD,KAAK,CAAC,GAAG,EAAE,CAAC;gBAEZ,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,MAAM,EAChC,CAAC;oBACG,MAAM,MAAM,GAAG,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC;oBACpD,IAAI,MAAM,KAAK,SAAS,IAAI,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,CAAC,EACnD,CAAC;wBACG,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI,EAAE,KAAK,CAAC,IAAI;4BAChB,SAAS,EAAE,WAAW;4BACtB,IAAI,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,CAAC,UAAU,EAAE;yBACM,CAAC,CAAC;oBAC1D,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/dist/rules/require-effect-disposal.d.ts

@@ -0,0 +1,3 @@
+import type { Rule } from 'eslint';
+export declare const requireEffectDisposal: Rule.RuleModule;
+//# sourceMappingURL=require-effect-disposal.d.ts.map

package/dist/rules/require-effect-disposal.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-effect-disposal.d.ts","sourceRoot":"","sources":["../../src/rules/require-effect-disposal.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAGnC,eAAO,MAAM,qBAAqB,EAAE,IAAI,CAAC,UAsCxC,CAAC"}

package/dist/rules/require-effect-disposal.js

@@ -0,0 +1,38 @@
+// azeroth/require-effect-disposal: a bare `createEffect(...)` statement at
+// MODULE scope has no owner - no root collects its disposer and the caller
+// discarded it, so the effect runs for the life of the page. Inside
+// functions, ownership is unknowable syntactically (the surrounding
+// component/render() usually provides a root), so the rule deliberately
+// stays silent there - module scope is the case that is always wrong.
+import { isCallTo } from "../ast.js";
+export const requireEffectDisposal = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'disallow undisposable module-scope createEffect calls'
+        },
+        messages: {
+            naked: 'Module-scope createEffect with a discarded disposer can never be stopped. Keep the returned dispose function, or create the effect inside a createRoot()/component.'
+        },
+        schema: []
+    },
+    create(context) {
+        return {
+            ExpressionStatement(node) {
+                const statement = node;
+                const parent = statement.parent;
+                if (parent === undefined || parent.type !== 'Program') {
+                    return;
+                }
+                const expression = statement.expression;
+                if (isCallTo(expression, 'createEffect')) {
+                    context.report({
+                        node: expression,
+                        messageId: 'naked'
+                    });
+                }
+            }
+        };
+    }
+};
+//# sourceMappingURL=require-effect-disposal.js.map

package/dist/rules/require-effect-disposal.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-effect-disposal.js","sourceRoot":"","sources":["../../src/rules/require-effect-disposal.ts"],"names":[],"mappings":"AAAA,2EAA2E;AAC3E,2EAA2E;AAC3E,oEAAoE;AACpE,oEAAoE;AACpE,wEAAwE;AACxE,sEAAsE;AAGtE,OAAO,EAAgB,QAAQ,EAAE,MAAM,WAAW,CAAC;AAEnD,MAAM,CAAC,MAAM,qBAAqB,GAAoB;IAClD,IAAI,EACJ;QACI,IAAI,EAAE,SAAS;QACf,IAAI,EACJ;YACI,WAAW,EAAE,uDAAuD;SACvE;QACD,QAAQ,EACR;YACI,KAAK,EAAE,qKAAqK;SAC/K;QACD,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAO;QAEV,OAAO;YACH,mBAAmB,CAAC,IAAI;gBAEpB,MAAM,SAAS,GAAG,IAA0B,CAAC;gBAC7C,MAAM,MAAM,GAAG,SAAS,CAAC,MAAM,CAAC;gBAChC,IAAI,MAAM,KAAK,SAAS,IAAI,MAAM,CAAC,IAAI,KAAK,SAAS,EACrD,CAAC;oBACG,OAAO;gBACX,CAAC;gBAED,MAAM,UAAU,GAAG,SAAS,CAAC,UAAiC,CAAC;gBAC/D,IAAI,QAAQ,CAAC,UAAU,EAAE,cAAc,CAAC,EACxC,CAAC;oBACG,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,UAAU;wBAChB,SAAS,EAAE,OAAO;qBAC8B,CAAC,CAAC;gBAC1D,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC"}

package/package.json

@@ -0,0 +1,54 @@
+{
+    "name": "@azerothjs/eslint-plugin",
+    "version": "0.5.0-beta.1",
+    "description": "AzerothJS ESLint rules — reactivity foot-guns in plain .ts files",
+    "type": "module",
+    "main": "./dist/index.js",
+    "module": "./dist/index.js",
+    "types": "./dist/index.d.ts",
+    "exports":
+    {
+        ".":
+        {
+            "types": "./dist/index.d.ts",
+            "import": "./dist/index.js",
+            "default": "./dist/index.js"
+        }
+    },
+    "files":
+    [
+        "dist"
+    ],
+    "sideEffects": false,
+    "scripts":
+    {
+        "build": "tsc -p tsconfig.build.json",
+        "prepublishOnly": "npm run build"
+    },
+    "peerDependencies":
+    {
+        "eslint": ">=9"
+    },
+    "publishConfig":
+    {
+        "access": "public"
+    },
+    "author":
+    {
+        "name": "IntelligentQuantum",
+        "email": "IntelligentQuantum@Gmail.Com",
+        "url": "https://IntelligentQuantum.Dev/"
+    },
+    "license": "MIT",
+    "repository":
+    {
+        "type": "git",
+        "url": "git+https://github.com/IntelligentQuantum-Dev/AzerothJS.git",
+        "directory": "packages/eslint-plugin"
+    },
+    "homepage": "https://github.com/IntelligentQuantum-Dev/AzerothJS/tree/main/packages/eslint-plugin",
+    "bugs":
+    {
+        "url": "https://github.com/IntelligentQuantum-Dev/AzerothJS/issues"
+    }
+}