@abloatai/humans 0.60.0 → 0.61.0

package/dist/local/BaseSyncedStore.js

@@ -1334,10 +1334,10 @@ export class BaseSyncedStore {
         const isCreate = !this.objectPool.get(model.id);
         if (isCreate) {
             model.updatedAt = new Date();
-            this.syncClient.add(model);
+            await this.syncClient.add(model);
         }
         else {
-            this.syncClient.update(model);
+            await this.syncClient.update(model);
         }
     }
     /** Save with an atomic server mutation (e.g., createSectionWithBlocks) */
@@ -1350,19 +1350,19 @@ export class BaseSyncedStore {
         const model = rowAsModel(entity);
         this.pendingDeletes.add(model.id);
         // SyncClient.delete handles: pool remove, transaction queue
-        this.syncClient.delete(model);
+        await this.syncClient.delete(model);
     }
     /** Archive a model. Accepts schema-inferred entity shapes (see `save`). */
     async archive(entity) {
         const model = rowAsModel(entity);
         model.archivedAt = new Date();
-        this.syncClient.archive(model);
+        await this.syncClient.archive(model);
     }
     /** Unarchive a model. Accepts schema-inferred entity shapes (see `save`). */
     async unarchive(entity) {
         const model = rowAsModel(entity);
         model.archivedAt = null;
-        this.syncClient.update(model);
+        await this.syncClient.update(model);
     }
     // ── Query API ────────────────────────────────────────────────────────────
     // `ablo.<model>.local.get` / `.local.list` is the read surface for

package/dist/local/Model.js

