@delmaredigital/payload-experiments 0.4.0-rc.1 → 0.4.0-rc.2

package/README.md

@@ -1,6 +1,6 @@
 # @delmaredigital/payload-experiments
 
-> ⚠️ Upgrading to 0.4? Run `payload migrate:create` for four nullable plugin columns (`xp_experiments.page_collection` and its `_xp_experiments_v` counterpart, `xp_phases.placement_docs`, `xp_state.retired_slugs`), plus the Experiment block and `xpGoal` columns if you adopt them, then `payload generate:importmap`. Saves on pages, blocks and Puck collections can now be refused while a test runs on them (`page-busy`, `interacting-tests`); Puck tests whose slot is only on a draft can't start until the page is published; host `metricSources` ids may not start with `click:`. See the CHANGELOG's BREAKING section.
+> ⚠️ Upgrading to 0.4? Run `payload migrate:create` for four nullable plugin columns (`xp_experiments.page_collection` and its `_xp_experiments_v` counterpart, `xp_phases.placement_docs`, `xp_state.retired_slugs`), plus the Experiment block and `xpGoal` columns if you adopt them, then `payload generate:importmap`. Saves on pages, blocks and Puck collections can now be refused while a test runs on them (`page-busy`, `interacting-tests`); Puck tests whose slot is only on a draft can't start until the page is published; host `metricSources` ids may not start with `click:`; `admin.editorUrl` now receives a required `editor` argument. See the CHANGELOG's BREAKING section.
 
 > ⚠️ Upgrading to 0.3? For 0.3.0-rc.1, every `linkIdentity` call must supply `verified`; it now returns an outcome. Run `payload migrate:create` for the nullable `xp_identities.verified_at` column and `user_id` index, plus the release-pages columns listed in the CHANGELOG (`decision_pages_released_at` / `decision_pages_released_by` on `xp_experiments` and their `version_` counterparts on `_xp_experiments_v`). Existing identity links become unverified; see the CHANGELOG for optional backfill.
 
@@ -620,13 +620,15 @@ for these patterns.
 
 ## Admin and client
 
-The sidebar includes a **Tests** group with **All tests** and **Goals**. The
-dashboard also includes **Tests** and **Goals** cards above its widgets. Goals
-appears only for users who can manage tests. Both additions follow the host's
-admin route and keep host-provided navigation and dashboard entries first.
-After upgrading, run `payload generate:importmap` to register the dashboard card
-component and the server nav wrapper.
-A host nav group also labelled "Tests" shares Payload's NavGroup DOM id and open-state preference with the plugin's group, so pick another label.
+**Tests** is a regular navigation group for managers only, with **All tests**
+and **Goals**. It is the first named group, after Payload's default
+Collections/Globals groups when the host has ungrouped entities. Payload's own
+dashboard shows the Tests cards. A host collection with `group: 'Tests'` joins
+the plugin's group.
+
+After upgrading, run `payload generate:importmap` to register the two list views
+and the Goals edit redirect. Without regeneration, both list URLs and the Goal
+create/edit URLs show Payload's raw views.
 
 ### Client assignments and exposure
 

package/dist/admin/TestsDashboardCards.d.ts

@@ -1,3 +1,4 @@
 import './theme.css';
 import './dashboard.css';
+/** @deprecated Retained for existing import maps until 0.5.0. */
 export declare function TestsDashboardCards(): import("react").JSX.Element | null;

package/dist/admin/TestsDashboardCards.js

@@ -1,20 +1,16 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { surfaceUrls } from './surfaceUrls.js';
 import './theme.css';
 import './dashboard.css';
 import { Card, useAuth, useConfig } from '@payloadcms/ui';
-import { formatAdminURL } from 'payload/shared';
 import { SURFACE_COPY } from './copy/surface.js';
