@trafficgroup/knex-rel 0.1.9 → 0.1.11-rc0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trafficgroup/knex-rel",
-  "version": "0.1.9",
+  "version": "0.1.11-rc0",
   "description": "Knex Module",
   "main": "dist/index.js",
   "scripts": {

package/src/constants/video.constants.ts

@@ -0,0 +1,19 @@
+/**
+ * Video sorting constants
+ * Shared between DAO and API controller to ensure validation consistency
+ */
+
+export const VALID_VIDEO_SORT_FIELDS = ['created_at', 'name', 'status'] as const;
+export type VideoSortField = typeof VALID_VIDEO_SORT_FIELDS[number];
+
+export const VALID_SORT_ORDERS = ['ASC', 'DESC'] as const;
+export type SortOrder = typeof VALID_SORT_ORDERS[number];
+
+/**
+ * Maps API sort field names to database column names with table aliases
+ */
+export const VIDEO_SORT_COLUMN_MAP: Record<VideoSortField, string> = {
+    'created_at': 'v.created_at',
+    'name': 'v.name',
+    'status': 'v.status'
+};

package/src/d.types.ts

@@ -1,22 +1,18 @@
 export interface IDataPaginator<T> {
-  success: boolean;
-  data: T[];
-  page: number;
-  limit: number;
-  count: number;
-  totalCount: number;
-  totalPages: number;
+    success: boolean;
+    data: T[];
+    page: number;
+    limit: number;
+    count: number;
+    totalCount: number;
+    totalPages: number;
 }
 
 export interface IBaseDAO<T> {
-  create(item: T): Promise<T>;
-  getById(id: number): Promise<T | null>;
-  getByUuid(uuid: string): Promise<T | null>;
-  getAll(
-    page: number,
-    limit: number,
-    entityId?: any | null,
-  ): Promise<IDataPaginator<T>>;
-  update(id: number, item: T): Promise<T | null>;
-  delete(id: number): Promise<boolean>;
-}
+    create(item: T): Promise<T>;
+    getById(id: number): Promise<T | null>;
+    getByUuid(uuid: string): Promise<T | null>;
+    getAll(page: number, limit: number, entityId?: any | null): Promise<IDataPaginator<T>>;
+    update(id: number, item: T): Promise<T | null>;
+    delete(id: number): Promise<boolean>;
+}

package/src/dao/VideoMinuteResultDAO.ts

