@recordtimelabel/core 0.2.0 → 0.2.1

package/README.md

@@ -20,7 +20,7 @@ During local development an app can consume a sibling checkout with:
 For release builds, consume a fixed npm package, git tag, or private registry version so builds do not depend on a sibling folder path. The v2 gateway contract release is:
 
 ```json
-"@recordtimelabel/core": "0.2.0"
+"@recordtimelabel/core": "0.2.1"
 ```
 
 If this checkout's `package.json` is ahead of the published version, publish the new package before updating consumers to that version.
@@ -50,7 +50,7 @@ If this checkout's `package.json` is ahead of the published version, publish the
 - `extendFirestoreV2OperationReadPlanWithRecords(readPlan, records)`
 - `planFirestoreV2OperationChanges({ documents, operations, localState, now })`
 - `estimateFirestoreV2WriteUnits(changes, overhead)`
-- `buildOperationsFromSnapshotDiff({ previousState, nextState, now, operationIdPrefix, batchSize })`
+- `buildOperationsFromSnapshotDiff({ previousState, nextState, now, operationIdPrefix, batchSize })`（一般操作在 `batches`，`folder.delete` 在 `bulkOperations`）
 - `RTL_SYNC_PROTOCOL_VERSION`
 - `hasMeaningfulRecordTimeLabelCloudState(data, options)`
 - `buildRecordTimeLabelContentFingerprint(data)`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@recordtimelabel/core",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "type": "module",
   "description": "Shared RecordTimeLabel data model, merge logic, operations, and sync engine.",
   "main": "./src/index.js",

package/src/index.js

@@ -5,7 +5,7 @@ const REQUIRED_FOLDERS = [
   { id: DEFAULT_FOLDER_ID, name: 'Uncategorized' }
 ];
 
-export const RECORD_TIMELABEL_CORE_VERSION = '0.2.0';
+export const RECORD_TIMELABEL_CORE_VERSION = '0.2.1';
 export const RTL_SYNC_PROTOCOL_VERSION = 1;
 export const RTL_MAX_OPERATIONS_PER_REQUEST = 20;
 export const RTL_MAX_REQUEST_BYTES = 256 * 1024;
@@ -46,9 +46,20 @@ export const RECORD_TIMELABEL_CLOUD_SCHEMAS = Object.freeze({
 export const FIRESTORE_V2_SETTINGS_DOC_ID = 'main';
 
 const toArray = (value) => (Array.isArray(value) ? value : []);
-const clone = (value) => {
-  if (value === undefined || value === null) return value;
-  return JSON.parse(JSON.stringify(value));
+const clone = (value, seen = new WeakMap()) => {
+  if (value === undefined || value === null || typeof value !== 'object') return value;
+  if (value instanceof Date) return new Date(value.getTime());
+  const prototype = Object.getPrototypeOf(value);
+  if (prototype !== Object.prototype && prototype !== null && !Array.isArray(value)) {
+    return value;
+  }
+  if (seen.has(value)) return seen.get(value);
+  const copy = Array.isArray(value) ? [] : {};
+  seen.set(value, copy);
+  Reflect.ownKeys(value).forEach((key) => {
+    copy[key] = clone(value[key], seen);
+  });
+  return copy;
 };
 
 const toFiniteTimestamp = (value) => {
@@ -2526,21 +2537,23 @@ export const planFirestoreV2OperationChanges = ({
         });
         const recordDocument = applied.documents.records[recordId];
         if (!recordDocument) { reason = 'blocked_by_tombstone'; break; }
-        rtlSetRoot(nextDocuments.root, applied.documents.root);
-        nextDocuments.records[recordId] = recordDocument;
+        const candidateRoot = clone(applied.documents.root);
+        const candidateFolders = clone(nextDocuments.folders);
         const targetFolder = rtlEnsureTargetFolder({
-          root: nextDocuments.root,
-          folders: nextDocuments.folders,
+          root: candidateRoot,
+          folders: candidateFolders,
           localState: normalizedLocalState,
           folderId: targetFolderId,
           now: operationNow
         });
         if (!targetFolder) {
           reason = 'folder_not_found';
-          delete nextDocuments.records[recordId];
           break;
         }
         targetFolder.recordOrder = rtlMergeOrder([recordId], targetFolder.recordOrder);
+        rtlSetRoot(nextDocuments.root, candidateRoot);
+        nextDocuments.folders = candidateFolders;
+        nextDocuments.records[recordId] = recordDocument;
         changed = true;
         break;
       }
@@ -2751,6 +2764,7 @@ export const buildOperationsFromSnapshotDiff = ({
   const previousFolders = new Map(previous.folders.map((folder) => [folder.id, folder]));
   const nextFolders = new Map(next.folders.map((folder) => [folder.id, folder]));
   const drafts = [];
+  const bulkDrafts = [];
 
   nextFolders.forEach((folder, folderId) => {
     if (RTL_PROTECTED_FOLDER_IDS.has(folderId)) return;
@@ -2787,7 +2801,10 @@ export const buildOperationsFromSnapshotDiff = ({
 
   previousFolders.forEach((folder, folderId) => {
     if (!RTL_PROTECTED_FOLDER_IDS.has(folderId) && !nextFolders.has(folderId)) {
-      drafts.push({type: OPERATION_TYPES.FOLDER_DELETE, payload: {folderId}});
+      bulkDrafts.push({
+        type: OPERATION_TYPES.FOLDER_DELETE,
+        payload: {folderId, deletedAt: now}
+      });
     }
   });
 
@@ -2812,6 +2829,11 @@ export const buildOperationsFromSnapshotDiff = ({
     createdAt: now + index,
     ...draft
   }));
+  const bulkOperations = bulkDrafts.map((draft, index) => ({
+    id: `${operationIdPrefix}:bulk:${String(index).padStart(6, '0')}`,
+    createdAt: now + operations.length + index,
+    ...draft
+  }));
   const boundedBatchSize = Math.max(1, Math.min(
     RTL_MAX_OPERATIONS_PER_REQUEST,
     Number.isSafeInteger(batchSize) ? batchSize : RTL_MAX_OPERATIONS_PER_REQUEST
@@ -2826,7 +2848,14 @@ export const buildOperationsFromSnapshotDiff = ({
   const changes = buildFirestoreV2DocumentChangeSet(previousDocuments, nextDocuments, {
     allowDeletes: true
   });
-  return {operations, batches, changes, previousDocuments, nextDocuments};
+  return {
+    operations,
+    batches,
+    bulkOperations,
+    changes,
+    previousDocuments,
+    nextDocuments
+  };
 };
 
 export default {