@byline/admin 3.12.1 → 3.12.2

package/src/forms/form-renderer.tsx

@@ -20,22 +20,25 @@ import type {
 } from '@byline/core'
 import type { DocumentPatch } from '@byline/core/patches'
 import { useTranslation } from '@byline/i18n/react'
-import { Alert, Button, CloseIcon, ComboButton, IconButton, Modal } from '@byline/ui/react'
+import { Alert, Button, ComboButton } from '@byline/ui/react'
 import cx from 'classnames'
 
 import { FieldRenderer } from '../fields/field-renderer'
 import { useBylineFieldServices } from '../fields/field-services-context'
-import { LocalDateTime } from '../fields/local-date-time'
 import { AdminGroup } from '../presentation/group'
 import { AdminRow } from '../presentation/row'
 import { AdminTabs } from '../presentation/tabs'
 import { AvailableLocalesWidget } from './available-locales-widget'
 import { DocumentActions, type DocumentActionsLocaleOption } from './document-actions'
 import { FormProvider, useFieldValue, useFormContext } from './form-context'
+import { NavigationGuardModal, SystemFieldsConfirmModal, UnsavedChangesModal } from './form-modals'
 import styles from './form-renderer.module.css'
+import { FormStatusDisplay } from './form-status-display'
 import { useNavigationGuardAdapter } from './navigation-guard'
 import { PathWidget } from './path-widget'
+import { computeStatusTransitions } from './status-transitions'
 import { executeUploadsWithProgress } from './upload-executor'
+import { useFormLayout } from './use-form-layout'
 import type { UseNavigationGuard } from './navigation-guard'
 
 /** Metadata about a previously published version that is still live. */
@@ -156,166 +159,6 @@ export interface FormRendererProps {
   useNavigationGuard?: UseNavigationGuard
 }
 
