@elytracms/studio 0.0.92 → 0.0.94

package/dist/components/cms/cms-store.d.ts

@@ -1,8 +1,8 @@
 import type { ReactNode } from 'react';
 import type { CmsDocument, CmsValidationIssue, CollectionDef, CollectionRegistry, DocumentPatch, DocumentRef, FieldDef, ValuePath, Locale, LocaleConfig, LocalizedString, RestoreScope, ValidateDocumentOptions } from '@elytracms/core/cms-core';
 import type { CmsDocumentRecord, DocumentVersionRecord, IdFactory, PresenceEntry } from '@elytracms/core/persistence';
-import type { CommitOutcome, DocumentConflict, OperationError, OperationsClient } from '@elytracms/studio/lib/operations';
-import type { TableViewState } from '@elytracms/studio/lib/cms-views';
+import type { CommitOutcome, DocumentConflict, OperationError, OperationsClient } from '../../lib/operations/index.js';
+import type { TableViewState } from '../../lib/cms-views/index.js';
 import type { CellCommitInput, CellCommitOutcome } from './cell-edit.js';
 import type { BulkAction, BulkReport, BulkSetField } from './bulk-actions.js';
 /**
@@ -142,10 +142,11 @@ export interface CmsStore {
      */
     documentHasUnpublishedChanges(ref: DocumentRef): boolean;
     /**
-     * Publish: snapshot the draft as a new version and pin it live (EC-224).
-     * Flips published-ness optimistically, then reconciles against the backend
-     * record (EC-225) so the badge can never claim a publish the backend refused
-     * (e.g. blocked by publish-time field validation).
+     * Publish: the ONE publish operation (EC-585) — it clears the draft's hidden values,
+     * validates, and pins a new version (EC-224). Flips published-ness optimistically,
+     * then reconciles against the backend record (EC-225) so the badge can never claim a
+     * publish the backend refused — and a refusal REJECTS (`PublishRefusedError`, after
+     * reporting it on the persist banner), so no caller shows success for it.
      */
     publishDocument(ref: DocumentRef): Promise<void>;
     /**

package/dist/components/cms/cms-store.js

@@ -2,23 +2,23 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import { applyDocumentPatches, createCollectionRegistry, createDocumentStore, documentKey, documentReachabilityIssues, documentSchema, resolveDocumentTitle, seedDefaults, titleFieldOf, validateDocument, valuesInLocale, writeField, } from '@elytracms/core/cms-core';
 import { createUuidIdFactory } from '@elytracms/core/persistence';
-import { translate } from '@elytracms/studio/lib/i18n';
-import { configureStudioLocales, useStudioLocale } from '@elytracms/studio/lib/locale';
-import { studioManifestLookup } from '@elytracms/studio/lib/registry';
-import { resolveDeploymentCollections, resolveDeploymentLocaleConfig, resolveDeploymentRoutes, } from '@elytracms/studio/lib/workspace';
+import { translate } from '../../lib/i18n/index.js';
+import { configureStudioLocales, useStudioLocale } from '../../lib/locale/index.js';
+import { studioManifestLookup } from '../../lib/registry/index.js';
+import { resolveDeploymentCollections, resolveDeploymentLocaleConfig, resolveDeploymentRoutes, } from '../../lib/workspace/index.js';
 import { Heading, Panel, Stack, Text } from '@elytracms/studio/ui';
 import { assetLibraryOracle, gateValidateOptions } from './gate-validation.js';
 import { matchesRef, removeDoc, replaceDoc, upsertDoc } from './document-ref-utils.js';
-import { createRequiredContext } from '@elytracms/studio/lib/required-context';
-import { createCmsPersister, getStudioActor, getStudioOperations } from '@elytracms/studio/lib/operations';
-import { publishWithRevision as publishRevision } from '@elytracms/studio/lib/operations/publish-with-revision';
-import { fieldWritePatch, fieldPositionPatches } from '@elytracms/studio/lib/cms/field-patch';
-import { ALL_VIEW_ID, DEFAULT_VIEW_OVERRIDE_ID, canonicalViewState, configViewIssues, defaultConfigViewId, defaultViewState, deprecatedDefaultFlagViewIds, hasDefaultViewOverride, pseudoColumns, resolveCollectionViews, viewStatesEqual, } from '@elytracms/studio/lib/cms-views';
+import { createRequiredContext } from '../../lib/required-context.js';
+import { createCmsPersister, getStudioActor, getStudioOperations } from '../../lib/operations/index.js';
+import { publishInSession, publishWithRevision as publishRevision, } from '../../lib/operations/publish-with-revision.js';
+import { operationErrorMessage } from '../../lib/persona/labels.js';
+import { fieldWritePatch } from '../../lib/cms/field-patch.js';
+import { ALL_VIEW_ID, DEFAULT_VIEW_OVERRIDE_ID, canonicalViewState, configViewIssues, defaultConfigViewId, defaultViewState, deprecatedDefaultFlagViewIds, hasDefaultViewOverride, pseudoColumns, resolveCollectionViews, viewStatesEqual, } from '../../lib/cms-views/index.js';
 import { useOptionalAssets } from '../assets/assets-store.js';
 import { consumeStagedDocuments } from './document-staging.js';
 import { commitCellEdit } from './cell-edit.js';
 import { runBulkAction } from './bulk-actions.js';
-import { cleanHiddenForPublish } from './publish-clean.js';
 const { Context: CmsContext, useRequired: useCms } = createRequiredContext('Cms');
 /** The stable empty conflict list (referential stability for the surfaces' memos). */
 const NO_CONFLICTS = [];
