stimeo-ui 0.16.0 → 0.17.0

package/dist/controllers/nested_form_controller.js

@@ -1,8 +1,5 @@
 import { Controller } from '@hotwired/stimulus';
 
-// src/controllers/nested_form_controller.ts
-
-// src/utils/announce.ts
 function announce(message, options = {}) {
   const text = message.trim();
   if (text.length === 0) return;
@@ -19,77 +16,153 @@ function fillTemplate(template, values) {
   });
 }
 
-// src/utils/attribute_lease.ts
-var AttributeLease = class {
+function sharedRegistry(key, create) {
+  const symbol = Symbol.for(key);
+  const scope = globalThis;
+  const existing = scope[symbol];
+  if (existing !== void 0) return existing;
+  const registry = create();
+  Object.defineProperty(globalThis, symbol, { value: registry });
+  return registry;
+}
+
+function isAttributeValue(value) {
+  return value === null || typeof value === "string";
+}
+function readLeaseRecord(element, record) {
+  try {
+    const parsed = JSON.parse(element.getAttribute(record) ?? "null");
+    if (!Array.isArray(parsed)) return void 0;
+    const [original, written, ...rest] = parsed;
+    const linked = rest[0] !== false;
+    const beneath = linked ? rest : rest.slice(1);
+    if (!beneath.every((under) => typeof under === "string")) return void 0;
+    return isAttributeValue(original) && isAttributeValue(written) ? { original, written, linked, beneath } : void 0;
+  } catch {
+    return void 0;
+  }
+}
+var AttributeLease = class _AttributeLease {
+  static #registry = sharedRegistry(
+    "stimeo-ui.attribute-lease.registry.v1",
+    () => ({ stacks: /* @__PURE__ */ new WeakMap(), names: /* @__PURE__ */ new Map() })
+  );
   #attribute;
-  #records = /* @__PURE__ */ new Map();
-  /** @param attribute - The attribute whose temporary values this lease owns. */
-  constructor(attribute) {
+  #record;
+  #base;
+  #held = /* @__PURE__ */ new Set();
+  constructor(attribute, owner, options = {}) {
     this.#attribute = attribute;
+    this.#record = `data-${owner}-${attribute}-lease`;
+    this.#base = options.base === true;
+    const names = _AttributeLease.#registry.names;
+    names.set(attribute, (names.get(attribute) ?? /* @__PURE__ */ new Set()).add(this.#record));
   }
-  /** Writes or removes the leased attribute while preserving its authored value. */
   write(element, value) {
-    const existing = this.#records.get(element);
-    if (existing) {
-      existing.written = value;
-    } else {
-      this.#records.set(element, {
-        original: element.getAttribute(this.#attribute),
-        written: value
-      });
-    }
-    this.#reflect(element, value);
+    const stack = this.#stack(element);
+    const hold = this.#take(stack) ?? this.#acquire(element, stack);
+    this.#held.add(element);
+    const shows = !this.#covered(element, stack, hold);
+    hold.written = value;
+    this.#mark(element, stack);
+    if (shows) this.#reflect(element, value);
   }
-  /** Returns one lease without overwriting a value subsequently authored by a consumer. */
   return(element) {
-    const record = this.#records.get(element);
-    if (!record) return;
-    this.#records.delete(element);
-    const stillOwned = element.getAttribute(this.#attribute) === record.written;
-    if (stillOwned) this.#reflect(element, record.original);
+    const held = this.#held.delete(element);
+    const known = _AttributeLease.#registry.stacks.get(element)?.has(this.#attribute);
+    const stack = held || known || element.hasAttribute(this.#record) ? this.#stack(element) : [];
+    const hold = this.#take(stack);
+    if (!hold) return false;
+    const carries = element.getAttribute(this.#attribute) === hold.written;
+    const owns = carries && !this.#covered(element, stack, hold);
+    const index = stack.indexOf(hold);
+    stack.splice(index, 1);
+    const above = stack[index];
+    if (above) above.linked &&= hold.linked;
+    this.#mark(element, stack);
+    if (!owns) return true;
+    const below = hold.linked ? stack[index - 1] : void 0;
+    this.#reflect(element, below ? below.written : hold.original);
+    return true;
+  }
+  returnAll() {
+    for (const element of Array.from(this.#held)) this.return(element);
+  }
+  #stack(element) {
+    const stacks = _AttributeLease.#registry.stacks;
+    const byAttribute = stacks.get(element) ?? /* @__PURE__ */ new Map();
+    stacks.set(element, byAttribute);
+    const stack = byAttribute.get(this.#attribute) ?? this.#seed(element);
+    byAttribute.set(this.#attribute, stack);
+    return stack;
+  }
+  #seed(element) {
+    const found = [];
+    for (const record of _AttributeLease.#registry.names.get(this.#attribute) ?? []) {
+      const read = readLeaseRecord(element, record);
+      if (read) found.push({ ...read, record });
+    }
+    const depth = ({ beneath }) => beneath.filter((name) => found.some((other) => other.record === name)).length;
+    return found.sort((one, other) => depth(one) - depth(other)).map(({ record, written, original, linked }, index) => ({
+      lease: null,
+      record,
+      written,
+      original,
+      linked: linked && index > 0
+    }));
+  }
+  #take(stack) {
+    const hold = stack.find((held) => held.lease === this) ?? stack.find((held) => held.lease === null && held.record === this.#record);
+    if (hold) hold.lease = this;
+    return hold;
+  }
+  #acquire(element, stack) {
+    const current = element.getAttribute(this.#attribute);
+    const top = stack.at(-1);
+    const bottom = stack[0];
+    const over = top?.written === current ? top : void 0;
+    const from = this.#base ? bottom : over;
+    const hold = {
+      lease: this,
+      record: this.#record,
+      written: current,
+      original: from ? from.original : current,
+      linked: !this.#base && over !== void 0
+    };
+    if (this.#base && bottom) {
+      bottom.linked = true;
+      stack.unshift(hold);
+    } else stack.push(hold);
+    return hold;
+  }
+  #covered(element, stack, hold) {
+    const current = element.getAttribute(this.#attribute);
+    return stack.slice(stack.indexOf(hold) + 1).some((above) => above.written === current);
+  }
+  #mark(element, stack) {
+    const records = /* @__PURE__ */ new Map([[this.#record, null]]);
+    stack.forEach((hold, index) => {
+      const names = stack.slice(0, index).map((under) => under.record);
+      const beneath = [...new Set(names.reverse())].filter((name) => name !== hold.record);
+      const marker = hold.linked || beneath.length === 0 ? [] : [false];
+      records.set(
+        hold.record,
+        hold.written === hold.original && beneath.length === 0 && index === stack.length - 1 ? null : JSON.stringify([hold.original, hold.written, ...marker, ...beneath])
+      );
+    });
+    for (const [name, recorded] of records) {
+      if (element.getAttribute(name) === recorded) continue;
+      if (recorded === null) element.removeAttribute(name);
+      else element.setAttribute(name, recorded);
+    }
   }
-  /** Reflects only a real value transition, avoiding self-triggered mutation work. */
   #reflect(element, value) {
     if (element.getAttribute(this.#attribute) === value) return;
     if (value === null) element.removeAttribute(this.#attribute);
     else element.setAttribute(this.#attribute, value);
   }
-  /** Returns every outstanding lease using the same ownership check as {@link return}. */
-  returnAll() {
-    for (const element of Array.from(this.#records.keys())) this.return(element);
-  }
-};
-
-// src/utils/before_cache_reset.ts
-var BeforeCacheReset = class _BeforeCacheReset {
-  /** Every subscribed instance, iterated by the one shared document listener. */
-  static #subscribers = /* @__PURE__ */ new Set();
-  /** The shared listener; installed while at least one instance is subscribed. */
-  static #onBeforeCache = () => {
-    for (const subscriber of _BeforeCacheReset.#subscribers) subscriber.#rewind();
-  };
-  #rewind;
-  /** @param rewind - the pass that returns this controller's state to its initial form. */
-  constructor(rewind) {
-    this.#rewind = rewind;
-  }
-  /** Subscribes to `turbo:before-cache`; call from `connect()`. Idempotent. */
-  activate() {
-    const first = _BeforeCacheReset.#subscribers.size === 0;
-    _BeforeCacheReset.#subscribers.add(this);
-    if (first) {
-      document.addEventListener("turbo:before-cache", _BeforeCacheReset.#onBeforeCache);
-    }
-  }
-  /** Unsubscribes; call from `disconnect()`. Safe when never subscribed. */
-  deactivate() {
-    _BeforeCacheReset.#subscribers.delete(this);
-    if (_BeforeCacheReset.#subscribers.size > 0) return;
-    document.removeEventListener("turbo:before-cache", _BeforeCacheReset.#onBeforeCache);
-  }
 };
 
-// src/utils/focus_candidate.ts
 function inheritsFieldsetDisabled(control) {
   let fieldset = control.closest("fieldset[disabled]");
   while (fieldset) {
@@ -99,8 +172,19 @@ function inheritsFieldsetDisabled(control) {
   }
   return false;
 }
+function unslotted(element) {
+  return element.assignedSlot === null && element.parentElement?.shadowRoot != null;
+}
+function flatTreeParent(element) {
+  if (unslotted(element)) return null;
+  const parent = element.assignedSlot ?? element.parentNode;
+  if (parent instanceof ShadowRoot) return parent.host;
+  return parent instanceof Element ? parent : null;
+}
 function canTakeFocus(element) {
-  if (element.closest("[hidden], [inert]")) return false;
+  for (let node = element; node; node = flatTreeParent(node)) {
+    if (unslotted(node) || node.matches("[hidden], [inert]")) return false;
+  }
   if (element instanceof HTMLInputElement && element.type === "hidden") return false;
   if (!("disabled" in element)) return true;
   if (element.disabled) return false;
@@ -164,27 +248,33 @@ function firstTabStop(root) {
   return null;
 }
 
-// src/utils/microtask_coalescer.ts
+var ListenerSet = class {
+  #abort = new AbortController();
+  add(target, type, handler, options) {
+    target.addEventListener(type, handler, { ...options, signal: this.#abort.signal });
+  }
+  dispose() {
+    this.#abort.abort();
+    this.#abort = new AbortController();
+  }
+};
+
 var MicrotaskCoalescer = class {
   #run;
   #queued = false;
   #active = false;
   #generation = 0;
-  /** @param run - the single reconciliation pass, invoked at most once per batch. */
   constructor(run) {
     this.#run = run;
   }
-  /** Opens the window in which {@link schedule} is honoured; call from `connect()`. */
   activate() {
     this.#active = true;
   }
-  /** Closes the window and drops any pending pass; call from `disconnect()`. */
   cancel() {
     this.#active = false;
     this.#queued = false;
     this.#generation += 1;
   }
-  /** Requests one pass after the batch settles. Idempotent; inert outside the window. */
   schedule() {
     if (!this.#active || this.#queued) return;
     this.#queued = true;
@@ -197,56 +287,195 @@ var MicrotaskCoalescer = class {
   }
 };
 
-// src/utils/tabindex_loan.ts
-var TabindexLoan = class {
+var MorphRenderWatcher = class {
+  #listeners = new ListenerSet();
+  #pass;
+  #element = null;
+  #onMorph = () => this.#pass.schedule();
+  constructor(run) {
+    this.#pass = new MicrotaskCoalescer(run);
+  }
+  observe(element) {
+    if (this.#element === element) return;
+    this.disconnect();
+    this.#element = element;
+    this.#pass.activate();
+    this.#listeners.add(element, "turbo:morph-element", this.#onMorph);
+  }
+  schedule() {
+    this.#pass.schedule();
+  }
+  disconnect() {
+    this.#listeners.dispose();
+    this.#pass.cancel();
+    this.#element = null;
+  }
+};
+
+var NUMBER_BOUNDS = {
+  nonNegative: { finite: true, min: 0 }};
+function matchesNumberBounds(value, bounds) {
+  if (!Number.isFinite(value)) {
+    const direction = value === Infinity ? "positive" : value === -Infinity ? "negative" : null;
+    if (direction === null) return false;
+    if (bounds.allowInfinity !== "both" && bounds.allowInfinity !== direction) return false;
+  }
+  if (bounds.min !== void 0 && value < bounds.min) return false;
+  if (bounds.max !== void 0 && value > bounds.max) return false;
+  if (bounds.exclusiveMin !== void 0 && value <= bounds.exclusiveMin) return false;
+  if (bounds.integer && !Number.isInteger(value)) return false;
+  if (bounds.allowedValues !== void 0 && !bounds.allowedValues.includes(value)) return false;
+  return true;
+}
+
+function readNumber(raw, fallback, bounds) {
+  return matchesNumberBounds(raw, bounds) ? raw : fallback;
+}
+
+var NumberValueReader = class {
+  #lastRejected = /* @__PURE__ */ new Map();
+  read(owner, name, raw, fallback, bounds) {
+    const resolved = readNumber(raw, fallback, bounds);
+    if (matchesNumberBounds(raw, bounds)) {
+      this.#lastRejected.delete(name);
+      return resolved;
+    }
+    const attribute = `data-${owner.identifier}-${name.replace(/[A-Z]/g, (letter) => `-${letter.toLowerCase()}`)}-value`;
+    const literal = owner.element.getAttribute(attribute);
+    if (literal === null) {
+      this.#lastRejected.delete(name);
+      return resolved;
+    }
+    if (this.#lastRejected.get(name) !== literal) {
+      this.#lastRejected.set(name, literal);
+      console.warn(
+        `Stimeo UI: "${owner.identifier}" has an invalid number Value "${name}" declaration ${JSON.stringify(literal)}; using ${fallback}.`
+      );
+    }
+    return resolved;
+  }
+};
+
+function queryOpenDescendants(root, selector) {
+  const matches = [];
+  const roots = [root];
+  for (const scope of roots) {
+    matches.push(...scope.querySelectorAll(selector));
+    if (scope.nodeType === Node.ELEMENT_NODE) {
+      const shadow = scope.shadowRoot;
+      if (shadow) roots.push(shadow);
+    }
+    for (const element of scope.querySelectorAll("*")) {
+      if (element.shadowRoot) roots.push(element.shadowRoot);
+    }
+  }
+  return matches;
+}
+
+var TabindexLoan = class _TabindexLoan {
+  static #registry = sharedRegistry(
+    "stimeo-ui.tabindex-loan.registry.v1",
+    () => ({ stacks: /* @__PURE__ */ new WeakMap(), names: /* @__PURE__ */ new Set() })
+  );
   #value;
+  #record;
   #lent = /* @__PURE__ */ new Set();
-  /** Returns live loans before Turbo can copy them into its page snapshot. */
-  #beforeCache = new BeforeCacheReset(() => this.returnAll());
-  /**
-   * @param value - the `tabindex` to lend. `"-1"` (the default) is
-   *   programmatically focusable but not a Tab stop; `"0"` is a real Tab stop,
-   *   which a scroll region with no focusable content of its own needs.
-   */
-  constructor(value = "-1") {
+  constructor(value, owner) {
     this.#value = value;
+    this.#record = `data-${owner}-tabindex-loan`;
+    _TabindexLoan.#registry.names.add(this.#record);
   }
-  /** Lends `element` the value; no-ops when it already carries a `tabindex`. */
   lend(element) {
-    if (element.hasAttribute("tabindex")) return;
-    element.setAttribute("tabindex", this.#value);
+    const stack = this.#stack(element);
+    const current = element.getAttribute("tabindex");
+    const held = stack.find((hold) => hold.loan === this) ?? stack.find(
+      (hold) => hold.loan === null && hold.record === this.#record && hold.value === this.#value
+    );
+    const linked = stack.at(-1)?.value === current;
+    if (!held && current !== null && !linked) return;
     this.#lent.add(element);
-    this.#beforeCache.activate();
+    if (held) held.loan = this;
+    else stack.push({ loan: this, record: this.#record, value: this.#value, linked });
+    this.#mark(element, stack);
+    if (current === null || !held && current !== this.#value) {
+      element.setAttribute("tabindex", this.#value);
+    }
   }
-  /** Takes back every loan whose value is still the one that was lent. */
   returnAll() {
     for (const element of this.#lent) {
-      if (element.getAttribute("tabindex") === this.#value) element.removeAttribute("tabindex");
+      const stack = this.#stack(element);
+      const hold = stack.find((held) => held.loan === this);
+      if (hold) this.#release(element, stack, hold);
     }
     this.#lent.clear();
-    this.#beforeCache.deactivate();
+  }
+  reclaim(element) {
+    if (element.getAttribute(this.#record) !== this.#value) return;
+    const stack = this.#stack(element);
+    const hold = stack.find((held) => held.loan === null && held.record === this.#record);
+    if (hold && !element.matches(":focus")) this.#release(element, stack, hold);
+  }
+  reclaimWithin(root) {
+    for (const element of queryOpenDescendants(root, `[${this.#record}]`)) {
+      this.reclaim(element);
+    }
+  }
+  #stack(element) {
+    const stacks = _TabindexLoan.#registry.stacks;
+    const stack = stacks.get(element) ?? this.#seed(element);
+    stacks.set(element, stack);
+    return stack;
+  }
+  #seed(element) {
+    const current = element.getAttribute("tabindex");
+    const holds = [];
+    for (const record of _TabindexLoan.#registry.names) {
+      const value = element.getAttribute(record);
+      if (value !== null) holds.push({ loan: null, record, value, linked: true });
+    }
+    holds.sort((one, other) => Number(one.value === current) - Number(other.value === current));
+    for (const [index, hold] of holds.entries()) hold.linked = index > 0;
+    return holds;
+  }
+  #release(element, stack, hold) {
+    const current = element.getAttribute("tabindex");
+    const index = stack.indexOf(hold);
+    const owns = current === hold.value && !stack.slice(index + 1).some((above2) => above2.value === current);
+    stack.splice(index, 1);
+    const above = stack[index];
+    if (above) above.linked &&= hold.linked;
+    this.#mark(element, stack);
+    if (!owns) return;
+    const below = hold.linked ? stack[index - 1] : void 0;
+    if (!below) element.removeAttribute("tabindex");
+    else if (below.value !== current) element.setAttribute("tabindex", below.value);
+  }
+  #mark(element, stack) {
+    const records = /* @__PURE__ */ new Map([[this.#record, null]]);
+    for (const hold of stack) records.set(hold.record, hold.value);
+    for (const [name, recorded] of records) {
+      if (element.getAttribute(name) === recorded) continue;
+      if (recorded === null) element.removeAttribute(name);
+      else element.setAttribute(name, recorded);
+    }
   }
 };
 
-// src/utils/target_selector.ts
 function targetSelector(identifier, name) {
   return `[data-${identifier}-target~="${name}"]`;
 }
 
-// src/controllers/nested_form_controller.ts
 var REMOVE_PART = "remove";
 var DESTROY_FLAG_PART = "destroyFlag";
 var DESTROYED_VALUES = /* @__PURE__ */ new Set(["1", "true"]);
-var NestedFormController = class extends Controller {
-  /** Selects any root of this controller; the nearest one owns an element. */
+var NestedFormController = class _NestedFormController extends Controller {
+  #numbers = new NumberValueReader();
   get #rootSelector() {
     return `[data-controller~="${this.identifier}"]`;
   }
-  /** Selects per-row remove buttons, in the namespace this controller is registered under. */
   get #removeSelector() {
     return targetSelector(this.identifier, REMOVE_PART);
   }
-  /** Selects the hidden `_destroy` input a persisted row carries. */
   get #destroyFlagSelector() {
     return targetSelector(this.identifier, DESTROY_FLAG_PART);
   }
@@ -258,27 +487,20 @@ var NestedFormController = class extends Controller {
     announce: { type: Boolean, default: true },
     countMessage: { type: String, default: "" }
   };
+  static valueConstraints = {
+    min: NUMBER_BOUNDS.nonNegative,
+    max: NUMBER_BOUNDS.nonNegative
+  };
   static actions = ["add"];
   static events = ["add", "remove", "reconcile"];
-  /** Monotonic source for unique row indices; never a row-state counter. */
   #lastIndex = 0;
   #warnedMissing = false;
   #warnedTemplate = false;
-  /** The state last written to the hooks; reconciliation reports only real moves. */
   #published = null;
-  /** Watches the list for row changes the controller did not perform itself. */
   #observer = null;
-  #reconcile = new MicrotaskCoalescer(() => this.#reconcileNow());
-  /** Restores the authored add-button `disabled` when a lease ends. */
-  #addDisabled = new AttributeLease("disabled");
-  #beforeCache = new BeforeCacheReset(() => this.#rewindForCache());
-  /** Makes the root a programmatic focus destination when no other candidate survives. */
-  #tabindex = new TabindexLoan();
-  /**
-   * Delegated click handler for the per-row remove buttons (dynamic-safe). Only
-   * buttons whose nearest nested-form root is this instance are acted on, so a
-   * nested inner form's buttons never remove an outer row.
-   */
+  #reconcile = new MorphRenderWatcher(() => this.#reconcileNow());
+  #addDisabled = new AttributeLease("disabled", this.identifier);
+  #tabindex = new TabindexLoan("-1", this.identifier);
   #onClick = (event) => {
     const target = event.target;
     const button = target?.closest(this.#removeSelector);
@@ -287,11 +509,11 @@ var NestedFormController = class extends Controller {
     if (row) this.#removeRow(row);
   };
   connect() {
+    this.#tabindex.reclaim(this.element);
     this.#warnedMissing = false;
     this.#warnedTemplate = false;
     this.element.addEventListener("click", this.#onClick);
-    this.#reconcile.activate();
-    this.#beforeCache.activate();
+    this.#reconcile.observe(this.element);
     if (!this.hasListTarget || !this.hasTemplateTarget) this.#warnMissing();
     this.#refresh();
   }
@@ -299,28 +521,19 @@ var NestedFormController = class extends Controller {
     this.element.removeEventListener("click", this.#onClick);
     this.#observer?.disconnect();
     this.#observer = null;
-    this.#reconcile.cancel();
-    this.#beforeCache.deactivate();
-    this.#rewindForCache();
+    this.#reconcile.disconnect();
+    this.#addDisabled.returnAll();
     this.#tabindex.returnAll();
     this.#published = null;
   }
-  /** Follows an arriving or swapped-in list: rebind to the primary, then reconcile. */
   listTargetConnected() {
     this.#rebindObserver();
     this.#reconcile.schedule();
   }
-  /** Follows a departing list the same way — the primary may have changed. */
   listTargetDisconnected() {
     this.#rebindObserver();
     this.#reconcile.schedule();
   }
-  /**
-   * Points the observer at the current primary list. Re-deriving on every list
-   * arrival and departure makes the binding independent of the order Stimulus
-   * reports an overlapping swap in — a staggered swap (successor appended before
-   * the old list leaves) ends observed and reconciled either way.
-   */
   #rebindObserver() {
     this.#observer?.disconnect();
     this.#observer = null;
@@ -333,7 +546,6 @@ var NestedFormController = class extends Controller {
       attributeFilter: ["value"]
     });
   }
-  /** Returns the lease with a departing add button; a new one re-arms on refresh. */
   addTargetDisconnected(target) {
     this.#addDisabled.return(target);
     this.#reconcile.schedule();
@@ -341,21 +553,12 @@ var NestedFormController = class extends Controller {
   addTargetConnected() {
     this.#reconcile.schedule();
   }
-  /** Re-clamps when application code or a Turbo morph changes `min`. */
   minValueChanged() {
     this.#reconcile.schedule();
   }
-  /** Re-clamps when application code or a Turbo morph changes `max`. */
   maxValueChanged() {
     this.#reconcile.schedule();
   }
-  /**
-   * Clones the template row, replaces the index placeholder with a unique value,
-   * appends it, focuses its first tab stop, and announces the new count. No-ops at
-   * `max`, when the required targets are missing (named on the console once per
-   * connection), or when the template does not produce exactly one root element
-   * (also named once; the insertion is rolled back so nothing accumulates).
-   */
   add() {
     if (!this.hasListTarget || !this.hasTemplateTarget) {
       this.#warnMissing();
@@ -386,13 +589,6 @@ var NestedFormController = class extends Controller {
     this.dispatch("add", { detail: { index, element: row } });
     this.#announce();
   }
-  /**
-   * Removes a row: a persisted row (one carrying its own `destroyFlag`) has the
-   * flag set to `1` and is hidden so Rails destroys it on submit; an unsaved row
-   * is dropped from the DOM. Returns focus to a surviving row. No-ops at `min`.
-   *
-   * @stimeoRuntimeOnly `min` decides whether this one removal is allowed.
-   */
   #removeRow(row) {
     const rows = this.#effectiveRows;
     if (this.#destroyed(row)) {
@@ -400,7 +596,7 @@ var NestedFormController = class extends Controller {
       this.#focusAfterRemove(this.#positionAmong(rows, row));
       return;
     }
-    if (rows.length <= this.minValue) return;
+    if (rows.length <= this.#safeMin) return;
     const position = rows.indexOf(row);
     const flag = this.#destroyFlagOf(row);
     const persisted = flag !== null;
@@ -415,21 +611,11 @@ var NestedFormController = class extends Controller {
     this.dispatch("remove", { detail: { element: row, persisted } });
     this.#announce();
   }
-  /**
-   * The index of the first effective row following `row` in document order.
-   * -1 (no following row) feeds the focus slices as a negative index, which
-   * yields the same fully-reversed nearest-first order as `rows.length` would.
-   */
   #positionAmong(rows, row) {
     return rows.findIndex(
       (candidate) => (row.compareDocumentPosition(candidate) & Node.DOCUMENT_POSITION_FOLLOWING) !== 0
     );
   }
-  /**
-   * Moves focus to the first tab stop of the nearest surviving row — following
-   * rows first, then preceding ones — falling back to the add button and finally
-   * to the root via a temporary `tabindex`.
-   */
   #focusAfterRemove(position) {
     const rows = this.#effectiveRows;
     const ordered = [...rows.slice(position), ...rows.slice(0, position).reverse()];
@@ -447,31 +633,20 @@ var NestedFormController = class extends Controller {
     this.#tabindex.lend(this.element);
     this.element.focus();
   }
-  /**
-   * Recomputes the count and min/max hooks from the DOM and records them as published.
-   *
-   * @stimeoRenderRoot
-   */
   #refresh() {
     if (!this.hasListTarget) return;
     const count = this.#effectiveRows.length;
-    const atMin = count <= this.minValue;
-    const atMax = this.maxValue > 0 && count >= this.maxValue;
+    const atMin = count <= this.#safeMin;
+    const atMax = this.#safeMax > 0 && count >= this.#safeMax;
     this.element.setAttribute("data-nested-count", String(count));
     this.#reflect("data-nested-at-max", atMax);
     this.#reflect("data-nested-at-min", atMin);
     if (this.hasAddTarget) {
-      if (this.maxValue > 0) this.#addDisabled.write(this.addTarget, atMax ? "" : null);
+      if (this.#safeMax > 0) this.#addDisabled.write(this.addTarget, atMax ? "" : null);
       else this.#addDisabled.return(this.addTarget);
     }
     this.#published = { count, atMin, atMax };
   }
-  /**
-   * Applies row changes the controller did not perform itself (Turbo Streams,
-   * morphs, runtime Value changes): refreshes the hooks and reports a moved
-   * public state as `reconcile`. The controller's own operations refresh
-   * synchronously first, so their observer echo arrives here as a no-move.
-   */
   #reconcileNow() {
     const previous = this.#published;
     this.#refresh();
@@ -480,25 +655,14 @@ var NestedFormController = class extends Controller {
     const moved = previous.count !== current.count || previous.atMin !== current.atMin || previous.atMax !== current.atMax;
     if (moved) this.dispatch("reconcile", { detail: { ...current } });
   }
-  /**
-   * Bridges the count change to the shared announcer when configured.
-   *
-   * @stimeoRuntimeOnly `announce` and `countMessage` word the one announcement of this change.
-   */
   #announce() {
     if (!this.announceValue || this.countMessageValue === "") return;
     announce(fillTemplate(this.countMessageValue, { count: this.#effectiveRows.length }));
   }
-  /** Sets `attribute` to `"true"` when `on`, else removes it. */
   #reflect(attribute, on) {
     if (on) this.element.setAttribute(attribute, "true");
     else this.element.removeAttribute(attribute);
   }
-  /** Returns the disabled lease so an authored value never leaks into a snapshot. */
-  #rewindForCache() {
-    if (this.hasAddTarget) this.#addDisabled.return(this.addTarget);
-  }
-  /** Names the missing required target(s) once per connection. */
   #warnMissing() {
     if (this.#warnedMissing) return;
     this.#warnedMissing = true;
@@ -510,49 +674,38 @@ var NestedFormController = class extends Controller {
       `Stimeo UI: "${this.identifier}" cannot manage rows because its markup lacks ${missing}.`
     );
   }
-  /** Names a template that does not produce exactly one element, once per connection. */
   #warnBadTemplate(produced) {
     if (this.#warnedTemplate) return;
     this.#warnedTemplate = true;
     const reason = produced === 0 ? "produces no element" : "must produce exactly one root element";
     console.warn(`Stimeo UI: "${this.identifier}" added no row because its template ${reason}.`);
   }
-  /** A strictly-increasing unique index (collision-free even on rapid adds). */
   #nextIndex() {
     const index = Math.max(Date.now(), this.#lastIndex + 1);
     this.#lastIndex = index;
     return index;
   }
   get #atMax() {
-    return this.maxValue > 0 && this.#effectiveRows.length >= this.maxValue;
+    return this.#safeMax > 0 && this.#effectiveRows.length >= this.#safeMax;
   }
-  /**
-   * Direct child rows of the list whose own destroy flag is not set. Callers
-   * reach this only behind a list-presence gate (`add`, `#refresh`, and the
-   * click path through `#rowContaining`).
-   */
   get #effectiveRows() {
     return Array.from(this.listTarget.children).filter(
       (row) => !this.#destroyed(row)
     );
   }
-  /** Whether `row` is flagged for destruction; the flag value is the truth source. */
   #destroyed(row) {
     const flag = this.#destroyFlagOf(row);
     return flag !== null && DESTROYED_VALUES.has(flag.value);
   }
-  /** The row's own destroy flag, skipping flags owned by a nested inner form. */
   #destroyFlagOf(row) {
     for (const flag of row.querySelectorAll(this.#destroyFlagSelector)) {
       if (this.#ownerOf(flag) === this.element) return flag;
     }
     return null;
   }
-  /** The nearest nested-form root that owns `el`. */
   #ownerOf(el) {
     return el.closest(this.#rootSelector);
   }
-  /** The nearest ancestor of `el` that is a direct child of the list, else null. */
   #rowContaining(el) {
     if (!this.hasListTarget) return null;
     let node = el;
@@ -561,8 +714,25 @@ var NestedFormController = class extends Controller {
     }
     return node;
   }
+  get #safeMin() {
+    return this.#numbers.read(
+      this,
+      "min",
+      this.minValue,
+      _NestedFormController.values.min.default,
+      _NestedFormController.valueConstraints.min
+    );
+  }
+  get #safeMax() {
+    return this.#numbers.read(
+      this,
+      "max",
+      this.maxValue,
+      _NestedFormController.values.max.default,
+      _NestedFormController.valueConstraints.max
+    );
+  }
 };
 
 export { NestedFormController };
-//# sourceMappingURL=nested_form_controller.js.map
 //# sourceMappingURL=nested_form_controller.js.map