Please address the comments from this code review:

## Overall Comments
- Extract the trailing-’t’ suffix parsing logic from both the install and remove commands into a shared helper to DRY up the code and keep behavior consistent.
- Add validation to error out or warn when both a trailing ‘t’ in the request and the --free-threaded flag are provided so users aren’t sending redundant or conflicting options.
- Consider pinning an upper bound for the pbs-installer dependency (e.g. <2026.0) even when using calver to guard against future breaking changes.

## Individual Comments

### Comment 1
<location> `src/poetry/console/commands/python/remove.py:100-110` </location>
<code_context>
    def handle(self) -> int:
        implementation = self.option("implementation").lower()
        free_threaded = self.option("free-threaded")

        result = 0

        for request in self.argument("python"):
            result += self.remove_python_installation(
                request, implementation, free_threaded, self.io
            )

        return result

</code_context>

<issue_to_address>
**issue (code-quality):** We've found these issues:

- Convert for loop into call to sum() ([`sum-comprehension`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/sum-comprehension/))
- Inline variable that is immediately returned ([`inline-immediately-returned-variable`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/inline-immediately-returned-variable/))
</issue_to_address>

### Comment 2
<location> `tests/console/commands/python/test_python_remove.py:110-114` </location>
<code_context>
@pytest.mark.parametrize("free_threaded", [True, False])
@pytest.mark.parametrize("option", [True, False])
def test_remove_version_free_threaded(
    tester: CommandTester,
    config: Config,
    mocked_poetry_managed_python_register: MockedPoetryPythonRegister,
    free_threaded: bool,
    option: bool,
) -> None:
    standard_path = mocked_poetry_managed_python_register("3.14.0", "cpython")
    free_threaded_path = mocked_poetry_managed_python_register(
        "3.14.0", "cpython", free_threaded=True
    )

    args = "3.14.0"
    if free_threaded:
        if option:
            args += " --free-threaded"
        else:
            args += "t"

    assert tester.execute(args) == 0, tester.io.fetch_error()

    details = "cpython"
    if free_threaded:
        details += ", free-threaded"
    assert (
        tester.io.fetch_output()
        == f"Removing installation 3.14.0 ({details}) ... Done\n"
    )
    if free_threaded:
        assert not free_threaded_path.exists()
        assert standard_path.exists()
    else:
        assert not standard_path.exists()
        assert free_threaded_path.exists()

</code_context>

<issue_to_address>
**suggestion (code-quality):** We've found these issues:

- Swap positions of nested conditionals ([`swap-nested-ifs`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/swap-nested-ifs/))
- Hoist nested repeated code outside conditional statements ([`hoist-similar-statement-from-if`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/hoist-similar-statement-from-if/))
- Replace if statement with if expression ([`assign-if-exp`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/assign-if-exp/))

```suggestion
        args += " --free-threaded" if option else "t"
```
</issue_to_address>