devrev-Python-SDK 2.9.0__tar.gz → 2.10.1__tar.gz

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/.github/workflows/ci.yml

@@ -52,7 +52,7 @@ jobs:
           fi
 
       - name: Upload coverage
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v5
         with:
           files: ./coverage.xml
           fail_ci_if_error: false

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/.github/workflows/deploy.yml

@@ -42,7 +42,7 @@ jobs:
           echo "Deploying version: $VERSION"
 
       - name: Authenticate to Google Cloud (WIF)
-        uses: google-github-actions/auth@v2
+        uses: google-github-actions/auth@v3
         with:
           workload_identity_provider: ${{ secrets.WIF_PROVIDER }}
           service_account: ${{ secrets.WIF_SERVICE_ACCOUNT }}

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/.github/workflows/release.yml

@@ -93,7 +93,7 @@ jobs:
       url: https://pypi.org/project/py-devrev/
     steps:
       - name: Download artifacts
-        uses: actions/download-artifact@v7
+        uses: actions/download-artifact@v8
         with:
           name: dist
           path: dist/
@@ -115,7 +115,7 @@ jobs:
           fetch-depth: 0
 
       - name: Download artifacts
-        uses: actions/download-artifact@v7
+        uses: actions/download-artifact@v8
         with:
           name: dist
           path: dist/

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 2.9.0
+Version: 2.10.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
@@ -545,6 +545,16 @@ client.articles.update_with_content(
     applies_to_parts=["don:core:dvrv-us-1:devo/1:part/123", "don:core:dvrv-us-1:devo/1:part/456"],
 )
 
+# Update access level and tags
+from devrev.models.articles import ArticleAccessLevel
+from devrev.models.base import SetTagWithValue
+
+client.articles.update_with_content(
+    id=article.id,
+    access_level=ArticleAccessLevel.INTERNAL,  # internal, external, private, public
+    tags=[SetTagWithValue(id="don:core:dvrv-us-1:devo/1:tag/123")],
+)
+
 # List published articles (metadata only)
 published = client.articles.list(limit=20)
 for article in published:

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/README.md

@@ -470,6 +470,16 @@ client.articles.update_with_content(
     applies_to_parts=["don:core:dvrv-us-1:devo/1:part/123", "don:core:dvrv-us-1:devo/1:part/456"],
 )
 
+# Update access level and tags
+from devrev.models.articles import ArticleAccessLevel
+from devrev.models.base import SetTagWithValue
+
+client.articles.update_with_content(
+    id=article.id,
+    access_level=ArticleAccessLevel.INTERNAL,  # internal, external, private, public
+    tags=[SetTagWithValue(id="don:core:dvrv-us-1:devo/1:tag/123")],
+)
+
 # List published articles (metadata only)
 published = client.articles.list(limit=20)
 for article in published:

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/docs/api/services/articles.md

@@ -94,6 +94,8 @@ article = client.articles.create_with_content(
     status=ArticleStatus.PUBLISHED,  # Optional: draft, published, archived
     content_format="text/html",  # Optional: default is text/html
     applies_to_parts=["don:core:dvrv-us-1:devo/1:part/123"],  # Optional: associate with parts
+    scope=1,  # Optional: 1=internal, 2=external (API default is external)
+    tags=[SetTagWithValue(id="don:core:dvrv-us-1:devo/1:tag/123")],  # Optional: apply tags
 )
 print(f"Created article with content: {article.id}")
 ```
@@ -188,6 +190,16 @@ updated_article = client.articles.update_with_content(
     id="don:core:dvrv-us-1:devo/1:article/123",
     applies_to_parts=["don:core:dvrv-us-1:devo/1:part/123", "don:core:dvrv-us-1:devo/1:part/456"],
 )
+
+# Update access level and tags
+from devrev.models.articles import ArticleAccessLevel
+from devrev.models.base import SetTagWithValue
+
+updated_article = client.articles.update_with_content(
+    id="don:core:dvrv-us-1:devo/1:article/123",
+    access_level=ArticleAccessLevel.INTERNAL,  # Change to internal visibility
+    tags=[SetTagWithValue(id="don:core:dvrv-us-1:devo/1:tag/123")],  # Apply tags
+)
 ```
 
 ### Async Usage

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/docs/changelog.md

