Clarify when we are subsetting the lockfile for 3rdparty dependencies (Cherry-pick of pantsbuild#12435)

[ci skip-rust]

[ci skip-build-wheels]

Author: Eric-Arellano

src/python/pants/backend/python/util_rules/pex.py

@@ -654,24 +654,13 @@ async def build_pex(
     if request.pex_path:
         argv.extend(["--pex-path", ":".join(pex.name for pex in request.pex_path)])
 
-    description = request.description
-    if description is None:
-        if request.requirements:
-            description = (
-                f"Building {request.output_filename} with "
-                f"{pluralize(len(request.requirements), 'requirement')}: "
-                f"{', '.join(request.requirements)}"
-            )
-        else:
-            description = f"Building {request.output_filename}"
-
     process = await Get(
         Process,
         PexCliProcess(
             python=python,
             argv=argv,
             additional_input_digest=merged_digest,
-            description=description,
+            description=_build_pex_description(request),
             output_files=[request.output_filename],
         ),
     )
@@ -699,6 +688,23 @@ async def build_pex(
     )
 
 
+def _build_pex_description(request: PexRequest) -> str:
+    if request.description:
+        return request.description
+    if not request.requirements:
+        return f"Building {request.output_filename}"
+    if request.repository_pex:
+        return (
+            f"Extracting {pluralize(len(request.requirements), 'requirement')} "
+            f"to build {request.output_filename} from {request.repository_pex.name}: "
+            f"{', '.join(request.requirements)}"
+        )
+    return (
+        f"Building {request.output_filename} with "
+        f"{pluralize(len(request.requirements), 'requirement')}: {', '.join(request.requirements)}"
+    )
+
+
 @rule
 async def create_pex(request: PexRequest) -> Pex:
     result = await Get(BuildPexResult, PexRequest, request)

src/python/pants/backend/python/util_rules/pex_test.py

@@ -33,6 +33,7 @@
     PexResolveInfo,
     VenvPex,
     VenvPexProcess,
+    _build_pex_description,
     resolve_requirements_constraints_file,
 )
 from pants.backend.python.util_rules.pex import rules as pex_rules
@@ -681,3 +682,49 @@ def test_venv_pex_resolve_info(rule_runner: RuleRunner, pex_type: type[Pex | Ven
     assert dists[3].version == Version("2.23.0")
     assert Requirement.parse('PySocks!=1.5.7,>=1.5.6; extra == "socks"') in dists[3].requires_dists
     assert dists[4].project_name == "urllib3"
+
+
+def test_build_pex_description() -> None:
+    def assert_description(
+        requirements: PexRequirements,
+        *,
+        use_repo_pex: bool = False,
+        description: str | None = None,
+        expected: str,
+    ) -> None:
+        repo_pex = Pex(EMPTY_DIGEST, "repo.pex", None) if use_repo_pex else None
+        request = PexRequest(
+            output_filename="new.pex",
+            internal_only=True,
+            requirements=requirements,
+            description=description,
+            repository_pex=repo_pex,
+        )
+        assert _build_pex_description(request) == expected
+
+    assert_description(PexRequirements(), description="Custom!", expected="Custom!")
+    assert_description(
+        PexRequirements(), description="Custom!", use_repo_pex=True, expected="Custom!"
+    )
+
+    assert_description(PexRequirements(), expected="Building new.pex")
+    assert_description(PexRequirements(), use_repo_pex=True, expected="Building new.pex")
+
+    assert_description(
+        PexRequirements(["req"]), expected="Building new.pex with 1 requirement: req"
+    )
+    assert_description(
+        PexRequirements(["req"]),
+        use_repo_pex=True,
+        expected="Extracting 1 requirement to build new.pex from repo.pex: req",
+    )
+
+    assert_description(
+        PexRequirements(["req1", "req2"]),
+        expected="Building new.pex with 2 requirements: req1, req2",
+    )
+    assert_description(
+        PexRequirements(["req1", "req2"]),
+        use_repo_pex=True,
+        expected="Extracting 2 requirements to build new.pex from repo.pex: req1, req2",
+    )