phase1: fix two bugs surfaced by the GPU smoke

1. Don't rmtree clip_root/frames when re-bridging: egoinfinity import stores
   real frames there (symlink at extract_frames/frames pointing back), so
   re-running phase1 on an imported clip was deleting the frames. Skip the
   bridge when it already resolves to extract_frames/frames.
2. Pass --only=<id> not --only <id> to batch_pipeline: clip ids can start
   with '-' (YouTube-derived), which argparse mistook for a flag.

Author: Vector-Wangel

egoinfinity/stages/phase1.py

@@ -53,9 +53,16 @@ def run(self, ctx: StageContext) -> None:
         src_frames = ctx.artifacts.path("extract_frames", "frames")
         if not src_frames.exists():
             raise FileNotFoundError(f"frames missing at {src_frames}; run extract_frames first")
-        if bridge_frames.exists() or bridge_frames.is_symlink():
-            bridge_frames.unlink() if bridge_frames.is_symlink() else shutil.rmtree(bridge_frames)
-        bridge_frames.symlink_to(src_frames.resolve())
+        # Don't touch clip_root/frames if it already resolves to the same dir as
+        # extract_frames/frames. `egoinfinity import` stores the REAL frames at
+        # clip_root/frames with a symlink at extract_frames/frames pointing back,
+        # so blindly rmtree-ing clip_root/frames here would delete the imported
+        # frames. Only (re)create the bridge when it's missing or points elsewhere.
+        already = bridge_frames.exists() and src_frames.resolve() == bridge_frames.resolve()
+        if not already:
+            if bridge_frames.exists() or bridge_frames.is_symlink():
+                bridge_frames.unlink() if bridge_frames.is_symlink() else shutil.rmtree(bridge_frames)
+            bridge_frames.symlink_to(src_frames.resolve())
 
         # batch_pipeline relies on a favorites-style manifest with objects[].
         # The pipeline manifest already has video_uri + objects + start + end.
@@ -81,9 +88,11 @@ def run(self, ctx: StageContext) -> None:
         if not ctx.stage_config.args.get("run_track", True):
             env["EGOINFINITY_RUN_TRACK"] = "0"
 
+        # `--only=<id>` (not `--only <id>`): clip ids can start with '-'
+        # (YouTube-derived), which argparse would otherwise treat as a flag.
         cmd = [sys.executable, "-m", "tools.batch_pipeline",
                "--favorites-dir", str(favorites_dir),
-               "--only", clip_id, "--force"]
+               f"--only={clip_id}", "--force"]
         if not ctx.stage_config.args.get("with_sam3d_worker", False):
             cmd.append("--no-sam3d-worker")
         sam3d_preset = ctx.stage_config.args.get("sam3d_preset")