@llui/compiler 0.4.0 → 0.5.0

package/dist/lint-modules.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lint-modules.d.ts","sourceRoot":"","sources":["../src/lint-modules.ts"],"names":[],"mappings":"AAeA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AA2CjD;;;;;;GAMG;AACH,wBAAgB,iBAAiB,IAAI,cAAc,EAAE,CA4CpD"}
+{"version":3,"file":"lint-modules.d.ts","sourceRoot":"","sources":["../src/lint-modules.ts"],"names":[],"mappings":"AAeA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AA6CjD;;;;;;GAMG;AACH,wBAAgB,iBAAiB,IAAI,cAAc,EAAE,CA8CpD"}

package/dist/lint-modules.js

@@ -51,6 +51,8 @@ import { staticOnModule } from './modules/static-on.js';
 import { noListRenderInSampleModule } from './modules/no-list-render-in-sample.js';
 import { noSampleInAccessorModule } from './modules/no-sample-in-accessor.js';
 import { noSampleInReactivePositionModule } from './modules/no-sample-in-reactive-position.js';
+import { noSampleInEventHandlerModule } from './modules/no-sample-in-event-handler.js';
+import { noRepeatedItemCurrentModule } from './modules/no-repeated-item-current.js';
 import { agentEmitsDriftModule } from './modules/agent-emits-drift.js';
 import { agentMsgResolvableModule } from './modules/agent-msg-resolvable.js';
 /**
@@ -101,6 +103,8 @@ export function createLintModules() {
         noListRenderInSampleModule(),
         noSampleInAccessorModule(),
         noSampleInReactivePositionModule(),
+        noSampleInEventHandlerModule(),
+        noRepeatedItemCurrentModule(),
         agentEmitsDriftModule(),
         agentMsgResolvableModule(),
     ];

package/dist/lint-modules.js.map

@@ -1 +1 @@
-{"version":3,"file":"lint-modules.js","sourceRoot":"","sources":["../src/lint-modules.ts"],"names":[],"mappings":"AAAA,0BAA0B;AAC1B,EAAE;AACF,uEAAuE;AACvE,qEAAqE;AACrE,wEAAwE;AACxE,uEAAuE;AACvE,yEAAyE;AACzE,6CAA6C;AAC7C,EAAE;AACF,kEAAkE;AAClE,yEAAyE;AACzE,uEAAuE;AACvE,wEAAwE;AACxE,gEAAgE;AAGhE,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAA;AACvE,OAAO,EAAE,wBAAwB,EAAE,MAAM,oCAAoC,CAAA;AAC7E,OAAO,EAAE,uBAAuB,EAAE,MAAM,mCAAmC,CAAA;AAC3E,OAAO,EAAE,yBAAyB,EAAE,MAAM,qCAAqC,CAAA;AAC/E,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAA;AACjE,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,8BAA8B,EAAE,MAAM,yCAAyC,CAAA;AACxF,OAAO,EAAE,yBAAyB,EAAE,MAAM,qCAAqC,CAAA;AAC/E,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,sBAAsB,EAAE,MAAM,gCAAgC,CAAA;AACvE,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,+BAA+B,EAAE,MAAM,0CAA0C,CAAA;AAC1F,OAAO,EAAE,oCAAoC,EAAE,MAAM,gDAAgD,CAAA;AACrG,OAAO,EAAE,mCAAmC,EAAE,MAAM,+CAA+C,CAAA;AACnG,OAAO,EAAE,gCAAgC,EAAE,MAAM,2CAA2C,CAAA;AAC5F,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAA;AAC3D,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAA;AAChE,OAAO,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAA;AAClE,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,qCAAqC,EAAE,MAAM,mDAAmD,CAAA;AACzG,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAA;AACxE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAA;AACvD,OAAO,EAAE,0BAA0B,EAAE,MAAM,uCAAuC,CAAA;AAClF,OAAO,EAAE,wBAAwB,EAAE,MAAM,oCAAoC,CAAA;AAC7E,OAAO,EAAE,gCAAgC,EAAE,MAAM,6CAA6C,CAAA;AAC9F,OAAO,EAAE,qBAAqB,EAAE,MAAM,gCAAgC,CAAA;AACtE,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAE5E;;;;;;GAMG;AACH,MAAM,UAAU,iBAAiB;IAC/B,OAAO;QACL,qBAAqB,EAAE;QACvB,iBAAiB,EAAE;QACnB,qBAAqB,EAAE;QACvB,wBAAwB,EAAE;QAC1B,uBAAuB,EAAE;QACzB,yBAAyB,EAAE;QAC3B,wBAAwB,EAAE;QAC1B,mBAAmB,EAAE;QACrB,0BAA0B,EAAE;QAC5B,8BAA8B,EAAE;QAChC,yBAAyB,EAAE;QAC3B,wBAAwB,EAAE;QAC1B,sBAAsB,EAAE;QACxB,2BAA2B,EAAE;QAC7B,0BAA0B,EAAE;QAC5B,0BAA0B,EAAE;QAC5B,wBAAwB,EAAE;QAC1B,2BAA2B,EAAE;QAC7B,2BAA2B,EAAE;QAC7B,+BAA+B,EAAE;QACjC,oCAAoC,EAAE;QACtC,mCAAmC,EAAE;QACrC,gCAAgC,EAAE;QAClC,0BAA0B,EAAE;QAC5B,gBAAgB,EAAE;QAClB,qBAAqB,EAAE;QACvB,mBAAmB,EAAE;QACrB,mBAAmB,EAAE;QACrB,qBAAqB,EAAE;QACvB,iBAAiB,EAAE;QACnB,qBAAqB,EAAE;QACvB,qCAAqC,EAAE;QACvC,qBAAqB,EAAE;QACvB,sBAAsB,EAAE;QACxB,iBAAiB,EAAE;QACnB,cAAc,EAAE;QAChB,0BAA0B,EAAE;QAC5B,wBAAwB,EAAE;QAC1B,gCAAgC,EAAE;QAClC,qBAAqB,EAAE;QACvB,wBAAwB,EAAE;KAC3B,CAAA;AACH,CAAC","sourcesContent":["// Always-on lint modules.\n//\n// Every entry here is a zero-arg `CompilerModule` factory whose output\n// is registered unconditionally on every `transformLlui` invocation.\n// The function is the single source of truth — `transform.ts`'s active-\n// module list spreads it, and `scripts/generate-rule-docs.ts` calls it\n// to enumerate diagnostic IDs for the rule reference. Adding or removing\n// a rule in one place propagates everywhere.\n//\n// Note: this does NOT include modules with per-file options (e.g.\n// `maskLegendModule({ fieldBits, fieldBitsHi })`, `coreSynthesisModule`,\n// etc.) — those stay constructed inline in `transform.ts` because they\n// take per-file context. The `compilerStampModule` is unconditional but\n// it's an instance-not-factory; appended separately by callers.\n\nimport type { CompilerModule } from './module.js'\nimport { bitmaskOverflowModule } from './modules/bitmask-overflow.js'\nimport { asyncUpdateModule } from './modules/async-update.js'\nimport { mapOnStateArrayModule } from './modules/map-on-state-array.js'\nimport { nestedSendInUpdateModule } from './modules/nested-send-in-update.js'\nimport { directStateInViewModule } from './modules/direct-state-in-view.js'\nimport { imperativeDomInViewModule } from './modules/imperative-dom-in-view.js'\nimport { accessorSideEffectModule } from './modules/accessor-side-effect.js'\nimport { stateMutationModule } from './modules/state-mutation.js'\nimport { effectWithoutHandlerModule } from './modules/effect-without-handler.js'\nimport { exhaustiveEffectHandlingModule } from './modules/exhaustive-effect-handling.js'\nimport { noEagerItemAccessorModule } from './modules/no-eager-item-accessor.js'\nimport { pureUpdateFunctionModule } from './modules/pure-update-function.js'\nimport { exhaustiveUpdateModule } from './modules/exhaustive-update.js'\nimport { noLetReactiveAccessorModule } from './modules/no-let-reactive-accessor.js'\nimport { eachClosureViolationModule } from './modules/each-closure-violation.js'\nimport { stringEffectCallbackModule } from './modules/string-effect-callback.js'\nimport { agentMissingIntentModule } from './modules/agent-missing-intent.js'\nimport { agentWarningOnConfirmModule } from './modules/agent-warning-on-confirm.js'\nimport { agentExampleOnPayloadModule } from './modules/agent-example-on-payload.js'\nimport { agentExclusiveAnnotationsModule } from './modules/agent-exclusive-annotations.js'\nimport { agentOptionalFieldUndocumentedModule } from './modules/agent-optional-field-undocumented.js'\nimport { agentTagsendTranslatorMissingModule } from './modules/agent-tagsend-translator-missing.js'\nimport { agentNonextractableHandlerModule } from './modules/agent-nonextractable-handler.js'\nimport { subappRequiresReasonModule } from './modules/subapp-requires-reason.js'\nimport { emptyPropsModule } from './modules/empty-props.js'\nimport { forgottenSpreadModule } from './modules/forgotten-spread.js'\nimport { accessibilityModule } from './modules/accessibility.js'\nimport { viewBagImportModule } from './modules/view-bag-import.js'\nimport { controlledInputModule } from './modules/controlled-input.js'\nimport { missingMemoModule } from './modules/missing-memo.js'\nimport { namespaceImportModule } from './modules/namespace-import.js'\nimport { noBarrelImportWhenSubpathExistsModule } from './modules/no-barrel-import-when-subpath-exists.js'\nimport { formBoilerplateModule } from './modules/form-boilerplate.js'\nimport { spreadInChildrenModule } from './modules/spread-in-children.js'\nimport { staticItemsModule } from './modules/static-items.js'\nimport { staticOnModule } from './modules/static-on.js'\nimport { noListRenderInSampleModule } from './modules/no-list-render-in-sample.js'\nimport { noSampleInAccessorModule } from './modules/no-sample-in-accessor.js'\nimport { noSampleInReactivePositionModule } from './modules/no-sample-in-reactive-position.js'\nimport { agentEmitsDriftModule } from './modules/agent-emits-drift.js'\nimport { agentMsgResolvableModule } from './modules/agent-msg-resolvable.js'\n\n/**\n * Construct fresh instances of every always-on lint module.\n *\n * Returns a new array per call. Modules are stateful within a single\n * `ModuleRegistry.run()` (slot accumulators), so reusing instances\n * across files would leak state — always call this once per file.\n */\nexport function createLintModules(): CompilerModule[] {\n  return [\n    bitmaskOverflowModule(),\n    asyncUpdateModule(),\n    mapOnStateArrayModule(),\n    nestedSendInUpdateModule(),\n    directStateInViewModule(),\n    imperativeDomInViewModule(),\n    accessorSideEffectModule(),\n    stateMutationModule(),\n    effectWithoutHandlerModule(),\n    exhaustiveEffectHandlingModule(),\n    noEagerItemAccessorModule(),\n    pureUpdateFunctionModule(),\n    exhaustiveUpdateModule(),\n    noLetReactiveAccessorModule(),\n    eachClosureViolationModule(),\n    stringEffectCallbackModule(),\n    agentMissingIntentModule(),\n    agentWarningOnConfirmModule(),\n    agentExampleOnPayloadModule(),\n    agentExclusiveAnnotationsModule(),\n    agentOptionalFieldUndocumentedModule(),\n    agentTagsendTranslatorMissingModule(),\n    agentNonextractableHandlerModule(),\n    subappRequiresReasonModule(),\n    emptyPropsModule(),\n    forgottenSpreadModule(),\n    accessibilityModule(),\n    viewBagImportModule(),\n    controlledInputModule(),\n    missingMemoModule(),\n    namespaceImportModule(),\n    noBarrelImportWhenSubpathExistsModule(),\n    formBoilerplateModule(),\n    spreadInChildrenModule(),\n    staticItemsModule(),\n    staticOnModule(),\n    noListRenderInSampleModule(),\n    noSampleInAccessorModule(),\n    noSampleInReactivePositionModule(),\n    agentEmitsDriftModule(),\n    agentMsgResolvableModule(),\n  ]\n}\n"]}
+{"version":3,"file":"lint-modules.js","sourceRoot":"","sources":["../src/lint-modules.ts"],"names":[],"mappings":"AAAA,0BAA0B;AAC1B,EAAE;AACF,uEAAuE;AACvE,qEAAqE;AACrE,wEAAwE;AACxE,uEAAuE;AACvE,yEAAyE;AACzE,6CAA6C;AAC7C,EAAE;AACF,kEAAkE;AAClE,yEAAyE;AACzE,uEAAuE;AACvE,wEAAwE;AACxE,gEAAgE;AAGhE,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAA;AACvE,OAAO,EAAE,wBAAwB,EAAE,MAAM,oCAAoC,CAAA;AAC7E,OAAO,EAAE,uBAAuB,EAAE,MAAM,mCAAmC,CAAA;AAC3E,OAAO,EAAE,yBAAyB,EAAE,MAAM,qCAAqC,CAAA;AAC/E,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAA;AACjE,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,8BAA8B,EAAE,MAAM,yCAAyC,CAAA;AACxF,OAAO,EAAE,yBAAyB,EAAE,MAAM,qCAAqC,CAAA;AAC/E,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,sBAAsB,EAAE,MAAM,gCAAgC,CAAA;AACvE,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAC5E,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,+BAA+B,EAAE,MAAM,0CAA0C,CAAA;AAC1F,OAAO,EAAE,oCAAoC,EAAE,MAAM,gDAAgD,CAAA;AACrG,OAAO,EAAE,mCAAmC,EAAE,MAAM,+CAA+C,CAAA;AACnG,OAAO,EAAE,gCAAgC,EAAE,MAAM,2CAA2C,CAAA;AAC5F,OAAO,EAAE,0BAA0B,EAAE,MAAM,qCAAqC,CAAA;AAChF,OAAO,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAA;AAC3D,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAA;AAChE,OAAO,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAA;AAClE,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,qCAAqC,EAAE,MAAM,mDAAmD,CAAA;AACzG,OAAO,EAAE,qBAAqB,EAAE,MAAM,+BAA+B,CAAA;AACrE,OAAO,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAA;AACxE,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAC7D,OAAO,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAA;AACvD,OAAO,EAAE,0BAA0B,EAAE,MAAM,uCAAuC,CAAA;AAClF,OAAO,EAAE,wBAAwB,EAAE,MAAM,oCAAoC,CAAA;AAC7E,OAAO,EAAE,gCAAgC,EAAE,MAAM,6CAA6C,CAAA;AAC9F,OAAO,EAAE,4BAA4B,EAAE,MAAM,yCAAyC,CAAA;AACtF,OAAO,EAAE,2BAA2B,EAAE,MAAM,uCAAuC,CAAA;AACnF,OAAO,EAAE,qBAAqB,EAAE,MAAM,gCAAgC,CAAA;AACtE,OAAO,EAAE,wBAAwB,EAAE,MAAM,mCAAmC,CAAA;AAE5E;;;;;;GAMG;AACH,MAAM,UAAU,iBAAiB;IAC/B,OAAO;QACL,qBAAqB,EAAE;QACvB,iBAAiB,EAAE;QACnB,qBAAqB,EAAE;QACvB,wBAAwB,EAAE;QAC1B,uBAAuB,EAAE;QACzB,yBAAyB,EAAE;QAC3B,wBAAwB,EAAE;QAC1B,mBAAmB,EAAE;QACrB,0BAA0B,EAAE;QAC5B,8BAA8B,EAAE;QAChC,yBAAyB,EAAE;QAC3B,wBAAwB,EAAE;QAC1B,sBAAsB,EAAE;QACxB,2BAA2B,EAAE;QAC7B,0BAA0B,EAAE;QAC5B,0BAA0B,EAAE;QAC5B,wBAAwB,EAAE;QAC1B,2BAA2B,EAAE;QAC7B,2BAA2B,EAAE;QAC7B,+BAA+B,EAAE;QACjC,oCAAoC,EAAE;QACtC,mCAAmC,EAAE;QACrC,gCAAgC,EAAE;QAClC,0BAA0B,EAAE;QAC5B,gBAAgB,EAAE;QAClB,qBAAqB,EAAE;QACvB,mBAAmB,EAAE;QACrB,mBAAmB,EAAE;QACrB,qBAAqB,EAAE;QACvB,iBAAiB,EAAE;QACnB,qBAAqB,EAAE;QACvB,qCAAqC,EAAE;QACvC,qBAAqB,EAAE;QACvB,sBAAsB,EAAE;QACxB,iBAAiB,EAAE;QACnB,cAAc,EAAE;QAChB,0BAA0B,EAAE;QAC5B,wBAAwB,EAAE;QAC1B,gCAAgC,EAAE;QAClC,4BAA4B,EAAE;QAC9B,2BAA2B,EAAE;QAC7B,qBAAqB,EAAE;QACvB,wBAAwB,EAAE;KAC3B,CAAA;AACH,CAAC","sourcesContent":["// Always-on lint modules.\n//\n// Every entry here is a zero-arg `CompilerModule` factory whose output\n// is registered unconditionally on every `transformLlui` invocation.\n// The function is the single source of truth — `transform.ts`'s active-\n// module list spreads it, and `scripts/generate-rule-docs.ts` calls it\n// to enumerate diagnostic IDs for the rule reference. Adding or removing\n// a rule in one place propagates everywhere.\n//\n// Note: this does NOT include modules with per-file options (e.g.\n// `maskLegendModule({ fieldBits, fieldBitsHi })`, `coreSynthesisModule`,\n// etc.) — those stay constructed inline in `transform.ts` because they\n// take per-file context. The `compilerStampModule` is unconditional but\n// it's an instance-not-factory; appended separately by callers.\n\nimport type { CompilerModule } from './module.js'\nimport { bitmaskOverflowModule } from './modules/bitmask-overflow.js'\nimport { asyncUpdateModule } from './modules/async-update.js'\nimport { mapOnStateArrayModule } from './modules/map-on-state-array.js'\nimport { nestedSendInUpdateModule } from './modules/nested-send-in-update.js'\nimport { directStateInViewModule } from './modules/direct-state-in-view.js'\nimport { imperativeDomInViewModule } from './modules/imperative-dom-in-view.js'\nimport { accessorSideEffectModule } from './modules/accessor-side-effect.js'\nimport { stateMutationModule } from './modules/state-mutation.js'\nimport { effectWithoutHandlerModule } from './modules/effect-without-handler.js'\nimport { exhaustiveEffectHandlingModule } from './modules/exhaustive-effect-handling.js'\nimport { noEagerItemAccessorModule } from './modules/no-eager-item-accessor.js'\nimport { pureUpdateFunctionModule } from './modules/pure-update-function.js'\nimport { exhaustiveUpdateModule } from './modules/exhaustive-update.js'\nimport { noLetReactiveAccessorModule } from './modules/no-let-reactive-accessor.js'\nimport { eachClosureViolationModule } from './modules/each-closure-violation.js'\nimport { stringEffectCallbackModule } from './modules/string-effect-callback.js'\nimport { agentMissingIntentModule } from './modules/agent-missing-intent.js'\nimport { agentWarningOnConfirmModule } from './modules/agent-warning-on-confirm.js'\nimport { agentExampleOnPayloadModule } from './modules/agent-example-on-payload.js'\nimport { agentExclusiveAnnotationsModule } from './modules/agent-exclusive-annotations.js'\nimport { agentOptionalFieldUndocumentedModule } from './modules/agent-optional-field-undocumented.js'\nimport { agentTagsendTranslatorMissingModule } from './modules/agent-tagsend-translator-missing.js'\nimport { agentNonextractableHandlerModule } from './modules/agent-nonextractable-handler.js'\nimport { subappRequiresReasonModule } from './modules/subapp-requires-reason.js'\nimport { emptyPropsModule } from './modules/empty-props.js'\nimport { forgottenSpreadModule } from './modules/forgotten-spread.js'\nimport { accessibilityModule } from './modules/accessibility.js'\nimport { viewBagImportModule } from './modules/view-bag-import.js'\nimport { controlledInputModule } from './modules/controlled-input.js'\nimport { missingMemoModule } from './modules/missing-memo.js'\nimport { namespaceImportModule } from './modules/namespace-import.js'\nimport { noBarrelImportWhenSubpathExistsModule } from './modules/no-barrel-import-when-subpath-exists.js'\nimport { formBoilerplateModule } from './modules/form-boilerplate.js'\nimport { spreadInChildrenModule } from './modules/spread-in-children.js'\nimport { staticItemsModule } from './modules/static-items.js'\nimport { staticOnModule } from './modules/static-on.js'\nimport { noListRenderInSampleModule } from './modules/no-list-render-in-sample.js'\nimport { noSampleInAccessorModule } from './modules/no-sample-in-accessor.js'\nimport { noSampleInReactivePositionModule } from './modules/no-sample-in-reactive-position.js'\nimport { noSampleInEventHandlerModule } from './modules/no-sample-in-event-handler.js'\nimport { noRepeatedItemCurrentModule } from './modules/no-repeated-item-current.js'\nimport { agentEmitsDriftModule } from './modules/agent-emits-drift.js'\nimport { agentMsgResolvableModule } from './modules/agent-msg-resolvable.js'\n\n/**\n * Construct fresh instances of every always-on lint module.\n *\n * Returns a new array per call. Modules are stateful within a single\n * `ModuleRegistry.run()` (slot accumulators), so reusing instances\n * across files would leak state — always call this once per file.\n */\nexport function createLintModules(): CompilerModule[] {\n  return [\n    bitmaskOverflowModule(),\n    asyncUpdateModule(),\n    mapOnStateArrayModule(),\n    nestedSendInUpdateModule(),\n    directStateInViewModule(),\n    imperativeDomInViewModule(),\n    accessorSideEffectModule(),\n    stateMutationModule(),\n    effectWithoutHandlerModule(),\n    exhaustiveEffectHandlingModule(),\n    noEagerItemAccessorModule(),\n    pureUpdateFunctionModule(),\n    exhaustiveUpdateModule(),\n    noLetReactiveAccessorModule(),\n    eachClosureViolationModule(),\n    stringEffectCallbackModule(),\n    agentMissingIntentModule(),\n    agentWarningOnConfirmModule(),\n    agentExampleOnPayloadModule(),\n    agentExclusiveAnnotationsModule(),\n    agentOptionalFieldUndocumentedModule(),\n    agentTagsendTranslatorMissingModule(),\n    agentNonextractableHandlerModule(),\n    subappRequiresReasonModule(),\n    emptyPropsModule(),\n    forgottenSpreadModule(),\n    accessibilityModule(),\n    viewBagImportModule(),\n    controlledInputModule(),\n    missingMemoModule(),\n    namespaceImportModule(),\n    noBarrelImportWhenSubpathExistsModule(),\n    formBoilerplateModule(),\n    spreadInChildrenModule(),\n    staticItemsModule(),\n    staticOnModule(),\n    noListRenderInSampleModule(),\n    noSampleInAccessorModule(),\n    noSampleInReactivePositionModule(),\n    noSampleInEventHandlerModule(),\n    noRepeatedItemCurrentModule(),\n    agentEmitsDriftModule(),\n    agentMsgResolvableModule(),\n  ]\n}\n"]}

