DONE.md

@@ -2,91 +2,6 @@
 
 This file tracks completed features, fixes, and milestones.
 
-## Version 0.1.0-dev20+ (2025-12-28) - Queue UI Performance & Remux Safety
-
-### Performance Optimizations
-- ✅ **Queue View Button Responsiveness**
-  - Fixed Windows-specific button lag after conversion completion
-  - Eliminated redundant UI refreshes in queue button handlers (Pause, Resume, Cancel, Remove, Move Up/Down, etc.)
-  - Queue onChange callback now handles all refreshes automatically - removed duplicate manual calls
-  - Added stopQueueAutoRefresh() before navigation to prevent conflicting UI updates
-  - Result: Instant button response on Windows (was 1-3 second lag)
-  - Reported by: Jake & Stu
-
-- ✅ **Main Menu Performance**
-  - Fixed main menu lag when sidebar visible and queue active
-  - Implemented 300ms throttling for main menu rebuilds (prevents excessive redraws)
-  - Cached jobQueue.List() calls to eliminate multiple expensive copies (was 2-3 copies per refresh)
-  - Smart conditional refresh: only rebuild sidebar when history actually changes
-  - Result: 3-5x improvement in main menu responsiveness, especially on Windows
-  - RAM usage confirmed: 220MB (lean and efficient for video processing app)
-
-- ✅ **Queue Auto-Refresh Optimization**
-  - Reduced auto-refresh interval from 500ms to 1000ms (1 second)
-  - Reduces UI thread pressure on Windows while maintaining smooth progress updates
-  - Combined with 500ms manual throttle in refreshQueueView() for optimal balance
-
-### User Experience Improvements
-- ✅ **Queue Position Labeling**
-  - Fixed confusing priority display in queue view
-  - Changed from internal priority numbers (3, 2, 1) to user-friendly queue positions (1, 2, 3)
-  - Now displays "Queue Position: 1" for first job, "Queue Position: 2" for second, etc.
-  - Applied to both Pending and Paused jobs
-  - Much clearer for users to understand execution order
-
-### Remux Safety System (Fool-Proof Implementation)
-- ✅ **Comprehensive Codec Compatibility Validation**
-  - Added validateRemuxCompatibility() function with format-specific checks
-  - Automatically detects incompatible codec/container combinations
-  - Validates before ANY remux operation to prevent silent failures
-
-- ✅ **Container-Specific Validation**
-  - MP4: Blocks VP8, VP9, AV1, Theora, Vorbis, Opus (not reliably supported)
-  - MKV: Allows almost everything (ultra-flexible)
-  - WebM: Enforces VP8/VP9/AV1 video + Vorbis/Opus audio only
-  - MOV: Apple-friendly codecs (H.264, H.265, ProRes, MJPEG)
-
-- ✅ **Automatic Fallback to Re-encoding**
-  - WMV/ASF sources automatically re-encode (timestamp/codec issues)
-  - FLV with legacy codecs (Sorenson/VP6) auto re-encode
-  - Incompatible codec/container pairs auto re-encode to safe default (H.264)
-  - User never gets broken files - system handles it transparently
-
-- ✅ **Auto-Fixable Format Detection**
-  - AVI: Applies -fflags +genpts for timestamp regeneration
-  - FLV (H.264): Applies timestamp fixes
-  - MPEG-TS/M2TS/MTS: Extended analysis + timestamp fixes
-  - VOB (DVD rips): Full timestamp regeneration
-  - All apply -avoid_negative_ts make_zero automatically
-
-- ✅ **Enhanced FFmpeg Safety Flags**
-  - All remux operations now include:
-    - `-fflags +genpts` (regenerate timestamps)
-    - `-avoid_negative_ts make_zero` (fix negative timestamps)
-    - `-map 0` (preserve all streams)
-    - `-map_chapters 0` (preserve chapters)
-  - MPEG-TS sources get extended analysis parameters
-  - Result: Robust, reliable remuxing with zero risk of corruption
-
-- ✅ **Codec Name Normalization**
-  - Added normalizeCodecName() to handle codec name variations
-  - Maps h264/avc/avc1/h.264/x264 → h264
-  - Maps h265/hevc/h.265/x265 → h265
-  - Maps divx/xvid/mpeg-4 → mpeg4
-  - Ensures accurate validation regardless of FFprobe output variations
-
-### Technical Improvements
-- ✅ **Smart UI Update Strategy**
-  - Throttled refreshes prevent cascading rebuilds
-  - Conditional updates only when state actually changes
-  - Queue list caching eliminates redundant memory allocations
-  - Windows-optimized rendering pipeline
-
-- ✅ **Debug Logging**
-  - Added comprehensive logging for remux compatibility decisions
-  - Clear messages when auto-fixing vs auto re-encoding
-  - Helps debugging and user understanding
-
 ## Version 0.1.0-dev20+ (2025-12-26) - Author Module & UI Enhancements
 
 ### Features