@@ -311,7 +311,7 @@ export class Model {
         for (const key of keys) {
             if (key === 'id')
                 continue;
-            const mod = modified?.get(key);
+            const mod = modified.get(key);
             if (mod) {
                 out[key] = mod.old;
             }
@@ -358,25 +358,23 @@ export class Model {
         const errors = [];
         const modelName = this.getModelName();
         const properties = getActiveRegistry().getProperties(modelName);
-        if (properties) {
-            const json = this.toJSON();
-            for (const [propName, metadata] of properties) {
-                // Check required fields
-                if (!metadata.nullable && !metadata.optional) {
-                    const value = json[propName];
-                    if (value == null || value === '') {
-                        errors.push(`${propName} is required`);
-                    }
+        const json = this.toJSON();
+        for (const [propName, metadata] of properties) {
+            // Check required fields
+            if (!metadata.nullable && !metadata.optional) {
+                const value = json[propName];
+                if (value == null || value === '') {
+                    errors.push(`${propName} is required`);
                 }
-                // Run custom validation rules
-                const rules = this.validationRules[propName];
-                if (rules) {
-                    const value = json[propName];
-                    for (const rule of rules) {
-                        const error = rule(value);
-                        if (error)
-                            errors.push(error);
-                    }
+            }
+            // Run custom validation rules
+            const rules = this.validationRules[propName];
+            if (rules) {
+                const value = json[propName];
+                for (const rule of rules) {
+                    const error = rule(value);
+                    if (error)
+                        errors.push(error);
                 }
             }
         }
@@ -679,20 +677,18 @@ export class Model {
         if (this.archivedAt !== undefined) {
             result.archivedAt = this.archivedAt?.toISOString() ?? null;
         }
-        if (properties) {
-            const self = this;
-            for (const [propName, metadata] of properties) {
-                // Skip certain types
-                if (metadata.type === 'ephemeralProperty')
-                    continue;
-                if (metadata.type === 'referenceModel')
-                    continue;
-                if (metadata.type === 'referenceCollection')
-                    continue;
-                const value = self[propName];
-                if (value !== undefined) {
-                    result[propName] = value;
-                }
+        const self = this;
+        for (const [propName, metadata] of properties) {
+            // Skip certain types
+            if (metadata.type === 'ephemeralProperty')
+                continue;
+            if (metadata.type === 'referenceModel')
+                continue;
+            if (metadata.type === 'referenceCollection')
+                continue;
+            const value = self[propName];
+            if (value !== undefined) {
+                result[propName] = value;
             }
         }
         return result;
@@ -793,14 +789,12 @@ export class Model {
         if (hasActiveRegistry()) {
             const modelName = this.getModelName();
             const properties = getActiveRegistry().getProperties(modelName);
-            if (properties) {
-                const self = this;
-                for (const [propName, metadata] of properties) {
-                    if (metadata.type === 'referenceCollection') {
-                        const collection = self[propName];
-                        if (collection?.dispose) {
-                            collection.dispose();
-                        }
+            const self = this;
+            for (const [propName, metadata] of properties) {
+                if (metadata.type === 'referenceCollection') {
+                    const collection = self[propName];
+                    if (collection) {
+                        collection.dispose();
                     }
                 }
             }
@@ -825,11 +819,9 @@ export class Model {
         const snapshot = {};
         const modelName = this.getModelName();
         const properties = getActiveRegistry().getProperties(modelName);
-        if (properties) {
-            const json = this.toJSON();
-            for (const [propName] of properties) {
-                snapshot[propName] = json[propName];
-            }
+        const json = this.toJSON();
+        for (const [propName] of properties) {
+            snapshot[propName] = json[propName];
         }
         return snapshot;
     }
@@ -874,17 +866,15 @@ export class Model {
             snapshot.archivedAt = this.archivedAt;
         const properties = getActiveRegistry().getProperties(this.getModelName());
         const self = this;
-        if (properties) {
-            for (const [propName, metadata] of properties) {
-                if (metadata.type === 'ephemeralProperty' ||
-                    metadata.type === 'referenceModel' ||
-                    metadata.type === 'referenceCollection') {
-                    continue;
-                }
-                // Reading through the observable getter is the point: it subscribes the
-                // enclosing MobX reaction to this field.
-                snapshot[propName] = self[propName];
+        for (const [propName, metadata] of properties) {
+            if (metadata.type === 'ephemeralProperty' ||
+                metadata.type === 'referenceModel' ||
+                metadata.type === 'referenceCollection') {
+                continue;
             }
+            // Reading through the observable getter is the point: it subscribes the
+            // enclosing MobX reaction to this field.
+            snapshot[propName] = self[propName];
         }
         for (const name of this.getDerivedGetterNames()) {
             Object.defineProperty(snapshot, name, {

package/dist/local/NetworkMonitor.js

@@ -12,6 +12,8 @@ export class NetworkMonitor extends EventEmitter {
     // Only `navigator.onLine === false` means offline. Node 18+ exposes a global
     // `navigator` with `onLine === undefined`, so the naive `navigator.onLine`
     // would seed `false` (offline) on every server client — start optimistic.
+    // DOM types say `onLine` is boolean, but Node exposes it as undefined.
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-boolean-literal-compare
     isOnline = !(typeof navigator !== 'undefined' && navigator.onLine === false);
     lastOnlineCheck = new Date();
     constructor(runtime = globalRuntime) {

package/dist/local/RuntimeContext.js

@@ -47,6 +47,8 @@ export const browserOnlineStatus = {
         // signal. Don't use `!navigator.onLine`: Node 18+ exposes a global
         // `navigator` whose `onLine` is `undefined`, which `!` would read as offline —
         // wedging every Node/server client (agents, worker, MCP) into a false offline.
+        // DOM types say `onLine` is boolean, but Node exposes it as undefined.
+        // eslint-disable-next-line @typescript-eslint/no-unnecessary-boolean-literal-compare
         return !(typeof navigator !== 'undefined' && navigator.onLine === false);
     },
 };

package/dist/local/SyncClient.d.ts

@@ -21,6 +21,7 @@ import type { WriteOptions } from './interfaces/index.js';
 import { LogPosition } from './logPosition.js';
 import { type RehydrationStats, type SyncObserver, type SyncState } from './syncClientTypes.js';
 import type { BootstrapSnapshot } from './syncClientTypes.js';
+import { type BatchFileUploadOptions, type FileUploadOptions } from './fileUploads.js';
 export type { BootstrapSnapshot, RehydrationStats } from './syncClientTypes.js';
 export declare class SyncClient extends EventEmitter {
     private readonly runtime;
@@ -201,36 +202,11 @@ export declare class SyncClient extends EventEmitter {
     get gql(): import("./interfaces/index.js").MutationExecutor;
     /** Delete model (DELETE) - works offline */
     delete(model: Model, options?: WriteOptions): Promise<void> | undefined;
-    /**
-     * Upload a file and create its attachment record. The upload runs through
-     * the {@link MutationQueue}, and a model is built from the server's
-     * response and added to the pool.
-     */
-    uploadFile(file: File, options: {
-        id: string;
-        attachableType: string;
-        attachableId: string;
-        metadata?: Record<string, unknown>;
-    }): Promise<Model | null>;
-    /**
-     * Batch upload files — single GraphQL call + parallel S3 PUTs.
-     *
-     * Returns the raw `Model[]` built by the object pool (typename is
-     * determined by the payload the server returns — currently always
-     * `Attachment`). The SDK has no knowledge of app-specific model classes,
-     * so it cannot honestly claim a narrower return type; consumers that
-     * need an `Attachment[]` project through their own typed accessor
-     * (e.g. `store.query.attachments.findMany({ where: { id: IN ids } })`)
-     * after the upload resolves.
-     */
-    batchUploadFiles(files: File[], options: {
-        ids: string[];
-        attachableType: string;
-        attachableId: string;
-        metadata?: Record<string, unknown>;
-    }): Promise<Model[]>;
+    private fileUploadContext;
+    uploadFile(file: File, options: FileUploadOptions): Promise<Model | null>;
+    batchUploadFiles(files: File[], options: BatchFileUploadOptions): Promise<Model[]>;
     /** Archive model (ARCHIVE) - works offline */
-    archive(model: Model): void;
+    archive(model: Model): Promise<void> | undefined;
     /**
      * Append a mutation to the pending queue and schedule its sync work.
      *

package/dist/local/SyncClient.js

@@ -14,7 +14,7 @@ import { deepEqual, snapshotJsonValue } from '@abloatai/transaction/utils/json';
 // ModelRegistry instance accessed via this.objectPool.registry
 import { LoadStrategy } from '@abloatai/transaction/types';
 import { globalRuntime } from './context.js';
-import { AbloAuthenticationError, AbloError, AbloValidationError } from '@abloatai/transaction/errors';
+import { AbloError, AbloValidationError } from '@abloatai/transaction/errors';
 import { EventEmitter } from 'events';
 import { NetworkMonitor } from './NetworkMonitor.js';
 import { MutationQueue, } from './transactions/mutations/MutationQueue.js';
@@ -25,6 +25,8 @@ import { createLocalMutationPort } from './transactions/localMutation.js';
 import { createReconnectDrain } from './transactions/reconnectDrain.js';
 import { DatabaseCommitOutboxStore } from './transactions/databaseCommitOutbox.js';
 import { toEpochMs, } from './syncClientTypes.js';
+import { batchUploadFiles, uploadFile, } from './fileUploads.js';
+const ignoreSeparatelyObservedMutationFailure = () => undefined;
 export class SyncClient extends EventEmitter {
     runtime;
     objectPool;
@@ -856,7 +858,7 @@ export class SyncClient extends EventEmitter {
         if (capturedChanges === undefined)
             return;
         this.objectPool.upsert(model, ModelScope.live);
-        this.stageMutation('update', model, capturedChanges);
+        void this.stageMutation('update', model, capturedChanges);
         this.notifyObservers({
             type: 'update',
             modelType: model.getModelName(),
@@ -876,99 +878,27 @@ export class SyncClient extends EventEmitter {
         this.mutationQueue.cancelTransactionsForModel(model.id);
         return this.mutate('delete', model, () => this.objectPool.remove(model.id), options);
     }
-    /**
-     * Upload a file and create its attachment record. The upload runs through
-     * the {@link MutationQueue}, and a model is built from the server's
-     * response and added to the pool.
-     */
-    async uploadFile(file, options) {
-        if (!this.userId || !this.organizationId) {
-            throw new AbloAuthenticationError('Authentication required for file uploads', {
-                code: 'file_upload_auth_required',
-            });
-        }
-        try {
-            // Use MutationQueue to handle the upload mutation
-            const result = await this.mutationQueue.uploadAttachment(file, {
-                id: options.id,
-                attachableType: options.attachableType,
-                attachableId: options.attachableId,
-                metadata: options.metadata,
-            }, {
-                userId: this.userId,
-                organizationId: this.organizationId,
-            });
-            if (result) {
-                // Create model from response using ModelRegistry (generic — no concrete class import)
-                const model = this.objectPool.createFromData({
-                    id: options.id,
-                    ...result,
-                });
-                if (model) {
-                    this.objectPool.add(model, ModelScope.live);
-                    this.notifyObservers({
-                        type: 'create',
-                        modelType: model.getModelName(),
-                        model,
-                    });
-                    return model;
-                }
-            }
-            return null;
-        }
-        catch (error) {
-            this.runtime.observability.captureMutationFailure({
-                context: 'file-upload',
-                error: error instanceof Error ? error : new Error(String(error)),
-            });
-            throw error;
-        }
-    }
-    /**
-     * Batch upload files — single GraphQL call + parallel S3 PUTs.
-     *
-     * Returns the raw `Model[]` built by the object pool (typename is
-     * determined by the payload the server returns — currently always
-     * `Attachment`). The SDK has no knowledge of app-specific model classes,
-     * so it cannot honestly claim a narrower return type; consumers that
-     * need an `Attachment[]` project through their own typed accessor
-     * (e.g. `store.query.attachments.findMany({ where: { id: IN ids } })`)
-     * after the upload resolves.
-     */
-    async batchUploadFiles(files, options) {
-        if (!this.userId || !this.organizationId) {
-            throw new AbloAuthenticationError('Authentication required for file uploads', {
-                code: 'file_upload_auth_required',
-            });
-        }
-        const items = options.ids.map((id) => ({
-            id,
-            attachableType: options.attachableType,
-            attachableId: options.attachableId,
-            metadata: options.metadata,
-        }));
-        const results = await this.mutationQueue.batchUploadAttachments(files, items, {
+    fileUploadContext() {
+        return {
             userId: this.userId,
             organizationId: this.organizationId,
-        });
-        const models = [];
-        for (const result of results) {
-            const model = this.objectPool.createFromData({ ...result });
-            if (model) {
-                this.objectPool.add(model, ModelScope.live);
-                this.notifyObservers({
-                    type: 'create',
-                    modelType: model.getModelName(),
-                    model,
-                });
-                models.push(model);
-            }
-        }
-        return models;
+            mutationQueue: this.mutationQueue,
+            objectPool: this.objectPool,
+            observability: this.runtime.observability,
+            notifyCreated: (model) => {
+                this.notifyObservers({ type: 'create', modelType: model.getModelName(), model });
+            },
+        };
+    }
+    uploadFile(file, options) {
+        return uploadFile(this.fileUploadContext(), file, options);
+    }
+    batchUploadFiles(files, options) {
+        return batchUploadFiles(this.fileUploadContext(), files, options);
     }
     /** Archive model (ARCHIVE) - works offline */
     archive(model) {
-        this.mutate('archive', model, () => { this.objectPool.updateScope(model.id, ModelScope.archived); });
+        return this.mutate('archive', model, () => { this.objectPool.updateScope(model.id, ModelScope.archived); });
     }
     /**
      * Append a mutation to the pending queue and schedule its sync work.
@@ -999,7 +929,7 @@ export class SyncClient extends EventEmitter {
         // Most internal callers intentionally use fire-and-forget writes. Observe
         // their rejection without replacing the exact promise returned to model
         // operations that need authoritative per-transaction confirmation.
-        void confirmation.catch(() => undefined);
+        void confirmation.catch(ignoreSeparatelyObservedMutationFailure);
         const pending = staging.then(() => undefined).catch((error) => {
             this.runtime.observability.captureMutationFailure({
                 context: `stage-mutation-${type}`,
@@ -1225,14 +1155,11 @@ export class SyncClient extends EventEmitter {
     markConnected() {
         this.setConnectionState('connected');
         // Browser online state may have marked the client connected before the
-        // WebSocket itself was ready. Always kick both durable lanes on the real
-        // socket event, even when the high-level state did not change.
-        void this.drainPendingConfirmations().catch((error) => {
-            this.runtime.observability.captureMutationFailure({
-                context: 'restore-commit-outbox',
-                error: error instanceof Error ? error : new Error(String(error)),
-            });
-        });
+        // WebSocket itself was ready. Kick the durable lanes through the staging
+        // barrier: a model mutation enters the in-memory store before its journal
+        // row finishes saving, so a direct reconnect drain can otherwise try to
+        // seal a source record that does not exist yet. The pending drain also
+        // starts the atomic commit lane, so one ordered entry point covers both.
         void this.processPendingMutations();
     }
     drainPendingConfirmations() {

package/dist/local/client/createModelOperations.js

@@ -31,6 +31,8 @@ import { ModelScope } from '@abloatai/transaction/types';
 import { bindClaimLifetime, claimLifetimeOf, } from '@abloatai/transaction/claims/lifetime';
 import { claimQueueView, resolveClaimContentionOptions, } from '@abloatai/transaction/client/resources/modelOperations';
 import { capturePointRead, prepareReadSet, } from '@abloatai/transaction/internal/read-set';
+const ignoreSeparatelyObservedMutationFailure = () => undefined;
+const ignoreBestEffortClaimReleaseFailure = () => undefined;
 const modelClientMeta = new WeakMap();
 export function getModelClientMeta(modelClient) {
     if (typeof modelClient !== 'object' || modelClient === null)
@@ -113,7 +115,7 @@ hydration, collaboration, readSetContext) {
             // await does not create an unhandled-rejection process error. Returning
             // the original promise preserves normal rejection for callers that do
             // await or attach their own catch handler.
-            void confirmation.catch(() => undefined);
+            void confirmation.catch(ignoreSeparatelyObservedMutationFailure);
             return confirmation;
         };
     };
@@ -226,9 +228,9 @@ hydration, collaboration, readSetContext) {
         // This runs after authoritative confirmation. A best-effort abandon frame
         // cannot turn a committed write into an apparent failure; the server has
         // already fulfilled the participant's claims as part of that commit.
-        await releaseClaimsForEntity(entityId).catch(() => undefined);
+        await releaseClaimsForEntity(entityId).catch(ignoreBestEffortClaimReleaseFailure);
         if (explicit && !explicitWasLocal) {
-            await explicit.release?.().catch(() => undefined);
+            await explicit.release?.().catch(ignoreBestEffortClaimReleaseFailure);
         }
     };
     const takeClaim = async (params) => {
@@ -819,7 +821,7 @@ hydration, collaboration, readSetContext) {
             return modelAsRow(model);
         }
         finally {
-            await autoLease?.release?.().catch(() => { });
+            await autoLease?.release?.().catch(ignoreBestEffortClaimReleaseFailure);
         }
     });
     function createRows(params) {

package/dist/local/fileUploads.d.ts

@@ -0,0 +1,27 @@
+/** File-upload behavior owned beneath the SyncClient boundary. */
+import type { RuntimeContext } from './RuntimeContext.js';
+import { Model } from './Model.js';
+import { InstanceCache } from './InstanceCache.js';
+import type { MutationQueue } from './transactions/mutations/MutationQueue.js';
+export interface FileUploadOptions {
+    readonly id: string;
+    readonly attachableType: string;
+    readonly attachableId: string;
+    readonly metadata?: Record<string, unknown>;
+}
+export interface BatchFileUploadOptions {
+    readonly ids: string[];
+    readonly attachableType: string;
+    readonly attachableId: string;
+    readonly metadata?: Record<string, unknown>;
+}
+export interface FileUploadContext {
+    readonly userId: string | null;
+    readonly organizationId: string | null;
+    readonly mutationQueue: MutationQueue;
+    readonly objectPool: InstanceCache;
+    readonly observability: RuntimeContext['observability'];
+    readonly notifyCreated: (model: Model) => void;
+}
+export declare function uploadFile(context: FileUploadContext, file: File, options: FileUploadOptions): Promise<Model | null>;
+export declare function batchUploadFiles(context: FileUploadContext, files: File[], options: BatchFileUploadOptions): Promise<Model[]>;

package/dist/local/fileUploads.js

@@ -0,0 +1,55 @@
+/** File-upload behavior owned beneath the SyncClient boundary. */
+import { AbloAuthenticationError } from '@abloatai/transaction/errors';
+import { Model } from './Model.js';
+import { InstanceCache, ModelScope } from './InstanceCache.js';
+function authenticatedContext(context) {
+    if (!context.userId || !context.organizationId) {
+        throw new AbloAuthenticationError('Authentication required for file uploads', {
+            code: 'file_upload_auth_required',
+        });
+    }
+    return { userId: context.userId, organizationId: context.organizationId };
+}
+function acceptUploadedModel(context, data) {
+    const model = context.objectPool.createFromData(data);
+    if (!model)
+        return null;
+    context.objectPool.add(model, ModelScope.live);
+    context.notifyCreated(model);
+    return model;
+}
+export async function uploadFile(context, file, options) {
+    const identity = authenticatedContext(context);
+    try {
+        const result = await context.mutationQueue.uploadAttachment(file, {
+            id: options.id,
+            attachableType: options.attachableType,
+            attachableId: options.attachableId,
+            metadata: options.metadata,
+        }, identity);
+        return result
+            ? acceptUploadedModel(context, { id: options.id, ...result })
+            : null;
+    }
+    catch (error) {
+        context.observability.captureMutationFailure({
+            context: 'file-upload',
+            error: error instanceof Error ? error : new Error(String(error)),
+        });
+        throw error;
+    }
+}
+export async function batchUploadFiles(context, files, options) {
+    const identity = authenticatedContext(context);
+    const items = options.ids.map((id) => ({
+        id,
+        attachableType: options.attachableType,
+        attachableId: options.attachableId,
+        metadata: options.metadata,
+    }));
+    const results = await context.mutationQueue.batchUploadAttachments(files, items, identity);
+    return results.flatMap((result) => {
+        const model = acceptUploadedModel(context, { ...result });
+        return model ? [model] : [];
+    });
+}

package/dist/local/stores/syncAction.d.ts

@@ -10,10 +10,10 @@ export declare const syncActionSchema: z.ZodObject<{
     modelName: z.ZodString;
     modelId: z.ZodString;
     action: z.ZodEnum<{
-        A: "A";
         I: "I";
         U: "U";
         D: "D";
+        A: "A";
         V: "V";
         C: "C";
         G: "G";

package/dist/local/sync/contextOnChange.js

@@ -37,7 +37,7 @@ export function contextOnChange(transport, pool, reads, listener) {
     // already advanced this exact row in the pool.
     for (const read of rowReads) {
         const resident = pool.peek(read.id);
-        if (!resident || resident.getModelName().toLowerCase() !== read.model.toLowerCase()) {
+        if (resident?.getModelName().toLowerCase() !== read.model.toLowerCase()) {
             continue;
         }
         const observed = pool.watermarks.of(resident);

package/dist/local/sync/createClaimStream.js

@@ -284,7 +284,7 @@ export function createClaimStream(config, transport = null) {
             }
             ownClaims.clear();
             for (const claimId of [...pendingHeartbeats.keys()]) {
-                settleHeartbeat(claimId, ({ reject }) => reject(error));
+                settleHeartbeat(claimId, ({ reject }) => { reject(error); });
             }
         }));
     }

package/dist/local/sync/deltaPipeline.js

@@ -67,7 +67,9 @@ export function deduplicateDeltas(deltas) {
         return deltas;
     let strictlyOrdered = true;
     for (let index = 1; index < deltas.length; index += 1) {
-        if (deltas[index - 1].id >= deltas[index].id) {
+        const previous = deltas[index - 1];
+        const current = deltas[index];
+        if (!previous || !current || previous.id >= current.id) {
             strictlyOrdered = false;
             break;
         }
@@ -277,10 +279,13 @@ export function sliceApplyChanges(changes, maxDeltas) {
     while (index < changes.length) {
         // The indivisible unit starting here: one transaction's run, or a single
         // untransacted change.
-        const transactionId = changes[index].transactionId;
+        const change = changes[index];
+        if (!change)
+            break;
+        const transactionId = change.transactionId;
         let end = index + 1;
         if (transactionId !== undefined) {
-            while (end < changes.length && changes[end].transactionId === transactionId)
+            while (changes[end]?.transactionId === transactionId)
                 end += 1;
         }
         const groupSize = end - index;
@@ -327,9 +332,11 @@ async function flushDeltaBatchInner(ctx, queuedDeltas) {
     if (customDeltas.length > 0) {
         runInAction(() => {
             for (const delta of customDeltas) {
+                if (delta.data === null)
+                    continue;
                 const data = typeof delta.data === 'string'
                     ? JSON.parse(delta.data)
-                    : (delta.data);
+                    : delta.data;
                 // 'C' (Covering) is treated identically to 'I' here — the client
                 // gained permission to see the entity, so we insert it into the
                 // pool as if newly created.
@@ -395,7 +402,7 @@ async function flushDeltaBatchInner(ctx, queuedDeltas) {
         // slice. Slices stay the atomicity unit; the budget only decides where
         // the loop breathes.
         let sliceStartedAt = performance.now();
-        for (let index = 0; index < slices.length; index++) {
+        for (const [index, slice] of slices.entries()) {
             if (index > 0 && performance.now() - sliceStartedAt > APPLY_YIELD_BUDGET_MS) {
                 pipelineDebug.phase = `apply-yield-${index}`;
                 pipelineDebug.applyYields += 1;
@@ -404,7 +411,6 @@ async function flushDeltaBatchInner(ctx, queuedDeltas) {
             }
             pipelineDebug.phase = `apply-slice-${index}`;
             pipelineDebug.applySlices += 1;
-            const slice = slices[index];
             if (hasApplyPlugins) {
                 runStage(stagePlugins, 'apply', { changes: slice });
             }