-export function TestsDashboardCards() {
+/** @deprecated Retained for existing import maps until 0.5.0. */ export function TestsDashboardCards() {
     const { config } = useConfig();
     const { permissions } = useAuth();
     const testPermissions = permissions?.collections?.['xp-experiments'];
     const canManage = testPermissions?.update === true;
     if (!canManage) return null;
-    const href = (path)=>formatAdminURL({
-            adminRoute: config.routes.admin,
-            path
-        });
     return /*#__PURE__*/ _jsxs("section", {
         className: "xp-dash-group",
         "aria-labelledby": "xp-dash-title",
@@ -32,7 +28,7 @@ export function TestsDashboardCards() {
                             title: SURFACE_COPY.navAllTests,
                             titleAs: "h3",
                             buttonAriaLabel: SURFACE_COPY.dashShowTests,
-                            href: href('/experiments'),
+                            href: surfaceUrls.home(config.routes.admin),
                             id: "card-xp-tests"
                         })
                     }),
@@ -41,7 +37,7 @@ export function TestsDashboardCards() {
                             title: SURFACE_COPY.navGoals,
                             titleAs: "h3",
                             buttonAriaLabel: SURFACE_COPY.dashShowGoals,
-                            href: href('/experiments/goals'),
+                            href: surfaceUrls.goals(config.routes.admin),
                             id: "card-xp-goals"
                         })
                     })

package/dist/admin/TestsNavLink.d.ts

@@ -1,4 +1,5 @@
 import './theme.css';
+/** @deprecated Retained for existing import maps until 0.5.0. */
 declare function TestsNav({ isOpen }: {
     isOpen?: boolean;
 }): import("react").JSX.Element | null;

package/dist/admin/TestsNavLink.js

@@ -1,22 +1,16 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { surfaceUrls } from './surfaceUrls.js';
 import './theme.css';
 import { Link, NavGroup, useAuth, useConfig } from '@payloadcms/ui';
-import { formatAdminURL } from 'payload/shared';
 import { usePathname } from 'next/navigation.js';
 import { SURFACE_COPY } from './copy/surface.js';
