@forcecalendar/core 2.0.0 → 2.1.0

package/core/events/Event.js

@@ -374,7 +374,7 @@ export class Event {
    * @returns {boolean} True if event spans multiple days
    */
   get isMultiDay() {
-    if (!this._cache.hasOwnProperty('isMultiDay')) {
+    if (!Object.prototype.hasOwnProperty.call(this._cache, 'isMultiDay')) {
       const startDay = this.start.toDateString();
       const endDay = this.end.toDateString();
       this._cache.isMultiDay = startDay !== endDay;
@@ -417,10 +417,10 @@ export class Event {
       dayEnd.setHours(23, 59, 59, 999);
 
       return this.start <= dayEnd && this.end >= dayStart;
-    } else {
+    } 
       // Single day event: check if it's on the same day
       return startString === dateString;
-    }
+    
   }
 
   /**
@@ -436,9 +436,9 @@ export class Event {
     } else if (otherEvent && otherEvent.start && otherEvent.end) {
       // Allow checking against time ranges
       return !(this.end <= otherEvent.start || this.start >= otherEvent.end);
-    } else {
+    } 
       throw new Error('Parameter must be an Event instance or have start/end properties');
-    }
+    
   }
 
   /**

package/core/events/EventStore.js

@@ -226,12 +226,12 @@ export class EventStore {
     }
 
     // Filter by all-day events
-    if (filters.hasOwnProperty('allDay')) {
+    if (Object.prototype.hasOwnProperty.call(filters, 'allDay')) {
       results = results.filter(event => event.allDay === filters.allDay);
     }
 
     // Filter by recurring
-    if (filters.hasOwnProperty('recurring')) {
+    if (Object.prototype.hasOwnProperty.call(filters, 'recurring')) {
       results = results.filter(event => event.recurring === filters.recurring);
     }
 
@@ -250,7 +250,7 @@ export class EventStore {
     }
 
     // Filter by having attendees
-    if (filters.hasOwnProperty('hasAttendees')) {
+    if (Object.prototype.hasOwnProperty.call(filters, 'hasAttendees')) {
       results = results.filter(event => filters.hasAttendees ? event.hasAttendees : !event.hasAttendees);
     }
 

package/core/events/RecurrenceEngine.js

@@ -40,7 +40,7 @@ export class RecurrenceEngine {
     let lastOffset = tzManager.getTimezoneOffset(currentDate, eventTimezone);
 
     // Track last date to detect infinite loop (date not advancing)
-    let lastDateTimestamp = null;
+    const lastDateTimestamp = null;
     let stuckCount = 0;
     const maxStuckIterations = 3;
 
@@ -267,11 +267,11 @@ export class RecurrenceEngine {
           return Math.abs(exceptionDate.getTime() - dateTime) < 1000; // Within 1 second
         }
         return exceptionDate.toDateString() === dateStr;
-      } else {
+      } 
         // Simple date exception
         const exceptionDate = exDate instanceof Date ? exDate : new Date(exDate);
         return exceptionDate.toDateString() === dateStr;
-      }
+      
     });
   }
 
@@ -327,9 +327,9 @@ export class RecurrenceEngine {
 
       if (dateStr.endsWith('Z')) {
         return new Date(Date.UTC(year, month, day, hour, minute, second));
-      } else {
+      } 
         return new Date(year, month, day, hour, minute, second);
-      }
+      
     }
 
     // Fallback to standard date parsing

package/core/events/RecurrenceEngineV2.js

@@ -609,7 +609,7 @@ export class RecurrenceEngineV2 {
      */
     clearEventCache(eventId) {
         for (const key of this.occurrenceCache.keys()) {
-            if (key.startsWith(eventId + '_')) {
+            if (key.startsWith(`${eventId  }_`)) {
                 this.occurrenceCache.delete(key);
             }
         }

package/core/ics/ICSParser.js

@@ -39,7 +39,7 @@ export class ICSParser {
         let inEvent = false;
         let inAlarm = false;
 
-        for (let line of lines) {
+        for (const line of lines) {
             // Skip empty lines
             if (!line.trim()) continue;
 
@@ -248,7 +248,7 @@ export class ICSParser {
                 event.category = value.split(',')[0]; // Take first category
                 break;
 
-            case 'STATUS':
+            case 'STATUS': {
                 const statusMap = {
                     'TENTATIVE': 'tentative',
                     'CONFIRMED': 'confirmed',
@@ -256,6 +256,7 @@ export class ICSParser {
                 };
                 event.status = statusMap[value] || 'confirmed';
                 break;
+            }
 
             case 'TRANSP':
                 event.showAs = value === 'TRANSPARENT' ? 'free' : 'busy';
@@ -265,7 +266,7 @@ export class ICSParser {
                 event.organizer = value.replace('mailto:', '');
                 break;
 
-            case 'ATTENDEE':
+            case 'ATTENDEE': {
                 if (!event.attendees) event.attendees = [];
                 const email = value.replace('mailto:', '');
                 event.attendees.push({
@@ -273,6 +274,7 @@ export class ICSParser {
                     name: email.split('@')[0] // Use email prefix as name
                 });
                 break;
+            }
 
             case 'RRULE':
                 event.recurrence = value;
@@ -308,10 +310,10 @@ export class ICSParser {
         if (dateString.endsWith('Z')) {
             // UTC time
             return new Date(Date.UTC(year, month, day, hour, minute, second));
-        } else {
+        } 
             // Local time
             return new Date(year, month, day, hour, minute, second);
-        }
+        
     }
 
     /**
@@ -365,7 +367,7 @@ export class ICSParser {
             // Continuation lines (with space prefix)
             while (remaining.length > 0) {
                 const chunk = remaining.substr(0, this.maxLineLength - 1);
-                folded.push(' ' + chunk);
+                folded.push(` ${  chunk}`);
                 remaining = remaining.substr(chunk.length);
             }
 

package/core/search/EventSearch.js

@@ -288,7 +288,7 @@ export class EventSearch {
         const events = this.eventStore.getAllEvents();
 
         for (const event of events) {
-            const value = event[field] || (includeEmpty ? '(No ' + field + ')' : null);
+            const value = event[field] || (includeEmpty ? `(No ${  field  })` : null);
             if (value === null) continue;
 
             if (!groups.has(value)) {

package/core/timezone/TimezoneManager.js

@@ -343,13 +343,13 @@ export class TimezoneManager {
         if (!tzString) return 'UTC';
 
         // Check if it's already an IANA identifier
-        if (this.database.timezones.hasOwnProperty(tzString)) {
+        if (Object.prototype.hasOwnProperty.call(this.database.timezones, tzString)) {
             return tzString;
         }
 
         // Check abbreviations
         const upperTz = tzString.toUpperCase();
-        if (this.database.abbreviations && this.database.abbreviations.hasOwnProperty(upperTz)) {
+        if (this.database.abbreviations && Object.prototype.hasOwnProperty.call(this.database.abbreviations, upperTz)) {
             return this.database.abbreviations[upperTz];
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forcecalendar/core",
-  "version": "2.0.0",
+  "version": "2.1.0",
   "type": "module",
   "private": false,
   "description": "A modern, lightweight, framework-agnostic calendar engine optimized for Salesforce",
@@ -11,7 +11,12 @@
   "scripts": {
     "test": "node tests/run-all.js",
     "test:ics": "node tests/integration/test-ics.js",
-    "test:search": "node tests/integration/test-search.js"
+    "test:search": "node tests/integration/test-search.js",
+    "lint": "eslint core/ --ext .js",
+    "lint:fix": "eslint core/ --ext .js --fix",
+    "format": "prettier --write \"core/**/*.js\"",
+    "format:check": "prettier --check \"core/**/*.js\"",
+    "quality": "npm run lint && npm run format:check"
   },
   "exports": {
     ".": "./core/index.js",