devrev-Python-SDK 2.11.0__tar.gz → 2.12.0__tar.gz

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 2.11.0
+Version: 2.12.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

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "devrev-Python-SDK"
-version = "2.11.0"
+version = "2.12.0"
 description = "A modern, type-safe Python SDK for the DevRev API"
 readme = "README.md"
 requires-python = ">=3.11"

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/src/devrev_mcp/tools/parts.py

@@ -117,6 +117,12 @@ if _config.enable_destructive_tools:
                     f"Invalid part type: {e.args[0]}. "
                     f"Valid types: {', '.join(t.name for t in PartType)}"
                 ) from e
+            # Validate parent_part is provided for non-product parts
+            if part_type != PartType.PRODUCT and not parent_part:
+                raise RuntimeError(
+                    f"parent_part is required when creating a {part_type.name} part. "
+                    "Only PRODUCT parts can be created without a parent."
+                )
             request = PartsCreateRequest(
                 name=name,
                 type=part_type,

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/tests/unit/mcp/test_tools_parts.py

@@ -150,7 +150,12 @@ class TestPartsCreateTool:
         mock_part = _make_mock_part(id="FEAT-789", name="New Feature", type="feature")
         mock_client.parts.create.return_value = mock_part
 
-        result = await devrev_parts_create(mock_ctx, name="New Feature", type="FEATURE")
+        result = await devrev_parts_create(
+            mock_ctx,
+            name="New Feature",
+            type="FEATURE",
+            parent_part=["don:core:dvrv-us-1:devo/1:part/1"],
+        )
 
         assert result["id"] == "FEAT-789"
         call_args = mock_client.parts.create.call_args[0][0]
@@ -245,6 +250,52 @@ class TestPartsCreateTool:
                 type="invalid",
             )
 
+    @pytest.mark.asyncio
+    async def test_create_capability_without_parent_part_raises(self, mock_ctx, mock_client):
+        """Test that creating a CAPABILITY without parent_part raises RuntimeError. (#184)"""
+        with pytest.raises(RuntimeError, match="parent_part is required"):
+            await devrev_parts_create(
+                mock_ctx,
+                name="Orphan Capability",
+                type="capability",
+            )
+
+    @pytest.mark.asyncio
+    async def test_create_feature_without_parent_part_raises(self, mock_ctx, mock_client):
+        """Test that creating a FEATURE without parent_part raises RuntimeError. (#184)"""
+        with pytest.raises(RuntimeError, match="parent_part is required"):
+            await devrev_parts_create(
+                mock_ctx,
+                name="Orphan Feature",
+                type="feature",
+            )
+
+    @pytest.mark.asyncio
+    async def test_create_enhancement_without_parent_part_raises(self, mock_ctx, mock_client):
+        """Test that creating an ENHANCEMENT without parent_part raises RuntimeError. (#184)"""
+        with pytest.raises(RuntimeError, match="parent_part is required"):
+            await devrev_parts_create(
+                mock_ctx,
+                name="Orphan Enhancement",
+                type="enhancement",
+            )
+
+    @pytest.mark.asyncio
+    async def test_create_product_without_parent_part_succeeds(self, mock_ctx, mock_client):
+        """Test that creating a PRODUCT without parent_part succeeds. (#184)"""
+        mock_part = _make_mock_part(id="PROD-999", name="Standalone Product")
+        mock_client.parts.create.return_value = mock_part
+
+        result = await devrev_parts_create(
+            mock_ctx,
+            name="Standalone Product",
+            type="product",
+        )
+
+        assert result["id"] == "PROD-999"
+        call_args = mock_client.parts.create.call_args[0][0]
+        assert call_args.parent_part is None
+
 
 class TestPartsUpdateTool:
     """Tests for devrev_parts_update tool."""

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/tests/unit/services/test_articles.py

@@ -15,6 +15,7 @@ from devrev.models.articles import (
     ArticlesUpdateRequestOwnedBy,
     ArticlesUpdateRequestTags,
     ArticleType,
+    SetSharedWithMembership,
 )
 from devrev.models.base import SetTagWithValue
 from devrev.services.articles import ArticlesService
@@ -355,3 +356,79 @@ class TestArticlesService:
         call_data = mock_http_client.post.call_args.kwargs["data"]
         assert call_data["language"] == "fr"
         assert call_data["authored_by"] == ["author-1", "author-2"]
+
+    def test_update_with_content_shared_with_serialization(
+        self,
+        mock_http_client: MagicMock,
+        sample_article_data: dict[str, Any],
+    ) -> None:
+        """Test that shared_with is wrapped in ArticlesUpdateRequestSharedWith for updates.
+
+        The update path wraps shared_with in {"set": [...]} unlike the create path
+        which passes a raw list. This test asserts the serialized payload shape.
+        """
+        mock_http_client.post.return_value = create_mock_response({"article": sample_article_data})
+
+        mock_parent = MagicMock()
+        service = ArticlesService(mock_http_client, parent_client=mock_parent)
+
+        memberships = [
+            SetSharedWithMembership(member="don:identity:user:100"),
+            SetSharedWithMembership(member="don:identity:user:200", role="editor"),
+        ]
+        service.update_with_content(
+            "don:core:article:123",
+            shared_with=memberships,
+        )
+
+        # update_with_content calls update -> _post, so check the serialized payload
+        mock_http_client.post.assert_called_once()
+        call_data = mock_http_client.post.call_args.kwargs["data"]
+        assert "shared_with" in call_data
+        assert "set" in call_data["shared_with"]
+        assert len(call_data["shared_with"]["set"]) == 2
+        assert call_data["shared_with"]["set"][0]["member"] == "don:identity:user:100"
+        assert call_data["shared_with"]["set"][1]["member"] == "don:identity:user:200"
+        assert call_data["shared_with"]["set"][1]["role"] == "editor"
+
+    def test_update_with_content_empty_shared_with_clears_sharing(
+        self,
+        mock_http_client: MagicMock,
+        sample_article_data: dict[str, Any],
+    ) -> None:
+        """Test that an empty shared_with list serializes as {"set": []} to clear sharing."""
+        mock_http_client.post.return_value = create_mock_response({"article": sample_article_data})
+
+        mock_parent = MagicMock()
+        service = ArticlesService(mock_http_client, parent_client=mock_parent)
+
+        service.update_with_content(
+            "don:core:article:123",
+            shared_with=[],
+        )
+
+        mock_http_client.post.assert_called_once()
+        call_data = mock_http_client.post.call_args.kwargs["data"]
+        assert "shared_with" in call_data
+        assert call_data["shared_with"] == {"set": []}
+
+    def test_update_with_content_without_shared_with_omits_field(
+        self,
+        mock_http_client: MagicMock,
+        sample_article_data: dict[str, Any],
+    ) -> None:
+        """Test that shared_with is omitted from payload when not provided."""
+        mock_http_client.post.return_value = create_mock_response({"article": sample_article_data})
+
+        mock_parent = MagicMock()
+        service = ArticlesService(mock_http_client, parent_client=mock_parent)
+
+        service.update_with_content(
+            "don:core:article:123",
+            title="Updated Title",
+        )
+
+        mock_http_client.post.assert_called_once()
+        call_data = mock_http_client.post.call_args.kwargs["data"]
+        # shared_with should be truly absent from the payload, not just None
+        assert "shared_with" not in call_data

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.0}/uv.lock

@@ -483,7 +483,7 @@ wheels = [
 
 [[package]]
 name = "devrev-python-sdk"
-version = "2.11.0"
+version = "2.12.0"
 source = { editable = "." }
 dependencies = [
     { name = "beautifulsoup4" },