fix(eval): validate --only-qids before loading the BIRD dataset

eval_baseline loaded the BIRD dump before parsing --only-qids, so on hosts
without the local dataset (CI) an invalid qid list died with FileNotFoundError
instead of the intended exit-3 usage error the CLI contract (and its test)
promises. Parse and validate the flag first; the dataset load stays where the
data is actually needed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: JuliaEdom

scripts/eval_baseline.py

@@ -209,18 +209,22 @@ def main(argv: list[str] | None = None) -> int:
     )
     args = parser.parse_args(argv)
 
+    # Validate CLI input before the (filesystem-dependent) dataset load, so a
+    # typo fails fast with exit 3 instead of a FileNotFoundError on hosts
+    # without the BIRD dump.
+    try:
+        only_qids = [int(x) for x in args.only_qids.split(",") if x.strip()]
+    except ValueError:
+        print("[error] invalid --only-qids: expected comma-separated integers", file=sys.stderr)
+        return 3
+
     examples = load_bird_mini_dev(Path(args.bird_root))
     if args.db:
         examples = [e for e in examples if e.registry_db_id == args.db]
         if not examples:
             print(f"[error] no examples for db {args.db!r}", file=sys.stderr)
             return 3
 
-    try:
-        only_qids = [int(x) for x in args.only_qids.split(",") if x.strip()]
-    except ValueError:
-        print("[error] invalid --only-qids: expected comma-separated integers", file=sys.stderr)
-        return 3
     if only_qids:
         examples_by_qid = {e.question_id: e for e in examples}
         sample = [examples_by_qid[qid] for qid in only_qids if qid in examples_by_qid]