relink.py: source_root is now a positional argument.

Author: samsrabin

relink.py

@@ -233,7 +233,8 @@ def parse_arguments():
         )
     )
     parser.add_argument(
-        "--source-root",
+        "source_root",
+        nargs="?",
         type=validate_directory,
         default=DEFAULT_SOURCE_ROOT,
         help=(

tests/relink/test_args.py

@@ -51,7 +51,7 @@ def test_custom_source_root(self, mock_default_dirs, tmp_path):
         _, target_dir = mock_default_dirs
         custom_source = tmp_path / "custom_source"
         custom_source.mkdir()
-        with patch("sys.argv", ["relink.py", "--source-root", str(custom_source)]):
+        with patch("sys.argv", ["relink.py", str(custom_source)]):
             args = relink.parse_arguments()
             assert args.source_root == str(custom_source.resolve())
             assert args.target_root == target_dir
@@ -76,7 +76,6 @@ def test_both_custom_paths(self, tmp_path):
             "sys.argv",
             [
                 "relink.py",
-                "--source-root",
                 str(source_path),
                 "--target-root",
                 str(target_path),

tests/relink/test_cmdline.py

@@ -40,7 +40,6 @@ def test_command_line_execution_dry_run(mock_dirs):
     command = [
         sys.executable,
         relink_script,
-        "--source-root",
         str(source_dir),
         "--target-root",
         str(target_dir),
@@ -76,7 +75,6 @@ def test_command_line_execution_actual_run(mock_dirs):
     command = [
         sys.executable,
         relink_script,
-        "--source-root",
         str(source_dir),
         "--target-root",
         str(target_dir),

tests/relink/test_timing.py

@@ -37,7 +37,6 @@ def test_timing_logging(tmp_path, caplog, use_timing, should_log_timing):
     # Build argv with or without --timing flag
     test_argv = [
         "relink.py",
-        "--source-root",
         str(source_dir),
         "--target-root",
         str(target_dir),
@@ -75,7 +74,6 @@ def test_timing_shows_in_quiet_mode(tmp_path, caplog):
     # Build argv with both --timing and --quiet flags
     test_argv = [
         "relink.py",
-        "--source-root",
         str(source_dir),
         "--target-root",
         str(target_dir),