pdfdancer-client-python 0.3.12__tar.gz → 0.3.13__tar.gz

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/.github/workflows/ci.yml

@@ -12,6 +12,10 @@ jobs:
     # Quick minimal test on non-main branches
     if: github.ref != 'refs/heads/main'
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      issues: read
+      pull-requests: read
     strategy:
       fail-fast: false
       max-parallel: 2
@@ -23,16 +27,51 @@ jobs:
         with:
           submodules: true
 
-      - name: Detect API URL override
-        if: github.event_name == 'pull_request'
-        id: api-url
-        run: |
-          if [ -f .api-url ]; then
-            url=$(cat .api-url | tr -d '[:space:]')
-            if [ -n "$url" ]; then
-              echo "base_url=$url" >> "$GITHUB_OUTPUT"
-            fi
-          fi
+      - name: Resolve API base URL
+        id: resolve-api-base-url
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.PROPAGATE_TOKEN }}
+          script: |
+            const fallbackUrl = 'https://api-staging.pdfdancer.com';
+            const pr = context.payload.pull_request;
+            if (!pr) {
+              core.setOutput('base_url', fallbackUrl);
+              return;
+            }
+
+            const body = pr.body || '';
+            const apiPrMatch = body.match(/## Source API PR[\s\S]*?^- Repository:\s*MenschMachine\/pdfdancer-api\s*$[\s\S]*?^- PR:\s*#?(\d+)\s*$/im);
+            if (!apiPrMatch) {
+              core.info('No Source API PR block found in the PR body; using staging API.');
+              core.setOutput('base_url', fallbackUrl);
+              return;
+            }
+
+            const apiPrNumber = Number(apiPrMatch[1]);
+            const comments = await github.paginate(github.rest.issues.listComments, {
+              owner: 'MenschMachine',
+              repo: 'pdfdancer-api',
+              issue_number: apiPrNumber,
+              per_page: 100,
+            });
+            const previewComments = comments.filter((comment) =>
+              typeof comment.body === 'string' && comment.body.includes('### Preview Environment')
+            );
+            if (previewComments.length === 0) {
+              core.setFailed(`Source API PR #${apiPrNumber} is referenced by this PR, but no preview comment was found.`);
+              return;
+            }
+
+            const previewComment = previewComments[previewComments.length - 1];
+            const urlMatch = previewComment.body.match(/\*\*API URL:\*\*\s*(https?:\/\/\S+)/);
+            if (!urlMatch) {
+              core.setFailed(`Preview comment on source API PR #${apiPrNumber} does not contain a parseable API URL.`);
+              return;
+            }
+
+            core.info(`Using preview API URL from source API PR #${apiPrNumber}: ${urlMatch[1]}`);
+            core.setOutput('base_url', urlMatch[1]);
 
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v5
@@ -53,8 +92,8 @@ jobs:
 
       - name: Run tests
         run: |
-          PDFDANCER_BASE_URL=${{ steps.api-url.outputs.base_url || 'https://api-staging.pdfdancer.com' }} \
-          PDFDANCER_TOKEN=42 \
+          PDFDANCER_BASE_URL=${{ steps.resolve-api-base-url.outputs.base_url }} \
+          PDFDANCER_API_TOKEN=42 \
           venv/bin/python -m pytest tests/ -v --maxfail=3
 
       - name: Build distribution packages
@@ -68,6 +107,10 @@ jobs:
     # Full matrix only on main
     if: github.ref == 'refs/heads/main'
     runs-on: ${{ matrix.os }}
+    permissions:
+      contents: read
+      issues: read
+      pull-requests: read
     strategy:
       fail-fast: false
       max-parallel: 2
@@ -80,17 +123,13 @@ jobs:
         with:
           submodules: true
 
-      - name: Detect API URL override
-        if: github.event_name == 'pull_request'
-        id: api-url
-        shell: bash
-        run: |
-          if [ -f .api-url ]; then
-            url=$(cat .api-url | tr -d '[:space:]')
-            if [ -n "$url" ]; then
-              echo "base_url=$url" >> "$GITHUB_OUTPUT"
-            fi
-          fi
+      - name: Resolve API base URL
+        id: resolve-api-base-url
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.PROPAGATE_TOKEN }}
+          script: |
+            core.setOutput('base_url', 'https://api-staging.pdfdancer.com');
 
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v5
@@ -115,8 +154,8 @@ jobs:
       - name: Run tests (Unix)
         if: runner.os != 'Windows'
         run: |