package/dist/modules/no-repeated-item-current.d.ts

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

package/dist/modules/no-repeated-item-current.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-repeated-item-current.d.ts","sourceRoot":"","sources":["../../src/modules/no-repeated-item-current.ts"],"names":[],"mappings":"AA6BA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AAoGlD,wBAAgB,2BAA2B,IAAI,cAAc,CA4C5D"}

package/dist/modules/no-repeated-item-current.js

@@ -0,0 +1,164 @@
+// `no-repeated-item-current` — warns when an `each.render` callback's
+// accessor body calls `item.current()` more than once with a property
+// chain after each call (e.g. `item.current().facts[K]` repeated
+// inside the same text/show.when accessor).
+//
+// Two reasons the pattern is dangerous:
+//
+//   1. **Bitmask trap.** `item.current().X` hides the read from the
+//      compiler's static analyzer — the accessor falls back to
+//      FULL_MASK and fires on every state change instead of only when
+//      `X` changes.
+//   2. **Reconcile-race undefined.** Repeated `.current()` calls
+//      across a single accessor body can observe intermediate state
+//      during a structural transition. The chained property access
+//      then throws `Cannot read properties of undefined (reading
+//      'X')`. The dungeonlogs 2026-05-20 report named exactly this
+//      class of bug.
+//
+// The fix is one of:
+//   - destructure once: `const e = item.current(); use e.X, e.Y`
+//   - project to a row type in `items` so each cell becomes a simple
+//     field read (`item.X` shorthand)
+//
+// Severity: warn. A single `item.current()` call is fine (sometimes
+// necessary, e.g. guarding for primitive T's `current` accessor); the
+// warning fires only when the same accessor body calls it 2+ times.
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+/** True when `node` is `item.current()` — bare `item` identifier root. */
+function isItemCurrentCall(node) {
+    if (!ts.isCallExpression(node))
+        return false;
+    if (node.arguments.length !== 0)
+        return false;
+    if (!ts.isPropertyAccessExpression(node.expression))
+        return false;
+    const obj = node.expression.expression;
+    if (!ts.isIdentifier(obj) || obj.text !== 'item')
+        return false;
+    const name = node.expression.name;
+    return ts.isIdentifier(name) && name.text === 'current';
+}
+/**
+ * Find direct-children `item.current()` calls of `body` that are
+ * followed by a property access (e.g. `item.current().X`). Skip nested
+ * function bodies — a `.current()` inside an inner arrow runs in a
+ * different scope.
+ */
+function findChainedItemCurrents(body) {
+    const out = [];
+    const walk = (n) => {
+        if (isItemCurrentCall(n)) {
+            // We only flag chained access — `item.current()` alone is fine
+            // (used as a return value, passed to a helper, etc.).
+            const parent = n.parent;
+            if (parent && ts.isPropertyAccessExpression(parent) && parent.expression === n) {
+                out.push(n);
+            }
+            else if (parent && ts.isElementAccessExpression(parent) && parent.expression === n) {
+                out.push(n);
+            }
+            return;
+        }
+        if ((ts.isArrowFunction(n) || ts.isFunctionExpression(n) || ts.isFunctionDeclaration(n)) &&
+            n !== body) {
+            return;
+        }
+        ts.forEachChild(n, walk);
+    };
+    walk(body);
+    return out;
+}
+/**
+ * Walk a function body and report when any inner accessor (arrow arg
+ * to `text` / `unsafeHtml` / `h.show({ when })` / similar) calls
+ * `item.current().X` more than once.
+ */
+function checkRenderBody(body, sf, report) {
+    const visit = (n) => {
+        if (ts.isArrowFunction(n) || ts.isFunctionExpression(n)) {
+            const calls = findChainedItemCurrents(n.body);
+            if (calls.length >= 2) {
+                // Report the SECOND call — the first one is fine on its own;
+                // the warning is "you're doing this repeatedly."
+                report(calls[1]);
+            }
+        }
+        ts.forEachChild(n, visit);
+    };
+    visit(body);
+    void sf;
+}
+/**
+ * Detect an `each(...)` or `h.each(...)` call. Returns the render
+ * function's body if present.
+ */
+function eachRenderBody(call) {
+    let isEach = false;
+    if (ts.isIdentifier(call.expression) && call.expression.text === 'each')
+        isEach = true;
+    else if (ts.isPropertyAccessExpression(call.expression) &&
+        ts.isIdentifier(call.expression.name) &&
+        call.expression.name.text === 'each') {
+        isEach = true;
+    }
+    if (!isEach)
+        return undefined;
+    const arg = call.arguments[0];
+    if (!arg || !ts.isObjectLiteralExpression(arg))
+        return undefined;
+    for (const prop of arg.properties) {
+        if (ts.isPropertyAssignment(prop) &&
+            ts.isIdentifier(prop.name) &&
+            prop.name.text === 'render' &&
+            (ts.isArrowFunction(prop.initializer) || ts.isFunctionExpression(prop.initializer))) {
+            return prop.initializer.body;
+        }
+    }
+    return undefined;
+}
+export function noRepeatedItemCurrentModule() {
+    return {
+        name: 'no-repeated-item-current',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-repeated-item-current',
+                description: 'Repeated `item.current().X` calls inside the same accessor — bitmask falls back to FULL_MASK and chained access can throw during reconcile races. Destructure once or project to a row type.',
+            },
+        ],
+        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)) {
+                        const body = eachRenderBody(n);
+                        if (body) {
+                            checkRenderBody(body, sf, (offender) => {
+                                ctx.reportDiagnostic({
+                                    id: 'llui/no-repeated-item-current',
+                                    severity: 'warning',
+                                    category: 'reactivity',
+                                    message: `Repeated \`item.current()\` calls inside an each.render accessor. The compiler can't trace through ` +
+                                        `\`item.current().X\` so the binding falls back to FULL_MASK (fires on every state change), and ` +
+                                        `chained access can throw \`Cannot read properties of undefined\` during structural reconciles. ` +
+                                        `Either destructure once at the top — \`const e = item.current(); /* use e.X, e.Y */\` — or project ` +
+                                        `to a row type in \`items\` so each cell becomes a simple field read (\`item.X\` shorthand).`,
+                                    location: {
+                                        file: sf.fileName,
+                                        range: rangeFromOffsets(sf.text, offender.getStart(sf), offender.getEnd()),
+                                    },
+                                });
+                            });
+                        }
+                    }
+                    ts.forEachChild(n, walk);
+                };
+                walk(sf);
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-repeated-item-current.js.map

