@forcecalendar/core 2.3.0 → 2.5.0

package/core/events/EventStore.js

@@ -86,19 +86,11 @@ export class EventStore {
       this._indexEvent(event);
 
       // Notify listeners (batch if in batch mode)
-      if (this.isBatchMode) {
-        this.batchNotifications.push({
-          type: 'add',
-          event,
-          version: ++this.version
-        });
-      } else {
-        this._notifyChange({
-          type: 'add',
-          event,
-          version: ++this.version
-        });
-      }
+      this._queueChange({
+        type: 'add',
+        event,
+        version: ++this.version
+      });
 
       return event;
     });
@@ -106,35 +98,24 @@ export class EventStore {
 
   /**
    * Update an existing event
-   * @param {string} eventId - The event ID
+   *
+   * An occurrence id (see {@link Event.occurrenceId}) updates the recurring
+   * master the occurrence belongs to, i.e. the whole series.
+   * @param {string} eventId - Event id or occurrence id
    * @param {Partial<import('../types.js').EventData>} updates - Properties to update
-   * @returns {Event} The updated event
+   * @returns {Event} The updated event (the master for an occurrence id)
    * @throws {Error} If event not found
    */
   updateEvent(eventId, updates) {
-    const existingEvent = this.events.get(eventId);
+    const existingEvent = this.events.get(eventId) || this._resolveOccurrenceMaster(eventId);
     if (!existingEvent) {
       throw new Error(`Event with id ${eventId} not found`);
     }
 
-    // Remove old indices
-    this._unindexEvent(existingEvent);
-
     // Create updated event
     const updatedEvent = existingEvent.clone(updates);
 
-    // Store updated event
-    this.events.set(eventId, updatedEvent);
-
-    // Update cache with new event data
-    this.optimizer.cache(eventId, updatedEvent, 'event');
-
-    // Clear query and date range caches since results may have changed
-    this.optimizer.queryCache.clear();
-    this.optimizer.dateRangeCache.clear();
-
-    // Re-index
-    this._indexEvent(updatedEvent);
+    this._replaceEvent(existingEvent, updatedEvent);
 
     // Notify listeners
     this._notifyChange({
@@ -147,27 +128,46 @@ export class EventStore {
     return updatedEvent;
   }
 
+  /**
+   * Swap a stored event for a new instance with the same id, keeping
+   * indices and caches in sync. Does not notify listeners.
+   * @param {Event} existingEvent - Event currently in the store
+   * @param {Event} replacement - Event instance that takes its place
+   * @private
+   */
+  _replaceEvent(existingEvent, replacement) {
+    // Remove old indices
+    this._unindexEvent(existingEvent);
+
+    // Store replacement
+    this.events.set(replacement.id, replacement);
+
+    // Update cache with new event data
+    this.optimizer.cache(replacement.id, replacement, 'event');
+
+    // Clear query and date range caches since results may have changed
+    this.optimizer.queryCache.clear();
+    this.optimizer.dateRangeCache.clear();
+
+    // Re-index
+    this._indexEvent(replacement);
+  }
+
   /**
    * Remove an event from the store
-   * @param {string} eventId - The event ID to remove
+   *
+   * An occurrence id (see {@link Event.occurrenceId}) removes the recurring
+   * master the occurrence belongs to, i.e. the whole series.
+   * @param {string} eventId - Event id or occurrence id
    * @returns {boolean} True if removed, false if not found
    */
   removeEvent(eventId) {
-    const event = this.events.get(eventId);
+    const event = this.events.get(eventId) || this._resolveOccurrenceMaster(eventId);
     if (!event) {
       return false;
     }
 
-    // Remove from primary storage
-    this.events.delete(eventId);
-
-    // Invalidate caches
-    this.optimizer.eventCache.delete(eventId);
-    this.optimizer.queryCache.clear();
-    this.optimizer.dateRangeCache.clear();
-
-    // Remove from indices
-    this._unindexEvent(event);
+    this._detachEvent(event);
 
     // Notify listeners
     this._notifyChange({
@@ -179,10 +179,33 @@ export class EventStore {
     return true;
   }
 
+  /**
+   * Remove an event from storage, caches and indices. Does not notify listeners.
+   * @param {Event} event - Event currently in the store
+   * @private
+   */
+  _detachEvent(event) {
+    // Remove from primary storage
+    this.events.delete(event.id);
+
+    // Invalidate caches
+    this.optimizer.eventCache.delete(event.id);
+    this.optimizer.queryCache.clear();
+    this.optimizer.dateRangeCache.clear();
+
+    // Remove from indices
+    this._unindexEvent(event);
+  }
+
   /**
    * Get an event by ID
-   * @param {string} eventId - The event ID
-   * @returns {Event|null} The event or null if not found
+   *
+   * Occurrence ids produced by {@link EventStore#expandRecurringEvent}
+   * (`<masterId>_<startMs>`, see {@link Event.occurrenceId}) resolve to the
+   * stored recurring master they were derived from, so an id taken from view
+   * data can always be looked up. Occurrences themselves are never stored.
+   * @param {string} eventId - Event id or occurrence id
+   * @returns {Event|null} The stored event (the master for an occurrence id) or null
    */
   getEvent(eventId) {
     // Check cache first
@@ -197,9 +220,26 @@ export class EventStore {
     // Cache if found
     if (event) {
       this.optimizer.cache(eventId, event, 'event');
+      return event;
     }
 
-    return event;
+    return this._resolveOccurrenceMaster(eventId);
+  }
+
+  /**
+   * Resolve an occurrence id to the stored recurring master it belongs to.
+   * Not cached: the master's own cache entry is the one kept in sync.
+   * @param {string} eventId - Candidate occurrence id
+   * @returns {Event|null} The master event or null
+   * @private
+   */
+  _resolveOccurrenceMaster(eventId) {
+    const parsed = Event.parseOccurrenceId(eventId);
+    if (!parsed) {
+      return null;
+    }
+    const master = this.events.get(parsed.recurringEventId);
+    return master && master.recurring ? master : null;
   }
 
   /**
@@ -331,14 +371,103 @@ export class EventStore {
 
   /**
    * Get events for a specific date
+   *
+   * Recurring series are expanded for the day, so the result holds their
+   * occurrences (see {@link EventStore#expandRecurringEvent}) rather than the
+   * master events. When building a grid of days use
+   * {@link EventStore#getEventsByDate}, which expands once for the whole range.
    * @param {Date} date - The date to query
    * @param {string} [timezone] - Timezone for the query (defaults to store timezone)
    * @returns {Event[]} Events occurring on the date, sorted by start time
    */
   getEventsForDate(date, timezone = null) {
     timezone = timezone || this.defaultTimezone;
+    const dayStart = DateUtils.startOfDay(date);
+    const dayEnd = DateUtils.endOfDay(date);
+
+    const candidates = [];
+    for (const id of this._collectDateCandidateIds(date)) {
+      const event = this.events.get(id);
+      // Recurring masters are represented by their occurrences below
+      if (event && !event.recurring) {
+        candidates.push(event);
+      }
+    }
+
+    for (const id of this.indices.recurring) {
+      const event = this.events.get(id);
+      if (event) {
+        candidates.push(...this.expandRecurringEvent(event, dayStart, dayEnd, timezone));
+      }
+    }
+
+    return this._selectEventsForDay(candidates, dayStart, dayEnd, timezone);
+  }
+
+  /**
+   * Get the events for every day in a range, keyed by local date (YYYY-MM-DD)
+   *
+   * Recurring series are expanded once for the whole range rather than once
+   * per day, which is what a month or week grid needs. Every day in the range
+   * has an entry (an empty array when nothing occurs) and multi-day events
+   * appear under each day they span. Each array is sorted like
+   * {@link EventStore#getEventsForDate}.
+   *
+   * @example
+   * const byDate = store.getEventsByDate(gridStart, gridEnd);
+   * const events = byDate.get(DateUtils.getLocalDateString(cellDate)) || [];
+   *
+   * @param {Date} start - First day of the range
+   * @param {Date} end - Last day of the range
+   * @param {string} [timezone] - Timezone deciding which day an event falls on (defaults to store timezone)
+   * @returns {Map<string, Event[]>} Local date string -> events on that day
+   */
+  getEventsByDate(start, end, timezone = null) {
+    timezone = timezone || this.defaultTimezone;
+    const rangeStart = DateUtils.startOfDay(start);
+    const rangeEnd = DateUtils.endOfDay(end);
+
+    /** @type {Map<string, Event[]>} */
+    const byDate = new Map();
+    for (const day of DateUtils.getDateRange(rangeStart, rangeEnd)) {
+      byDate.set(DateUtils.getLocalDateString(day), []);
+    }
 
-    // Collect candidate event IDs from indices
+    // Query one day beyond each edge so events that fall on an edge day in
+    // the requested timezone are not lost to the UTC-based range filter.
+    const events = this.getEventsInRange(
+      DateUtils.addDays(rangeStart, -1),
+      DateUtils.addDays(rangeEnd, 1),
+      true,
+      timezone
+    );
+
+    for (const event of events) {
+      const eventStart = event.getStartInTimezone(timezone);
+      const eventEnd = event.getEndInTimezone(timezone);
+      const lastDay = eventEnd < rangeEnd ? eventEnd : rangeEnd;
+      let day = DateUtils.startOfDay(eventStart > rangeStart ? eventStart : rangeStart);
+      while (day <= lastDay) {
+        byDate.get(DateUtils.getLocalDateString(day))?.push(event);
+        day = DateUtils.addDays(day, 1);
+      }
+    }
+
+    const compare = this._compareByStart(timezone);
+    for (const dayEvents of byDate.values()) {
+      dayEvents.sort(compare);
+    }
+
+    return byDate;
+  }
+
+  /**
+   * Collect the ids of stored events that may occur on a date.
+   * @param {Date} date - The date to query
+   * @returns {Set<string>} Candidate event ids
+   * @private
+   */
+  _collectDateCandidateIds(date) {
     const candidateIds = new Set();
 
     // Check byDate index for nearby dates (handles most events)
@@ -362,35 +491,41 @@ export class EventStore {
       monthEventIds.forEach(id => candidateIds.add(id));
     }
 
-    // Filter candidates to events that actually overlap with the requested date
-    const allEvents = [];
-    const startOfDay = new Date(date);
-    startOfDay.setHours(0, 0, 0, 0);
-    const endOfDay = new Date(date);
-    endOfDay.setHours(23, 59, 59, 999);
+    return candidateIds;
+  }
 
-    for (const id of candidateIds) {
-      const event = this.events.get(id);
-      if (event) {
-        // Check if event actually occurs on the requested date in the given timezone
-        const eventStartLocal = event.getStartInTimezone(timezone);
-        const eventEndLocal = event.getEndInTimezone(timezone);
+  /**
+   * Keep the events that overlap a day in the given timezone, sorted by start.
+   * @param {Event[]} events - Candidate events
+   * @param {Date} dayStart - Start of the day
+   * @param {Date} dayEnd - End of the day
+   * @param {string} timezone - Timezone deciding whether an event falls on the day
+   * @returns {Event[]} Events on the day, sorted
+   * @private
+   */
+  _selectEventsForDay(events, dayStart, dayEnd, timezone) {
+    const onDay = events.filter(event => {
+      // Event overlaps with this day if it starts before end of day and ends after start of day
+      const eventStartLocal = event.getStartInTimezone(timezone);
+      const eventEndLocal = event.getEndInTimezone(timezone);
+      return eventStartLocal <= dayEnd && eventEndLocal >= dayStart;
+    });
 
-        // Event overlaps with this day if it starts before end of day and ends after start of day
-        if (eventStartLocal <= endOfDay && eventEndLocal >= startOfDay) {
-          allEvents.push(event);
-        }
-      }
-    }
+    return onDay.sort(this._compareByStart(timezone));
+  }
 
-    return allEvents.sort((a, b) => {
-      // Sort by start time in the specified timezone
-      const aStart = a.getStartInTimezone(timezone);
-      const bStart = b.getStartInTimezone(timezone);
-      const timeCompare = aStart - bStart;
+  /**
+   * Comparator ordering events by start time in a timezone, longer events first.
+   * @param {string} timezone - Timezone used for the start comparison
+   * @returns {(a: Event, b: Event) => number} Comparator
+   * @private
+   */
+  _compareByStart(timezone) {
+    return (a, b) => {
+      const timeCompare = a.getStartInTimezone(timezone) - b.getStartInTimezone(timezone);
       if (timeCompare !== 0) return timeCompare;
       return b.duration - a.duration; // Longer events first
-    });
+    };
   }
 
   /**
@@ -466,11 +601,19 @@ export class EventStore {
    * @returns {Array<Event[]>} Array of event groups that overlap
    */
   getOverlapGroups(date, timedOnly = true) {
-    let events = this.getEventsForDate(date);
+    return this.groupOverlappingEvents(this.getEventsForDate(date), timedOnly);
+  }
 
-    if (timedOnly) {
-      events = events.filter(e => !e.allDay);
-    }
+  /**
+   * Group a list of events into clusters of overlapping time slots
+   * Same result as {@link EventStore#getOverlapGroups} for events already fetched
+   * (for example one day of {@link EventStore#getEventsByDate}).
+   * @param {Event[]} events - Events to group; the array is not modified
+   * @param {boolean} [timedOnly=true] - Only include timed events (not all-day)
+   * @returns {Array<Event[]>} Array of event groups that overlap
+   */
+  groupOverlappingEvents(events, timedOnly = true) {
+    events = timedOnly ? events.filter(e => !e.allDay) : [...events];
 
     if (events.length === 0) return [];
 
@@ -632,6 +775,16 @@ export class EventStore {
 
   /**
    * Expand a recurring event into individual occurrences
+   *
+   * Returns every occurrence that overlaps the range, including ones that
+   * start before it but run into it (multi-day series). Each occurrence is an
+   * {@link Event} cloned from the master with:
+   * - `id` from {@link Event.occurrenceId} (`<masterId>_<startMs>`), stable
+   *   across ranges and resolvable with {@link EventStore#getEvent},
+   * - `isOccurrence: true`, `recurringEventId` and `occurrenceStart`,
+   * - `metadata.recurringEventId`, `metadata.occurrenceId` (same as `id`) and
+   *   `metadata.occurrenceIndex` (position within this expansion).
+   * Non-recurring events are returned as-is in a one-element array.
    * @param {Event} event - The recurring event
    * @param {Date} rangeStart - Start of the expansion range
    * @param {Date} rangeEnd - End of the expansion range
@@ -647,27 +800,141 @@ export class EventStore {
 
     // Expand in the event's timezone for accurate recurrence calculation
     const eventTimezone = event.timeZone || timezone;
-    const occurrences = this.recurrenceEngine.expandEvent(event, rangeStart, rangeEnd, {
+
+    // The engine selects occurrences by start, so look back one event
+    // duration to catch occurrences that began before the range but overlap it
+    const duration = Math.max(0, event.end - event.start);
+    const expandStart = new Date(rangeStart.getTime() - duration);
+    const occurrences = this.recurrenceEngine.expandEvent(event, expandStart, rangeEnd, {
       timezone: eventTimezone
     });
 
-    return occurrences.map((occurrence, index) => {
-      // Create a new event instance for each occurrence
-      const occurrenceEvent = event.clone({
-        id: `${event.id}_occurrence_${index}`,
-        start: occurrence.start,
-        end: occurrence.end,
-        timeZone: occurrence.timezone || eventTimezone,
-        metadata: {
-          ...event.metadata,
-          recurringEventId: event.id,
-          occurrenceId: occurrence.id,
-          occurrenceIndex: index
-        }
-      });
+    const expanded = [];
+    for (const occurrence of occurrences) {
+      if (occurrence.end < rangeStart || occurrence.start > rangeEnd) {
+        continue;
+      }
+      expanded.push(this._createOccurrence(event, occurrence, eventTimezone, expanded.length));
+    }
+
+    return expanded;
+  }
 
-      return occurrenceEvent;
+  /**
+   * Build the Event instance for one occurrence of a recurring master.
+   * @param {Event} event - The recurring master
+   * @param {{start: Date, end: Date, timezone?: string}} occurrence - Engine occurrence
+   * @param {string} eventTimezone - Timezone the series was expanded in
+   * @param {number} index - Position within the current expansion
+   * @returns {Event} Occurrence event
+   * @private
+   */
+  _createOccurrence(event, occurrence, eventTimezone, index) {
+    const occurrenceStart = new Date(occurrence.start);
+    const id = Event.occurrenceId(event.id, occurrenceStart);
+
+    const occurrenceEvent = event.clone({
+      id,
+      start: occurrenceStart,
+      end: new Date(occurrence.end),
+      timeZone: occurrence.timezone || eventTimezone,
+      metadata: {
+        ...event.metadata,
+        recurringEventId: event.id,
+        occurrenceId: id,
+        occurrenceIndex: index
+      }
     });
+
+    occurrenceEvent.isOccurrence = true;
+    occurrenceEvent.recurringEventId = event.id;
+    occurrenceEvent.occurrenceStart = new Date(occurrenceStart);
+
+    return occurrenceEvent;
+  }
+
+  /**
+   * Lazily iterate the occurrences of a stored event in chronological order.
+   *
+   * Occurrences come one at a time from the store's recurrence engine
+   * (RecurrenceEngineV2 by default), so taking the next few occurrences of
+   * an open-ended series does not expand the series. `after` and `before`
+   * are exclusive unless `inclusive` is set; see
+   * RecurrenceEngineV2.iterateOccurrences for the full semantics. The
+   * expansion timezone defaults to the event's, then the store's.
+   *
+   * @example
+   * for (const occurrence of store.iterateOccurrences('standup', { after: new Date() })) {
+   *   if (occurrence.start > deadline) break;
+   *   remind(occurrence);
+   * }
+   *
+   * @param {string} eventId - The event ID
+   * @param {import('../types.js').ExpandedOccurrenceIteratorOptions} [options={}] - Window and expansion options
+   * @returns {Generator<import('../types.js').ExpandedOccurrence, void, undefined>} Occurrences in chronological order
+   * @throws {Error} If no event with the ID exists
+   */
+  iterateOccurrences(eventId, options = {}) {
+    const query = this._occurrenceQuery(eventId, options);
+    return this.recurrenceEngine.iterateOccurrences(query.event, query.options);
+  }
+
+  /**
+   * First occurrence of a stored event after an instant, or null when the
+   * series has no occurrence after it. `after` is exclusive unless
+   * `options.inclusive` is set.
+   *
+   * @example
+   * const upcoming = store.getNextOccurrence('standup', new Date());
+   *
+   * @param {string} eventId - The event ID
+   * @param {Date|number} [after=null] - Instant to search from (defaults to the series start)
+   * @param {import('../types.js').ExpandedOccurrenceIteratorOptions} [options={}] - Further options
+   * @returns {import('../types.js').ExpandedOccurrence|null} The next occurrence, or null
+   * @throws {Error} If no event with the ID exists
+   */
+  getNextOccurrence(eventId, after = null, options = {}) {
+    const query = this._occurrenceQuery(eventId, options);
+    return this.recurrenceEngine.nextOccurrence(query.event, after, query.options);
+  }
+
+  /**
+   * The first `count` occurrences of a stored event inside a window,
+   * generated lazily. `count` is capped at the engine's
+   * MAX_OCCURRENCES_HARD_LIMIT.
+   *
+   * @example
+   * const nextFive = store.takeOccurrences('standup', 5, { after: new Date() });
+   *
+   * @param {string} eventId - The event ID
+   * @param {number} count - Maximum number of occurrences to return
+   * @param {import('../types.js').ExpandedOccurrenceIteratorOptions} [options={}] - Window and expansion options
+   * @returns {import('../types.js').ExpandedOccurrence[]} Up to `count` occurrences in chronological order
+   * @throws {Error} If no event with the ID exists
+   */
+  takeOccurrences(eventId, count, options = {}) {
+    const query = this._occurrenceQuery(eventId, options);
+    return this.recurrenceEngine.takeOccurrences(query.event, count, query.options);
+  }
+
+  /**
+   * Resolve an occurrence query to the stored event and its options, with
+   * the timezone defaulted as expandRecurringEvent does
+   * @param {string} eventId - The event ID
+   * @param {Object} options - Caller options
+   * @returns {{ event: Event, options: Object }}
+   * @throws {Error} If no event with the ID exists
+   * @private
+   */
+  _occurrenceQuery(eventId, options) {
+    const event = this.events.get(eventId);
+    if (!event) {
+      throw new Error(`Event with id ${eventId} not found`);
+    }
+    return {
+      event,
+      options: { ...options, timezone: options.timezone || event.timeZone || this.defaultTimezone }
+    };
   }
 
   /**
@@ -705,6 +972,118 @@ export class EventStore {
     this.commitBatch();
   }
 
+  /**
+   * Reconcile the store with a snapshot of events, applying only the differences.
+   *
+   * Compared with {@link EventStore#loadEvents} (clear + re-add everything) this:
+   * - keeps the existing {@link Event} instance for every entry that is
+   *   equivalent to the stored one (identity is preserved, no notification),
+   * - replaces stored events whose incoming data differs (`update` change),
+   * - adds events whose id is not in the store (`add` change),
+   * - removes stored events missing from the snapshot (`remove` change),
+   *   unless `removeMissing` is `false`,
+   * - emits a single `batch` notification listing those changes, or nothing at
+   *   all when the snapshot matches the store. When called while a batch is
+   *   already open the changes are queued on that batch instead.
+   *
+   * Input is validated up front: invalid event data or duplicate ids throw
+   * before the store is modified. Any error raised while applying the diff
+   * rolls the store back to its previous state.
+   *
+   * @example
+   * // periodic server snapshot
+   * const { added, updated, removed } = store.reconcile(rowsFromServer);
+   * if (added.length || updated.length || removed.length) rerender();
+   *
+   * @param {Array<Event|import('../types.js').EventData>} events - Complete snapshot of events
+   * @param {import('../types.js').ReconcileOptions} [options={}] - Reconcile options
+   * @returns {import('../types.js').ReconcileResult} Events that were added, updated, removed and left untouched
+   * @throws {Error} If an entry fails validation or two entries share an id
+   */
+  reconcile(events, options = {}) {
+    const { removeMissing = true, isEquivalent = Event.isEquivalent } = options;
+
+    if (!events || typeof events[Symbol.iterator] !== 'function') {
+      throw new Error('reconcile() expects an iterable of events');
+    }
+    if (typeof isEquivalent !== 'function') {
+      throw new Error('reconcile() option isEquivalent must be a function');
+    }
+
+    return this.optimizer.measure('reconcile', () => {
+      // Normalize and validate everything before touching the store
+      /** @type {Map<string, Event>} */
+      const incoming = new Map();
+      for (const eventData of events) {
+        const event = eventData instanceof Event ? eventData : new Event(eventData);
+        if (incoming.has(event.id)) {
+          throw new Error(`Duplicate event id in reconcile input: ${event.id}`);
+        }
+        incoming.set(event.id, event);
+      }
+
+      /** @type {import('../types.js').ReconcileResult} */
+      const result = { added: [], updated: [], removed: [], unchanged: [] };
+
+      // Nest inside an existing batch if one is open, otherwise own one
+      const ownsBatch = !this.isBatchMode;
+      if (ownsBatch) {
+        this.startBatch(true);
+      }
+
+      try {
+        if (removeMissing) {
+          for (const existing of Array.from(this.events.values())) {
+            if (!incoming.has(existing.id)) {
+              this._detachEvent(existing);
+              this._queueChange({ type: 'remove', event: existing, version: ++this.version });
+              result.removed.push(existing);
+            }
+          }
+        }
+
+        for (const event of incoming.values()) {
+          const existing = this.events.get(event.id);
+          if (!existing) {
+            this.events.set(event.id, event);
+            this.optimizer.cache(event.id, event, 'event');
+            this._indexEvent(event);
+            this._queueChange({ type: 'add', event, version: ++this.version });
+            result.added.push(event);
+          } else if (existing === event || isEquivalent(existing, event)) {
+            result.unchanged.push(existing);
+          } else {
+            this._replaceEvent(existing, event);
+            this._queueChange({
+              type: 'update',
+              event,
+              oldEvent: existing,
+              version: ++this.version
+            });
+            result.updated.push({ event, oldEvent: existing });
+          }
+        }
+
+        if (result.added.length > 0) {
+          // Newly indexed events may change range/query results
+          this.optimizer.queryCache.clear();
+          this.optimizer.dateRangeCache.clear();
+        }
+      } catch (error) {
+        if (ownsBatch) {
+          this.rollbackBatch();
+        }
+        throw error;
+      }
+
+      if (ownsBatch) {
+        this.commitBatch();
+      }
+
+      return result;
+    });
+  }
+
   /**
    * Subscribe to store changes
    * @param {Function} callback - Callback function
@@ -949,6 +1328,19 @@ export class EventStore {
    * Notify listeners of changes
    * @private
    */
+  /**
+   * Deliver a change now, or queue it when a batch is open
+   * @param {import('../types.js').EventStoreChange} change - Change to deliver
+   * @private
+   */
+  _queueChange(change) {
+    if (this.isBatchMode) {
+      this.batchNotifications.push(change);
+    } else {
+      this._notifyChange(change);
+    }
+  }
+
   _notifyChange(change) {
     for (const listener of this.listeners) {
       try {
@@ -1066,7 +1458,7 @@ export class EventStore {
       this.batchBackup = null;
 
       // Clear cache
-      this.optimizer.clearCache();
+      this.clearCaches();
     }
 
     this.batchNotifications = [];