@llui/compiler 0.3.0

package/dist/modules/no-barrel-import-when-subpath-exists.d.ts

@@ -0,0 +1,3 @@
+import type { CompilerModule } from '../module.js';
+export declare function noBarrelImportWhenSubpathExistsModule(): CompilerModule;
+//# sourceMappingURL=no-barrel-import-when-subpath-exists.d.ts.map

package/dist/modules/no-barrel-import-when-subpath-exists.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-barrel-import-when-subpath-exists.d.ts","sourceRoot":"","sources":["../../src/modules/no-barrel-import-when-subpath-exists.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AA+BlD,wBAAgB,qCAAqC,IAAI,cAAc,CAqDtE"}

package/dist/modules/no-barrel-import-when-subpath-exists.js

@@ -0,0 +1,100 @@
+// `no-barrel-import-when-subpath-exists` — errors when a name is imported
+// from an `@llui/*` package's barrel and the package ships a sub-path
+// export for that name. Sub-paths skip the barrel parse — smaller
+// bundle, faster cold builds. Migrated from
+// `@llui/eslint-plugin/src/rules/no-barrel-import-when-subpath-exists.ts`.
+// Autofix dropped per the migration plan; the error includes the exact
+// split-import replacement.
+import { createRequire } from 'node:module';
+import { dirname } from 'node:path';
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+const TARGETS = new Set(['@llui/components']);
+const subpathCache = new Map();
+function loadSubpaths(packageName, fromDir) {
+    const cacheKey = `${packageName}@${fromDir}`;
+    const cached = subpathCache.get(cacheKey);
+    if (cached !== undefined)
+        return cached;
+    let result = null;
+    try {
+        const req = createRequire(`${fromDir}/index.js`);
+        const pkgPath = req.resolve(`${packageName}/package.json`);
+        const pkg = req(pkgPath);
+        if (pkg.exports && typeof pkg.exports === 'object') {
+            const subs = new Set();
+            for (const key of Object.keys(pkg.exports)) {
+                if (key === '.' || key.includes('*'))
+                    continue;
+                if (!key.startsWith('./'))
+                    continue;
+                subs.add(key.slice(2));
+            }
+            result = subs;
+        }
+    }
+    catch {
+        result = null;
+    }
+    subpathCache.set(cacheKey, result);
+    return result;
+}
+export function noBarrelImportWhenSubpathExistsModule() {
+    return {
+        name: 'no-barrel-import-when-subpath-exists',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-barrel-import-when-subpath-exists',
+                description: 'Barrel import has a sub-path export — use the sub-path for smaller bundles.',
+            },
+        ],
+        visitors: {
+            [ts.SyntaxKind.SourceFile]: (ctx, node) => {
+                const visited = node;
+                const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true);
+                // Look up sub-paths relative to the file's directory. If the
+                // package isn't installed (test fixture, sandboxed env), the
+                // helper returns null and the rule silently skips.
+                const fromDir = sf.fileName ? dirname(sf.fileName) : process.cwd();
+                for (const stmt of sf.statements) {
+                    if (!ts.isImportDeclaration(stmt))
+                        continue;
+                    if (!ts.isStringLiteral(stmt.moduleSpecifier))
+                        continue;
+                    const src = stmt.moduleSpecifier.text;
+                    if (!TARGETS.has(src))
+                        continue;
+                    const clause = stmt.importClause;
+                    if (!clause || !clause.namedBindings || !ts.isNamedImports(clause.namedBindings))
+                        continue;
+                    const subpaths = loadSubpaths(src, fromDir);
+                    if (!subpaths)
+                        continue;
+                    for (const spec of clause.namedBindings.elements) {
+                        const importedName = (spec.propertyName ?? spec.name).text;
+                        if (!subpaths.has(importedName))
+                            continue;
+                        const localName = spec.name.text;
+                        const fixLine = importedName === localName
+                            ? `import { ${localName} } from '${src}/${importedName}'`
+                            : `import { ${importedName} as ${localName} } from '${src}/${importedName}'`;
+                        ctx.reportDiagnostic({
+                            id: 'llui/no-barrel-import-when-subpath-exists',
+                            severity: 'error',
+                            category: 'perf',
+                            message: `Import \`${importedName}\` from the sub-path '${src}/${importedName}' instead of ` +
+                                `the barrel '${src}'. Sub-path imports skip the barrel parse — smaller bundle, ` +
+                                `faster cold builds. Fix: \`${fixLine}\` (split off from the existing barrel import).`,
+                            location: {
+                                file: sf.fileName,
+                                range: rangeFromOffsets(sf.text, spec.getStart(sf), spec.getEnd()),
+                            },
+                        });
+                    }
+                }
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-barrel-import-when-subpath-exists.js.map

package/dist/modules/no-barrel-import-when-subpath-exists.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-barrel-import-when-subpath-exists.js","sourceRoot":"","sources":["../../src/modules/no-barrel-import-when-subpath-exists.ts"],"names":[],"mappings":"AAAA,0EAA0E;AAC1E,sEAAsE;AACtE,kEAAkE;AAClE,4CAA4C;AAC5C,2EAA2E;AAC3E,uEAAuE;AACvE,4BAA4B;AAE5B,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAA;AAC3C,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AACnC,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAGnD,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAA;AAE7C,MAAM,YAAY,GAAG,IAAI,GAAG,EAAsB,CAAA;AAElD,SAAS,YAAY,CAAC,WAAmB,EAAE,OAAe;IACxD,MAAM,QAAQ,GAAG,GAAG,WAAW,IAAI,OAAO,EAAE,CAAA;IAC5C,MAAM,MAAM,GAAG,YAAY,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAA;IACzC,IAAI,MAAM,KAAK,SAAS;QAAE,OAAO,MAAM,CAAA;IACvC,IAAI,MAAM,GAAe,IAAI,CAAA;IAC7B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,aAAa,CAAC,GAAG,OAAO,WAAW,CAAC,CAAA;QAChD,MAAM,OAAO,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,WAAW,eAAe,CAAC,CAAA;QAC1D,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAA0C,CAAA;QACjE,IAAI,GAAG,CAAC,OAAO,IAAI,OAAO,GAAG,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;YACnD,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAA;YAC9B,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC3C,IAAI,GAAG,KAAK,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC;oBAAE,SAAQ;gBAC9C,IAAI,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,CAAC;oBAAE,SAAQ;gBACnC,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAA;YACxB,CAAC;YACD,MAAM,GAAG,IAAI,CAAA;QACf,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,GAAG,IAAI,CAAA;IACf,CAAC;IACD,YAAY,CAAC,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAA;IAClC,OAAO,MAAM,CAAA;AACf,CAAC;AAED,MAAM,UAAU,qCAAqC;IACnD,OAAO;QACL,IAAI,EAAE,sCAAsC;QAC5C,eAAe,EAAE,QAAQ;QACzB,WAAW,EAAE;YACX;gBACE,EAAE,EAAE,2CAA2C;gBAC/C,WAAW,EAAE,6EAA6E;aAC3F;SACF;QACD,QAAQ,EAAE;YACR,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE;gBACxC,MAAM,OAAO,GAAG,IAAqB,CAAA;gBACrC,MAAM,EAAE,GAAG,EAAE,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAA;gBAC5F,6DAA6D;gBAC7D,6DAA6D;gBAC7D,mDAAmD;gBACnD,MAAM,OAAO,GAAG,EAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,EAAE,CAAA;gBAClE,KAAK,MAAM,IAAI,IAAI,EAAE,CAAC,UAAU,EAAE,CAAC;oBACjC,IAAI,CAAC,EAAE,CAAC,mBAAmB,CAAC,IAAI,CAAC;wBAAE,SAAQ;oBAC3C,IAAI,CAAC,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,eAAe,CAAC;wBAAE,SAAQ;oBACvD,MAAM,GAAG,GAAG,IAAI,CAAC,eAAe,CAAC,IAAI,CAAA;oBACrC,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC;wBAAE,SAAQ;oBAC/B,MAAM,MAAM,GAAG,IAAI,CAAC,YAAY,CAAA;oBAChC,IAAI,CAAC,MAAM,IAAI,CAAC,MAAM,CAAC,aAAa,IAAI,CAAC,EAAE,CAAC,cAAc,CAAC,MAAM,CAAC,aAAa,CAAC;wBAAE,SAAQ;oBAC1F,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAA;oBAC3C,IAAI,CAAC,QAAQ;wBAAE,SAAQ;oBACvB,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,aAAa,CAAC,QAAQ,EAAE,CAAC;wBACjD,MAAM,YAAY,GAAG,CAAC,IAAI,CAAC,YAAY,IAAI,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAA;wBAC1D,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,YAAY,CAAC;4BAAE,SAAQ;wBACzC,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAA;wBAChC,MAAM,OAAO,GACX,YAAY,KAAK,SAAS;4BACxB,CAAC,CAAC,YAAY,SAAS,YAAY,GAAG,IAAI,YAAY,GAAG;4BACzD,CAAC,CAAC,YAAY,YAAY,OAAO,SAAS,YAAY,GAAG,IAAI,YAAY,GAAG,CAAA;wBAChF,GAAG,CAAC,gBAAgB,CAAC;4BACnB,EAAE,EAAE,2CAA2C;4BAC/C,QAAQ,EAAE,OAAO;4BACjB,QAAQ,EAAE,MAAM;4BAChB,OAAO,EACL,YAAY,YAAY,yBAAyB,GAAG,IAAI,YAAY,eAAe;gCACnF,eAAe,GAAG,8DAA8D;gCAChF,8BAA8B,OAAO,iDAAiD;4BACxF,QAAQ,EAAE;gCACR,IAAI,EAAE,EAAE,CAAC,QAAQ;gCACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,IAAI,CAAC,MAAM,EAAE,CAAC;6BACnE;yBACF,CAAC,CAAA;oBACJ,CAAC;gBACH,CAAC;YACH,CAAC;SACF;KACF,CAAA;AACH,CAAC","sourcesContent":["// `no-barrel-import-when-subpath-exists` — errors when a name is imported\n// from an `@llui/*` package's barrel and the package ships a sub-path\n// export for that name. Sub-paths skip the barrel parse — smaller\n// bundle, faster cold builds. Migrated from\n// `@llui/eslint-plugin/src/rules/no-barrel-import-when-subpath-exists.ts`.\n// Autofix dropped per the migration plan; the error includes the exact\n// split-import replacement.\n\nimport { createRequire } from 'node:module'\nimport { dirname } from 'node:path'\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\n\nconst TARGETS = new Set(['@llui/components'])\ntype SubpathSet = Set<string> | null\nconst subpathCache = new Map<string, SubpathSet>()\n\nfunction loadSubpaths(packageName: string, fromDir: string): SubpathSet {\n  const cacheKey = `${packageName}@${fromDir}`\n  const cached = subpathCache.get(cacheKey)\n  if (cached !== undefined) return cached\n  let result: SubpathSet = null\n  try {\n    const req = createRequire(`${fromDir}/index.js`)\n    const pkgPath = req.resolve(`${packageName}/package.json`)\n    const pkg = req(pkgPath) as { exports?: Record<string, unknown> }\n    if (pkg.exports && typeof pkg.exports === 'object') {\n      const subs = new Set<string>()\n      for (const key of Object.keys(pkg.exports)) {\n        if (key === '.' || key.includes('*')) continue\n        if (!key.startsWith('./')) continue\n        subs.add(key.slice(2))\n      }\n      result = subs\n    }\n  } catch {\n    result = null\n  }\n  subpathCache.set(cacheKey, result)\n  return result\n}\n\nexport function noBarrelImportWhenSubpathExistsModule(): CompilerModule {\n  return {\n    name: 'no-barrel-import-when-subpath-exists',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-barrel-import-when-subpath-exists',\n        description: 'Barrel import has a sub-path export — use the sub-path for smaller bundles.',\n      },\n    ],\n    visitors: {\n      [ts.SyntaxKind.SourceFile]: (ctx, node) => {\n        const visited = node as ts.SourceFile\n        const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true)\n        // Look up sub-paths relative to the file's directory. If the\n        // package isn't installed (test fixture, sandboxed env), the\n        // helper returns null and the rule silently skips.\n        const fromDir = sf.fileName ? dirname(sf.fileName) : process.cwd()\n        for (const stmt of sf.statements) {\n          if (!ts.isImportDeclaration(stmt)) continue\n          if (!ts.isStringLiteral(stmt.moduleSpecifier)) continue\n          const src = stmt.moduleSpecifier.text\n          if (!TARGETS.has(src)) continue\n          const clause = stmt.importClause\n          if (!clause || !clause.namedBindings || !ts.isNamedImports(clause.namedBindings)) continue\n          const subpaths = loadSubpaths(src, fromDir)\n          if (!subpaths) continue\n          for (const spec of clause.namedBindings.elements) {\n            const importedName = (spec.propertyName ?? spec.name).text\n            if (!subpaths.has(importedName)) continue\n            const localName = spec.name.text\n            const fixLine =\n              importedName === localName\n                ? `import { ${localName} } from '${src}/${importedName}'`\n                : `import { ${importedName} as ${localName} } from '${src}/${importedName}'`\n            ctx.reportDiagnostic({\n              id: 'llui/no-barrel-import-when-subpath-exists',\n              severity: 'error',\n              category: 'perf',\n              message:\n                `Import \\`${importedName}\\` from the sub-path '${src}/${importedName}' instead of ` +\n                `the barrel '${src}'. Sub-path imports skip the barrel parse — smaller bundle, ` +\n                `faster cold builds. Fix: \\`${fixLine}\\` (split off from the existing barrel import).`,\n              location: {\n                file: sf.fileName,\n                range: rangeFromOffsets(sf.text, spec.getStart(sf), spec.getEnd()),\n              },\n            })\n          }\n        }\n      },\n    },\n  }\n}\n"]}

package/dist/modules/no-eager-item-accessor.d.ts

@@ -0,0 +1,3 @@
+import type { CompilerModule } from '../module.js';
+export declare function noEagerItemAccessorModule(): CompilerModule;
+//# sourceMappingURL=no-eager-item-accessor.d.ts.map

package/dist/modules/no-eager-item-accessor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-eager-item-accessor.d.ts","sourceRoot":"","sources":["../../src/modules/no-eager-item-accessor.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AAYlD,wBAAgB,yBAAyB,IAAI,cAAc,CAyD1D"}

package/dist/modules/no-eager-item-accessor.js

@@ -0,0 +1,74 @@
+// `no-eager-item-accessor` — errors when an each() `item.<prop>()` is
+// invoked eagerly inside `text(...)` / `unsafeHtml(...)`. The
+// ItemAccessor returns a `() => V`; passing the call result captures
+// a static value, so the cell never updates when the row data
+// changes in place. Pass the accessor itself instead (drop the `()`).
+// Migrated from `@llui/eslint-plugin/src/rules/no-eager-item-accessor.ts`.
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+const EAGER_TARGETS = new Set(['text', 'unsafeHtml']);
+/** True when `node` is `item.<prop>()` — bare `item` identifier root. */
+function isItemMemberCall(node) {
+    if (!ts.isCallExpression(node))
+        return false;
+    if (!ts.isPropertyAccessExpression(node.expression))
+        return false;
+    const obj = node.expression.expression;
+    return ts.isIdentifier(obj) && obj.text === 'item';
+}
+export function noEagerItemAccessorModule() {
+    return {
+        name: 'no-eager-item-accessor',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-eager-item-accessor',
+                description: 'Eager call of an ItemAccessor (`text(item.x())`) captures a static value — drop the `()` to pass the accessor.',
+            },
+        ],
+        visitors: {
+            [ts.SyntaxKind.SourceFile]: (ctx, node) => {
+                const visited = node;
+                const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true);
+                const walk = (n) => {
+                    if (ts.isCallExpression(n)) {
+                        // Bare `text(...)` or member-form `h.text(...)`.
+                        let calleeName;
+                        if (ts.isIdentifier(n.expression) && EAGER_TARGETS.has(n.expression.text)) {
+                            calleeName = n.expression.text;
+                        }
+                        else if (ts.isPropertyAccessExpression(n.expression) &&
+                            ts.isIdentifier(n.expression.name) &&
+                            EAGER_TARGETS.has(n.expression.name.text)) {
+                            calleeName = n.expression.name.text;
+                        }
+                        if (calleeName) {
+                            const arg = n.arguments[0];
+                            if (arg && isItemMemberCall(arg)) {
+                                const memberArg = arg;
+                                const member = memberArg.expression;
+                                const propText = ts.isIdentifier(member.name)
+                                    ? `item.${member.name.text}`
+                                    : 'item.<prop>';
+                                ctx.reportDiagnostic({
+                                    id: 'llui/no-eager-item-accessor',
+                                    severity: 'error',
+                                    category: 'reactivity',
+                                    message: `\`${calleeName}(${propText}())\` reads the item value once at view-construction and never updates when the row data changes in place. ` +
+                                        `Drop the \`()\` to pass the accessor itself: \`${calleeName}(${propText})\`. The runtime detects the zero-arg form and re-reads on every commit.`,
+                                    location: {
+                                        file: sf.fileName,
+                                        range: rangeFromOffsets(sf.text, memberArg.getStart(sf), memberArg.getEnd()),
+                                    },
+                                });
+                            }
+                        }
+                    }
+                    ts.forEachChild(n, walk);
+                };
+                walk(sf);
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-eager-item-accessor.js.map

package/dist/modules/no-eager-item-accessor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-eager-item-accessor.js","sourceRoot":"","sources":["../../src/modules/no-eager-item-accessor.ts"],"names":[],"mappings":"AAAA,sEAAsE;AACtE,8DAA8D;AAC9D,qEAAqE;AACrE,8DAA8D;AAC9D,sEAAsE;AACtE,2EAA2E;AAE3E,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAGnD,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,CAAA;AAErD,yEAAyE;AACzE,SAAS,gBAAgB,CAAC,IAAa;IACrC,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAA;IAC5C,IAAI,CAAC,EAAE,CAAC,0BAA0B,CAAC,IAAI,CAAC,UAAU,CAAC;QAAE,OAAO,KAAK,CAAA;IACjE,MAAM,GAAG,GAAG,IAAI,CAAC,UAAU,CAAC,UAAU,CAAA;IACtC,OAAO,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,IAAI,KAAK,MAAM,CAAA;AACpD,CAAC;AAED,MAAM,UAAU,yBAAyB;IACvC,OAAO;QACL,IAAI,EAAE,wBAAwB;QAC9B,eAAe,EAAE,QAAQ;QACzB,WAAW,EAAE;YACX;gBACE,EAAE,EAAE,6BAA6B;gBACjC,WAAW,EACT,gHAAgH;aACnH;SACF;QACD,QAAQ,EAAE;YACR,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE;gBACxC,MAAM,OAAO,GAAG,IAAqB,CAAA;gBACrC,MAAM,EAAE,GAAG,EAAE,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAA;gBAC5F,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;oBAChC,IAAI,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC,EAAE,CAAC;wBAC3B,iDAAiD;wBACjD,IAAI,UAA8B,CAAA;wBAClC,IAAI,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;4BAC1E,UAAU,GAAG,CAAC,CAAC,UAAU,CAAC,IAAI,CAAA;wBAChC,CAAC;6BAAM,IACL,EAAE,CAAC,0BAA0B,CAAC,CAAC,CAAC,UAAU,CAAC;4BAC3C,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC;4BAClC,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,EACzC,CAAC;4BACD,UAAU,GAAG,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAA;wBACrC,CAAC;wBACD,IAAI,UAAU,EAAE,CAAC;4BACf,MAAM,GAAG,GAAG,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAA;4BAC1B,IAAI,GAAG,IAAI,gBAAgB,CAAC,GAAG,CAAC,EAAE,CAAC;gCACjC,MAAM,SAAS,GAAG,GAAG,CAAA;gCACrB,MAAM,MAAM,GAAG,SAAS,CAAC,UAAyC,CAAA;gCAClE,MAAM,QAAQ,GAAG,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAAC;oCAC3C,CAAC,CAAC,QAAQ,MAAM,CAAC,IAAI,CAAC,IAAI,EAAE;oCAC5B,CAAC,CAAC,aAAa,CAAA;gCACjB,GAAG,CAAC,gBAAgB,CAAC;oCACnB,EAAE,EAAE,6BAA6B;oCACjC,QAAQ,EAAE,OAAO;oCACjB,QAAQ,EAAE,YAAY;oCACtB,OAAO,EACL,KAAK,UAAU,IAAI,QAAQ,6GAA6G;wCACxI,kDAAkD,UAAU,IAAI,QAAQ,0EAA0E;oCACpJ,QAAQ,EAAE;wCACR,IAAI,EAAE,EAAE,CAAC,QAAQ;wCACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,SAAS,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,SAAS,CAAC,MAAM,EAAE,CAAC;qCAC7E;iCACF,CAAC,CAAA;4BACJ,CAAC;wBACH,CAAC;oBACH,CAAC;oBACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA;gBAC1B,CAAC,CAAA;gBACD,IAAI,CAAC,EAAE,CAAC,CAAA;YACV,CAAC;SACF;KACF,CAAA;AACH,CAAC","sourcesContent":["// `no-eager-item-accessor` — errors when an each() `item.<prop>()` is\n// invoked eagerly inside `text(...)` / `unsafeHtml(...)`. The\n// ItemAccessor returns a `() => V`; passing the call result captures\n// a static value, so the cell never updates when the row data\n// changes in place. Pass the accessor itself instead (drop the `()`).\n// Migrated from `@llui/eslint-plugin/src/rules/no-eager-item-accessor.ts`.\n\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\n\nconst EAGER_TARGETS = new Set(['text', 'unsafeHtml'])\n\n/** True when `node` is `item.<prop>()` — bare `item` identifier root. */\nfunction isItemMemberCall(node: ts.Node): node is ts.CallExpression {\n  if (!ts.isCallExpression(node)) return false\n  if (!ts.isPropertyAccessExpression(node.expression)) return false\n  const obj = node.expression.expression\n  return ts.isIdentifier(obj) && obj.text === 'item'\n}\n\nexport function noEagerItemAccessorModule(): CompilerModule {\n  return {\n    name: 'no-eager-item-accessor',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-eager-item-accessor',\n        description:\n          'Eager call of an ItemAccessor (`text(item.x())`) captures a static value — drop the `()` to pass the accessor.',\n      },\n    ],\n    visitors: {\n      [ts.SyntaxKind.SourceFile]: (ctx, node) => {\n        const visited = node as ts.SourceFile\n        const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true)\n        const walk = (n: ts.Node): void => {\n          if (ts.isCallExpression(n)) {\n            // Bare `text(...)` or member-form `h.text(...)`.\n            let calleeName: string | undefined\n            if (ts.isIdentifier(n.expression) && EAGER_TARGETS.has(n.expression.text)) {\n              calleeName = n.expression.text\n            } else if (\n              ts.isPropertyAccessExpression(n.expression) &&\n              ts.isIdentifier(n.expression.name) &&\n              EAGER_TARGETS.has(n.expression.name.text)\n            ) {\n              calleeName = n.expression.name.text\n            }\n            if (calleeName) {\n              const arg = n.arguments[0]\n              if (arg && isItemMemberCall(arg)) {\n                const memberArg = arg\n                const member = memberArg.expression as ts.PropertyAccessExpression\n                const propText = ts.isIdentifier(member.name)\n                  ? `item.${member.name.text}`\n                  : 'item.<prop>'\n                ctx.reportDiagnostic({\n                  id: 'llui/no-eager-item-accessor',\n                  severity: 'error',\n                  category: 'reactivity',\n                  message:\n                    `\\`${calleeName}(${propText}())\\` reads the item value once at view-construction and never updates when the row data changes in place. ` +\n                    `Drop the \\`()\\` to pass the accessor itself: \\`${calleeName}(${propText})\\`. The runtime detects the zero-arg form and re-reads on every commit.`,\n                  location: {\n                    file: sf.fileName,\n                    range: rangeFromOffsets(sf.text, memberArg.getStart(sf), memberArg.getEnd()),\n                  },\n                })\n              }\n            }\n          }\n          ts.forEachChild(n, walk)\n        }\n        walk(sf)\n      },\n    },\n  }\n}\n"]}

package/dist/modules/no-let-reactive-accessor.d.ts

@@ -0,0 +1,3 @@
+import type { CompilerModule } from '../module.js';
+export declare function noLetReactiveAccessorModule(): CompilerModule;
+//# sourceMappingURL=no-let-reactive-accessor.d.ts.map

package/dist/modules/no-let-reactive-accessor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-let-reactive-accessor.d.ts","sourceRoot":"","sources":["../../src/modules/no-let-reactive-accessor.ts"],"names":[],"mappings":"AAeA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AAwHlD,wBAAgB,2BAA2B,IAAI,cAAc,CAuG5D"}

package/dist/modules/no-let-reactive-accessor.js

@@ -0,0 +1,227 @@
+// `no-let-reactive-accessor` — errors when a top-level `let` / `var`
+// binding with a callable initializer is referenced at a reactive
+// accessor position. The compiler's `resolveAccessorBody` refuses to
+// follow `let` / `var` bindings (reassignment would invalidate any
+// compile-time mask analysis), so the binding silently falls back to
+// FULL_MASK at runtime — correct but suboptimal. Forcing `const`
+// preserves the precise-mask optimization. Migrated from
+// `@llui/eslint-plugin/src/rules/no-let-reactive-accessor.ts`.
+//
+// Note: autofix dropped (per the lint→compiler migration plan); the
+// error message includes the exact `let`/`var` → `const` substitution
+// so an LLM consuming the error can apply it directly.
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+import { ELEMENT_HELPERS } from './_element-helpers.js';
+const REACTIVE_API_NAMES = new Set([
+    ...ELEMENT_HELPERS,
+    'each',
+    'branch',
+    'scope',
+    'show',
+    'memo',
+    'portal',
+    'foreign',
+    'child',
+    'errorBoundary',
+]);
+const FIRST_ARG_BINDING_HELPERS = new Set(['text', 'unsafeHtml', 'memo']);
+function isAtReactivePosition(node) {
+    const parent = node.parent;
+    if (!parent)
+        return false;
+    if (ts.isCallExpression(parent) && parent.arguments[0] === node) {
+        if (ts.isIdentifier(parent.expression)) {
+            const name = parent.expression.text;
+            if (name === 'item' || name === 'sample')
+                return false;
+            return FIRST_ARG_BINDING_HELPERS.has(name) || REACTIVE_API_NAMES.has(name);
+        }
+        if (ts.isPropertyAccessExpression(parent.expression) &&
+            ts.isIdentifier(parent.expression.name)) {
+            return FIRST_ARG_BINDING_HELPERS.has(parent.expression.name.text);
+        }
+        return false;
+    }
+    if (ts.isPropertyAssignment(parent) && parent.initializer === node) {
+        if (!ts.isIdentifier(parent.name))
+            return false;
+        const key = parent.name.text;
+        if (/^on[A-Z]/.test(key))
+            return false;
+        if (key === 'key' || key === 'name')
+            return false;
+        let ancestor = parent.parent;
+        while (ancestor && !ts.isCallExpression(ancestor))
+            ancestor = ancestor.parent;
+        if (!ancestor)
+            return false;
+        const callExpr = ancestor;
+        if (!ts.isIdentifier(callExpr.expression))
+            return false;
+        return REACTIVE_API_NAMES.has(callExpr.expression.text);
+    }
+    return false;
+}
+function describeReactiveContext(id) {
+    const parent = id.parent;
+    if (!parent)
+        return 'reactive position';
+    if (ts.isCallExpression(parent) && parent.arguments[0] === id) {
+        if (ts.isIdentifier(parent.expression))
+            return `${parent.expression.text}(…)`;
+        if (ts.isPropertyAccessExpression(parent.expression) &&
+            ts.isIdentifier(parent.expression.name)) {
+            return `…${parent.expression.name.text}(…)`;
+        }
+    }
+    if (ts.isPropertyAssignment(parent) &&
+        parent.initializer === id &&
+        ts.isIdentifier(parent.name)) {
+        let ancestor = parent.parent;
+        while (ancestor && !ts.isCallExpression(ancestor))
+            ancestor = ancestor.parent;
+        const calleeName = ancestor && ts.isCallExpression(ancestor) && ts.isIdentifier(ancestor.expression)
+            ? ancestor.expression.text
+            : null;
+        return calleeName ? `${calleeName}({ ${parent.name.text}: … })` : `{ ${parent.name.text}: … }`;
+    }
+    return 'reactive position';
+}
+function isCallableInitializer(init) {
+    if (!init)
+        return false;
+    if (ts.isArrowFunction(init) || ts.isFunctionExpression(init))
+        return true;
+    if (ts.isCallExpression(init) &&
+        ts.isIdentifier(init.expression) &&
+        init.expression.text === 'memo' &&
+        init.arguments.length >= 1) {
+        const inner = init.arguments[0];
+        return ts.isArrowFunction(inner) || ts.isFunctionExpression(inner);
+    }
+    return false;
+}
+/**
+ * True when `n` is the target of a write (assignment LHS or
+ * pre/postfix increment). We don't try to detect destructuring or
+ * compound forms exhaustively — the common case (`foo = …`,
+ * `foo += …`, `foo++`) covers the practical reassignment patterns.
+ */
+function isWriteTarget(n) {
+    const parent = n.parent;
+    if (!parent)
+        return false;
+    if (ts.isBinaryExpression(parent) && parent.left === n) {
+        const op = parent.operatorToken.kind;
+        return (op === ts.SyntaxKind.EqualsToken ||
+            op === ts.SyntaxKind.PlusEqualsToken ||
+            op === ts.SyntaxKind.MinusEqualsToken ||
+            op === ts.SyntaxKind.AsteriskEqualsToken ||
+            op === ts.SyntaxKind.SlashEqualsToken);
+    }
+    if (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent)) {
+        return (parent.operator === ts.SyntaxKind.PlusPlusToken ||
+            parent.operator === ts.SyntaxKind.MinusMinusToken);
+    }
+    return false;
+}
+export function noLetReactiveAccessorModule() {
+    return {
+        name: 'no-let-reactive-accessor',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-let-reactive-accessor',
+                description: '`let` / `var` accessor referenced at a reactive position — compiler falls back to FULL_MASK. Use `const`.',
+            },
+        ],
+        visitors: {
+            [ts.SyntaxKind.SourceFile]: (ctx, node) => {
+                const visited = node;
+                const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true);
+                const decls = [];
+                for (const stmt of sf.statements) {
+                    if (!ts.isVariableStatement(stmt))
+                        continue;
+                    const flags = stmt.declarationList.flags;
+                    let kind = null;
+                    if (flags & ts.NodeFlags.Let)
+                        kind = 'let';
+                    else if (!(flags & ts.NodeFlags.Const))
+                        kind = 'var';
+                    if (!kind)
+                        continue;
+                    if (stmt.declarationList.declarations.length !== 1)
+                        continue;
+                    const decl = stmt.declarationList.declarations[0];
+                    if (!ts.isIdentifier(decl.name))
+                        continue;
+                    if (!isCallableInitializer(decl.initializer))
+                        continue;
+                    decls.push({ name: decl.name.text, kind, stmt });
+                }
+                if (decls.length === 0)
+                    return;
+                // For each declaration, scan all identifier references in
+                // the file. Track reads at reactive positions and any
+                // write (reassignment). The declarator's own identifier is
+                // excluded.
+                const declMap = new Map(decls.map((d) => [d.name, d]));
+                const state = new Map();
+                for (const d of decls) {
+                    state.set(d.name, { hasReactive: false, reactiveContext: null, hasReassign: false });
+                }
+                // We don't track shadowing rigorously — a nested `const foo`
+                // would still let the outer `let foo`'s name fire. The
+                // common case (top-level let, no inner shadowing) is what we
+                // care about; shadowing is rare and an inner const would
+                // change behavior anyway.
+                const walk = (n) => {
+                    if (ts.isIdentifier(n) && declMap.has(n.text)) {
+                        const decl = declMap.get(n.text);
+                        const slot = state.get(n.text);
+                        // Skip the declarator's own identifier node — that's not
+                        // a reference, it's the declaration itself.
+                        const declIdent = decl.stmt.declarationList.declarations[0].name;
+                        if (n !== declIdent) {
+                            if (isWriteTarget(n))
+                                slot.hasReassign = true;
+                            if (!slot.hasReactive && isAtReactivePosition(n)) {
+                                slot.hasReactive = true;
+                                slot.reactiveContext = describeReactiveContext(n);
+                            }
+                        }
+                    }
+                    ts.forEachChild(n, walk);
+                };
+                walk(sf);
+                for (const d of decls) {
+                    const s = state.get(d.name);
+                    if (!s.hasReactive)
+                        continue;
+                    const reactiveCtx = s.reactiveContext ?? 'reactive position';
+                    const fixHint = s.hasReassign
+                        ? `Either avoid reassignment (use \`const\` and a different binding for the new value) or accept the FULL_MASK fallback.`
+                        : `Fix: change \`${d.kind} ${d.name} = …\` to \`const ${d.name} = …\`.`;
+                    const reasonClause = s.hasReassign
+                        ? `, but it's reassigned later in the file — the compiler can't follow \`${d.kind}\` bindings`
+                        : '';
+                    ctx.reportDiagnostic({
+                        id: 'llui/no-let-reactive-accessor',
+                        severity: 'error',
+                        category: 'reactivity',
+                        message: `\`${d.name}\` is a \`${d.kind}\`-bound accessor used at a reactive position ` +
+                            `(${reactiveCtx})${reasonClause}. The compiler only follows \`const\` bindings; ` +
+                            `\`${d.kind}\` falls back to FULL_MASK at runtime. ${fixHint}`,
+                        location: {
+                            file: sf.fileName,
+                            range: rangeFromOffsets(sf.text, d.stmt.getStart(sf), d.stmt.getEnd()),
+                        },
+                    });
+                }
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-let-reactive-accessor.js.map

package/dist/modules/no-let-reactive-accessor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-let-reactive-accessor.js","sourceRoot":"","sources":["../../src/modules/no-let-reactive-accessor.ts"],"names":[],"mappings":"AAAA,qEAAqE;AACrE,kEAAkE;AAClE,qEAAqE;AACrE,mEAAmE;AACnE,qEAAqE;AACrE,iEAAiE;AACjE,yDAAyD;AACzD,+DAA+D;AAC/D,EAAE;AACF,oEAAoE;AACpE,sEAAsE;AACtE,uDAAuD;AAEvD,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAEnD,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAA;AAEvD,MAAM,kBAAkB,GAAG,IAAI,GAAG,CAAS;IACzC,GAAG,eAAe;IAClB,MAAM;IACN,QAAQ;IACR,OAAO;IACP,MAAM;IACN,MAAM;IACN,QAAQ;IACR,SAAS;IACT,OAAO;IACP,eAAe;CAChB,CAAC,CAAA;AACF,MAAM,yBAAyB,GAAG,IAAI,GAAG,CAAC,CAAC,MAAM,EAAE,YAAY,EAAE,MAAM,CAAC,CAAC,CAAA;AAEzE,SAAS,oBAAoB,CAAC,IAAmB;IAC/C,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAA;IAC1B,IAAI,CAAC,MAAM;QAAE,OAAO,KAAK,CAAA;IACzB,IAAI,EAAE,CAAC,gBAAgB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAChE,IAAI,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,UAAU,CAAC,EAAE,CAAC;YACvC,MAAM,IAAI,GAAG,MAAM,CAAC,UAAU,CAAC,IAAI,CAAA;YACnC,IAAI,IAAI,KAAK,MAAM,IAAI,IAAI,KAAK,QAAQ;gBAAE,OAAO,KAAK,CAAA;YACtD,OAAO,yBAAyB,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,kBAAkB,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QAC5E,CAAC;QACD,IACE,EAAE,CAAC,0BAA0B,CAAC,MAAM,CAAC,UAAU,CAAC;YAChD,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,EACvC,CAAC;YACD,OAAO,yBAAyB,CAAC,GAAG,CAAC,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACnE,CAAC;QACD,OAAO,KAAK,CAAA;IACd,CAAC;IACD,IAAI,EAAE,CAAC,oBAAoB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,WAAW,KAAK,IAAI,EAAE,CAAC;QACnE,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAAC;YAAE,OAAO,KAAK,CAAA;QAC/C,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAA;QAC5B,IAAI,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC;YAAE,OAAO,KAAK,CAAA;QACtC,IAAI,GAAG,KAAK,KAAK,IAAI,GAAG,KAAK,MAAM;YAAE,OAAO,KAAK,CAAA;QACjD,IAAI,QAAQ,GAAwB,MAAM,CAAC,MAAM,CAAA;QACjD,OAAO,QAAQ,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,QAAQ,CAAC;YAAE,QAAQ,GAAG,QAAQ,CAAC,MAAM,CAAA;QAC7E,IAAI,CAAC,QAAQ;YAAE,OAAO,KAAK,CAAA;QAC3B,MAAM,QAAQ,GAAG,QAA6B,CAAA;QAC9C,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,QAAQ,CAAC,UAAU,CAAC;YAAE,OAAO,KAAK,CAAA;QACvD,OAAO,kBAAkB,CAAC,GAAG,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,CAAA;IACzD,CAAC;IACD,OAAO,KAAK,CAAA;AACd,CAAC;AAED,SAAS,uBAAuB,CAAC,EAAiB;IAChD,MAAM,MAAM,GAAG,EAAE,CAAC,MAAM,CAAA;IACxB,IAAI,CAAC,MAAM;QAAE,OAAO,mBAAmB,CAAA;IACvC,IAAI,EAAE,CAAC,gBAAgB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC;QAC9D,IAAI,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,UAAU,CAAC;YAAE,OAAO,GAAG,MAAM,CAAC,UAAU,CAAC,IAAI,KAAK,CAAA;QAC7E,IACE,EAAE,CAAC,0BAA0B,CAAC,MAAM,CAAC,UAAU,CAAC;YAChD,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,EACvC,CAAC;YACD,OAAO,IAAI,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,CAAA;QAC7C,CAAC;IACH,CAAC;IACD,IACE,EAAE,CAAC,oBAAoB,CAAC,MAAM,CAAC;QAC/B,MAAM,CAAC,WAAW,KAAK,EAAE;QACzB,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAAC,EAC5B,CAAC;QACD,IAAI,QAAQ,GAAwB,MAAM,CAAC,MAAM,CAAA;QACjD,OAAO,QAAQ,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,QAAQ,CAAC;YAAE,QAAQ,GAAG,QAAQ,CAAC,MAAM,CAAA;QAC7E,MAAM,UAAU,GACd,QAAQ,IAAI,EAAE,CAAC,gBAAgB,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,YAAY,CAAC,QAAQ,CAAC,UAAU,CAAC;YAC/E,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI;YAC1B,CAAC,CAAC,IAAI,CAAA;QACV,OAAO,UAAU,CAAC,CAAC,CAAC,GAAG,UAAU,MAAM,MAAM,CAAC,IAAI,CAAC,IAAI,QAAQ,CAAC,CAAC,CAAC,KAAK,MAAM,CAAC,IAAI,CAAC,IAAI,OAAO,CAAA;IAChG,CAAC;IACD,OAAO,mBAAmB,CAAA;AAC5B,CAAC;AAED,SAAS,qBAAqB,CAAC,IAA+B;IAC5D,IAAI,CAAC,IAAI;QAAE,OAAO,KAAK,CAAA;IACvB,IAAI,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAA;IAC1E,IACE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC;QACzB,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,UAAU,CAAC;QAChC,IAAI,CAAC,UAAU,CAAC,IAAI,KAAK,MAAM;QAC/B,IAAI,CAAC,SAAS,CAAC,MAAM,IAAI,CAAC,EAC1B,CAAC;QACD,MAAM,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAE,CAAA;QAChC,OAAO,EAAE,CAAC,eAAe,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAA;IACpE,CAAC;IACD,OAAO,KAAK,CAAA;AACd,CAAC;AAED;;;;;GAKG;AACH,SAAS,aAAa,CAAC,CAAgB;IACrC,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAA;IACvB,IAAI,CAAC,MAAM;QAAE,OAAO,KAAK,CAAA;IACzB,IAAI,EAAE,CAAC,kBAAkB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;QACvD,MAAM,EAAE,GAAG,MAAM,CAAC,aAAa,CAAC,IAAI,CAAA;QACpC,OAAO,CACL,EAAE,KAAK,EAAE,CAAC,UAAU,CAAC,WAAW;YAChC,EAAE,KAAK,EAAE,CAAC,UAAU,CAAC,eAAe;YACpC,EAAE,KAAK,EAAE,CAAC,UAAU,CAAC,gBAAgB;YACrC,EAAE,KAAK,EAAE,CAAC,UAAU,CAAC,mBAAmB;YACxC,EAAE,KAAK,EAAE,CAAC,UAAU,CAAC,gBAAgB,CACtC,CAAA;IACH,CAAC;IACD,IAAI,EAAE,CAAC,wBAAwB,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,uBAAuB,CAAC,MAAM,CAAC,EAAE,CAAC;QAC9E,OAAO,CACL,MAAM,CAAC,QAAQ,KAAK,EAAE,CAAC,UAAU,CAAC,aAAa;YAC/C,MAAM,CAAC,QAAQ,KAAK,EAAE,CAAC,UAAU,CAAC,eAAe,CAClD,CAAA;IACH,CAAC;IACD,OAAO,KAAK,CAAA;AACd,CAAC;AAED,MAAM,UAAU,2BAA2B;IACzC,OAAO;QACL,IAAI,EAAE,0BAA0B;QAChC,eAAe,EAAE,QAAQ;QACzB,WAAW,EAAE;YACX;gBACE,EAAE,EAAE,+BAA+B;gBACnC,WAAW,EACT,2GAA2G;aAC9G;SACF;QACD,QAAQ,EAAE;YACR,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE;gBACxC,MAAM,OAAO,GAAG,IAAqB,CAAA;gBACrC,MAAM,EAAE,GAAG,EAAE,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAA;gBAU5F,MAAM,KAAK,GAAW,EAAE,CAAA;gBACxB,KAAK,MAAM,IAAI,IAAI,EAAE,CAAC,UAAU,EAAE,CAAC;oBACjC,IAAI,CAAC,EAAE,CAAC,mBAAmB,CAAC,IAAI,CAAC;wBAAE,SAAQ;oBAC3C,MAAM,KAAK,GAAG,IAAI,CAAC,eAAe,CAAC,KAAK,CAAA;oBACxC,IAAI,IAAI,GAAyB,IAAI,CAAA;oBACrC,IAAI,KAAK,GAAG,EAAE,CAAC,SAAS,CAAC,GAAG;wBAAE,IAAI,GAAG,KAAK,CAAA;yBACrC,IAAI,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC,SAAS,CAAC,KAAK,CAAC;wBAAE,IAAI,GAAG,KAAK,CAAA;oBACpD,IAAI,CAAC,IAAI;wBAAE,SAAQ;oBACnB,IAAI,IAAI,CAAC,eAAe,CAAC,YAAY,CAAC,MAAM,KAAK,CAAC;wBAAE,SAAQ;oBAC5D,MAAM,IAAI,GAAG,IAAI,CAAC,eAAe,CAAC,YAAY,CAAC,CAAC,CAAE,CAAA;oBAClD,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;wBAAE,SAAQ;oBACzC,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,WAAW,CAAC;wBAAE,SAAQ;oBACtD,KAAK,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,CAAA;gBAClD,CAAC;gBACD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;oBAAE,OAAM;gBAE9B,0DAA0D;gBAC1D,sDAAsD;gBACtD,2DAA2D;gBAC3D,YAAY;gBACZ,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAU,CAAC,CAAC,CAAA;gBAC/D,MAAM,KAAK,GAAG,IAAI,GAAG,EAGlB,CAAA;gBACH,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;oBACtB,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,eAAe,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC,CAAA;gBACtF,CAAC;gBACD,6DAA6D;gBAC7D,uDAAuD;gBACvD,6DAA6D;gBAC7D,yDAAyD;gBACzD,0BAA0B;gBAC1B,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;oBAChC,IAAI,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC;wBAC9C,MAAM,IAAI,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAE,CAAA;wBACjC,MAAM,IAAI,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAE,CAAA;wBAC/B,yDAAyD;wBACzD,4CAA4C;wBAC5C,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,eAAe,CAAC,YAAY,CAAC,CAAC,CAAE,CAAC,IAAI,CAAA;wBACjE,IAAI,CAAC,KAAK,SAAS,EAAE,CAAC;4BACpB,IAAI,aAAa,CAAC,CAAC,CAAC;gCAAE,IAAI,CAAC,WAAW,GAAG,IAAI,CAAA;4BAC7C,IAAI,CAAC,IAAI,CAAC,WAAW,IAAI,oBAAoB,CAAC,CAAC,CAAC,EAAE,CAAC;gCACjD,IAAI,CAAC,WAAW,GAAG,IAAI,CAAA;gCACvB,IAAI,CAAC,eAAe,GAAG,uBAAuB,CAAC,CAAC,CAAC,CAAA;4BACnD,CAAC;wBACH,CAAC;oBACH,CAAC;oBACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA;gBAC1B,CAAC,CAAA;gBACD,IAAI,CAAC,EAAE,CAAC,CAAA;gBAER,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;oBACtB,MAAM,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAE,CAAA;oBAC5B,IAAI,CAAC,CAAC,CAAC,WAAW;wBAAE,SAAQ;oBAC5B,MAAM,WAAW,GAAG,CAAC,CAAC,eAAe,IAAI,mBAAmB,CAAA;oBAC5D,MAAM,OAAO,GAAG,CAAC,CAAC,WAAW;wBAC3B,CAAC,CAAC,uHAAuH;wBACzH,CAAC,CAAC,iBAAiB,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,IAAI,qBAAqB,CAAC,CAAC,IAAI,SAAS,CAAA;oBACzE,MAAM,YAAY,GAAG,CAAC,CAAC,WAAW;wBAChC,CAAC,CAAC,yEAAyE,CAAC,CAAC,IAAI,aAAa;wBAC9F,CAAC,CAAC,EAAE,CAAA;oBACN,GAAG,CAAC,gBAAgB,CAAC;wBACnB,EAAE,EAAE,+BAA+B;wBACnC,QAAQ,EAAE,OAAO;wBACjB,QAAQ,EAAE,YAAY;wBACtB,OAAO,EACL,KAAK,CAAC,CAAC,IAAI,aAAa,CAAC,CAAC,IAAI,gDAAgD;4BAC9E,IAAI,WAAW,IAAI,YAAY,kDAAkD;4BACjF,KAAK,CAAC,CAAC,IAAI,0CAA0C,OAAO,EAAE;wBAChE,QAAQ,EAAE;4BACR,IAAI,EAAE,EAAE,CAAC,QAAQ;4BACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC;yBACvE;qBACF,CAAC,CAAA;gBACJ,CAAC;YACH,CAAC;SACF;KACF,CAAA;AACH,CAAC","sourcesContent":["// `no-let-reactive-accessor` — errors when a top-level `let` / `var`\n// binding with a callable initializer is referenced at a reactive\n// accessor position. The compiler's `resolveAccessorBody` refuses to\n// follow `let` / `var` bindings (reassignment would invalidate any\n// compile-time mask analysis), so the binding silently falls back to\n// FULL_MASK at runtime — correct but suboptimal. Forcing `const`\n// preserves the precise-mask optimization. Migrated from\n// `@llui/eslint-plugin/src/rules/no-let-reactive-accessor.ts`.\n//\n// Note: autofix dropped (per the lint→compiler migration plan); the\n// error message includes the exact `let`/`var` → `const` substitution\n// so an LLM consuming the error can apply it directly.\n\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\nimport { ELEMENT_HELPERS } from './_element-helpers.js'\n\nconst REACTIVE_API_NAMES = new Set<string>([\n  ...ELEMENT_HELPERS,\n  'each',\n  'branch',\n  'scope',\n  'show',\n  'memo',\n  'portal',\n  'foreign',\n  'child',\n  'errorBoundary',\n])\nconst FIRST_ARG_BINDING_HELPERS = new Set(['text', 'unsafeHtml', 'memo'])\n\nfunction isAtReactivePosition(node: ts.Identifier): boolean {\n  const parent = node.parent\n  if (!parent) return false\n  if (ts.isCallExpression(parent) && parent.arguments[0] === node) {\n    if (ts.isIdentifier(parent.expression)) {\n      const name = parent.expression.text\n      if (name === 'item' || name === 'sample') return false\n      return FIRST_ARG_BINDING_HELPERS.has(name) || REACTIVE_API_NAMES.has(name)\n    }\n    if (\n      ts.isPropertyAccessExpression(parent.expression) &&\n      ts.isIdentifier(parent.expression.name)\n    ) {\n      return FIRST_ARG_BINDING_HELPERS.has(parent.expression.name.text)\n    }\n    return false\n  }\n  if (ts.isPropertyAssignment(parent) && parent.initializer === node) {\n    if (!ts.isIdentifier(parent.name)) return false\n    const key = parent.name.text\n    if (/^on[A-Z]/.test(key)) return false\n    if (key === 'key' || key === 'name') return false\n    let ancestor: ts.Node | undefined = parent.parent\n    while (ancestor && !ts.isCallExpression(ancestor)) ancestor = ancestor.parent\n    if (!ancestor) return false\n    const callExpr = ancestor as ts.CallExpression\n    if (!ts.isIdentifier(callExpr.expression)) return false\n    return REACTIVE_API_NAMES.has(callExpr.expression.text)\n  }\n  return false\n}\n\nfunction describeReactiveContext(id: ts.Identifier): string {\n  const parent = id.parent\n  if (!parent) return 'reactive position'\n  if (ts.isCallExpression(parent) && parent.arguments[0] === id) {\n    if (ts.isIdentifier(parent.expression)) return `${parent.expression.text}(…)`\n    if (\n      ts.isPropertyAccessExpression(parent.expression) &&\n      ts.isIdentifier(parent.expression.name)\n    ) {\n      return `…${parent.expression.name.text}(…)`\n    }\n  }\n  if (\n    ts.isPropertyAssignment(parent) &&\n    parent.initializer === id &&\n    ts.isIdentifier(parent.name)\n  ) {\n    let ancestor: ts.Node | undefined = parent.parent\n    while (ancestor && !ts.isCallExpression(ancestor)) ancestor = ancestor.parent\n    const calleeName =\n      ancestor && ts.isCallExpression(ancestor) && ts.isIdentifier(ancestor.expression)\n        ? ancestor.expression.text\n        : null\n    return calleeName ? `${calleeName}({ ${parent.name.text}: … })` : `{ ${parent.name.text}: … }`\n  }\n  return 'reactive position'\n}\n\nfunction isCallableInitializer(init: ts.Expression | undefined): boolean {\n  if (!init) return false\n  if (ts.isArrowFunction(init) || ts.isFunctionExpression(init)) return true\n  if (\n    ts.isCallExpression(init) &&\n    ts.isIdentifier(init.expression) &&\n    init.expression.text === 'memo' &&\n    init.arguments.length >= 1\n  ) {\n    const inner = init.arguments[0]!\n    return ts.isArrowFunction(inner) || ts.isFunctionExpression(inner)\n  }\n  return false\n}\n\n/**\n * True when `n` is the target of a write (assignment LHS or\n * pre/postfix increment). We don't try to detect destructuring or\n * compound forms exhaustively — the common case (`foo = …`,\n * `foo += …`, `foo++`) covers the practical reassignment patterns.\n */\nfunction isWriteTarget(n: ts.Identifier): boolean {\n  const parent = n.parent\n  if (!parent) return false\n  if (ts.isBinaryExpression(parent) && parent.left === n) {\n    const op = parent.operatorToken.kind\n    return (\n      op === ts.SyntaxKind.EqualsToken ||\n      op === ts.SyntaxKind.PlusEqualsToken ||\n      op === ts.SyntaxKind.MinusEqualsToken ||\n      op === ts.SyntaxKind.AsteriskEqualsToken ||\n      op === ts.SyntaxKind.SlashEqualsToken\n    )\n  }\n  if (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent)) {\n    return (\n      parent.operator === ts.SyntaxKind.PlusPlusToken ||\n      parent.operator === ts.SyntaxKind.MinusMinusToken\n    )\n  }\n  return false\n}\n\nexport function noLetReactiveAccessorModule(): CompilerModule {\n  return {\n    name: 'no-let-reactive-accessor',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-let-reactive-accessor',\n        description:\n          '`let` / `var` accessor referenced at a reactive position — compiler falls back to FULL_MASK. Use `const`.',\n      },\n    ],\n    visitors: {\n      [ts.SyntaxKind.SourceFile]: (ctx, node) => {\n        const visited = node as ts.SourceFile\n        const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true)\n        // Collect single-declarator top-level `let foo = …` /\n        // `var foo = …` declarations with a callable initializer.\n        // Nested let/var are out of scope — the compiler resolver\n        // only follows top-level bindings anyway.\n        type Decl = {\n          name: string\n          kind: 'let' | 'var'\n          stmt: ts.VariableStatement\n        }\n        const decls: Decl[] = []\n        for (const stmt of sf.statements) {\n          if (!ts.isVariableStatement(stmt)) continue\n          const flags = stmt.declarationList.flags\n          let kind: 'let' | 'var' | null = null\n          if (flags & ts.NodeFlags.Let) kind = 'let'\n          else if (!(flags & ts.NodeFlags.Const)) kind = 'var'\n          if (!kind) continue\n          if (stmt.declarationList.declarations.length !== 1) continue\n          const decl = stmt.declarationList.declarations[0]!\n          if (!ts.isIdentifier(decl.name)) continue\n          if (!isCallableInitializer(decl.initializer)) continue\n          decls.push({ name: decl.name.text, kind, stmt })\n        }\n        if (decls.length === 0) return\n\n        // For each declaration, scan all identifier references in\n        // the file. Track reads at reactive positions and any\n        // write (reassignment). The declarator's own identifier is\n        // excluded.\n        const declMap = new Map(decls.map((d) => [d.name, d] as const))\n        const state = new Map<\n          string,\n          { hasReactive: boolean; reactiveContext: string | null; hasReassign: boolean }\n        >()\n        for (const d of decls) {\n          state.set(d.name, { hasReactive: false, reactiveContext: null, hasReassign: false })\n        }\n        // We don't track shadowing rigorously — a nested `const foo`\n        // would still let the outer `let foo`'s name fire. The\n        // common case (top-level let, no inner shadowing) is what we\n        // care about; shadowing is rare and an inner const would\n        // change behavior anyway.\n        const walk = (n: ts.Node): void => {\n          if (ts.isIdentifier(n) && declMap.has(n.text)) {\n            const decl = declMap.get(n.text)!\n            const slot = state.get(n.text)!\n            // Skip the declarator's own identifier node — that's not\n            // a reference, it's the declaration itself.\n            const declIdent = decl.stmt.declarationList.declarations[0]!.name\n            if (n !== declIdent) {\n              if (isWriteTarget(n)) slot.hasReassign = true\n              if (!slot.hasReactive && isAtReactivePosition(n)) {\n                slot.hasReactive = true\n                slot.reactiveContext = describeReactiveContext(n)\n              }\n            }\n          }\n          ts.forEachChild(n, walk)\n        }\n        walk(sf)\n\n        for (const d of decls) {\n          const s = state.get(d.name)!\n          if (!s.hasReactive) continue\n          const reactiveCtx = s.reactiveContext ?? 'reactive position'\n          const fixHint = s.hasReassign\n            ? `Either avoid reassignment (use \\`const\\` and a different binding for the new value) or accept the FULL_MASK fallback.`\n            : `Fix: change \\`${d.kind} ${d.name} = …\\` to \\`const ${d.name} = …\\`.`\n          const reasonClause = s.hasReassign\n            ? `, but it's reassigned later in the file — the compiler can't follow \\`${d.kind}\\` bindings`\n            : ''\n          ctx.reportDiagnostic({\n            id: 'llui/no-let-reactive-accessor',\n            severity: 'error',\n            category: 'reactivity',\n            message:\n              `\\`${d.name}\\` is a \\`${d.kind}\\`-bound accessor used at a reactive position ` +\n              `(${reactiveCtx})${reasonClause}. The compiler only follows \\`const\\` bindings; ` +\n              `\\`${d.kind}\\` falls back to FULL_MASK at runtime. ${fixHint}`,\n            location: {\n              file: sf.fileName,\n              range: rangeFromOffsets(sf.text, d.stmt.getStart(sf), d.stmt.getEnd()),\n            },\n          })\n        }\n      },\n    },\n  }\n}\n"]}

package/dist/modules/no-list-render-in-sample.d.ts

@@ -0,0 +1,3 @@
+import type { CompilerModule } from '../module.js';
+export declare function noListRenderInSampleModule(): CompilerModule;
+//# sourceMappingURL=no-list-render-in-sample.d.ts.map

package/dist/modules/no-list-render-in-sample.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-list-render-in-sample.d.ts","sourceRoot":"","sources":["../../src/modules/no-list-render-in-sample.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AA4BlD,wBAAgB,0BAA0B,IAAI,cAAc,CAqD3D"}

package/dist/modules/no-list-render-in-sample.js

@@ -0,0 +1,89 @@
+// `no-list-render-in-sample` — errors when `sample()` wraps a `.map()`
+// over state-derived items. `sample` is a one-shot imperative read; the
+// rendered cells go stale on in-place row updates. Use `each + ItemAccessor`.
+// Migrated from
+// `@llui/eslint-plugin/src/rules/no-list-render-in-sample.ts`.
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+const STATE_PARAM_NAMES = new Set(['s', 'state', 'props']);
+function isStateRootedMemberMap(n) {
+    if (!ts.isCallExpression(n))
+        return false;
+    if (!ts.isPropertyAccessExpression(n.expression))
+        return false;
+    if (!ts.isIdentifier(n.expression.name) || n.expression.name.text !== 'map')
+        return false;
+    let cursor = n.expression.expression;
+    while (ts.isPropertyAccessExpression(cursor))
+        cursor = cursor.expression;
+    if (!ts.isIdentifier(cursor))
+        return false;
+    return STATE_PARAM_NAMES.has(cursor.text);
+}
+function bodyContainsStateMap(body) {
+    let found = false;
+    const walk = (n) => {
+        if (found)
+            return;
+        if (isStateRootedMemberMap(n)) {
+            found = true;
+            return;
+        }
+        ts.forEachChild(n, walk);
+    };
+    walk(body);
+    return found;
+}
+export function noListRenderInSampleModule() {
+    return {
+        name: 'no-list-render-in-sample',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-list-render-in-sample',
+                description: '`sample()` wrapping `.map()` over state-derived items — cells go stale.',
+            },
+        ],
+        visitors: {
+            [ts.SyntaxKind.SourceFile]: (ctx, node) => {
+                const visited = node;
+                const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true);
+                const walk = (n) => {
+                    if (ts.isCallExpression(n)) {
+                        let isSample = false;
+                        if (ts.isIdentifier(n.expression) && n.expression.text === 'sample')
+                            isSample = true;
+                        else if (ts.isPropertyAccessExpression(n.expression) &&
+                            ts.isIdentifier(n.expression.name) &&
+                            n.expression.name.text === 'sample')
+                            isSample = true;
+                        if (isSample) {
+                            const arg = n.arguments[0];
+                            if (arg && (ts.isArrowFunction(arg) || ts.isFunctionExpression(arg)) && arg.body) {
+                                if (bodyContainsStateMap(arg.body)) {
+                                    ctx.reportDiagnostic({
+                                        id: 'llui/no-list-render-in-sample',
+                                        severity: 'error',
+                                        category: 'reactivity',
+                                        message: `\`sample()\` is a one-shot read — \`.map()\` over state-derived items inside ` +
+                                            `it captures the rows at view-construction and the cells go stale when row ` +
+                                            `data updates in place. Use \`each({ items: (s) => s.<list>, key, render })\` ` +
+                                            `and bind cells reactively via \`text(item.field)\` / ` +
+                                            `\`show({ when: () => item.flag() })\`. See cookbook recipe "List of editable rows."`,
+                                        location: {
+                                            file: sf.fileName,
+                                            range: rangeFromOffsets(sf.text, n.getStart(sf), n.getEnd()),
+                                        },
+                                    });
+                                }
+                            }
+                        }
+                    }
+                    ts.forEachChild(n, walk);
+                };
+                walk(sf);
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-list-render-in-sample.js.map

package/dist/modules/no-list-render-in-sample.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-list-render-in-sample.js","sourceRoot":"","sources":["../../src/modules/no-list-render-in-sample.ts"],"names":[],"mappings":"AAAA,uEAAuE;AACvE,wEAAwE;AACxE,8EAA8E;AAC9E,gBAAgB;AAChB,+DAA+D;AAE/D,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAGnD,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC,CAAA;AAE1D,SAAS,sBAAsB,CAAC,CAAU;IACxC,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC;QAAE,OAAO,KAAK,CAAA;IACzC,IAAI,CAAC,EAAE,CAAC,0BAA0B,CAAC,CAAC,CAAC,UAAU,CAAC;QAAE,OAAO,KAAK,CAAA;IAC9D,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,KAAK;QAAE,OAAO,KAAK,CAAA;IACzF,IAAI,MAAM,GAAkB,CAAC,CAAC,UAAU,CAAC,UAAU,CAAA;IACnD,OAAO,EAAE,CAAC,0BAA0B,CAAC,MAAM,CAAC;QAAE,MAAM,GAAG,MAAM,CAAC,UAAU,CAAA;IACxE,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,MAAM,CAAC;QAAE,OAAO,KAAK,CAAA;IAC1C,OAAO,iBAAiB,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,CAAA;AAC3C,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAa;IACzC,IAAI,KAAK,GAAG,KAAK,CAAA;IACjB,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;QAChC,IAAI,KAAK;YAAE,OAAM;QACjB,IAAI,sBAAsB,CAAC,CAAC,CAAC,EAAE,CAAC;YAC9B,KAAK,GAAG,IAAI,CAAA;YACZ,OAAM;QACR,CAAC;QACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA;IAC1B,CAAC,CAAA;IACD,IAAI,CAAC,IAAI,CAAC,CAAA;IACV,OAAO,KAAK,CAAA;AACd,CAAC;AAED,MAAM,UAAU,0BAA0B;IACxC,OAAO;QACL,IAAI,EAAE,0BAA0B;QAChC,eAAe,EAAE,QAAQ;QACzB,WAAW,EAAE;YACX;gBACE,EAAE,EAAE,+BAA+B;gBACnC,WAAW,EAAE,yEAAyE;aACvF;SACF;QACD,QAAQ,EAAE;YACR,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE;gBACxC,MAAM,OAAO,GAAG,IAAqB,CAAA;gBACrC,MAAM,EAAE,GAAG,EAAE,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAA;gBAC5F,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;oBAChC,IAAI,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC,EAAE,CAAC;wBAC3B,IAAI,QAAQ,GAAG,KAAK,CAAA;wBACpB,IAAI,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,IAAI,KAAK,QAAQ;4BAAE,QAAQ,GAAG,IAAI,CAAA;6BAC/E,IACH,EAAE,CAAC,0BAA0B,CAAC,CAAC,CAAC,UAAU,CAAC;4BAC3C,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC;4BAClC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ;4BAEnC,QAAQ,GAAG,IAAI,CAAA;wBACjB,IAAI,QAAQ,EAAE,CAAC;4BACb,MAAM,GAAG,GAAG,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAA;4BAC1B,IAAI,GAAG,IAAI,CAAC,EAAE,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;gCACjF,IAAI,oBAAoB,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;oCACnC,GAAG,CAAC,gBAAgB,CAAC;wCACnB,EAAE,EAAE,+BAA+B;wCACnC,QAAQ,EAAE,OAAO;wCACjB,QAAQ,EAAE,YAAY;wCACtB,OAAO,EACL,+EAA+E;4CAC/E,4EAA4E;4CAC5E,+EAA+E;4CAC/E,uDAAuD;4CACvD,qFAAqF;wCACvF,QAAQ,EAAE;4CACR,IAAI,EAAE,EAAE,CAAC,QAAQ;4CACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC;yCAC7D;qCACF,CAAC,CAAA;gCACJ,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;oBACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA;gBAC1B,CAAC,CAAA;gBACD,IAAI,CAAC,EAAE,CAAC,CAAA;YACV,CAAC;SACF;KACF,CAAA;AACH,CAAC","sourcesContent":["// `no-list-render-in-sample` — errors when `sample()` wraps a `.map()`\n// over state-derived items. `sample` is a one-shot imperative read; the\n// rendered cells go stale on in-place row updates. Use `each + ItemAccessor`.\n// Migrated from\n// `@llui/eslint-plugin/src/rules/no-list-render-in-sample.ts`.\n\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\n\nconst STATE_PARAM_NAMES = new Set(['s', 'state', 'props'])\n\nfunction isStateRootedMemberMap(n: ts.Node): boolean {\n  if (!ts.isCallExpression(n)) return false\n  if (!ts.isPropertyAccessExpression(n.expression)) return false\n  if (!ts.isIdentifier(n.expression.name) || n.expression.name.text !== 'map') return false\n  let cursor: ts.Expression = n.expression.expression\n  while (ts.isPropertyAccessExpression(cursor)) cursor = cursor.expression\n  if (!ts.isIdentifier(cursor)) return false\n  return STATE_PARAM_NAMES.has(cursor.text)\n}\n\nfunction bodyContainsStateMap(body: ts.Node): boolean {\n  let found = false\n  const walk = (n: ts.Node): void => {\n    if (found) return\n    if (isStateRootedMemberMap(n)) {\n      found = true\n      return\n    }\n    ts.forEachChild(n, walk)\n  }\n  walk(body)\n  return found\n}\n\nexport function noListRenderInSampleModule(): CompilerModule {\n  return {\n    name: 'no-list-render-in-sample',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-list-render-in-sample',\n        description: '`sample()` wrapping `.map()` over state-derived items — cells go stale.',\n      },\n    ],\n    visitors: {\n      [ts.SyntaxKind.SourceFile]: (ctx, node) => {\n        const visited = node as ts.SourceFile\n        const sf = ts.createSourceFile(visited.fileName, visited.text, ts.ScriptTarget.Latest, true)\n        const walk = (n: ts.Node): void => {\n          if (ts.isCallExpression(n)) {\n            let isSample = false\n            if (ts.isIdentifier(n.expression) && n.expression.text === 'sample') isSample = true\n            else if (\n              ts.isPropertyAccessExpression(n.expression) &&\n              ts.isIdentifier(n.expression.name) &&\n              n.expression.name.text === 'sample'\n            )\n              isSample = true\n            if (isSample) {\n              const arg = n.arguments[0]\n              if (arg && (ts.isArrowFunction(arg) || ts.isFunctionExpression(arg)) && arg.body) {\n                if (bodyContainsStateMap(arg.body)) {\n                  ctx.reportDiagnostic({\n                    id: 'llui/no-list-render-in-sample',\n                    severity: 'error',\n                    category: 'reactivity',\n                    message:\n                      `\\`sample()\\` is a one-shot read — \\`.map()\\` over state-derived items inside ` +\n                      `it captures the rows at view-construction and the cells go stale when row ` +\n                      `data updates in place. Use \\`each({ items: (s) => s.<list>, key, render })\\` ` +\n                      `and bind cells reactively via \\`text(item.field)\\` / ` +\n                      `\\`show({ when: () => item.flag() })\\`. See cookbook recipe \"List of editable rows.\"`,\n                    location: {\n                      file: sf.fileName,\n                      range: rangeFromOffsets(sf.text, n.getStart(sf), n.getEnd()),\n                    },\n                  })\n                }\n              }\n            }\n          }\n          ts.forEachChild(n, walk)\n        }\n        walk(sf)\n      },\n    },\n  }\n}\n"]}

package/dist/modules/no-sample-in-accessor.d.ts

@@ -0,0 +1,3 @@
+import type { CompilerModule } from '../module.js';
+export declare function noSampleInAccessorModule(): CompilerModule;
+//# sourceMappingURL=no-sample-in-accessor.d.ts.map

package/dist/modules/no-sample-in-accessor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-sample-in-accessor.d.ts","sourceRoot":"","sources":["../../src/modules/no-sample-in-accessor.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AAqDlD,wBAAgB,wBAAwB,IAAI,cAAc,CAiFzD"}