package/dist/modules/no-repeated-item-current.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-repeated-item-current.js","sourceRoot":"","sources":["../../src/modules/no-repeated-item-current.ts"],"names":[],"mappings":"AAAA,sEAAsE;AACtE,sEAAsE;AACtE,iEAAiE;AACjE,4CAA4C;AAC5C,EAAE;AACF,wCAAwC;AACxC,EAAE;AACF,oEAAoE;AACpE,+DAA+D;AAC/D,sEAAsE;AACtE,oBAAoB;AACpB,iEAAiE;AACjE,oEAAoE;AACpE,mEAAmE;AACnE,iEAAiE;AACjE,mEAAmE;AACnE,qBAAqB;AACrB,EAAE;AACF,qBAAqB;AACrB,iEAAiE;AACjE,qEAAqE;AACrE,sCAAsC;AACtC,EAAE;AACF,oEAAoE;AACpE,sEAAsE;AACtE,oEAAoE;AAEpE,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAGnD,0EAA0E;AAC1E,SAAS,iBAAiB,CAAC,IAAa;IACtC,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAA;IAC5C,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAA;IAC7C,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,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,IAAI,KAAK,MAAM;QAAE,OAAO,KAAK,CAAA;IAC9D,MAAM,IAAI,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,CAAA;IACjC,OAAO,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,CAAA;AACzD,CAAC;AAED;;;;;GAKG;AACH,SAAS,uBAAuB,CAAC,IAAa;IAC5C,MAAM,GAAG,GAAwB,EAAE,CAAA;IACnC,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;QAChC,IAAI,iBAAiB,CAAC,CAAC,CAAC,EAAE,CAAC;YACzB,+DAA+D;YAC/D,sDAAsD;YACtD,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAA;YACvB,IAAI,MAAM,IAAI,EAAE,CAAC,0BAA0B,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,UAAU,KAAK,CAAC,EAAE,CAAC;gBAC/E,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;YACb,CAAC;iBAAM,IAAI,MAAM,IAAI,EAAE,CAAC,yBAAyB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,UAAU,KAAK,CAAC,EAAE,CAAC;gBACrF,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;YACb,CAAC;YACD,OAAM;QACR,CAAC;QACD,IACE,CAAC,EAAE,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;YACpF,CAAC,KAAK,IAAI,EACV,CAAC;YACD,OAAM;QACR,CAAC;QACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA;IAC1B,CAAC,CAAA;IACD,IAAI,CAAC,IAAI,CAAC,CAAA;IACV,OAAO,GAAG,CAAA;AACZ,CAAC;AAED;;;;GAIG;AACH,SAAS,eAAe,CACtB,IAAa,EACb,EAAiB,EACjB,MAA6C;IAE7C,MAAM,KAAK,GAAG,CAAC,CAAU,EAAQ,EAAE;QACjC,IAAI,EAAE,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,CAAC,CAAC,EAAE,CAAC;YACxD,MAAM,KAAK,GAAG,uBAAuB,CAAC,CAAC,CAAC,IAAI,CAAC,CAAA;YAC7C,IAAI,KAAK,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;gBACtB,6DAA6D;gBAC7D,iDAAiD;gBACjD,MAAM,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,CAAA;YACnB,CAAC;QACH,CAAC;QACD,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,KAAK,CAAC,CAAA;IAC3B,CAAC,CAAA;IACD,KAAK,CAAC,IAAI,CAAC,CAAA;IACX,KAAK,EAAE,CAAA;AACT,CAAC;AAED;;;GAGG;AACH,SAAS,cAAc,CAAC,IAAuB;IAC7C,IAAI,MAAM,GAAG,KAAK,CAAA;IAClB,IAAI,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,IAAI,KAAK,MAAM;QAAE,MAAM,GAAG,IAAI,CAAA;SACjF,IACH,EAAE,CAAC,0BAA0B,CAAC,IAAI,CAAC,UAAU,CAAC;QAC9C,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QACrC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,MAAM,EACpC,CAAC;QACD,MAAM,GAAG,IAAI,CAAA;IACf,CAAC;IACD,IAAI,CAAC,MAAM;QAAE,OAAO,SAAS,CAAA;IAC7B,MAAM,GAAG,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAA;IAC7B,IAAI,CAAC,GAAG,IAAI,CAAC,EAAE,CAAC,yBAAyB,CAAC,GAAG,CAAC;QAAE,OAAO,SAAS,CAAA;IAChE,KAAK,MAAM,IAAI,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;QAClC,IACE,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC;YAC7B,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;YAC1B,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ;YAC3B,CAAC,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,EACnF,CAAC;YACD,OAAO,IAAI,CAAC,WAAW,CAAC,IAAI,CAAA;QAC9B,CAAC;IACH,CAAC;IACD,OAAO,SAAS,CAAA;AAClB,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,8LAA8L;aACjM;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,MAAM,IAAI,GAAG,cAAc,CAAC,CAAC,CAAC,CAAA;wBAC9B,IAAI,IAAI,EAAE,CAAC;4BACT,eAAe,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,QAAQ,EAAE,EAAE;gCACrC,GAAG,CAAC,gBAAgB,CAAC;oCACnB,EAAE,EAAE,+BAA+B;oCACnC,QAAQ,EAAE,SAAS;oCACnB,QAAQ,EAAE,YAAY;oCACtB,OAAO,EACL,qGAAqG;wCACrG,iGAAiG;wCACjG,iGAAiG;wCACjG,qGAAqG;wCACrG,6FAA6F;oCAC/F,QAAQ,EAAE;wCACR,IAAI,EAAE,EAAE,CAAC,QAAQ;wCACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,QAAQ,CAAC,MAAM,EAAE,CAAC;qCAC3E;iCACF,CAAC,CAAA;4BACJ,CAAC,CAAC,CAAA;wBACJ,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-repeated-item-current` — warns when an `each.render` callback's\n// accessor body calls `item.current()` more than once with a property\n// chain after each call (e.g. `item.current().facts[K]` repeated\n// inside the same text/show.when accessor).\n//\n// Two reasons the pattern is dangerous:\n//\n//   1. **Bitmask trap.** `item.current().X` hides the read from the\n//      compiler's static analyzer — the accessor falls back to\n//      FULL_MASK and fires on every state change instead of only when\n//      `X` changes.\n//   2. **Reconcile-race undefined.** Repeated `.current()` calls\n//      across a single accessor body can observe intermediate state\n//      during a structural transition. The chained property access\n//      then throws `Cannot read properties of undefined (reading\n//      'X')`. The dungeonlogs 2026-05-20 report named exactly this\n//      class of bug.\n//\n// The fix is one of:\n//   - destructure once: `const e = item.current(); use e.X, e.Y`\n//   - project to a row type in `items` so each cell becomes a simple\n//     field read (`item.X` shorthand)\n//\n// Severity: warn. A single `item.current()` call is fine (sometimes\n// necessary, e.g. guarding for primitive T's `current` accessor); the\n// warning fires only when the same accessor body calls it 2+ times.\n\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\n\n/** True when `node` is `item.current()` — bare `item` identifier root. */\nfunction isItemCurrentCall(node: ts.Node): node is ts.CallExpression {\n  if (!ts.isCallExpression(node)) return false\n  if (node.arguments.length !== 0) return false\n  if (!ts.isPropertyAccessExpression(node.expression)) return false\n  const obj = node.expression.expression\n  if (!ts.isIdentifier(obj) || obj.text !== 'item') return false\n  const name = node.expression.name\n  return ts.isIdentifier(name) && name.text === 'current'\n}\n\n/**\n * Find direct-children `item.current()` calls of `body` that are\n * followed by a property access (e.g. `item.current().X`). Skip nested\n * function bodies — a `.current()` inside an inner arrow runs in a\n * different scope.\n */\nfunction findChainedItemCurrents(body: ts.Node): ts.CallExpression[] {\n  const out: ts.CallExpression[] = []\n  const walk = (n: ts.Node): void => {\n    if (isItemCurrentCall(n)) {\n      // We only flag chained access — `item.current()` alone is fine\n      // (used as a return value, passed to a helper, etc.).\n      const parent = n.parent\n      if (parent && ts.isPropertyAccessExpression(parent) && parent.expression === n) {\n        out.push(n)\n      } else if (parent && ts.isElementAccessExpression(parent) && parent.expression === n) {\n        out.push(n)\n      }\n      return\n    }\n    if (\n      (ts.isArrowFunction(n) || ts.isFunctionExpression(n) || ts.isFunctionDeclaration(n)) &&\n      n !== body\n    ) {\n      return\n    }\n    ts.forEachChild(n, walk)\n  }\n  walk(body)\n  return out\n}\n\n/**\n * Walk a function body and report when any inner accessor (arrow arg\n * to `text` / `unsafeHtml` / `h.show({ when })` / similar) calls\n * `item.current().X` more than once.\n */\nfunction checkRenderBody(\n  body: ts.Node,\n  sf: ts.SourceFile,\n  report: (offender: ts.CallExpression) => void,\n): void {\n  const visit = (n: ts.Node): void => {\n    if (ts.isArrowFunction(n) || ts.isFunctionExpression(n)) {\n      const calls = findChainedItemCurrents(n.body)\n      if (calls.length >= 2) {\n        // Report the SECOND call — the first one is fine on its own;\n        // the warning is \"you're doing this repeatedly.\"\n        report(calls[1]!)\n      }\n    }\n    ts.forEachChild(n, visit)\n  }\n  visit(body)\n  void sf\n}\n\n/**\n * Detect an `each(...)` or `h.each(...)` call. Returns the render\n * function's body if present.\n */\nfunction eachRenderBody(call: ts.CallExpression): ts.Node | undefined {\n  let isEach = false\n  if (ts.isIdentifier(call.expression) && call.expression.text === 'each') isEach = true\n  else if (\n    ts.isPropertyAccessExpression(call.expression) &&\n    ts.isIdentifier(call.expression.name) &&\n    call.expression.name.text === 'each'\n  ) {\n    isEach = true\n  }\n  if (!isEach) return undefined\n  const arg = call.arguments[0]\n  if (!arg || !ts.isObjectLiteralExpression(arg)) return undefined\n  for (const prop of arg.properties) {\n    if (\n      ts.isPropertyAssignment(prop) &&\n      ts.isIdentifier(prop.name) &&\n      prop.name.text === 'render' &&\n      (ts.isArrowFunction(prop.initializer) || ts.isFunctionExpression(prop.initializer))\n    ) {\n      return prop.initializer.body\n    }\n  }\n  return undefined\n}\n\nexport function noRepeatedItemCurrentModule(): CompilerModule {\n  return {\n    name: 'no-repeated-item-current',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-repeated-item-current',\n        description:\n          'Repeated `item.current().X` calls inside the same accessor — bitmask falls back to FULL_MASK and chained access can throw during reconcile races. Destructure once or project to a row type.',\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            const body = eachRenderBody(n)\n            if (body) {\n              checkRenderBody(body, sf, (offender) => {\n                ctx.reportDiagnostic({\n                  id: 'llui/no-repeated-item-current',\n                  severity: 'warning',\n                  category: 'reactivity',\n                  message:\n                    `Repeated \\`item.current()\\` calls inside an each.render accessor. The compiler can't trace through ` +\n                    `\\`item.current().X\\` so the binding falls back to FULL_MASK (fires on every state change), and ` +\n                    `chained access can throw \\`Cannot read properties of undefined\\` during structural reconciles. ` +\n                    `Either destructure once at the top — \\`const e = item.current(); /* use e.X, e.Y */\\` — or project ` +\n                    `to a row type in \\`items\\` so each cell becomes a simple field read (\\`item.X\\` shorthand).`,\n                  location: {\n                    file: sf.fileName,\n                    range: rangeFromOffsets(sf.text, offender.getStart(sf), offender.getEnd()),\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-event-handler.d.ts

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

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-sample-in-event-handler.d.ts","sourceRoot":"","sources":["../../src/modules/no-sample-in-event-handler.ts"],"names":[],"mappings":"AAkBA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,cAAc,CAAA;AA0ClD,wBAAgB,4BAA4B,IAAI,cAAc,CAgD7D"}

package/dist/modules/no-sample-in-event-handler.js

@@ -0,0 +1,101 @@
+// `no-sample-in-event-handler` — errors when `sample()` / `h.sample()`
+// appears inside an event-handler property (`onClick`, `onInput`,
+// `onSubmit`, …). Event handlers run AFTER mount, with no active
+// render context, so the runtime `sample()` throws `[LLui] sample()
+// can only be called inside a component's view() function`.
+//
+// Catching at compile time turns "open the dev console, click the
+// button, see the error" into a build failure on the offending file.
+// Aligned with the framework's "compile-time errors not lint warnings"
+// philosophy.
+//
+// The right pattern is to capture at render time:
+//   const id = h.sample((s) => s.id)
+//   button({ onClick: () => send({ type: 'select', id }) })
+// or to use the mount handle: `handle.getState()` inside the handler.
+import ts from 'typescript';
+import { rangeFromOffsets } from '../diagnostic.js';
+const EVENT_HANDLER_KEY_RE = /^on[A-Z]/;
+function isSampleCall(n) {
+    if (!ts.isCallExpression(n))
+        return false;
+    if (ts.isIdentifier(n.expression) && n.expression.text === 'sample')
+        return true;
+    if (ts.isPropertyAccessExpression(n.expression) &&
+        ts.isIdentifier(n.expression.name) &&
+        n.expression.name.text === 'sample') {
+        return true;
+    }
+    return false;
+}
+function findFirstSampleInside(body) {
+    let found;
+    const walk = (n) => {
+        if (found)
+            return;
+        if (isSampleCall(n)) {
+            found = n;
+            return;
+        }
+        // DON'T descend into nested functions — sample() inside an inner
+        // function (e.g. a setTimeout body or another arrow that captures
+        // the event handler's closure) runs at a different time and has
+        // its own context check at runtime. Only the directly-synchronous
+        // sample call in the handler body is the trap this rule targets.
+        if ((ts.isArrowFunction(n) || ts.isFunctionExpression(n) || ts.isFunctionDeclaration(n)) &&
+            n !== body) {
+            return;
+        }
+        ts.forEachChild(n, walk);
+    };
+    walk(body);
+    return found;
+}
+export function noSampleInEventHandlerModule() {
+    return {
+        name: 'no-sample-in-event-handler',
+        compilerVersion: '^0.3.0',
+        diagnostics: [
+            {
+                id: 'llui/no-sample-in-event-handler',
+                description: '`sample()` inside an event handler — handlers run with no render context; throws at runtime. Capture at render time instead.',
+            },
+        ],
+        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.isPropertyAssignment(n)) {
+                        const key = n.name;
+                        if (ts.isIdentifier(key) && EVENT_HANDLER_KEY_RE.test(key.text)) {
+                            const value = n.initializer;
+                            if (ts.isArrowFunction(value) || ts.isFunctionExpression(value)) {
+                                const offender = findFirstSampleInside(value.body);
+                                if (offender) {
+                                    ctx.reportDiagnostic({
+                                        id: 'llui/no-sample-in-event-handler',
+                                        severity: 'error',
+                                        category: 'reactivity',
+                                        message: `\`sample()\` is being called inside the \`${key.text}\` handler. Handlers run AFTER mount with no active render context, so this throws at runtime ` +
+                                            `(\`[LLui] sample() can only be called inside a component's view() function\`). Capture the value at render time instead: ` +
+                                            `\`const id = h.sample(s => s.id); button({ ${key.text}: () => send({ ..., id }) })\`. ` +
+                                            `If you need the LATEST state at click time (rare), use the mount handle: ` +
+                                            `\`handle.getState()\` inside the handler.`,
+                                        location: {
+                                            file: sf.fileName,
+                                            range: rangeFromOffsets(sf.text, offender.getStart(sf), offender.getEnd()),
+                                        },
+                                    });
+                                }
+                            }
+                        }
+                    }
+                    ts.forEachChild(n, walk);
+                };
+                walk(sf);
+            },
+        },
+    };
+}
+//# sourceMappingURL=no-sample-in-event-handler.js.map