-const FormStatusDisplay = ({
-  initialData,
-  workflowStatuses,
-  publishedVersion,
-  onUnpublish,
-}: {
-  initialData?: Record<string, any>
-  workflowStatuses?: WorkflowStatus[]
-  publishedVersion?: PublishedVersionInfo | null
-  onUnpublish?: () => Promise<void>
-}) => {
-  const { t } = useTranslation('byline-admin')
-  const statusCode = initialData?.status
-  const statusLabel = workflowStatuses?.find((s) => s.name === statusCode)?.label ?? statusCode
-  // Single-status workflows (e.g. lookups) have no editorial lifecycle —
-  // suppress the "Status: …" cell since there is nothing meaningful to convey.
-  const showStatusCell = (workflowStatuses?.length ?? 0) > 1
-
-  return (
-    <div className={cx('byline-form-status', styles.status)}>
-      <div className={cx('byline-form-status-meta', styles['status-meta'])}>
-        {showStatusCell && (
-          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
-            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
-              {t('forms.status.label')}
-            </span>
-            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
-              {statusLabel}
-            </span>
-          </div>
-        )}
-
-        {initialData?.updatedAt != null && (
-          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
-            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
-              {t('forms.status.lastModified')}
-            </span>
-            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
-              <LocalDateTime value={initialData.updatedAt} />
-            </span>
-          </div>
-        )}
-
-        {initialData?.createdAt != null && (
-          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
-            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
-              {t('forms.status.created')}
-            </span>
-            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
-              <LocalDateTime value={initialData.createdAt} />
-            </span>
-          </div>
-        )}
-      </div>
-
-      {publishedVersion != null && (
-        <div className={cx('byline-form-status-published', styles['status-published'])}>
-          <span className={cx('byline-form-status-muted', styles['status-muted'])}>
-            {t('forms.status.publishedLive')}{' '}
-            {publishedVersion.updatedAt ? (
-              <span>
-                {t('forms.status.publishedOn', { date: new Date(publishedVersion.updatedAt) })}
-              </span>
-            ) : (
-              ''
-            )}
-          </span>
-          {onUnpublish && (
-            <>
-              {' '}
-              <button
-                type="button"
-                onClick={onUnpublish}
-                className={cx('byline-form-status-unpublish', styles['status-unpublish'])}
-              >
-                {t('common.actions.unpublish')}
-              </button>
-            </>
-          )}
-        </div>
-      )}
-    </div>
-  )
-}
-
-/**
- * Compute the primary and secondary status transitions for the ComboButton.
- * - Primary: the main action (forward step), or the current status itself
- *   when the document has reached the final workflow step (terminal state).
- * - Secondary: other available transitions to show as dropdown options.
- * - isTerminal: true when the document is at the final workflow status —
- *   the primary button renders as a non-actionable indicator and all
- *   back-steps move into the dropdown.
- */
-function computeStatusTransitions(
-  currentStatus: string | undefined,
-  workflowStatuses: WorkflowStatus[] | undefined,
-  nextStatus: WorkflowStatus | undefined
-): {
-  primaryStatus: WorkflowStatus | undefined
-  secondaryStatuses: WorkflowStatus[]
-  isTerminal: boolean
-} {
-  if (!workflowStatuses || workflowStatuses.length === 0 || !currentStatus) {
-    return { primaryStatus: nextStatus, secondaryStatuses: [], isTerminal: false }
-  }
-
-  // Single-status workflows (e.g. SINGLE_STATUS_WORKFLOW for lookups) have
-  // no transitions — short-circuit so the form shows only Close / Save.
-  if (workflowStatuses.length <= 1) {
-    return { primaryStatus: undefined, secondaryStatuses: [], isTerminal: false }
-  }
-
-  const currentIndex = workflowStatuses.findIndex((s) => s.name === currentStatus)
-  if (currentIndex === -1) {
-    return { primaryStatus: nextStatus, secondaryStatuses: [], isTerminal: false }
-  }
-
-  const isAtEnd = currentIndex === workflowStatuses.length - 1
-  const isAtStart = currentIndex === 0
-
-  // Collect all available target statuses
-  const availableTargets: WorkflowStatus[] = []
-
-  // Reset to first (if not at first)
-  if (!isAtStart && workflowStatuses[0]) {
-    availableTargets.push(workflowStatuses[0])
-  }
-
-  // Back one step (if not at start and the previous is not already the first)
-  const prev = workflowStatuses[currentIndex - 1]
-  if (currentIndex > 1 && prev) {
-    availableTargets.push(prev)
-  }
-
-  // Forward one step (if not at end) - this is the nextStatus
-  const next = workflowStatuses[currentIndex + 1]
-  if (!isAtEnd && next) {
-    availableTargets.push(next)
-  }
-
-  if (isAtEnd) {
-    // Terminal state: the primary button is a non-actionable indicator of the
-    // current status; both back-steps (revert to previous / reset to first)
-    // are surfaced in the dropdown.
-    return {
-      primaryStatus: workflowStatuses[currentIndex],
-      secondaryStatuses: availableTargets,
-      isTerminal: true,
-    }
-  }
-
-  // Not at end: primary is the forward step (nextStatus)
-  return {
-    primaryStatus: nextStatus,
-    secondaryStatuses: availableTargets.filter((s) => s.name !== nextStatus?.name),
-    isTerminal: false,
-  }
-}
-
 const FormContent = ({
   mode,
   fields,
@@ -394,80 +237,12 @@ const FormContent = ({
     if (initialLocale) setContentLocale(initialLocale)
   }, [initialLocale])
 
-  // ---------------------------------------------------------------------
-  // Layout primitives + lookup tables.
-  //
-  // Built once per render from `adminConfig`. The validator at startup
-  // guarantees every reachable name resolves and every schema field is
-  // placed at most once, so render-time lookups are unguarded.
-  // ---------------------------------------------------------------------
-
-  const fieldByName = useMemo(() => {
-    const map = new Map<string, Field>()
-    for (const field of fields) {
-      if ('name' in field) map.set(field.name, field)
-    }
-    return map
-  }, [fields])
-
-  const tabSetByName = useMemo(() => {
-    const map = new Map<string, TabSetDefinition>()
-    for (const set of adminConfig?.tabSets ?? []) map.set(set.name, set)
-    return map
-  }, [adminConfig])
-
-  const rowByName = useMemo(() => {
-    const map = new Map<string, RowDefinition>()
-    for (const row of adminConfig?.rows ?? []) map.set(row.name, row)
-    return map
-  }, [adminConfig])
-
-  const groupByName = useMemo(() => {
-    const map = new Map<string, GroupDefinition>()
-    for (const group of adminConfig?.groups ?? []) map.set(group.name, group)
-    return map
-  }, [adminConfig])
-
-  // When `layout` is omitted, synthesise main = all schema fields in order.
-  const layout = useMemo(() => {
-    if (adminConfig?.layout) return adminConfig.layout
-    return { main: fields.filter((f) => 'name' in f).map((f) => (f as { name: string }).name) }
-  }, [adminConfig, fields])
-
-  // Reverse index: schema field name → which tab set + tab it lives in.
-  // Powers per-tab-set error badge counts. Fields not under any tab set
-  // (e.g. raw-field placement directly in `layout.main`) are absent from
-  // this map.
-  const fieldToTabPath = useMemo(() => {
-    const map = new Map<string, { tabSetName: string; tabName: string }>()
-    const visit = (
-      names: readonly string[],
-      tabSetName: string,
-      tabName: string,
-      seen: Set<string>
-    ) => {
-      for (const name of names) {
-        if (fieldByName.has(name)) {
-          map.set(name, { tabSetName, tabName })
-        } else if (seen.has(name)) {
-        } else if (rowByName.has(name)) {
-          const row = rowByName.get(name)!
-          const next = new Set(seen).add(name)
-          visit(row.fields, tabSetName, tabName, next)
-        } else if (groupByName.has(name)) {
-          const group = groupByName.get(name)!
-          const next = new Set(seen).add(name)
-          visit(group.fields, tabSetName, tabName, next)
-        }
-      }
-    }
-    for (const set of adminConfig?.tabSets ?? []) {
-      for (const tab of set.tabs) {
-        visit(tab.fields, set.name, tab.name, new Set())
-      }
-    }
-    return map
-  }, [adminConfig, fieldByName, rowByName, groupByName])
+  // Layout primitives + lookup tables — pure derivations of `adminConfig` +
+  // `fields`. The validator at startup guarantees every reachable name
+  // resolves and every schema field is placed at most once, so the render-time
+  // lookups below are unguarded. See ./use-form-layout.
+  const { fieldByName, tabSetByName, rowByName, groupByName, layout, fieldToTabPath } =
+    useFormLayout(adminConfig, fields)
 
   // ---------------------------------------------------------------------
   // Active-tab state — one tab name per declared tab set.
@@ -901,159 +676,21 @@ const FormContent = ({
           {(layout.sidebar ?? []).map((name) => renderItem(name))}
         </div>
       </div>
-      {showUnsavedModal && (
-        <Modal
-          isOpen={true}
-          closeOnOverlayClick={true}
-          onDismiss={() => setShowUnsavedModal(false)}
-        >
-          <Modal.Container style={{ maxWidth: '460px' }}>
-            <Modal.Header
-              className={cx('byline-form-guard-modal-head', styles['guard-modal-head'])}
-            >
-              <h3 className={cx('byline-form-guard-modal-title', styles['guard-modal-title'])}>
-                {t('forms.unsavedChanges.title')}
-              </h3>
-            </Modal.Header>
-            <Modal.Content>
-              <p className={cx('byline-form-guard-modal-text', styles['guard-modal-text'])}>
-                {t('forms.unsavedChanges.message')}
-              </p>
-            </Modal.Content>
-            <Modal.Actions>
-              <Button
-                size="sm"
-                style={{ minWidth: '60px' }}
-                intent="primary"
-                type="button"
-                onClick={() => setShowUnsavedModal(false)}
-              >
-                {t('forms.unsavedChanges.okButton')}
-              </Button>
-            </Modal.Actions>
-          </Modal.Container>
-        </Modal>
-      )}
+      {showUnsavedModal && <UnsavedChangesModal onClose={() => setShowUnsavedModal(false)} />}
       {pendingSystemFieldsSubmit != null && (
-        <Modal
-          isOpen={true}
-          closeOnOverlayClick={true}
-          onDismiss={() => setPendingSystemFieldsSubmit(null)}
-        >
-          <Modal.Container style={{ maxWidth: '520px' }}>
-            <Modal.Header
-              className={cx('byline-form-guard-modal-head', styles['guard-modal-head'])}
-            >
-              <h3 className={cx('byline-form-guard-modal-title', styles['guard-modal-title'])}>
-                {pendingSystemFieldsSubmit.contentDirty
-                  ? t('forms.systemFieldsConfirm.bothTitle')
-                  : t('forms.systemFieldsConfirm.title')}
-              </h3>
-              <IconButton
-                aria-label={t('common.actions.close')}
-                size="xs"
-                onClick={() => setPendingSystemFieldsSubmit(null)}
-              >
-                <CloseIcon width="16px" height="16px" svgClassName="white-icon" />
-              </IconButton>
-            </Modal.Header>
-            <Modal.Content className="prose">
-              {/* Lead with reassurance: content edits follow the normal
-                  revision + publish workflow. The immediate, document-level
-                  system-field write is explained below the divider. */}
-              {pendingSystemFieldsSubmit.contentDirty && (
-                <p className={cx('byline-form-system-fields-content-note', 'm-0 mt-2')}>
-                  {t('forms.systemFieldsConfirm.contentNote')}
-                </p>
-              )}
-              <p
-                className="m-0 mt-2"
-                style={
-                  pendingSystemFieldsSubmit.contentDirty
-                    ? {
-                        marginTop: 'var(--spacing-8)',
-                        paddingTop: 'var(--spacing-12)',
-                        borderTop: '1px solid var(--border-color)',
-                      }
-                    : undefined
-                }
-              >
-                {t('forms.systemFieldsConfirm.intro')}
-              </p>
-              <ul
-                className={cx('byline-form-system-fields-list', styles['guard-modal-text'], 'm-0')}
-              >
-                {pendingSystemFieldsSubmit.pathDirty && (
-                  <li>{t('forms.systemFieldsConfirm.bulletPath')}</li>
-                )}
-                {pendingSystemFieldsSubmit.availableLocalesDirty && (
-                  <li>{t('forms.systemFieldsConfirm.bulletLocales')}</li>
-                )}
-              </ul>
-              <p
-                className={cx('byline-form-system-fields-effect', styles['guard-modal-text'])}
-                style={{
-                  marginTop: 'var(--spacing-4)',
-                  marginBottom: 0,
-                  color: 'var(--text-subtle)',
-                }}
-              >
-                {t('forms.systemFieldsConfirm.effectLine')}
-              </p>
-            </Modal.Content>
-            <Modal.Actions>
-              <Button
-                size="sm"
-                style={{ minWidth: '80px' }}
-                intent="noeffect"
-                type="button"
-                onClick={() => setPendingSystemFieldsSubmit(null)}
-              >
-                {t('common.actions.cancel')}
-              </Button>
-              <Button
-                size="sm"
-                style={{ minWidth: '80px' }}
-                intent="primary"
-                type="button"
-                onClick={() => {
-                  const payload = pendingSystemFieldsSubmit
-                  setPendingSystemFieldsSubmit(null)
-                  submitPayload(payload)
-                }}
-              >
-                {t('forms.systemFieldsConfirm.confirmButton')}
-              </Button>
-            </Modal.Actions>
-          </Modal.Container>
-        </Modal>
-      )}
-      {guard.isBlocked && (
-        <Modal isOpen={true} closeOnOverlayClick={false} onDismiss={guard.stay}>
-          <Modal.Container style={{ maxWidth: '460px' }}>
-            <Modal.Header
-              className={cx('byline-form-guard-modal-head', styles['guard-modal-head'])}
-            >
-              <h3 className={cx('byline-form-guard-modal-title', styles['guard-modal-title'])}>
-                {t('forms.navigationGuard.title')}
-              </h3>
-            </Modal.Header>
-            <Modal.Content>
-              <p className={cx('byline-form-guard-modal-text', styles['guard-modal-text'])}>
-                {t('forms.navigationGuard.message')}
-              </p>
-            </Modal.Content>
-            <Modal.Actions>
-              <Button size="sm" intent="noeffect" type="button" onClick={guard.stay}>
-                {t('forms.navigationGuard.stayButton')}
-              </Button>
-              <Button size="sm" intent="danger" type="button" onClick={guard.proceed}>
-                {t('forms.navigationGuard.leaveButton')}
-              </Button>
-            </Modal.Actions>
-          </Modal.Container>
-        </Modal>
+        <SystemFieldsConfirmModal
+          contentDirty={pendingSystemFieldsSubmit.contentDirty}
+          pathDirty={pendingSystemFieldsSubmit.pathDirty}
+          availableLocalesDirty={pendingSystemFieldsSubmit.availableLocalesDirty}
+          onCancel={() => setPendingSystemFieldsSubmit(null)}
+          onConfirm={() => {
+            const payload = pendingSystemFieldsSubmit
+            setPendingSystemFieldsSubmit(null)
+            submitPayload(payload)
+          }}
+        />
       )}
+      {guard.isBlocked && <NavigationGuardModal onStay={guard.stay} onProceed={guard.proceed} />}
     </form>
   )
 }

package/src/forms/form-status-display.tsx

@@ -0,0 +1,108 @@
+'use client'
+
+/**
+ * This Source Code is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * Copyright (c) Infonomic Company Limited
+ */
+
+import type { WorkflowStatus } from '@byline/core'
+import { useTranslation } from '@byline/i18n/react'
+import cx from 'classnames'
+
+import { LocalDateTime } from '../fields/local-date-time'
+import styles from './form-renderer.module.css'
+import type { PublishedVersionInfo } from './form-renderer'
+
+/**
+ * The status / metadata strip at the top of a document form — current
+ * workflow status (suppressed for single-status workflows), last-modified and
+ * created timestamps, and a "published live" indicator with an inline
+ * Unpublish action when a previously-published version is still live.
+ */
+export const FormStatusDisplay = ({
+  initialData,
+  workflowStatuses,
+  publishedVersion,
+  onUnpublish,
+}: {
+  initialData?: Record<string, any>
+  workflowStatuses?: WorkflowStatus[]
+  publishedVersion?: PublishedVersionInfo | null
+  onUnpublish?: () => Promise<void>
+}) => {
+  const { t } = useTranslation('byline-admin')
+  const statusCode = initialData?.status
+  const statusLabel = workflowStatuses?.find((s) => s.name === statusCode)?.label ?? statusCode
+  // Single-status workflows (e.g. lookups) have no editorial lifecycle —
+  // suppress the "Status: …" cell since there is nothing meaningful to convey.
+  const showStatusCell = (workflowStatuses?.length ?? 0) > 1
+
+  return (
+    <div className={cx('byline-form-status', styles.status)}>
+      <div className={cx('byline-form-status-meta', styles['status-meta'])}>
+        {showStatusCell && (
+          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
+            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
+              {t('forms.status.label')}
+            </span>
+            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
+              {statusLabel}
+            </span>
+          </div>
+        )}
+
+        {initialData?.updatedAt != null && (
+          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
+            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
+              {t('forms.status.lastModified')}
+            </span>
+            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
+              <LocalDateTime value={initialData.updatedAt} />
+            </span>
+          </div>
+        )}
+
+        {initialData?.createdAt != null && (
+          <div className={cx('byline-form-status-cell', styles['status-cell'])}>
+            <span className={cx('byline-form-status-muted', styles['status-muted'])}>
+              {t('forms.status.created')}
+            </span>
+            <span className={cx('byline-form-status-trunc', styles['status-trunc'])}>
+              <LocalDateTime value={initialData.createdAt} />
+            </span>
+          </div>
+        )}
+      </div>
+
+      {publishedVersion != null && (
+        <div className={cx('byline-form-status-published', styles['status-published'])}>
+          <span className={cx('byline-form-status-muted', styles['status-muted'])}>
+            {t('forms.status.publishedLive')}{' '}
+            {publishedVersion.updatedAt ? (
+              <span>
+                {t('forms.status.publishedOn', { date: new Date(publishedVersion.updatedAt) })}
+              </span>
+            ) : (
+              ''
+            )}
+          </span>
+          {onUnpublish && (
+            <>
+              {' '}
+              <button
+                type="button"
+                onClick={onUnpublish}
+                className={cx('byline-form-status-unpublish', styles['status-unpublish'])}
+              >
+                {t('common.actions.unpublish')}
+              </button>
+            </>
+          )}
+        </div>
+      )}
+    </div>
+  )
+}

package/src/forms/status-transitions.test.node.ts

@@ -0,0 +1,87 @@
+/**
+ * This Source Code is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * Copyright (c) Infonomic Company Limited
+ */
+
+import type { WorkflowStatus } from '@byline/core'
+import { describe, expect, it } from 'vitest'
+
+import { computeStatusTransitions } from './status-transitions'
+
+const draft: WorkflowStatus = { name: 'draft', label: 'Draft' }
+const review: WorkflowStatus = { name: 'review', label: 'Review' }
+const published: WorkflowStatus = { name: 'published', label: 'Published' }
+const flow = [draft, review, published]
+
+describe('computeStatusTransitions', () => {
+  it('falls back to nextStatus when there is no workflow or no current status', () => {
+    expect(computeStatusTransitions(undefined, undefined, published)).toEqual({
+      primaryStatus: published,
+      secondaryStatuses: [],
+      isTerminal: false,
+    })
+    expect(computeStatusTransitions('draft', [], published)).toEqual({
+      primaryStatus: published,
+      secondaryStatuses: [],
+      isTerminal: false,
+    })
+    expect(computeStatusTransitions(undefined, flow, published)).toEqual({
+      primaryStatus: published,
+      secondaryStatuses: [],
+      isTerminal: false,
+    })
+  })
+
+  it('exposes no transitions for a single-status workflow', () => {
+    expect(computeStatusTransitions('draft', [draft], undefined)).toEqual({
+      primaryStatus: undefined,
+      secondaryStatuses: [],
+      isTerminal: false,
+    })
+  })
+
+  it('falls back to nextStatus when the current status is not in the workflow', () => {
+    expect(computeStatusTransitions('archived', flow, review)).toEqual({
+      primaryStatus: review,
+      secondaryStatuses: [],
+      isTerminal: false,
+    })
+  })
+
+  it('at the first step: primary is the forward step, no back-steps', () => {
+    const result = computeStatusTransitions('draft', flow, review)
+    expect(result.primaryStatus).toBe(review)
+    expect(result.isTerminal).toBe(false)
+    // Forward step is the primary, so it is filtered out of the dropdown.
+    expect(result.secondaryStatuses).toEqual([])
+  })
+
+  it('in the middle: primary is forward, dropdown offers reset-to-first', () => {
+    const result = computeStatusTransitions('review', flow, published)
+    expect(result.primaryStatus).toBe(published)
+    expect(result.isTerminal).toBe(false)
+    // Reset-to-first (draft) is surfaced; the forward step (published) is the
+    // primary and filtered out. The "back one step" is draft itself here, so
+    // it is not duplicated.
+    expect(result.secondaryStatuses).toEqual([draft])
+  })
+
+  it('at the terminal step: primary is the current status, back-steps in dropdown', () => {
+    const result = computeStatusTransitions('published', flow, undefined)
+    expect(result.primaryStatus).toBe(published)
+    expect(result.isTerminal).toBe(true)
+    // Both reset-to-first (draft) and back-one-step (review) are offered.
+    expect(result.secondaryStatuses).toEqual([draft, review])
+  })
+
+  it('terminal back-one-step is omitted when the previous is already the first', () => {
+    const twoStep = [draft, published]
+    const result = computeStatusTransitions('published', twoStep, undefined)
+    expect(result.isTerminal).toBe(true)
+    // Only reset-to-first (draft); no separate back-one-step since prev === first.
+    expect(result.secondaryStatuses).toEqual([draft])
+  })
+})