fix: excluding non-existing groups for pdm remove

news/3404.bugfix.md

@@ -0,0 +1 @@
+Excluding non-existing groups for `pdm remove`.

src/pdm/cli/commands/remove.py

@@ -125,7 +125,7 @@ def do_remove(
         if sync:
             do_sync(
                 project,
-                selection=GroupSelection(project, default=False, groups=[group]),
+                selection=GroupSelection(project, default=False, groups=[group], exclude_non_existing=True),
                 clean=True,
                 no_editable=no_editable,
                 no_self=no_self,

src/pdm/cli/filters.py

@@ -23,13 +23,15 @@ def __init__(
         groups: Sequence[str] = (),
         group: str | None = None,
         excluded_groups: Sequence[str] = (),
+        exclude_non_existing: bool = False,
     ):
         self.project = project
         self.groups = groups
         self.group = group
         self.default = default
         self.dev = dev
         self.excluded_groups = excluded_groups
+        self.exclude_non_existing = exclude_non_existing
 
     @classmethod
     def from_options(cls, project: Project, options: argparse.Namespace) -> GroupSelection:
@@ -101,6 +103,10 @@ def _translated_groups(self) -> list[str]:
                 err=True,
             )
             groups_set -= invalid_groups
+
+        if self.exclude_non_existing and self.project.lockfile.groups is not None:
+            groups_set &= {normalize_name(g) for g in self.project.lockfile.groups}
+
         # Sorts the result in ascending order instead of in random order
         # to make this function pure
         result = sorted(groups_set, key=lambda x: (x != "default", x))

tests/cli/test_remove.py

@@ -105,3 +105,12 @@ def test_remove_group_not_in_lockfile(project, pdm, mocker):
     pdm(["remove", "--group", "tz", "pytz"], obj=project, strict=True)
     assert "optional-dependencies" not in project.pyproject.metadata
     locker.assert_not_called()
+
+
+@pytest.mark.usefixtures("working_set")
+def test_remove_exclude_non_existing_dev_group_in_lockfile(project, pdm):
+    pdm(["add", "requests"], obj=project, strict=True)
+    project.add_dependencies(["pytz"], to_group="tz", dev=True)
+    assert project.lockfile.groups == ["default"]
+    result = pdm(["remove", "requests"], obj=project)
+    assert result.exit_code == 0