feat: batch queued planning edits (#375)

frontend/later-sync.js

@@ -184,19 +184,29 @@ function createLaterSync({ storage, getLogin, fetchJson, onRemoteRecords, onStat
     if (!ownerKey) return false;
     ensureChannel();
     try {
-      let plan = await fetchJson('api/v1/later');
       let operations = pending();
+      let plan;
+      if (!operations.length) plan = await fetchJson('api/v1/later');
       while (operations.length) {
         if (key() !== ownerKey) return false;
-        const operation = operations[0];
+        const batch = operations.slice(0, 50);
         plan = await fetchJson('api/v1/later', {
           method: 'PATCH',
           headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify(operation),
+          body: JSON.stringify({ operations: batch }),
         });
-        if (pending().some(candidate => candidate.operation_id === operation.operation_id) &&
-            !removeOperation(operation.operation_id)) {
-          throw new Error('Could not persist Later delivery receipt');
+        const hasReceipts = Array.isArray(plan.accepted_operation_ids) ||
+          Array.isArray(plan.duplicate_operation_ids) || Array.isArray(plan.rejected_operations);
+        const received = hasReceipts ? [
+          ...(plan.accepted_operation_ids || []),
+          ...(plan.duplicate_operation_ids || []),
+          ...(plan.rejected_operations || []).map(item => item.operation_id),
+        ] : batch.map(item => item.operation_id);
+        for (const operationId of received) {
+          if (pending().some(candidate => candidate.operation_id === operationId) &&
+              !removeOperation(operationId)) {
+            throw new Error('Could not persist Later delivery receipt');
+          }
         }
         operations = pending();
       }

frontend/service-worker.js

@@ -1,6 +1,6 @@
 const BASE = new URL('./', self.location.href).pathname;
 importScripts(BASE + 'static/background-issue-sync.js');
-const CACHE = 'stackchain-dashboard-shell-v53';
+const CACHE = 'stackchain-dashboard-shell-v54';
 const OUTAGE_STATUSES = new Set([500, 502, 503, 504]);
 const NAVIGATION_TIMEOUT_MS = self.__STACKCHAIN_NAVIGATION_TIMEOUT_MS || 4000;
 const SHELL = [

frontend/today-sync.js

@@ -170,30 +170,31 @@ function createTodaySync({ storage, getLogin, fetchJson, onRemoteIds, onStatus,
     if (!ownerKey) return false;
     ensureChannel();
     try {
-      let plan = await fetchJson('api/v1/today');
       let operations = pending();
+      let plan;
       let hadConflict = false;
+      if (!operations.length) plan = await fetchJson('api/v1/today');
       while (operations.length) {
         if (key() !== ownerKey) return false;
-        const operation = operations[0];
-        try {
-          plan = await fetchJson('api/v1/today', {
-            method: 'PATCH',
-            headers: { 'Content-Type': 'application/json' },
-            body: JSON.stringify(operation),
-          });
-        } catch (error) {
-          if (error?.status !== 409) throw error;
-          if (!removeOperation(operation.operation_id)) {
-            throw new Error('Could not persist rejected Today operation');
+        const batch = operations.slice(0, 50);
+        plan = await fetchJson('api/v1/today', {
+          method: 'PATCH',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify({ operations: batch }),
+        });
+        const hasReceipts = Array.isArray(plan.accepted_operation_ids) ||
+          Array.isArray(plan.duplicate_operation_ids) || Array.isArray(plan.rejected_operations);
+        const received = hasReceipts ? [
+          ...(plan.accepted_operation_ids || []),
+          ...(plan.duplicate_operation_ids || []),
+          ...(plan.rejected_operations || []).map(item => item.operation_id),
+        ] : batch.map(item => item.operation_id);
+        hadConflict = hadConflict || Boolean(plan.rejected_operations?.length);
+        for (const operationId of received) {
+          if (pending().some(candidate => candidate.operation_id === operationId) &&
+              !removeOperation(operationId)) {
+            throw new Error('Could not persist Today delivery receipt');
           }
-          hadConflict = true;
-          operations = pending();
-          continue;
-        }
-        if (pending().some(candidate => candidate.operation_id === operation.operation_id) &&
-            !removeOperation(operation.operation_id)) {
-          throw new Error('Could not persist Today delivery receipt');
         }
         operations = pending();
       }