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

{devrev_python_sdk-2.10.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.10.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.10.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.10.0 → devrev_python_sdk-2.10.1}/PKG-INFO

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

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

@@ -7,6 +7,21 @@ 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

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "devrev-Python-SDK"
-version = "2.10.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.10.0 → devrev_python_sdk-2.10.1}/src/devrev_mcp/tools/articles.py

@@ -22,7 +22,7 @@ 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__)
 
@@ -41,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.
@@ -54,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
 
@@ -142,7 +143,9 @@ if _config.enable_destructive_tools:
                     ) 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
+            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,
@@ -219,7 +222,9 @@ if _config.enable_destructive_tools:
                     ) 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
+            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,

{devrev_python_sdk-2.10.0 → devrev_python_sdk-2.10.1}/tests/unit/mcp/test_tools_articles.py

@@ -44,12 +44,16 @@ class TestArticlesListTool:
     @pytest.mark.asyncio
     async def test_list_empty(self, mock_ctx, mock_client):
         """Test listing articles when none exist."""
-        mock_client.articles.list.return_value = []
+        response = MagicMock()
+        response.articles = []
+        response.next_cursor = None
+        mock_client.articles.list.return_value = response
 
         result = await devrev_articles_list(mock_ctx)
 
         assert result["count"] == 0
         assert result["articles"] == []
+        assert "next_cursor" not in result
         mock_client.articles.list.assert_called_once()
 
     @pytest.mark.asyncio
@@ -59,7 +63,10 @@ class TestArticlesListTool:
             _make_mock_article(id="article-1", title="Article 1"),
             _make_mock_article(id="article-2", title="Article 2"),
         ]
-        mock_client.articles.list.return_value = mock_articles
+        response = MagicMock()
+        response.articles = mock_articles
+        response.next_cursor = None
+        mock_client.articles.list.return_value = response
 
         result = await devrev_articles_list(mock_ctx, limit=10)
 
@@ -68,6 +75,24 @@ class TestArticlesListTool:
         assert result["articles"][0]["id"] == "article-1"
         assert result["articles"][1]["id"] == "article-2"
 
+    @pytest.mark.asyncio
+    async def test_list_with_pagination(self, mock_ctx, mock_client):
+        """Test listing articles returns pagination cursor."""
+        response = MagicMock()
+        response.articles = [_make_mock_article(id="article-1", title="Article 1")]
+        response.next_cursor = "cursor-123"
+        mock_client.articles.list.return_value = response
+
+        result = await devrev_articles_list(mock_ctx, cursor="prev-cursor", limit=10)
+
+        assert result["count"] == 1
+        assert result["next_cursor"] == "cursor-123"
+        mock_client.articles.list.assert_called_once()
+        call_args = mock_client.articles.list.call_args
+        request = call_args[0][0]
+        assert request.cursor == "prev-cursor"
+        assert request.limit == 10
+
     @pytest.mark.asyncio
     async def test_list_error(self, mock_ctx, mock_client):
         """Test listing articles with API error."""

{devrev_python_sdk-2.10.0 → devrev_python_sdk-2.10.1}/uv.lock

@@ -35,14 +35,14 @@ wheels = [
 
 [[package]]
 name = "authlib"
-version = "1.6.6"
+version = "1.6.9"
 source = { registry = "https://pypi.org/simple" }
 dependencies = [
     { name = "cryptography" },
 ]
-sdist = { url = "https://files.pythonhosted.org/packages/bb/9b/b1661026ff24bc641b76b78c5222d614776b0c085bcfdac9bd15a1cb4b35/authlib-1.6.6.tar.gz", hash = "sha256:45770e8e056d0f283451d9996fbb59b70d45722b45d854d58f32878d0a40c38e", size = 164894, upload-time = "2025-12-12T08:01:41.464Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/af/98/00d3dd826d46959ad8e32af2dbb2398868fd9fd0683c26e56d0789bd0e68/authlib-1.6.9.tar.gz", hash = "sha256:d8f2421e7e5980cc1ddb4e32d3f5fa659cfaf60d8eaf3281ebed192e4ab74f04", size = 165134, upload-time = "2026-03-02T07:44:01.998Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/54/51/321e821856452f7386c4e9df866f196720b1ad0c5ea1623ea7399969ae3b/authlib-1.6.6-py2.py3-none-any.whl", hash = "sha256:7d9e9bc535c13974313a87f53e8430eb6ea3d1cf6ae4f6efcd793f2e949143fd", size = 244005, upload-time = "2025-12-12T08:01:40.209Z" },
+    { url = "https://files.pythonhosted.org/packages/53/23/b65f568ed0c22f1efacb744d2db1a33c8068f384b8c9b482b52ebdbc3ef6/authlib-1.6.9-py2.py3-none-any.whl", hash = "sha256:f08b4c14e08f0861dc18a32357b33fbcfd2ea86cfe3fe149484b4d764c4a0ac3", size = 244197, upload-time = "2026-03-02T07:44:00.307Z" },
 ]
 
 [[package]]