-function TestsNav({ isOpen = true }) {
+/** @deprecated Retained for existing import maps until 0.5.0. */ function TestsNav({ isOpen = true }) {
     const { config } = useConfig();
     const { permissions } = useAuth();
     const pathname = usePathname();
-    const href = formatAdminURL({
-        adminRoute: config.routes.admin,
-        path: '/experiments'
-    });
-    const goalsHref = formatAdminURL({
-        adminRoute: config.routes.admin,
-        path: '/experiments/goals'
-    });
+    const href = surfaceUrls.home(config.routes.admin);
+    const goalsHref = surfaceUrls.goals(config.routes.admin);
     const base = href.replace(/\/$/, '');
     const goalsBase = goalsHref.replace(/\/$/, '');
     const goalsActive = pathname === goalsBase || Boolean(pathname?.startsWith(`${goalsBase}/`));

package/dist/admin/api.d.ts

@@ -13,6 +13,13 @@ export type ApiResult<T> = {
 };
 type RequestOptions = NonNullable<Parameters<typeof refusalMessage>[3]>;
 export declare function createApi(apiBase: string): {
+    findTestIdBySlug: (slug: string) => Promise<Extract<ApiResult<DocId>, {
+        ok: false;
+    }> | {
+        ok: true;
+        data: DocId;
+    }>;
+    deleteTest: (id: DocId) => Promise<ApiResult<unknown>>;
     wrapBlock: (body: WrapBlockBody) => Promise<ApiResult<Record<string, unknown>>>;
     pages: (query: {
         collection: string;

package/dist/admin/api.js

@@ -1,6 +1,8 @@
 import { API_MESSAGES, refusalMessage } from './copy/common.js';
+import { BLOCK_COPY } from './copy/blocks.js';
 import { CONTENT_ENTRY_REFUSALS } from './copy/actions.js';
 export function createApi(apiBase) {
+    const collectionBase = `${apiBase.replace(/\/$/, '')}/xp-experiments`;
     const base = `${apiBase.replace(/\/$/, '')}/xp/experiments`;
     const route = (id, action)=>`${base}/${encodeURIComponent(id)}/${action}`;
     async function request(path, method, body, options, signal) {
@@ -64,6 +66,31 @@ export function createApi(apiBase) {
     }
     const post = (id, action, body, options)=>request(()=>route(id, action), 'POST', body, options);
     return {
+        findTestIdBySlug: async (slug)=>{
+            const result = await request(()=>`${collectionBase}?where[slug][equals]=${encodeURIComponent(slug)}&depth=0&limit=1`, 'GET');
+            if (!result.ok) return result;
+            const id = result.data?.docs?.[0]?.id;
+            if (id === undefined) return {
+                ok: false,
+                status: 404,
+                code: 'not-found',
+                message: refusalMessage('not-found', 404, undefined, {
+                    notFound: BLOCK_COPY.alreadyDeleted
+                }),
+                retryable: false
+            };
+            return {
+                ok: true,
+                data: id
+            };
+        },
+        deleteTest: (id)=>request(()=>`${collectionBase}/${encodeURIComponent(id)}`, 'DELETE', undefined, {
+                notFound: BLOCK_COPY.noLongerDraft,
+                forbidden: BLOCK_COPY.noLongerDraft,
+                codes: {
+                    'invalid-status': BLOCK_COPY.noLongerDraft
+                }
+            }),
         wrapBlock: (body)=>request(()=>`${apiBase.replace(/\/$/, '')}/xp/blocks/wrap`, 'POST', body),
         pages: (query, signal)=>request(()=>`${apiBase.replace(/\/$/, '')}/xp/pages?${new URLSearchParams({
                     collection: query.collection,

package/dist/admin/blocks/BlockPanel.js

@@ -1,20 +1,28 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { Banner, Button, ConfirmationModal, useModal, useFormInitializing, useRowLabel } from '@payloadcms/ui';
+import { Banner, Button, ConfirmationModal, toast, useModal, useFormInitializing, useRowLabel } from '@payloadcms/ui';
+import { formatAdminURL } from 'payload/shared';
 import { useEffect, useId, useRef, useState } from 'react';
 import { copyOriginalIntoVersion, syncExperimentSlots, unwrapExperimentBlock } from '../../blocks/formState.js';
-import { BLOCK_COPY, blockSyncNotice } from '../copy/blocks.js';
+import { BLOCK_COPY, blockSyncNotice, blockUnwrapNotice, blockDeleteConfirmation, blockUnwrapConfirmation, blockRunningUnwrap, blockDecidedUnwrap, blockFinishedCopy } from '../copy/blocks.js';
 import { API_MESSAGES } from '../copy/common.js';
 import { isOrphan, liveStatus, slotLabel, statusSentence } from './model.js';
-import { useBlockStatus } from './statusCache.js';
+import { createApi } from '../api.js';
+import { useBlockStatus, setBlockStatus } from './statusCache.js';
 import { useBlockForm } from './useBlockForm.js';
+import { BlockChangedError, RefusalError } from './formApi.js';
+function errorMessage(error) {
+    return error instanceof BlockChangedError || error instanceof RefusalError ? error.message : API_MESSAGES.failed;
+}
 export function BlockStatusField({ path }) {
     const context = useBlockForm(path), { block, field, manager, config, collection, replace } = context;
     const initializing = useFormInitializing();
     const status = useBlockStatus(config.routes.api, block?.experiment, true);
     const synced = useRef(false), syncing = useRef(false);
     const [notice, setNotice] = useState(false), [error, setError] = useState(), [busy, setBusy] = useState(false);
-    const [syncFailed, setSyncFailed] = useState(false);
+    const [syncError, setSyncError] = useState();
+    const { openModal } = useModal();
+    const modalSlug = `xp-unwrap-${useId()}`;
     const canSync = !initializing && manager && block && field && liveStatus(status) && [
         'draft',
         'stopped'
@@ -23,7 +31,7 @@ export function BlockStatusField({ path }) {
         if (!canSync || syncing.current) return;
         syncing.current = true;
         setBusy(true);
-        setSyncFailed(false);
+        setSyncError(undefined);
         try {
             const next = syncExperimentSlots(block, status.arms, field);
             if (JSON.stringify(next.slots) !== JSON.stringify(block.slots)) {
@@ -32,8 +40,8 @@ export function BlockStatusField({ path }) {
                 ]);
                 setNotice(true);
             }
-        } catch  {
-            setSyncFailed(true);
+        } catch (error) {
+            setSyncError(errorMessage(error));
         } finally{
             syncing.current = false;
             setBusy(false);
@@ -44,45 +52,83 @@ export function BlockStatusField({ path }) {
         synced.current = true;
         void sync();
     });
-    if (!status) return null;
+    if (!status || !manager) return null;
+    const draft = liveStatus(status) && status.status === 'draft';
+    const stopped = liveStatus(status) && status.status === 'stopped';
     async function unwrap() {
         if (!block || !field || busy) return;
         setBusy(true);
         setError(undefined);
+        let deleted = false;
         try {
+            if (draft) {
+                const api = createApi(config.routes.api);
+                const found = await api.findTestIdBySlug(block.experiment);
+                if (!found.ok) throw new RefusalError(found.message);
+                const removed = await api.deleteTest(found.data);
+                if (!removed.ok) throw new RefusalError(removed.message);
+                deleted = true;
+                setBlockStatus(config.routes.api, block.experiment, {
+                    deleted: true
+                });
+            }
             await replace(unwrapExperimentBlock(block, field));
-        } catch  {
-            setError(API_MESSAGES.failed);
+            toast.success(blockUnwrapNotice(Boolean(collection?.versions?.drafts), deleted));
+        } catch (error) {
+            setError(deleted ? BLOCK_COPY.finishUnwrap : errorMessage(error));
         } finally{
             setBusy(false);
         }
     }
     const sentence = statusSentence(status);
+    const linkedCopy = liveStatus(status) ? status.status === 'running' ? blockRunningUnwrap : status.status === 'decided' ? blockDecidedUnwrap : status.status === 'archived' && status.winnerArm ? blockFinishedCopy(slotLabel(status.winnerArm, status)) : undefined : undefined;
+    const href = formatAdminURL({
+        adminRoute: config.routes.admin,
+        path: `/experiments/by-slug/${encodeURIComponent(block?.experiment ?? '')}`
+    });
     return /*#__PURE__*/ _jsxs("div", {
         children: [
             liveStatus(status) && status.status === 'running' ? /*#__PURE__*/ _jsx(Banner, {
                 children: sentence
-            }) : /*#__PURE__*/ _jsx("p", {
+            }) : linkedCopy && liveStatus(status) && status.status === 'archived' ? null : /*#__PURE__*/ _jsx("p", {
                 children: sentence
             }),
+            linkedCopy ? /*#__PURE__*/ _jsxs("p", {
+                children: [
+                    linkedCopy.before,
+                    /*#__PURE__*/ _jsx("a", {
+                        href: href,
+                        children: linkedCopy.link
+                    }),
+                    linkedCopy.after
+                ]
+            }) : null,
             notice ? /*#__PURE__*/ _jsx("p", {
                 role: "status",
                 children: blockSyncNotice(Boolean(collection?.versions?.drafts))
             }) : null,
-            manager && isOrphan(status) ? /*#__PURE__*/ _jsx(Button, {
+            isOrphan(status) || draft || stopped ? /*#__PURE__*/ _jsx(Button, {
                 disabled: busy,
-                onClick: ()=>void unwrap(),
-                children: BLOCK_COPY.unwrap
+                onClick: ()=>draft || stopped ? openModal(modalSlug) : void unwrap(),
+                children: draft ? BLOCK_COPY.removeAndUnwrap : BLOCK_COPY.unwrap
+            }) : null,
+            draft || stopped ? /*#__PURE__*/ _jsx(ConfirmationModal, {
+                modalSlug: modalSlug,
+                heading: draft ? BLOCK_COPY.removeAndUnwrap : BLOCK_COPY.unwrap,
+                body: draft ? blockDeleteConfirmation(status.name) : blockUnwrapConfirmation(status.name),
+                confirmLabel: draft ? BLOCK_COPY.deleteAndUnwrap : BLOCK_COPY.unwrap,
+                cancelLabel: BLOCK_COPY.cancel,
+                onConfirm: unwrap
             }) : null,
             error ? /*#__PURE__*/ _jsx("p", {
                 role: "alert",
                 children: error
             }) : null,
-            syncFailed ? /*#__PURE__*/ _jsx("p", {
+            syncError ? /*#__PURE__*/ _jsx("p", {
                 role: "alert",
-                children: API_MESSAGES.failed
+                children: syncError
             }) : null,
-            syncFailed && canSync ? /*#__PURE__*/ _jsx(Button, {
+            syncError && canSync ? /*#__PURE__*/ _jsx(Button, {
                 disabled: busy,
                 onClick: ()=>void sync(),
                 children: BLOCK_COPY.retry
@@ -119,8 +165,8 @@ export function CopyOriginalField({ path }) {
             await replace([
                 copyOriginalIntoVersion(block, slot, field)
             ]);
-        } catch  {
-            setError(API_MESSAGES.failed);
+        } catch (error) {
+            setError(errorMessage(error));
         } finally{
             setBusy(false);
         }

package/dist/admin/blocks/formApi.d.ts

@@ -5,6 +5,9 @@ import { type HostBlock } from '../../blocks/model.js';
 export declare class BlockChangedError extends Error {
     constructor();
 }
+/** An API refusal already translated into editor-facing copy. */
+export declare class RefusalError extends Error {
+}
 export type RowsField = Extract<Field, {
     type: 'blocks';
 }>;

package/dist/admin/blocks/formApi.js

@@ -5,6 +5,8 @@ import { BLOCK_COPY } from '../copy/blocks.js';
         super(BLOCK_COPY.blockChanged);
     }
 }
+/** An API refusal already translated into editor-facing copy. */ export class RefusalError extends Error {
+}
 export const wrapMode = 'form-api';
 /** Client twin: unnamed layout containers do not contribute to a data path. */ export function findClientBlocksField(fields, name) {
     for (const field of fields){
@@ -71,14 +73,18 @@ export function rowLocation(path) {
 export function dataAt(data, path) {
     return path.split('.').reduce((value, key)=>value && typeof value === 'object' ? value[key] : undefined, data);
 }
+/** Stable object keys; array order and scalar values remain significant. */ function stringify(value) {
+    return JSON.stringify(value, (_, item)=>item && typeof item === 'object' && !Array.isArray(item) ? Object.fromEntries(Object.entries(item).sort(([a], [b])=>a < b ? -1 : a > b ? 1 : 0)) : item);
+}
 /** Payload 3.90.2 and 4 beta.6 use the same row APIs. Sync replaces the whole
  * wrapper so add/drop/reorder and the server-rendered component slots stay together.
  * N=0 needs UPDATE: removeFieldRow alone serializes the empty field as number 0.
  */ export async function replaceBlockRow(args) {
     const { form, getFormState, collectionSlug, id, field, index, rows } = args;
     const data = form.getData(), before = hostBlocks(data[field]);
-    if (args.expectedBlock && JSON.stringify(before[index]) !== JSON.stringify(args.expectedBlock)) throw new BlockChangedError();
-    const original = JSON.stringify(data[field]);
+    if (args.expectedBlock && stringify(form.getDataByPath(`${field}.${index}`)) !== stringify(args.expectedBlock)) throw new BlockChangedError();
+    // Compare the same reader on both sides: getData() keeps empty row counts as 0.
+    const original = stringify(form.getDataByPath(field));
     const next = [
         ...before.slice(0, index),
         ...rows,
@@ -100,7 +106,7 @@ export function dataAt(data, path) {
         renderAllFields: true,
         skipValidation: true
     });
-    if (JSON.stringify(form.getDataByPath(field)) !== original) throw new BlockChangedError();
+    if (stringify(form.getDataByPath(field)) !== original) throw new BlockChangedError();
     if (!result.state) throw new Error('Missing server form state');
     const state = result.state;
     function subFieldState(rowIndex) {

package/dist/admin/blocks/statusCache.d.ts

@@ -1,5 +1,7 @@
 import type { BlockStatus } from './model.js';
 export declare const validBlockSlug: (slug: unknown) => slug is string;
+/** Update a mounted owner's snapshot and invalidate an older pending read. */
+export declare function setBlockStatus(api: string, slug: string, value: BlockStatus): void;
 /** Only status fields own reads. StrictMode replays the same mount token; a
  * different owner after the last cleanup starts fresh, even in the same commit.
  * Deferred eviction removes every entry with no live owner after that commit.

package/dist/admin/blocks/statusCache.js

@@ -9,6 +9,13 @@ const keyOf = (api, slug)=>JSON.stringify([
     ]);
 const publish = (key)=>listeners.get(key)?.callbacks.forEach((listener)=>listener());
 export const validBlockSlug = (slug)=>typeof slug === 'string' && slug.length > 0 && slug.length <= 64;
+/** Update a mounted owner's snapshot and invalidate an older pending read. */ export function setBlockStatus(api, slug, value) {
+    const key = keyOf(api, slug), current = cache.get(key);
+    if (!current) return;
+    current.revision++;
+    current.value = value;
+    publish(key);
+}
 /** Only status fields own reads. StrictMode replays the same mount token; a
  * different owner after the last cleanup starts fresh, even in the same commit.
  * Deferred eviction removes every entry with no live owner after that commit.
@@ -47,7 +54,8 @@ export const validBlockSlug = (slug)=>typeof slug === 'string' && slug.length >
             current = {
                 value: null,
                 owners: 0,
-                mounts: new WeakSet()
+                mounts: new WeakSet(),
+                revision: 0
             };
             cache.set(key, current);
         }
@@ -55,12 +63,13 @@ export const validBlockSlug = (slug)=>typeof slug === 'string' && slug.length >
         owned.owners++;
         owned.mounts.add(token.current);
         publish(key);
+        const revision = owned.revision;
         if (!owned.pending) owned.pending = createApi(api).placementStatus({
             slugs: [
                 slug
             ]
         }).then((result)=>{
-            if (cache.get(key) !== owned) return;
+            if (cache.get(key) !== owned || owned.revision !== revision) return;
             owned.value = result.ok ? result.data?.slugs[slug] ?? null : null;
             publish(key);
         });

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

@@ -11,6 +11,11 @@ export declare const BLOCK_COPY: {
     copy: string;
     confirmCopy: string;
     unwrap: string;
+    removeAndUnwrap: string;
+    deleteAndUnwrap: string;
+    alreadyDeleted: string;
+    noLongerDraft: string;
+    finishUnwrap: string;
     deleted: string;
     discarded: string;
     running: string;
@@ -22,6 +27,17 @@ export declare const blockPosition: (label: string, position: number) => string;
 export declare const blockTestName: (title: string, label: string) => string;
 export declare const blockDraftStatus: (name: string) => string;
 export declare const blockDecidedStatus: (winner: string, rollout: boolean) => string;
+export type BlockLinkedCopy = {
+    before: string;
+    link: string;
+    after: string;
+};
+export declare const blockRunningUnwrap: BlockLinkedCopy;
+export declare const blockDecidedUnwrap: BlockLinkedCopy;
+export declare const blockFinishedCopy: (winner: string) => BlockLinkedCopy;
 export declare const blockFinishedStatus: (winner: string) => string;
+export declare const blockDeleteConfirmation: (name: string) => string;
+export declare const blockUnwrapConfirmation: (name: string) => string;
+export declare const blockUnwrapNotice: (drafts: boolean, deleted: boolean) => string;
 export declare const blockSaveNotice: (drafts: boolean) => "Save a draft of this page to add the test." | "Save this page to add the test.";
 export declare const blockSyncNotice: (drafts: boolean) => "Versions were updated to match the test. Save draft to keep them." | "Versions were updated to match the test. Save to keep them.";

package/dist/admin/copy/blocks.js

@@ -11,6 +11,11 @@ export const BLOCK_COPY = {
     copy: 'Copy the Original into this version',
     confirmCopy: 'Replace this version’s content with a copy of the Original? Edits you made to this version will be lost.',
     unwrap: 'Unwrap',
+    removeAndUnwrap: 'Remove the test and unwrap',
+    deleteAndUnwrap: 'Delete and unwrap',
+    alreadyDeleted: 'This test was already deleted. Reload the page.',
+    noLongerDraft: 'This test is no longer a draft. Reload the page.',
+    finishUnwrap: 'The test was removed; use Unwrap to finish.',
     deleted: "This block's test was deleted; the page shows the Original.",
     discarded: 'This test has ended; the page shows the Original.',
     running: "Running: visitors see different versions. Edit a version's tab to change it; the change is noted on the results.",
@@ -33,6 +38,27 @@ export const blockPosition = (label, position)=>`${label} (block ${position})`;
 export const blockTestName = (title, label)=>`${title}: ${label}`;
 export const blockDraftStatus = (name)=>`This block is being tested in the draft test ${name}. Visitors see the Original until it starts.`;
 export const blockDecidedStatus = (winner, rollout)=>`Decided: visitors see ${winner}${rollout ? '; a holdout still sees the Original' : ''}.`;
-export const blockFinishedStatus = (winner)=>`Finished: visitors see ${winner}. Use "Make it the content" on the test to replace this block.`;
+export const blockRunningUnwrap = {
+    before: 'Stop ',
+    link: 'the test',
+    after: ' to unwrap.'
+};
+export const blockDecidedUnwrap = {
+    before: 'To put the winner into this page, use "Make it the content" on ',
+    link: 'the test',
+    after: '.'
+};
+export const blockFinishedCopy = (winner)=>({
+        before: `Finished: visitors see ${winner}. Use "Make it the content" on `,
+        link: 'the test',
+        after: ' to replace this block.'
+    });
+export const blockFinishedStatus = (winner)=>{
+    const copy = blockFinishedCopy(winner);
+    return copy.before + copy.link + copy.after;
+};
+export const blockDeleteConfirmation = (name)=>`Delete the draft test ${name} and put the Original back in its place? Deleting the test can't be undone.`;
+export const blockUnwrapConfirmation = (name)=>`Put the Original back in this block's place? The test ${name} and its results are kept; it can't be started again.`;
+export const blockUnwrapNotice = (drafts, deleted)=>`${deleted ? 'The draft test was deleted and the block unwrapped.' : 'The block was unwrapped.'} ${drafts ? 'Save draft' : 'Save'} to keep it.`;
 export const blockSaveNotice = (drafts)=>drafts ? 'Save a draft of this page to add the test.' : 'Save this page to add the test.';
 export const blockSyncNotice = (drafts)=>drafts ? 'Versions were updated to match the test. Save draft to keep them.' : 'Versions were updated to match the test. Save to keep them.';

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

@@ -22,6 +22,7 @@ export declare const LOCKED_MESSAGES: {
 };
 export declare function refusalMessage(code: string | undefined, status: number, message?: string, options?: {
     notFound?: string;
+    forbidden?: string;
     codes?: Readonly<Record<string, string>>;
     fallback?: string;
 }): string;

package/dist/admin/copy/common.js

@@ -28,6 +28,6 @@ export function refusalMessage(code, status, message, options) {
     if (code === 'not-found' || status === 404) return options?.notFound ?? API_MESSAGES.notFound;
     if (code === 'commit-unverified') return API_MESSAGES.commitUnverified;
     if (code === 'not-implemented' || status === 501) return API_MESSAGES.notImplemented;
-    if (code === 'forbidden' || status === 403) return API_MESSAGES.forbidden;
+    if (code === 'forbidden' || status === 403) return options?.forbidden ?? API_MESSAGES.forbidden;
     return options?.fallback ?? (message || API_MESSAGES.failed);
 }

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

@@ -1,5 +1,4 @@
 export declare const PAGE_COPY: {
-    readonly search: "Search pages by title";
     readonly more: "More";
     readonly loading: "Loading pages…";
     readonly empty: "There are no pages here yet.";
@@ -8,8 +7,6 @@ export declare const PAGE_COPY: {
     readonly draft: "Draft";
     readonly noPath: "This page has no address yet.";
     readonly choose: "Choose a page";
-    readonly change: "Change";
-    readonly close: "Close";
     readonly where: "Where it shows";
     readonly open: "Open";
     readonly preview: "Preview";
@@ -17,10 +14,11 @@ export declare const PAGE_COPY: {
     readonly duplicateStep: "Choose Duplicate from the document's actions. Keep the Original's title on the copy, unless the title is what you're testing.";
     readonly ownAddress: "Give the copy its own address and save it as a draft. Publish it when it's ready; the test can't start until it is.";
     readonly thenChoose: "Then choose the page you made.";
-    readonly noPage: "No page chosen yet.";
 };
 export declare const usedPageCopy: (label: string) => string;
 export declare const usedAddressCopy: (label: string, path: string, collection: string) => string;
 export declare const openOriginalCopy: (collection: string) => string;
 export declare const openCollectionCopy: (collection: string) => string;
 export declare const pageStepCopy: (label: string) => string;
+export declare const pageNotFoundCopy: (path: string, collection: string) => string;
+export declare const pageFieldCopy: (label: string) => string;

package/dist/admin/copy/pages.js

@@ -1,5 +1,4 @@
 export const PAGE_COPY = {
-    search: 'Search pages by title',
     more: 'More',
     loading: 'Loading pages…',
     empty: 'There are no pages here yet.',
@@ -8,19 +7,18 @@ export const PAGE_COPY = {
     draft: 'Draft',
     noPath: 'This page has no address yet.',
     choose: 'Choose a page',
-    change: 'Change',
-    close: 'Close',
     where: 'Where it shows',
     open: 'Open',
     preview: 'Preview',
     contentChecksOff: 'Content checks are off for this page type.',
     duplicateStep: "Choose Duplicate from the document's actions. Keep the Original's title on the copy, unless the title is what you're testing.",
     ownAddress: "Give the copy its own address and save it as a draft. Publish it when it's ready; the test can't start until it is.",
-    thenChoose: 'Then choose the page you made.',
-    noPage: 'No page chosen yet.'
+    thenChoose: 'Then choose the page you made.'
 };
 export const usedPageCopy = (label)=>`This is ${label}.`;
 export const usedAddressCopy = (label, path, collection)=>`Same address as ${label} (${path}). Change this page's address in ${collection}, publish it, then choose it.`;
 export const openOriginalCopy = (collection)=>`Open the Original in ${collection}.`;
 export const openCollectionCopy = (collection)=>`Open ${collection}`;
 export const pageStepCopy = (label)=>`A page for ${label}`;
+export const pageNotFoundCopy = (path, collection)=>`${path} isn't in ${collection}. Choose a page.`;
+export const pageFieldCopy = (label)=>`Page for ${label}`;

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

@@ -25,6 +25,7 @@ export declare const READINESS_COPY: {
     readonly everyone: "Everyone already takes part.";
     readonly participation: "How many people take part (%)";
     readonly noPlacements: "Not on a page yet. Open a page, add an Experiment block and choose this test.";
+    readonly placementRemoved: "The block for this test was removed from its page, so it can't start again. Its results are kept. To test the block again, start a new test from the page.";
     readonly noPoint: "Choose the setting to test.";
     readonly busy: "Another test is already running on this setting. Stop it first.";
     readonly noGoal: "Choose a goal.";

package/dist/admin/copy/readiness.js

@@ -25,6 +25,7 @@ export const READINESS_COPY = {
     everyone: 'Everyone already takes part.',
     participation: 'How many people take part (%)',
     noPlacements: 'Not on a page yet. Open a page, add an Experiment block and choose this test.',
+    placementRemoved: "The block for this test was removed from its page, so it can't start again. Its results are kept. To test the block again, start a new test from the page.",
     noPoint: 'Choose the setting to test.',
     busy: 'Another test is already running on this setting. Stop it first.',
     noGoal: 'Choose a goal.',

package/dist/admin/goals/XpGoalField.d.ts

@@ -1,2 +1,4 @@
 import type { TextFieldClientProps } from 'payload';
+import '../theme.css';
+import './xpGoalField.css';
 export declare function XpGoalField({ path, field, readOnly }: TextFieldClientProps): import("react").JSX.Element;