-          PDFDANCER_BASE_URL=${{ steps.api-url.outputs.base_url || 'https://api-staging.pdfdancer.com' }} \
-          PDFDANCER_TOKEN=42 \
+          PDFDANCER_BASE_URL=${{ steps.resolve-api-base-url.outputs.base_url }} \
+          PDFDANCER_API_TOKEN=42 \
           venv/bin/python -m pytest tests/ -v --maxfail=3
 
       - name: Build & Validate (Unix)
@@ -146,8 +185,8 @@ jobs:
         if: runner.os == 'Windows'
         shell: cmd
         run: |
-          set PDFDANCER_BASE_URL=${{ steps.api-url.outputs.base_url || 'https://api-staging.pdfdancer.com' }}
-          set PDFDANCER_TOKEN=42
+          set PDFDANCER_BASE_URL=${{ steps.resolve-api-base-url.outputs.base_url }}
+          set PDFDANCER_API_TOKEN=42
           venv\Scripts\python -m pytest tests/ -v --maxfail=3
 
       - name: Build & Validate (Windows)
@@ -155,4 +194,4 @@ jobs:
         shell: cmd
         run: |
           venv\Scripts\python -m build
-          venv\Scripts\python -m twine check dist/*
+          venv\Scripts\python -m twine check dist/*

pdfdancer_client_python-0.3.13/.github/workflows/sdk-backward-compat.yml

@@ -0,0 +1,88 @@
+name: SDK Backward Compatibility Tests
+
+on:
+  repository_dispatch:
+    types: [api-preview-deployed]
+
+  workflow_dispatch:
+    inputs:
+      api_url:
+        description: 'API base URL to test against'
+        required: true
+        type: string
+      sdk_token:
+        description: 'API token for testing'
+        required: true
+        type: string
+        default: '42'
+
+jobs:
+  get-tags:
+    runs-on: ubuntu-latest
+    outputs:
+      tags: ${{ steps.tags.outputs.tags }}
+    steps:
+      - name: Fetch tags
+        id: tags
+        run: |
+          TAGS=$(git ls-remote --tags https://github.com/MenschMachine/pdfdancer-client-python.git \
+            | grep -v '\^{}' \
+            | awk -F'/' '{print $NF}' \
+            | grep '^v' \
+            | sort -V \
+            | tail -3 \
+            | jq -R -s -c 'split("\n") | map(select(length > 0))')
+          echo "tags=$TAGS" >> "$GITHUB_OUTPUT"
+          echo "Last 3 SDK tags: $TAGS"
+
+  test-compat:
+    needs: get-tags
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+    strategy:
+      fail-fast: false
+      max-parallel: 1
+      matrix:
+        tag: ${{ fromJSON(needs.get-tags.outputs.tags) }}
+
+    steps:
+      - name: Checkout SDK at ${{ matrix.tag }}
+        uses: actions/checkout@v4
+        with:
+          repository: MenschMachine/pdfdancer-client-python
+          ref: ${{ matrix.tag }}
+          path: sdk
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.12'
+
+      - name: Create virtual environment
+        run: python -m venv venv
+
+      - name: Install SDK at ${{ matrix.tag }}
+        run: |
+          venv/bin/pip install --upgrade pip
+          venv/bin/pip install -e "./sdk[dev]"
+
+      - name: Run e2e tests
+        working-directory: sdk
+        run: |
+          PDFDANCER_BASE_URL=${{ github.event.client_payload.api_url || inputs.api_url }} \
+          PDFDANCER_API_TOKEN=${{ github.event.client_payload.sdk_token || inputs.sdk_token }} \
+          ../venv/bin/python -m pytest -v -x \
+            -k "not test_redact_multiple_paths and not test_context_manager_vs_manual_apply" \
+            --junitxml=test-results.xml
+        env:
+          PYTHONWARNINGS: "ignore:Unverified HTTPS request"
+
+      - name: Upload test results
+        if: failure()
+        uses: actions/upload-artifact@v4
+        with:
+          name: test-results-${{ matrix.tag }}
+          path: |
+            sdk/test-results.xml
+            sdk/.pytest_cache/
+          retention-days: 7

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pdfdancer-client-python
-Version: 0.3.12
+Version: 0.3.13
 Summary: Python client for PDFDancer API
 Author-email: "The Famous Cat Ltd." <hi@thefamouscat.com>
 License: 

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pdfdancer-client-python"
-version = "0.3.12"
+version = "0.3.13"
 description = "Python client for PDFDancer API"
 readme = "README.md"
 authors = [

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer/__init__.py

@@ -28,6 +28,7 @@ from .models import (
     ImageTransformRequest,
     ImageTransformType,
     Line,
+    ModifyPathRequest,
     ObjectRef,
     ObjectType,
     Orientation,
@@ -36,6 +37,7 @@ from .models import (
     Paragraph,
     Path,
     PathGroupInfo,
+    PathObjectRef,
     PathSegment,
     Point,
     Position,
@@ -55,7 +57,7 @@ from .paragraph_builder import ParagraphBuilder
 from .path_builder import BezierBuilder, LineBuilder, PathBuilder
 from .text_line_builder import TextLineBuilder
 
-__version__ = "0.3.12"
+__version__ = "0.3.13"
 __all__ = [
     "PDFDancer",
     "ParagraphBuilder",
@@ -65,6 +67,8 @@ __all__ = [
     "LineBuilder",
     "BezierBuilder",
     "ObjectRef",
+    "PathObjectRef",
+    "ModifyPathRequest",
     "Position",
     "ObjectType",
     "Font",

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer/models.py

@@ -1895,3 +1895,85 @@ class ImageTransformRequest:
             result["fillColor"] = self.fill_color
 
         return result
+
+
+@dataclass
+class ModifyPathRequest:
+    """Request to modify path stroke and fill colors.
+
+    Parameters:
+    - object_ref: Reference to the path to modify.
+    - stroke_color: New stroke color (optional - null means don't change).
+    - fill_color: New fill color (optional - null means don't change).
+
+    Example:
+    ```python
+    req = ModifyPathRequest(object_ref=path_ref, stroke_color=Color(255, 0, 0), fill_color=None)
+    payload = req.to_dict()
+    ```
+    """
+
+    object_ref: ObjectRef
+    stroke_color: Optional[Color] = None
+    fill_color: Optional[Color] = None
+
+    def to_dict(self) -> dict:
+        """Convert to dictionary for JSON serialization."""
+        result: Dict[str, Any] = {
+            "ref": self.object_ref.to_dict(),
+        }
+
+        if self.stroke_color is not None:
+            result["strokeColor"] = {
+                "red": self.stroke_color.r,
+                "green": self.stroke_color.g,
+                "blue": self.stroke_color.b,
+                "alpha": self.stroke_color.a,
+            }
+
+        if self.fill_color is not None:
+            result["fillColor"] = {
+                "red": self.fill_color.r,
+                "green": self.fill_color.g,
+                "blue": self.fill_color.b,
+                "alpha": self.fill_color.a,
+            }
+
+        return result
+
+
+class PathObjectRef(ObjectRef):
+    """
+    Reference to a path object with stroke and fill color information.
+
+    Parameters (typically provided by the server):
+    - internal_id: Identifier of the path object.
+    - position: Position of the path.
+    - object_type: Should be ObjectType.PATH.
+    - stroke_color: Stroke/outline color of the path (optional).
+    - fill_color: Fill color of the path (optional).
+
+    Usage:
+    - Returned by find/snapshot APIs when querying paths.
+    - Pass to ModifyPathRequest to update path colors.
+    """
+
+    def __init__(
+        self,
+        internal_id: str,
+        position: Position,
+        object_type: ObjectType,
+        stroke_color: Optional[Color] = None,
+        fill_color: Optional[Color] = None,
+    ):
+        super().__init__(internal_id, position, object_type)
+        self.stroke_color = stroke_color
+        self.fill_color = fill_color
+
+    def get_stroke_color(self) -> Optional[Color]:
+        """Get the stroke/outline color of the path."""
+        return self.stroke_color
+
+    def get_fill_color(self) -> Optional[Color]:
+        """Get the fill color of the path."""
+        return self.fill_color

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer/pdfdancer_v1.py

@@ -47,6 +47,7 @@ from .models import (
     FontType,
     FormFieldRef,
     Image,
+    ModifyPathRequest,
     ModifyRequest,
     ModifyTextRequest,
     MoveRequest,
@@ -58,6 +59,7 @@ from .models import (
     PageSize,
     PageSnapshot,
     Paragraph,
+    PathObjectRef,
     Position,
     PositionMode,
     RedactRequest,
@@ -2755,6 +2757,38 @@ class PDFDancer:
         self._invalidate_snapshots()
         return result
 
+    def _modify_path(
+        self,
+        object_ref: ObjectRef,
+        stroke_color: Optional[Color],
+        fill_color: Optional[Color],
+    ) -> CommandResult:
+        """
+        Modifies the stroke and fill colors of a path object.
+
+        Args:
+            object_ref: Reference to the path to modify.
+            stroke_color: New stroke color (None means don't change).
+            fill_color: New fill color (None means don't change).
+
+        Returns:
+            CommandResult indicating success or failure.
+        """
+        if object_ref is None:
+            raise ValidationException("Object reference cannot be null")
+
+        request_data = ModifyPathRequest(
+            object_ref=object_ref,
+            stroke_color=stroke_color,
+            fill_color=fill_color,
+        ).to_dict()
+        response = self._make_request("PUT", "/pdf/modify/path", data=request_data)
+        result = CommandResult.from_dict(response.json())
+
+        # Invalidate snapshot caches after mutation
+        self._invalidate_snapshots()
+        return result
+
     # Font Operations
 
     def find_fonts(self, font_name: str, font_size: int) -> List[Font]:
@@ -3159,6 +3193,43 @@ class PDFDancer:
             value=obj_data["value"] if "value" in obj_data else None,
         )
 
+    def _parse_path_object_ref(self, obj_data: dict) -> PathObjectRef:
+        """Parse JSON object data into PathObjectRef instance with color information."""
+        position_data = obj_data.get("position", {})
+        position = self._parse_position(position_data) if position_data else None
+
+        object_type = ObjectType(obj_data["type"])
+
+        # Parse stroke color if present
+        stroke_color = None
+        stroke_color_data = obj_data.get("strokeColor")
+        if isinstance(stroke_color_data, dict):
+            red = stroke_color_data.get("red")
+            green = stroke_color_data.get("green")
+            blue = stroke_color_data.get("blue")
+            alpha = stroke_color_data.get("alpha", 255)
+            if all(isinstance(v, int) for v in [red, green, blue]):
+                stroke_color = Color(red, green, blue, alpha)
+
+        # Parse fill color if present
+        fill_color = None
+        fill_color_data = obj_data.get("fillColor")
+        if isinstance(fill_color_data, dict):
+            red = fill_color_data.get("red")
+            green = fill_color_data.get("green")
+            blue = fill_color_data.get("blue")
+            alpha = fill_color_data.get("alpha", 255)
+            if all(isinstance(v, int) for v in [red, green, blue]):
+                fill_color = Color(red, green, blue, alpha)
+
+        return PathObjectRef(
+            internal_id=obj_data["internalId"] if "internalId" in obj_data else None,
+            position=position,
+            object_type=object_type,
+            stroke_color=stroke_color,
+            fill_color=fill_color,
+        )
+
     @staticmethod
     def _parse_position(pos_data: dict) -> Position:
         """Parse JSON position data into Position instance."""
@@ -3473,6 +3544,9 @@ class PDFDancer:
                 ):
                     # Parse as FormFieldRef to capture name and value
                     elements.append(self._parse_form_field_ref(elem_data))
+                elif elem_type == ObjectType.PATH:
+                    # Parse as PathObjectRef to capture stroke/fill colors
+                    elements.append(self._parse_path_object_ref(elem_data))
                 else:
                     # Parse as basic ObjectRef
                     elements.append(self._parse_object_ref(elem_data))
@@ -3526,9 +3600,7 @@ class PDFDancer:
             self._client.close()
 
     def _to_path_objects(self, refs: List[ObjectRef]) -> List[PathObject]:
-        return [
-            PathObject(self, ref.internal_id, ref.type, ref.position) for ref in refs
-        ]
+        return [PathObject(self, ref) for ref in refs]
 
     def _to_paragraph_objects(self, refs: List[TextObjectRef]) -> List[ParagraphObject]:
         return [ParagraphObject(self, ref) for ref in refs]

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer/types.py

@@ -4,7 +4,7 @@ import sys
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Optional
 
-from . import FormFieldRef, ObjectRef, ObjectType, Point, Position, TextObjectRef
+from . import FormFieldRef, ObjectRef, ObjectType, PathObjectRef, Point, Position, TextObjectRef
 from .exceptions import ValidationException
 
 if TYPE_CHECKING:
@@ -86,11 +86,44 @@ class PDFObjectBase:
 class PathObject(PDFObjectBase):
     """Represents a vector path object inside a PDF page."""
 
+    def __init__(self, client: "PDFDancer", object_ref):
+        """
+        Initialize a PathObject.
+
+        Args:
+            client: PDFDancer client instance
+            object_ref: ObjectRef or PathObjectRef with path data
+        """
+        super().__init__(
+            client, object_ref.internal_id, object_ref.type, object_ref.position
+        )
+        self._object_ref = object_ref
+
     @property
     def bounding_box(self) -> Optional[BoundingRect]:
         """Optional bounding rectangle (if available)."""
         return self.position.bounding_rect
 
+    def edit(self) -> PathEditSession:
+        """Start a fluent editing session to modify path colors."""
+        return PathEditSession(self._client, self.object_ref())
+
+    def object_ref(self):
+        """Return an ObjectRef for this path."""
+        return self._object_ref
+
+    def get_stroke_color(self) -> Optional["Color"]:
+        """Get the stroke/outline color of the path, or None if not set."""
+        if isinstance(self._object_ref, PathObjectRef):
+            return self._object_ref.get_stroke_color()
+        return None
+
+    def get_fill_color(self) -> Optional["Color"]:
+        """Get the fill color of the path, or None if not set."""
+        if isinstance(self._object_ref, PathObjectRef):
+            return self._object_ref.get_fill_color()
+        return None
+
     def __eq__(self, other):
         if not isinstance(other, PathObject):
             return False
@@ -748,3 +781,61 @@ class FormFieldObject(PDFObjectBase):
             and self.name == other.name
             and self.value == other.value
         )
+
+
+class PathEditSession:
+    """
+    Fluent editing helper for modifying path stroke and fill colors.
+    """
+
+    def __init__(self, client: "PDFDancer", object_ref):
+        self._client = client
+        self._object_ref = object_ref
+        self._stroke_color = None
+        self._fill_color = None
+
+    def __enter__(self):
+        return self
+
+    def __exit__(self, exc_type, exc_val, exc_tb):
+        if exc_type:
+            return False
+        self.apply()
+        return False
+
+    def stroke_color(self, color) -> "PathEditSession":
+        """
+        Set the stroke/outline color.
+
+        Args:
+            color: The stroke color (Color object)
+
+        Returns:
+            Self for method chaining
+        """
+        self._stroke_color = color
+        return self
+
+    def fill_color(self, color) -> "PathEditSession":
+        """
+        Set the fill color.
+
+        Args:
+            color: The fill color (Color object)
+
+        Returns:
+            Self for method chaining
+        """
+        self._fill_color = color
+        return self
+
+    def apply(self):
+        """
+        Apply the color modifications to the path.
+
+        Returns:
+            CommandResult indicating success or failure
+        """
+        return self._client._modify_path(
+            self._object_ref, self._stroke_color, self._fill_color
+        )

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer_client_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pdfdancer-client-python
-Version: 0.3.12
+Version: 0.3.13
 Summary: Python client for PDFDancer API
 Author-email: "The Famous Cat Ltd." <hi@thefamouscat.com>
 License: 

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/src/pdfdancer_client_python.egg-info/SOURCES.txt

@@ -1,4 +1,3 @@
-.api-url
 .flake8
 .gitignore
 .gitmodules
@@ -16,6 +15,7 @@ test.sh
 .github/workflows/ci.yml
 .github/workflows/daily-tests.yml
 .github/workflows/release.yml
+.github/workflows/sdk-backward-compat.yml
 docs/capabilities/.gitkeep
 docs/capabilities/CLEAR_CLIPPING.md
 media/logo-orange-512h.webp

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/tests/e2e/test_context_manager.py

@@ -398,6 +398,12 @@ def test_context_manager_example_from_docs():
         )
 
 
+@pytest.mark.skip(reason="""
+The following test is disabled because it fails intermittently:
+Minor variations in PDF byte output result in failures, even though output is visually and functionally identical.
+Such spurious differences can be caused by library version changes, metadata, timestamps, and non-semantic structure within PDFs.
+Therefore, strict byte count comparison is too brittle for a reliable automated test.
+""")
 def test_context_manager_vs_manual_apply():
     """Test that context manager produces same result as manual apply()"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/tests/e2e/test_path.py

@@ -1,6 +1,6 @@
 import pytest
 
-from pdfdancer import ObjectType
+from pdfdancer import Color, ObjectType
 from pdfdancer.pdfdancer_v1 import PDFDancer
 from tests.e2e import _require_env_and_fixture
 from tests.e2e.pdf_assertions import PDFAssertions
@@ -79,3 +79,50 @@ def test_move_path():
             .assert_no_path_at(80, 720)
             .assert_path_is_at("PATH_0_000001", 50.1, 100)
         )
+
+
+def test_modify_path_colors():
+    """Test modifying stroke and fill colors of a path."""
+    base_url, token, pdf_path = _require_env_and_fixture("basic-paths.pdf")
+
+    with PDFDancer.open(pdf_path, token=token, base_url=base_url, timeout=30.0) as pdf:
+        # PATH_0_000001 is a line at (80, 720)
+        path = pdf.page(1).select_paths_at(80, 720)[0]
+        assert path.internal_id == "PATH_0_000001"
+
+        # Modify the stroke color to red
+        red = Color(255, 0, 0)
+        result = path.edit().stroke_color(red).apply()
+        assert result.success, f"Expected success but got: {result.message}"
+
+        # Re-fetch path via select_paths() to verify the stroke color was actually changed
+        # Note: select_paths_at() goes to API which doesn't return colors,
+        # select_paths() uses snapshot which includes color data
+        paths_after_stroke = [p for p in pdf.select_paths() if p.internal_id == "PATH_0_000001"]
+        assert len(paths_after_stroke) == 1
+        path_after_stroke = paths_after_stroke[0]
+        stroke_color = path_after_stroke.get_stroke_color()
+        assert stroke_color is not None, "Expected stroke color to be set after modification"
+        assert stroke_color.r == 255, f"Expected red=255 but got {stroke_color.r}"
+        assert stroke_color.g == 0, f"Expected green=0 but got {stroke_color.g}"
+        assert stroke_color.b == 0, f"Expected blue=0 but got {stroke_color.b}"
+
+        # Modify both stroke and fill colors
+        blue = Color(0, 0, 255)
+        result = path.edit().stroke_color(red).fill_color(blue).apply()
+        assert result.success, f"Expected success but got: {result.message}"
+
+        # Re-fetch via select_paths() to verify both colors were actually changed
+        paths_after_both = [p for p in pdf.select_paths() if p.internal_id == "PATH_0_000001"]
+        assert len(paths_after_both) == 1
+        path_after_both = paths_after_both[0]
+        stroke_color = path_after_both.get_stroke_color()
+        fill_color = path_after_both.get_fill_color()
+        assert stroke_color is not None, "Expected stroke color to be set"
+        assert fill_color is not None, "Expected fill color to be set after modification"
+        assert stroke_color.r == 255, f"Expected stroke red=255 but got {stroke_color.r}"
+        assert stroke_color.g == 0, f"Expected stroke green=0 but got {stroke_color.g}"
+        assert stroke_color.b == 0, f"Expected stroke blue=0 but got {stroke_color.b}"
+        assert fill_color.r == 0, f"Expected fill red=0 but got {fill_color.r}"
+        assert fill_color.g == 0, f"Expected fill green=0 but got {fill_color.g}"
+        assert fill_color.b == 255, f"Expected fill blue=255 but got {fill_color.b}"

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/tests/e2e/test_redact.py

@@ -202,6 +202,7 @@ def test_redact_path():
         assertions.assert_number_of_paths(8)
 
 
+@pytest.mark.skip(reason="TODO cannot make a meaningful assertion currently")
 def test_redact_multiple_paths():
     """Test batch redacting multiple paths"""
     base_url, token, pdf_path = _require_env_and_fixture("basic-paths.pdf")

{pdfdancer_client_python-0.3.12 → pdfdancer_client_python-0.3.13}/tests/test_pdf_object_equality.py

@@ -6,7 +6,7 @@ from unittest.mock import Mock
 
 import pytest
 
-from pdfdancer import Color, ObjectType, Position, TextObjectRef
+from pdfdancer import Color, ObjectRef, ObjectType, Position, TextObjectRef
 from pdfdancer.types import (
     FormFieldObject,
     FormObject,
@@ -24,9 +24,10 @@ class TestPDFObjectEquality:
         """PathObject instances with same internal_id and type should be equal."""
         mock_client = Mock()
         position = Position.at_page(1)
+        ref = ObjectRef("id123", position, ObjectType.PATH)
 
-        obj1 = PathObject(mock_client, "id123", ObjectType.PATH, position)
-        obj2 = PathObject(mock_client, "id123", ObjectType.PATH, position)
+        obj1 = PathObject(mock_client, ref)
+        obj2 = PathObject(mock_client, ref)
 
         assert obj1 == obj2
 
@@ -34,9 +35,11 @@ class TestPDFObjectEquality:
         """PathObject instances with different internal_id should not be equal."""
         mock_client = Mock()
         position = Position.at_page(1)
+        ref1 = ObjectRef("id123", position, ObjectType.PATH)
+        ref2 = ObjectRef("id456", position, ObjectType.PATH)
 
-        obj1 = PathObject(mock_client, "id123", ObjectType.PATH, position)
-        obj2 = PathObject(mock_client, "id456", ObjectType.PATH, position)
+        obj1 = PathObject(mock_client, ref1)
+        obj2 = PathObject(mock_client, ref2)
 
         assert obj1 != obj2
 
@@ -46,8 +49,11 @@ class TestPDFObjectEquality:
         position1 = Position.at_page(1)
         position2 = Position.at_page(2)
 
-        obj1 = PathObject(mock_client, "id123", ObjectType.PATH, position1)
-        obj2 = PathObject(mock_client, "id123", ObjectType.PATH, position2)
+        ref1 = ObjectRef("id123", position1, ObjectType.PATH)
+        ref2 = ObjectRef("id123", position2, ObjectType.PATH)
+
+        obj1 = PathObject(mock_client, ref1)
+        obj2 = PathObject(mock_client, ref2)
 
         assert obj1 != obj2
 
@@ -55,8 +61,9 @@ class TestPDFObjectEquality:
         """PathObject should not equal non-PathObject."""
         mock_client = Mock()
         position = Position.at_page(1)
+        path_ref = ObjectRef("id123", position, ObjectType.PATH)
 
-        obj1 = PathObject(mock_client, "id123", ObjectType.PATH, position)
+        obj1 = PathObject(mock_client, path_ref)
         obj2 = ImageObject(mock_client, "id123", ObjectType.IMAGE, position)
 
         assert obj1 != obj2
@@ -296,8 +303,9 @@ class TestPDFObjectEquality:
         """PDFObjectBase subclasses should not equal None."""
         mock_client = Mock()
         position = Position.at_page(1)
+        ref = ObjectRef("id123", position, ObjectType.PATH)
 
-        obj = PathObject(mock_client, "id123", ObjectType.PATH, position)
+        obj = PathObject(mock_client, ref)
 
         assert obj != None
 
@@ -305,7 +313,8 @@ class TestPDFObjectEquality:
         """PDFObjectBase subclasses should not equal strings."""
         mock_client = Mock()
         position = Position.at_page(1)
+        ref = ObjectRef("id123", position, ObjectType.PATH)
 
-        obj = PathObject(mock_client, "id123", ObjectType.PATH, position)
+        obj = PathObject(mock_client, ref)
 
         assert obj != "id123"

pdfdancer_client_python-0.3.12/.api-url

@@ -1 +0,0 @@
-http://46.225.120.69:8080