also fail fast on unknown flags.

Author: xzrderek

eval_protocol/cli.py

@@ -521,8 +521,11 @@ def _hide_suppressed_subparsers(parser: argparse.ArgumentParser) -> None:
 def parse_args(args=None):
     """Parse command line arguments."""
     parser = build_parser()
-    # Use parse_known_args to allow Hydra to handle its own arguments
-    return parser.parse_known_args(args)
+    # Fail fast on unknown flags so typos don't silently get ignored.
+    parsed, remaining = parser.parse_known_args(args)
+    if remaining:
+        parser.error(f"unrecognized arguments: {' '.join(remaining)}")
+    return parsed, remaining
 
 
 def main():

tests/test_cli_args.py

@@ -6,6 +6,16 @@
 from eval_protocol.cli import parse_args
 
 
+def test_unknown_flag_fails_fast(capsys):
+    with pytest.raises(SystemExit) as e:
+        parse_args(["create", "rft", "--definitely-not-a-real-flag"])
+    assert e.value.code == 2
+    out = capsys.readouterr()
+    # argparse writes errors to stderr
+    assert "unrecognized arguments" in out.err
+    assert "--definitely-not-a-real-flag" in out.err
+
+
 @pytest.mark.skip(reason="preview and deploy commands are currently disabled in cli.py")
 class TestCliArgParsing:
     # --- Tests for 'preview' command ---