@wordpress/core-data 7.33.0 → 7.33.1-next.36001005c.0

package/src/actions.js

@@ -19,7 +19,7 @@ import { receiveItems, removeItems, receiveQueriedItems } from './queried-data';
 import { DEFAULT_ENTITY_KEY } from './entities';
 import { createBatch } from './batch';
 import { STORE_NAME } from './name';
-import { LOCAL_EDITOR_ORIGIN, syncManager } from './sync';
+import { LOCAL_EDITOR_ORIGIN, getSyncManager } from './sync';
 import logEntityDeprecation from './utils/log-entity-deprecation';
 
 /**
@@ -415,7 +415,7 @@ export const editEntityRecord =
 				const objectType = `${ kind }/${ name }`;
 				const objectId = recordId;
 
-				syncManager.update(
+				getSyncManager()?.update(
 					objectType,
 					objectId,
 					edit.edits,

package/src/entities.js

@@ -272,41 +272,7 @@ async function loadPostTypeEntities() {
 		);
 		const namespace = postType?.rest_namespace ?? 'wp/v2';
 
-		/**
-		 * @type {import('@wordpress/sync').SyncConfig}
-		 */
-		const syncConfig = {
-			/**
-			 * Apply changes from the local editor to the local CRDT document so
-			 * that those changes can be synced to other peers (via the provider).
-			 *
-			 * @param {import('@wordpress/sync').CRDTDoc}               crdtDoc
-			 * @param {Partial< import('@wordpress/sync').ObjectData >} changes
-			 * @return {void}
-			 */
-			applyChangesToCRDTDoc: ( crdtDoc, changes ) =>
-				applyPostChangesToCRDTDoc( crdtDoc, changes, postType ),
-
-			/**
-			 * Extract changes from a CRDT document that can be used to update the
-			 * local editor state.
-			 *
-			 * @param {import('@wordpress/sync').CRDTDoc}    crdtDoc
-			 * @param {import('@wordpress/sync').ObjectData} editedRecord
-			 * @return {Partial< import('@wordpress/sync').ObjectData >} Changes to record
-			 */
-			getChangesFromCRDTDoc: ( crdtDoc, editedRecord ) =>
-				getPostChangesFromCRDTDoc( crdtDoc, editedRecord, postType ),
-
-			/**
-			 * Sync features supported by the entity.
-			 *
-			 * @type {Record< string, boolean >}
-			 */
-			supports: {},
-		};
-
-		return {
+		const entity = {
 			kind: 'postType',
 			baseURL: `/${ namespace }/${ postType.rest_base }`,
 			baseURLParams: { context: 'edit' },
@@ -326,7 +292,6 @@ async function loadPostTypeEntities() {
 					: String( record.id ) ),
 			__unstablePrePersist: isTemplate ? undefined : prePersistPostType,
 			__unstable_rest_base: postType.rest_base,
-			syncConfig,
 			supportsPagination: true,
 			getRevisionsUrl: ( parentId, revisionId ) =>
 				`/${ namespace }/${
@@ -336,6 +301,50 @@ async function loadPostTypeEntities() {
 				}`,
 			revisionKey: DEFAULT_ENTITY_KEY,
 		};
+
+		if ( window.__experimentalEnableSync ) {
+			if ( globalThis.IS_GUTENBERG_PLUGIN ) {
+				/**
+				 * @type {import('@wordpress/sync').SyncConfig}
+				 */
+				entity.syncConfig = {
+					/**
+					 * Apply changes from the local editor to the local CRDT document so
+					 * that those changes can be synced to other peers (via the provider).
+					 *
+					 * @param {import('@wordpress/sync').CRDTDoc}               crdtDoc
+					 * @param {Partial< import('@wordpress/sync').ObjectData >} changes
+					 * @return {void}
+					 */
+					applyChangesToCRDTDoc: ( crdtDoc, changes ) =>
+						applyPostChangesToCRDTDoc( crdtDoc, changes, postType ),
+
+					/**
+					 * Extract changes from a CRDT document that can be used to update the
+					 * local editor state.
+					 *
+					 * @param {import('@wordpress/sync').CRDTDoc}    crdtDoc
+					 * @param {import('@wordpress/sync').ObjectData} editedRecord
+					 * @return {Partial< import('@wordpress/sync').ObjectData >} Changes to record
+					 */
+					getChangesFromCRDTDoc: ( crdtDoc, editedRecord ) =>
+						getPostChangesFromCRDTDoc(
+							crdtDoc,
+							editedRecord,
+							postType
+						),
+
+					/**
+					 * Sync features supported by the entity.
+					 *
+					 * @type {Record< string, boolean >}
+					 */
+					supports: {},
+				};
+			}
+		}
+
+		return entity;
 	} );
 }
 
