devrev-Python-SDK 2.12.2__tar.gz → 2.14.0__tar.gz

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/.github/workflows/docs.yml

@@ -59,5 +59,5 @@ jobs:
     steps:
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@v4
+        uses: actions/deploy-pages@v5
 

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/.github/workflows/integration-tests.yml

@@ -102,7 +102,7 @@ jobs:
       
       - name: Create issue on failure
         if: failure()
-        uses: actions/github-script@v8
+        uses: actions/github-script@v9
         with:
           script: |
             github.rest.issues.create({

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/.github/workflows/release.yml

@@ -155,7 +155,7 @@ jobs:
           EOF
 
       - name: Create Release
-        uses: softprops/action-gh-release@v2
+        uses: softprops/action-gh-release@v3
         with:
           name: v${{ steps.version.outputs.version }}
           body_path: release_notes.md

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 2.12.2
+Version: 2.14.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.12.2 → devrev_python_sdk-2.14.0}/docs/changelog.md

@@ -7,6 +7,58 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.14.0] - 2026-04-14
+
+### Added
+
+- **DON ID type validation with helpful error messages** (CUSS-396, PR #207)
+  - New `validate_don_id()` utility in `src/devrev_mcp/utils/don_id.py`
+  - `parse_don_type()` extracts object type segment from DON IDs
+  - `DON_TYPE_MAP` maps DON type segments to human-friendly names
+  - `TOOL_SUGGESTIONS` maps DON types to the correct MCP tool for actionable suggestions
+  - Integrated into all 14 MCP tool files (`_get`, `_update`, `_delete` operations)
+  - Example: *"This is a rev_org ID, try using devrev_rev_orgs_get instead"*
+  - Raises `RuntimeError` caught by existing MCP error handlers
+
+- **Rev Orgs (revenue organizations) service** (CUSS-397, PR #207)
+  - Pydantic v2 models: `RevOrg`, `RevOrgSummary`, request/response models (`src/devrev/models/rev_orgs.py`)
+  - Sync `RevOrgsService` and async `AsyncRevOrgsService` (`src/devrev/services/rev_orgs.py`)
+  - MCP tools: `devrev_rev_orgs_list`, `devrev_rev_orgs_get`, `devrev_rev_orgs_create`, `devrev_rev_orgs_update`, `devrev_rev_orgs_delete`
+  - Destructive tools gated behind `enable_destructive_tools` config flag
+  - Registered in `DevRevClient` and `AsyncDevRevClient` as `rev_orgs` property
+
+### Security
+
+- **Bump cryptography from 46.0.5 to 46.0.7** (PR #206) — Fixes CVE-2026-39892 (buffer overflow with non-contiguous buffers) and CVE-2026-34073 (name constraint bypass with wildcard DNS SAN)
+
+### Maintenance
+
+- **CI dependency updates** — Folded 3 Dependabot PRs into PR #207:
+  - PR #208: Bump `softprops/action-gh-release` from v2 to v3 (Node 24)
+  - PR #209: Bump `actions/deploy-pages` from v4 to v5 (Node 24)
+  - PR #210: Bump `actions/github-script` from v8 to v9 (Node 24)
+
+### Testing
+
+- 22 new unit tests for DON ID validation (`tests/unit/test_don_id_validation.py`)
+- 19 new unit tests for Rev Orgs models and services (`tests/unit/test_rev_orgs.py`)
+- All 1,091 tests passing
+
+## [2.13.0] - 2026-04-08
+
+### Added
+
+- **Article content format detection and bidirectional conversion** (#160, PR #205)
+  - `detect_content_format()` — detects `devrev/rt`, `text/markdown`, `text/html`, or `text/plain`
+  - `devrev_rt_to_markdown()` — converts ProseMirror JSON to Markdown
+  - `devrev_rt_to_html()` — converts ProseMirror JSON to HTML with XSS-safe escaping
+  - `OutputFormat` type alias (`Literal["text/markdown", "text/html", "devrev/rt"]`)
+  - Four format constants: `CONTENT_FORMAT_DEVREV_RT`, `CONTENT_FORMAT_MARKDOWN`, `CONTENT_FORMAT_HTML`, `CONTENT_FORMAT_PLAIN`
+  - `get_with_content()` (sync + async) now accepts optional `output_format` parameter for automatic content conversion
+  - MCP tool `devrev_articles_get` accepts `output_format` when `include_content=True`
+  - `_convert_content()` validates output format and raises `ValueError` for invalid values
+  - All HTML output uses `html.escape()` for text and attributes; links include `rel="noopener noreferrer"`
+
 ## [2.12.2] - 2026-04-08
 
 ### Maintenance

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/pyproject.toml

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

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/src/devrev/client.py

@@ -38,6 +38,7 @@ from devrev.services.recommendations import (
     AsyncRecommendationsService,
     RecommendationsService,
 )
+from devrev.services.rev_orgs import AsyncRevOrgsService, RevOrgsService
 from devrev.services.rev_users import AsyncRevUsersService, RevUsersService
 from devrev.services.search import AsyncSearchService, SearchService
 from devrev.services.slas import AsyncSlasService, SlasService
@@ -160,6 +161,7 @@ class DevRevClient:
         self._groups = GroupsService(self._http)
         self._links = LinksService(self._http)
         self._parts = PartsService(self._http)
+        self._rev_orgs = RevOrgsService(self._http)
         self._rev_users = RevUsersService(self._http)
         self._slas = SlasService(self._http)
         self._tags = TagsService(self._http)
@@ -242,6 +244,11 @@ class DevRevClient:
         """Access the Parts service."""
         return self._parts
 
+    @property
+    def rev_orgs(self) -> RevOrgsService:
+        """Access the Rev Orgs service for managing revenue organizations."""
+        return self._rev_orgs
+
     @property
     def rev_users(self) -> RevUsersService:
         """Access the Rev Users service."""
@@ -482,6 +489,7 @@ class AsyncDevRevClient:
         self._groups = AsyncGroupsService(self._http)
         self._links = AsyncLinksService(self._http)
         self._parts = AsyncPartsService(self._http)
+        self._rev_orgs = AsyncRevOrgsService(self._http)
         self._rev_users = AsyncRevUsersService(self._http)
         self._slas = AsyncSlasService(self._http)
         self._tags = AsyncTagsService(self._http)
@@ -564,6 +572,11 @@ class AsyncDevRevClient:
         """Access the Parts service."""
         return self._parts
 
+    @property
+    def rev_orgs(self) -> AsyncRevOrgsService:
+        """Access the Rev Orgs service for managing revenue organizations."""
+        return self._rev_orgs
+
     @property
     def rev_users(self) -> AsyncRevUsersService:
         """Access the Rev Users service."""

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/src/devrev/models/__init__.py

@@ -260,6 +260,20 @@ from devrev.models.recommendations import (
     MessageRole,
     TokenUsage,
 )
+from devrev.models.rev_orgs import (
+    RevOrg,
+    RevOrgsCreateRequest,
+    RevOrgsCreateResponse,
+    RevOrgsDeleteRequest,
+    RevOrgsDeleteResponse,
+    RevOrgsGetRequest,
+    RevOrgsGetResponse,
+    RevOrgsListRequest,
+    RevOrgsListResponse,
+    RevOrgSummary,
+    RevOrgsUpdateRequest,
+    RevOrgsUpdateResponse,
+)
 from devrev.models.rev_users import (
     RevUser,
     RevUsersAssociationsAddRequest,
@@ -694,6 +708,19 @@ __all__ = [
     "ChatCompletionResponse",
     "GetReplyRequest",
     "GetReplyResponse",
+    # Rev Orgs
+    "RevOrg",
+    "RevOrgSummary",
+    "RevOrgsCreateRequest",
+    "RevOrgsCreateResponse",
+    "RevOrgsGetRequest",
+    "RevOrgsGetResponse",
+    "RevOrgsListRequest",
+    "RevOrgsListResponse",
+    "RevOrgsUpdateRequest",
+    "RevOrgsUpdateResponse",
+    "RevOrgsDeleteRequest",
+    "RevOrgsDeleteResponse",
     # Rev Users
     "RevUser",
     "RevUserSummary",
@@ -889,6 +916,12 @@ from devrev.models.accounts import (
     AccountsListResponse as _AccountsListResponse,
 )
 from devrev.models.base import TagWithValue as _TagWithValue  # noqa: F811
+from devrev.models.rev_orgs import (  # noqa: F811
+    RevOrg as _RevOrg,
+)
+from devrev.models.rev_orgs import (
+    RevOrgsListResponse as _RevOrgsListResponse,
+)
 from devrev.models.works import (  # noqa: F811
     Work as _Work,
 )
@@ -904,6 +937,8 @@ _TagWithValue.model_rebuild()
 _Account.model_rebuild()
 _AccountsListResponse.model_rebuild()
 _AccountsExportResponse.model_rebuild()
+_RevOrg.model_rebuild()
+_RevOrgsListResponse.model_rebuild()
 _Work.model_rebuild()
 _WorksListResponse.model_rebuild()
 _WorksExportResponse.model_rebuild()

devrev_python_sdk-2.14.0/src/devrev/models/rev_orgs.py

@@ -0,0 +1,159 @@
+"""Rev Org models for DevRev SDK.
+
+This module contains Pydantic models for Rev Org (Revenue Organization)-related
+API operations. Rev Orgs represent customer organizations in DevRev.
+"""
+
+from __future__ import annotations
+
+from datetime import datetime
+from typing import Any
+
+from pydantic import Field
+
+from devrev.models.accounts import AccountSummary
+from devrev.models.base import (
+    CustomSchemaSpec,
+    DateFilter,
+    DevRevBaseModel,
+    DevRevResponseModel,
+    PaginatedResponse,
+    TagWithValue,
+    UserSummary,
+)
+
+
+class RevOrg(DevRevResponseModel):
+    """DevRev Rev Org (Revenue Organization) model.
+
+    Represents a customer organization in DevRev, typically associated
+    with an Account.
+    """
+
+    id: str = Field(..., description="Rev org ID")
+    display_id: str | None = Field(default=None, description="Human-readable display ID")
+    display_name: str | None = Field(default=None, description="Rev org display name")
+    description: str | None = Field(default=None, description="Rev org description")
+    account: AccountSummary | None = Field(default=None, description="Associated account")
+    created_date: datetime | None = Field(default=None, description="Creation timestamp")
+    modified_date: datetime | None = Field(default=None, description="Last modification timestamp")
+    created_by: UserSummary | None = Field(
+        default=None, description="User who created this rev org"
+    )
+    modified_by: UserSummary | None = Field(
+        default=None, description="User who last modified this rev org"
+    )
+    domain: str | None = Field(default=None, description="Domain")
+    external_ref: str | None = Field(default=None, description="External reference identifier")
+    external_refs: list[str] | None = Field(default=None, description="External references")
+    custom_fields: dict[str, Any] | None = Field(default=None, description="Custom fields")
+    sub_type: str | None = Field(default=None, description="Rev org subtype")
+    tags: list[TagWithValue] | None = Field(default=None, description="Tags")
+    tier: str | None = Field(default=None, description="Rev org tier")
+    artifacts: list[str] | None = Field(default=None, description="Associated artifact IDs")
+
+
+class RevOrgSummary(DevRevResponseModel):
+    """Summary of a Rev Org for list/reference operations."""
+
+    id: str = Field(..., description="Rev org ID")
+    display_id: str | None = Field(default=None, description="Human-readable display ID")
+    display_name: str | None = Field(default=None, description="Rev org display name")
+
+
+# Request Models
+
+
+class RevOrgsGetRequest(DevRevBaseModel):
+    """Request to get a rev org."""
+
+    id: str | None = Field(default=None, description="Rev org ID")
+    account: str | None = Field(
+        default=None, description="Account ID to get the default rev org for"
+    )
+
+
+class RevOrgsListRequest(DevRevBaseModel):
+    """Request to list rev orgs."""
+
+    account: list[str] | None = Field(default=None, description="Filter by account IDs")
+    created_by: list[str] | None = Field(default=None, description="Filter by creator user IDs")
+    created_date: DateFilter | None = Field(default=None, description="Filter by creation date")
+    cursor: str | None = Field(default=None, description="Pagination cursor")
+    display_name: list[str] | None = Field(default=None, description="Filter by display names")
+    domains: list[str] | None = Field(default=None, description="Filter by domains")
+    external_refs: list[str] | None = Field(default=None, description="Filter by external refs")
+    limit: int | None = Field(default=None, ge=1, le=100, description="Max results to return")
+    modified_date: DateFilter | None = Field(
+        default=None, description="Filter by modification date"
+    )
+    owned_by: list[str] | None = Field(default=None, description="Filter by owner user IDs")
+    tags: list[str] | None = Field(default=None, description="Filter by tag IDs")
+
+
+class RevOrgsCreateRequest(DevRevBaseModel):
+    """Request to create a rev org."""
+
+    display_name: str = Field(..., description="Rev org display name", min_length=1, max_length=256)
+    account: str = Field(..., description="Parent account ID")
+    description: str | None = Field(default=None, description="Rev org description")
+    external_ref: str | None = Field(default=None, description="External reference identifier")
+    tier: str | None = Field(default=None, description="Rev org tier")
+    custom_fields: dict[str, Any] | None = Field(default=None, description="Custom fields")
+    custom_schema_spec: CustomSchemaSpec | None = Field(
+        default=None, description="Custom schema spec"
+    )
+
+
+class RevOrgsUpdateRequest(DevRevBaseModel):
+    """Request to update a rev org."""
+
+    id: str = Field(..., description="Rev org ID")
+    display_name: str | None = Field(default=None, description="New display name")
+    description: str | None = Field(default=None, description="New description")
+    tier: str | None = Field(default=None, description="New tier")
+    custom_fields: dict[str, Any] | None = Field(
+        default=None, description="Custom fields to update"
+    )
+    artifacts: dict[str, Any] | None = Field(
+        default=None, description="Artifact set/remove operations"
+    )
+
+
+class RevOrgsDeleteRequest(DevRevBaseModel):
+    """Request to delete a rev org."""
+
+    id: str = Field(..., description="Rev org ID to delete")
+
+
+# Response Models
+
+
+class RevOrgsGetResponse(DevRevResponseModel):
+    """Response from getting a rev org."""
+
+    rev_org: RevOrg = Field(..., description="Retrieved rev org")
+
+
+class RevOrgsListResponse(PaginatedResponse):
+    """Response from listing rev orgs."""
+
+    rev_orgs: list[RevOrg] = Field(..., description="List of rev orgs")
+
+
+class RevOrgsCreateResponse(DevRevResponseModel):
+    """Response from creating a rev org."""
+
+    rev_org: RevOrg = Field(..., description="Created rev org")
+
+
+class RevOrgsUpdateResponse(DevRevResponseModel):
+    """Response from updating a rev org."""
+
+    rev_org: RevOrg = Field(..., description="Updated rev org")
+
+
+class RevOrgsDeleteResponse(DevRevResponseModel):
+    """Response from deleting a rev org."""
+
+    pass  # Empty response body

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/src/devrev/services/__init__.py

@@ -34,6 +34,7 @@ from devrev.services.recommendations import (
     AsyncRecommendationsService,
     RecommendationsService,
 )
+from devrev.services.rev_orgs import AsyncRevOrgsService, RevOrgsService
 from devrev.services.rev_users import AsyncRevUsersService, RevUsersService
 from devrev.services.search import AsyncSearchService, SearchService
 from devrev.services.slas import AsyncSlasService, SlasService
@@ -99,6 +100,9 @@ __all__ = [
     # Recommendations
     "RecommendationsService",
     "AsyncRecommendationsService",
+    # Rev Orgs
+    "RevOrgsService",
+    "AsyncRevOrgsService",
     # Rev Users
     "RevUsersService",
     "AsyncRevUsersService",

{devrev_python_sdk-2.12.2 → devrev_python_sdk-2.14.0}/src/devrev/services/articles.py

@@ -35,15 +35,16 @@ from devrev.models.artifacts import (
 )
 from devrev.models.base import SetTagWithValue
 from devrev.services.base import AsyncBaseService, BaseService
-from devrev.utils.content_converter import html_to_devrev_rt
-
-# Content format to file extension mapping
-_CONTENT_FORMAT_EXTENSIONS: dict[str, str] = {
-    "text/html": ".html",
-    "text/markdown": ".md",
-    "text/plain": ".txt",
-    "devrev/rt": "",  # DevRev rich text uses no extension (file name "Article")
-}
+from devrev.utils.content_converter import (
+    CONTENT_FORMAT_DEVREV_RT,
+    CONTENT_FORMAT_HTML,
+    CONTENT_FORMAT_MARKDOWN,
+    OutputFormat,
+    detect_content_format,
+    devrev_rt_to_html,
+    devrev_rt_to_markdown,
+    html_to_devrev_rt,
+)
 
 
 def _extract_content_artifact_id(resource: dict[str, object]) -> str | None:
@@ -101,6 +102,69 @@ def _extract_content_format(resource: dict[str, object]) -> str:
     return "text/plain"
 
 
+def _convert_content(
+    content: str,
+    source_format: str,
+    target_format: str,
+) -> tuple[str, str]:
+    """Convert article content between formats.
+
+    Args:
+        content: The raw content string.
+        source_format: The MIME type of *content* (e.g. ``"devrev/rt"``).
+        target_format: The desired output MIME type.
+
+    Returns:
+        A ``(converted_content, actual_format)`` tuple.  If no conversion
+        is necessary (source == target, or conversion is not possible)
+        the original content and format are returned.
+    Raises:
+        ValueError: If *target_format* is not a recognised format.
+    """
+    _VALID_FORMATS = {
+        CONTENT_FORMAT_MARKDOWN,
+        CONTENT_FORMAT_HTML,
+        CONTENT_FORMAT_DEVREV_RT,
+    }
+    if target_format not in _VALID_FORMATS:
+        raise ValueError(
+            f"Invalid output_format {target_format!r}. Accepted values: {sorted(_VALID_FORMATS)}"
+        )
+
+    if source_format == target_format:
+        return content, source_format
+
+    # Auto-detect source format when unknown / generic
+    if source_format in ("text/plain", ""):
+        source_format = detect_content_format(content)
+
+    if target_format == CONTENT_FORMAT_MARKDOWN:
+        if source_format == CONTENT_FORMAT_DEVREV_RT:
+            return devrev_rt_to_markdown(content), CONTENT_FORMAT_MARKDOWN
+        # HTML or unknown → convert to devrev/rt first, then to markdown
+        if source_format == CONTENT_FORMAT_HTML:
+            rt = html_to_devrev_rt(content)
+            return devrev_rt_to_markdown(rt), CONTENT_FORMAT_MARKDOWN
+        # Already markdown or plain text
+        return content, source_format
+
+    if target_format == CONTENT_FORMAT_HTML:
+        if source_format == CONTENT_FORMAT_DEVREV_RT:
+            return devrev_rt_to_html(content), CONTENT_FORMAT_HTML
+        if source_format == CONTENT_FORMAT_MARKDOWN:
+            rt = html_to_devrev_rt(content)
+            return devrev_rt_to_html(rt), CONTENT_FORMAT_HTML
+        return content, source_format
+
+    if target_format == CONTENT_FORMAT_DEVREV_RT:
+        if source_format != CONTENT_FORMAT_DEVREV_RT:
+            return html_to_devrev_rt(content), CONTENT_FORMAT_DEVREV_RT
+        return content, source_format
+
+    # Unknown target format – return unchanged
+    return content, source_format
+
+
 class ArticlesService(BaseService):
     """Service for managing DevRev Articles."""
 
@@ -313,17 +377,27 @@ class ArticlesService(BaseService):
             # Re-raise the original error
             raise DevRevError(f"Failed to create article with content: {e}") from e
 
-    def get_with_content(self, id: str) -> ArticleWithContent:
+    def get_with_content(
+        self,
+        id: str,
+        *,
+        output_format: OutputFormat | None = None,
+    ) -> ArticleWithContent:
         """Get an article with its content loaded.
 
         This is a high-level method that:
         1. Fetches article metadata
         2. Locates the content artifact
         3. Downloads artifact content
-        4. Returns combined model
+        4. Optionally converts to the requested output format
+        5. Returns combined model
 
         Args:
             id: Article ID
+            output_format: Desired output format for the content.  Accepted
+                values: ``"text/markdown"``, ``"text/html"``, ``"devrev/rt"``.
+                When ``None`` (the default) the raw stored content is returned
+                as-is.
 
         Returns:
             ArticleWithContent with metadata and content
@@ -336,6 +410,9 @@ class ArticlesService(BaseService):
             >>> article_with_content = client.articles.get_with_content("ART-123")
             >>> print(article_with_content.article.title)
             >>> print(article_with_content.content)
+            >>> # Get content as Markdown
+            >>> md = client.articles.get_with_content("ART-123", output_format="text/markdown")
+            >>> print(md.content)
         """
         if not self._parent_client:
             raise DevRevError(
@@ -364,12 +441,18 @@ class ArticlesService(BaseService):
             # Get content format from resource metadata (more reliable than artifact.get)
             content_format = _extract_content_format(article.resource)
 
+            # Step 4: Convert to requested output format if specified
+            if output_format is not None:
+                content, content_format = _convert_content(content, content_format, output_format)
+
             return ArticleWithContent(
                 article=article,
                 content=content,
                 content_format=content_format,
                 content_version=None,
             )
+        except DevRevError:
+            raise
         except Exception as e:
             raise DevRevError(f"Failed to download content for article {id}: {e}") from e
 
@@ -772,17 +855,27 @@ class AsyncArticlesService(AsyncBaseService):
             # Re-raise the original error
             raise DevRevError(f"Failed to create article with content: {e}") from e
 
-    async def get_with_content(self, id: str) -> ArticleWithContent:
+    async def get_with_content(
+        self,
+        id: str,
+        *,
+        output_format: OutputFormat | None = None,
+    ) -> ArticleWithContent:
         """Get an article with its content loaded (async).
 
         This is a high-level method that:
         1. Fetches article metadata
         2. Locates the content artifact
         3. Downloads artifact content
-        4. Returns combined model
+        4. Optionally converts to the requested output format
+        5. Returns combined model
 
         Args:
             id: Article ID
+            output_format: Desired output format for the content.  Accepted
+                values: ``"text/markdown"``, ``"text/html"``, ``"devrev/rt"``.
+                When ``None`` (the default) the raw stored content is returned
+                as-is.
 
         Returns:
             ArticleWithContent with metadata and content
@@ -818,12 +911,18 @@ class AsyncArticlesService(AsyncBaseService):
             # Get content format from resource metadata (more reliable than artifact.get)
             content_format = _extract_content_format(article.resource)
 
+            # Step 4: Convert to requested output format if specified
+            if output_format is not None:
+                content, content_format = _convert_content(content, content_format, output_format)
+
             return ArticleWithContent(
                 article=article,
                 content=content,
                 content_format=content_format,
                 content_version=None,
             )
+        except DevRevError:
+            raise
         except Exception as e:
             raise DevRevError(f"Failed to download content for article {id}: {e}") from e