@forcecalendar/core 1.0.4 → 1.0.6

package/core/calendar/Calendar.js

@@ -15,8 +15,8 @@ export class Calendar {
    * @param {import('../../types.js').CalendarConfig} [config={}] - Configuration options
    */
   constructor(config = {}) {
-    // Initialize timezone manager first
-    this.timezoneManager = new TimezoneManager();
+    // Initialize timezone manager first (use singleton to share cache)
+    this.timezoneManager = TimezoneManager.getInstance();
 
     // Initialize configuration
     this.config = {

package/core/calendar/DateUtils.js

@@ -37,8 +37,9 @@ export class DateUtils {
     const day = result.getDay();
     const diff = (day < weekStartsOn ? 7 : 0) + day - weekStartsOn;
 
-    // Use setTime to handle month/year boundaries correctly
-    result.setTime(result.getTime() - (diff * 24 * 60 * 60 * 1000));
+    // Use setDate() to handle DST transitions correctly
+    // (millisecond arithmetic fails when days are 23 or 25 hours)
+    result.setDate(result.getDate() - diff);
     result.setHours(0, 0, 0, 0);
     return result;
   }
@@ -51,8 +52,9 @@ export class DateUtils {
    */
   static endOfWeek(date, weekStartsOn = 0) {
     const result = DateUtils.startOfWeek(date, weekStartsOn);
-    // Use setTime to handle month/year boundaries correctly
-    result.setTime(result.getTime() + (6 * 24 * 60 * 60 * 1000));
+    // Use setDate() to handle DST transitions correctly
+    // (millisecond arithmetic fails when days are 23 or 25 hours)
+    result.setDate(result.getDate() + 6);
     result.setHours(23, 59, 59, 999);
     return result;
   }
@@ -101,8 +103,9 @@ export class DateUtils {
    */
   static addDays(date, days) {
     const result = new Date(date);
-    // Use setTime to handle month/year boundaries correctly
-    result.setTime(result.getTime() + (days * 24 * 60 * 60 * 1000));
+    // Use setDate() to handle DST transitions correctly
+    // (millisecond arithmetic fails when days are 23 or 25 hours)
+    result.setDate(result.getDate() + days);
     return result;
   }
 
@@ -402,8 +405,9 @@ export class DateUtils {
 
     while (current.getTime() <= endTime) {
       dates.push(new Date(current));
-      // Use setTime to handle month/year boundaries correctly
-      current.setTime(current.getTime() + (24 * 60 * 60 * 1000));
+      // Use setDate() to handle DST transitions correctly
+      // (millisecond arithmetic fails when days are 23 or 25 hours)
+      current.setDate(current.getDate() + 1);
     }
 
     return dates;

package/core/events/Event.js

@@ -221,8 +221,9 @@ export class Event {
     this.id = normalized.id;
     this.title = normalized.title;
 
-    // Initialize timezone manager
-    this._timezoneManager = new TimezoneManager();
+    // Use shared timezone manager singleton to avoid memory bloat
+    // (previously each Event created its own TimezoneManager instance)
+    this._timezoneManager = TimezoneManager.getInstance();
 
     // Timezone handling
     // Store the timezone the event was created in (wall-clock time)

package/core/events/EventStore.js

@@ -30,8 +30,8 @@ export class EventStore {
       byStatus: new Map()
     };
 
-    // Timezone manager for conversions
-    this.timezoneManager = new TimezoneManager();
+    // Timezone manager for conversions (use singleton to share cache)
+    this.timezoneManager = TimezoneManager.getInstance();
 
     // Default timezone for the store (can be overridden)
     this.defaultTimezone = config.timezone || this.timezoneManager.getSystemTimezone();
@@ -367,7 +367,8 @@ export class EventStore {
     // Collect all events from those dates
     const checkedIds = new Set();
     dates.forEach(date => {
-      const dateStr = date.toDateString();
+      // Use getLocalDateString to match the index key format (YYYY-MM-DD)
+      const dateStr = DateUtils.getLocalDateString(date);
       const eventIds = this.indices.byDate.get(dateStr) || new Set();
 
       eventIds.forEach(id => {

package/core/events/RecurrenceEngine.js

@@ -25,7 +25,7 @@ export class RecurrenceEngine {
     const occurrences = [];
     const duration = event.end - event.start;
     const eventTimezone = timezone || event.timeZone || 'UTC';
-    const tzManager = new TimezoneManager();
+    const tzManager = TimezoneManager.getInstance();
 
     // Work in event's timezone for accurate recurrence calculation
     let currentDate = new Date(event.start);

package/core/events/RecurrenceEngineV2.js

@@ -8,7 +8,8 @@ import { RRuleParser } from './RRuleParser.js';
 
 export class RecurrenceEngineV2 {
     constructor() {
-        this.tzManager = new TimezoneManager();
+        // Use singleton to share cache across all components
+        this.tzManager = TimezoneManager.getInstance();
 
         // Cache for expanded occurrences
         this.occurrenceCache = new Map();

package/core/timezone/TimezoneManager.js

@@ -7,7 +7,33 @@
 
 import { TimezoneDatabase } from './TimezoneDatabase.js';
 
+// Singleton instance for shared use across the application
+let sharedInstance = null;
+
 export class TimezoneManager {
+    /**
+     * Get the shared singleton instance of TimezoneManager
+     * This should be used instead of creating new instances to avoid memory bloat
+     * @returns {TimezoneManager} The shared instance
+     */
+    static getInstance() {
+        if (!sharedInstance) {
+            sharedInstance = new TimezoneManager();
+        }
+        return sharedInstance;
+    }
+
+    /**
+     * Reset the singleton instance (useful for testing)
+     * @private
+     */
+    static _resetInstance() {
+        if (sharedInstance) {
+            sharedInstance.clearCache();
+        }
+        sharedInstance = null;
+    }
+
     constructor() {
         // Initialize comprehensive timezone database
         this.database = new TimezoneDatabase();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forcecalendar/core",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "type": "module",
   "private": false,
   "description": "A modern, lightweight, framework-agnostic calendar engine optimized for Salesforce",