@delmaredigital/payload-experiments 0.2.0-rc.7 → 0.3.0-rc.4

package/dist/admin/surface/test/fields.js

@@ -1,6 +1,7 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useEffect, useId, useState } from 'react';
+import { TextInput, NumberInput, Textarea, Select, SearchInput, CheckboxList as UICheckboxList } from '../../../ui/Inputs.js';
 import { TEST_PAGE as C } from '../../copy/testPage.js';
 import { EDITABLE_BY_STATUS } from '../../../collections/experiments/editability.js';
 import { lockedReason } from '../../lockedNotice.model.js';
@@ -21,13 +22,19 @@ export function FieldErrors({ test, path }) {
             }, key))
     });
 }
-export function ReadOnly({ label, value, reason }) {
+export function ReadOnly({ label, value, reason, labelAction }) {
     return /*#__PURE__*/ _jsxs("div", {
         className: "xp-read-only",
         title: reason ?? undefined,
         children: [
-            /*#__PURE__*/ _jsx("strong", {
-                children: label
+            /*#__PURE__*/ _jsxs("div", {
+                className: "xp-ui-field-label",
+                children: [
+                    /*#__PURE__*/ _jsx("strong", {
+                        children: label
+                    }),
+                    labelAction
+                ]
             }),
             /*#__PURE__*/ _jsx("p", {
                 children: value
@@ -38,7 +45,7 @@ export function ReadOnly({ label, value, reason }) {
         ]
     });
 }
-/** Local typing is intentionally separate from staging: build the current group on commit. */ export function TextEdit({ label, value, onCommit, reason, multiline, placeholder, type = 'text', min, max, required, onDraft, validate, invalidMessage }) {
+/** Local typing is intentionally separate from staging: build the current group on commit. */ export function TextEdit({ label, value, onCommit, reason, multiline, placeholder, type = 'text', min, max, required, onDraft, validate, invalidMessage, hideLabel = false, disabled = false, help }) {
     const id = useId();
     const [draft, setDraft] = useState(String(value));
     const [error, setError] = useState(null);
@@ -55,111 +62,103 @@ export function ReadOnly({ label, value, reason }) {
         }
         onCommit(draft);
     };
-    if (reason) return /*#__PURE__*/ _jsx(ReadOnly, {
+    if (reason && !disabled) return /*#__PURE__*/ _jsx(ReadOnly, {
         label: label,
         value: value === '' && placeholder ? placeholder : value,
         reason: reason
     });
-    return /*#__PURE__*/ _jsxs(_Fragment, {
-        children: [
-            /*#__PURE__*/ _jsxs("label", {
-                className: "xp-field",
-                htmlFor: id,
-                children: [
-                    /*#__PURE__*/ _jsx("span", {
-                        children: label
-                    }),
-                    multiline ? /*#__PURE__*/ _jsx("textarea", {
-                        id: id,
-                        value: draft,
-                        placeholder: placeholder,
-                        onChange: (e)=>{
-                            setDraft(e.target.value);
-                            onDraft?.(e.target.value);
-                        },
-                        onBlur: commit
-                    }) : /*#__PURE__*/ _jsx("input", {
-                        id: id,
-                        type: type,
-                        value: draft,
-                        min: min,
-                        max: max,
-                        required: required,
-                        placeholder: placeholder,
-                        "aria-invalid": error ? true : undefined,
-                        "aria-describedby": error ? `${id}-error` : undefined,
-                        onChange: (e)=>{
-                            setError(null);
-                            setDraft(e.target.value);
-                            onDraft?.(e.target.value);
-                        },
-                        onBlur: commit,
-                        onKeyDown: (e)=>{
-                            if (e.key === 'Enter') e.currentTarget.blur();
-                        }
-                    })
-                ]
-            }),
-            error ? /*#__PURE__*/ _jsx("p", {
-                className: "xp-field-error",
-                id: `${id}-error`,
-                role: "alert",
-                children: error
-            }) : null
-        ]
+    const Input = multiline ? Textarea : type === 'number' ? NumberInput : TextInput;
+    return /*#__PURE__*/ _jsx("div", {
+        className: hideLabel ? 'xp-grid-field' : undefined,
+        title: reason ?? undefined,
+        children: /*#__PURE__*/ _jsx(Input, {
+            id: id,
+            label: label,
+            "aria-label": label,
+            value: draft,
+            disabled: disabled,
+            placeholder: placeholder,
+            min: min,
+            max: max,
+            required: required,
+            help: help,
+            error: error ?? undefined,
+            onChange: (e)=>{
+                setError(null);
+                setDraft(e.target.value);
+                onDraft?.(e.target.value);
+            },
+            onBlur: commit,
+            onKeyDown: (e)=>{
+                if (!multiline && e.key === 'Enter') e.currentTarget.blur();
+            }
+        })
     });
 }
 export const positiveInteger = (raw)=>raw.trim() !== '' && Number.isInteger(Number(raw)) && Number(raw) > 0;
-export function SelectEdit({ label, value, options, reason, onCommit, readOnlyValue }) {
+export function SelectEdit({ label, value, options, reason, onCommit, readOnlyValue, labelAction }) {
     const id = useId();
     if (reason) {
         return /*#__PURE__*/ _jsx(ReadOnly, {
             label: label,
             reason: reason,
+            labelAction: labelAction,
             value: readOnlyValue ?? options.find((o)=>o.value === value)?.label ?? value
         });
     }
-    return /*#__PURE__*/ _jsxs("label", {
-        className: "xp-field",
-        htmlFor: id,
-        children: [
-            /*#__PURE__*/ _jsx("span", {
-                children: label
-            }),
-            /*#__PURE__*/ _jsx("select", {
-                id: id,
-                value: value,
-                onChange: (e)=>onCommit(e.target.value),
-                children: options.map((option)=>/*#__PURE__*/ _jsx("option", {
-                        value: option.value,
-                        children: option.label
-                    }, option.value))
-            })
-        ]
+    return /*#__PURE__*/ _jsx(Select, {
+        id: id,
+        label: label,
+        labelAction: labelAction,
+        value: value,
+        onChange: (e)=>onCommit(e.target.value),
+        children: options.map((option)=>/*#__PURE__*/ _jsx("option", {
+                value: option.value,
+                children: option.label
+            }, option.value))
     });
 }
-export function CheckboxList({ label, value, options, reason, onToggle, readOnlyValue }) {
+export function CheckboxList({ label, value, options, reason, onToggle, readOnlyValue, searchable = false }) {
+    const id = useId();
+    const [query, setQuery] = useState('');
+    const visible = searchable ? options.filter((option)=>value.includes(option.value) || option.label.toLowerCase().includes(query.toLowerCase())) : options;
     if (reason) return /*#__PURE__*/ _jsx(ReadOnly, {
         label: label,
         reason: reason,
         value: readOnlyValue ?? value.map((v)=>options.find((o)=>o.value === v)?.label ?? v).join(', ')
     });
-    return /*#__PURE__*/ _jsxs("fieldset", {
-        className: "xp-field",
+    return /*#__PURE__*/ _jsx(UICheckboxList, {
+        id: id,
+        label: label,
+        value: value,
+        options: visible,
+        onChange: (next)=>{
+            const changed = options.find((option)=>next.includes(option.value) !== value.includes(option.value));
+            if (changed) onToggle(changed.value, next.includes(changed.value));
+        },
+        children: searchable ? /*#__PURE__*/ _jsx(SearchInput, {
+            id: `${id}-search`,
+            label: C.searchGoals,
+            value: query,
+            onChange: (event)=>setQuery(event.target.value)
+        }) : null
+    });
+}
+export function TestName({ test }) {
+    return /*#__PURE__*/ _jsxs(_Fragment, {
         children: [
-            /*#__PURE__*/ _jsx("legend", {
-                children: label
+            /*#__PURE__*/ _jsx(TextEdit, {
+                label: C.name,
+                value: test.doc?.name ?? '',
+                reason: reasonFor(test, 'name'),
+                onCommit: (value)=>save(test, 'name', ()=>({
+                            name: value
+                        }))
             }),
-            options.map((option)=>/*#__PURE__*/ _jsxs("label", {
-                    children: [
-                        /*#__PURE__*/ _jsx("input", {
-                            type: "checkbox",
-                            checked: value.includes(option.value),
-                            onChange: (e)=>onToggle(option.value, e.target.checked)
-                        }),
-                        option.label
-                    ]
-                }, option.value))
+            /*#__PURE__*/ _jsx(FieldErrors, {
+                test: test,
+                path: "name"
+            })
         ]
     });
 }

package/dist/admin/surface/test/jumpToSection.d.ts

@@ -0,0 +1,3 @@
+import type { SectionId } from '../contracts.js';
+/** Open disclosures before scrolling, then give keyboard users a meaningful landing. */
+export declare function jumpToSection(section: SectionId): void;

package/dist/admin/surface/test/jumpToSection.js

@@ -0,0 +1,10 @@
+/** Open disclosures before scrolling, then give keyboard users a meaningful landing. */ export function jumpToSection(section) {
+    const target = document.getElementById(`section-${section}`);
+    if (target instanceof HTMLDetailsElement) target.open = true;
+    target?.scrollIntoView?.({
+        block: 'start'
+    });
+    document.getElementById(`title-${section}`)?.focus({
+        preventScroll: true
+    });
+}

package/dist/admin/surface/test/pageParts.js

@@ -1,17 +1,23 @@
 'use client';
-import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { TEST_PAGE as C } from '../../copy/testPage.js';
 import { pageTargetIssue, setPageTarget, showsPageTarget } from '../../arms/pageTarget.model.js';
 import { FieldErrors, reasonFor, save, TextEdit } from './fields.js';
 function VersionContent({ test, armIndex }) {
     const arm = test.doc?.arms[armIndex];
     if (!arm || !test.doc) return null;
-    if (!showsPageTarget(test.doc.kind, arm.kind)) return /*#__PURE__*/ _jsx("p", {
+    if (!showsPageTarget(test.doc.kind, arm.kind)) return /*#__PURE__*/ _jsx("span", {
+        className: "xp-content-chip",
         children: C.holdoutContent
     });
     const issue = pageTargetIssue(arm.pageTarget, arm.kind);
-    return /*#__PURE__*/ _jsxs(_Fragment, {
+    return /*#__PURE__*/ _jsxs("div", {
+        className: "xp-version-content",
         children: [
+            /*#__PURE__*/ _jsx("span", {
+                className: `xp-content-chip ${issue ? 'xp-content-empty' : 'xp-content-ready'}`,
+                children: issue ? C.addPageAddress : C.pageReady
+            }),
             /*#__PURE__*/ _jsx(TextEdit, {
                 label: `${arm.label}: ${C.pageAddress}`,
                 value: arm.pageTarget ?? '',

package/dist/admin/surface/test/testPage.css

@@ -1,23 +1,90 @@
 .xp-test-page { color: var(--xp-text); background: var(--xp-bg); max-width: 90rem; margin: 0 auto; padding: 1.5rem; }
+.xp-test-page--setup { min-width: 0; padding: 0 0 160px; }
 .xp-test-header { position: sticky; top: 0; z-index: 10; background: var(--xp-bg); padding: 1rem 0; border-bottom: 1px solid var(--xp-border); }
 .xp-header-meta, .xp-section-actions { display: flex; flex-wrap: wrap; align-items: center; gap: 1rem; margin-block: 1rem; }
-.xp-kind-chip, .xp-status-pill { border: 1px solid var(--xp-border-control); border-radius: 1rem; padding: .25rem .75rem; }
+.xp-kind-chip, .xp-status-pill { border: 1px solid var(--xp-border-strong); border-radius: 1rem; padding: .25rem .75rem; }
 .xp-status-pill--success, .xp-content-ready, .xp-section-check--done { color: var(--xp-success-text); }
 .xp-content-empty, .xp-status-pill--warning { color: var(--xp-warning-text); }
-.xp-test-section { padding-block: 1.5rem; border-bottom: 1px solid var(--xp-border); scroll-margin-top: 15rem; }
-.xp-version-rows { display: grid; gap: 1rem; grid-template-columns: repeat(auto-fit, minmax(min(100%, 16rem), 1fr)); }
-.xp-version-row { border: 1px solid var(--xp-border); border-radius: var(--xp-radius); padding: 1rem; }
-.xp-version-colour { display: inline-block; width: .75rem; height: .75rem; margin-right: .5rem; border-radius: 50%; }
-.xp-traffic-bar { display: flex; height: 1.25rem; background: var(--xp-surface-raised); margin-block: 1.5rem; overflow: hidden; border-radius: var(--xp-radius); }
+.xp-test-section, .xp-test-advanced { scroll-margin-top: var(--xp-space-6); }
+.xp-version-rows { display: grid; grid-template-columns: minmax(0, 1.3fr) var(--xp-version-share-width) minmax(var(--xp-version-state-width), 1fr) var(--xp-version-remove-width); gap: var(--xp-space-3); align-items: center; }
+.xp-version-row, .xp-version-grid-head { display: grid; grid-column: 1 / -1; grid-template-columns: subgrid; gap: var(--xp-space-3); align-items: center; }
+.xp-version-grid-head { margin-bottom: var(--xp-space-2); font: var(--xp-text-small); color: var(--xp-text-2); }
+.xp-version-row > .xp-field-error, .xp-version-row > [role=alert] { grid-column: 1 / -1; }
+.xp-version-remove { grid-column: 4; }
+.xp-grid-field .xp-ui-field { margin: 0; }
+.xp-grid-field .xp-ui-field > label { position: absolute; width: 1px; height: 1px; overflow: hidden; clip-path: inset(50%); white-space: nowrap; }
+.xp-version-content { display: flex; flex-wrap: nowrap; gap: var(--xp-space-2); align-items: center; min-width: 0; }
+.xp-version-content .xp-ui-button { white-space: nowrap; }
+.xp-content-chip { flex-shrink: 0; display: inline-flex; width: fit-content; border-radius: var(--xp-radius-chip); padding: var(--xp-space-half) var(--xp-space-2); font: var(--xp-text-small); background: var(--xp-surface-2); color: var(--xp-text-2); }
+.xp-content-chip.xp-content-ready { background: var(--xp-success-bg); color: var(--xp-success); }
+.xp-content-chip.xp-content-empty { background: var(--xp-warning-bg); color: var(--xp-warning); }
+.xp-traffic-bar { display: flex; height: var(--xp-space-2); gap: var(--xp-space-half); margin-block: var(--xp-space-5) var(--xp-space-3); overflow: hidden; border-radius: var(--xp-space-1); }
 .xp-traffic-bar > span { display: block; }
-.xp-field { display: flex; flex-direction: column; gap: .5rem; margin-block: 1rem; }
-.xp-field input, .xp-field textarea, .xp-field select { font: inherit; color: var(--xp-text); background: var(--xp-input-bg); border: 1px solid var(--xp-border-control); border-radius: var(--xp-radius); padding: .65rem; }
-.xp-field textarea { min-height: 6rem; resize: vertical; }
-.xp-field input[type=range] { padding: 0; }
+.xp-split-legend { display: flex; flex-wrap: wrap; gap: var(--xp-space-4); font: var(--xp-text-small); color: var(--xp-text-2); }
+.xp-split-legend i { display: inline-block; width: var(--xp-legend-key-size); height: var(--xp-legend-key-size); border-radius: var(--xp-legend-key-radius); margin-right: var(--xp-space-one-half); }
+.xp-field-grid { display: grid; grid-template-columns: repeat(2, minmax(0, 1fr)); gap: var(--xp-space-4); }
+.xp-test-advanced .xp-field-grid { padding-top: var(--xp-space-4); }
+.xp-field-grid > div { min-width: 0; }
+.xp-field-grid p, .xp-page-binding p { color: var(--xp-text-2); font: var(--xp-text-small); }
+.xp-read-only code { font-family: var(--xp-mono); overflow-wrap: anywhere; }
+.xp-ui-checkbox-list { display: flex; flex-wrap: wrap; gap: var(--xp-space-2) var(--xp-space-4); }
+.xp-duration-suggestion { align-items: center; margin-top: var(--xp-space-4); }
 .xp-read-only { padding-block: .5rem; }
 .xp-read-only p { margin-block: .5rem; }
 .xp-field-error { color: var(--xp-error-text); }
-.xp-test-page button { cursor: pointer; color: var(--xp-text); background: var(--xp-surface-raised); border: 1px solid var(--xp-border-control); padding: .65rem 1rem; border-radius: var(--xp-radius); font: inherit; }
-.xp-test-page button:disabled { opacity: .5; cursor: default; }
-.xp-test-page summary { cursor: pointer; padding-block: .75rem; }
-.xp-targeting-row { border-bottom: 1px solid var(--xp-border); padding-bottom: 1rem; }
+
+.xp-targeting-row { display: grid; grid-template-columns: var(--xp-targeting-attribute-width) var(--xp-targeting-comparison-width) minmax(0, 1fr); align-items: start; gap: var(--xp-space-3); border: 1px solid var(--xp-border); border-radius: var(--xp-radius-control); padding: var(--xp-space-4); margin-bottom: var(--xp-space-4); }
+.xp-targeting-row > .xp-ui-button-group, .xp-targeting-row > p { grid-column: 1 / -1; }
+@media (max-width: 720px) {
+  .xp-version-rows { grid-template-columns: minmax(0, 1fr) var(--xp-version-share-width-mobile); }
+  .xp-version-content { flex-wrap: wrap; }
+  .xp-version-remove:empty { display: none; }
+  .xp-version-content, .xp-setting-values, .xp-version-remove { grid-column: 1 / -1; }
+  .xp-version-grid-head > :nth-child(n+3) { display: none; }
+  .xp-targeting-row, .xp-field-grid { grid-template-columns: 1fr; }
+}
+
+.xp-test-layout { display: grid; grid-template-columns: 220px minmax(0, 1fr); align-items: start; gap: var(--xp-space-8); }
+.xp-test-content { display: grid; gap: var(--xp-space-8); min-width: 0; max-width: 760px; }
+.xp-test-rail { position: sticky; top: var(--xp-space-6); min-width: 0; }
+.xp-test-rail-list { list-style: none; padding: 0; margin: 0; display: flex; flex-direction: column; gap: var(--xp-space-1); }
+.xp-test-rail-item { display: flex; align-items: center; gap: var(--xp-space-2); padding: var(--xp-space-2); border-radius: var(--xp-radius-control); color: var(--xp-text-2); text-decoration: none; font: var(--xp-text-small); }
+.xp-test-rail-item:hover, .xp-test-rail-item.is-active { background: var(--xp-surface-2); color: var(--xp-text); }
+.xp-test-rail-item.is-active { font-weight: 600; }
+.xp-test-rail-item--advanced { margin-top: var(--xp-space-3); color: var(--xp-text-3); }
+.xp-test-rail-mark { flex: none; color: var(--xp-text-3); }
+.xp-test-rail-mark--done { color: var(--xp-success); }
+.xp-test-sr-only { position: absolute; width: 1px; height: 1px; overflow: hidden; clip-path: inset(50%); white-space: nowrap; }
+.xp-test-rail-item:focus-visible, .xp-test-advanced > summary:focus-visible, .xp-test-content h2:focus { outline: 2px solid var(--xp-focus); outline-offset: 2px; }
+.xp-test-advanced { border: 1px solid var(--xp-border); border-radius: var(--xp-radius-card); background: var(--xp-surface); }
+.xp-test-advanced > summary { cursor: pointer; display: flex; align-items: center; justify-content: space-between; gap: var(--xp-space-3); padding: var(--xp-space-4) var(--xp-space-5); color: var(--xp-text-2); }
+.xp-test-advanced > summary h2 { margin: 0; font: var(--xp-text-body); }
+.xp-test-advanced[open] > summary { border-bottom: 1px solid var(--xp-border); }
+.xp-test-readiness-bar { position: fixed; bottom: 0; inset-inline: 0; z-index: 20; background: var(--xp-surface); border-top: 1px solid var(--xp-border); }
+.xp-readiness { color: var(--xp-text); background: var(--xp-surface); padding: var(--xp-space-4); border: 1px solid var(--xp-border-strong); border-radius: var(--xp-radius-control); }
+.xp-readiness--ready:not(.xp-readiness--bar) { background: var(--xp-success-bg); }
+.xp-readiness--bar { display: grid; grid-template-columns: minmax(0, 1fr) auto; align-items: center; gap: var(--xp-space-2) var(--xp-space-4); padding-block: var(--xp-space-3); min-height: 64px; border: 0; border-radius: 0; }
+.xp-readiness__state { display: flex; align-items: center; flex-wrap: wrap; gap: var(--xp-space-2); min-width: 0; }
+.xp-readiness__state p, .xp-readiness--bar > p { margin: 0; }
+.xp-readiness__state > p { font-weight: 500; }
+.xp-readiness--bar .xp-save-status, .xp-readiness__separator { color: var(--xp-text-3); font: var(--xp-text-small); }
+.xp-readiness__actions { display: flex; flex-wrap: wrap; align-items: center; gap: var(--xp-space-2); }
+.xp-readiness--bar > .xp-readiness__actions { grid-column: 2; grid-row: 1 / span 3; justify-content: flex-end; }
+@media (max-width: 960px) {
+  .xp-test-layout { grid-template-columns: minmax(0, 1fr); gap: var(--xp-space-5); }
+  .xp-test-rail { top: 0; z-index: 2; background: var(--xp-bg); padding-block: var(--xp-space-2); border-bottom: 1px solid var(--xp-border); }
+  .xp-test-rail-list { flex-direction: row; overflow-x: auto; }
+  .xp-test-rail-item { white-space: nowrap; }
+  .xp-test-rail-item--advanced { margin-top: 0; }
+  .xp-test-content { max-width: none; }
+  .xp-test-section, .xp-test-advanced { scroll-margin-top: 64px; }
+}
+@media (max-width: 720px) {
+  .xp-readiness--bar { grid-template-columns: minmax(0, 1fr); }
+  /* The sentence wraps under the save state here, so a trailing separator would dangle. */
+  .xp-readiness--bar .xp-readiness__separator { position: absolute; width: 1px; height: 1px; overflow: hidden; clip-path: inset(50%); }
+  .xp-readiness--bar > .xp-readiness__actions { grid-column: 1; grid-row: auto; justify-content: flex-start; }
+}
+
+.xp-ui-checkbox-list > .xp-ui-field:has(input[type="search"]) { flex-basis: 100%; }
+.xp-targeting-row .xp-ui-chips { margin-block: 0; }

package/dist/admin/surface/test/testPage.model.d.ts

@@ -1,4 +1,4 @@
-import type { Arm, ArmKey, ExperimentKind, ExposurePolicy } from '../../../types/contracts.js';
+import type { Arm, ExperimentKind, ExposurePolicy } from '../../../types/contracts.js';
 import type { PreflightResult } from '../../../types/endpoints.js';
 export declare function orderedVersions(arms: readonly Arm[]): Array<{
     arm: Arm;
@@ -13,5 +13,4 @@ export declare function durationSuggestion(preflight: PreflightResult | null, sa
     feasible: boolean;
 } | null;
 export declare function exposureOptions(kind: ExperimentKind): readonly ExposurePolicy[];
-/** Keep the edited share exact and distribute the remaining whole percentages proportionally. */
-export declare function balanceWeight(arms: Arm[], key: ArmKey, weight: number): Arm[];
+export { balanceWeight } from '../../../arms/model.js';

package/dist/admin/surface/test/testPage.model.js

@@ -1,4 +1,3 @@
-import { setWeight } from '../../arms/arms.model.js';
 const ORDER = [
     'control',
     'variantA',
@@ -31,23 +30,4 @@ export function exposureOptions(kind) {
         'manual'
     ];
 }
-/** Keep the edited share exact and distribute the remaining whole percentages proportionally. */ export function balanceWeight(arms, key, weight) {
-    if (!arms.some((arm)=>arm.key === key) || !Number.isInteger(weight) || weight < 0 || weight > 100) return arms;
-    const others = arms.filter((arm)=>arm.key !== key);
-    if (!others.length) return arms;
-    const total = others.reduce((sum, arm)=>sum + Math.max(0, Number.isFinite(arm.weight) ? arm.weight : 0), 0);
-    const denominator = total || others.length;
-    const raw = others.map((arm)=>(total ? Math.max(0, Number.isFinite(arm.weight) ? arm.weight : 0) : 1) * (100 - weight) / denominator);
-    const whole = raw.map(Math.floor);
-    const order = raw.map((_, i)=>i).sort((a, b)=>raw[b] - whole[b] - (raw[a] - whole[a]) || a - b);
-    const remainder = 100 - weight - whole.reduce((sum, value)=>sum + value, 0);
-    for(let i = 0; i < remainder; i++)whole[order[i]]++;
-    const shares = new Map(others.map((arm, i)=>[
-            arm.key,
-            whole[i]
-        ]));
-    return setWeight(arms, key, weight).map((arm)=>shares.has(arm.key) ? {
-            ...arm,
-            weight: shares.get(arm.key)
-        } : arm);
-}
+export { balanceWeight } from '../../../arms/model.js';

package/dist/admin/surface/useTestDocument.js

@@ -155,11 +155,13 @@ function stage(session, partial) {
             }
             session.publish();
         }
-        async function readDocument() {
+        async function readDocument(forSave = false) {
             const response = await fetch(url, {
                 method: 'GET',
                 credentials: 'include',
-                signal: controller.signal
+                ...forSave ? {} : {
+                    signal: controller.signal
+                }
             });
             if (!response.ok) throw new Error(API_MESSAGES.unreadable);
             const doc = await response.json();
@@ -181,6 +183,10 @@ function stage(session, partial) {
                 let failed = false;
                 try {
                     if (!session.saved) return;
+                    if (typeof session.saved.updatedAt !== 'string' || !session.saved.updatedAt) {
+                        session.saved = await readDocument(true);
+                        if (typeof session.saved.updatedAt !== 'string' || !session.saved.updatedAt) throw new Error(API_MESSAGES.unreadable);
+                    }
                     const patch = changedPatch(session.saved, queued);
                     changed = Object.keys(patch).length > 0;
                     if (changed) {
@@ -209,8 +215,9 @@ function stage(session, partial) {
                                     ...edit
                                 }), {});
                             session.queue = [];
+                            // Server text is never shown; the stale-edit sentence is the admin's own.
                             session.view.errors = {
-                                _section: staleError.message || TEST_PAGE.staleEdit
+                                _section: TEST_PAGE.staleEdit
                             };
                         } else if (!response.ok) session.view.errors = {
                             ...session.view.errors,
@@ -221,8 +228,8 @@ function stage(session, partial) {
                             const doc = body?.doc;
                             const readable = doc && doc.id != null && doc.status;
                             // A 2xx has applied even if its response body was lost. Preserve it while re-reading.
-                            // The old timestamp is no longer a valid baseline. Omit the header until a
-                            // readable PATCH or GET restores it, including when this read is aborted.
+                            // The old timestamp is no longer a valid baseline. The next queued save
+                            // must recover it with a GET before sending another PATCH.
                             session.saved = readable ? doc : {
                                 ...session.saved,
                                 ...patch,

package/dist/admin/targeting/targeting.model.d.ts

@@ -1,6 +1,7 @@
 import type { TargetingRule } from '../../types/contracts.js';
 import type { SerializedAttributeKeys } from '../../types/admin.js';
 import type { TargetingOp } from '../../targeting/ops.js';
+export { readListValue } from '../../targeting/values.js';
 type Attribute = SerializedAttributeKeys[string];
 export type AttributeType = Attribute['type'];
 export declare function opsFor(type?: AttributeType): readonly TargetingOp[];
@@ -13,11 +14,8 @@ export declare function valueInput(attribute?: Attribute, op?: TargetingOp): {
     }[];
 };
 export declare function coerceValue(type: AttributeType, op: TargetingOp, raw: unknown): string | number | boolean | string[] | null;
-/** Only the read boundary accepts legacy comma-separated document values. */
-export declare function readListValue(raw: unknown): string[];
 /** Normalize the read boundary before any whole-group edit. */
 export declare function readTargetingRules(rules: TargetingRule[]): TargetingRule[];
 export declare function rowPath(path: string): string;
 export declare function emptyListHint(op: TargetingOp, value: unknown): string | undefined;
 export declare function humaniseAttribute(key: string): string;
-export {};

package/dist/admin/targeting/targeting.model.js

@@ -1,5 +1,7 @@
 import { TARGETING_OPS, isOpValid } from '../../targeting/ops.js';
+import { coerceListValue, readListValue } from '../../targeting/values.js';
 import { NUMBER_OP_LABELS, STRING_OP_LABELS, AUDIENCE_COPY } from '../copy/audience.js';
+export { readListValue } from '../../targeting/values.js';
 export function opsFor(type) {
     return type ? TARGETING_OPS[type] : [];
 }
@@ -27,11 +29,7 @@ export function valueInput(attribute, op = 'eq') {
 export function coerceValue(type, op, raw) {
     if (!isOpValid(type, op)) return null;
     if (type === 'string') {
-        if (op === 'in' || op === 'nin') return [
-            ...new Set((Array.isArray(raw) ? raw.map(String) : [
-                String(raw ?? '')
-            ]).map((value)=>value.trim()).filter(Boolean))
-        ];
+        if (op === 'in' || op === 'nin') return coerceListValue(raw);
         return raw == null ? '' : String(raw);
     }
     if (type === 'number') {
@@ -41,9 +39,6 @@ export function coerceValue(type, op, raw) {
     }
     return raw === true || raw === 'true' ? true : raw === false || raw === 'false' ? false : null;
 }
-/** Only the read boundary accepts legacy comma-separated document values. */ export function readListValue(raw) {
-    return coerceValue('string', 'in', typeof raw === 'string' ? raw.split(',') : raw);
-}
 /** Normalize the read boundary before any whole-group edit. */ export function readTargetingRules(rules) {
     return rules.map((rule)=>rule.op === 'in' || rule.op === 'nin' ? {
             ...rule,