More sync edge case fixes

Author: zefhemel

client/spaces/evented_space_primitives.ts

@@ -78,9 +78,16 @@ export class EventedSpacePrimitives implements SpacePrimitives {
       // Some other operation (read, write, list, meta) is already going on
       // this will likely trigger events, so let's not worry about any of that and avoid race condition and inconsistent data.
       console.info(
-        "alreadyFetching is on, skipping even triggering for fetchFileList.",
+        "operationInProgress, deferring event processing for fetchFileList.",
       );
-      return this.wrapped.fetchFileList();
+      const result = await this.wrapped.fetchFileList();
+      // Schedule a retry after current operation completes
+      setTimeout(() => {
+        if (!this.operationInProgress) {
+          void this.fetchFileList();
+        }
+      }, 50);
+      return result;
     }
     if (!this.enabled) {
       return this.wrapped.fetchFileList();

plugs/sync/sync.plug.yaml

@@ -15,6 +15,10 @@ functions:
     path: "./sync.ts:spaceSyncComplete"
     events:
       - "service-worker:space-sync-complete"
+  fileSyncComplete:
+    path: "./sync.ts:fileSyncComplete"
+    events:
+      - "service-worker:file-sync-complete"
   updateSyncStatus:
     path: "./sync.ts:updateSyncStatus"
     events:

plugs/sync/sync.ts

@@ -26,6 +26,15 @@ export async function spaceSyncComplete(message: { operations: number }) {
   void index.ensureFullIndex();
 }
 
+export async function fileSyncComplete(
+  message: { path: string; operations: number },
+) {
+  if (message.operations > 0) {
+    // Re-read the specific file's metadata to trigger file:changed event
+    await space.getFileMeta(message.path);
+  }
+}
+
 export async function updateSyncStatus(event: {
   status: {
     filesProcessed: number;