refactor: update work item relations endpoints to use direct identifiers for removal

Author: akhil-vamshi-konam

plane/api/work_items/custom_relations.py

@@ -4,7 +4,6 @@
 
 from ...models.work_items import (
     CreateWorkItemCustomRelation,
-    RemoveWorkItemCustomRelation,
     WorkItemWithRelationType,
 )
 from ..base_resource import BaseResource
@@ -68,17 +67,16 @@ def remove(
         workspace_slug: str,
         project_id: str,
         work_item_id: str,
-        data: RemoveWorkItemCustomRelation,
+        related_work_item_id: str,
     ) -> None:
         """Remove a custom relation between this work item and a target.
 
         Args:
             workspace_slug: The workspace slug identifier
             project_id: UUID of the project
             work_item_id: UUID of the work item
-            data: Removal payload containing the related work item UUID
+            related_work_item_id: UUID of the related work item to remove the relation with
         """
-        return self._post(
-            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/work-item-relations/remove/",
-            data.model_dump(exclude_none=True),
+        self._delete(
+            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/work-item-relations/{related_work_item_id}/"
         )

plane/api/work_items/dependencies.py

@@ -4,7 +4,6 @@
 
 from ...models.work_items import (
     CreateWorkItemDependency,
-    RemoveWorkItemDependency,
     WorkItemDependencyResponse,
     WorkItemWithRelationType,
 )
@@ -32,7 +31,7 @@ def list(
             work_item_id: UUID of the work item
         """
         response = self._get(
-            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/relation-dependencies/"
+            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/dependencies/"
         )
         return WorkItemDependencyResponse.model_validate(response)
 
@@ -52,7 +51,7 @@ def create(
             data: Dependency creation payload
         """
         response = self._post(
-            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/relation-dependencies/",
+            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/dependencies/",
             data.model_dump(exclude_none=True),
         )
         return [WorkItemWithRelationType.model_validate(item) for item in response]
@@ -62,17 +61,16 @@ def remove(
         workspace_slug: str,
         project_id: str,
         work_item_id: str,
-        data: RemoveWorkItemDependency,
+        related_work_item_id: str,
     ) -> None:
         """Remove a dependency relation between this work item and a target.
 
         Args:
             workspace_slug: The workspace slug identifier
             project_id: UUID of the project
             work_item_id: UUID of the work item
-            data: Removal payload containing the related work item UUID
+            related_work_item_id: UUID of the related work item to remove the dependency with
         """
-        return self._post(
-            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/relation-dependencies/remove/",
-            data.model_dump(exclude_none=True),
+        self._delete(
+            f"{workspace_slug}/projects/{project_id}/work-items/{work_item_id}/dependencies/{related_work_item_id}/"
         )

tests/unit/test_work_item_relations.py

@@ -15,8 +15,6 @@
     CreateWorkItem,
     CreateWorkItemCustomRelation,
     CreateWorkItemDependency,
-    RemoveWorkItemCustomRelation,
-    RemoveWorkItemDependency,
     WorkItemDependencyResponse,
     WorkItemWithRelationType,
 )
@@ -74,7 +72,7 @@ def custom_definition(client: PlaneClient, workspace_slug: str):
 
 
 class TestWorkItemDependencies:
-    """Tests for the /relation-dependencies/ endpoint."""
+    """Tests for the /dependencies/ endpoint."""
 
     def test_list_dependencies_empty(
         self, client: PlaneClient, workspace_slug: str, project: Project, work_item_a
@@ -154,7 +152,7 @@ def test_remove_dependency(
             workspace_slug,
             project.id,
             work_item_a.id,
-            RemoveWorkItemDependency(work_item_id=work_item_b.id),
+            work_item_b.id,
         )
         result = client.work_items.dependencies.list(workspace_slug, project.id, work_item_a.id)
         blocking_ids = [wi.id for wi in result.blocking]
@@ -195,7 +193,7 @@ def test_create_all_dependency_types(
                 workspace_slug,
                 project.id,
                 work_item_a.id,
-                RemoveWorkItemDependency(work_item_id=work_item_b.id),
+                work_item_b.id,
             )
         assert created_any
 
@@ -292,7 +290,7 @@ def test_remove_custom_relation(
             workspace_slug,
             project.id,
             work_item_a.id,
-            RemoveWorkItemCustomRelation(work_item_id=work_item_b.id),
+            work_item_b.id,
         )
         result = client.work_items.custom_relations.list(workspace_slug, project.id, work_item_a.id)
         outward_ids = [wi.id for wi in result.get(custom_definition.outward, [])]
@@ -327,7 +325,7 @@ def test_create_custom_relation_inward(
             workspace_slug,
             project.id,
             work_item_a.id,
-            RemoveWorkItemCustomRelation(work_item_id=work_item_b.id),
+            work_item_b.id,
         )
 
     def test_create_relation_invalid_definition(