minotor 7.0.1 → 7.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "minotor",
-  "version": "7.0.1",
+  "version": "7.0.2",
   "description": "A lightweight client-side transit routing library.",
   "keywords": [
     "minotor",

package/src/routing/router.ts

@@ -190,6 +190,10 @@ export class Router {
         options.transportModes,
       );
       markedStops.clear();
+      const earliestArrivalAtAnyDestination = this.earliestArrivalAtAnyStop(
+        earliestArrivals,
+        destinations,
+      );
       // for each route that can be reached with at least round - 1 trips
       const reachableRoutesArray = Array.from(reachableRoutes.entries());
       for (let i = 0; i < reachableRoutesArray.length; i++) {
@@ -214,9 +218,7 @@ export class Router {
             if (
               currentDropOffType !== 'NOT_AVAILABLE' &&
               currentArrivalTime.isBefore(earliestArrivalAtCurrentStop) &&
-              currentArrivalTime.isBefore(
-                this.earliestArrivalAtAnyStop(earliestArrivals, destinations),
-              )
+              currentArrivalTime.isBefore(earliestArrivalAtAnyDestination)
             ) {
               const bestHopOnDepartureTime = route.departureFrom(
                 currentTrip.bestHopOnStop,

package/src/timetable/time.ts

@@ -193,9 +193,16 @@ export class Time {
     if (times.length === 0) {
       throw new Error('At least one Time instance is required.');
     }
-    return times.reduce((maxTime, currentTime) => {
-      return currentTime.isAfter(maxTime) ? currentTime : maxTime;
-    });
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    let maxTime = times[0]!;
+    for (let i = 1; i < times.length; i++) {
+      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+      if (times[i]!.minutesSinceMidnight > maxTime.minutesSinceMidnight) {
+        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+        maxTime = times[i]!;
+      }
+    }
+    return maxTime;
   }
 
   /**
@@ -208,9 +215,16 @@ export class Time {
     if (times.length === 0) {
       throw new Error('At least one Time instance is required.');
     }
-    return times.reduce((minTime, currentTime) => {
-      return currentTime.isBefore(minTime) ? currentTime : minTime;
-    });
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    let minTime = times[0]!;
+    for (let i = 1; i < times.length; i++) {
+      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+      if (times[i]!.minutesSinceMidnight < minTime.minutesSinceMidnight) {
+        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+        minTime = times[i]!;
+      }
+    }
+    return minTime;
   }
 
   /**
@@ -220,7 +234,7 @@ export class Time {
    * @returns True if the current Time instance is after the other Time instance, otherwise false.
    */
   isAfter(otherTime: Time): boolean {
-    return this.minutesSinceMidnight > otherTime.toMinutes();
+    return this.minutesSinceMidnight > otherTime.minutesSinceMidnight;
   }
 
   /**
@@ -230,7 +244,7 @@ export class Time {
    * @returns True if the current Time instance is before the other Time instance, otherwise false.
    */
   isBefore(otherTime: Time): boolean {
-    return this.minutesSinceMidnight < otherTime.toMinutes();
+    return this.minutesSinceMidnight < otherTime.minutesSinceMidnight;
   }
 
   /**
@@ -240,6 +254,6 @@ export class Time {
    * @returns True if the current Time instance is equal to the other Time instance, otherwise false.
    */
   equals(otherTime: Time): boolean {
-    return this.minutesSinceMidnight === otherTime.toMinutes();
+    return this.minutesSinceMidnight === otherTime.minutesSinceMidnight;
   }
 }

package/src/timetable/timetable.ts

@@ -184,9 +184,7 @@ export class Timetable {
         `Service route not found for route ID: ${route.serviceRoute()}`,
       );
     }
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const { routes, ...serviceRouteInfo } = serviceRoute;
-    return serviceRouteInfo;
+    return { type: serviceRoute.type, name: serviceRoute.name };
   }
 
   /**