Fix two bugs from post-implementation audit

1. clip_extractor.py: snap loops were breaking on FIRST boundary within
tolerance instead of finding the NEAREST one. With a sorted boundary
list, this snapped to the lowest-valued boundary in range, not the
closest. Fixed by tracking best_dist and iterating all boundaries.

2. orchestrator.py: adjust_to_scene_boundaries() call was still passing
tolerance=5.0 and min/max_duration — leftover from the old heavy
implementation. The new method is a lightweight 1s safety snap that
ignores min/max_duration entirely. Removed stale kwargs so the call
uses the correct defaults.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

core/clip_extractor.py

@@ -256,17 +256,21 @@ class ClipExtractor:
             new_start = candidate.start_time
             new_end   = candidate.end_time
 
-            # Snap start to nearest boundary within tolerance
+            # Snap start to the NEAREST boundary within tolerance
+            best_start_dist = tolerance
             for boundary in scene_boundaries:
-                if abs(boundary - candidate.start_time) <= tolerance:
+                dist = abs(boundary - candidate.start_time)
+                if dist <= best_start_dist:
+                    best_start_dist = dist
                     new_start = boundary
-                    break
 
-            # Snap end to nearest boundary within tolerance
+            # Snap end to the NEAREST boundary within tolerance
+            best_end_dist = tolerance
             for boundary in scene_boundaries:
-                if abs(boundary - candidate.end_time) <= tolerance:
+                dist = abs(boundary - candidate.end_time)
+                if dist <= best_end_dist:
+                    best_end_dist = dist
                     new_end = boundary
-                    break
 
             # Sanity: don't snap if it collapses the clip
             if new_end <= new_start:

pipeline/orchestrator.py

@@ -428,9 +428,6 @@ class PipelineOrchestrator:
                 optimized_candidates = self._clip_extractor.adjust_to_scene_boundaries(
                     optimized_candidates,
                     scene_boundaries,
-                    tolerance=5.0,  # Allow up to 5s adjustment for scene alignment
-                    min_duration=min_duration,
-                    max_duration=max_duration,
                 )
                 # Log the resulting duration distribution
                 durations = [c.end_time - c.start_time for c in optimized_candidates]