Address potential integer overflow in timestamp comparison

Converting int64 subtraction to int could overflow on 32-bit
systems. Though unlikely in practice, use comparison instead of
subtraction.

This was identified by the Cursor AI tool.

Signed-off-by: Tom Pantelis <[email protected]>

Author: tpantelis

pkg/agent/controller/service_import_conflict.go

@@ -218,10 +218,12 @@ func sortServiceImportsByTimestamp(siList []runtime.Object, aggregatedType mcsv1
 		tsA := parseTimestamp(siA)
 		tsB := parseTimestamp(siB)
 
-		if tsA == tsB {
-			return strings.Compare(siA.Labels[mcsv1a1.LabelSourceCluster], siB.Labels[mcsv1a1.LabelSourceCluster])
+		if tsA < tsB {
+			return -1
+		} else if tsA > tsB {
+			return 1
 		}
 
-		return int(tsA - tsB)
+		return strings.Compare(siA.Labels[mcsv1a1.LabelSourceCluster], siB.Labels[mcsv1a1.LabelSourceCluster])
 	})
 }