package/dist/modules/no-sample-in-event-handler.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-sample-in-event-handler.js","sourceRoot":"","sources":["../../src/modules/no-sample-in-event-handler.ts"],"names":[],"mappings":"AAAA,uEAAuE;AACvE,kEAAkE;AAClE,iEAAiE;AACjE,oEAAoE;AACpE,4DAA4D;AAC5D,EAAE;AACF,kEAAkE;AAClE,qEAAqE;AACrE,uEAAuE;AACvE,cAAc;AACd,EAAE;AACF,kDAAkD;AAClD,qCAAqC;AACrC,4DAA4D;AAC5D,sEAAsE;AAEtE,OAAO,EAAE,MAAM,YAAY,CAAA;AAC3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAA;AAGnD,MAAM,oBAAoB,GAAG,UAAU,CAAA;AAEvC,SAAS,YAAY,CAAC,CAAU;IAC9B,IAAI,CAAC,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC;QAAE,OAAO,KAAK,CAAA;IACzC,IAAI,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,IAAI,KAAK,QAAQ;QAAE,OAAO,IAAI,CAAA;IAChF,IACE,EAAE,CAAC,0BAA0B,CAAC,CAAC,CAAC,UAAU,CAAC;QAC3C,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC;QAClC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ,EACnC,CAAC;QACD,OAAO,IAAI,CAAA;IACb,CAAC;IACD,OAAO,KAAK,CAAA;AACd,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAa;IAC1C,IAAI,KAAoC,CAAA;IACxC,MAAM,IAAI,GAAG,CAAC,CAAU,EAAQ,EAAE;QAChC,IAAI,KAAK;YAAE,OAAM;QACjB,IAAI,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC;YACpB,KAAK,GAAG,CAAsB,CAAA;YAC9B,OAAM;QACR,CAAC;QACD,iEAAiE;QACjE,kEAAkE;QAClE,gEAAgE;QAChE,kEAAkE;QAClE,iEAAiE;QACjE,IACE,CAAC,EAAE,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;YACpF,CAAC,KAAK,IAAI,EACV,CAAC;YACD,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,4BAA4B;IAC1C,OAAO;QACL,IAAI,EAAE,4BAA4B;QAClC,eAAe,EAAE,QAAQ;QACzB,WAAW,EAAE;YACX;gBACE,EAAE,EAAE,iCAAiC;gBACrC,WAAW,EACT,8HAA8H;aACjI;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,oBAAoB,CAAC,CAAC,CAAC,EAAE,CAAC;wBAC/B,MAAM,GAAG,GAAG,CAAC,CAAC,IAAI,CAAA;wBAClB,IAAI,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,oBAAoB,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;4BAChE,MAAM,KAAK,GAAG,CAAC,CAAC,WAAW,CAAA;4BAC3B,IAAI,EAAE,CAAC,eAAe,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,oBAAoB,CAAC,KAAK,CAAC,EAAE,CAAC;gCAChE,MAAM,QAAQ,GAAG,qBAAqB,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;gCAClD,IAAI,QAAQ,EAAE,CAAC;oCACb,GAAG,CAAC,gBAAgB,CAAC;wCACnB,EAAE,EAAE,iCAAiC;wCACrC,QAAQ,EAAE,OAAO;wCACjB,QAAQ,EAAE,YAAY;wCACtB,OAAO,EACL,6CAA6C,GAAG,CAAC,IAAI,gGAAgG;4CACrJ,2HAA2H;4CAC3H,8CAA8C,GAAG,CAAC,IAAI,kCAAkC;4CACxF,2EAA2E;4CAC3E,2CAA2C;wCAC7C,QAAQ,EAAE;4CACR,IAAI,EAAE,EAAE,CAAC,QAAQ;4CACjB,KAAK,EAAE,gBAAgB,CAAC,EAAE,CAAC,IAAI,EAAE,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC,EAAE,QAAQ,CAAC,MAAM,EAAE,CAAC;yCAC3E;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-sample-in-event-handler` — errors when `sample()` / `h.sample()`\n// appears inside an event-handler property (`onClick`, `onInput`,\n// `onSubmit`, …). Event handlers run AFTER mount, with no active\n// render context, so the runtime `sample()` throws `[LLui] sample()\n// can only be called inside a component's view() function`.\n//\n// Catching at compile time turns \"open the dev console, click the\n// button, see the error\" into a build failure on the offending file.\n// Aligned with the framework's \"compile-time errors not lint warnings\"\n// philosophy.\n//\n// The right pattern is to capture at render time:\n//   const id = h.sample((s) => s.id)\n//   button({ onClick: () => send({ type: 'select', id }) })\n// or to use the mount handle: `handle.getState()` inside the handler.\n\nimport ts from 'typescript'\nimport { rangeFromOffsets } from '../diagnostic.js'\nimport type { CompilerModule } from '../module.js'\n\nconst EVENT_HANDLER_KEY_RE = /^on[A-Z]/\n\nfunction isSampleCall(n: ts.Node): boolean {\n  if (!ts.isCallExpression(n)) return false\n  if (ts.isIdentifier(n.expression) && n.expression.text === 'sample') return true\n  if (\n    ts.isPropertyAccessExpression(n.expression) &&\n    ts.isIdentifier(n.expression.name) &&\n    n.expression.name.text === 'sample'\n  ) {\n    return true\n  }\n  return false\n}\n\nfunction findFirstSampleInside(body: ts.Node): ts.CallExpression | undefined {\n  let found: ts.CallExpression | undefined\n  const walk = (n: ts.Node): void => {\n    if (found) return\n    if (isSampleCall(n)) {\n      found = n as ts.CallExpression\n      return\n    }\n    // DON'T descend into nested functions — sample() inside an inner\n    // function (e.g. a setTimeout body or another arrow that captures\n    // the event handler's closure) runs at a different time and has\n    // its own context check at runtime. Only the directly-synchronous\n    // sample call in the handler body is the trap this rule targets.\n    if (\n      (ts.isArrowFunction(n) || ts.isFunctionExpression(n) || ts.isFunctionDeclaration(n)) &&\n      n !== body\n    ) {\n      return\n    }\n    ts.forEachChild(n, walk)\n  }\n  walk(body)\n  return found\n}\n\nexport function noSampleInEventHandlerModule(): CompilerModule {\n  return {\n    name: 'no-sample-in-event-handler',\n    compilerVersion: '^0.3.0',\n    diagnostics: [\n      {\n        id: 'llui/no-sample-in-event-handler',\n        description:\n          '`sample()` inside an event handler — handlers run with no render context; throws at runtime. Capture at render time instead.',\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.isPropertyAssignment(n)) {\n            const key = n.name\n            if (ts.isIdentifier(key) && EVENT_HANDLER_KEY_RE.test(key.text)) {\n              const value = n.initializer\n              if (ts.isArrowFunction(value) || ts.isFunctionExpression(value)) {\n                const offender = findFirstSampleInside(value.body)\n                if (offender) {\n                  ctx.reportDiagnostic({\n                    id: 'llui/no-sample-in-event-handler',\n                    severity: 'error',\n                    category: 'reactivity',\n                    message:\n                      `\\`sample()\\` is being called inside the \\`${key.text}\\` handler. Handlers run AFTER mount with no active render context, so this throws at runtime ` +\n                      `(\\`[LLui] sample() can only be called inside a component's view() function\\`). Capture the value at render time instead: ` +\n                      `\\`const id = h.sample(s => s.id); button({ ${key.text}: () => send({ ..., id }) })\\`. ` +\n                      `If you need the LATEST state at click time (rare), use the mount handle: ` +\n                      `\\`handle.getState()\\` inside the handler.`,\n                    location: {\n                      file: sf.fileName,\n                      range: rangeFromOffsets(sf.text, offender.getStart(sf), offender.getEnd()),\n                    },\n                  })\n                }\n              }\n            }\n          }\n          ts.forEachChild(n, walk)\n        }\n        walk(sf)\n      },\n    },\n  }\n}\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llui/compiler",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "type": "module",
   "sideEffects": false,
   "exports": {