@forcecalendar/interface 1.0.31 → 1.0.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forcecalendar/interface",
-  "version": "1.0.31",
+  "version": "1.0.33",
   "type": "module",
   "description": "Official interface layer for forceCalendar Core - Enterprise calendar components",
   "main": "dist/force-calendar-interface.umd.js",

package/src/components/ForceCalendar.js

@@ -91,21 +91,23 @@ export class ForceCalendar extends BaseComponent {
         forwardEventAction('remove', data);
       })
     );
+    // Specific lifecycle events — do NOT call forwardEventAction here; the
+    // canonical event:add/update/remove handlers above already forward the
+    // generic CustomEvent.  These handlers emit only the specific variant so
+    // consumers that care about the distinction can subscribe to it without
+    // receiving the generic event a second time.
     this._busUnsubscribers.push(
       eventBus.on('event:added', data => {
-        forwardEventAction('add', data);
         this.emit('calendar-event-added', data);
       })
     );
     this._busUnsubscribers.push(
       eventBus.on('event:updated', data => {
-        forwardEventAction('update', data);
         this.emit('calendar-event-updated', data);
       })
     );
     this._busUnsubscribers.push(
       eventBus.on('event:deleted', data => {
-        forwardEventAction('remove', data);
         this.emit('calendar-event-deleted', data);
       })
     );

package/src/core/StateManager.js

@@ -46,12 +46,21 @@ class StateManager {
 
   /**
    * Sync state.events from Core calendar (single source of truth)
-   * This ensures state.events always matches Core's event store
+   * This ensures state.events always matches Core's event store.
+   *
+   * @param {object} options
+   * @param {boolean} options.silent  - suppress subscriber notifications
+   * @param {boolean} options.force   - always update even when IDs match
+   *                                    (required after updateEvent where IDs
+   *                                    are unchanged but content has changed)
    */
   _syncEventsFromCore(options = {}) {
+    const { force = false } = options;
     const coreEvents = this.calendar.getEvents() || [];
-    // Only update if different to avoid unnecessary re-renders
+    // Skip the update when nothing changed, unless the caller forces a sync
+    // (e.g. after updateEvent where IDs are the same but content differs)
     if (
+      force ||
       this.state.events.length !== coreEvents.length ||
       !this._eventsMatch(this.state.events, coreEvents)
     ) {
@@ -61,7 +70,9 @@ class StateManager {
   }
 
   /**
-   * Check if two event arrays have the same events (by id)
+   * Check if two event arrays have the same events by id.
+   * Only used for add/delete guards — updateEvent must pass force:true
+   * to bypass this check because IDs are unchanged after an update.
    */
   _eventsMatch(arr1, arr2) {
     if (arr1.length !== arr2.length) return false;
@@ -240,8 +251,9 @@ class StateManager {
       return null;
     }
 
-    // Sync from Core to ensure consistency (single source of truth)
-    this._syncEventsFromCore();
+    // Force sync from Core — IDs are unchanged after an update so the
+    // ID-only guard in _eventsMatch would otherwise skip the state update
+    this._syncEventsFromCore({ force: true });
     eventBus.emit('event:update', { event });
     eventBus.emit('event:updated', { event });
     return event;