@delmaredigital/payload-experiments 0.3.0-rc.1 → 0.3.0-rc.4

package/dist/admin/copy/results.js

@@ -9,7 +9,7 @@ export const RESULTS_COPY = {
         sequential: 'Sequential',
         bayesian: 'Bayesian'
     },
-    asOfDecision: (date)=>`as of the decision, ${date}`,
+    asOfDecision: (date)=>`numbers from ${date}, used for the decision`,
     kinds: {
         binomial: 'yes/no',
         count: 'amount',
@@ -57,6 +57,8 @@ export const RESULTS_COPY = {
     infeasible: 'A clear answer would take more than 90 days at this traffic.',
     noCap: 'Not capped: fewer than 20 people had this outcome in the 90 days before Start, so every value counts in full.',
     quality: 'Data quality',
+    missingWinner: 'The stored winner is missing from this round’s results.',
+    missingWinnerNext: 'Recompute; if it persists, contact a developer before trusting these results.',
     srm: "Traffic isn't splitting as set. Check for caching or a page placed twice.",
     multiGroup: 'People who saw more than one group were excluded.',
     multiGroupNext: 'Check for caching or a page placed twice before trusting the results.',

package/dist/admin/copy/review.d.ts

@@ -1,8 +1,8 @@
 export declare const REVIEW_COPY: {
     readonly noBaseline: "No data yet for this metric.";
     readonly preview: "Preview";
+    readonly previewBlocked: "Your browser blocked the preview window. Allow pop-ups for this site, then try again.";
     readonly placedTimes: (n: number) => string;
-    readonly noPreview: "Preview isn't set up for this page type";
     readonly raise: "Raise allocation";
     readonly addMetric: "Add a metric";
     readonly allocationBody: "Raising allocation adds people; nobody already taking part changes group. Lowering needs a new round.";

package/dist/admin/copy/review.js

@@ -1,8 +1,8 @@
 export const REVIEW_COPY = {
     noBaseline: 'No data yet for this metric.',
     preview: 'Preview',
+    previewBlocked: 'Your browser blocked the preview window. Allow pop-ups for this site, then try again.',
     placedTimes: (n)=>`(placed ${n} times on this page)`,
-    noPreview: "Preview isn't set up for this page type",
     raise: 'Raise allocation',
     addMetric: 'Add a metric',
     allocationBody: 'Raising allocation adds people; nobody already taking part changes group. Lowering needs a new round.',

package/dist/admin/copy/surface.d.ts

@@ -10,7 +10,6 @@ export declare const SURFACE_COPY: {
     readonly dashShowTests: "Show all tests";
     readonly dashShowGoals: "Show all goals";
     readonly backToAdmin: "Back to admin";
-    readonly limitedList: "Showing the 200 most recent tests.";
     readonly notInstalled: "The Tests screen isn't installed yet. Run `payload generate:importmap` and reload.";
     readonly noAccess: "You don't have access to tests.";
     readonly loading: "Loading tests…";

package/dist/admin/copy/surface.js

@@ -10,7 +10,6 @@ export const SURFACE_COPY = {
     dashShowTests: 'Show all tests',
     dashShowGoals: 'Show all goals',
     backToAdmin: 'Back to admin',
-    limitedList: 'Showing the 200 most recent tests.',
     notInstalled: "The Tests screen isn't installed yet. Run `payload generate:importmap` and reload.",
     noAccess: "You don't have access to tests.",
     loading: 'Loading tests…',

package/dist/admin/paths.d.ts

@@ -4,6 +4,7 @@ export declare const XP_ARM_SELECT = "@delmaredigital/payload-experiments/admin/
 export declare const XP_TESTS_VIEW = "@delmaredigital/payload-experiments/admin/rsc#ExperimentsRootView";
 export declare const XP_TESTS_LIST_REDIRECT = "@delmaredigital/payload-experiments/admin/rsc#CollectionListRedirect";
 export declare const XP_TEST_EDIT_REDIRECT = "@delmaredigital/payload-experiments/admin/rsc#CollectionEditRedirect";
-export declare const XP_TESTS_NAV_LINK = "@delmaredigital/payload-experiments/admin/client#TestsNavLink";
+export declare const XP_TESTS_NAV_LINK = "@delmaredigital/payload-experiments/admin/rsc#TestsNavLinkServer";
+export declare const XP_TESTS_NAV_CLIENT = "@delmaredigital/payload-experiments/admin/client#TestsNavLink";
 export declare const XP_TESTS_SURFACE = "@delmaredigital/payload-experiments/admin/client#TestsSurface";
 export declare const XP_TESTS_DASHBOARD_CARDS = "@delmaredigital/payload-experiments/admin/client#TestsDashboardCards";

package/dist/admin/paths.js

@@ -4,6 +4,7 @@ export const XP_ARM_SELECT = '@delmaredigital/payload-experiments/admin/client#A
 export const XP_TESTS_VIEW = '@delmaredigital/payload-experiments/admin/rsc#ExperimentsRootView';
 export const XP_TESTS_LIST_REDIRECT = '@delmaredigital/payload-experiments/admin/rsc#CollectionListRedirect';
 export const XP_TEST_EDIT_REDIRECT = '@delmaredigital/payload-experiments/admin/rsc#CollectionEditRedirect';
-export const XP_TESTS_NAV_LINK = '@delmaredigital/payload-experiments/admin/client#TestsNavLink';
+export const XP_TESTS_NAV_LINK = '@delmaredigital/payload-experiments/admin/rsc#TestsNavLinkServer';
+export const XP_TESTS_NAV_CLIENT = '@delmaredigital/payload-experiments/admin/client#TestsNavLink';
 export const XP_TESTS_SURFACE = '@delmaredigital/payload-experiments/admin/client#TestsSurface';
 export const XP_TESTS_DASHBOARD_CARDS = '@delmaredigital/payload-experiments/admin/client#TestsDashboardCards';

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

@@ -6,4 +6,6 @@ type PreviewPage = Placement & {
 };
 /** Preview URLs identify pages; counts use the most placements in any one version. */
 export declare function previewPages(placements: readonly Placement[], flagPoint?: string): PreviewPage[];
+/** Pages that offer something to do: a preview or an editor link. The section is hidden when none do. */
+export declare function actionablePreviewPages(placements: readonly Placement[], flagPoint?: string): PreviewPage[];
 export {};

package/dist/admin/previewLinks.model.js

@@ -30,3 +30,6 @@
         ...pages.values()
     ].map(({ page })=>page);
 }
+/** Pages that offer something to do: a preview or an editor link. The section is hidden when none do. */ export function actionablePreviewPages(placements, flagPoint) {
+    return previewPages(placements, flagPoint).filter((page)=>page.url || page.editorUrl);
+}

package/dist/admin/results/legacy.d.ts

@@ -0,0 +1,16 @@
+import type { PhaseSummary } from '../../types/endpoints.js';
+import type { ResultsContext } from './results.model.js';
+import type { LegacySnapshot } from './snapshot.js';
+/** Stands in for CORE-15a winnerArm, borrowing the live decision only for the newest rollout. */
+export declare function legacyRolloutLine(snapshot: LegacySnapshot, phase: PhaseSummary, phases: PhaseSummary[], decision?: {
+    winnerArm?: string;
+}): string;
+/** Stands in for CORE-15a minExposuresPerArm; absent historical minima keep the old dash. */
+export declare function legacyMinimum(snapshot: LegacySnapshot, ctx: ResultsContext): number | undefined;
+/** Stands in for CORE-15a minExposuresPerArm by deriving the old notice from the frozen verdict. */
+export declare function legacyTooFew(snapshot: LegacySnapshot): boolean;
+/** Stands in for CORE-15a winnerArm in rollout recommendations using the old recommended arm and label. */
+export declare function legacyRecommendationWinner(snapshot: LegacySnapshot, phase?: PhaseSummary): {
+    key: string | null;
+    label: string;
+};

package/dist/admin/results/legacy.js

@@ -0,0 +1,27 @@
+import { RESULTS_COPY, shippingTitle } from '../copy/results.js';
+import { armLabel } from '../labels.js';
+/** Stands in for CORE-15a winnerArm, borrowing the live decision only for the newest rollout. */ export function legacyRolloutLine(snapshot, phase, phases, decision) {
+    const newestRollout = phases.filter((phase)=>phase.phaseKind === 'rollout').reduce((latest, phase)=>latest === null || phase.phase > latest ? phase.phase : latest, null);
+    const winner = snapshot.winnerArm ?? (snapshot.phase === newestRollout ? decision?.winnerArm : undefined);
+    return shippingTitle(winner ? phase.arms.find((arm)=>arm.key === winner)?.label ?? armLabel(winner) : RESULTS_COPY.shippedGroup);
+}
+/** Stands in for CORE-15a minExposuresPerArm; absent historical minima keep the old dash. */ export function legacyMinimum(snapshot, ctx) {
+    return snapshot.minExposuresPerArm ?? ctx.minExposuresPerArm;
+}
+/** Stands in for CORE-15a minExposuresPerArm by deriving the old notice from the frozen verdict. */ export function legacyTooFew(snapshot) {
+    return snapshot.verdict.code === 'collecting' || snapshot.verdict.code === 'rollout-collecting';
+}
+/** Stands in for CORE-15a winnerArm in rollout recommendations using the old recommended arm and label. */ export function legacyRecommendationWinner(snapshot, phase) {
+    const key = snapshot.winnerArm ?? snapshot.verdict.recommendedArm;
+    const stored = phase?.arms.find((arm)=>arm.key === key)?.label.trim();
+    return {
+        key,
+        label: stored || (key && [
+            'control',
+            'variantA',
+            'variantB',
+            'variantC',
+            'holdout'
+        ].includes(key) ? armLabel(key) : RESULTS_COPY.shippedGroup)
+    };
+}

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

@@ -1,13 +1,13 @@
 import type { DocId, Status } from '../../types/contracts.js';
 import type { PhaseSummary, ResultsResponse } from '../../types/endpoints.js';
 import type { Snapshot } from '../../types/snapshot.js';
+import { type DisplaySnapshot } from './snapshot.js';
 type Metric = Snapshot['metrics'][number];
 type Arm = Metric['perArm'][number];
 type Decision = {
     phase: number;
     winnerArm?: string;
 };
-type DisplaySnapshot = Snapshot;
 export declare function defaultRound({ phases, decision, status }: Pick<ResultsResponse, 'phases'> & {
     decision?: Decision | null;
     status: Status;
@@ -26,8 +26,8 @@ export type VerdictCardModel = {
     guardrailLine?: string;
 };
 export declare function verdictCard({ snapshot, decisionSnapshot, isDecidedTestRound, winnerLabel }: {
-    snapshot: Snapshot;
-    decisionSnapshot?: Snapshot;
+    snapshot: DisplaySnapshot;
+    decisionSnapshot?: DisplaySnapshot;
     isDecidedTestRound: boolean;
     winnerLabel: string;
 }): VerdictCardModel;
@@ -73,11 +73,11 @@ export interface ArmRow {
     comparison?: ComparisonModel;
 }
 export declare function armRows({ snapshot, phase, metricId }: {
-    snapshot: Snapshot;
+    snapshot: DisplaySnapshot;
     phase: PhaseSummary;
     metricId: DocId;
 }): ArmRow[];
-export declare function guardrailChips(snapshot: Snapshot): {
+export declare function guardrailChips(snapshot: DisplaySnapshot): {
     name: string;
     status: "OK" | "Watch" | "Failing" | "Too little data";
 }[];
@@ -105,5 +105,5 @@ export interface DataQualityItem {
     shares?: string[];
 }
 export declare function dataQuality(snapshot: DisplaySnapshot, ctx?: ResultsContext): DataQualityItem[];
-export declare function verdictLabel(snapshot: Snapshot): string;
+export declare function verdictLabel(snapshot: DisplaySnapshot): string;
 export {};

package/dist/admin/results/results.model.js

@@ -2,6 +2,8 @@ import { READINESS_COPY, versionPageUnavailableCopy, versionPageUncheckedCopy }
 import { armLabel, phaseKindLabel, verdictCodeLabel } from '../labels.js';
 import { RESULTS_COPY, cappedCount, collectingCopy, conflictCopy, decidedTitle, exclusionCopy, ignoredCopy, metricUnavailable, moreDays, roundLabel, shareCopy, shippingTitle, srmP, uncappedTotal } from '../copy/results.js';
 import { formatCI, formatCount, formatMetric, formatPercent, formatProbability } from './format.js';
+import { legacyMinimum, legacyRolloutLine, legacyTooFew } from './legacy.js';
+import { isLegacySnapshot } from './snapshot.js';
 export function defaultRound({ phases, decision, status }) {
     if ((status === 'decided' || status === 'archived') && decision) return decision.phase;
     return phases.reduce((latest, phase)=>latest === null || phase.phase > latest ? phase.phase : latest, null);
@@ -29,9 +31,10 @@ export function verdictCard({ snapshot, decisionSnapshot, isDecidedTestRound, wi
 }
 export function rolloutLine(snapshot, phase, phases, decision) {
     if (snapshot.phaseKind !== 'rollout') return null;
-    const newestRollout = phases.filter((phase)=>phase.phaseKind === 'rollout').reduce((latest, phase)=>latest === null || phase.phase > latest ? phase.phase : latest, null);
-    const winner = snapshot.winnerArm ?? (snapshot.phase === newestRollout ? decision?.winnerArm : undefined);
-    return shippingTitle(winner ? phase.arms.find((arm)=>arm.key === winner)?.label ?? armLabel(winner) : RESULTS_COPY.shippedGroup);
+    return isLegacySnapshot(snapshot) ? legacyRolloutLine(snapshot, phase, phases, decision) : currentRolloutLine(snapshot, phase);
+}
+function currentRolloutLine(snapshot, phase) {
+    return snapshot.winnerArm === null ? null : shippingTitle(phase.arms.find((arm)=>arm.key === snapshot.winnerArm)?.label ?? armLabel(snapshot.winnerArm));
 }
 export function cappedCell({ metric, arm, caps }) {
     const cap = caps?.[metric.metricId];
@@ -125,7 +128,7 @@ function smallest(values) {
     if (phaseKind !== 'rollout' || snapshot.verdict.code !== 'rollout-collecting') return null;
     const keys = comparedKeys(snapshot, ctx.phase);
     const primary = snapshot.metrics.find((metric)=>metric.role === 'primary');
-    return collectingCopy(formatCount(smallest(keys.map((key)=>snapshot.exposures.perArm[key]))), formatCount(snapshot.minExposuresPerArm ?? ctx.minExposuresPerArm), formatCount(smallest(keys.map((key)=>primary?.perArm.find((arm)=>arm.arm === key)?.converters))), formatCount(primary ? snapshot.params.minSampleSizes[primary.metricId] : undefined));
+    return collectingCopy(formatCount(smallest(keys.map((key)=>snapshot.exposures.perArm[key]))), formatCount(isLegacySnapshot(snapshot) ? legacyMinimum(snapshot, ctx) : snapshot.minExposuresPerArm), formatCount(smallest(keys.map((key)=>primary?.perArm.find((arm)=>arm.arm === key)?.converters))), formatCount(primary ? snapshot.params.minSampleSizes[primary.metricId] : undefined));
 }
 export function dataQuality(snapshot, ctx = {}) {
     const items = (snapshot.health ?? []).map((item)=>({
@@ -133,6 +136,12 @@ export function dataQuality(snapshot, ctx = {}) {
             message: READINESS_COPY.needsAttention,
             nextStep: item.code === 'version-page-unavailable' ? versionPageUnavailableCopy(item.path, ctx.status) : versionPageUncheckedCopy(item.path, ctx.status)
         }));
+    // Test snapshots store null by design; only rollout snapshots record the shipped winner.
+    if (!isLegacySnapshot(snapshot) && snapshot.phaseKind === 'rollout' && snapshot.winnerArm === null) items.push({
+        id: 'missing-winner',
+        message: RESULTS_COPY.missingWinner,
+        nextStep: RESULTS_COPY.missingWinnerNext
+    });
     const srmTotal = Object.keys(snapshot.srm.expected).reduce((total, key)=>total + (snapshot.exposures.perArm[key] ?? 0), 0);
     if (snapshot.srm.flagged) items.push({
         id: 'srm',
@@ -159,7 +168,7 @@ export function dataQuality(snapshot, ctx = {}) {
         detail: conflictCopy(formatPercent(exposures.identityConflictShare)),
         nextStep: RESULTS_COPY.identityNext
     });
-    if (snapshot.verdict.code === 'collecting' || snapshot.verdict.code === 'rollout-collecting') items.push({
+    if (isLegacySnapshot(snapshot) ? legacyTooFew(snapshot) : snapshot.verdict.code === 'collecting' || snapshot.verdict.code === 'rollout-collecting') items.push({
         id: 'too-few',
         message: RESULTS_COPY.tooFew,
         nextStep: RESULTS_COPY.tooFewNext

package/dist/admin/results/snapshot.d.ts

@@ -0,0 +1,14 @@
+import type { Snapshot } from '../../types/snapshot.js';
+type Core15aFields = Pick<Snapshot, 'winnerArm' | 'minExposuresPerArm'>;
+/** Immutable history written before CORE-15a, with at least one added field absent. */
+export type LegacySnapshot = Omit<Snapshot, keyof Core15aFields> & ({
+    winnerArm?: undefined;
+    minExposuresPerArm?: number;
+} | {
+    winnerArm?: string | null;
+    minExposuresPerArm?: undefined;
+});
+export type DisplaySnapshot = Snapshot | LegacySnapshot;
+/** A stored null winner is current data; only missing CORE-15a fields mark history. */
+export declare function isLegacySnapshot(snapshot: DisplaySnapshot): snapshot is LegacySnapshot;
+export {};

package/dist/admin/results/snapshot.js

@@ -0,0 +1,3 @@
+/** A stored null winner is current data; only missing CORE-15a fields mark history. */ export function isLegacySnapshot(snapshot) {
+    return snapshot.winnerArm === undefined || snapshot.minExposuresPerArm === undefined;
+}

package/dist/admin/surface/TestsHome.js

@@ -3,6 +3,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import { useEffect, useMemo, useState } from 'react';
 import { Link } from '@payloadcms/ui';
 import { formatAdminURL } from 'payload/shared';
+import { fetchAllDocs } from '../../lib/fetchAllDocs.js';
 import { createApi } from '../api.js';
 import { noMatchesLine, SURFACE_COPY } from '../copy/surface.js';
 import { Banner, EmptyState, StatusChip } from '../../ui/Feedback.js';
@@ -19,7 +20,8 @@ const HOME_FIELDS = [
     'decidedAt',
     'decision',
     'unitType',
-    'flagPoint'
+    'flagPoint',
+    'updatedAt'
 ];
 // Typecheck fails if HomeTest gains a field the list request does not select (Payload always returns id).
 const homeFieldsComplete = true;
@@ -42,19 +44,18 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
         });
         async function load() {
             try {
-                const response = await fetch(`${apiRoute.replace(/\/$/, '')}/xp-experiments?depth=0&sort=-updatedAt&limit=200&${HOME_SELECT}`, {
+                const response = await fetchAllDocs(`${apiRoute.replace(/\/$/, '')}/xp-experiments?depth=0&sort=id&${HOME_SELECT}`, {
                     credentials: 'include',
                     signal: controller.signal
                 });
                 if (!response.ok) throw new Error('list');
-                const { docs, totalDocs } = await response.json();
+                const docs = response.docs.sort((a, b)=>(Date.parse(b.updatedAt ?? '') || 0) - (Date.parse(a.updatedAt ?? '') || 0));
                 if (!active) return;
                 const summary = await api.summary(docs.map((doc)=>doc.id));
                 if (active) setState({
                     loading: false,
                     error: false,
                     tests: docs,
-                    limited: totalDocs > docs.length,
                     summaries: summary.ok ? summary.data ?? {} : {},
                     summaryError: !summary.ok
                 });
@@ -156,11 +157,7 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
                     flagPoints: flagPoints,
                     search: search,
                     clear: ()=>setSearch('')
-                }),
-                state.limited ? /*#__PURE__*/ _jsx("p", {
-                    className: "xp-tests-limited",
-                    children: SURFACE_COPY.limitedList
-                }) : null
+                })
             ]
         })
     });

package/dist/admin/surface/goals/data.js

@@ -1,3 +1,4 @@
+import { DocsResponseError, fetchAllDocs } from '../../../lib/fetchAllDocs.js';
 import { refusalMessage } from '../../copy/common.js';
 import { GOALS_COPY as C } from '../../copy/goals.js';
 import { payloadFieldErrors } from '../testDocument.model.js';
@@ -9,24 +10,23 @@ export class GoalResponseError extends Error {
 }
 /** Map REST refusals while retaining Payload's field validation details. */ export async function goalResponse(response, fallback, options) {
     const body = await response.json().catch(()=>undefined);
-    if (!response.ok) {
-        const fields = payloadFieldErrors(body, '_section', fallback);
-        const first = body?.errors?.[0];
-        throw new GoalResponseError(refusalMessage(first?.code, response.status, Object.values(fields).join(' ') || fallback, options), fields);
-    }
+    if (!response.ok) goalRefusal(body, response.status, fallback, options);
     if (body == null) throw new Error(fallback);
     return body;
 }
+function goalRefusal(body, status, fallback, options) {
+    const fields = payloadFieldErrors(body, '_section', fallback);
+    const first = body?.errors?.[0];
+    throw new GoalResponseError(refusalMessage(first?.code, status, Object.values(fields).join(' ') || fallback, options), fields);
+}
 export async function goalPages(base, collection, signal) {
-    const all = [];
-    for(let page = 1;; page++){
-        const response = await fetch(`${base}/${collection}?depth=0&limit=100&page=${page}`, {
-            credentials: 'include',
-            signal
-        });
-        const body = await goalResponse(response, C.loadError);
-        if (!Array.isArray(body.docs)) throw new Error(C.loadError);
-        all.push(...body.docs);
-        if (!body.hasNextPage) return all;
-    }
+    const result = await fetchAllDocs(`${base}/${collection}?depth=0`, {
+        credentials: 'include',
+        signal
+    }).catch((cause)=>{
+        if (cause instanceof DocsResponseError) throw new Error(C.loadError);
+        throw cause;
+    });
+    if (!result.ok) goalRefusal(result.body, result.status, C.loadError);
+    return result.docs;
 }

package/dist/admin/surface/home.model.d.ts

@@ -7,6 +7,7 @@ export interface HomeTest {
     kind: ExperimentKind;
     status: Status;
     unitType: UnitType;
+    updatedAt?: string | null;
     flagPoint?: string | null;
     analysis?: {
         plannedDurationDays?: number | null;

package/dist/admin/surface/newTest/NewTest.js

@@ -1,5 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { fetchAllDocs } from '../../../lib/fetchAllDocs.js';
 import { useEffect, useRef, useState } from 'react';
 import { useRouter } from 'next/navigation.js';
 import { formatAdminURL } from 'payload/shared';
@@ -28,14 +29,13 @@ export function NewTest({ surface, pages }) {
         setLayerState('loading');
         void (async ()=>{
             try {
-                const response = await fetch(`${surface.apiRoute.replace(/\/$/, '')}/xp-layers?depth=0&limit=100`, {
+                const response = await fetchAllDocs(`${surface.apiRoute.replace(/\/$/, '')}/xp-layers?depth=0`, {
                     credentials: 'include',
                     signal: controller.signal
                 });
-                const body = await response.json();
-                if (!response.ok || !Array.isArray(body?.docs)) throw new Error('groups');
+                if (!response.ok) throw new Error('groups');
                 if (!controller.signal.aborted) {
-                    setLayers(body.docs);
+                    setLayers(response.docs);
                     setLayerState('ready');
                 }
             } catch  {

package/dist/admin/surface/readiness/ActionDialogs.js

@@ -1,5 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { DocsResponseError, fetchAllDocs } from '../../../lib/fetchAllDocs.js';
 import { useEffect, useState } from 'react';
 import { Button } from '../../../ui/Button.js';
 import { TextInput } from '../../../ui/Inputs.js';
@@ -192,16 +193,14 @@ import { useTransition } from './useTransition.js';
         const controller = new AbortController();
         void (async ()=>{
             try {
-                const response = await fetch(`${surface.apiRoute.replace(/\/$/, '')}/xp-metrics?depth=0&limit=100`, {
+                const response = await fetchAllDocs(`${surface.apiRoute.replace(/\/$/, '')}/xp-metrics?depth=0`, {
                     credentials: 'include',
                     signal: controller.signal
                 });
                 if (!response.ok) throw new Error(refusalMessage(undefined, response.status));
-                const data = await response.json();
-                if (!Array.isArray(data.docs)) throw new Error(API_MESSAGES.unreadable);
-                if (!controller.signal.aborted) setMetrics(data.docs);
+                if (!controller.signal.aborted) setMetrics(response.docs);
             } catch (cause) {
-                if (!controller.signal.aborted) setLoadError(cause instanceof Error ? cause.message : API_MESSAGES.network);
+                if (!controller.signal.aborted) setLoadError(cause instanceof DocsResponseError ? API_MESSAGES.unreadable : cause instanceof Error ? cause.message : API_MESSAGES.network);
             } finally{
                 if (!controller.signal.aborted) setLoading(false);
             }

package/dist/admin/surface/readiness/readiness.model.js

@@ -1,6 +1,6 @@
 import { formatDate } from '../../format/date.js';
 import { actionGroupLabel } from '../../actions/actions.model.js';
-import { unsupportedContentCopy, versionPageUnavailableCopy, versionPageUncheckedCopy, placementFailureCopy, unpublishedPageCopy, READINESS_COPY as C, attributeNext, contentNext, goalPeopleNext, parameterNext, placementNext, ruleNext, versionsNext } from '../../copy/readiness.js';
+import { unsupportedContentCopy, versionPageUnavailableCopy, versionPageUncheckedCopy, placementFailureCopy, pageInUseCopy, unpublishedPageCopy, READINESS_COPY as C, attributeNext, contentNext, goalPeopleNext, parameterNext, placementNext, ruleNext, versionsNext } from '../../copy/readiness.js';
 import { SECTION_ORDER } from '../contracts.js';
 const object = (value)=>value && typeof value === 'object' && !Array.isArray(value) ? value : {};
 const first = (value)=>object(Array.isArray(value) ? value[0] : value);
@@ -49,6 +49,13 @@ const versions = row('versions', (detail)=>versionsNext(detailText(detail, C.ver
     'no-reach-per-day': row('length', ()=>C.noReach),
     'layer-full': row('versions', ()=>C.full),
     'layer-unit-mismatch': row('header', ()=>C.differentPeople),
+    'page-in-use': {
+        ...row('versions', (detail)=>{
+            const paths = object(detail).paths;
+            return pageInUseCopy(text(Array.isArray(paths) ? paths[0] : undefined, 'this page'));
+        }),
+        nextStep: C.chooseDifferentPage
+    },
     'page-no-targets': row('versions', ()=>C.noTargets),
     'page-target-unpublished': row('versions', (detail)=>unpublishedPageCopy(text(detail, 'This page'))),
     'page-targets-unchecked': row('versions', ()=>C.uncheckedPages),

package/dist/admin/surface/results/ResultsBody.js

@@ -12,6 +12,7 @@ import { METRICS_COPY } from '../../copy/metrics.js';
 import { API_MESSAGES } from '../../copy/common.js';
 import { armLabel } from '../../labels.js';
 import { PreviewLinks } from '../../PreviewLinks.js';
+import { actionablePreviewPages } from '../../previewLinks.model.js';
 import { VerdictHero } from '../../results/VerdictHero.js';
 import { StatTile } from '../../results/StatTile.js';
 import { DataTable, Swatch } from '../../results/DataTable.js';
@@ -329,7 +330,7 @@ function ResultsContent({ id, surface, onRecomputed, onPagesReleased, renderPage
                     }) : null
                 ]
             }),
-            manage && placements && placements.length > 0 ? /*#__PURE__*/ _jsxs("section", {
+            manage && placements && actionablePreviewPages(placements, doc?.flagPoint).length > 0 ? /*#__PURE__*/ _jsxs("section", {
                 className: "xp-results-block",
                 children: [
                     /*#__PURE__*/ _jsx("div", {

package/dist/admin/surface/results/recommendation.model.d.ts

@@ -1,8 +1,9 @@
 import type { PhaseSummary } from '../../../types/endpoints.js';
 import type { Status } from '../../../types/contracts.js';
-import type { Snapshot, VerdictCode } from '../../../types/snapshot.js';
+import type { VerdictCode } from '../../../types/snapshot.js';
+import { type DisplaySnapshot } from '../../results/snapshot.js';
 export interface RecommendationContext {
-    snapshot: Snapshot;
+    snapshot: DisplaySnapshot;
     phase?: PhaseSummary;
     plannedDurationDays?: number | null;
     status?: Status;

package/dist/admin/surface/results/recommendation.model.js

@@ -1,6 +1,9 @@
 import { formatDate } from '../../format/date.js';
 import { armLabel } from '../../labels.js';
 import { formatCount, formatPercent } from '../../results/format.js';
+import { RESULTS_COPY } from '../../copy/results.js';
+import { legacyRecommendationWinner } from '../../results/legacy.js';
+import { isLegacySnapshot } from '../../results/snapshot.js';
 import { RECOMMENDATION_COPY as C, collectingRecommendation, guardrailRecommendation, mixedRecommendation, rolloutCollectingRecommendation, rolloutReadoutRecommendation, runningRecommendation, winnerRecommendation, worseRecommendation, storedGuardrailFinding } from '../../copy/recommendation.js';
 function label(ctx, key, fallback = C.group) {
     const stored = ctx.phase?.arms.find((arm)=>arm.key === key)?.label.trim();
@@ -14,7 +17,10 @@ function label(ctx, key, fallback = C.group) {
 }
 const primary = ({ snapshot })=>snapshot.metrics.find((metric)=>metric.role === 'primary');
 const worse = (ctx)=>primary(ctx)?.comparisons.find((comparison)=>comparison.significant === 'worse' && ctx.phase?.arms.some((arm)=>arm.key === comparison.arm && arm.kind === 'variant'))?.arm;
-const winner = (ctx)=>label(ctx, ctx.snapshot.winnerArm ?? ctx.snapshot.verdict.recommendedArm, C.winner);
+const rolloutWinner = (ctx)=>isLegacySnapshot(ctx.snapshot) ? legacyRecommendationWinner(ctx.snapshot, ctx.phase) : {
+        key: ctx.snapshot.winnerArm,
+        label: label(ctx, ctx.snapshot.winnerArm)
+    };
 /** Select stored findings and format them; never run inference in the admin. */ export const RECOMMENDATION = {
     unavailable: ()=>C.unavailable,
     'data-problem': ()=>C.dataProblem,
@@ -46,10 +52,11 @@ const winner = (ctx)=>label(ctx, ctx.snapshot.winnerArm ?? ctx.snapshot.verdict.
         return Number.isFinite(end) ? runningRecommendation(formatDate(end)) : C.keepRunning;
     },
     'no-difference': ()=>C.noDifference,
-    'rollout-collecting': (ctx)=>rolloutCollectingRecommendation(winner(ctx)),
-    'rollout-readout': (ctx)=>rolloutReadoutRecommendation(winner(ctx), formatPercent(primary(ctx)?.comparisons.find((comparison)=>comparison.arm === (ctx.snapshot.winnerArm ?? ctx.snapshot.verdict.recommendedArm))?.lift, true))
+    'rollout-collecting': (ctx)=>rolloutCollectingRecommendation(rolloutWinner(ctx).label),
+    'rollout-readout': (ctx)=>rolloutReadoutRecommendation(rolloutWinner(ctx).label, formatPercent(primary(ctx)?.comparisons.find((comparison)=>comparison.arm === rolloutWinner(ctx).key)?.lift, true))
 };
 export const recommendation = (ctx)=>{
+    if (!isLegacySnapshot(ctx.snapshot) && ctx.snapshot.phaseKind === 'rollout' && ctx.snapshot.winnerArm === null) return RESULTS_COPY.missingWinner;
     const sentence = RECOMMENDATION[ctx.snapshot.verdict.code];
     return sentence(ctx);
 };

package/dist/admin/surface/surface.css

@@ -43,7 +43,6 @@
 .xp-tests-group-chevron { align-self: center; color: var(--xp-text-3); }
 .xp-tests-group[open] .xp-tests-group-chevron { transform: rotate(90deg); }
 .xp-tests-group > summary + .xp-tests-list { margin-top: 10px; }
-.xp-tests-limited { margin-top: var(--xp-space-4); color: var(--xp-text-2); font: var(--xp-text-small); }
 .xp-tests-no-matches { display: flex; flex-wrap: wrap; align-items: center; gap: var(--xp-space-2); }
 .xp-tests-no-matches p { margin: 0; }
 @media (max-width: 720px) {

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

@@ -1,5 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { fetchAllDocs } from '../../../lib/fetchAllDocs.js';
 import { Button } from '../../../ui/Button.js';
 import { useEffect, useState } from 'react';
 import { formatAdminURL } from 'payload/shared';
@@ -19,22 +20,13 @@ export function GoalSection({ test, surface }) {
             try {
                 await Promise.resolve();
                 if (controller.signal.aborted) return;
-                const all = [];
-                let page = 1;
-                for(;;){
-                    const response = await fetch(`${surface.apiRoute.replace(/\/$/, '')}/xp-metrics?depth=0&limit=100&page=${page}`, {
-                        credentials: 'include',
-                        signal: controller.signal
-                    });
-                    if (!response.ok) throw new Error();
-                    const body = await response.json();
-                    if (!Array.isArray(body.docs)) throw new Error();
-                    all.push(...body.docs);
-                    if (!body.hasNextPage) break;
-                    page++;
-                }
+                const response = await fetchAllDocs(`${surface.apiRoute.replace(/\/$/, '')}/xp-metrics?depth=0`, {
+                    credentials: 'include',
+                    signal: controller.signal
+                });
+                if (!response.ok) throw new Error();
                 if (!controller.signal.aborted) {
-                    setMetrics(all);
+                    setMetrics(response.docs);
                     setFailed(false);
                     setLoaded(true);
                 }

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

@@ -215,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,