refactor(bootstrap): make --max-release-age optional with IntRange(min=1)

Change --max-release-age to default=None with IntRange(min=1) so that
0 is rejected by Click and the flag is no longer required with
--multiple-versions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Rohan Devasthale <rdevasth@redhat.com>

Author: rd4398

src/fromager/commands/bootstrap.py

@@ -113,9 +113,9 @@ def _get_requirements_from_args(
 @click.option(
     "--max-release-age",
     "max_release_age",
-    type=click.IntRange(min=0),
-    default=0,
-    help="Reject package versions published more than this many days ago (0 disables the check). Required with --multiple-versions.",
+    type=click.IntRange(min=1),
+    default=None,
+    help="Reject package versions published more than this many days ago.",
 )
 @click.argument("toplevel", nargs=-1)
 @click.pass_obj
@@ -128,7 +128,7 @@ def bootstrap(
     skip_constraints: bool,
     test_mode: bool,
     multiple_versions: bool,
-    max_release_age: int,
+    max_release_age: int | None,
     toplevel: list[str],
 ) -> None:
     """Compute and build the dependencies of a set of requirements recursively
@@ -164,10 +164,6 @@ def bootstrap(
         )
 
     if multiple_versions:
-        if max_release_age <= 0:
-            raise click.UsageError(
-                "--max-release-age is required when using --multiple-versions"
-            )
         logger.info(
             "multiple versions mode enabled: will bootstrap all matching versions"
         )
@@ -180,7 +176,7 @@ def bootstrap(
             )
             skip_constraints = True
 
-    if max_release_age > 0:
+    if max_release_age is not None:
         wkctx.set_max_release_age(max_release_age)
         logger.info(
             "max release age: rejecting versions older than %d days",
@@ -514,9 +510,9 @@ def write_constraints_file(
 @click.option(
     "--max-release-age",
     "max_release_age",
-    type=click.IntRange(min=0),
-    default=0,
-    help="Reject package versions published more than this many days ago (0 disables the check). Required with --multiple-versions.",
+    type=click.IntRange(min=1),
+    default=None,
+    help="Reject package versions published more than this many days ago.",
 )
 @click.argument("toplevel", nargs=-1)
 @click.pass_obj
@@ -532,7 +528,7 @@ def bootstrap_parallel(
     force: bool,
     max_workers: int | None,
     multiple_versions: bool,
-    max_release_age: int,
+    max_release_age: int | None,
     toplevel: list[str],
 ) -> None:
     """Bootstrap and build-parallel

tests/test_bootstrap.py

@@ -684,10 +684,10 @@ def test_without_multiple_versions_constraints_not_disabled(
         assert mock_write_constraints.called
 
 
-def test_multiple_versions_requires_max_release_age(
+def test_max_release_age_rejects_zero(
     tmp_context: context.WorkContext,
 ) -> None:
-    """Test that --multiple-versions without --max-release-age fails."""
+    """Test that --max-release-age 0 is rejected by Click's IntRange(min=1)."""
     runner = CliRunner()
     with runner.isolated_filesystem():
         pathlib.Path("req.txt").write_text("setuptools>=60\n")
@@ -696,12 +696,12 @@ def test_multiple_versions_requires_max_release_age(
             [
                 "-r",
                 "req.txt",
-                "--multiple-versions",
+                "--max-release-age",
+                "0",
             ],
             obj=tmp_context,
         )
     assert result.exit_code == 2
-    assert "--max-release-age is required" in result.output
 
 
 @patch("fromager.commands.bootstrap.bootstrapper.Bootstrapper")