@optifye/dashboard-core 6.10.51 → 6.10.53

package/dist/index.d.mts

@@ -1298,6 +1298,7 @@ interface S3ListObjectsParams {
 interface VideoMetadata {
     camera_id?: string;
     workspace_id?: string;
+    cycle_time?: number;
     sop_name?: string;
     violation_start_frame?: number;
     violation_end_frame?: number;

package/dist/index.d.ts

@@ -1298,6 +1298,7 @@ interface S3ListObjectsParams {
 interface VideoMetadata {
     camera_id?: string;
     workspace_id?: string;
+    cycle_time?: number;
     sop_name?: string;
     violation_start_frame?: number;
     violation_end_frame?: number;

package/dist/index.js

@@ -5590,6 +5590,25 @@ var CLIP_COUNTS_CACHE_TTL = 30 * 1e3;
 var CLIP_BY_ID_CACHE_TTL = 2 * 60 * 1e3;
 var CLIP_BY_ID_CACHE_MAX = 200;
 var CLIPS_INIT_CACHE_TTL = 30 * 1e3;
+var parseFiniteNumber = (value) => {
+  if (typeof value === "number" && Number.isFinite(value)) {
+    return value;
+  }
+  if (typeof value === "string" && value.trim().length > 0) {
+    const parsed = Number(value);
+    if (Number.isFinite(parsed)) {
+      return parsed;
+    }
+  }
+  return void 0;
+};
+var getClipCycleTimeFrames = (metadata) => {
+  const flattenedCycleTime = parseFiniteNumber(metadata?.cycle_time);
+  if (flattenedCycleTime !== void 0) {
+    return flattenedCycleTime;
+  }
+  return parseFiniteNumber(metadata?.request?.metadata?.cycle_time);
+};
 var getSupabaseClient = () => {
   const existing = _getSupabaseInstanceOptional();
   if (existing) {
@@ -6099,20 +6118,31 @@ var S3ClipsSupabaseService = class {
         limit
       });
       console.log(`[S3ClipsSupabase] Fetched ${response.clips?.length || 0} ${type} clips`);
-      const transformedClips = (response.clips || []).map((clip) => ({
-        id: clip.clip_id,
-        src: clip.playlist,
-        // Raw playlist content
-        timestamp: clip.timestamp,
-        // Use pre-formatted timestamp from API (already in 12-hour format with seconds)
-        severity: this.getSeverityFromClipType(clip.clip_type_name),
-        description: this.getDescriptionFromClipType(clip.clip_type_name),
-        type: clip.clip_type_name,
-        originalUri: `clips:${clip.clip_id}`,
-        cycle_time_seconds: clip.metadata?.request?.metadata?.cycle_time ? clip.metadata.request.metadata.cycle_time / (clip.metadata?.playlist?.fps || 20) : void 0,
-        creation_timestamp: clip.created_at,
-        percentile: clip.cycle_time_percentile || clip.idle_time_percentile
-      }));
+      const transformedClips = (response.clips || []).map((clip) => {
+        const clipId = clip.id ?? clip.clip_id;
+        const clipType = clip.type ?? clip.clip_type_name;
+        const cycleTimeSeconds = parseFiniteNumber(clip.cycle_time_seconds) ?? (() => {
+          const cycleTimeFrames = getClipCycleTimeFrames(clip.metadata);
+          return cycleTimeFrames !== void 0 ? cycleTimeFrames / (clip.metadata?.playlist?.fps || 20) : void 0;
+        })();
+        return {
+          id: clipId,
+          src: clip.src ?? clip.playlist,
+          // Current API returns src; keep playlist fallback for legacy responses
+          timestamp: clip.timestamp,
+          // Use pre-formatted timestamp from API (already in 12-hour format with seconds)
+          severity: clip.severity ?? this.getSeverityFromClipType(clipType),
+          description: clip.description ?? this.getDescriptionFromClipType(clipType),
+          type: clipType,
+          originalUri: clip.originalUri ?? `clips:${clipId}`,
+          cycle_time_seconds: cycleTimeSeconds,
+          creation_timestamp: clip.creation_timestamp ?? clip.created_at,
+          percentile: clip.percentile ?? clip.cycle_time_percentile ?? clip.idle_time_percentile,
+          duration: clip.duration,
+          idle_start_time: clip.idle_start_time,
+          idle_end_time: clip.idle_end_time
+        };
+      });
       return {
         clips: transformedClips,
         total: response.total || transformedClips.length,
@@ -6142,10 +6172,13 @@ var S3ClipsSupabaseService = class {
    */
   getSeverityFromClipType(clipType) {
     switch (clipType) {
+      case "long_cycle_time":
+      case "worst_cycle_time":
       case "sop_deviations":
         return "high";
       case "idle_time":
         return "medium";
+      case "best_cycle_time":
       case "cycle_completion":
       default:
         return "low";
@@ -6157,11 +6190,14 @@ var S3ClipsSupabaseService = class {
    */
   getDescriptionFromClipType(clipType) {
     const descriptions = {
+      "long_cycle_time": "Long Cycle Time Detected",
       "idle_time": "Idle Time Detected",
+      "best_cycle_time": "Best Cycle Time Performance",
+      "worst_cycle_time": "Worst Cycle Time Performance",
       "sop_deviations": "SOP Deviations",
       "cycle_completion": "Cycle Completion"
     };
-    return descriptions[clipType] || "Analysis Clip";
+    return clipType ? descriptions[clipType] || "Analysis Clip" : "Analysis Clip";
   }
 };
 
@@ -62452,6 +62488,21 @@ var ShiftsView = ({
         }
         const factoryId = existingLineThreshold?.factory_id || lineFactoryId;
         if (factoryId) {
+          let resolvedLineThresholdSkuId = existingLineThreshold?.sku_id || null;
+          if (!resolvedLineThresholdSkuId) {
+            const { data: dummySkuRows, error: dummySkuError } = await supabase.from("skus").select("id").eq("line_id", lineId).eq("sku_definition", "dummy_definition").eq("is_active", true).limit(1);
+            if (dummySkuError) {
+              throw new Error(
+                `Failed to resolve dummy SKU for line ${lineId}, shift ${shift.shiftId}: ${dummySkuError.message}`
+              );
+            }
+            resolvedLineThresholdSkuId = dummySkuRows?.[0]?.id || null;
+          }
+          if (!resolvedLineThresholdSkuId) {
+            throw new Error(
+              `No active SKU found for line ${lineId}, shift ${shift.shiftId} while updating line thresholds`
+            );
+          }
           const lineThresholdPayload = {
             factory_id: factoryId,
             line_id: lineId,
@@ -62459,12 +62510,10 @@ var ShiftsView = ({
             shift_id: shift.shiftId,
             product_code: existingLineThreshold?.product_code || "",
             threshold_day_output: thresholdDayOutput,
-            threshold_pph: thresholdPPH
+            threshold_pph: thresholdPPH,
+            sku_id: resolvedLineThresholdSkuId
           };
-          if (existingLineThreshold?.sku_id) {
-            lineThresholdPayload.sku_id = existingLineThreshold.sku_id;
-          }
-          const { error: lineThresholdUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdPayload, { onConflict: "factory_id,line_id,date,shift_id" });
+          const { error: lineThresholdUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdPayload, { onConflict: "factory_id,line_id,date,shift_id,sku_id" });
           if (lineThresholdUpsertError) {
             throw new Error(
               `Failed to update line thresholds for line ${lineId}, shift ${shift.shiftId}: ${lineThresholdUpsertError.message}`
@@ -64252,6 +64301,17 @@ var TargetsView = ({
       await workspaceService.updateActionThresholds(workspaceThresholdUpdates);
       console.log(`[handleSaveLine] Successfully updated action thresholds for ${lineId}`);
       const packagingWorkspaces = lineDataToSave.workspaces.filter((ws) => ws.actionType === "packaging");
+      let resolvedLineThresholdSkuId = lineDataToSave.selectedSKU?.id || null;
+      if (!resolvedLineThresholdSkuId) {
+        const { data: dummySkuRows, error: dummySkuError } = await supabase.from("skus").select("id").eq("line_id", lineId).eq("sku_definition", "dummy_definition").eq("is_active", true).limit(1);
+        if (dummySkuError) {
+          throw dummySkuError;
+        }
+        resolvedLineThresholdSkuId = dummySkuRows?.[0]?.id || null;
+      }
+      if (!resolvedLineThresholdSkuId) {
+        throw new Error(`No active SKU found for line ${lineId} while saving line thresholds`);
+      }
       const lineThresholdData = {
         factory_id: currentFactoryId,
         line_id: lineId,
@@ -64261,10 +64321,10 @@ var TargetsView = ({
         threshold_day_output: packagingWorkspaces.reduce((acc, ws) => acc + (Number(ws.targetDayOutput) || 0), 0),
         threshold_pph: packagingWorkspaces.reduce((acc, ws) => acc + (ws.targetPPH ? Math.round(Number(ws.targetPPH)) : 0), 0),
         // Round each PPH value
-        ...skuEnabled && lineDataToSave.selectedSKU ? { sku_id: lineDataToSave.selectedSKU.id } : {}
+        sku_id: resolvedLineThresholdSkuId
       };
       console.log(`[handleSaveLine] lineThresholdData for upsert on ${lineId}:`, lineThresholdData);
-      const { error: lineUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdData, { onConflict: "factory_id,line_id,date,shift_id" });
+      const { error: lineUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdData, { onConflict: "factory_id,line_id,date,shift_id,sku_id" });
       if (lineUpsertError) {
         console.error(`[handleSaveLine] Error upserting line_thresholds for ${lineId}:`, lineUpsertError);
         sonner.toast.error(`Failed to save line thresholds for ${mergedLineNames[lineId] || lineId}`);

package/dist/index.mjs

@@ -5561,6 +5561,25 @@ var CLIP_COUNTS_CACHE_TTL = 30 * 1e3;
 var CLIP_BY_ID_CACHE_TTL = 2 * 60 * 1e3;
 var CLIP_BY_ID_CACHE_MAX = 200;
 var CLIPS_INIT_CACHE_TTL = 30 * 1e3;
+var parseFiniteNumber = (value) => {
+  if (typeof value === "number" && Number.isFinite(value)) {
+    return value;
+  }
+  if (typeof value === "string" && value.trim().length > 0) {
+    const parsed = Number(value);
+    if (Number.isFinite(parsed)) {
+      return parsed;
+    }
+  }
+  return void 0;
+};
+var getClipCycleTimeFrames = (metadata) => {
+  const flattenedCycleTime = parseFiniteNumber(metadata?.cycle_time);
+  if (flattenedCycleTime !== void 0) {
+    return flattenedCycleTime;
+  }
+  return parseFiniteNumber(metadata?.request?.metadata?.cycle_time);
+};
 var getSupabaseClient = () => {
   const existing = _getSupabaseInstanceOptional();
   if (existing) {
@@ -6070,20 +6089,31 @@ var S3ClipsSupabaseService = class {
         limit
       });
       console.log(`[S3ClipsSupabase] Fetched ${response.clips?.length || 0} ${type} clips`);
-      const transformedClips = (response.clips || []).map((clip) => ({
-        id: clip.clip_id,
-        src: clip.playlist,
-        // Raw playlist content
-        timestamp: clip.timestamp,
-        // Use pre-formatted timestamp from API (already in 12-hour format with seconds)
-        severity: this.getSeverityFromClipType(clip.clip_type_name),
-        description: this.getDescriptionFromClipType(clip.clip_type_name),
-        type: clip.clip_type_name,
-        originalUri: `clips:${clip.clip_id}`,
-        cycle_time_seconds: clip.metadata?.request?.metadata?.cycle_time ? clip.metadata.request.metadata.cycle_time / (clip.metadata?.playlist?.fps || 20) : void 0,
-        creation_timestamp: clip.created_at,
-        percentile: clip.cycle_time_percentile || clip.idle_time_percentile
-      }));
+      const transformedClips = (response.clips || []).map((clip) => {
+        const clipId = clip.id ?? clip.clip_id;
+        const clipType = clip.type ?? clip.clip_type_name;
+        const cycleTimeSeconds = parseFiniteNumber(clip.cycle_time_seconds) ?? (() => {
+          const cycleTimeFrames = getClipCycleTimeFrames(clip.metadata);
+          return cycleTimeFrames !== void 0 ? cycleTimeFrames / (clip.metadata?.playlist?.fps || 20) : void 0;
+        })();
+        return {
+          id: clipId,
+          src: clip.src ?? clip.playlist,
+          // Current API returns src; keep playlist fallback for legacy responses
+          timestamp: clip.timestamp,
+          // Use pre-formatted timestamp from API (already in 12-hour format with seconds)
+          severity: clip.severity ?? this.getSeverityFromClipType(clipType),
+          description: clip.description ?? this.getDescriptionFromClipType(clipType),
+          type: clipType,
+          originalUri: clip.originalUri ?? `clips:${clipId}`,
+          cycle_time_seconds: cycleTimeSeconds,
+          creation_timestamp: clip.creation_timestamp ?? clip.created_at,
+          percentile: clip.percentile ?? clip.cycle_time_percentile ?? clip.idle_time_percentile,
+          duration: clip.duration,
+          idle_start_time: clip.idle_start_time,
+          idle_end_time: clip.idle_end_time
+        };
+      });
       return {
         clips: transformedClips,
         total: response.total || transformedClips.length,
@@ -6113,10 +6143,13 @@ var S3ClipsSupabaseService = class {
    */
   getSeverityFromClipType(clipType) {
     switch (clipType) {
+      case "long_cycle_time":
+      case "worst_cycle_time":
       case "sop_deviations":
         return "high";
       case "idle_time":
         return "medium";
+      case "best_cycle_time":
       case "cycle_completion":
       default:
         return "low";
@@ -6128,11 +6161,14 @@ var S3ClipsSupabaseService = class {
    */
   getDescriptionFromClipType(clipType) {
     const descriptions = {
+      "long_cycle_time": "Long Cycle Time Detected",
       "idle_time": "Idle Time Detected",
+      "best_cycle_time": "Best Cycle Time Performance",
+      "worst_cycle_time": "Worst Cycle Time Performance",
       "sop_deviations": "SOP Deviations",
       "cycle_completion": "Cycle Completion"
     };
-    return descriptions[clipType] || "Analysis Clip";
+    return clipType ? descriptions[clipType] || "Analysis Clip" : "Analysis Clip";
   }
 };
 
@@ -62423,6 +62459,21 @@ var ShiftsView = ({
         }
         const factoryId = existingLineThreshold?.factory_id || lineFactoryId;
         if (factoryId) {
+          let resolvedLineThresholdSkuId = existingLineThreshold?.sku_id || null;
+          if (!resolvedLineThresholdSkuId) {
+            const { data: dummySkuRows, error: dummySkuError } = await supabase.from("skus").select("id").eq("line_id", lineId).eq("sku_definition", "dummy_definition").eq("is_active", true).limit(1);
+            if (dummySkuError) {
+              throw new Error(
+                `Failed to resolve dummy SKU for line ${lineId}, shift ${shift.shiftId}: ${dummySkuError.message}`
+              );
+            }
+            resolvedLineThresholdSkuId = dummySkuRows?.[0]?.id || null;
+          }
+          if (!resolvedLineThresholdSkuId) {
+            throw new Error(
+              `No active SKU found for line ${lineId}, shift ${shift.shiftId} while updating line thresholds`
+            );
+          }
           const lineThresholdPayload = {
             factory_id: factoryId,
             line_id: lineId,
@@ -62430,12 +62481,10 @@ var ShiftsView = ({
             shift_id: shift.shiftId,
             product_code: existingLineThreshold?.product_code || "",
             threshold_day_output: thresholdDayOutput,
-            threshold_pph: thresholdPPH
+            threshold_pph: thresholdPPH,
+            sku_id: resolvedLineThresholdSkuId
           };
-          if (existingLineThreshold?.sku_id) {
-            lineThresholdPayload.sku_id = existingLineThreshold.sku_id;
-          }
-          const { error: lineThresholdUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdPayload, { onConflict: "factory_id,line_id,date,shift_id" });
+          const { error: lineThresholdUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdPayload, { onConflict: "factory_id,line_id,date,shift_id,sku_id" });
           if (lineThresholdUpsertError) {
             throw new Error(
               `Failed to update line thresholds for line ${lineId}, shift ${shift.shiftId}: ${lineThresholdUpsertError.message}`
@@ -64223,6 +64272,17 @@ var TargetsView = ({
       await workspaceService.updateActionThresholds(workspaceThresholdUpdates);
       console.log(`[handleSaveLine] Successfully updated action thresholds for ${lineId}`);
       const packagingWorkspaces = lineDataToSave.workspaces.filter((ws) => ws.actionType === "packaging");
+      let resolvedLineThresholdSkuId = lineDataToSave.selectedSKU?.id || null;
+      if (!resolvedLineThresholdSkuId) {
+        const { data: dummySkuRows, error: dummySkuError } = await supabase.from("skus").select("id").eq("line_id", lineId).eq("sku_definition", "dummy_definition").eq("is_active", true).limit(1);
+        if (dummySkuError) {
+          throw dummySkuError;
+        }
+        resolvedLineThresholdSkuId = dummySkuRows?.[0]?.id || null;
+      }
+      if (!resolvedLineThresholdSkuId) {
+        throw new Error(`No active SKU found for line ${lineId} while saving line thresholds`);
+      }
       const lineThresholdData = {
         factory_id: currentFactoryId,
         line_id: lineId,
@@ -64232,10 +64292,10 @@ var TargetsView = ({
         threshold_day_output: packagingWorkspaces.reduce((acc, ws) => acc + (Number(ws.targetDayOutput) || 0), 0),
         threshold_pph: packagingWorkspaces.reduce((acc, ws) => acc + (ws.targetPPH ? Math.round(Number(ws.targetPPH)) : 0), 0),
         // Round each PPH value
-        ...skuEnabled && lineDataToSave.selectedSKU ? { sku_id: lineDataToSave.selectedSKU.id } : {}
+        sku_id: resolvedLineThresholdSkuId
       };
       console.log(`[handleSaveLine] lineThresholdData for upsert on ${lineId}:`, lineThresholdData);
-      const { error: lineUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdData, { onConflict: "factory_id,line_id,date,shift_id" });
+      const { error: lineUpsertError } = await supabase.from("line_thresholds").upsert(lineThresholdData, { onConflict: "factory_id,line_id,date,shift_id,sku_id" });
       if (lineUpsertError) {
         console.error(`[handleSaveLine] Error upserting line_thresholds for ${lineId}:`, lineUpsertError);
         toast.error(`Failed to save line thresholds for ${mergedLineNames[lineId] || lineId}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@optifye/dashboard-core",
-  "version": "6.10.51",
+  "version": "6.10.53",
   "description": "Reusable UI & logic for Optifye dashboard",
   "main": "dist/index.js",
   "module": "dist/index.mjs",