@flytedan/flytebot-design-system 0.29.0 → 0.31.0

package/dist/index.js

@@ -1517,7 +1517,7 @@ var BANNER_TONE_STYLE = {
 };
 function Banner({ tone = "info", title, detail, actions, role }) {
   const t = BANNER_TONE_STYLE[tone];
-  const liveRole = role || (tone === "danger" || tone === "warn" ? "alert" : "status");
+  const liveRole = role || (tone === "danger" ? "alert" : "status");
   return /* @__PURE__ */ jsxs19(
     "div",
     {
@@ -2769,8 +2769,10 @@ function NumberInput({
   max = Infinity,
   step = 1,
   bigStep,
+  integer = false,
   value,
   onChange,
+  onValidityChange,
   format = true,
   className = "",
   style,
@@ -2786,17 +2788,34 @@ function NumberInput({
     const n = Number(s);
     return isNaN(n) ? null : n;
   };
+  const validationMessage = (n) => {
+    if (integer && !Number.isInteger(n)) return "Enter a whole number";
+    if (n < min) return "Must be \u2265 " + min;
+    if (n > max) return "Must be \u2264 " + max;
+    return null;
+  };
   const [text, setText] = React21.useState(value == null || value === "" ? "" : String(value));
   const [editing, setEditing] = React21.useState(false);
+  const [invalidMessage, setInvalidMessage] = React21.useState(null);
   const timer = React21.useRef(null);
   React21.useEffect(() => {
-    if (!editing) setText(value == null || value === "" ? "" : String(value));
+    if (!editing) {
+      setText(value == null || value === "" ? "" : String(value));
+      setInvalidMessage(null);
+    }
   }, [value, editing]);
+  React21.useEffect(() => {
+    onValidityChange && onValidityChange(invalidMessage == null);
+  }, [invalidMessage]);
   const clamp = (n) => Math.min(max, Math.max(min, n));
   const fire = (n) => onChange && onChange({ target: { value: n } });
   const bump = (dir, mult = 1) => {
-    const cur = num(text) ?? num(value) ?? (min === -Infinity ? 0 : min);
-    const next = clamp(cur + dir * (bigStep && mult > 1 ? bigStep : step) * (bigStep ? 1 : mult));
+    const cur = (invalidMessage ? null : num(text)) ?? num(value) ?? (min === -Infinity ? 0 : min);
+    const amount = (bigStep && mult > 1 ? bigStep : step) * (bigStep ? 1 : mult);
+    let next = cur + dir * amount;
+    if (integer) next = Math.round(next);
+    next = clamp(next);
+    setInvalidMessage(null);
     setText(String(next));
     fire(next);
   };
@@ -2820,7 +2839,8 @@ function NumberInput({
     const n = num(text);
     return n == null ? "" : format ? n.toLocaleString() : String(n);
   })();
-  const box = ["fd-input", "fd-input-num", error ? "fd-input-error" : "", disabled ? "fd-input-disabled" : ""].filter(Boolean).join(" ");
+  const shownError = error || invalidMessage || void 0;
+  const box = ["fd-input", "fd-input-num", shownError ? "fd-input-error" : "", disabled ? "fd-input-disabled" : ""].filter(Boolean).join(" ");
   const fieldName = label || ariaLabel;
   const stepperName = (verb) => fieldName ? verb + " " + fieldName : verb;
   return /* @__PURE__ */ jsxs36("div", { className: ["fd-field", className].filter(Boolean).join(" "), style, children: [
@@ -2836,30 +2856,29 @@ function NumberInput({
           disabled,
           placeholder,
           value: shown,
-          "aria-invalid": error ? "true" : void 0,
-          "aria-describedby": error ? errorId : void 0,
+          "aria-invalid": shownError ? "true" : void 0,
+          "aria-describedby": shownError ? errorId : void 0,
           onFocus: () => setEditing(true),
           onBlur: () => {
             setEditing(false);
-            if (text.trim() === "") {
-              fire(void 0);
-              return;
-            }
-            const n = num(text);
-            if (n != null) {
-              const c = clamp(n);
-              setText(String(c));
-              fire(c);
-            }
+            if (text.trim() === "") fire(void 0);
           },
           onChange: (e) => {
-            setText(e.target.value);
-            if (e.target.value.trim() === "") {
+            const raw = e.target.value;
+            setText(raw);
+            if (raw.trim() === "") {
+              setInvalidMessage(null);
               fire(void 0);
               return;
             }
-            const n = num(e.target.value);
-            if (n != null) fire(n);
+            const n = num(raw);
+            if (n == null) {
+              setInvalidMessage("Enter a number");
+              return;
+            }
+            const message = validationMessage(n);
+            setInvalidMessage(message);
+            if (!message) fire(n);
           },
           onKeyDown: (e) => {
             if (e.key === "ArrowUp") {
@@ -2902,9 +2921,9 @@ function NumberInput({
         )
       ] })
     ] }),
-    error ? /* @__PURE__ */ jsxs36("span", { id: errorId, className: "fd-field-error", children: [
+    shownError ? /* @__PURE__ */ jsxs36("span", { id: errorId, className: "fd-field-error", children: [
       /* @__PURE__ */ jsx42("i", { className: "ph ph-warning-circle", "aria-hidden": "true" }),
-      error
+      shownError
     ] }) : help ? /* @__PURE__ */ jsx42("span", { className: "fd-field-help", children: help }) : null
   ] });
 }
@@ -3182,6 +3201,91 @@ function isFieldSet(field, value) {
     }
   }
 }
+function validateFilterValue(field, value) {
+  switch (field.type) {
+    case "text":
+    case "select":
+      return { ok: true };
+    case "multiSelect": {
+      const v = value;
+      if (!v || !Array.isArray(v.values) || v.values.length === 0) {
+        return { ok: false, reason: "Select at least one value" };
+      }
+      if (!operatorsFor(field).includes(v.operator)) {
+        return { ok: false, reason: "That operator isn't allowed for " + field.label };
+      }
+      return { ok: true };
+    }
+    case "range": {
+      const v = value;
+      if (!v || v.min == null && v.max == null) {
+        return { ok: false, reason: "Enter at least one bound" };
+      }
+      const lo = field.min ?? -Infinity;
+      const hi = field.max ?? Infinity;
+      for (const n of [v.min, v.max]) {
+        if (n == null) continue;
+        if (field.integer && !Number.isInteger(n)) return { ok: false, reason: "Enter a whole number" };
+        if (n < lo) return { ok: false, reason: "Must be \u2265 " + lo };
+        if (n > hi) return { ok: false, reason: "Must be \u2264 " + hi };
+      }
+      if (v.min != null && v.max != null && v.min > v.max) {
+        return { ok: false, reason: "Must be \u2265 " + v.min };
+      }
+      return { ok: true };
+    }
+    case "date": {
+      const v = value;
+      if (!v) return { ok: false, reason: "Pick a date value" };
+      if (v.kind === "withinDays") {
+        const maxDays = field.maxDays ?? 3650;
+        if (v.days == null) return { ok: false, reason: "Enter a number of days" };
+        if (!Number.isInteger(v.days)) return { ok: false, reason: "Enter a whole number" };
+        if (v.days < 1) return { ok: false, reason: "Must be \u2265 1" };
+        if (v.days > maxDays) return { ok: false, reason: "Must be \u2264 " + maxDays };
+        return { ok: true };
+      }
+      if (!v.from || !v.to) return { ok: false, reason: "Pick both dates" };
+      if (v.from > v.to) return { ok: false, reason: "The start date must be on or before the end date" };
+      return { ok: true };
+    }
+  }
+}
+function isEmptyCandidate(field, candidate) {
+  switch (field.type) {
+    case "text":
+    case "select":
+      return false;
+    case "multiSelect": {
+      const v = candidate;
+      return !v || !Array.isArray(v.values) || v.values.length === 0;
+    }
+    case "range": {
+      const v = candidate;
+      return !v || v.min == null && v.max == null;
+    }
+    case "date": {
+      const v = candidate;
+      if (!v) return true;
+      return v.kind === "withinDays" ? v.days == null : !(v.from && v.to);
+    }
+  }
+}
+function assertValidDateField(field) {
+  if (process.env.NODE_ENV === "production") return;
+  const defaultDays = field.defaultDays ?? 30;
+  const maxDays = field.maxDays ?? 3650;
+  if (!Number.isInteger(maxDays) || maxDays < 1 || maxDays > 3650) {
+    throw new Error(
+      'FilterBar: date field "' + field.key + '" has an invalid maxDays (' + field.maxDays + ") \u2014 must be a whole number from 1 to 3650."
+    );
+  }
+  if (!Number.isInteger(defaultDays) || defaultDays < 1 || defaultDays > maxDays) {
+    throw new Error(
+      'FilterBar: date field "' + field.key + '" has an invalid defaultDays (' + field.defaultDays + ") \u2014 must be a whole number from 1 to maxDays (" + maxDays + ")."
+    );
+  }
+}
 function chipLabel(field, value) {
   switch (field.type) {
     case "text":
@@ -3234,18 +3338,30 @@ function MultiSelectPanel({
   field,
   value,
   anchorRef,
-  onChange,
-  onRemove,
+  onCommit,
   onClose
 }) {
   const allowedOps = operatorsFor(field);
-  const operator = allowedOps.length === 1 ? allowedOps[0] : value?.operator ?? allowedOps[0];
+  const [draftOperator, setDraftOperator] = React24.useState(value?.operator ?? allowedOps[0]);
+  const operator = allowedOps.length === 1 ? allowedOps[0] : value?.operator ?? draftOperator;
   const selected = value?.values ?? [];
   const selectedRef = React24.useRef(selected);
   selectedRef.current = selected;
+  const operatorRef = React24.useRef(operator);
+  operatorRef.current = operator;
+  const mountedRef = React24.useRef(true);
+  React24.useEffect(() => {
+    mountedRef.current = true;
+    return () => {
+      mountedRef.current = false;
+    };
+  }, []);
   const commit = (nextValues, nextOperator = operator) => {
-    if (nextValues.length === 0) onRemove();
-    else onChange({ operator: nextOperator, values: nextValues });
+    onCommit({ operator: nextOperator, values: nextValues });
+  };
+  const setOperator = (op) => {
+    setDraftOperator(op);
+    if (selected.length > 0) commit(selected, op);
   };
   const [mode2, setMode] = React24.useState("search");
   const [query, setQuery] = React24.useState("");
@@ -3283,18 +3399,20 @@ function MultiSelectPanel({
   const options = field.loadOptions ? asyncState.options ?? [] : (field.options ?? []).filter((o) => o.label.toLowerCase().includes(query.toLowerCase()));
   const showLoading = !!field.loadOptions && asyncState.loading;
   const showError = !!field.loadOptions && asyncState.error;
-  const showListbox = !showLoading && !showError;
+  const hasListbox = !showLoading && !showError && options.length > 0;
   const isSelected = (o) => selected.some((s) => s.value === o.value);
   const toggle = (o) => {
     const next = isSelected(o) ? selected.filter((s) => s.value !== o.value) : [...selected, o];
     commit(next);
   };
   const nav = useListboxNav({ count: options.length, onSelect: (i) => toggle(options[i]) });
-  useListboxScrollIntoView(nav.active, listRef, showListbox);
+  useListboxScrollIntoView(nav.active, listRef, hasListbox);
   React24.useEffect(() => {
     nav.setActive(0);
   }, [query, options.length]);
-  const activeId = showListbox && options[nav.active] ? listId + "-opt-" + nav.active : void 0;
+  const activeId = hasListbox && options[nav.active] ? listId + "-opt-" + nav.active : void 0;
+  const liveRole = showError ? "alert" : "status";
+  const liveMessage = showLoading ? "Loading\u2026" : showError ? "Couldn't load options." : options.length === 0 ? "No matches" : "";
   const [pasteText, setPasteText] = React24.useState("");
   const [pasteLoading, setPasteLoading] = React24.useState(false);
   const [unmatchedTokens, setUnmatchedTokens] = React24.useState([]);
@@ -3310,19 +3428,21 @@ function MultiSelectPanel({
     setPasteLoading(true);
     setPasteError(null);
     field.resolve(tokens).then((result) => {
+      if (!mountedRef.current) return;
       setPasteLoading(false);
       setUnmatchedTokens(result.unmatched);
       const merged = new Map(selectedRef.current.map((s) => [s.value, s]));
       for (const m of result.matched) merged.set(m.value, { value: m.value, label: m.label });
-      commit(Array.from(merged.values()));
+      commit(Array.from(merged.values()), operatorRef.current);
       setPasteText("");
     }).catch((err) => {
+      if (!mountedRef.current) return;
       setPasteLoading(false);
       setPasteError(err instanceof Error ? err.message : "Could not resolve the pasted list.");
     });
   };
   const header = /* @__PURE__ */ jsxs39("span", { className: "fd-stack fd-gap-sm", children: [
-    allowedOps.length > 1 ? /* @__PURE__ */ jsx45(OperatorToggle, { operator, onChange: (op) => commit(selected, op), label: field.label }) : null,
+    allowedOps.length > 1 ? /* @__PURE__ */ jsx45(OperatorToggle, { operator, onChange: setOperator, label: field.label }) : null,
     field.resolve ? /* @__PURE__ */ jsx45(
       SegmentedControl,
       {
@@ -3339,8 +3459,8 @@ function MultiSelectPanel({
         {
           ref: inputRef,
           role: "combobox",
-          "aria-expanded": showListbox,
-          "aria-controls": showListbox ? listId : void 0,
+          "aria-expanded": hasListbox,
+          "aria-controls": hasListbox ? listId : void 0,
           "aria-autocomplete": "list",
           "aria-activedescendant": activeId,
           "aria-label": field.label,
@@ -3396,33 +3516,41 @@ function MultiSelectPanel({
       ] }),
       /* @__PURE__ */ jsx45("span", { className: "fd-row fd-wrap fd-gap-xs", children: selected.map((v) => /* @__PURE__ */ jsx45(Tag, { onRemove: () => commit(selected.filter((s) => s.value !== v.value)), children: v.label }, v.value)) })
     ] }) : null
-  ] }) : showLoading ? /* @__PURE__ */ jsxs39("span", { className: "fd-row fd-gap-xs", role: "status", children: [
-    /* @__PURE__ */ jsx45("span", { className: "fd-spinner", "aria-hidden": "true" }),
-    " Loading\u2026"
-  ] }) : showError ? /* @__PURE__ */ jsxs39("span", { className: "fd-stack fd-gap-sm", children: [
-    /* @__PURE__ */ jsx45("span", { className: "fd-body-sm", role: "alert", children: "Couldn't load options." }),
-    /* @__PURE__ */ jsx45(Button, { size: "sm", variant: "secondary", onClick: () => setRetryTick((n) => n + 1), children: "Retry" })
-  ] }) : options.length === 0 ? /* @__PURE__ */ jsx45("span", { className: "fd-body-sm fd-muted", role: "status", children: "No matches" }) : /* @__PURE__ */ jsx45("span", { id: listId, role: "listbox", "aria-label": field.label, "aria-multiselectable": "true", ref: listRef, className: "fd-pop-list", children: options.map((o, i) => {
-    const on = isSelected(o);
-    return /* @__PURE__ */ jsxs39(
-      "div",
+  ] }) : /* @__PURE__ */ jsxs39("span", { className: "fd-stack fd-gap-sm", children: [
+    /* @__PURE__ */ jsxs39(
+      "span",
       {
-        id: listId + "-opt-" + i,
-        "data-i": i,
-        role: "option",
-        tabIndex: -1,
-        "aria-selected": on,
-        className: ["fd-opt", i === nav.active ? "is-active" : "", on ? "is-selected" : ""].filter(Boolean).join(" "),
-        onMouseEnter: () => nav.setActive(i),
-        onClick: () => toggle(o),
+        className: "fd-row fd-gap-xs" + (liveMessage && !showError ? " fd-muted" : ""),
+        role: liveRole,
         children: [
-          /* @__PURE__ */ jsx45("i", { className: "fd-filterbar-opt-check ph " + (on ? "ph-check-square is-checked" : "ph-square"), "aria-hidden": "true" }),
-          /* @__PURE__ */ jsx45("span", { className: "fd-opt-label", children: o.label })
+          showLoading ? /* @__PURE__ */ jsx45("span", { className: "fd-spinner", "aria-hidden": "true" }) : null,
+          liveMessage ? /* @__PURE__ */ jsx45("span", { className: "fd-body-sm", children: liveMessage }) : null
         ]
-      },
-      o.value
-    );
-  }) });
+      }
+    ),
+    showError ? /* @__PURE__ */ jsx45(Button, { size: "sm", variant: "secondary", onClick: () => setRetryTick((n) => n + 1), children: "Retry" }) : null,
+    hasListbox ? /* @__PURE__ */ jsx45("span", { id: listId, role: "listbox", "aria-label": field.label, "aria-multiselectable": "true", ref: listRef, className: "fd-pop-list", children: options.map((o, i) => {
+      const on = isSelected(o);
+      return /* @__PURE__ */ jsxs39(
+        "div",
+        {
+          id: listId + "-opt-" + i,
+          "data-i": i,
+          role: "option",
+          tabIndex: -1,
+          "aria-selected": on,
+          className: ["fd-opt", i === nav.active ? "is-active" : "", on ? "is-selected" : ""].filter(Boolean).join(" "),
+          onMouseEnter: () => nav.setActive(i),
+          onClick: () => toggle(o),
+          children: [
+            /* @__PURE__ */ jsx45("i", { className: "fd-filterbar-opt-check ph " + (on ? "ph-check-square is-checked" : "ph-square"), "aria-hidden": "true" }),
+            /* @__PURE__ */ jsx45("span", { className: "fd-opt-label", children: o.label })
+          ]
+        },
+        o.value
+      );
+    }) }) : null
+  ] });
   const footer = mode2 === "paste" ? /* @__PURE__ */ jsx45(Button, { size: "sm", variant: "secondary", onClick: doResolve, disabled: pasteLoading || !pasteText.trim(), children: pasteLoading ? "Resolving\u2026" : "Resolve" }) : void 0;
   return /* @__PURE__ */ jsx45(
     Popover,
@@ -3444,22 +3572,21 @@ function RangePanel({
   field,
   value,
   anchorRef,
-  onChange,
-  onRemove,
+  onCommit,
   onClose
 }) {
   const [draftMin, setDraftMin] = React24.useState(value?.min);
   const [draftMax, setDraftMax] = React24.useState(value?.max);
-  const invalid = draftMin != null && draftMax != null && draftMin > draftMax;
+  const [minTextValid, setMinTextValid] = React24.useState(true);
+  const [maxTextValid, setMaxTextValid] = React24.useState(true);
+  const isEmpty = draftMin == null && draftMax == null;
+  const validation = isEmpty ? { ok: true } : validateFilterValue(field, { min: draftMin, max: draftMax });
+  const comboError = !isEmpty && !validation.ok ? validation.reason : void 0;
+  const doneDisabled = !minTextValid || !maxTextValid || !!comboError;
   const apply = (nextMin, nextMax) => {
     setDraftMin(nextMin);
     setDraftMax(nextMax);
-    if (nextMin == null && nextMax == null) {
-      onRemove();
-      return;
-    }
-    if (nextMin != null && nextMax != null && nextMin > nextMax) return;
-    onChange({ min: nextMin, max: nextMax });
+    onCommit(nextMin == null && nextMax == null ? void 0 : { min: nextMin, max: nextMax });
   };
   return /* @__PURE__ */ jsx45(
     Popover,
@@ -3471,7 +3598,7 @@ function RangePanel({
       offset: 6,
       width: PANEL_WIDTH.range,
       label: field.label,
-      footer: /* @__PURE__ */ jsx45(Button, { size: "sm", variant: "secondary", onClick: onClose, children: "Done" }),
+      footer: /* @__PURE__ */ jsx45(Button, { size: "sm", variant: "secondary", onClick: onClose, disabled: doneDisabled, children: "Done" }),
       children: /* @__PURE__ */ jsxs39("span", { className: "fd-stack fd-gap-sm", children: [
         /* @__PURE__ */ jsx45(
           NumberInput,
@@ -3481,8 +3608,10 @@ function RangePanel({
             min: field.min,
             max: field.max,
             step: field.step,
+            integer: field.integer,
             placeholder: field.minPlaceholder,
-            onChange: (e) => apply(e.target.value, draftMax)
+            onChange: (e) => apply(e.target.value, draftMax),
+            onValidityChange: setMinTextValid
           }
         ),
         /* @__PURE__ */ jsx45(
@@ -3493,9 +3622,11 @@ function RangePanel({
             min: field.min,
             max: field.max,
             step: field.step,
-            error: invalid ? "Must be \u2265 " + draftMin : void 0,
+            integer: field.integer,
+            error: comboError,
             placeholder: field.maxPlaceholder,
-            onChange: (e) => apply(draftMin, e.target.value)
+            onChange: (e) => apply(draftMin, e.target.value),
+            onValidityChange: setMaxTextValid
           }
         )
       ] })
@@ -3506,49 +3637,35 @@ function DatePanel({
   field,
   value,
   anchorRef,
-  onChange,
-  onRemove,
+  onCommit,
   onClose
 }) {
   const defaultDays = field.defaultDays ?? 30;
-  const [draft, setDraft] = React24.useState(value ?? { kind: "withinDays", days: defaultDays });
-  const committedRef = React24.useRef(!!value);
+  const maxDays = field.maxDays ?? 3650;
+  const [kind, setKind] = React24.useState(value?.kind ?? "withinDays");
+  const [daysDraft, setDaysDraft] = React24.useState(
+    value?.kind === "withinDays" ? value.days : defaultDays
+  );
+  const [between, setBetween] = React24.useState(
+    value?.kind === "between" ? { from: value.from, to: value.to } : { from: "", to: "" }
+  );
   const applyWithinDays = (days) => {
-    const next = { kind: "withinDays", days: days ?? defaultDays };
-    setDraft(next);
-    if (days != null) {
-      committedRef.current = true;
-      onChange(next);
-    }
+    setDaysDraft(days);
+    onCommit(days == null ? void 0 : { kind: "withinDays", days });
   };
   const applyBetween = (from, to) => {
-    const next = { kind: "between", from, to };
-    setDraft(next);
-    if (from && to) {
-      committedRef.current = true;
-      onChange(next);
-    } else if (committedRef.current) {
-      committedRef.current = false;
-      onRemove();
-    }
+    setBetween({ from, to });
+    onCommit(from && to ? { kind: "between", from, to } : void 0);
   };
-  const switchKind = (kind) => {
-    if (kind === "withinDays") {
-      const days = draft.kind === "withinDays" ? draft.days : defaultDays;
-      setDraft({ kind: "withinDays", days });
-    } else {
-      setDraft({ kind: "between", from: "", to: "" });
-      if (committedRef.current) {
-        committedRef.current = false;
-        onRemove();
-      }
-    }
+  const switchKind = (next) => {
+    if (next === kind) return;
+    setKind(next);
+    if (next === "between") setBetween({ from: "", to: "" });
+    else setDaysDraft(defaultDays);
+    onCommit(void 0);
   };
   const confirm = () => {
-    if (draft.kind === "withinDays") {
-      committedRef.current = true;
-      onChange(draft);
-    }
+    if (kind === "withinDays" && daysDraft != null) onCommit({ kind: "withinDays", days: daysDraft });
     onClose();
   };
   return /* @__PURE__ */ jsx45(
@@ -3571,16 +3688,16 @@ function DatePanel({
               { value: "withinDays", label: "Within last N days" },
               { value: "between", label: "Between dates" }
             ],
-            value: draft.kind,
+            value: kind,
             onChange: (k) => switchKind(k)
           }
         ),
-        draft.kind === "withinDays" ? /* @__PURE__ */ jsx45(NumberInput, { label: "Days", min: 1, value: draft.days, onChange: (e) => applyWithinDays(e.target.value) }) : /* @__PURE__ */ jsx45(
+        kind === "withinDays" ? /* @__PURE__ */ jsx45(NumberInput, { label: "Days", min: 1, max: maxDays, step: 1, integer: true, value: daysDraft ?? "", onChange: (e) => applyWithinDays(e.target.value) }) : /* @__PURE__ */ jsx45(
           DatePicker,
           {
             range: true,
             label: "Date range",
-            value: { start: draft.from, end: draft.to },
+            value: { start: between.from, end: between.to },
             onChange: (e) => applyBetween(e.target.value.start || "", e.target.value.end || "")
           }
         )
@@ -3594,6 +3711,21 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
   const addAnchorRef = React24.useRef(null);
   const textInputRef = React24.useRef(null);
   const optionsRef = React24.useRef(null);
+  if (process.env.NODE_ENV !== "production") {
+    for (const f of fields) if (f.type === "date") assertValidDateField(f);
+  }
+  const commit = (field, candidate) => {
+    if (isEmptyCandidate(field, candidate)) {
+      removeField(field);
+      return { ok: true };
+    }
+    const result = validateFilterValue(field, candidate);
+    if (result.ok) onFilterChange(field.key, candidate);
+    return result;
+  };
+  const removeField = (field) => {
+    if (isFieldSet(field, filters[field.key])) onFilterRemove(field.key);
+  };
   const editingField = panel && panel.kind === "edit" ? fields.find((f) => f.key === panel.key) : void 0;
   const startEditing = (key) => setPanel({ kind: "edit", key });
   const closeEditing = () => setPanel(null);
@@ -3625,7 +3757,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
               value: value ?? "",
               placeholder: field.placeholder || field.label,
               "aria-label": field.label,
-              onChange: (e) => onFilterChange(field.key, e.target.value),
+              onChange: (e) => commit(field, e.target.value),
               onKeyDown: (e) => {
                 if (e.key === "Enter") setPanel(null);
                 else if (e.key === "Escape") {
@@ -3644,7 +3776,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
               "aria-label": "Remove " + field.label,
               onClick: (e) => {
                 e.stopPropagation();
-                onFilterRemove(field.key);
+                removeField(field);
                 setPanel(null);
               },
               children: /* @__PURE__ */ jsx45("i", { className: "ph ph-x", "aria-hidden": "true" })
@@ -3657,7 +3789,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
       const chipRef = editing ? editAnchorRef : void 0;
       const toggleOrClose = () => editing ? closeEditing() : startEditing(field.key);
       const remove = () => {
-        if (set) onFilterRemove(field.key);
+        removeField(field);
         closeEditing();
       };
       const chipText = field.label + (set ? ": " + chipLabel(field, value) : "");
@@ -3687,7 +3819,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
                     "aria-selected": on,
                     className: ["fd-opt", on ? "is-selected" : ""].filter(Boolean).join(" "),
                     onClick: () => {
-                      onFilterChange(field.key, o.value);
+                      commit(field, o.value);
                       setPanel(null);
                     },
                     children: /* @__PURE__ */ jsx45("span", { className: "fd-opt-label", children: o.label })
@@ -3708,8 +3840,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
               field,
               value,
               anchorRef: editAnchorRef,
-              onChange: (v) => onFilterChange(field.key, v),
-              onRemove: () => onFilterRemove(field.key),
+              onCommit: (v) => commit(field, v),
               onClose: closeEditing
             }
           ) : null
@@ -3724,8 +3855,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
               field,
               value,
               anchorRef: editAnchorRef,
-              onChange: (v) => onFilterChange(field.key, v),
-              onRemove: () => onFilterRemove(field.key),
+              onCommit: (v) => commit(field, v),
               onClose: closeEditing
             }
           ) : null
@@ -3740,8 +3870,7 @@ function FilterBar({ fields = [], filters, onFilterChange, onFilterRemove, align
               field,
               value,
               anchorRef: editAnchorRef,
-              onChange: (v) => onFilterChange(field.key, v),
-              onRemove: () => onFilterRemove(field.key),
+              onCommit: (v) => commit(field, v),
               onClose: closeEditing
             }
           ) : null