@delmaredigital/payload-experiments 0.2.0-rc.6 → 0.3.0-rc.1

package/dist/transitions/releasePages.js

@@ -0,0 +1,26 @@
+import { relationshipId } from './checkStart.js';
+import { withTransition } from './kernel.js';
+import { readExperiment, refuse, updateExperiment } from './start.js';
+export async function releasePages(req, { experimentId }) {
+    const before = await readExperiment(req, experimentId);
+    const { result: doc, revision } = await withTransition(req, {
+        experimentId,
+        layerId: relationshipId(before.layer)
+    }, async ({ req })=>{
+        const current = await readExperiment(req, experimentId);
+        const decision = current.decision;
+        if (current.kind !== 'page' || current.status !== 'archived' || !decision?.winnerArm || decision.discardedAt || decision.pagesReleasedAt) refuse('invalid-status');
+        const at = new Date().toISOString();
+        return updateExperiment(req, experimentId, {
+            decision: {
+                ...decision,
+                pagesReleasedAt: at,
+                pagesReleasedBy: req.user ? String(req.user.id) : null
+            }
+        });
+    });
+    return {
+        doc,
+        revision: revision
+    };
+}

package/dist/transitions/restoreContentEntry.d.ts

@@ -0,0 +1,8 @@
+import type { PayloadRequest } from 'payload';
+import type { DocId } from '../types/contracts.js';
+import type { RestoreContentEntryBody } from '../types/endpoints.js';
+export declare function restoreContentEntry(req: PayloadRequest, { experimentId, collection, id }: RestoreContentEntryBody & {
+    experimentId: DocId;
+}): Promise<{
+    doc: import("payload").JsonObject & import("payload").TypeWithID;
+}>;

package/dist/transitions/restoreContentEntry.js

@@ -0,0 +1,47 @@
+import { relationshipId } from './checkStart.js';
+import { withTransition } from './kernel.js';
+import { readExperiment, refuse, updateExperiment } from './start.js';
+export async function restoreContentEntry(req, { experimentId, collection, id }) {
+    const before = await readExperiment(req, experimentId);
+    // Only the audit report changes; assignment and the manifest stay the same.
+    const { result: doc } = await withTransition(req, {
+        experimentId,
+        layerId: relationshipId(before.layer),
+        flagPointId: before.kind === 'flag' ? before.flagPoint : undefined,
+        bump: false
+    }, async ({ req })=>{
+        const current = await readExperiment(req, experimentId);
+        const decision = current.decision;
+        const report = decision?.contentReport;
+        const index = Array.isArray(report?.docs) ? report.docs.findIndex((entry)=>entry.collection === collection && String(entry.id) === String(id)) : -1;
+        const entry = report?.docs?.[index];
+        if (current.status !== 'decided' || !decision?.winnerArm || decision.discardedAt) refuse('not-decided');
+        if (!entry) refuse('entry-not-found');
+        if (!entry.dismissedAt) refuse('entry-not-dismissed');
+        const docs = report.docs.map((entry, i)=>{
+            if (i !== index) return entry;
+            const { dismissedAt: _at, dismissedBy: _by, ...restored } = entry;
+            return {
+                ...restored,
+                ok: false
+            };
+        });
+        return updateExperiment(req, experimentId, {
+            decision: {
+                ...decision,
+                contentReport: {
+                    ...report,
+                    docs
+                }
+            }
+        });
+    });
+    req.payload.logger.info({
+        experimentId,
+        collection,
+        id
+    }, 'Experiment content entry restored');
+    return {
+        doc
+    };
+}

package/dist/transitions/start.js

@@ -1,4 +1,7 @@
+import { lockMetricsForFreeze } from '../db/metricUsage.js';
+import { DEFAULT_SEQUENTIAL_TUNING_PARAMETER } from '../constants.js';
 import { claimSlices } from '../db/slices.js';
