@mono-lit/helper 0.0.1 → 0.0.3

package/dist/{mono-data-chart-ChU4Ii3p.js → mono-data-chart-ChkelNT7.js}

@@ -1,15 +1,17 @@
-import { n as isPath, t as getFieldValue } from "./field-path-C92eGLg3.js";
-import { r as readAllRows } from "./data-source-read-DL88aH8t.js";
-import { n as andPredicates, r as compileFilterPredicate, t as andFilters } from "./filter-eval-DIfU2EAn.js";
-import { a as resolveDataSourceOptions, d as createApplyGate, f as isRolledUp, h as monoArraySource, i as normalizeSortList, n as extraLoadOptions, o as toList, p as loadApply, r as mergeDataSourceOptions, u as summaryFilterOf } from "./data-source-options-CQSTZHPE.js";
-import { t as arrayToODataString } from "./filter-odata-C1vSGZaY.js";
-import { t as createNotifier } from "./notifier-CE4yxMUQ.js";
+import { n as isPath, t as getFieldValue } from "./field-path-BabiMrrA.js";
+import { r as readAllRows } from "./data-source-read-FxqzQkk2.js";
+import { n as andPredicates, r as compileFilterPredicate, t as andFilters } from "./filter-eval-ChDSaAUH.js";
+import { n as describeError, o as isCanceled } from "./normalize-error-D93O4q8g.js";
+import { a as resolveDataSourceOptions, d as createApplyGate, f as isRolledUp, h as monoArraySource, i as normalizeSortList, n as extraLoadOptions, o as toList, p as loadApply, r as mergeDataSourceOptions, u as summaryFilterOf } from "./data-source-options-QjuI73g8.js";
+import { t as arrayToODataString } from "./filter-odata-Bi27XUpQ.js";
+import { t as createNotifier } from "./notifier-BIrqUzRh.js";
+
 //#region src/components/chart/chart-odata.ts
 /**
 * `MonoChartAgg` → the OData aggregate keyword. Every value the chart supports has
 * an exact equivalent, so the server path never has to fall back to draining.
 */