@@ -574,16 +574,16 @@ function CmsSession({ children, projectId, initialCollections, initialDocuments,
             validate: (doc) => validateRef.current(doc),
         });
     }, [projectId]);
-    const publishDocument = useCallback(async (ref) => {
-        setPublished(ref, true); // optimistic
-        // EC-273: no clearing here — the draft is already free of conditionally-hidden
-        // values (the editor clears them on write), so the snapshot the publish op
-        // pins is clean by construction.
-        const result = await publishWithRevision(ref);
-        if (!result.ok)
-            setPersistError(result.error);
-        await reconcilePublished(ref);
-    }, [setPublished, reconcilePublished, publishWithRevision]);
+    const publishDocument = useCallback((ref) => 
+    // EC-585: the hidden-clear runs INSIDE `documents.publish` (for this and every
+    // other caller), and a refusal rejects — see `publishInSession`.
+    publishInSession(ref, {
+        markPublished: (target) => setPublished(target, true), // optimistic
+        publish: publishWithRevision,
+        reconcile: reconcilePublished,
+        report: setPersistError,
+        describe: operationErrorMessage,
+    }), [setPublished, reconcilePublished, publishWithRevision]);
     const unpublishDocument = useCallback(async (ref) => {
         setPublished(ref, false); // optimistic
         persisterRef.current?.unpublishDocument(ref);
@@ -899,28 +899,9 @@ function CmsSession({ children, projectId, initialCollections, initialDocuments,
                 isPublished,
                 ...(findAsset ? { findAsset } : {}),
             })),
-            // EC-425: bulk publish joins the single-document clean point — the
-            // SAME shared hidden-strip `doPublish` runs (EC-273 field values +
-            // EC-394 component props, session registry), per item.
-            // EC-580: in the session's locale views — the single publish's same step.
-            ...(collection
-                ? {
-                    cleanForPublish: (doc) => cleanHiddenForPublish(collection, doc, undefined, localeConfig),
-                }
-                : {}),
-            // Land a cleaned draft exactly the way the editor's publish does:
-            // session apply + the persister's self-typing upsert (EC-277 manifest
-            // lens), flushed so the serial queue lands the upsert BEFORE the
-            // awaited publish command snapshots the stored draft.
-            // EC-595 A2: the strip emits the field positions it cleared (computed in the
-            // clean writer from its own input/output, never by the persister).
-            applyCleaned: async (ref, cleaned) => {
-                const current = documentsRef.current.find((d) => matchesRef(d, ref));
-                if (!current)
-                    return;
-                await commitPatches(ref, fieldPositionPatches(current, cleaned));
-                await persisterRef.current?.flush(ref);
-            },
+            // EC-585: no clean step here — the hidden-clear runs inside the one publish
+            // operation each item's `publishWithRevision` dispatches (EC-425's shared
+            // clean point, moved to where every publish path crosses it).
             // EC-224: a succeeded publish/unpublish moves the version pointer
             // (`publishedKeys`) via setPublished — there is no separate flat
             // `state` mirror to keep in sync (published-ness is derived from the
@@ -957,7 +938,6 @@ function CmsSession({ children, projectId, initialCollections, initialDocuments,
         findAsset,
         stampHasUnpublishedChanges,
         publishWithRevision,
-        commitPatches,
     ]);
     // EC-595 point 7 — follow one open document: pushed rows go to the persister (applied
     // under the local edits), presence to the caller. The capability lives on the