@@ -69,15 +69,15 @@ interface IGroupedResponse {
 }
 
 interface IStudyTimeGroupResult {
-  absoluteTime: string; // ISO 8601 start of bucket
+  absoluteTime: string;      // ISO 8601 start of bucket
   groupIndex: number;
-  startMinute: number; // Start minute number (0-based from video start)
-  endMinute: number; // End minute number (0-based from video start)
-  label: string; // Formatted label
+  startMinute: number;        // Start minute number (0-based from video start)
+  endMinute: number;          // End minute number (0-based from video start)
+  label: string;              // Formatted label
   results: ITMCResult | IATRResult;
   minuteCount: number;
   videoCount: number;
-  contributingVideos: string[]; // Video UUIDs
+  contributingVideos: string[];  // Video UUIDs
 }
 
 interface IGroupedStudyResponse {
@@ -87,7 +87,7 @@ interface IGroupedStudyResponse {
   study: {
     uuid: string;
     name: string;
-    type: "TMC" | "ATR";
+    type: 'TMC' | 'ATR';
     status: string;
   };
   videoCount: number;
@@ -423,41 +423,37 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
     // Use Knex query builder for safe parameter binding
     const groupingQuery = this.knex(this.tableName)
       .select(
-        this.knex.raw("FLOOR(minute_number / ?) as group_index", [
-          groupingMinutes,
-        ]),
-        this.knex.raw("MIN(minute_number) as start_minute"),
-        this.knex.raw("MAX(minute_number) as end_minute"),
-        this.knex.raw("COUNT(*) as minute_count"),
-        this.knex.raw(
-          "array_agg(results ORDER BY minute_number) as all_results",
-        ),
+        this.knex.raw('FLOOR(minute_number / ?) as group_index', [groupingMinutes]),
+        this.knex.raw('MIN(minute_number) as start_minute'),
+        this.knex.raw('MAX(minute_number) as end_minute'),
+        this.knex.raw('COUNT(*) as minute_count'),
+        this.knex.raw('array_agg(results ORDER BY minute_number) as all_results')
       )
-      .where("video_id", video.id);
+      .where('video_id', video.id);
 
     if (startMinute !== undefined) {
-      groupingQuery.where("minute_number", ">=", startMinute);
+      groupingQuery.where('minute_number', '>=', startMinute);
     }
 
     if (endMinute !== undefined) {
-      groupingQuery.where("minute_number", "<=", endMinute);
+      groupingQuery.where('minute_number', '<=', endMinute);
     }
 
     const rows = await groupingQuery
-      .groupBy("group_index")
-      .orderBy("group_index");
+      .groupBy('group_index')
+      .orderBy('group_index');
 
     // Aggregate the results in TypeScript based on video type
     const aggregatedGroups: IGroupedResult[] = rows.map((row: any) => {
-      if (!row || typeof row !== "object") {
-        throw new Error("Invalid row data received from database query");
+      if (!row || typeof row !== 'object') {
+        throw new Error('Invalid row data received from database query');
       }
 
       const allResults = Array.isArray(row.all_results) ? row.all_results : [];
-
+      
       // Determine video type based on multiple factors
-      let studyType = video.videoType || "ATR"; // Default fallback to ATR
-
+      let studyType = video.videoType || 'ATR'; // Default fallback to ATR
+      
       // Check if minute data has study_type field (ATR usually does)
       if (allResults.length > 0 && allResults[0].study_type) {
         studyType = allResults[0].study_type;
@@ -469,24 +465,20 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
           const vehicleKeys = Object.keys(sampleResult.vehicles);
           if (vehicleKeys.length > 0) {
             const firstVehicleType = sampleResult.vehicles[vehicleKeys[0]];
-            if (firstVehicleType && typeof firstVehicleType === "object") {
+            if (firstVehicleType && typeof firstVehicleType === 'object') {
               const directions = Object.keys(firstVehicleType);
-              if (
-                directions.includes("NORTH") ||
-                directions.includes("SOUTH") ||
-                directions.includes("EAST") ||
-                directions.includes("WEST")
-              ) {
-                studyType = "TMC";
+              if (directions.includes('NORTH') || directions.includes('SOUTH') || 
+                  directions.includes('EAST') || directions.includes('WEST')) {
+                studyType = 'TMC';
               }
             }
           }
         }
       }
-
+      
       // Aggregate based on determined video type
       let aggregatedResult;
-      if (studyType === "TMC") {
+      if (studyType === 'TMC') {
         aggregatedResult = this.aggregateTMCResults(allResults);
       } else {
         aggregatedResult = this.aggregateATRResults(allResults);
@@ -577,8 +569,8 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
         "v.uuid as videoUuid",
         "v.recordingStartedAt",
         this.knex.raw(
-          '"v"."recordingStartedAt" + (vmr.minute_number || \' minutes\')::INTERVAL as "absoluteTime"',
-        ),
+          '"v"."recordingStartedAt" + (vmr.minute_number || \' minutes\')::INTERVAL as "absoluteTime"'
+        )
       )
       .whereIn("v.id", videoIds)
       .orderBy("absoluteTime", "asc");
@@ -616,7 +608,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
     for (const minute of minuteResults) {
       const absoluteTime = new Date(minute.absoluteTime);
       const minutesSinceEarliest = Math.floor(
-        (absoluteTime.getTime() - earliestTime.getTime()) / (1000 * 60),
+        (absoluteTime.getTime() - earliestTime.getTime()) / (1000 * 60)
       );
       const bucketIndex = Math.floor(minutesSinceEarliest / groupingMinutes);
 
@@ -624,7 +616,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
         // Calculate bucket start time
         const bucketStartMinutes = bucketIndex * groupingMinutes;
         const bucketStartTime = new Date(
-          earliestTime.getTime() + bucketStartMinutes * 60 * 1000,
+          earliestTime.getTime() + bucketStartMinutes * 60 * 1000
         );
 
         buckets.set(bucketIndex, {
@@ -641,14 +633,12 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
     }
 
     // Step 5: Aggregate using existing methods based on study type
-    const aggregatedGroups: IStudyTimeGroupResult[] = Array.from(
-      buckets.values(),
-    )
+    const aggregatedGroups: IStudyTimeGroupResult[] = Array.from(buckets.values())
       .sort((a, b) => a.groupIndex - b.groupIndex)
       .map((bucket) => {
         let aggregatedResult: ITMCResult | IATRResult;
 
-        if (study.type === "TMC") {
+        if (study.type === 'TMC') {
           aggregatedResult = this.aggregateTMCResults(bucket.results);
         } else {
           aggregatedResult = this.aggregateATRResults(bucket.results);
@@ -659,10 +649,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
           groupIndex: bucket.groupIndex,
           startMinute: bucket.groupIndex * groupingMinutes,
           endMinute: (bucket.groupIndex + 1) * groupingMinutes - 1,
-          label: this.formatStudyTimeLabel(
-            bucket.absoluteTime,
-            groupingMinutes,
-          ),
+          label: this.formatStudyTimeLabel(bucket.absoluteTime, groupingMinutes),
           results: aggregatedResult,
           minuteCount: bucket.results.length,
           videoCount: bucket.videoUuids.size,
@@ -725,27 +712,25 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
       vehicles: {},
       counts: {
         total_vehicles: 0,
-        entry_vehicles: 0,
+        entry_vehicles: 0
       },
       total: 0,
       totalcount: 0,
       detected_classes: {},
-      study_type: "TMC",
+      study_type: "TMC"
     };
 
     for (const minute of minutes) {
       const results = minute; // minute is already the results object from array_agg
 
       // Aggregate vehicle movements by class and direction
-      if (results.vehicles && typeof results.vehicles === "object") {
-        for (const [vehicleClass, directions] of Object.entries(
-          results.vehicles,
-        )) {
+      if (results.vehicles && typeof results.vehicles === 'object') {
+        for (const [vehicleClass, directions] of Object.entries(results.vehicles)) {
           // Skip the 'total' pseudo vehicle class - validate it's actually aggregate data
-          if (vehicleClass === "total" && typeof directions === "number") {
+          if (vehicleClass === 'total' && typeof directions === 'number') {
             continue; // This is aggregate total data, not a vehicle class
           }
-
+          
           if (!aggregated.vehicles[vehicleClass]) {
             aggregated.vehicles[vehicleClass] = {};
           }
@@ -768,12 +753,11 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
               for (const [turnType, count] of Object.entries(turns)) {
                 const turnCount = (count as number) || 0;
                 aggregated.vehicles[vehicleClass][direction][turnType] =
-                  (aggregated.vehicles[vehicleClass][direction][turnType] ||
-                    0) + turnCount;
-
+                  (aggregated.vehicles[vehicleClass][direction][turnType] || 0) + turnCount;
+                
                 // Add to detected_classes count for this vehicle type
                 aggregated.detected_classes[vehicleClass] += turnCount;
-
+                
                 // Add to total counts
                 aggregated.total += turnCount;
                 aggregated.totalcount += turnCount;
@@ -781,12 +765,10 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
             }
           }
         }
-
+        
         // Also process the 'total' entry for validation but don't count it as a vehicle class
         if (results.vehicles.total) {
-          for (const [direction, turns] of Object.entries(
-            results.vehicles.total as any,
-          )) {
+          for (const [direction, turns] of Object.entries(results.vehicles.total as any)) {
             if (typeof turns === "object" && turns !== null) {
               for (const [turnType, count] of Object.entries(turns)) {
                 const turnCount = (count as number) || 0;
@@ -799,8 +781,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
     }
 
     // Set entry_vehicles same as total_vehicles for TMC
-    aggregated.counts.entry_vehicles =
-      aggregated.counts.total_vehicles || aggregated.total;
+    aggregated.counts.entry_vehicles = aggregated.counts.total_vehicles || aggregated.total;
 
     return aggregated;
   }
@@ -824,7 +805,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
       if (results.vehicles) {
         for (const [vehicleClass, lanes] of Object.entries(results.vehicles)) {
           // Skip 'total' pseudo-class if present
-          if (vehicleClass === "total") {
+          if (vehicleClass === 'total') {
             continue;
           }
 
@@ -833,8 +814,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
           }
 
           for (const [laneId, count] of Object.entries(lanes as any)) {
-            const numericCount =
-              typeof count === "number" ? count : parseInt(String(count)) || 0;
+            const numericCount = typeof count === 'number' ? count : (parseInt(String(count)) || 0);
             aggregated.vehicles[vehicleClass][laneId] =
               (aggregated.vehicles[vehicleClass][laneId] || 0) + numericCount;
           }
@@ -941,10 +921,7 @@ export class VideoMinuteResultDAO implements IBaseDAO<IVideoMinuteResult> {
    * Used when results are already grouped by date in the UI
    * Uses UTC time for consistency with absoluteTime field
    */
-  private formatStudyTimeLabel(
-    startTime: Date,
-    durationMinutes: number,
-  ): string {
+  private formatStudyTimeLabel(startTime: Date, durationMinutes: number): string {
     const endTime = new Date(startTime.getTime() + durationMinutes * 60 * 1000);
 
     const formatTime = (date: Date): string => {

package/src/dao/auth/auth.dao.ts

@@ -4,61 +4,58 @@ import { IAuth } from "../../interfaces/auth/auth.interfaces";
 import KnexManager from "../../KnexConnection";
 
 export class AuthDAO implements IBaseDAO<IAuth> {
-  private _knex: Knex<any, unknown[]> = KnexManager.getConnection();
-
-  async create(item: IAuth): Promise<IAuth> {
-    const [createdAuth] = await this._knex("auth").insert(item).returning("*");
-    return createdAuth;
-  }
-
-  async getById(id: number): Promise<IAuth | null> {
-    const auth = await this._knex("auth").where({ id }).first();
-    return auth || null;
-  }
-
-  async getByUuid(uuid: string): Promise<IAuth | null> {
-    // Auth table doesn't have uuid, so we'll return null or throw error
-    return null;
-  }
-
-  async getByUserId(userId: number): Promise<IAuth | null> {
-    const auth = await this._knex("auth").where({ userId }).first();
-    return auth || null;
-  }
-
-  async getByEmailToken(emailToken: string): Promise<IAuth | null> {
-    const auth = await this._knex("auth").where({ emailToken }).first();
-    return auth || null;
-  }
-
-  async update(id: number, item: Partial<IAuth>): Promise<IAuth | null> {
-    const [updatedAuth] = await this._knex("auth")
-      .where({ id })
-      .update(item)
-      .returning("*");
-    return updatedAuth || null;
-  }
-
-  async delete(id: number): Promise<boolean> {
-    const result = await this._knex("auth").where({ id }).del();
-    return result > 0;
-  }
-
-  async getAll(page: number, limit: number): Promise<IDataPaginator<IAuth>> {
-    const offset = (page - 1) * limit;
-
-    const [countResult] = await this._knex("auth").count("* as count");
-    const totalCount = +countResult.count;
-    const auths = await this._knex("auth").limit(limit).offset(offset);
-
-    return {
-      success: true,
-      data: auths,
-      page,
-      limit,
-      count: auths.length,
-      totalCount,
-      totalPages: Math.ceil(totalCount / limit),
-    };
-  }
-}
+    private _knex: Knex<any, unknown[]> = KnexManager.getConnection();
+
+    async create(item: IAuth): Promise<IAuth> {
+        const [createdAuth] = await this._knex("auth").insert(item).returning("*");
+        return createdAuth;
+    }
+
+    async getById(id: number): Promise<IAuth | null> {
+        const auth = await this._knex("auth").where({ id }).first();
+        return auth || null;
+    }
+
+    async getByUuid(uuid: string): Promise<IAuth | null> {
+        // Auth table doesn't have uuid, so we'll return null or throw error
+        return null;
+    }
+
+    async getByUserId(userId: number): Promise<IAuth | null> {
+        const auth = await this._knex("auth").where({ userId }).first();
+        return auth || null;
+    }
+
+    async getByEmailToken(emailToken: string): Promise<IAuth | null> {
+        const auth = await this._knex("auth").where({ emailToken }).first();
+        return auth || null;
+    }
+
+    async update(id: number, item: Partial<IAuth>): Promise<IAuth | null> {
+        const [updatedAuth] = await this._knex("auth").where({ id }).update(item).returning("*");
+        return updatedAuth || null;
+    }
+
+    async delete(id: number): Promise<boolean> {
+        const result = await this._knex("auth").where({ id }).del();
+        return result > 0;
+    }
+
+    async getAll(page: number, limit: number): Promise<IDataPaginator<IAuth>> {
+        const offset = (page - 1) * limit;
+
+        const [countResult] = await this._knex("auth").count("* as count");
+        const totalCount = +countResult.count;
+        const auths = await this._knex("auth").limit(limit).offset(offset);
+
+        return {
+            success: true,
+            data: auths,
+            page,
+            limit,
+            count: auths.length,
+            totalCount,
+            totalPages: Math.ceil(totalCount / limit),
+        };
+    }
+}