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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 2.11.0
+Version: 2.12.1
 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.1}/deploy/cloudbuild.yaml

@@ -71,7 +71,7 @@ substitutions:
 # Build options
 options:
   logging: CLOUD_LOGGING_ONLY
-  machineType: 'N1_HIGHCPU_8'
+  machineType: 'E2_HIGHCPU_8'
 
 # Timeout for the entire build
 timeout: '1200s'

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.1}/docs/api/beta/search.md

@@ -90,7 +90,7 @@ from devrev.models.search import CoreSearchRequest, HybridSearchRequest, SearchN
 # Core search with request object
 request = CoreSearchRequest(
     query="type:incident AND severity:sev0",
-    namespace=SearchNamespace.WORK,
+    namespaces=[SearchNamespace.WORK],
     limit=50,
 )
 results = client.search.core(request)
@@ -98,6 +98,7 @@ results = client.search.core(request)
 # Hybrid search with request object
 request = HybridSearchRequest(
     query="database performance issues",
+    namespaces=[SearchNamespace.WORK],
     semantic_weight=0.8,
     limit=20,
 )

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.1}/docs/changelog.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.12.1] - 2026-04-07
+
+### Fixed
+
+- **Search API Bad Request error** (#204) — Fixed `devrev_search_core_devrev` and `devrev_search_hybrid_devrev` MCP tools returning "Validation error: Bad Request" by changing `CoreSearchRequest` and `HybridSearchRequest` models to send `namespaces` (plural, array) instead of `namespace` (singular, string), matching the DevRev API specification.
+
+### Maintenance
+
+- **Cloud Build machine type** — Updated Cloud Build machine type from `N1_HIGHCPU_8` to `E2_HIGHCPU_8`.
+
 ## [2.10.2] - 2026-03-11
 
 ### Fixed

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "devrev-Python-SDK"
-version = "2.11.0"
+version = "2.12.1"
 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.1}/src/devrev/models/search.py

@@ -93,8 +93,8 @@ class CoreSearchRequest(DevRevBaseModel):
     """Request model for core search."""
 
     query: str = Field(..., description="Search query")
-    namespace: SearchNamespace = Field(
-        ..., description="Namespace to search (e.g. account, work, article)"
+    namespaces: list[SearchNamespace] = Field(
+        ..., description="Namespaces to search in (e.g. ['account'], ['work', 'ticket'])"
     )
     limit: int | None = Field(
         default=None,
@@ -109,8 +109,8 @@ class HybridSearchRequest(DevRevBaseModel):
     """Request model for hybrid search."""
 
     query: str = Field(..., description="Search query")
-    namespace: SearchNamespace = Field(
-        ..., description="Namespace to search (e.g. account, work, article)"
+    namespaces: list[SearchNamespace] = Field(
+        ..., description="Namespaces to search in (e.g. ['account'], ['work', 'ticket'])"
     )
     semantic_weight: float | None = Field(
         default=None, description="Weight for semantic search (0-1)"

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.1}/src/devrev/services/search.py

@@ -94,7 +94,7 @@ class SearchService(BaseService):
                 )
             request = CoreSearchRequest(
                 query=request_or_query,
-                namespace=namespace,
+                namespaces=[namespace],
                 limit=limit,
                 cursor=cursor,
             )
@@ -180,7 +180,7 @@ class SearchService(BaseService):
                 )
             request = HybridSearchRequest(
                 query=request_or_query,
-                namespace=namespace,
+                namespaces=[namespace],
                 semantic_weight=semantic_weight,
                 limit=limit,
                 cursor=cursor,
@@ -266,7 +266,7 @@ class AsyncSearchService(AsyncBaseService):
                 )
             request = CoreSearchRequest(
                 query=request_or_query,
-                namespace=namespace,
+                namespaces=[namespace],
                 limit=limit,
                 cursor=cursor,
             )
@@ -352,7 +352,7 @@ class AsyncSearchService(AsyncBaseService):
                 )
             request = HybridSearchRequest(
                 query=request_or_query,
-                namespace=namespace,
+                namespaces=[namespace],
                 semantic_weight=semantic_weight,
                 limit=limit,
                 cursor=cursor,

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.1}/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.1}/tests/integration/test_specialized_services_phase3.py

@@ -69,7 +69,7 @@ class TestSearchEndpoints:
     )
     def test_search_core_with_request(self, beta_client: DevRevClient) -> None:
         """Test search.core endpoint with request object."""
-        request = CoreSearchRequest(query="test", namespace=SearchNamespace.WORK, limit=5)
+        request = CoreSearchRequest(query="test", namespaces=[SearchNamespace.WORK], limit=5)
         result = beta_client.search.core(request)
         assert result is not None
         assert hasattr(result, "results")
@@ -96,7 +96,7 @@ class TestSearchEndpoints:
     def test_search_hybrid_with_request(self, beta_client: DevRevClient) -> None:
         """Test search.hybrid endpoint with request object."""
         request = HybridSearchRequest(
-            query="test", namespace=SearchNamespace.WORK, limit=5, semantic_weight=0.5
+            query="test", namespaces=[SearchNamespace.WORK], limit=5, semantic_weight=0.5
         )
         result = beta_client.search.hybrid(request)
         assert result is not None

{devrev_python_sdk-2.11.0 → devrev_python_sdk-2.12.1}/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.1}/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.1}/tests/unit/services/test_search.py

@@ -46,7 +46,7 @@ class TestSearchService:
         service = SearchService(mock_http_client)
         request = CoreSearchRequest(
             query="type:ticket AND status:open",
-            namespace=SearchNamespace.WORK,
+            namespaces=[SearchNamespace.WORK],
             limit=20,
         )
         result = service.core(request)
@@ -101,7 +101,7 @@ class TestSearchService:
         service = SearchService(mock_http_client)
         request = HybridSearchRequest(
             query="authentication issues",
-            namespace=SearchNamespace.CONVERSATION,
+            namespaces=[SearchNamespace.CONVERSATION],
             semantic_weight=0.7,
             limit=10,
         )

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

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