+import { unpublishedPageCopy } from '../admin/copy/readiness.js';
 import { XpRefusal } from '../errors.js';
 import { isOpValid } from '../targeting/ops.js';
 import { newSeed } from '../runtime/seed.js';
@@ -27,21 +30,31 @@ export async function readExperiment(req, id) {
 }
 export function refuse(code, detail) {
     const messages = {
+        'page-target-unpublished': unpublishedPageCopy(typeof detail === 'string' ? detail : 'This page'),
         'invalid-status': 'This action is not available in the current status.',
         'stale-draft': 'The experiment changed. Review it again before starting.',
         'flag-point-busy': 'Another experiment is running or decided on this flag point.',
         'layer-full': 'There is not enough room in this layer.',
         'allocation-lower': 'Allocation can only increase while running. Stop and start a new round to lower it.',
-        'invalid-body': 'Check the experiment settings and try again.'
+        'invalid-body': 'Check the experiment settings and try again.',
+        'not-decided': 'This test has no standing winner.',
+        'entry-not-found': 'That page is not in the report.',
+        'entry-ok': 'That page was already updated.',
+        'entry-dismissed': 'That page was already dismissed.',
+        'entry-not-dismissed': 'That page is not dismissed.'
     };
     const conflict = [
         'invalid-status',
         'stale-draft',
         'flag-point-busy',
         'layer-full',
-        'allocation-lower'
+        'allocation-lower',
+        'not-decided',
+        'entry-ok',
+        'entry-dismissed',
+        'entry-not-dismissed'
     ].includes(code);
-    throw new XpRefusal(code, messages[code] ?? 'The experiment is not ready to start.', detail, conflict ? 409 : 422);
+    throw new XpRefusal(code, messages[code] ?? 'The experiment is not ready to start.', detail, code === 'entry-not-found' ? 404 : conflict ? 409 : 422);
 }
 export async function updateExperiment(req, id, data) {
     // §7.8: access-authorised transition writes locked plugin fields.
@@ -78,8 +91,10 @@ export async function start(req, { experimentId, options }) {
         kind: full.kind,
         analysis: full.analysis
     };
+    // Warning-only codes always have ok: true. These exclusions narrow
+    // ReadinessCode to RefusalCode, which intentionally excludes warnings.
     const failed = checkStart(input).find((check)=>!check.ok);
-    if (failed) refuse(failed.code, failed.detail);
+    if (failed && failed.code !== 'page-targets-unchecked' && failed.code !== 'make-content-unsupported') refuse(failed.code, failed.detail);
     const before = input.experiment;
     const updatedAt = before.updatedAt;
     const at = new Date();
@@ -134,8 +149,17 @@ export async function start(req, { experimentId, options }) {
                 ...input,
                 experiment: current
             }).find((check)=>!check.ok);
-            if (recheck) refuse(recheck.code, recheck.detail);
+            if (recheck && recheck.code !== 'page-targets-unchecked' && recheck.code !== 'make-content-unsupported') refuse(recheck.code, recheck.detail);
         }
