feat: add a solver.min-release-age-exclude-source config option

Author: radoering

docs/configuration.md

@@ -442,6 +442,26 @@ regardless of their upload age.
 poetry config solver.min-release-age-exclude "my-package,other-package"
 ```
 
+### `solver.min-release-age-exclude-source`
+
+**Type**: `string`
+
+**Default**: *not set*
+
+**Environment Variable**: `POETRY_SOLVER_MIN_RELEASE_AGE_EXCLUDE_SOURCE`
+
+*Introduced in 2.4.0*
+
+A comma-separated list of source names or URLs that should be excluded from the
+[`solver.min-release-age`](#solvermin-release-age) filter.
+All packages from these sources will always be considered by the solver,
+regardless of their upload age.
+Sources can be referenced by the name defined in `pyproject.toml` or by URL.
+
+```bash
+poetry config solver.min-release-age-exclude-source "private-repo,https://example.com/simple/"
+```
+
 ### `system-git-client`
 
 **Type**: `boolean`

src/poetry/config/config.py

@@ -44,6 +44,10 @@ def int_normalizer(val: str) -> int:
     return int(val)
 
 
+def str_list_normalizer(val: str) -> list[str]:
+    return [vs for v in val.split(",") if (vs := v.strip())]
+
+
 def build_config_setting_validator(val: str) -> bool:
     try:
         value = build_config_setting_normalizer(val)
@@ -114,9 +118,8 @@ def normalize(cls, policy: str) -> list[str]:
 
         return list(
             {
-                name.strip() if cls.is_reserved(name) else canonicalize_name(name)
-                for name in policy.strip().split(",")
-                if name
+                name if cls.is_reserved(name) else canonicalize_name(name)
+                for name in str_list_normalizer(policy)
             }
         )
 
@@ -176,6 +179,7 @@ class Config:
             "lazy-wheel": True,
             "min-release-age": 0,
             "min-release-age-exclude": None,
+            "min-release-age-exclude-source": None,
         },
         "system-git-client": False,
         "keyring": {
@@ -411,6 +415,9 @@ def _get_normalizer(name: str) -> Callable[[str], Any]:
         }:
             return PackageFilterPolicy.normalize
 
+        if name == "solver.min-release-age-exclude-source":
+            return str_list_normalizer
+
         if name.startswith("installer.build-config-settings."):
             return build_config_setting_normalizer
 

src/poetry/console/commands/config.py

@@ -19,6 +19,7 @@
 from poetry.config.config import build_config_setting_normalizer
 from poetry.config.config import build_config_setting_validator
 from poetry.config.config import int_normalizer
+from poetry.config.config import str_list_normalizer
 from poetry.config.config_source import UNSET
 from poetry.config.config_source import ConfigSourceMigration
 from poetry.config.config_source import PropertyNotFoundError
@@ -107,6 +108,10 @@ def unique_config_values(self) -> dict[str, tuple[Any, Any]]:
                 PackageFilterPolicy.validator,
                 PackageFilterPolicy.normalize,
             ),
+            "solver.min-release-age-exclude-source": (
+                lambda val: bool(val.strip()),
+                str_list_normalizer,
+            ),
             "keyring.enabled": (boolean_validator, boolean_normalizer),
             "python.installation-dir": (str, lambda val: str(Path(val))),
         }

src/poetry/repositories/http_repository.py

@@ -84,13 +84,21 @@ def __init__(
         self._min_release_age_cutoff: datetime | None = None
         self._min_release_age_exclude: set[NormalizedName] = set()
         if self._min_release_age:
-            self._min_release_age_cutoff = datetime.now(tz=timezone.utc) - timedelta(
-                days=self._min_release_age
+            exclude_sources: set[str] = set(
+                config.get("solver.min-release-age-exclude-source") or []
             )
-            self._min_release_age_exclude = {
-                canonicalize_name(n)
-                for n in (config.get("solver.min-release-age-exclude") or [])
-            }
+            if self._is_name_excluded_from_min_release_age(
+                exclude_sources
+            ) or self._is_url_excluded_from_min_release_age(exclude_sources):
+                self._min_release_age = 0
+            else:
+                self._min_release_age_cutoff = datetime.now(
+                    tz=timezone.utc
+                ) - timedelta(days=self._min_release_age)
+                self._min_release_age_exclude = {
+                    canonicalize_name(n)
+                    for n in (config.get("solver.min-release-age-exclude") or [])
+                }
         self._age_filtered_versions: dict[NormalizedName, set[Version]] = {}
         # We are tracking if a domain supports range requests or not to avoid
         # unnecessary requests.
@@ -101,6 +109,12 @@ def __init__(
         # - False: The domain does not support range requests for the files we tried.
         self._supports_range_requests: dict[str, bool] = {}
 
+    def _is_name_excluded_from_min_release_age(self, exclude_sources: set[str]) -> bool:
+        return self.name.lower() in {s.lower() for s in exclude_sources}
+
+    def _is_url_excluded_from_min_release_age(self, exclude_sources: set[str]) -> bool:
+        return self.url.rstrip("/") in {s.rstrip("/") for s in exclude_sources}
+
     @property
     def session(self) -> Authenticator:
         return self._authenticator

tests/config/test_config.py

@@ -15,6 +15,7 @@
 from poetry.config.config import Config
 from poetry.config.config import boolean_normalizer
 from poetry.config.config import int_normalizer
+from poetry.config.config import str_list_normalizer
 from poetry.utils.password_manager import PasswordManager
 from tests.helpers import flatten_dict
 from tests.helpers import isolated_environment
@@ -37,6 +38,18 @@ def get_options_based_on_normalizer(normalizer: Normalizer) -> Iterator[str]:
             yield k
 
 
+@pytest.mark.parametrize(
+    ("value", "expected"),
+    [
+        ("foo, bar ,  baz", ["foo", "bar", "baz"]),
+        (", ,", []),
+        ("", []),
+    ],
+)
+def test_str_list_normalizer(value: str, expected: list[str]) -> None:
+    assert str_list_normalizer(value) == expected
+
+
 @pytest.mark.parametrize(
     ("name", "value"),
     [
@@ -45,6 +58,7 @@ def get_options_based_on_normalizer(normalizer: Normalizer) -> Iterator[str]:
         ("requests.max-retries", 0),
         ("solver.min-release-age", 0),
         ("solver.min-release-age-exclude", None),
+        ("solver.min-release-age-exclude-source", None),
     ],
 )
 def test_config_get_default_value(

tests/console/commands/test_config.py

@@ -77,6 +77,7 @@ def test_list_displays_default_value_if_not_set(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = true
 virtualenvs.in-project = null
@@ -114,6 +115,7 @@ def test_list_displays_set_get_setting(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = false
 virtualenvs.in-project = null
@@ -172,6 +174,7 @@ def test_unset_setting(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = true
 virtualenvs.in-project = null
@@ -208,6 +211,7 @@ def test_unset_repo_setting(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = true
 virtualenvs.in-project = null
@@ -345,6 +349,7 @@ def test_list_displays_set_get_local_setting(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = false
 virtualenvs.in-project = null
@@ -391,6 +396,7 @@ def test_list_must_not_display_sources_from_pyproject_toml(
 solver.lazy-wheel = true
 solver.min-release-age = 0
 solver.min-release-age-exclude = null
+solver.min-release-age-exclude-source = null
 system-git-client = false
 virtualenvs.create = true
 virtualenvs.in-project = null
@@ -651,6 +657,23 @@ def test_config_solver_min_release_age_exclude(
     assert repo._min_release_age_exclude == {"my-pkg", "other-pkg"}
 
 
+def test_config_solver_min_release_age_exclude_source(
+    tester: CommandTester, command_tester_factory: CommandTesterFactory
+) -> None:
+    tester.execute("--local solver.min-release-age-exclude-source")
+    assert tester.io.fetch_output().strip() == "null"
+
+    tester.io.clear_output()
+    tester.execute(
+        "--local solver.min-release-age-exclude-source"
+        " 'private-repo,https://example.com/simple/'"
+    )
+    tester.execute("--local solver.min-release-age-exclude-source")
+    output = tester.io.fetch_output().strip()
+    assert "private-repo" in output
+    assert "https://example.com/simple/" in output
+
+
 current_config = """\
 [experimental]
 system-git-client = true

tests/repositories/test_http_repository.py

@@ -96,6 +96,40 @@ def test_min_release_age_exclude_default(config: Config) -> None:
     assert repo._min_release_age_exclude == set()
 
 
+@pytest.mark.parametrize(
+    ("exclude_sources", "expect_cutoff"),
+    [
+        ([], True),
+        (["bar"], True),  # name does not match
+        (["foo"], False),  # matches repo name
+        (["FOO"], False),  # matches repo name (repo names are case-insensitive)
+        (["https://foo.com"], False),  # matches repo URL
+        (["https://foo.com/"], False),  # matches repo URL with trailing slash
+        (["other", "https://foo.com"], False),  # URL in list
+    ],
+)
+def test_min_release_age_exclude_source(
+    config: Config,
+    exclude_sources: list[str],
+    expect_cutoff: bool,
+) -> None:
+    config.merge(
+        {
+            "solver": {
+                "min-release-age": 7,
+                "min-release-age-exclude-source": exclude_sources,
+            }
+        }
+    )
+    repo = MockRepository(config=config)
+    if expect_cutoff:
+        assert repo._min_release_age == 7
+        assert repo._min_release_age_cutoff is not None
+    else:
+        assert repo._min_release_age == 0
+        assert repo._min_release_age_cutoff is None
+
+
 @pytest.mark.parametrize(
     ("links", "expected"),
     [