author_module.go

@@ -381,19 +381,6 @@ func buildChaptersTab(state *appState) fyne.CanvasObject {
 			}
 			state.authorFile = src
 			fileLabel.SetText(fmt.Sprintf("File: %s", filepath.Base(src.Path)))
-            // Clear the custom title so it can be re-derived from the new content.
-            // This addresses the user's request for the title to "reset".
-            state.authorTitle = ""
-            state.updateAuthorSummary()
-            // Update the UI for the title entry if the settings tab is currently visible.
-			if state.active == "author" && state.window.Canvas() != nil {
-				app := fyne.CurrentApp()
-				if app != nil && app.Driver() != nil {
-					app.Driver().DoFromGoroutine(func() {
-						state.showAuthorView() // Rebuild the module to refresh titleEntry
-					}, false)
-				}
-			}
 			state.loadEmbeddedChapters(path)
 			refreshChapters()
 		}, state.window)
@@ -899,20 +886,7 @@ func (s *appState) addAuthorFiles(paths []string) {
 		s.authorChapters = nil
 		s.authorChapterSource = ""
 	}
-	s.authorTitle = ""
 	s.updateAuthorSummary()
-	// Update the UI for the title entry if the settings tab is currently visible.
-	// This ensures the title entry visually resets as well.
-	if s.active == "author" && s.window.Canvas() != nil {
-		app := fyne.CurrentApp()
-		if app != nil && app.Driver() != nil {
-			app.Driver().DoFromGoroutine(func() {
-				// Rebuild the settings tab to refresh its controls.
-				// This is a bit heavy, but ensures the titleEntry reflects the change.
-				s.showAuthorView()
-			}, false)
-		}
-	}
 }
 
 func (s *appState) updateAuthorSummary() {

internal/ui/queueview.go

@@ -253,18 +253,8 @@ func BuildQueueView(
 		emptyMsg.Alignment = fyne.TextAlignCenter
 		jobItems = append(jobItems, container.NewCenter(emptyMsg))
 	} else {
-		// Calculate queue positions for pending/paused jobs
-		queuePositions := make(map[string]int)
-		position := 1
 		for _, job := range jobs {
-			if job.Status == queue.JobStatusPending || job.Status == queue.JobStatusPaused {
-				queuePositions[job.ID] = position
-				position++
-			}
-		}
-
-		for _, job := range jobs {
-			jobItems = append(jobItems, buildJobItem(job, queuePositions, onPause, onResume, onCancel, onRemove, onMoveUp, onMoveDown, onCopyError, onViewLog, onCopyCommand, bgColor, textColor))
+			jobItems = append(jobItems, buildJobItem(job, onPause, onResume, onCancel, onRemove, onMoveUp, onMoveDown, onCopyError, onViewLog, onCopyCommand, bgColor, textColor))
 		}
 	}
 
@@ -286,7 +276,6 @@ func BuildQueueView(
 // buildJobItem creates a single job item in the queue list
 func buildJobItem(
 	job *queue.Job,
-	queuePositions map[string]int,
 	onPause func(string),
 	onResume func(string),
 	onCancel func(string),
@@ -335,7 +324,7 @@ func buildJobItem(
 	badge := BuildModuleBadge(job.Type)
 
 	// Status text
-	statusText := getStatusText(job, queuePositions)
+	statusText := getStatusText(job)
 	statusLabel := widget.NewLabel(statusText)
 	statusLabel.TextStyle = fyne.TextStyle{Monospace: true}
 	statusLabel.Wrapping = fyne.TextWrapWord
@@ -420,14 +409,10 @@ func buildJobItem(
 }
 
 // getStatusText returns a human-readable status string
-func getStatusText(job *queue.Job, queuePositions map[string]int) string {
+func getStatusText(job *queue.Job) string {
 	switch job.Status {
 	case queue.JobStatusPending:
-		// Display position in queue (1 = first to run, 2 = second, etc.)
-		if pos, ok := queuePositions[job.ID]; ok {
-			return fmt.Sprintf("Status: Pending | Queue Position: %d", pos)
-		}
-		return "Status: Pending"
+		return fmt.Sprintf("Status: Pending | Priority: %d", job.Priority)
 	case queue.JobStatusRunning:
 		elapsed := ""
 		if job.StartedAt != nil {
@@ -450,10 +435,6 @@ func getStatusText(job *queue.Job, queuePositions map[string]int) string {
 
 		return fmt.Sprintf("Status: Running | Progress: %.1f%%%s%s", job.Progress, elapsed, extras)
 	case queue.JobStatusPaused:
-		// Display position in queue for paused jobs too
-		if pos, ok := queuePositions[job.ID]; ok {
-			return fmt.Sprintf("Status: Paused | Queue Position: %d", pos)
-		}
 		return "Status: Paused"
 	case queue.JobStatusCompleted:
 		duration := ""

main.go

@@ -985,9 +985,6 @@ type appState struct {
 	queueAutoRefreshStop    chan struct{}
 	queueAutoRefreshRunning bool
 
-	// Main menu refresh throttling
-	mainMenuLastRefresh time.Time
-
 	// Subtitles module state
 	subtitleVideoPath   string
 	subtitleFilePath    string
@@ -1620,18 +1617,12 @@ func (s *appState) showMainMenu() {
 
 	titleColor := utils.MustHex("#4CE870")
 
-	// PERFORMANCE: Cache queue list to avoid multiple expensive copies
-	var queueList []*queue.Job
-	if s.jobQueue != nil {
-		queueList = s.jobQueue.List()
-	}
-
 	// Get queue stats - show completed jobs out of total
 	var queueCompleted, queueTotal int
 	if s.jobQueue != nil {
 		_, _, completed, _, _ := s.jobQueue.Stats()
 		queueCompleted = completed
-		queueTotal = len(queueList)
+		queueTotal = len(s.jobQueue.List())
 	}
 
 	// Build sidebar if visible
@@ -1640,7 +1631,7 @@ func (s *appState) showMainMenu() {
 		// Get active jobs from queue (running/pending)
 		var activeJobs []ui.HistoryEntry
 		if s.jobQueue != nil {
-			for _, job := range queueList {
+			for _, job := range s.jobQueue.List() {
 				if job.Status == queue.JobStatusRunning || job.Status == queue.JobStatusPending {
 					// Convert queue.Job to ui.HistoryEntry
 					entry := ui.HistoryEntry{
@@ -1680,9 +1671,9 @@ func (s *appState) showMainMenu() {
 	}
 
 	menu := ui.BuildMainMenu(mods, s.showModule, s.handleModuleDrop, s.showQueue, nil, s.showBenchmark, s.showBenchmarkHistory, func() {
-		// Toggle sidebar - use throttled refresh to prevent lag
+		// Toggle sidebar
 		s.sidebarVisible = !s.sidebarVisible
-		s.refreshMainMenuThrottled()
+		s.showMainMenu()
 	}, s.sidebarVisible, sidebar, titleColor, queueColor, textColor, queueCompleted, queueTotal, hasBenchmark)
 
 	// Update stats bar
@@ -1709,26 +1700,6 @@ func (s *appState) showMainMenu() {
 	s.setContent(content)
 }
 
-// refreshMainMenuThrottled rebuilds main menu but throttles to prevent excessive redraws
-// Windows GUI is sensitive to rapid rebuilds, so we enforce a minimum delay
-func (s *appState) refreshMainMenuThrottled() {
-	now := time.Now()
-	if !s.mainMenuLastRefresh.IsZero() && now.Sub(s.mainMenuLastRefresh) < 300*time.Millisecond {
-		// Too soon since last refresh - skip to prevent lag
-		return
-	}
-	s.mainMenuLastRefresh = now
-	s.showMainMenu()
-}
-
-// refreshMainMenuSidebar is a lightweight refresh for sidebar-only updates
-// This prevents full main menu rebuilds when only history changes
-func (s *appState) refreshMainMenuSidebar() {
-	// For now, use throttled refresh to prevent cascading rebuilds
-	// In the future, could optimize to only update sidebar component
-	s.refreshMainMenuThrottled()
-}
-
 func (s *appState) showQueue() {
 	s.stopPreview()
 	s.stopPlayer()
@@ -1782,8 +1753,6 @@ func (s *appState) refreshQueueView() {
 	view, scroll := ui.BuildQueueView(
 		jobs,
 		func() { // onBack
-			// Stop auto-refresh before navigating away for snappy response
-			s.stopQueueAutoRefresh()
 			target := s.queueBackTarget
 			if target == "" {
 				target = s.lastModule
@@ -1798,67 +1767,61 @@ func (s *appState) refreshQueueView() {
 			if err := s.jobQueue.Pause(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to pause job: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func(id string) { // onResume
 			if err := s.jobQueue.Resume(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to resume job: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func(id string) { // onCancel
 			if err := s.jobQueue.Cancel(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to cancel job: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func(id string) { // onRemove
 			if err := s.jobQueue.Remove(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to remove job: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func(id string) { // onMoveUp
 			if err := s.jobQueue.MoveUp(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to move job up: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func(id string) { // onMoveDown
 			if err := s.jobQueue.MoveDown(id); err != nil {
 				logging.Debug(logging.CatSystem, "failed to move job down: %v", err)
 			}
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView() // Refresh
 		},
 		func() { // onPauseAll
 			s.jobQueue.PauseAll()
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView()
 		},
 		func() { // onResumeAll
 			s.jobQueue.ResumeAll()
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView()
 		},
 		func() { // onStart
 			s.jobQueue.ResumeAll()
-			// Queue onChange callback handles refresh automatically
+			s.refreshQueueView()
 		},
 		func() { // onClear
-			// Stop auto-refresh to prevent double UI updates
-			s.stopQueueAutoRefresh()
 			s.jobQueue.Clear()
 
 			// Always return to main menu after clearing
 			if len(s.jobQueue.List()) == 0 {
 				s.showMainMenu()
 			} else {
-				// Restart auto-refresh and do single refresh
-				s.startQueueAutoRefresh()
-				s.refreshQueueView()
+				s.refreshQueueView() // Refresh if jobs remain
 			}
 		},
 		func() { // onClearAll
-			// Stop auto-refresh to prevent double UI updates during navigation
-			s.stopQueueAutoRefresh()
 			s.jobQueue.ClearAll()
 			// Return to the module we were working on if possible
 			if s.lastModule != "" && s.lastModule != "queue" && s.lastModule != "menu" {
@@ -1947,9 +1910,7 @@ func (s *appState) startQueueAutoRefresh() {
 	s.queueAutoRefreshStop = stop
 	s.queueAutoRefreshRunning = true
 	go func() {
-		// Use 1-second interval to reduce UI update frequency, especially on Windows
-		// The refreshQueueView method has its own 500ms throttle for other triggers
-		ticker := time.NewTicker(1000 * time.Millisecond)
+		ticker := time.NewTicker(500 * time.Millisecond)
 		defer ticker.Stop()
 		for {
 			select {
@@ -3979,30 +3940,6 @@ func (s *appState) executeConvertJob(ctx context.Context, job *queue.Job, progre
 		remux = true
 	}
 
-	// REMUX SAFETY: Validate compatibility and auto-fix issues
-	if remux {
-		src, probeErr := probeVideo(inputPath)
-		if probeErr != nil {
-			return fmt.Errorf("remux safety check failed - cannot probe source: %w", probeErr)
-		}
-
-		compatible, reason, autoFix := validateRemuxCompatibility(src, selectedFormat.Ext, inputPath)
-		if !compatible {
-			if autoFix {
-				logging.Debug(logging.CatFFMPEG, "remux compatibility issue detected (auto-fixable): %s", reason)
-				// Continue with remux but apply fixes below
-			} else {
-				logging.Debug(logging.CatFFMPEG, "remux not compatible: %s - forcing re-encode", reason)
-				remux = false
-				// Force to safe codec
-				if selectedFormat.VideoCodec == "copy" {
-					selectedFormat.VideoCodec = "libx264"
-					cfg["videoCodec"] = "H.264"
-				}
-			}
-		}
-	}
-
 	// DVD presets: enforce compliant codecs and audio settings
 	// Note: We do NOT force resolution - user can choose Source or specific resolution
 	if isDVD {
@@ -4025,19 +3962,8 @@ func (s *appState) executeConvertJob(ctx context.Context, job *queue.Job, progre
 		cfg["pixelFormat"] = "yuv420p"
 	}
 
-	// REMUX SAFETY FLAGS: Add comprehensive timestamp and compatibility fixes
 	if remux {
-		// Regenerate presentation timestamps to fix sync issues
 		args = append(args, "-fflags", "+genpts")
-
-		// Fix negative timestamp issues (common in AVI, FLV, MPEG-TS)
-		args = append(args, "-avoid_negative_ts", "make_zero")
-
-		// Analyze MPEG-2 and MPEG-TS more carefully for proper remuxing
-		sourceExt := strings.ToLower(filepath.Ext(inputPath))
-		if sourceExt == ".ts" || sourceExt == ".m2ts" || sourceExt == ".mts" {
-			args = append(args, "-analyzeduration", "10000000", "-probesize", "10000000")
-		}
 	}
 	args = append(args, "-i", inputPath)
 
@@ -4236,14 +4162,7 @@ func (s *appState) executeConvertJob(ctx context.Context, job *queue.Job, progre
 		}
 	}
 	if videoCodec == "Copy" && !isDVD {
-		// REMUX MODE: Copy all streams safely
 		args = append(args, "-c:v", "copy")
-
-		// Map all streams to preserve everything (video, audio, subtitles, etc.)
-		args = append(args, "-map", "0")
-
-		// Preserve chapters if they exist
-		args = append(args, "-map_chapters", "0")
 	} else {
 		// Determine the actual codec to use
 		var actualCodec string
@@ -5865,11 +5784,10 @@ func runGUI() {
 			if state.active == "queue" {
 				state.refreshQueueView()
 			}
-			// PERFORMANCE FIX: Only rebuild main menu if history ACTUALLY changed
-			// This prevents constant rebuilds on every queue progress update
 			if state.active == "mainmenu" && state.sidebarVisible && len(state.historyEntries) != historyCount {
-				// Only refresh sidebar, not entire menu (much faster)
-				state.refreshMainMenuSidebar()
+				state.navigationHistorySuppress = true
+				state.showMainMenu()
+				state.navigationHistorySuppress = false
 			}
 		}, false)
 	})
@@ -12367,155 +12285,6 @@ func (v *videoSource) IsProgressive() bool {
 	return false
 }
 
-// validateRemuxCompatibility checks if source codecs are compatible with target container
-// Returns: (compatible, reason, autoFixable)
-// - compatible: true if remux is safe
-// - reason: explanation of why it's incompatible (if false)
-// - autoFixable: true if we can fix with FFmpeg flags (genpts, avoid_negative_ts, etc)
-func validateRemuxCompatibility(src *videoSource, targetExt string, sourcePath string) (bool, string, bool) {
-	if src == nil {
-		return false, "source probe returned nil", false
-	}
-
-	videoCodec := strings.ToLower(src.VideoCodec)
-	audioCodec := strings.ToLower(src.AudioCodec)
-	sourceExt := strings.ToLower(filepath.Ext(sourcePath))
-	targetExt = strings.ToLower(targetExt)
-
-	// Normalize codec names for comparison
-	videoCodec = normalizeCodecName(videoCodec)
-	audioCodec = normalizeCodecName(audioCodec)
-
-	// === CRITICAL BLOCKS: Must re-encode ===
-
-	// 1. WMV/ASF: Known to have issues with MKV/MP4 remux
-	if sourceExt == ".wmv" || sourceExt == ".asf" {
-		return false, "WMV/ASF containers often have timestamp and codec issues - re-encoding recommended", false
-	}
-
-	// 2. Old FLV with proprietary codecs
-	if sourceExt == ".flv" {
-		if strings.Contains(videoCodec, "sorenson") || strings.Contains(videoCodec, "vp6") {
-			return false, "FLV with legacy codecs (Sorenson/VP6) not well supported - re-encoding required", false
-		}
-		// H.264 FLV can be remuxed but often has timestamp issues (auto-fixable)
-		if strings.Contains(videoCodec, "h264") || strings.Contains(videoCodec, "avc") {
-			return true, "FLV H.264 detected - will apply timestamp fixes", true
-		}
-	}
-
-	// 3. Codec compatibility with target container
-	switch targetExt {
-	case ".mp4":
-		// MP4 supports: H.264, H.265, MPEG-4, AAC, MP3, AC3
-		// Does NOT support: VP8, VP9, AV1 (reliably), Theora, Vorbis, Opus (without tricks)
-		if strings.Contains(videoCodec, "vp8") || strings.Contains(videoCodec, "vp9") {
-			return false, "VP8/VP9 not reliably supported in MP4 - use MKV or WebM", false
-		}
-		if strings.Contains(videoCodec, "av1") {
-			return false, "AV1 in MP4 is experimental - use MKV for better compatibility", false
-		}
-		if strings.Contains(videoCodec, "theora") {
-			return false, "Theora not supported in MP4 - use MKV or re-encode to H.264", false
-		}
-		if strings.Contains(audioCodec, "vorbis") || strings.Contains(audioCodec, "opus") {
-			return false, "Vorbis/Opus not reliably supported in MP4 - use MKV or convert to AAC", false
-		}
-
-	case ".mkv":
-		// MKV is ultra-flexible, supports almost everything
-		// Only block truly broken/exotic codecs
-		if strings.Contains(videoCodec, "wmv") && strings.Contains(videoCodec, "drm") {
-			return false, "DRM-protected WMV cannot be remuxed", false
-		}
-
-	case ".webm":
-		// WebM only supports: VP8, VP9, AV1, Vorbis, Opus
-		if !strings.Contains(videoCodec, "vp8") && !strings.Contains(videoCodec, "vp9") && !strings.Contains(videoCodec, "av1") {
-			return false, fmt.Sprintf("WebM only supports VP8/VP9/AV1 video (source: %s)", videoCodec), false
-		}
-		if !strings.Contains(audioCodec, "vorbis") && !strings.Contains(audioCodec, "opus") && audioCodec != "" {
-			return false, fmt.Sprintf("WebM only supports Vorbis/Opus audio (source: %s)", audioCodec), false
-		}
-
-	case ".mov":
-		// MOV/QuickTime is fairly flexible but has quirks
-		// Generally compatible with H.264, H.265, ProRes, MJPEG
-		// Can have issues with exotic codecs
-	}
-
-	// === AUTO-FIXABLE ISSUES ===
-
-	// AVI files often have timestamp issues (fixable with genpts)
-	if sourceExt == ".avi" {
-		return true, "AVI source - will apply timestamp regeneration (genpts)", true
-	}
-
-	// Old MPEG-TS/PS files may have timestamp issues
-	if sourceExt == ".ts" || sourceExt == ".m2ts" || sourceExt == ".mts" {
-		return true, "MPEG transport stream - will apply timestamp fixes", true
-	}
-
-	// VOB files (DVD rips) often need timestamp fixes
-	if sourceExt == ".vob" {
-		return true, "VOB source - will apply timestamp regeneration", true
-	}
-
-	// All checks passed
-	return true, "", false
-}
-
-// normalizeCodecName standardizes codec names for comparison
-func normalizeCodecName(codec string) string {
-	codec = strings.ToLower(strings.TrimSpace(codec))
-
-	// Map common variations to standard names
-	replacements := map[string]string{
-		"h264":           "h264",
-		"avc":            "h264",
-		"avc1":           "h264",
-		"h.264":          "h264",
-		"x264":           "h264",
-		"h265":           "h265",
-		"hevc":           "h265",
-		"h.265":          "h265",
-		"x265":           "h265",
-		"mpeg4":          "mpeg4",
-		"divx":           "mpeg4",
-		"xvid":           "mpeg4",
-		"mpeg-4":         "mpeg4",
-		"mpeg2":          "mpeg2",
-		"mpeg-2":         "mpeg2",
-		"mpeg2video":     "mpeg2",
-		"aac":            "aac",
-		"mp3":            "mp3",
-		"ac3":            "ac3",
-		"a_ac3":          "ac3",
-		"eac3":           "eac3",
-		"vorbis":         "vorbis",
-		"opus":           "opus",
-		"vp8":            "vp8",
-		"vp9":            "vp9",
-		"av1":            "av1",
-		"libaom-av1":     "av1",
-		"theora":         "theora",
-		"wmv3":           "wmv",
-		"vc1":            "vc1",
-		"prores":         "prores",
-		"prores_ks":      "prores",
-		"mjpeg":          "mjpeg",
-		"png":            "png",
-	}
-
-	for old, new := range replacements {
-		if strings.Contains(codec, old) {
-			return new
-		}
-	}
-
-	return codec
-}
-
 func probeVideo(path string) (*videoSource, error) {
 	ctx, cancel := context.WithTimeout(context.Background(), 15*time.Second)
 	defer cancel()