+        const lockedMetrics = await lockMetricsForFreeze(exec, caps.definitions.map((metric)=>metric.id));
+        if (!lockedMetrics) {
+            throw new XpRefusal('stale-draft', 'A goal in this test was deleted. Review the goals before starting.', undefined, 409);
+        }
+        // Reachable: a slug/source change takes FOR NO KEY UPDATE, which a draft's FK check does not wait on.
+        if (caps.definitions.some((metric)=>{
+            const locked = lockedMetrics.find((row)=>String(row.id) === String(metric.id));
+            return locked.slug !== metric.slug || locked.source !== metric.source;
+        })) throw new XpRefusal('stale-draft', 'A goal changed. Review it again before starting.', undefined, 409);
         const targeting = (current.targeting ?? []).map((rule)=>{
             const type = Object.hasOwn(options.attributeKeys ?? {}, rule.attribute) ? options.attributeKeys[rule.attribute].type : undefined;
             const validValue = type === 'string' && [
@@ -183,6 +207,7 @@ export async function start(req, { experimentId, options }) {
                 ...current.analysis,
                 method: current.analysis?.method ?? 'sequential',
                 alpha: 0.05,
+                sequentialTuningParameter: current.analysis?.sequentialTuningParameter ?? DEFAULT_SEQUENTIAL_TUNING_PARAMETER,
                 cupedLookbackDays: 14,
                 layerSlices,
                 minSampleSizes: Object.fromEntries(definitions.map((metric)=>[

package/dist/types/contracts.d.ts

@@ -105,6 +105,9 @@ export interface ExperimentsPluginOptions<TUser extends {
         readResults?: ExperimentsAccess;
         readIdentities?: ExperimentsAccess;
     };
+    theme?: {
+        accent?: string;
+    };
     admin?: {
         editorUrl?: (args: {
             collection: string;
@@ -131,6 +134,17 @@ export interface ExperimentsPluginOptions<TUser extends {
         type: 'string' | 'number' | 'boolean';
         values?: string[];
     }>;
+    pages?: {
+        collections: Array<{
+            slug: string;
+            pathOf: (doc: Record<string, unknown>) => string | null;
+            /** Published-only lookup using the supplied req and overrideAccess: false. Results checks supply an anonymous req. */
+            findByPath: (path: string, req: PayloadRequest) => Promise<{
+                id: string | number;
+                published: boolean;
+            } | null>;
+        }>;
+    };
     puck?: {
         enabled: boolean;
         collections: Array<string | {

package/dist/types/endpoints.d.ts

@@ -1,10 +1,13 @@
-import type { ArmKind, DocId, ExposurePolicy, MetricSource, ResolvedOptions } from './contracts.js';
+import type { ArmKind, DocId, ExperimentKind, ExposurePolicy, MetricSource, ResolvedOptions, Status } from './contracts.js';
+import type { ContentReportReason } from './reasons.js';
 import type { ArmSlot } from './puck.js';
 import type { ReadinessCode } from '../transitions/checkStart.js';
 export type { ReadinessCode } from '../transitions/checkStart.js';
 import type { Snapshot, VerdictCode } from './snapshot.js';
 export type ArmIssueCode = 'weights-total' | 'control-count' | 'control-first' | 'holdout-last' | 'holdout-count' | 'variant-count' | 'duplicate-key' | 'duplicate-slot' | 'slot-required' | 'invalid-key';
-export type RefusalCode = ArmIssueCode | 'not-found' | 'no-planned-duration' | 'no-primary-metric' | 'no-placements' | 'empty-slot' | 'page-no-targets' | 'page-user-unit' | 'flag-no-point' | 'flag-params-invalid' | 'flag-point-busy' | 'flag-placement-missing' | 'layer-full' | 'layer-unit-mismatch' | 'unknown-metric-source' | 'unit-source-mismatch' | 'no-reach-per-day' | 'invalid-status' | 'stale-draft' | 'commit-unverified' | 'allocation-lower' | 'confirmation-required' | 'invalid-winner' | 'slices-taken' | 'locked-placement' | 'not-implemented' | 'forbidden' | 'unauthenticated' | 'body-too-large' | 'invalid-body' | 'too-many-ids' | 'invalid-phase' | 'invalid-ids' | 'recompute-failed' | 'analysis-failed' | 'undeclared-targeting-key' | 'invalid-targeting-rule' | 'duplicate-metric';
+/** Content-report dismiss/restore refusals (CORE-44). */
+export type ContentEntryRefusal = 'not-decided' | 'entry-not-found' | 'entry-ok' | 'entry-dismissed' | 'entry-not-dismissed';
+export type RefusalCode = ArmIssueCode | 'not-found' | ContentEntryRefusal | 'no-planned-duration' | 'no-primary-metric' | 'no-placements' | 'empty-slot' | 'page-no-targets' | 'page-target-unpublished' | 'page-user-unit' | 'version-page-unavailable' | 'version-page-unchecked' | 'flag-no-point' | 'flag-params-invalid' | 'flag-point-busy' | 'flag-placement-missing' | 'layer-full' | 'layer-unit-mismatch' | 'unknown-metric-source' | 'unit-source-mismatch' | 'no-reach-per-day' | 'invalid-status' | 'stale-draft' | 'stale-edit' | 'commit-unverified' | 'allocation-lower' | 'confirmation-required' | 'invalid-winner' | 'slices-taken' | 'locked-placement' | 'not-implemented' | 'forbidden' | 'unauthenticated' | 'body-too-large' | 'invalid-body' | 'too-many-ids' | 'invalid-phase' | 'invalid-ids' | 'recompute-failed' | 'analysis-failed' | 'undeclared-targeting-key' | 'invalid-targeting-rule' | 'block-path-rule' | 'duplicate-metric';
 export type ErrorEnvelope = {
     errors: Array<{
         code: RefusalCode;
@@ -32,12 +35,25 @@ export type ArchiveBody = {
     mode: 'stopped' | 'after-content' | 'discard' | 'hard-coded';
     confirmation?: string;
 };
+export type DismissContentEntryBody = {
+    collection: string;
+    id: DocId;
+};
+export type DismissContentEntryResult = Pick<TransitionSuccess, 'doc'>;
+export type RestoreContentEntryBody = DismissContentEntryBody;
+export type RestoreContentEntryResult = Pick<TransitionSuccess, 'doc'>;
+export type ContentReport = {
+    docs: MakeContentResult['docs'];
+} | MakeContentResult['docs'];
 export type MakeContentResult = TransitionSuccess & {
     docs: Array<{
         collection: string;
         id: DocId;
         ok: boolean;
-        error?: string;
+        detail?: ContentReportReason | string;
+        error?: ContentReportReason | string;
+        dismissedAt?: string;
+        dismissedBy?: string | null;
     }>;
 };
 export type PreflightResult = {
@@ -45,6 +61,7 @@ export type PreflightResult = {
         code: ReadinessCode;
         ok: boolean;
         detail?: unknown;
+        warning?: boolean;
     }>;
     placements: Array<{
         collection: string;
@@ -93,6 +110,12 @@ export interface PhaseSummary {
     }>;
 }
 export type ResultsResponse = {
+    /** Filled by the results endpoint from CORE-48: what a results reader may see about the test. */
+    experiment?: {
+        name: string;
+        kind: ExperimentKind;
+        status: Status;
+    };
     phases: PhaseSummary[];
     snapshot: Snapshot | null;
     decisionSnapshot?: Snapshot;
@@ -110,6 +133,14 @@ export type ResultsResponse = {
 };
 export type SummaryResponse = Record<DocId, {
     ready?: boolean;
+    next?: {
+        code: ReadinessCode;
+        detail?: unknown;
+    };
+    health?: Array<{
+        code: 'version-page-unavailable' | 'version-page-unchecked';
+        path: string;
+    }>;
     exposures: number;
     days: number;
     verdict: {

package/dist/types/index.d.ts

@@ -6,3 +6,4 @@ export * from './puck.js';
 export * from './admin.js';
 export * from './endpoints.js';
 export * from './snapshot.js';
+export * from './reasons.js';

package/dist/types/index.js

@@ -6,3 +6,4 @@ export * from './puck.js';
 export * from './admin.js';
 export * from './endpoints.js';
 export * from './snapshot.js';
+export * from './reasons.js';

package/dist/types/reasons.d.ts

@@ -0,0 +1,3 @@
+/** Closed codes written by new results runs; stored snapshots may predate them. */
+export type MetricUnavailableReason = 'source-failed' | 'source-unavailable' | 'unit-type-unavailable' | 'non-finite-values' | 'non-finite-covariates' | 'budget' | 'not-supplied' | 'analysis-failed';
+export type ContentReportReason = 'localized' | 'locked' | 'forbidden' | 'not-found' | 'validation' | 'update-failed';

package/dist/types/reasons.js

@@ -0,0 +1 @@
+/** Closed codes written by new results runs; stored snapshots may predate them. */ export { };

package/dist/types/snapshot.d.ts

@@ -1,5 +1,6 @@
 import type { GaussianPrior } from '@delmaredigital/experiments-stats';
 import type { DocId, MetricSource } from './contracts.js';
+import type { MetricUnavailableReason } from './reasons.js';
 export type VerdictCode = 'unavailable' | 'data-problem' | 'guardrail-failing' | 'collecting' | 'original-better' | 'worse-none-better' | 'winner' | 'mixed' | 'keep-running' | 'no-difference' | 'rollout-collecting' | 'rollout-readout';
 export interface Snapshot {
     experiment: DocId;
@@ -54,7 +55,7 @@ export interface Snapshot {
         kind: MetricSource['kind'];
         role: 'primary' | 'secondary' | 'guardrail';
         cupedApplied: boolean;
-        unavailable?: string;
+        unavailable?: MetricUnavailableReason | string;
         perArm: Array<{
             arm: string;
             n: number;
@@ -107,6 +108,10 @@ export interface Snapshot {
         guardrailLine?: string;
         worstArm?: string;
     };
+    health?: Array<{
+        code: 'version-page-unavailable' | 'version-page-unchecked';
+        path: string;
+    }>;
     contentChanges: Array<{
         at: string;
         doc: {

package/dist/ui/Banner.d.ts

@@ -0,0 +1,2 @@
+import './ui.css';
+export { Banner } from './Feedback.js';

package/dist/ui/Banner.js

@@ -0,0 +1,2 @@
+import './ui.css';
+export { Banner } from './Feedback.js';

package/dist/ui/Breadcrumb.d.ts

@@ -0,0 +1,7 @@
+import './ui.css';
+export declare function Breadcrumb({ items }: {
+    items: Array<{
+        label: string;
+        href?: string;
+    }>;
+}): import("react").JSX.Element;

package/dist/ui/Breadcrumb.js

@@ -0,0 +1,25 @@
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import './ui.css';
+export function Breadcrumb({ items }) {
+    return /*#__PURE__*/ _jsx("nav", {
+        className: "xp-ui-breadcrumb",
+        "aria-label": "Breadcrumb",
+        children: /*#__PURE__*/ _jsx("ol", {
+            children: items.map((item, index)=>/*#__PURE__*/ _jsxs("li", {
+                    children: [
+                        index ? /*#__PURE__*/ _jsx("span", {
+                            "aria-hidden": "true",
+                            children: "›"
+                        }) : null,
+                        index === items.length - 1 ? /*#__PURE__*/ _jsx("span", {
+                            "aria-current": "page",
+                            children: item.label
+                        }) : /*#__PURE__*/ _jsx("a", {
+                            href: item.href,
+                            children: item.label
+                        })
+                    ]
+                }, index))
+        })
+    });
+}

package/dist/ui/Button.d.ts

@@ -0,0 +1,15 @@
+import type { AnchorHTMLAttributes, ButtonHTMLAttributes } from 'react';
+import './ui.css';
+type Shared = {
+    variant?: 'primary' | 'secondary' | 'quiet' | 'link' | 'danger';
+    size?: 'md' | 'sm';
+    disabled?: boolean;
+    busy?: boolean;
+};
+export type ButtonProps = Shared & (ButtonHTMLAttributes<HTMLButtonElement> & {
+    href?: never;
+} | AnchorHTMLAttributes<HTMLAnchorElement> & {
+    href: string;
+});
+export declare function Button({ variant, size, disabled, busy, className, children, ...props }: ButtonProps): import("react").JSX.Element;
+export {};

package/dist/ui/Button.js

@@ -0,0 +1,32 @@
+'use client';
+import { jsx as _jsx } from "react/jsx-runtime";
+import './ui.css';
+export function Button({ variant = 'secondary', size = 'md', disabled, busy, className = '', children, ...props }) {
+    const classes = `xp-ui-button xp-ui-button--${variant} xp-ui-button--${size} ${className}`;
+    const blocked = disabled || busy;
+    if (props.href !== undefined) {
+        const { href, onClick, ...rest } = props;
+        return /*#__PURE__*/ _jsx("a", {
+            ...rest,
+            className: classes,
+            href: blocked ? undefined : href,
+            role: blocked ? 'link' : undefined,
+            "aria-disabled": blocked || undefined,
+            "aria-busy": busy || undefined,
+            tabIndex: blocked ? -1 : rest.tabIndex,
+            onClick: (event)=>{
+                if (blocked) event.preventDefault();
+                else onClick?.(event);
+            },
+            children: children
+        });
+    }
+    return /*#__PURE__*/ _jsx("button", {
+        ...props,
+        type: props.type ?? 'button',
+        className: classes,
+        disabled: blocked,
+        "aria-busy": busy || undefined,
+        children: children
+    });
+}

package/dist/ui/Card.d.ts

@@ -0,0 +1,12 @@
+import type { HTMLAttributes, ReactNode } from 'react';
+import './ui.css';
+export declare function Card({ className, ...props }: HTMLAttributes<HTMLDivElement>): import("react").JSX.Element;
+export declare function SectionCard({ number, title, complete, children, id, headingId, className }: {
+    number: number;
+    title: string;
+    complete?: boolean;
+    children: ReactNode;
+    id?: string;
+    headingId?: string;
+    className?: string;
+}): import("react").JSX.Element;

package/dist/ui/Card.js

@@ -0,0 +1,40 @@
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import './ui.css';
+export function Card({ className = '', ...props }) {
+    return /*#__PURE__*/ _jsx("div", {
+        ...props,
+        className: `xp-ui-card ${className}`
+    });
+}
+export function SectionCard({ number, title, complete = false, children, id, headingId, className = '' }) {
+    return /*#__PURE__*/ _jsxs("section", {
+        id: id,
+        "aria-labelledby": headingId,
+        className: `xp-ui-card xp-ui-section ${className}`,
+        children: [
+            /*#__PURE__*/ _jsxs("header", {
+                className: "xp-ui-section-head",
+                children: [
+                    /*#__PURE__*/ _jsx("span", {
+                        className: "xp-ui-section-number",
+                        children: number
+                    }),
+                    /*#__PURE__*/ _jsx("h2", {
+                        id: headingId,
+                        tabIndex: -1,
+                        children: title
+                    }),
+                    /*#__PURE__*/ _jsx("span", {
+                        className: "xp-ui-section-complete",
+                        "aria-label": complete ? 'Complete' : 'Incomplete',
+                        children: complete ? '✓' : '○'
+                    })
+                ]
+            }),
+            /*#__PURE__*/ _jsx("div", {
+                className: "xp-ui-section-body",
+                children: children
+            })
+        ]
+    });
+}

package/dist/ui/Checkbox.d.ts

@@ -0,0 +1,2 @@
+import './ui.css';
+export { Checkbox, CheckboxList } from './Inputs.js';

package/dist/ui/Checkbox.js

@@ -0,0 +1,2 @@
+import './ui.css';
+export { Checkbox, CheckboxList } from './Inputs.js';

package/dist/{admin/components → ui}/ChipsInput.d.ts

@@ -1,12 +1,13 @@
-import '../theme.css';
-import './chipsInput.css';
+import './ui.css';
 export interface ChipsInputProps {
     value: string[];
     onChange: (value: string[]) => void;
     label: string;
     description?: string;
+    help?: string;
+    error?: string;
     placeholder?: string;
     disabled?: boolean;
     id: string;
 }
-export declare function ChipsInput({ value, onChange, label, description, placeholder, disabled, id }: ChipsInputProps): import("react").JSX.Element;
+export declare function ChipsInput({ value, onChange, label, description, help, error, placeholder, disabled, id }: ChipsInputProps): import("react").JSX.Element;

package/dist/{admin/components → ui}/ChipsInput.js

@@ -1,10 +1,8 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useRef, useState } from 'react';
-import { AUDIENCE_COPY } from '../copy/audience.js';
-import '../theme.css';
-import './chipsInput.css';
-export function ChipsInput({ value, onChange, label, description, placeholder, disabled = false, id }) {
+import './ui.css';
+export function ChipsInput({ value, onChange, label, description, help = description, error, placeholder, disabled = false, id }) {
     const [draft, setDraft] = useState('');
     const input = useRef(null);
     const commit = (text)=>{
@@ -24,7 +22,7 @@ export function ChipsInput({ value, onChange, label, description, placeholder, d
         input.current?.focus();
     };
     return /*#__PURE__*/ _jsxs("div", {
-        className: "xp-chips",
+        className: "xp-ui-chips",
         role: "group",
         "aria-labelledby": `${id}-label`,
         "aria-disabled": disabled,
@@ -35,10 +33,10 @@ export function ChipsInput({ value, onChange, label, description, placeholder, d
                 children: label
             }),
             /*#__PURE__*/ _jsxs("div", {
-                className: "xp-chips-control",
+                className: `xp-ui-chips-control${error ? ' xp-ui-chips-control--invalid' : ''}`,
                 children: [
                     value.map((item, index)=>/*#__PURE__*/ _jsxs("span", {
-                            className: "xp-chips-item",
+                            className: "xp-ui-chips-item",
                             children: [
                                 /*#__PURE__*/ _jsx("span", {
                                     children: item
@@ -46,9 +44,9 @@ export function ChipsInput({ value, onChange, label, description, placeholder, d
                                 /*#__PURE__*/ _jsx("button", {
                                     type: "button",
                                     disabled: disabled,
-                                    "aria-label": AUDIENCE_COPY.removeValue(item),
+                                    "aria-label": `Remove ${item}`,
                                     onClick: ()=>remove(index),
-                                    children: AUDIENCE_COPY.removeSymbol
+                                    children: '×'
                                 })
                             ]
                         }, item)),
@@ -59,7 +57,11 @@ export function ChipsInput({ value, onChange, label, description, placeholder, d
                         value: draft,
                         disabled: disabled,
                         placeholder: placeholder,
-                        "aria-describedby": description ? `${id}-description` : undefined,
+                        "aria-invalid": Boolean(error) || undefined,
+                        "aria-describedby": [
+                            help ? `${id}-description` : '',
+                            error ? `${id}-error` : ''
+                        ].filter(Boolean).join(' ') || undefined,
                         onChange: (event)=>setDraft(event.target.value),
                         onBlur: ()=>{
                             if (draft.trim()) commit(draft);
@@ -87,9 +89,15 @@ export function ChipsInput({ value, onChange, label, description, placeholder, d
                     })
                 ]
             }),
-            description ? /*#__PURE__*/ _jsx("p", {
+            help ? /*#__PURE__*/ _jsx("p", {
                 id: `${id}-description`,
-                children: description
+                children: help
+            }) : null,
+            error ? /*#__PURE__*/ _jsx("p", {
+                className: "xp-ui-error",
+                id: `${id}-error`,
+                role: "alert",
+                children: error
             }) : null
         ]
     });

package/dist/ui/Dialog.d.ts

@@ -0,0 +1,13 @@
+import type { ReactNode } from 'react';
+import './ui.css';
+/** Native modal stays in its rendering document, including a Puck canvas iframe.
+ * `initialFocus="cancel"` targets the focusable control marked `data-dialog-cancel`.
+ */
+export declare function Dialog({ open, title, onClose, children, dismissible, initialFocus }: {
+    open: boolean;
+    dismissible?: boolean;
+    initialFocus?: 'cancel';
+    title: string;
+    onClose: () => void;
+    children: ReactNode;
+}): import("react").JSX.Element;