devrev-Python-SDK 3.1.0__py3-none-any.whl → 3.2.0__py3-none-any.whl

devrev/client.py

@@ -160,7 +160,7 @@ class DevRevClient:
         self._dev_users = DevUsersService(self._http)
         self._groups = GroupsService(self._http)
         self._links = LinksService(self._http)
-        self._parts = PartsService(self._http)
+        self._parts = PartsService(self._http, parent_client=self)
         self._rev_orgs = RevOrgsService(self._http)
         self._rev_users = RevUsersService(self._http)
         self._slas = SlasService(self._http)
@@ -488,7 +488,7 @@ class AsyncDevRevClient:
         self._dev_users = AsyncDevUsersService(self._http)
         self._groups = AsyncGroupsService(self._http)
         self._links = AsyncLinksService(self._http)
-        self._parts = AsyncPartsService(self._http)
+        self._parts = AsyncPartsService(self._http, parent_client=self)
         self._rev_orgs = AsyncRevOrgsService(self._http)
         self._rev_users = AsyncRevUsersService(self._http)
         self._slas = AsyncSlasService(self._http)

devrev/models/__init__.py

@@ -217,6 +217,7 @@ from devrev.models.notifications import (
     NotificationsSendResponse,
 )
 from devrev.models.parts import (
+    ParentPartFilter,
     Part,
     PartsCreateRequest,
     PartsCreateResponse,
@@ -226,6 +227,9 @@ from devrev.models.parts import (
     PartsGetResponse,
     PartsListRequest,
     PartsListResponse,
+    PartsMovePlan,
+    PartsMoveRequest,
+    PartsMoveResult,
     PartSummary,
     PartsUpdateRequest,
     PartsUpdateResponse,
@@ -669,6 +673,7 @@ __all__ = [
     "NotificationsSendRequest",
     "NotificationsSendResponse",
     # Parts
+    "ParentPartFilter",
     "Part",
     "PartSummary",
     "PartType",
@@ -682,6 +687,9 @@ __all__ = [
     "PartsUpdateResponse",
     "PartsDeleteRequest",
     "PartsDeleteResponse",
+    "PartsMoveRequest",
+    "PartsMovePlan",
+    "PartsMoveResult",
     # Preferences
     "Preferences",
     "PreferencesGetRequest",

devrev/models/parts.py

@@ -11,6 +11,7 @@ from devrev.models.base import (
     DevRevBaseModel,
     DevRevResponseModel,
     PaginatedResponse,
+    TagWithValue,
     UserSummary,
 )
 
@@ -33,6 +34,7 @@ class Part(DevRevResponseModel):
     type: PartType | None = Field(default=None, description="Part type")
     description: str | None = Field(default=None, description="Description")
     owned_by: list[UserSummary] | None = Field(default=None, description="Owners")
+    tags: list[TagWithValue] | None = Field(default=None, description="Tags")
     created_date: datetime | None = Field(default=None, description="Creation date")
     modified_date: datetime | None = Field(default=None, description="Last modified")
 
@@ -87,12 +89,32 @@ class PartsDeleteRequest(DevRevBaseModel):
     id: str = Field(..., description="Part ID to delete")
 
 
+class ParentPartFilter(DevRevBaseModel):
+    """Hierarchy filter for ``parts.list`` to fetch parts under given parents.
+
+    Mirrors the DevRev ``parent_part`` query filter. ``parts`` lists the parent
+    part IDs to fetch the hierarchy for (required), and ``level`` optionally
+    bounds how many levels of the hierarchy to return (``level=1`` returns the
+    direct children of the given parents).
+    """
+
+    parts: list[str] = Field(
+        ..., min_length=1, description="Parent part IDs to fetch the hierarchy for"
+    )
+    level: int | None = Field(
+        default=None, ge=1, description="Number of levels of the hierarchy to fetch"
+    )
+
+
 class PartsListRequest(DevRevBaseModel):
     """Request to list parts."""
 
     cursor: str | None = Field(default=None, description="Pagination cursor")
     limit: int | None = Field(default=None, ge=1, le=100, description="Max results")
     type: list[PartType] | None = Field(default=None, description="Filter by type")
+    parent_part: ParentPartFilter | None = Field(
+        default=None, description="Filter by parent part hierarchy"
+    )
 
 
 class PartsUpdateRequest(DevRevBaseModel):
@@ -131,3 +153,75 @@ class PartsDeleteResponse(DevRevResponseModel):
     """Response from deleting a part."""
 
     pass
+
+
+class PartsMoveRequest(DevRevBaseModel):
+    """Request to move (re-parent) a part.
+
+    The DevRev REST API only accepts ``parent_part`` on ``parts.create``, not
+    ``parts.update``. This SDK-level convenience request describes a higher-level
+    "move" operation that re-parents a part by recreating it under a new parent
+    and relinking its dependents. It is NOT a direct DevRev API request body.
+    """
+
+    id: str = Field(..., description="ID of the part to move (re-parent)")
+    new_parent_part: str = Field(
+        ..., description="Target parent part ID under which the part should be moved"
+    )
+    dry_run: bool = Field(
+        default=False,
+        description="If True, compute and return the move plan without executing it",
+    )
+
+
+class PartsMovePlan(DevRevResponseModel):
+    """Plan describing what a part move (re-parent) would do.
+
+    Computed before (or instead of, for a dry run) executing a move so callers
+    can review the dependents that will be relinked or re-parented.
+    """
+
+    source_part_id: str = Field(..., description="ID of the part to be moved")
+    source_part_type: str | None = Field(
+        default=None, description="Type of the source part (product, capability, etc.)"
+    )
+    new_parent_part: str = Field(
+        ..., description="Target parent part ID the source will be moved under"
+    )
+    work_items_to_relink: list[str] = Field(
+        default_factory=list,
+        description="IDs of work items currently applies_to the source part that will be relinked",
+    )
+    child_parts_to_reparent: list[str] = Field(
+        default_factory=list,
+        description="IDs of child parts of the source that will be re-parented",
+    )
+    will_delete_source: bool = Field(
+        default=False,
+        description="Whether the original source part will be deleted after the move",
+    )
+
+
+class PartsMoveResult(DevRevResponseModel):
+    """Outcome of an executed (or dry-run) part move (re-parent) operation."""
+
+    new_part_id: str = Field(..., description="ID of the newly created part under the new parent")
+    source_part_id: str = Field(..., description="ID of the original source part that was moved")
+    relinked_work_items: list[str] = Field(
+        default_factory=list,
+        description="IDs of work items that were relinked from the source to the new part",
+    )
+    reparented_children: list[str] = Field(
+        default_factory=list,
+        description="IDs of child parts that were re-parented under the new part",
+    )
+    source_deleted: bool = Field(
+        default=False, description="Whether the original source part was deleted"
+    )
+    dry_run: bool = Field(
+        default=False,
+        description="Whether this result describes a dry run (no changes were applied)",
+    )
+    plan: PartsMovePlan = Field(
+        ..., description="The move plan that was computed for this operation"
+    )

devrev/services/parts.py

@@ -2,7 +2,12 @@
 
 from __future__ import annotations
 
+from typing import TYPE_CHECKING
+
+from devrev.exceptions import DevRevError
+from devrev.models.base import TagWithValue, UserSummary
 from devrev.models.parts import (
+    ParentPartFilter,
     Part,
     PartsCreateRequest,
     PartsCreateResponse,
@@ -12,11 +17,47 @@ from devrev.models.parts import (
     PartsGetResponse,
     PartsListRequest,
     PartsListResponse,
+    PartsMovePlan,
+    PartsMoveRequest,
+    PartsMoveResult,
     PartsUpdateRequest,
     PartsUpdateResponse,
 )
 from devrev.services.base import AsyncBaseService, BaseService
 
+if TYPE_CHECKING:
+    from devrev.client import AsyncDevRevClient, DevRevClient
+
+# Module-level type alias. Declared here so method-scoped annotations can refer
+# to ``list[str]`` without colliding with the ``list`` method defined on
+# :class:`PartsService` / :class:`AsyncPartsService`.
+_StrList = list[str]
+
+# Sentinel returned as ``new_part_id`` for a dry-run move: no part is created,
+# so there is no real ID to report.
+_DRY_RUN_NEW_PART_ID = ""
+
+
+def _owned_by_ids(owned_by: list[UserSummary] | None) -> list[str]:
+    """Map a part's ``owned_by`` UserSummary list to a list of owner IDs."""
+    if not owned_by:
+        return []
+    return [owner.id for owner in owned_by]
+
+
+def _tag_ids(tags: list[TagWithValue] | None) -> list[str]:
+    """Map a part's ``tags`` to a list of tag ID strings for recreation.
+
+    ``TagWithValue.tag`` is a full ``Tag`` object in responses but a tag ID
+    string in requests; this normalizes both forms to the ID string.
+    """
+    if not tags:
+        return []
+    ids: list[str] = []
+    for tag in tags:
+        ids.append(tag.tag if isinstance(tag.tag, str) else tag.tag.id)
+    return ids
+
 
 class PartsService(BaseService):
     """Service for managing DevRev Parts."""
@@ -36,17 +77,21 @@ class PartsService(BaseService):
         *,
         limit: int | None = None,
         cursor: str | None = None,
+        parent_part: ParentPartFilter | None = None,
     ) -> PartsListResponse:
         """List parts.
 
         Args:
             limit: Maximum number of results to return (1-100).
             cursor: Pagination cursor from previous response.
+            parent_part: Hierarchy filter to return parts under given parents
+                (e.g. ``ParentPartFilter(parts=[parent_id], level=1)`` for the
+                direct children of ``parent_id``).
 
         Returns:
             PartsListResponse with parts and next_cursor for pagination.
         """
-        request = PartsListRequest(limit=limit, cursor=cursor)
+        request = PartsListRequest(limit=limit, cursor=cursor, parent_part=parent_part)
         return self._post("/parts.list", request, PartsListResponse)
 
     def update(self, request: PartsUpdateRequest) -> Part:
@@ -58,6 +103,198 @@ class PartsService(BaseService):
         """Delete a part."""
         self._post("/parts.delete", request, PartsDeleteResponse)
 
+    def _list_child_part_ids(self, source_id: str) -> _StrList:
+        """Return the IDs of the direct child parts of ``source_id``.
+
+        Pages through ``parts.list`` with a ``parent_part`` hierarchy filter
+        (level 1). The source part itself is excluded from the result.
+        """
+        child_ids: _StrList = []
+        cursor: str | None = None
+        while True:
+            page = self.list(
+                cursor=cursor,
+                parent_part=ParentPartFilter(parts=[source_id], level=1),
+            )
+            for part in page.parts:
+                if part.id != source_id:
+                    child_ids.append(part.id)
+            if not page.next_cursor:
+                break
+            cursor = page.next_cursor
+        return child_ids
+
+    def _collect_descendant_ids(self, source_id: str) -> set[str]:
+        """Return the IDs of every descendant of ``source_id`` (whole subtree).
+
+        Performs a bounded breadth-first walk over the part hierarchy using the
+        existing ``_list_child_part_ids`` (level-1) lookup at each node. A
+        ``visited`` set guards against re-processing a node, so a malformed
+        hierarchy that contains a cycle cannot cause an infinite loop. The
+        source itself is never included in the returned set.
+        """
+        descendants: set[str] = set()
+        visited: set[str] = {source_id}
+        frontier: _StrList = [source_id]
+        while frontier:
+            current = frontier.pop()
+            for child_id in self._list_child_part_ids(current):
+                if child_id in visited:
+                    continue
+                visited.add(child_id)
+                descendants.add(child_id)
+                frontier.append(child_id)
+        return descendants
+
+    def _validate_move_target(self, source: Part, request: PartsMoveRequest) -> None:
+        """Reject self-moves and cycles before any mutation occurs.
+
+        A move whose ``new_parent_part`` is the source itself (self-move) or a
+        descendant of the source (cycle) would, under the recreate-and-relink
+        workaround, create the replacement part under the soon-to-be-deleted
+        source and then delete it -- destroying the new part (data loss). This
+        guard runs read-only lookups only and raises before create/relink/delete.
+        """
+        if request.new_parent_part == source.id:
+            raise DevRevError(
+                f"Cannot move part {source.id!r} under itself "
+                f"(new_parent_part == id): a self-move would orphan and delete "
+                "the recreated part."
+            )
+        if request.new_parent_part in self._collect_descendant_ids(source.id):
+            raise DevRevError(
+                f"Cannot move part {source.id!r} under its own descendant "
+                f"{request.new_parent_part!r}: this would create a cycle and "
+                "destroy the recreated part when the source is deleted."
+            )
+
+    def _list_applies_to_work_ids(self, source_id: str) -> _StrList:
+        """Return the IDs of all work items that apply to ``source_id``.
+
+        Uses the works service (via the parent client) and paginates fully.
+        """
+        works = self._require_parent_client().works
+        work_ids: _StrList = []
+        cursor: str | None = None
+        while True:
+            page = works.list(applies_to_part=[source_id], cursor=cursor)
+            work_ids.extend(work.id for work in page.works)
+            if not page.next_cursor:
+                break
+            cursor = page.next_cursor
+        return work_ids
+
+    def _require_parent_client(self) -> DevRevClient:
+        """Return the parent client, raising if it was not provided."""
+        if not self._parent_client:
+            raise DevRevError(
+                "move requires a parent client reference. "
+                "Ensure the parts service is accessed via DevRevClient.parts."
+            )
+        return self._parent_client
+
+    def _compute_move_plan(self, source: Part, request: PartsMoveRequest) -> PartsMovePlan:
+        """Compute the plan describing what moving ``source`` would do."""
+        return PartsMovePlan(
+            source_part_id=source.id,
+            source_part_type=source.type.value if source.type else None,
+            new_parent_part=request.new_parent_part,
+            work_items_to_relink=self._list_applies_to_work_ids(source.id),
+            child_parts_to_reparent=self._list_child_part_ids(source.id),
+            will_delete_source=not request.dry_run,
+        )
+
+    def move(self, request: PartsMoveRequest) -> PartsMoveResult:
+        """Move (re-parent) a part under a new parent part.
+
+        The DevRev REST API does not allow changing ``parent_part`` on
+        ``parts.update`` -- only ``parts.create`` accepts it. This method
+        therefore implements a recreate-and-relink workaround: it creates a new
+        part of the same type under ``new_parent_part`` (preserving name,
+        description, owners and tags), relinks every work item that applied to
+        the source onto the new part, re-parents the source's direct children,
+        and finally deletes the original source part.
+
+        Order of operations for a real move: create the new part first, relink
+        work items and re-parent children to the new part, then delete the
+        source last. If any relink or re-parent fails, the source part is left
+        intact (not deleted) so nothing is orphaned and the error is surfaced.
+
+        Before any mutation (and before computing the plan), the request is
+        validated against self-moves and cycles: ``new_parent_part`` may not be
+        the source itself nor any descendant of the source. Both checks run
+        read-only lookups only and apply to dry runs as well, since such a
+        request is invalid regardless of ``dry_run``.
+
+        Args:
+            request: The move request (source id, new parent id, dry_run flag).
+
+        Returns:
+            A fully-populated :class:`PartsMoveResult`. For a dry run,
+            ``new_part_id`` is an empty-string sentinel, ``source_deleted`` is
+            False, and ``plan`` describes what would happen with no mutations.
+
+        Raises:
+            DevRevError: If no parent client is available; if ``new_parent_part``
+                is the source itself (self-move) or one of its descendants
+                (cycle); or if a relink or re-parent fails (the source is not
+                deleted in that case).
+        """
+        source = self.get(PartsGetRequest(id=request.id))
+        self._validate_move_target(source, request)
+        plan = self._compute_move_plan(source, request)
+
+        if request.dry_run:
+            return PartsMoveResult(
+                new_part_id=_DRY_RUN_NEW_PART_ID,
+                source_part_id=source.id,
+                relinked_work_items=[],
+                reparented_children=[],
+                source_deleted=False,
+                dry_run=True,
+                plan=plan,
+            )
+
+        if source.type is None:
+            raise DevRevError(
+                f"Cannot move part {source.id!r}: its type is unknown, "
+                "so the replacement part cannot be created with the same type."
+            )
+
+        new_part = self.create(
+            PartsCreateRequest(
+                name=source.name,
+                type=source.type,
+                description=source.description,
+                owned_by=_owned_by_ids(source.owned_by) or None,
+                parent_part=[request.new_parent_part],
+                tags=_tag_ids(source.tags) or None,
+            )
+        )
+
+        works = self._require_parent_client().works
+        relinked: _StrList = []
+        for work_id in plan.work_items_to_relink:
+            works.update(work_id, applies_to_part=new_part.id)
+            relinked.append(work_id)
+
+        reparented: _StrList = []
+        for child_id in plan.child_parts_to_reparent:
+            self.move(PartsMoveRequest(id=child_id, new_parent_part=new_part.id, dry_run=False))
+            reparented.append(child_id)
+
+        self.delete(PartsDeleteRequest(id=source.id))
+
+        return PartsMoveResult(
+            new_part_id=new_part.id,
+            source_part_id=source.id,
+            relinked_work_items=relinked,
+            reparented_children=reparented,
+            source_deleted=True,
+            dry_run=False,
+            plan=plan,
+        )
+
 
 class AsyncPartsService(AsyncBaseService):
     """Async service for managing DevRev Parts."""
@@ -77,17 +314,21 @@ class AsyncPartsService(AsyncBaseService):
         *,
         limit: int | None = None,
         cursor: str | None = None,
+        parent_part: ParentPartFilter | None = None,
     ) -> PartsListResponse:
         """List parts.
 
         Args:
             limit: Maximum number of results to return (1-100).
             cursor: Pagination cursor from previous response.
+            parent_part: Hierarchy filter to return parts under given parents
+                (e.g. ``ParentPartFilter(parts=[parent_id], level=1)`` for the
+                direct children of ``parent_id``).
 
         Returns:
             PartsListResponse with parts and next_cursor for pagination.
         """
-        request = PartsListRequest(limit=limit, cursor=cursor)
+        request = PartsListRequest(limit=limit, cursor=cursor, parent_part=parent_part)
         return await self._post("/parts.list", request, PartsListResponse)
 
     async def update(self, request: PartsUpdateRequest) -> Part:
@@ -98,3 +339,197 @@ class AsyncPartsService(AsyncBaseService):
     async def delete(self, request: PartsDeleteRequest) -> None:
         """Delete a part."""
         await self._post("/parts.delete", request, PartsDeleteResponse)
+
+    async def _list_child_part_ids(self, source_id: str) -> _StrList:
+        """Return the IDs of the direct child parts of ``source_id``.
+
+        Pages through ``parts.list`` with a ``parent_part`` hierarchy filter
+        (level 1). The source part itself is excluded from the result.
+        """
+        child_ids: _StrList = []
+        cursor: str | None = None
+        while True:
+            page = await self.list(
+                cursor=cursor,
+                parent_part=ParentPartFilter(parts=[source_id], level=1),
+            )
+            for part in page.parts:
+                if part.id != source_id:
+                    child_ids.append(part.id)
+            if not page.next_cursor:
+                break
+            cursor = page.next_cursor
+        return child_ids
+
+    async def _collect_descendant_ids(self, source_id: str) -> set[str]:
+        """Return the IDs of every descendant of ``source_id`` (whole subtree).
+
+        Performs a bounded breadth-first walk over the part hierarchy using the
+        existing ``_list_child_part_ids`` (level-1) lookup at each node. A
+        ``visited`` set guards against re-processing a node, so a malformed
+        hierarchy that contains a cycle cannot cause an infinite loop. The
+        source itself is never included in the returned set.
+        """
+        descendants: set[str] = set()
+        visited: set[str] = {source_id}
+        frontier: _StrList = [source_id]
+        while frontier:
+            current = frontier.pop()
+            for child_id in await self._list_child_part_ids(current):
+                if child_id in visited:
+                    continue
+                visited.add(child_id)
+                descendants.add(child_id)
+                frontier.append(child_id)
+        return descendants
+
+    async def _validate_move_target(self, source: Part, request: PartsMoveRequest) -> None:
+        """Reject self-moves and cycles before any mutation occurs.
+
+        A move whose ``new_parent_part`` is the source itself (self-move) or a
+        descendant of the source (cycle) would, under the recreate-and-relink
+        workaround, create the replacement part under the soon-to-be-deleted
+        source and then delete it -- destroying the new part (data loss). This
+        guard runs read-only lookups only and raises before create/relink/delete.
+        """
+        if request.new_parent_part == source.id:
+            raise DevRevError(
+                f"Cannot move part {source.id!r} under itself "
+                f"(new_parent_part == id): a self-move would orphan and delete "
+                "the recreated part."
+            )
+        if request.new_parent_part in await self._collect_descendant_ids(source.id):
+            raise DevRevError(
+                f"Cannot move part {source.id!r} under its own descendant "
+                f"{request.new_parent_part!r}: this would create a cycle and "
+                "destroy the recreated part when the source is deleted."
+            )
+
+    async def _list_applies_to_work_ids(self, source_id: str) -> _StrList:
+        """Return the IDs of all work items that apply to ``source_id``.
+
+        Uses the works service (via the parent client) and paginates fully.
+        """
+        works = self._require_parent_client().works
+        work_ids: _StrList = []
+        cursor: str | None = None
+        while True:
+            page = await works.list(applies_to_part=[source_id], cursor=cursor)
+            work_ids.extend(work.id for work in page.works)
+            if not page.next_cursor:
+                break
+            cursor = page.next_cursor
+        return work_ids
+
+    def _require_parent_client(self) -> AsyncDevRevClient:
+        """Return the parent client, raising if it was not provided."""
+        if not self._parent_client:
+            raise DevRevError(
+                "move requires a parent client reference. "
+                "Ensure the parts service is accessed via AsyncDevRevClient.parts."
+            )
+        return self._parent_client
+
+    async def _compute_move_plan(self, source: Part, request: PartsMoveRequest) -> PartsMovePlan:
+        """Compute the plan describing what moving ``source`` would do."""
+        return PartsMovePlan(
+            source_part_id=source.id,
+            source_part_type=source.type.value if source.type else None,
+            new_parent_part=request.new_parent_part,
+            work_items_to_relink=await self._list_applies_to_work_ids(source.id),
+            child_parts_to_reparent=await self._list_child_part_ids(source.id),
+            will_delete_source=not request.dry_run,
+        )
+
+    async def move(self, request: PartsMoveRequest) -> PartsMoveResult:
+        """Move (re-parent) a part under a new parent part.
+
+        The DevRev REST API does not allow changing ``parent_part`` on
+        ``parts.update`` -- only ``parts.create`` accepts it. This method
+        therefore implements a recreate-and-relink workaround: it creates a new
+        part of the same type under ``new_parent_part`` (preserving name,
+        description, owners and tags), relinks every work item that applied to
+        the source onto the new part, re-parents the source's direct children,
+        and finally deletes the original source part.
+
+        Order of operations for a real move: create the new part first, relink
+        work items and re-parent children to the new part, then delete the
+        source last. If any relink or re-parent fails, the source part is left
+        intact (not deleted) so nothing is orphaned and the error is surfaced.
+
+        Before any mutation (and before computing the plan), the request is
+        validated against self-moves and cycles: ``new_parent_part`` may not be
+        the source itself nor any descendant of the source. Both checks run
+        read-only lookups only and apply to dry runs as well, since such a
+        request is invalid regardless of ``dry_run``.
+
+        Args:
+            request: The move request (source id, new parent id, dry_run flag).
+
+        Returns:
+            A fully-populated :class:`PartsMoveResult`. For a dry run,
+            ``new_part_id`` is an empty-string sentinel, ``source_deleted`` is
+            False, and ``plan`` describes what would happen with no mutations.
+
+        Raises:
+            DevRevError: If no parent client is available; if ``new_parent_part``
+                is the source itself (self-move) or one of its descendants
+                (cycle); or if a relink or re-parent fails (the source is not
+                deleted in that case).
+        """
+        source = await self.get(PartsGetRequest(id=request.id))
+        await self._validate_move_target(source, request)
+        plan = await self._compute_move_plan(source, request)
+
+        if request.dry_run:
+            return PartsMoveResult(
+                new_part_id=_DRY_RUN_NEW_PART_ID,
+                source_part_id=source.id,
+                relinked_work_items=[],
+                reparented_children=[],
+                source_deleted=False,
+                dry_run=True,
+                plan=plan,
+            )
+
+        if source.type is None:
+            raise DevRevError(
+                f"Cannot move part {source.id!r}: its type is unknown, "
+                "so the replacement part cannot be created with the same type."
+            )
+
+        new_part = await self.create(
+            PartsCreateRequest(
+                name=source.name,
+                type=source.type,
+                description=source.description,
+                owned_by=_owned_by_ids(source.owned_by) or None,
+                parent_part=[request.new_parent_part],
+                tags=_tag_ids(source.tags) or None,
+            )
+        )
+
+        works = self._require_parent_client().works
+        relinked: _StrList = []
+        for work_id in plan.work_items_to_relink:
+            await works.update(work_id, applies_to_part=new_part.id)
+            relinked.append(work_id)
+
+        reparented: _StrList = []
+        for child_id in plan.child_parts_to_reparent:
+            await self.move(
+                PartsMoveRequest(id=child_id, new_parent_part=new_part.id, dry_run=False)
+            )
+            reparented.append(child_id)
+
+        await self.delete(PartsDeleteRequest(id=source.id))
+
+        return PartsMoveResult(
+            new_part_id=new_part.id,
+            source_part_id=source.id,
+            relinked_work_items=relinked,
+            reparented_children=reparented,
+            source_deleted=True,
+            dry_run=False,
+            plan=plan,
+        )

devrev/services/works.py

@@ -194,6 +194,7 @@ class WorksService(BaseService):
         *,
         title: str | None = None,
         body: str | None = None,
+        applies_to_part: str | None = None,
         owned_by: Sequence[str] | None = None,
         stage: StageUpdate | None = None,
         priority: IssuePriority | None = None,
@@ -206,6 +207,7 @@ class WorksService(BaseService):
             id: Work item ID
             title: New title
             body: New body/description
+            applies_to_part: New part ID this work applies to
             owned_by: New owner IDs
             stage: New stage
             priority: New priority (for issues)
@@ -220,6 +222,7 @@ class WorksService(BaseService):
             id=id,
             title=title,
             body=body,
+            applies_to_part=applies_to_part,
             owned_by=owned_by_update,
             stage=stage,
             priority=priority,
@@ -471,16 +474,31 @@ class AsyncWorksService(AsyncBaseService):
         *,
         title: str | None = None,
         body: str | None = None,
+        applies_to_part: str | None = None,
         owned_by: Sequence[str] | None = None,
         priority: IssuePriority | None = None,
         severity: TicketSeverity | None = None,
     ) -> Work:
-        """Update a work item."""
+        """Update a work item.
+
+        Args:
+            id: Work item ID
+            title: New title
+            body: New body/description
+            applies_to_part: New part ID this work applies to
+            owned_by: New owner IDs
+            priority: New priority (for issues)
+            severity: New severity (for tickets)
+
+        Returns:
+            The updated Work item
+        """
         owned_by_update = WorksUpdateRequestOwnedBy(set=owned_by) if owned_by else None
         request = WorksUpdateRequest(
             id=id,
             title=title,
             body=body,
+            applies_to_part=applies_to_part,
             owned_by=owned_by_update,
             priority=priority,
             severity=severity,

devrev_mcp/tools/parts.py

@@ -16,6 +16,7 @@ from devrev.models.parts import (
     PartsCreateRequest,
     PartsDeleteRequest,
     PartsGetRequest,
+    PartsMoveRequest,
     PartsUpdateRequest,
     PartType,
 )
@@ -191,3 +192,55 @@ if _config.enable_destructive_tools:
             return {"success": True, "message": f"Part {id} deleted successfully"}
         except DevRevError as e:
             raise RuntimeError(format_devrev_error(e)) from e
+
+    @mcp.tool()
+    async def devrev_parts_move(
+        ctx: Context[Any, Any, Any],
+        id: str,
+        new_parent_part: str,
+        dry_run: bool = False,
+    ) -> dict[str, Any]:
+        """Move (re-parent) a DevRev part under a new parent part.
+
+        This is NOT an in-place update. The DevRev REST API does not allow
+        changing a part's parent, so this operation RECREATES the part under the
+        new parent (giving it a NEW id), relinks dependents (work items that
+        apply to the source part and child parts of the source), and then, on a
+        real run, DELETES the original source part.
+
+        When ``dry_run`` is True, the move plan is computed and returned WITHOUT
+        mutating anything: no part is created, no dependents are relinked, and
+        the source part is not deleted. Use a dry run to review the impact
+        (``plan.work_items_to_relink``, ``plan.child_parts_to_reparent``,
+        ``plan.will_delete_source``) before performing a real move.
+
+        Args:
+            id: The ID of the part to move (re-parent). This part is recreated
+                under the new parent and, on a real run, deleted.
+            new_parent_part: The ID of the target parent part to move under.
+            dry_run: If True, return the computed move plan without applying any
+                changes. Defaults to False (perform the move).
+
+        Returns:
+            Dictionary containing the serialized move result, including the
+            new part ID, the original source part ID, the relinked work items,
+            the re-parented child parts, whether the source was deleted, the
+            dry-run flag, and the nested move plan.
+
+        Raises:
+            RuntimeError: If a provided ID is not a valid part DON ID, or if the
+                DevRev API call fails.
+        """
+        validate_don_id(id, "part", "devrev_parts_move")
+        validate_don_id(new_parent_part, "part", "devrev_parts_move")
+        app = ctx.request_context.lifespan_context
+        try:
+            request = PartsMoveRequest(
+                id=id,
+                new_parent_part=new_parent_part,
+                dry_run=dry_run,
+            )
+            result = await app.get_client().parts.move(request)
+            return serialize_model(result)
+        except DevRevError as e:
+            raise RuntimeError(format_devrev_error(e)) from e

{devrev_python_sdk-3.1.0.dist-info → devrev_python_sdk-3.2.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 3.1.0
+Version: 3.2.0
 Summary: A modern, type-safe Python SDK for the DevRev API
 Project-URL: Homepage, https://github.com/mgmonteleone/py-dev-rev
 Project-URL: Documentation, https://github.com/mgmonteleone/py-dev-rev
@@ -505,6 +505,57 @@ export_result = client.works.export(
 )
 ```
 
+### Managing Parts
+
+```python
+from devrev.models.parts import PartsCreateRequest, PartsMoveRequest, PartType
+
+# Create a feature under a capability
+feature = client.parts.create(PartsCreateRequest(
+    name="Bulk import",
+    type=PartType.FEATURE,
+    parent_part=["don:core:dvrv-us-1:devo/1:part/1"],
+))
+
+# Move (re-parent) a part under a new parent part
+#
+# The DevRev REST API only accepts `parent_part` on parts.create (not
+# parts.update), so move() implements a recreate-and-relink workaround: it
+# creates a new same-type part under the new parent (preserving name,
+# description, owners and tags), relinks work items (applies_to_part) and child
+# parts onto the new part, then deletes the original source part.
+#
+# CAVEAT: a real move is NOT an in-place update. It produces a NEW part id and
+# DELETES the original part. Update any references you hold to the old id.
+
+# Dry run first: returns the plan, mutates nothing.
+plan = client.parts.move(PartsMoveRequest(
+    id=feature.id,
+    new_parent_part="don:core:dvrv-us-1:devo/1:part/2",
+    dry_run=True,
+))
+print(plan.plan.work_items_to_relink)       # work items that would be relinked
+print(plan.plan.child_parts_to_reparent)    # child parts that would be re-parented
+print(plan.plan.will_delete_source)         # True -> the source part will be deleted
+assert plan.source_deleted is False         # nothing was changed by the dry run
+
+# Real move: creates the new part, relinks dependents, deletes the source.
+result = client.parts.move(PartsMoveRequest(
+    id=feature.id,
+    new_parent_part="don:core:dvrv-us-1:devo/1:part/2",
+))
+print(result.new_part_id)        # the NEW part id (the source id is now deleted)
+print(result.relinked_work_items)
+print(result.reparented_children)
+print(result.source_deleted)     # True
+```
+
+> **Note**: `move` requires the parts service be accessed via the client
+> (`client.parts`) because it coordinates with the works service to relink work
+> items. Via the MCP server, this is the `devrev_parts_move` tool, which is
+> gated behind `MCP_ENABLE_DESTRUCTIVE_TOOLS=true` (it creates and deletes
+> parts).
+
 ### Articles and Knowledge Base
 
 ```python
@@ -995,7 +1046,7 @@ All settings are configurable via environment variables (prefix `MCP_`):
 | Users | 5 | dev list/get, rev list/get/create |
 | Conversations | 6 | list, get, create, update, delete, export |
 | Articles | 6 | list, get, create, update, delete, count |
-| Parts | 5 | list, get, create, update, delete |
+| Parts | 6 | list, get, create, update, delete, move |
 | Tags | 5 | list, get, create, update, delete |
 | Groups | 8 | list, get, create, update, delete, add/remove member, count |
 | Timeline | 5 | list, get, create, update, delete |
@@ -1028,13 +1079,13 @@ py-devrev/
 │       ├── __main__.py         # CLI entry point
 │       ├── server.py           # FastMCP server setup & lifecycle
 │       ├── config.py           # MCPServerConfig (pydantic-settings)
-│       ├── tools/              # 78 MCP tools across 15 categories
+│       ├── tools/              # 79 MCP tools across 15 categories
 │       │   ├── works.py        # Works: list, get, create, update, delete, count, search
 │       │   ├── accounts.py     # Accounts: list, get, create, update, delete, merge
 │       │   ├── users.py        # Users: list/get dev users, list/get rev users, whoami
 │       │   ├── conversations.py # Conversations: list, get, create, update, delete, export
 │       │   ├── articles.py     # Articles: list, get, create, update, delete, count
-│       │   ├── parts.py        # Parts: list, get, create, update, delete
+│       │   ├── parts.py        # Parts: list, get, create, update, delete, move
 │       │   ├── tags.py         # Tags: list, get, create, update, delete
 │       │   ├── groups.py       # Groups: CRUD + member management + count
 │       │   ├── timeline.py     # Timeline: list, get, create, update, delete

{devrev_python_sdk-3.1.0.dist-info → devrev_python_sdk-3.2.0.dist-info}/RECORD

@@ -1,9 +1,9 @@
 devrev/__init__.py,sha256=iBKIRvS10mFWpBgnMjQQiQR04rgb8ZyXVk1-7cYJikI,1555
-devrev/client.py,sha256=FXEadZd0NzBUzXXY60e4qnN2Osv5vHE_9rj8nPpVPxI,26458
+devrev/client.py,sha256=WfbWOWJo34bBzh2FwAsJfrWMwHSPTHINYbDUI4eE-Dk,26498
 devrev/config.py,sha256=BKFhD8lMJvq-I9kpchiA50fYefVFhnNwcK9ReFogh4k,8122
 devrev/exceptions.py,sha256=Bs0pYgOgG26cyI785rdJeTR72gM-rXHaCEt24HAJzEk,7540
 devrev/py.typed,sha256=AbpHGcgLb-kRsJGnwFEktk7uzpZOCcBY74-YBdrKVGs,1
-devrev/models/__init__.py,sha256=D9J3kwZrAUJijh4wuLbfvb7WbokL84_dRg93OxtT_k4,24033
+devrev/models/__init__.py,sha256=0R2_Khol0gx7AYMKLLge5CMGKv2h6wCd31CQqG8-j_g,24209
 devrev/models/accounts.py,sha256=LrXAit2gIr34XyRO1T0fL6EwdXHsfJOyXwcegPnKnaE,6974
 devrev/models/articles.py,sha256=DgrZC_wtLsXIfilAvHJBoQkhbGDg3Vx5UX7h9MD63xA,16732
 devrev/models/artifacts.py,sha256=3iA58RyVlX0S0U5iID3Cnoc096Y70QfGqYfKEhxfE9g,4375
@@ -17,7 +17,7 @@ devrev/models/groups.py,sha256=oB3eCuX14po5E8WhqkuuC-GlC1WXEPVfpqB5tn4NZ-o,4486
 devrev/models/incidents.py,sha256=e8Wq7MyXeDPiYMCNvxPlZa4D939196fUaND-MYtBbcw,6305
 devrev/models/links.py,sha256=0jtIXn90WQgGrSrnR72AgM9lMVSK6xv9fXKzq2Eki-A,2963
 devrev/models/notifications.py,sha256=BjVGx6XR1rQbQPR1hz05j5wU-8VMqH2m02KZmMl3Z98,867
-devrev/models/parts.py,sha256=A1EjliQfmGfxGeg2NlrAGOXB5Qk8MCyhus_LmmTdakA,4233
+devrev/models/parts.py,sha256=PcE_N3Gd9YRhBoXwgYVhXYsVtVMAzM_nIcGEQ3Ut2Hc,8003
 devrev/models/preferences.py,sha256=S98RoSJhSlBavzFotyqsx_r1LGS2WjRtmazN4yIQFwc,1806
 devrev/models/question_answers.py,sha256=lPgE-OXzxMkrQdQE4NPz8IRV0I5P8iJq7VVdpx4iovg,3606
 devrev/models/recommendations.py,sha256=4szM-gtm-oVN7lBqi1xtGKABJcdwTeFLF_iR1fUoImY,2727
@@ -50,7 +50,7 @@ devrev/services/groups.py,sha256=0VpOKTwHiRjrKXrO1kdHo8V_z2I02GphYQhyDFnImxg,496
 devrev/services/incidents.py,sha256=YhrSj89FTVXI7xEw_vm0SMR5iIrnvhwu4MRdQqqhBx8,8076
 devrev/services/links.py,sha256=e6ppPQzdIp2tkJ_VEpkNhACZUq2scu2EfCr1S7NPh9s,2266
 devrev/services/notifications.py,sha256=uZZrNfpfEFOMChoAeVZR-Ow2NtPBgAtZ5Qv2mPN6Fvw,2201
-devrev/services/parts.py,sha256=Ork8lMFTTIJ-MJhsHoktpzx6lFQlxeZvOroNXVk10c8,3195
+devrev/services/parts.py,sha256=EQbuhDFIPKcCxljTTaLe5fo1gdwNhmmk0ZsBH8x8sj4,22120
 devrev/services/preferences.py,sha256=MXdBsPcAUcCO14hKrIjhHkFj_XD1OZbtTHnOGZaWTwg,3327
 devrev/services/question_answers.py,sha256=bDgbtjI3RCHjJLwI7JsSgvxNh7OITNQZrNigq9CqL8E,4332
 devrev/services/recommendations.py,sha256=_UV9BBlxwfrek69Q5LzSZRE2D43FR3BSWUhRMDHeXWM,1962
@@ -63,7 +63,7 @@ devrev/services/timeline_entries.py,sha256=6nbVPcWlE3-ohLhvkM421eqYS5ztKtHmGsPek
 devrev/services/track_events.py,sha256=lI4wXkWu3uUuXtuRg1MGNkTZ7B0Lc1PjM8Kw-6sUnWc,1439
 devrev/services/uoms.py,sha256=AA3ymoHj24FIbsZpYC4tg2elSdQ3iINTVOz7MraZcj8,8163
 devrev/services/webhooks.py,sha256=-TSkcaya1y48WB24_vHd-bqO5xSqxRsCLiilncNzQZU,3917
-devrev/services/works.py,sha256=hAK7ZUqLph5Db15p4RDbeRFLZUALAF5tCFOQSYytj_g,20456
+devrev/services/works.py,sha256=wUVQ8EcBs7Ogn2x219G37HL_WTYI0TCdxn-KPOD3IaQ,21064
 devrev/utils/__init__.py,sha256=NOrbpkjDVLH8n9xf-xpZJiIIa_GVI_6vqTm3E8L3Udw,857
 devrev/utils/content_converter.py,sha256=emRBLiVoOfDGpPDzrMRnqQr4-QkqN13OdWlYOyU_LCg,28141
 devrev/utils/deprecation.py,sha256=7qB2Dx531oP7mNi7q2txOYsOKC9YwdHqlKPMFHOW9Ws,1275
@@ -101,7 +101,7 @@ devrev_mcp/tools/engagements.py,sha256=21zsMLlgY1FwKzSIxPf6XVdMY77p8NXrWZ7l8PRKI
 devrev_mcp/tools/groups.py,sha256=E29eptwl_1rbmMPPElcSblWs71t1XdvSZIl1UR8TZr0,8456
 devrev_mcp/tools/incidents.py,sha256=XO9FaZlLHCXULCFpMpCOftYNfvajH5PeXOkW2uUZ_Z4,7984
 devrev_mcp/tools/links.py,sha256=oma8sLKaz0i3BnQmdoXdneZJx62j2P5YHbLag2aLNkU,4230
-devrev_mcp/tools/parts.py,sha256=4Yw4EamM2Q0o7a7ejfwIjTEdvcaV5KUnzcloGMkJCdk,6499
+devrev_mcp/tools/parts.py,sha256=nyEBYznHbhAzR0zPu0-f24zLRKrTKdXpb6luQ4cZ-_I,8906
 devrev_mcp/tools/question_answers.py,sha256=57p4bZVQ52wGseBAamI-2qJoiZVbMRo46NNuOtGGhOo,5272
 devrev_mcp/tools/recommendations.py,sha256=Cg1NeDYgIlWLrtFYuxGlvQLpy6uDm2HoG91yGoSRC8A,3014
 devrev_mcp/tools/rev_orgs.py,sha256=Uge-rORmFTHOD1hhYy7pDLaGSxtisJsnn5ccazldfpU,5074
@@ -118,7 +118,7 @@ devrev_mcp/utils/don_id.py,sha256=jbVKXW_BeKjPHhz1F9p14fNmrXP2R36rgWZqdC4QlOc,63
 devrev_mcp/utils/errors.py,sha256=5mRAo76rJvvEVi6b1ZokPxDtX5JKkptaqmiYDLCkwBE,2110
 devrev_mcp/utils/formatting.py,sha256=6JssG5x1BxjdgSiQ8Ou3H-9Wo3wgWTWmejsrGez4wKc,2431
 devrev_mcp/utils/pagination.py,sha256=EOUgL-ZdSToM1Q-ydXmjhibsef5K1u1g3CaS9K8I2fY,1286
-devrev_python_sdk-3.1.0.dist-info/METADATA,sha256=2nIwUYQnBr8IAtS76Y-Fi7U3aFKhjAmunDexsvixqPk,40934
-devrev_python_sdk-3.1.0.dist-info/WHEEL,sha256=mffPy8wBnZQn2VnJUU5jE99KsxaSfiyMHV9Yt0aLVxs,87
-devrev_python_sdk-3.1.0.dist-info/entry_points.txt,sha256=XiV4J_yy0yzVZVxg7T66YERVIlqdPNp3O-NHTHkllqQ,63
-devrev_python_sdk-3.1.0.dist-info/RECORD,,
+devrev_python_sdk-3.2.0.dist-info/METADATA,sha256=boi9nuE-abiG-LLrdoLGjLyJLWZ5u9Vb21rG_1jViQQ,43054
+devrev_python_sdk-3.2.0.dist-info/WHEEL,sha256=mffPy8wBnZQn2VnJUU5jE99KsxaSfiyMHV9Yt0aLVxs,87
+devrev_python_sdk-3.2.0.dist-info/entry_points.txt,sha256=XiV4J_yy0yzVZVxg7T66YERVIlqdPNp3O-NHTHkllqQ,63
+devrev_python_sdk-3.2.0.dist-info/RECORD,,