@forcecalendar/interface 1.0.30 → 1.0.32

package/package.json

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

package/src/components/EventForm.js

@@ -347,7 +347,10 @@ export class EventForm extends BaseComponent {
       this.endInput.value = this.formatDateForInput(this._formData.end);
       this.updateColorSelection();
 
-      // Focus trapping
+      // Clean up previous focus trap before creating a new one
+      if (this._cleanupFocusTrap) {
+        this._cleanupFocusTrap();
+      }
       this._cleanupFocusTrap = DOMUtils.trapFocus(this.modalContent);
     }
   }

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;
@@ -71,7 +82,11 @@ class StateManager {
 
   // State management
   getState() {
-    return { ...this.state };
+    return {
+      ...this.state,
+      config: { ...this.state.config },
+      events: [...this.state.events]
+    };
   }
 
   setState(updates, options = {}) {
@@ -236,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;