Fix sub-second rounding and avoid name collision with existing utils.formatTime.

Current logic floors everything (e.g., 0.4s -> "0s"), which is misleading in CutOffsetButton and a UX regression from the prior 0.1s precision. Also, naming this helper formatTime collides conceptually with apps/desktop/src/routes/editor/utils.ts:formatTime (mm:ss[.ff]) and can confuse future readers.

Suggested: rename and improve formatting to preserve tenths under 10s, handle negatives, and keep integer seconds when minutes/hours are present.

-function formatTime(totalSeconds: number): string {
-	const hours = Math.floor(totalSeconds / 3600);
-	const minutes = Math.floor((totalSeconds % 3600) / 60);
-	const seconds = Math.floor(totalSeconds % 60);
-
-	if (hours > 0) {
-		return `${hours}h ${minutes}m ${seconds}s`;
-	} else if (minutes > 0) {
-		return `${minutes}m ${seconds}s`;
-	} else {
-		return `${seconds}s`;
-	}
-}
+function formatDuration(totalSeconds: number): string {
+	const sign = totalSeconds < 0 ? "-" : "";
+	const s = Math.abs(totalSeconds);
+	const hours = Math.floor(s / 3600);
+	const minutes = Math.floor((s % 3600) / 60);
+	const secFloat = s % 60;
+	// Mixed units: keep integer seconds to avoid 60s rollovers
+	if (hours > 0) return `${sign}${hours}h ${minutes}m ${Math.floor(secFloat)}s`;
+	if (minutes > 0) return `${sign}${minutes}m ${Math.floor(secFloat)}s`;
+	// Sub-minute: show tenths for <10s, else whole seconds
+	const secs = secFloat < 10 ? Math.round(secFloat * 10) / 10 : Math.round(secFloat);
+	return `${sign}${secs.toFixed(secFloat < 10 ? 1 : 0)}s`;
+}

Follow-up: consider centralizing this in editor/utils as a separate humanizer (e.g., humanizeDuration) to avoid two different “formatTime” semantics.

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "PWD: $(pwd)"
echo "Git branch:"
git rev-parse --abbrev-ref HEAD || true

echo
echo "Searching for 'formatTime' occurrences (definitions & calls)..."
rg -n --hidden -S -C2 '\bfunction\s+formatTime\b|\bformatTime\s*\(' || true

echo
echo "Show ClipTrack.tsx if present:"
if [ -f "apps/desktop/src/routes/editor/Timeline/ClipTrack.tsx" ]; then
  sed -n '1,240p' apps/desktop/src/routes/editor/Timeline/ClipTrack.tsx
else
  rg -n --hidden -S 'ClipTrack.tsx' || true
fi

echo
echo "Show editor utils if present (possible other formatTime):"
if [ -f "apps/desktop/src/routes/editor/utils.ts" ]; then
  sed -n '1,240p' apps/desktop/src/routes/editor/utils.ts
else
  rg -n --hidden -S --glob '!node_modules' 'src/routes/editor/utils.ts|routes/editor/utils.ts|/routes/editor/utils.ts' || true
  rg -n --hidden -S -C2 '\bformatTime\b' || true
fi

echo
echo "Search for CutOffsetButton usages:"
rg -n --hidden -S -C3 'CutOffsetButton' || true

echo
echo "Search for existing humanize/formatDuration helpers:"
rg -n --hidden -S -C2 'formatDuration|humanizeDuration|humanizeTime|formatTimeHuman' || true

Length of output: 22669

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.