@@ -373,13 +382,21 @@ async function loadSiteEntity() {
 		name: 'site',
 		kind: 'root',
 		baseURL: '/wp/v2/settings',
-		syncConfig: {
-			applyChangesToCRDTDoc: defaultApplyChangesToCRDTDoc,
-			getChangesFromCRDTDoc: defaultGetChangesFromCRDTDoc,
-		},
 		meta: {},
 	};
 
+	if ( window.__experimentalEnableSync ) {
+		if ( globalThis.IS_GUTENBERG_PLUGIN ) {
+			/**
+			 * @type {import('@wordpress/sync').SyncConfig}
+			 */
+			entity.syncConfig = {
+				applyChangesToCRDTDoc: defaultApplyChangesToCRDTDoc,
+				getChangesFromCRDTDoc: defaultGetChangesFromCRDTDoc,
+			};
+		}
+	}
+
 	const site = await apiFetch( {
 		path: entity.baseURL,
 		method: 'OPTIONS',

package/src/resolvers.js

@@ -15,7 +15,7 @@ import apiFetch from '@wordpress/api-fetch';
  */
 import { STORE_NAME } from './name';
 import { additionalEntityConfigLoaders, DEFAULT_ENTITY_KEY } from './entities';
-import { syncManager } from './sync';
+import { getSyncManager } from './sync';
 import {
 	forwardResolver,
 	getNormalizedCommaSeparable,
@@ -185,7 +185,7 @@ export const getEntityRecord =
 						} );
 
 					// Load the entity record for syncing.
-					await syncManager.load(
+					await getSyncManager()?.load(
 						entityConfig.syncConfig,
 						objectType,
 						objectId,

package/src/sync.ts

@@ -5,8 +5,20 @@ import {
 	CRDT_RECORD_MAP_KEY,
 	LOCAL_EDITOR_ORIGIN,
 	LOCAL_SYNC_MANAGER_ORIGIN,
+	type SyncManager,
 	createSyncManager,
 } from '@wordpress/sync';
 
 export { CRDT_RECORD_MAP_KEY, LOCAL_EDITOR_ORIGIN, LOCAL_SYNC_MANAGER_ORIGIN };
-export const syncManager = createSyncManager();
+
+let syncManager: SyncManager;
+
+export function getSyncManager(): SyncManager | undefined {
+	if ( syncManager ) {
+		return syncManager;
+	}
+
+	syncManager = createSyncManager();
+
+	return syncManager;
+}

package/src/test/resolvers.js

@@ -6,13 +6,11 @@ import triggerFetch from '@wordpress/api-fetch';
 /**
  * Internal dependencies
  */
-import { syncManager } from '../sync';
+import { getSyncManager } from '../sync';
 
 jest.mock( '@wordpress/api-fetch' );
 jest.mock( '../sync', () => ( {
-	syncManager: {
-		load: jest.fn(),
-	},
+	getSyncManager: jest.fn(),
 } ) );
 
 /**
@@ -42,6 +40,8 @@ describe( 'getEntityRecord', () => {
 	const resolveSelect = { getEntitiesConfig: jest.fn( () => ENTITIES ) };
 
 	let dispatch;
+	let syncManager;
+
 	beforeEach( async () => {
 		dispatch = Object.assign( jest.fn(), {
 			receiveEntityRecords: jest.fn(),
@@ -51,7 +51,11 @@ describe( 'getEntityRecord', () => {
 			finishResolutions: jest.fn(),
 		} );
 		triggerFetch.mockReset();
-		syncManager.load.mockClear();
+
+		syncManager = {
+			load: jest.fn(),
+		};
+		getSyncManager.mockImplementation( () => syncManager );
 	} );
 
 	afterEach( () => {