-var ODATA_AGG = {
+const ODATA_AGG = {
 	sum: "sum",
 	avg: "average",
 	min: "min",
@@ -17,7 +19,7 @@ var ODATA_AGG = {
 	count: "$count"
 };
 /** A dotted path (`Job.Budget`) addresses a nav property as `Job/Budget` in OData. */
-var toODataPath = (field) => field.replaceAll(".", "/");
+const toODataPath = (field) => field.replaceAll(".", "/");
 /**
 * Build `groupby((<groupBy>),aggregate(<field> with <agg> as <field>, …))`.
 *
@@ -86,6 +88,7 @@ function buildChartOdataRequest(input) {
 		aggregated: params.$apply !== void 0
 	};
 }
+
 //#endregion
 //#region src/components/chart/mono-data-chart.ts
 /**
@@ -96,7 +99,7 @@ function buildChartOdataRequest(input) {
 * time, so a colour preset, a flavour or dark mode recolours charts too. Past
 * five series the roles follow, so a sixth dataset is still its own colour.
 */
-var DEFAULT_COLORS = [
+const DEFAULT_COLORS = [
 	"chart-1",
 	"chart-2",
 	"chart-3",
@@ -111,7 +114,7 @@ var DEFAULT_COLORS = [
 * (reading `data()` in Node), or before an element mounts. ONE's light tokens,
 * so the shape is right even without a DOM.
 */
-var NAME_FALLBACK = {
+const NAME_FALLBACK = {
 	"chart-1": "oklch(0.859 0.069 267.7)",
 	"chart-2": "oklch(0.735 0.12 268.04)",
 	"chart-3": "oklch(0.61 0.12 267.95)",
@@ -383,7 +386,8 @@ function monoChart(source = null, opts = {}) {
 	}
 	const onChanged = () => {
 		if (drainDepth > 0) return;
-		sync();
+		if (opts.loadAll === false) succeed();
+		sync().catch(() => {});
 	};
 	/**
 	* Publish the loading state, notifying only on a real change.
@@ -406,11 +410,26 @@ function monoChart(source = null, opts = {}) {
 		syncLoading();
 		notify();
 	};
+	/** Record a load failure for the element to paint. Cancelled loads are not failures. */
+	function fail(err) {
+		if (isCanceled(err)) return;
+		ctrl.error = describeError(err);
+		notify();
+	}
+	/** A load answered: the chart has data and any earlier failure is over. */
+	function succeed() {
+		ctrl.hasLoaded = true;
+		ctrl.error = null;
+	}
+	const onLoadError = (err) => {
+		syncLoading();
+		fail(err ?? /* @__PURE__ */ new Error("Request failed"));
+	};
 	function detach() {
 		if (!bound) return;
 		bound.off("changed", onChanged);
 		bound.off("loadingChanged", onLoadingChanged);
-		bound.off("loadError", onLoadingChanged);
+		bound.off("loadError", onLoadError);
 		bound = null;
 	}
 	let dsOptionsSource = opts.dataSourceOptions;
@@ -508,8 +527,10 @@ function monoChart(source = null, opts = {}) {
 		preAggregated = false;
 		resolveBase();
 		if (!bound) rows = [];
-		else if (opts.loadAll === false) rows = [...bound.items?.() ?? []];
-		else {
+		else if (opts.loadAll === false) {
+			rows = [...bound.items?.() ?? []];
+			if (rows.length) succeed();
+		} else {
 			drainDepth++;
 			syncLoading();
 			try {
@@ -531,6 +552,10 @@ function monoChart(source = null, opts = {}) {
 					});
 					rows = isArray && typeof filter === "function" ? drained.filter(filter) : drained;
 				}
+				succeed();
+			} catch (err) {
+				fail(err);
+				throw err;
 			} finally {
 				drainDepth--;
 				syncLoading();
@@ -545,6 +570,7 @@ function monoChart(source = null, opts = {}) {
 			detach();
 			raw = next;
 			rows = [];
+			succeed();
 			ctrl.items = rows;
 			ctrl.dataSource = null;
 			project();
@@ -557,14 +583,16 @@ function monoChart(source = null, opts = {}) {
 		detach();
 		bound = src;
 		ctrl.dataSource = src;
+		ctrl.hasLoaded = false;
+		ctrl.error = null;
 		if (!src) {
-			sync();
+			sync().catch(() => {});
 			return;
 		}
 		src.on("changed", onChanged);
 		src.on("loadingChanged", onLoadingChanged);
-		src.on("loadError", onLoadingChanged);
-		sync();
+		src.on("loadError", onLoadError);
+		sync().catch(() => {});
 	}
 	/**
 	* Load through `monoOdataFetch`.
@@ -609,6 +637,10 @@ function monoChart(source = null, opts = {}) {
 			if (res?.error) throw new Error(res.error.message ?? "odata request failed");
 			rows = res?.data ?? [];
 			preAggregated = request.aggregated;
+			succeed();
+		} catch (err) {
+			fail(err);
+			throw err;
 		} finally {
 			setLoading(false);
 		}
@@ -627,7 +659,13 @@ function monoChart(source = null, opts = {}) {
 			if (opts.loadAll === false && bound?.load) {
 				resolveBase();
 				applyBaseToSource(bound);
-				await bound.load();
+				try {
+					await bound.load();
+					succeed();
+				} catch (err) {
+					fail(err);
+					throw err;
+				}
 			}
 			await sync();
 		} finally {
@@ -680,6 +718,8 @@ function monoChart(source = null, opts = {}) {
 	const ctrl = {
 		items: rows,
 		loading: false,
+		error: null,
+		hasLoaded: false,
 		type: opts.type ?? "bar",
 		dataSource: null,
 		get instance() {
@@ -705,6 +745,11 @@ function monoChart(source = null, opts = {}) {
 		bind,
 		reload,
 		refresh: reload,
+		async retry() {
+			ctrl.error = null;
+			notify();
+			await reload().catch(() => {});
+		},
 		setDataSourceOptions,
 		setOdataOptions,
 		resolvedDataSourceOptions,
@@ -729,10 +774,9 @@ function monoChart(source = null, opts = {}) {
 	};
 	if (source) bind(source);
 	else project();
-	if (!source && opts.odata?.url) loadOdata().catch((err) => {
-		console.error("[mono-chart] odata load failed:", err);
-	});
+	if (!source && opts.odata?.url) loadOdata().catch(() => {});
 	return ctrl;
 }
+
 //#endregion
-export { composeApply as i, buildChartApply as n, buildChartOdataRequest as r, monoChart as t };
+export { composeApply as i, buildChartApply as n, buildChartOdataRequest as r, monoChart as t };

package/dist/{mono-data-dropdown-BAUl58kW.js → mono-data-dropdown-DxpUUk2y.js}

@@ -1,5 +1,6 @@
-import { t as monoDataGrid } from "./mono-data-grid-Db80FzdZ.js";
-import { t as createNotifier } from "./notifier-CE4yxMUQ.js";
+import { t as monoDataGrid } from "./mono-data-grid-8jmeIXUU.js";
+import { t as createNotifier } from "./notifier-BIrqUzRh.js";
+
 //#region src/components/dropdown-table/mono-data-dropdown.ts
 function monoDataDropdown(source = null, opts = {}) {
 	const keyExpr = opts.keyExpr ?? "Id";
@@ -353,5 +354,6 @@ function monoDataDropdown(source = null, opts = {}) {
 	};
 	return ctrl;
 }
+
 //#endregion
-export { monoDataDropdown as t };
+export { monoDataDropdown as t };

package/dist/{mono-data-grid-Db80FzdZ.js → mono-data-grid-8jmeIXUU.js}

@@ -1,235 +1,13 @@
-import { a as parseFieldPath, c as toODataClause, l as toODataSelector, n as isPath, o as projectFields, r as mergePatch, s as setFieldValue, t as getFieldValue } from "./field-path-C92eGLg3.js";
-import { a as buildGroups, c as isGroupNode, i as searchFilterOf, o as collectGroupPaths, r as readAllRows, s as flattenLeaves } from "./data-source-read-DL88aH8t.js";
-import { i as joinFilters, n as andPredicates, r as compileFilterPredicate, t as andFilters } from "./filter-eval-DIfU2EAn.js";
-import { S as searchEntryFor, _ as hasWildcardSearch, b as plainSearchColumns, g as hasCustomSearch, m as customRemoteClause, o as mergeSearchFields, p as customPredicate, t as DEFAULT_SEARCH_FIELDS, x as resolveSearchEntries } from "./data-search-Bwo5bg8K.js";
-import { a as resolveDataSourceOptions, c as composeSummaryApply, d as createApplyGate, f as isRolledUp, h as monoArraySource, i as normalizeSortList, l as summaryAliases, m as storeRows, n as extraLoadOptions, o as toList, p as loadApply, s as buildSummaryApply, t as dataSourceOptionsKey, u as summaryFilterOf } from "./data-source-options-CQSTZHPE.js";
-import { n as unwrapReactive } from "./reactive-D3EERqgO.js";
-import { t as arrayToODataString } from "./filter-odata-C1vSGZaY.js";
-import { t as createNotifier } from "./notifier-CE4yxMUQ.js";
-import { t as monoFilterBuilder } from "./filter-builder-Bke3qOgJ.js";
-//#region src/utils/normalize-error.ts
-/** Fields worth reading off an object that is not an `Error`. */
-var MESSAGE_KEYS = [
-	"message",
-	"statusText",
-	"errorMessage",
-	"error",
-	"detail"
-];
-var text = (value) => typeof value === "string" && value.trim() ? value.trim() : "";
-var isObject = (value) => value !== null && typeof value === "object";
-/**
-* A readable message for `raw`, or `fallback` when nothing legible can be found.
-*
-* Never throws, and never returns an empty string — a caller rendering this into
-* an error bar has to have something to render, and a blank bar is worse than a
-* generic one.
-*/
-function normalizeError(raw, fallback = "Request failed") {
-	if (raw == null) return fallback;
-	if (raw instanceof Error) {
-		const own = text(raw.message);
-		if (own) return own;
-	} else {
-		if (typeof raw === "string") return text(raw) || fallback;
-		if (typeof raw === "number" || typeof raw === "boolean") return String(raw);
-	}
-	if (isObject(raw)) {
-		const obj = raw;
-		for (const key of MESSAGE_KEYS) {
-			const hit = text(obj[key]);
-			if (hit) return hit;
-		}
-		const response = obj.response;
-		if (isObject(response)) {
-			const nested = text(response.statusText) || text(response.message);
-			if (nested) return nested;
-		}
-		const status = errorStatus(obj);
-		if (status !== void 0 && status > 0) return `HTTP ${status}`;
-		if (raw instanceof Error) return text(raw.name) || fallback;
-		return fallback;
-	}
-	return text(String(raw)) || fallback;
-}
-/**
-* What the bar says for a status. English; an app puts its own language in
-* ONCE with `setErrorMessages()` rather than on every table.
-*/
-var DEFAULT_ERROR_MESSAGES = Object.freeze({
-	network: "Could not reach the server. Check your connection and try again.",
-	400: "The server rejected the request.",
-	401: "Your session has expired. Please sign in again.",
-	403: "You do not have permission to view this data.",
-	404: "The requested data could not be found.",
-	408: "The server took too long to respond. Please try again.",
-	409: "The request conflicts with the current state of the data.",
-	422: "The server could not process the request.",
-	429: "Too many requests. Please wait a moment and try again.",
-	500: "Something went wrong on the server. Please try again later.",
-	502: "The server is temporarily unavailable. Please try again later.",
-	503: "The server is temporarily unavailable. Please try again later.",
-	504: "The server took too long to respond. Please try again later.",
-	default: "Request failed"
-});
-var globalMessages = null;
-/**
-* App-wide wording, merged over the defaults; `null` restores them. A grid's own
-* `errorMessages` merges over this in turn, so a single table can still differ.
-*/
-function setErrorMessages(messages) {
-	globalMessages = messages ? { ...messages } : null;
-}
-/** Defaults ← app-wide ← `local`, same shape as the filter builder's `resolveTexts`. */
-function resolveErrorMessages(local) {
-	const merged = { ...DEFAULT_ERROR_MESSAGES };
-	for (const layer of [globalMessages, local]) {
-		if (!layer) continue;
-		for (const key of Object.keys(layer)) {
-			const value = layer[key];
-			if (typeof value === "string" && value.trim()) merged[key] = value;
-		}
-	}
-	return merged;
-}
-/**
-* The standard reason phrases. Not shown to anyone — they are what `describeError`
-* uses to recognise that an error's text is ONLY the phrase for its status
-* (`Error('Forbidden')` + `httpStatus: 403`), which adds nothing to the preset and
-* so is not a detail worth appending.
-*/
-var REASON_PHRASES = {
-	400: "Bad Request",
-	401: "Unauthorized",
-	402: "Payment Required",
-	403: "Forbidden",
-	404: "Not Found",
-	405: "Method Not Allowed",
-	408: "Request Timeout",
-	409: "Conflict",
-	410: "Gone",
-	412: "Precondition Failed",
-	413: "Payload Too Large",
-	415: "Unsupported Media Type",
-	422: "Unprocessable Entity",
-	429: "Too Many Requests",
-	500: "Internal Server Error",
-	501: "Not Implemented",
-	502: "Bad Gateway",
-	503: "Service Unavailable",
-	504: "Gateway Timeout"
-};
-/** Texts a transport mints for "no answer at all" — a status of 0 in words. */
-var NETWORK_TEXTS = [
-	"failed to fetch",
-	"networkerror when attempting to fetch resource",
-	"load failed",
-	"network error",
-	"unspecified network error",
-	"network connection timeout",
-	"network request failed"
-];
-var toStatus = (value) => {
-	if (value === null || value === void 0 || value === "") return void 0;
-	const n = Number(value);
-	return Number.isInteger(n) && n >= 0 && n < 1e3 ? n : void 0;
-};
-/**
-* The HTTP status an error carries, `0` for a network failure, `undefined` when
-* it has none.
-*
-* Reads the fields the common transports use — devextreme's `httpStatus`, a
-* fetch `Response` / axios error's `status` and `response.status`, ofetch's
-* `statusCode`, an OData error body's numeric `code` — on an `Error` instance as
-* readily as on a plain object, since devextreme `extend()`s them onto an
-* `Error`.
-*/
-function errorStatus(raw) {
-	if (!isObject(raw)) return void 0;
-	const response = isObject(raw.response) ? raw.response : void 0;
-	const direct = toStatus(raw.httpStatus) ?? toStatus(raw.status) ?? toStatus(raw.statusCode) ?? toStatus(response?.status) ?? toStatus(response?.statusCode);
-	if (direct !== void 0) return direct;
-	const code = toStatus((isObject(raw.errorDetails) ? raw.errorDetails : void 0)?.code);
-	if (code !== void 0 && code >= 400) return code;
-	const own = (raw instanceof Error ? text(raw.message) : text(raw.message)).toLowerCase();
-	if (own && NETWORK_TEXTS.some((t) => own.includes(t))) return 0;
-}
-/**
-* The text a SERVER wrote about the failure, if the error carries one — an OData
-* error body's message, a JSON body's `message` / `error`, an axios
-* `response.data`. `''` when there is nothing beyond the transport's own words.
-*/
-function serverDetail(raw) {
-	if (!isObject(raw)) return "";
-	const details = raw.errorDetails;
-	if (isObject(details)) {
-		const m = details.message;
-		const hit = text(m) || (isObject(m) ? text(m.value) : "");
-		if (hit) return hit;
-	}
-	const response = isObject(raw.response) ? raw.response : void 0;
-	const data = response?.data ?? response?._data ?? raw.data;
-	if (isObject(data)) {
-		const err = data.error;
-		const hit = text(data.message) || text(data.detail) || text(data.title) || text(err) || (isObject(err) ? text(err.message) : "");
-		if (hit) return hit;
-	} else if (text(data)) return text(data);
-	if (!(raw instanceof Error)) {
-		const err = raw.error;
-		const hit = text(raw.errorMessage) || text(raw.detail) || (isObject(err) ? text(err.message) : "");
-		if (hit) return hit;
-	}
-	return "";
-}
-/** Is `value` nothing more than the reason phrase / a generic label for `status`? */
-function isBoilerplate(value, status) {
-	const v = value.toLowerCase();
-	if (!v || v === "error" || v === "unknown error" || v === "request failed") return true;
-	if (/^http\s*\d{3}$/.test(v)) return true;
-	if (NETWORK_TEXTS.some((t) => v.includes(t))) return true;
-	if (status !== void 0) {
-		const phrase = REASON_PHRASES[status]?.toLowerCase();
-		if (phrase && (v === phrase || v === `${status} ${phrase}` || v === `${status}`)) return true;
-		if (v === String(status)) return true;
-	}
-	return Object.values(REASON_PHRASES).some((p) => p.toLowerCase() === v);
-}
-/**
-* The line a person should read for `raw`.
-*
-* Order: a preset for the status (`'network'` for 0) → the error's own text →
-* `messages.default`. When a preset is the headline, whatever the server said
-* on top of the status becomes `detail`; without a preset, the raw text IS the
-* headline and only a distinct server body is a detail. Never throws, never
-* returns an empty `message`.
-*/
-function describeError(raw, messages) {
-	const texts = resolveErrorMessages(messages);
-	const status = errorStatus(raw);
-	const fallback = texts.default || DEFAULT_ERROR_MESSAGES.default;
-	const rawText = normalizeError(raw, fallback);
-	const preset = status === void 0 ? void 0 : status === 0 ? texts.network : texts[status];
-	if (preset) {
-		const candidate = serverDetail(raw) || (rawText === fallback ? "" : rawText);
-		const detail = candidate && !isBoilerplate(candidate, status) && candidate !== preset ? candidate : "";
-		return detail ? {
-			message: preset,
-			status,
-			detail
-		} : {
-			message: preset,
-			status
-		};
-	}
-	const message = rawText || fallback;
-	const fromServer = serverDetail(raw);
-	const detail = fromServer && fromServer !== message && !isBoilerplate(fromServer, status) ? fromServer : "";
-	const out = { message };
-	if (status !== void 0) out.status = status;
-	if (detail) out.detail = detail;
-	return out;
-}
-//#endregion
+import { a as parseFieldPath, c as toODataClause, l as toODataSelector, n as isPath, o as projectFields, r as mergePatch, s as setFieldValue, t as getFieldValue } from "./field-path-BabiMrrA.js";
+import { a as buildGroups, c as isGroupNode, i as searchFilterOf, o as collectGroupPaths, r as readAllRows, s as flattenLeaves } from "./data-source-read-FxqzQkk2.js";
+import { i as joinFilters, n as andPredicates, r as compileFilterPredicate, t as andFilters } from "./filter-eval-ChDSaAUH.js";
+import { A as resolveSearchEntries, C as customRemoteClause, E as hasWildcardSearch, S as customPredicate, T as hasCustomSearch, h as mergeSearchFields, j as searchEntryFor, k as plainSearchColumns, n as describeError, o as isCanceled, u as DEFAULT_SEARCH_FIELDS } from "./normalize-error-D93O4q8g.js";
+import { a as resolveDataSourceOptions, c as composeSummaryApply, d as createApplyGate, f as isRolledUp, h as monoArraySource, i as normalizeSortList, l as summaryAliases, m as storeRows, n as extraLoadOptions, o as toList, p as loadApply, s as buildSummaryApply, t as dataSourceOptionsKey, u as summaryFilterOf } from "./data-source-options-QjuI73g8.js";
+import { n as unwrapReactive } from "./reactive-Dtd-esHO.js";
+import { t as arrayToODataString } from "./filter-odata-Bi27XUpQ.js";
+import { t as createNotifier } from "./notifier-BIrqUzRh.js";
+import { t as monoFilterBuilder } from "./filter-builder-C0T3pMx2.js";
+
 //#region src/components/table/store-group-source.ts
 /**
 * Build a {@link MonoServerGroupSource} that drives **server-side** group paging
@@ -267,7 +45,7 @@ function storeGroupSource(source, config) {
 	return {
 		async loadGroups(ctx) {
 			const filter = joinFilters([ctx.filter, searchFilter(ctx.search)], "and");
-			return rows(await store.load({
+			const res = await store.load({
 				group: [{
 					selector: groupField,
 					isExpanded: false
@@ -277,7 +55,8 @@ function storeGroupSource(source, config) {
 				...filter ? { filter } : {},
 				sort: ctx.sort ?? [{ selector: groupField }],
 				...ctx.loadOptions ?? {}
-			})).map((g) => ({
+			});
+			return rows(res).map((g) => ({
 				key: g.key,
 				count: Number(g.count ?? (Array.isArray(g.items) ? g.items.length : 0)) || 0,
 				aggregates: summaryDesc && Array.isArray(g.summary) ? Object.fromEntries(summaryDesc.map((s, i) => [s.selector, Number(g.summary[i]) || 0])) : void 0
@@ -290,7 +69,7 @@ function storeGroupSource(source, config) {
 				key
 			];
 			const rowSelect = select ?? ctx.select;
-			return rows(await store.load({
+			const res = await store.load({
 				filter: joinFilters([
 					groupFilter,
 					ctx.filter,
@@ -302,13 +81,15 @@ function storeGroupSource(source, config) {
 				take: ctx.take,
 				requireTotalCount: false,
 				...ctx.loadOptions ?? {}
-			}));
+			});
+			return rows(res);
 		}
 	};
 }
+
 //#endregion
 //#region src/components/table/date-filter-tree.ts
-var DATE_LEVELS = [
+const DATE_LEVELS = [
 	"year",
 	"month",
 	"day",
@@ -323,7 +104,7 @@ function isDateRange(v) {
 	const ok = (x) => x instanceof Date || typeof x === "string";
 	return ok(o.from) && ok(o.to);
 }
-var BLANK_KEY = "blank";
+const BLANK_KEY = "blank";
 /**
 * A column value as a Date, or `null` for a blank / unparsable one. Accepts a
 * `Date`, a number (epoch ms), an ISO string, and a bare `YYYY-MM-DD` — the
@@ -457,7 +238,7 @@ function buildDateTree(values, options = {}) {
 	});
 	return out;
 }
-var childMaps = /* @__PURE__ */ new WeakMap();
+const childMaps = /* @__PURE__ */ new WeakMap();
 function childMap(n) {
 	let m = childMaps.get(n);
 	if (!m) {
@@ -595,12 +376,9 @@ function inRange(d, r) {
 	const t = d.getTime();
 	return t >= r.from.getTime() && t < r.to.getTime();
 }
+
 //#endregion
 //#region src/components/table/mono-data-grid.ts
-function isCanceled(err) {
-	if (!err) return false;
-	return err === "canceled" || err.name === "canceled" || err.message === "canceled" || err.__id === "canceled";
-}
 function monoDataGrid(ds = null, opts = {}) {
 	let bound = null;
 	let searchFieldsOverride;
@@ -1345,8 +1123,10 @@ function monoDataGrid(ds = null, opts = {}) {
 			const has = checkedRows.has(key);
 			const next = checked === void 0 ? !has : checked;
 			const { max, min } = checkLimits();
-			if (!(next && !has && max !== void 0 && checkedRows.size >= max || !next && has && min !== void 0 && checkedRows.size <= min)) if (next) checkedRows.set(key, item);
-			else checkedRows.delete(key);
+			if (!(next && !has && max !== void 0 && checkedRows.size >= max || !next && has && min !== void 0 && checkedRows.size <= min)) {
+				if (next) checkedRows.set(key, item);
+				else checkedRows.delete(key);
+			}
 			notify();
 		},
 		selectPage() {
@@ -2496,7 +2276,8 @@ function monoDataGrid(ds = null, opts = {}) {
 	}
 	/** Build the full group tree from every matched row the source holds. */
 	function fullGroups() {
-		return buildGroups(flattenLeaves(bound ? [...bound.items?.() ?? []] : []), groupFields, collapsed);
+		const raw = bound ? [...bound.items?.() ?? []] : [];
+		return buildGroups(flattenLeaves(raw), groupFields, collapsed);
 	}
 	/** Index every group node by path (full, unsliced) for `groupPageInfo`. */
 	function indexPaths(nodes) {
@@ -3433,9 +3214,11 @@ function monoDataGrid(ds = null, opts = {}) {
 				if (isDateRange(v)) {
 					const d = parseDateValue(rv);
 					if (!d) return false;
+					const from = v.from instanceof Date ? v.from : new Date(v.from);
+					const to = v.to instanceof Date ? v.to : new Date(v.to);
 					return inRange(d, {
-						from: v.from instanceof Date ? v.from : new Date(v.from),
-						to: v.to instanceof Date ? v.to : new Date(v.to)
+						from,
+						to
 					});
 				}
 				return String(rv) === String(v);
@@ -3843,7 +3626,8 @@ function monoDataGrid(ds = null, opts = {}) {
 		if (!isPath(field)) loadOpts.select = [field];
 		for (const [key, value] of Object.entries(scoped ?? {})) if (value == null) delete loadOpts[key];
 		else loadOpts[key] = value;
-		return distinctFromRows(storeRows(await store.load(loadOpts)), field);
+		const res = await store.load(loadOpts);
+		return distinctFromRows(storeRows(res), field);
 	}
 	async function setData(next) {
 		const s = bound;
@@ -3913,14 +3697,16 @@ function monoDataGrid(ds = null, opts = {}) {
 			const current = sortOf(field);
 			const next = order !== void 0 ? order : current === "asc" ? "desc" : current === "desc" ? null : "asc";
 			if (!next) ctrl.sorts = multi ? ctrl.sorts.filter((s) => s.field !== field) : [];
-			else if (multi) ctrl.sorts = ctrl.sorts.findIndex((s) => s.field === field) >= 0 ? ctrl.sorts.map((s) => s.field === field ? {
-				field,
-				order: next
-			} : s) : [...ctrl.sorts, {
-				field,
-				order: next
-			}];
-			else ctrl.sorts = [{
+			else if (multi) {
+				const at = ctrl.sorts.findIndex((s) => s.field === field);
+				ctrl.sorts = at >= 0 ? ctrl.sorts.map((s) => s.field === field ? {
+					field,
+					order: next
+				} : s) : [...ctrl.sorts, {
+					field,
+					order: next
+				}];
+			} else ctrl.sorts = [{
 				field,
 				order: next
 			}];
@@ -3984,7 +3770,8 @@ function monoDataGrid(ds = null, opts = {}) {
 			sync();
 			return;
 		}
-		const tree = buildGroups(flattenLeaves(bound ? [...bound.items?.() ?? []] : []), groupFields);
+		const raw = bound ? [...bound.items?.() ?? []] : [];
+		const tree = buildGroups(flattenLeaves(raw), groupFields);
 		for (const path of collectGroupPaths(tree)) collapsed.add(path);
 		sync();
 	}
@@ -4021,11 +3808,13 @@ function monoDataGrid(ds = null, opts = {}) {
 		sync();
 	}
 	function clearGroupPaging(path) {
-		if (groupRowPaging.delete(path)) if (serverMode()) {
-			groupRowsKey.delete(path);
-			sync();
-			fetchVisibleRows();
-		} else sync();
+		if (groupRowPaging.delete(path)) {
+			if (serverMode()) {
+				groupRowsKey.delete(path);
+				sync();
+				fetchVisibleRows();
+			} else sync();
+		}
 	}
 	function groupPageInfo(path) {
 		if (pageSizeAll) {
@@ -4099,7 +3888,8 @@ function monoDataGrid(ds = null, opts = {}) {
 	*/
 	async function buildGroupTree(rows) {
 		if (!groupFields.length) return [];
-		return buildGroups(rows ?? await getData(), groupFields);
+		const source = rows ?? await getData();
+		return buildGroups(source, groupFields);
 	}
 	async function importSheet(options) {
 		const { importTable } = await import("./import.js");
@@ -4138,5 +3928,6 @@ function monoDataGrid(ds = null, opts = {}) {
 	if (ds) bind(ds);
 	return ctrl;
 }
+
 //#endregion
-export { seedFromRanges as a, DEFAULT_ERROR_MESSAGES as c, normalizeError as d, resolveErrorMessages as f, minimize as i, describeError as l, buildDateTree as n, setAllChecked as o, setErrorMessages as p, checkState as r, setChecked as s, monoDataGrid as t, errorStatus as u };
+export { seedFromRanges as a, minimize as i, buildDateTree as n, setAllChecked as o, checkState as r, setChecked as s, monoDataGrid as t };