@@ -83,6 +83,19 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/55/1a/5b0320642cca53a473e79c7d273071b5a9a8578f9e370b74da5daa2768d7/bandit-1.9.2-py3-none-any.whl", hash = "sha256:bda8d68610fc33a6e10b7a8f1d61d92c8f6c004051d5e946406be1fb1b16a868", size = 134377, upload-time = "2025-11-23T21:36:17.39Z" },
 ]
 
+[[package]]
+name = "beautifulsoup4"
+version = "4.14.3"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "soupsieve" },
+    { name = "typing-extensions" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/c3/b0/1c6a16426d389813b48d95e26898aff79abbde42ad353958ad95cc8c9b21/beautifulsoup4-4.14.3.tar.gz", hash = "sha256:6292b1c5186d356bba669ef9f7f051757099565ad9ada5dd630bd9de5fa7fb86", size = 627737, upload-time = "2025-11-30T15:08:26.084Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/1a/39/47f9197bdd44df24d67ac8893641e16f386c984a0619ef2ee4c51fbbc019/beautifulsoup4-4.14.3-py3-none-any.whl", hash = "sha256:0918bfe44902e6ad8d57732ba310582e98da931428d231a5ecb9e7c703a735bb", size = 107721, upload-time = "2025-11-30T15:08:24.087Z" },
+]
+
 [[package]]
 name = "boolean-py"
 version = "5.0"
@@ -470,10 +483,12 @@ wheels = [
 
 [[package]]
 name = "devrev-python-sdk"
-version = "2.4.1"
+version = "2.10.1"
 source = { editable = "." }
 dependencies = [
+    { name = "beautifulsoup4" },
     { name = "httpx" },
+    { name = "markdown" },
     { name = "pydantic" },
     { name = "pydantic-settings" },
     { name = "python-dotenv" },
@@ -510,6 +525,7 @@ dev = [
     { name = "pre-commit" },
     { name = "pytest" },
     { name = "pytest-asyncio" },
+    { name = "pytest-benchmark" },
     { name = "pytest-cov" },
     { name = "pytest-mock" },
     { name = "respx" },
@@ -535,8 +551,10 @@ security = [
 [package.metadata]
 requires-dist = [
     { name = "bandit", marker = "extra == 'security'", specifier = ">=1.7.0" },
+    { name = "beautifulsoup4", specifier = ">=4.12.0" },
     { name = "devrev-python-sdk", extras = ["dev", "docs", "benchmark", "security", "mcp"], marker = "extra == 'all'" },
     { name = "httpx", specifier = ">=0.25.0" },
+    { name = "markdown", specifier = ">=3.5.0" },
     { name = "mcp", marker = "extra == 'mcp'", specifier = ">=1.0.0" },
     { name = "mkdocs", marker = "extra == 'docs'", specifier = ">=1.5.0" },
     { name = "mkdocs-material", marker = "extra == 'docs'", specifier = ">=9.5.0" },
@@ -551,6 +569,7 @@ requires-dist = [
     { name = "pytest", marker = "extra == 'dev'", specifier = ">=7.4.0,<10.0.0" },
     { name = "pytest-asyncio", marker = "extra == 'dev'", specifier = ">=0.21.0" },
     { name = "pytest-benchmark", marker = "extra == 'benchmark'", specifier = ">=4.0.0" },
+    { name = "pytest-benchmark", marker = "extra == 'dev'", specifier = ">=4.0.0" },
     { name = "pytest-cov", marker = "extra == 'dev'", specifier = ">=4.1.0" },
     { name = "pytest-mock", marker = "extra == 'dev'", specifier = ">=3.12.0" },
     { name = "python-dotenv", specifier = ">=1.0.0" },
@@ -2072,6 +2091,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/32/46/9cb0e58b2deb7f82b84065f37f3bffeb12413f947f9388e4cac22c4621ce/sortedcontainers-2.4.0-py2.py3-none-any.whl", hash = "sha256:a163dcaede0f1c021485e957a39245190e74249897e2ae4b2aa38595db237ee0", size = 29575, upload-time = "2021-05-16T22:03:41.177Z" },
 ]
 
+[[package]]
+name = "soupsieve"
+version = "2.8.3"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/7b/ae/2d9c981590ed9999a0d91755b47fc74f74de286b0f5cee14c9269041e6c4/soupsieve-2.8.3.tar.gz", hash = "sha256:3267f1eeea4251fb42728b6dfb746edc9acaffc4a45b27e19450b676586e8349", size = 118627, upload-time = "2026-01-20T04:27:02.457Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/46/2c/1462b1d0a634697ae9e55b3cecdcb64788e8b7d63f54d923fcd0bb140aed/soupsieve-2.8.3-py3-none-any.whl", hash = "sha256:ed64f2ba4eebeab06cc4962affce381647455978ffc1e36bb79a545b91f45a95", size = 37016, upload-time = "2026-01-20T04:27:01.012Z" },
+]
+
 [[package]]
 name = "sse-starlette"
 version = "3.2.0"