@@ -7,6 +7,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.10.1] - 2026-03-10
+
+### Fixed
+
+- **MCP articles_list bug** (#170) — Fixed `'tuple' object has no attribute 'model_dump'` error in `devrev_articles_list` MCP tool by correctly accessing `response.articles` instead of calling `list()` on the Pydantic response model. Added `paginated_response()` helper for consistent pagination handling.
+- **CI formatting** (#175) — Resolved formatting check failures in `scripts/debug_articles.py`.
+
+### Maintenance
+
+- **Dependency updates** (#177) — Consolidated Dependabot updates:
+  - `actions/download-artifact` v7 → v8
+  - `codecov/codecov-action` v3 → v5
+  - `google-github-actions/auth` v2 → v3
+  - `authlib` 1.6.6 → 1.6.9 (security fix: removes "none" algorithm from default JWT instance)
+
+## [2.10.0] - 2026-03-09
+
+### Added
+
+- **Article Scope and Tags** (#173) — Added visibility and tagging parameters to MCP article tools:
+  - `devrev_articles_create`: Added `scope` (1=internal, 2=external) and `tags` (list of tag IDs) parameters
+  - `devrev_articles_update`: Added `access_level` (internal, external, private, public) and `tags` parameters
+  - SDK `update_with_content`: Added `access_level` and `tags` parameters for both sync and async methods
+
 ## [2.9.0] - 2026-03-09
 
 ### Added

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/docs/mcp/tools-reference.md

@@ -57,6 +57,8 @@ Tools are the primary way AI assistants interact with DevRev. Each tool maps to
 - **Optional Content Loading**: Use `include_content=true` in `get` to fetch article body
 - **Clear Parameters**: `content` for article body, `description` for metadata summary
 - **Part Association**: Use `applies_to_parts` to natively associate articles with products, capabilities, features, or enhancements
+- **Visibility Control**: Use `scope` (1=internal, 2=external) on create or `access_level` (internal, external, private, public) on update
+- **Tagging**: Apply tags using the `tags` parameter (list of tag IDs)
 
 ### Conversations
 

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/pyproject.toml

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

devrev_python_sdk-2.10.1/scripts/debug_articles.py

@@ -0,0 +1,200 @@
+#!/usr/bin/env python3
+"""Debug script for article content creation and inspection.
+
+Usage: python3 scripts/debug_articles.py
+Requires: DEVREV_API_TOKEN environment variable
+"""
+
+import json
+import os
+import sys
+
+# Add src to path
+sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "src"))
+
+from devrev.client import DevRevClient  # noqa: E402
+from devrev.models.articles import (  # noqa: E402
+    ArticlesGetRequest,
+    ArticleStatus,
+)
+from devrev.models.artifacts import ArtifactGetRequest  # noqa: E402
+
+
+def inspect_article(client: DevRevClient, article_id: str) -> None:
+    """Inspect an article and its artifact content."""
+    print("\n" + "=" * 60)
+    print(f"INSPECTING ARTICLE: {article_id}")
+    print("=" * 60)
+
+    # Get article metadata
+    article = client.articles.get(ArticlesGetRequest(id=article_id))
+    print(f"\nTitle: {article.title}")
+    print(f"Status: {article.status}")
+    print(f"Description: {article.description}")
+    print(f"Resource: {json.dumps(article.resource, indent=2, default=str)}")
+
+    # Try to get content artifact
+    if article.resource:
+        # Check for artifacts array (how DevRev stores them)
+        artifacts = article.resource.get("artifacts", [])
+        if artifacts:
+            for art_info in artifacts:
+                art_id = art_info.get("id", "")
+                file_info = art_info.get("file", {})
+                print(f"\n--- Artifact: {art_id} ---")
+                print(f"  File name: {file_info.get('name')}")
+                print(f"  File type: {file_info.get('type')}")
+                print(f"  File size: {file_info.get('size')}")
+
+                # Get full artifact metadata
+                try:
+                    artifact = client.artifacts.get(ArtifactGetRequest(id=art_id))
+                    print(f"  Artifact metadata - file_name: {artifact.file_name}")
+                    print(f"  Artifact metadata - file_type: {artifact.file_type}")
+                    print(f"  Artifact metadata - version: {artifact.version}")
+                except Exception as e:
+                    print(f"  Error getting artifact metadata: {e}")
+
+                # Download content
+                try:
+                    content_bytes = client.artifacts.download(art_id)
+                    content = content_bytes.decode("utf-8")
+                    print(f"  Content length: {len(content)}")
+                    print("  Content preview (first 2000 chars):")
+                    print(f"  {content[:2000]}")
+                    if len(content) > 2000:
+                        print(f"  ... ({len(content) - 2000} more chars)")
+                except Exception as e:
+                    print(f"  Error downloading content: {e}")
+
+        # Check for content_artifact reference (how our SDK stores them)
+        content_artifact_id = article.resource.get("content_artifact")
+        if content_artifact_id:
+            print(f"\n--- Content Artifact (SDK style): {content_artifact_id} ---")
+            try:
+                content_bytes = client.artifacts.download(content_artifact_id)
+                content = content_bytes.decode("utf-8")
+                print(f"  Content length: {len(content)}")
+                print(f"  Content preview: {content[:1000]}")
+            except Exception as e:
+                print(f"  Error: {e}")
+
+
+def test_create_article(client: DevRevClient, owner_id: str) -> None:
+    """Test creating an article with HTML content."""
+    print("\n" + "=" * 60)
+    print("TEST: Creating article with HTML content")
+    print("=" * 60)
+
+    html_content = """<h1>Test Article from py-dev-rev SDK</h1>
+<p>This is a <strong>test article</strong> created via the py-dev-rev SDK.</p>
+<h2>Features</h2>
+<ul>
+<li>Rich <em>formatted</em> text</li>
+<li>Lists and headings</li>
+<li>Code: <code>print("hello")</code></li>
+</ul>
+<p>Created at: 2026-03-07 for debugging purposes.</p>"""
+
+    try:
+        article = client.articles.create_with_content(
+            title="SDK Test - Rich Content Article",
+            content=html_content,
+            owned_by=[owner_id],
+            description="Test article created by py-dev-rev SDK debug script",
+            status=ArticleStatus.DRAFT,
+            content_format="text/html",
+        )
+        print(f"✅ Article created: {article.id} (display: {article.display_id})")
+        print(f"  Title: {article.title}")
+        print(f"  Resource: {json.dumps(article.resource, indent=2, default=str)}")
+
+        # Now test get_with_content on the new article
+        test_get_with_content(client, article.id)
+
+        # Now test update_content
+        test_update_content(client, article.id)
+
+    except Exception as e:
+        print(f"❌ Failed to create article: {e}")
+        import traceback
+
+        traceback.print_exc()
+
+
+def test_get_with_content(client: DevRevClient, article_id: str) -> None:
+    """Test the get_with_content method."""
+    print("\n" + "=" * 60)
+    print(f"TEST: get_with_content for {article_id}")
+    print("=" * 60)
+
+    try:
+        result = client.articles.get_with_content(article_id)
+        print("✅ get_with_content succeeded!")
+        print(f"  Title: {result.article.title}")
+        print(f"  Content format: {result.content_format}")
+        print(f"  Content version: {result.content_version}")
+        print(f"  Content length: {len(result.content)}")
+        print(f"  Content preview: {result.content[:500]}")
+    except Exception as e:
+        print(f"❌ get_with_content failed: {e}")
+        import traceback
+
+        traceback.print_exc()
+
+
+def test_update_content(client: DevRevClient, article_id: str) -> None:
+    """Test updating article content."""
+    print("\n" + "=" * 60)
+    print(f"TEST: update_content for {article_id}")
+    print("=" * 60)
+
+    updated_html = """<h1>Updated Article</h1>
+<p>This content was <strong>updated</strong> by the py-dev-rev SDK.</p>
+<p>Updated at: 2026-03-07</p>"""
+
+    try:
+        article = client.articles.update_content(article_id, updated_html)
+        print("✅ update_content succeeded!")
+        print(f"  Title: {article.title}")
+        print(f"  Resource: {json.dumps(article.resource, indent=2, default=str)}")
+
+        # Verify by re-reading
+        result = client.articles.get_with_content(article_id)
+        print(f"  Verified content length: {len(result.content)}")
+        print(f"  Verified content: {result.content[:300]}")
+    except Exception as e:
+        print(f"❌ update_content failed: {e}")
+        import traceback
+
+        traceback.print_exc()
+
+
+def main() -> None:
+    """Run article diagnostics."""
+    token = os.environ.get("DEVREV_API_TOKEN")
+    if not token:
+        print("ERROR: DEVREV_API_TOKEN environment variable not set")
+        sys.exit(1)
+
+    client = DevRevClient()
+
+    # 1. Inspect the known working article ART-2817
+    art_2817_id = "don:core:dvrv-us-1:devo/11Ca9baGrM:article/2817"
+    inspect_article(client, art_2817_id)
+
+    # 2. Test get_with_content on ART-2817
+    test_get_with_content(client, art_2817_id)
+
+    # 3. Get owner ID for creating test articles
+    owner_id = "don:identity:dvrv-us-1:devo/11Ca9baGrM:devu/4"  # MGM
+
+    # 4. Try creating a new article with HTML content
+    test_create_article(client, owner_id)
+
+    # 5. Test get_with_content on the newly created article
+    # (test_create_article already inspects it, but let's also test the unified method)
+
+
+if __name__ == "__main__":
+    main()

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/src/devrev/services/articles.py

@@ -9,6 +9,7 @@ from collections.abc import Sequence
 from devrev.exceptions import DevRevError
 from devrev.models.articles import (
     Article,
+    ArticleAccessLevel,
     ArticlesCountRequest,
     ArticlesCountResponse,
     ArticlesCreateRequest,
@@ -22,6 +23,7 @@ from devrev.models.articles import (
     ArticleStatus,
     ArticlesUpdateRequest,
     ArticlesUpdateRequestAppliesToParts,
+    ArticlesUpdateRequestTags,
     ArticlesUpdateResponse,
     ArticleType,
     ArticleWithContent,
@@ -439,6 +441,8 @@ class ArticlesService(BaseService):
         description: str | None = None,
         status: ArticleStatus | None = None,
         applies_to_parts: builtins.list[str] | None = None,
+        access_level: ArticleAccessLevel | None = None,
+        tags: builtins.list[SetTagWithValue] | None = None,
     ) -> Article:
         """Update article metadata and/or content.
 
@@ -454,6 +458,8 @@ class ArticlesService(BaseService):
             description: Optional new metadata description
             status: Optional new status
             applies_to_parts: Optional list of part IDs to associate with
+            access_level: Optional access level (internal, external, private, public)
+            tags: Optional list of tags to apply (list of SetTagWithValue objects)
 
         Returns:
             Updated article
@@ -487,6 +493,12 @@ class ArticlesService(BaseService):
             ...     "ART-123",
             ...     applies_to_parts=["don:core:...:capability/6"]
             ... )
+            >>>
+            >>> # Update access level to internal
+            >>> article = client.articles.update_with_content(
+            ...     "ART-123",
+            ...     access_level=ArticleAccessLevel.INTERNAL
+            ... )
         """
         if not self._parent_client:
             raise DevRevError(
@@ -503,12 +515,19 @@ class ArticlesService(BaseService):
         if applies_to_parts is not None:
             applies_to_parts_req = ArticlesUpdateRequestAppliesToParts(set=applies_to_parts)
 
+        # Build tags wrapper if provided
+        tags_req = None
+        if tags is not None:
+            tags_req = ArticlesUpdateRequestTags(set=tags)
+
         # Update metadata if any metadata fields provided
         has_metadata = (
             title is not None
             or description is not None
             or status is not None
             or applies_to_parts is not None
+            or access_level is not None
+            or tags is not None
         )
         if has_metadata:
             update_req = ArticlesUpdateRequest(
@@ -517,6 +536,8 @@ class ArticlesService(BaseService):
                 description=description,
                 status=status,
                 applies_to_parts=applies_to_parts_req,
+                access_level=access_level,
+                tags=tags_req,
             )
             return self.update(update_req)
 
@@ -844,6 +865,8 @@ class AsyncArticlesService(AsyncBaseService):
         description: str | None = None,
         status: ArticleStatus | None = None,
         applies_to_parts: builtins.list[str] | None = None,
+        access_level: ArticleAccessLevel | None = None,
+        tags: builtins.list[SetTagWithValue] | None = None,
     ) -> Article:
         """Update article metadata and/or content (async).
 
@@ -859,6 +882,8 @@ class AsyncArticlesService(AsyncBaseService):
             description: Optional new metadata description
             status: Optional new status
             applies_to_parts: Optional list of part IDs to associate with
+            access_level: Optional access level (internal, external, private, public)
+            tags: Optional list of tags to apply (list of SetTagWithValue objects)
 
         Returns:
             Updated article
@@ -881,12 +906,19 @@ class AsyncArticlesService(AsyncBaseService):
         if applies_to_parts is not None:
             applies_to_parts_req = ArticlesUpdateRequestAppliesToParts(set=applies_to_parts)
 
+        # Build tags wrapper if provided
+        tags_req = None
+        if tags is not None:
+            tags_req = ArticlesUpdateRequestTags(set=tags)
+
         # Update metadata if any metadata fields provided
         has_metadata = (
             title is not None
             or description is not None
             or status is not None
             or applies_to_parts is not None
+            or access_level is not None
+            or tags is not None
         )
         if has_metadata:
             update_req = ArticlesUpdateRequest(
@@ -895,6 +927,8 @@ class AsyncArticlesService(AsyncBaseService):
                 description=description,
                 status=status,
                 applies_to_parts=applies_to_parts_req,
+                access_level=access_level,
+                tags=tags_req,
             )
             return await self.update(update_req)
 

{devrev_python_sdk-2.9.0 → devrev_python_sdk-2.10.1}/src/devrev_mcp/tools/articles.py

@@ -12,15 +12,17 @@ from mcp.server.fastmcp import Context
 
 from devrev.exceptions import DevRevError
 from devrev.models.articles import (
+    ArticleAccessLevel,
     ArticlesDeleteRequest,
     ArticlesGetRequest,
     ArticlesListRequest,
     ArticleStatus,
 )
+from devrev.models.base import SetTagWithValue
 from devrev_mcp.server import _config, mcp
 from devrev_mcp.utils.errors import format_devrev_error
 from devrev_mcp.utils.formatting import serialize_model, serialize_models
-from devrev_mcp.utils.pagination import clamp_page_size
+from devrev_mcp.utils.pagination import clamp_page_size, paginated_response
 
 logger = logging.getLogger(__name__)
 
@@ -39,7 +41,8 @@ async def devrev_articles_list(
         limit: Maximum number of articles to return.
 
     Returns:
-        Dictionary containing count and list of articles.
+        Dictionary containing count, list of articles, and optional next_cursor
+        for pagination.
 
     Raises:
         RuntimeError: If the DevRev API call fails.
@@ -52,9 +55,9 @@ async def devrev_articles_list(
                 limit, default=app.config.default_page_size, maximum=app.config.max_page_size
             ),
         )
-        articles = await app.get_client().articles.list(request)
-        items = serialize_models(list(articles))
-        return {"count": len(items), "articles": items}
+        response = await app.get_client().articles.list(request)
+        items = serialize_models(response.articles)
+        return paginated_response(items, next_cursor=response.next_cursor, total_label="articles")
     except DevRevError as e:
         raise RuntimeError(format_devrev_error(e)) from e
 
@@ -103,6 +106,8 @@ if _config.enable_destructive_tools:
         status: str | None = None,
         content_format: str = "text/html",
         applies_to_parts: list[str] | None = None,
+        scope: int | None = None,
+        tags: list[str] | None = None,
     ) -> dict[str, Any]:
         """Create a new article with content.
 
@@ -116,6 +121,8 @@ if _config.enable_destructive_tools:
             content_format: Content MIME type (default: text/html).
             applies_to_parts: Optional list of part IDs (products, capabilities,
                 features, enhancements) to associate the article with.
+            scope: Optional visibility scope (1=internal, 2=external).
+            tags: Optional list of tag IDs to apply to the article.
 
         Returns:
             Dictionary containing the created article details.
@@ -135,6 +142,11 @@ if _config.enable_destructive_tools:
                         f"Valid statuses: {', '.join(s.name for s in ArticleStatus)}"
                     ) from e
 
+            # Convert tag IDs to SetTagWithValue objects
+            tags_list = (
+                [SetTagWithValue(id=tag_id) for tag_id in tags] if tags is not None else None
+            )
+
             article = await app.get_client().articles.create_with_content(
                 title=title,
                 content=content,
@@ -143,6 +155,8 @@ if _config.enable_destructive_tools:
                 status=article_status,
                 content_format=content_format,
                 applies_to_parts=applies_to_parts,
+                scope=scope,
+                tags=tags_list,
             )
             return serialize_model(article)
         except DevRevError as e:
@@ -157,6 +171,8 @@ if _config.enable_destructive_tools:
         description: str | None = None,
         status: str | None = None,
         applies_to_parts: list[str] | None = None,
+        access_level: str | None = None,
+        tags: list[str] | None = None,
     ) -> dict[str, Any]:
         """Update an existing article in DevRev.
 
@@ -170,6 +186,11 @@ if _config.enable_destructive_tools:
             applies_to_parts: Optional list of part IDs (products, capabilities,
                 features, enhancements) to associate the article with.
                 Pass an empty list to remove all associations.
+            access_level: Optional access level (internal, external, private, public).
+                Note: For updates, use ``access_level`` (string enum). For creation,
+                use the ``scope`` parameter (integer: 1=internal, 2=external).
+            tags: Optional list of tag IDs to apply to the article.
+                Pass an empty list to remove all tags.
 
         Returns:
             Dictionary containing the updated article details.
@@ -189,6 +210,22 @@ if _config.enable_destructive_tools:
                         f"Valid statuses: {', '.join(s.name for s in ArticleStatus)}"
                     ) from e
 
+            # Convert access_level string to ArticleAccessLevel enum
+            article_access_level = None
+            if access_level:
+                try:
+                    article_access_level = ArticleAccessLevel[access_level.upper()]
+                except KeyError as e:
+                    raise RuntimeError(
+                        f"Invalid access level: {e.args[0]}. "
+                        f"Valid levels: {', '.join(a.name for a in ArticleAccessLevel)}"
+                    ) from e
+
+            # Convert tag IDs to SetTagWithValue objects
+            tags_list = (
+                [SetTagWithValue(id=tag_id) for tag_id in tags] if tags is not None else None
+            )
+
             article = await app.get_client().articles.update_with_content(
                 id=id,
                 title=title,
@@ -196,6 +233,8 @@ if _config.enable_destructive_tools:
                 description=description,
                 status=article_status,
                 applies_to_parts=applies_to_parts,
+                access_level=article_access_level,
+                tags=tags_list,
             )
             return serialize_model(article)
         except DevRevError as e: