pdfdancer-client-python 0.2.27__tar.gz → 0.2.29__tar.gz

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/PKG-INFO

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

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/pyproject.toml

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

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/src/pdfdancer/types.py

@@ -160,14 +160,14 @@ class BaseTextEdit:
 
 class TextLineEdit(BaseTextEdit):
     def apply(self) -> bool:
-        # Text lines don't support line spacing - ignore if set
+        # Line spacing is NOT supported for text lines - fail hard
         if self._line_spacing is not None:
-            print(
-                "WARNING: Line spacing is not supported for text lines and will be ignored",
-                file=sys.stderr,
+            raise UnsupportedOperation(
+                "Line spacing changes are not supported for individual text lines. "
+                "Line spacing can only be modified on paragraphs, not individual text lines."
             )
 
-        # Simple text-only change
+        # If only text changed (no font, color, or position), use simple text modification
         only_text_changed = (
             self._new_text is not None
             and self._font_name is None
@@ -185,7 +185,7 @@ class TextLineEdit(BaseTextEdit):
                 print(f"WARNING: {result.warning}", file=sys.stderr)
             return result
 
-        # Position-only change (move operation)
+        # If only position changed (move operation)
         only_move = (
             self._position is not None
             and self._new_text is None
@@ -216,15 +216,54 @@ class TextLineEdit(BaseTextEdit):
             result = self._target_obj._client._move(self._object_ref, position)
             return result
 
-        # Text lines with font/color styling changes are not supported by the PDF API
-        # Text lines are components of paragraphs and can only have their text modified
-        # or be moved, but not have their styling changed independently
-        raise UnsupportedOperation(
-            "Text lines can only have their text content modified or be moved. "
-            "Font and color changes are not supported for individual text lines. "
-            "To modify text styling, please modify the parent paragraph instead."
+        # For font/color changes or combined operations, use TextLineBuilder
+        # This ensures proper handling of font/color fallbacks just like ParagraphEditSession
+        from .text_line_builder import TextLineBuilder
+
+        builder = TextLineBuilder.from_object_ref(
+            self._target_obj._client, self._object_ref
         )
 
+        # Apply modifications to builder
+        # IMPORTANT: Always explicitly set text to ensure it's preserved
+        if self._new_text is not None:
+            builder.text(self._new_text)
+        elif hasattr(self._object_ref, "text") and self._object_ref.text:
+            # Preserve original text when only changing font/color/position
+            builder.text(self._object_ref.text)
+
+        # IMPORTANT: Always explicitly set font to ensure it's preserved
+        if self._font_name is not None and self._font_size is not None:
+            builder.font(self._font_name, self._font_size)
+        elif hasattr(self._object_ref, "font_name") and hasattr(self._object_ref, "font_size"):
+            if self._object_ref.font_name and self._object_ref.font_size:
+                # Preserve original font when only changing color/position
+                builder.font(self._object_ref.font_name, self._object_ref.font_size)
+
+        if self._color is not None:
+            builder.color(self._color)
+        if self._position is not None:
+            x = self._position.x()
+            y = self._position.y()
+            if x is None or y is None:
+                raise ValidationException("Position must have x and y coordinates")
+            page_index = (
+                self._object_ref.position.page_index
+                if self._object_ref.position
+                else None
+            )
+            if page_index is None:
+                raise ValidationException(
+                    "Text line position must include a page index"
+                )
+            builder.at(page_index, x, y)
+
+        # Use builder's modify method which handles all the complexity
+        result = builder.modify(self._object_ref)
+        if result.warning:
+            print(f"WARNING: {result.warning}", file=sys.stderr)
+        return result
+
 
 class ParagraphObject(PDFObjectBase):
     """Represents a paragraph text block inside a PDF page."""

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/src/pdfdancer_client_python.egg-info/PKG-INFO

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

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/tests/e2e/test_context_manager.py

@@ -12,7 +12,7 @@ def test_context_manager_basic_usage():
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         paragraphs = pdf.select_paragraphs()
-        assert len(paragraphs) == 20
+        assert 20 <= len(paragraphs) <= 22  # strange, but differs on linux
 
 
 def test_context_manager_edit_text_only():

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/tests/e2e/test_page.py

@@ -1,4 +1,5 @@
 from pdfdancer import ObjectType, Orientation, PageSize, PDFDancer
+
 from tests.e2e import _require_env_and_fixture
 from tests.e2e.pdf_assertions import PDFAssertions
 
@@ -7,14 +8,15 @@ def test_get_all_elements():
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url, timeout=30.0) as pdf:
-        expected_total = 95
         assert (
-            len(pdf.select_elements()) == expected_total
-        ), f"{len(pdf.select_elements())} elements found but {expected_total} elements expected"
+                95 <= len(pdf.select_elements()) <= 97
+        ), f"{len(pdf.select_elements())} elements found but  95-97 elements expected"
         actual_total = 0
         for page in pdf.pages():
             actual_total += len(page.select_elements())
-        assert actual_total == expected_total
+        assert (
+                95 <= actual_total <= 97
+        ), f"{actual_total} elements found but  95-97 elements expected"
 
 
 def test_get_pages():

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/tests/e2e/test_paragraph.py

@@ -10,8 +10,8 @@ def test_find_paragraphs_by_position():
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url, timeout=30.0) as pdf:
-        paras = pdf.select_paragraphs()
-        assert len(paras) == 20
+        paragraphs = pdf.select_paragraphs()
+        assert 20 <= len(paragraphs) <= 22  # strange, but differs on linux
 
         paras_page0 = pdf.page(0).select_paragraphs()
         assert len(paras_page0) == 3

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/tests/e2e/test_snapshot.py

@@ -4,8 +4,8 @@ Validates that snapshot data matches select_* method results before, during, and
 """
 
 import pytest
-
 from pdfdancer import ObjectType, PDFDancer
+
 from tests.e2e import _require_env_and_fixture
 
 
@@ -34,7 +34,7 @@ def test_page_snapshot_matches_select_paragraphs():
         selected_ids = {p.internal_id for p in selected_paragraphs}
 
         assert (
-            selected_ids == snapshot_ids
+                selected_ids == snapshot_ids
         ), "Snapshot and select_paragraphs() should return identical paragraph IDs"
 
 
@@ -59,7 +59,7 @@ def test_page_snapshot_matches_select_images():
             selected_ids = {img.internal_id for img in selected_images}
 
             assert (
-                selected_ids == snapshot_ids
+                    selected_ids == snapshot_ids
             ), "Snapshot and select_images() should return identical image IDs"
 
 
@@ -86,7 +86,7 @@ def test_page_snapshot_matches_select_forms():
             selected_ids = {form.internal_id for form in selected_forms}
 
             assert (
-                selected_ids == snapshot_ids
+                    selected_ids == snapshot_ids
             ), "Snapshot and select_forms() should return identical form IDs"
 
 
@@ -102,12 +102,12 @@ def test_page_snapshot_matches_select_form_fields():
             e
             for e in snapshot.elements
             if e.type
-            in (
-                ObjectType.FORM_FIELD,
-                ObjectType.TEXT_FIELD,
-                ObjectType.CHECK_BOX,
-                ObjectType.RADIO_BUTTON,
-            )
+               in (
+                   ObjectType.FORM_FIELD,
+                   ObjectType.TEXT_FIELD,
+                   ObjectType.CHECK_BOX,
+                   ObjectType.RADIO_BUTTON,
+               )
         ]
 
         selected_form_fields = page.select_form_fields()
@@ -121,7 +121,7 @@ def test_page_snapshot_matches_select_form_fields():
             selected_ids = {field.internal_id for field in selected_form_fields}
 
             assert (
-                selected_ids == snapshot_ids
+                    selected_ids == snapshot_ids
             ), "Snapshot and select_form_fields() should return identical form field IDs"
 
 
@@ -143,7 +143,7 @@ def test_page_snapshot_contains_all_element_types():
 
         # Verify we have at least some text elements
         assert (
-            paragraph_count > 0 or text_line_count > 0
+                paragraph_count > 0 or text_line_count > 0
         ), "Page should have at least some text elements"
 
         # Verify all elements have required fields
@@ -173,7 +173,7 @@ def test_document_snapshot_matches_all_pages():
             individual_page_ids = {e.internal_id for e in individual_page_snap.elements}
 
             assert (
-                individual_page_ids == doc_page_ids
+                    individual_page_ids == doc_page_ids
             ), f"Page {i} should have identical elements in document and individual snapshots"
 
 
@@ -201,7 +201,7 @@ def test_type_filter_matches_select_method():
         selected_ids = {p.internal_id for p in selected_paragraphs}
 
         assert (
-            selected_ids == snapshot_ids
+                selected_ids == snapshot_ids
         ), "Filtered snapshot and select_paragraphs() should return identical IDs"
 
 
@@ -239,7 +239,7 @@ def test_total_element_count_matches_expected():
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         # Showcase.pdf - Python API filters certain types (638)
         all_elements = pdf.select_elements()
-        assert len(all_elements) == 95, "Showcase.pdf should have 95 total elements"
+        assert 95 <= len(all_elements) <= 97, "Showcase.pdf should have 95 total elements"
 
         doc_snapshot = pdf.get_document_snapshot()
         snapshot_total = sum(len(p.elements) for p in doc_snapshot.pages)
@@ -266,7 +266,7 @@ def test_snapshot_consistency_across_multiple_pages():
             page_snap = pdf.get_page_snapshot(i)
             assert page_snap is not None, f"Page {i} snapshot should not be None"
             assert (
-                page_snap.page_ref.position.page_index == i
+                    page_snap.page_ref.position.page_index == i
             ), "Page snapshot should have correct page index"
 
 
@@ -280,7 +280,7 @@ def test_document_snapshot_contains_fonts():
 
         # Should have fonts
         assert (
-            doc_snapshot.fonts is not None
+                doc_snapshot.fonts is not None
         ), "Document snapshot should have fonts list"
         assert len(doc_snapshot.fonts) > 0, "Document should have at least one font"
 
@@ -289,5 +289,5 @@ def test_document_snapshot_contains_fonts():
             assert font.font_name is not None, "Font should have a name"
             assert font.font_type is not None, "Font should have a type"
             assert (
-                font.similarity_score is not None
+                    font.similarity_score is not None
             ), "Font should have similarity score"

{pdfdancer_client_python-0.2.27 → pdfdancer_client_python-0.2.29}/tests/e2e/test_text_line_edit.py

@@ -1,7 +1,7 @@
 import pytest
-
 from pdfdancer import Color
 from pdfdancer.pdfdancer_v1 import PDFDancer
+
 from tests.e2e import _require_env_and_fixture
 from tests.e2e.pdf_assertions import PDFAssertions
 
@@ -15,8 +15,8 @@ def test_text_line_edit_text_only():
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
@@ -34,49 +34,45 @@ def test_text_line_edit_text_only():
 
 
 def test_text_line_edit_font_only():
-    """Test that text line font-only changes raise UnsupportedOperation"""
+    """Test text line font-only changes"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
+        original_text = text_line.text
 
-        # Font changes on text lines should raise UnsupportedOperation
-        from pdfdancer.types import UnsupportedOperation
+        # Font changes on text lines should work
+        with text_line.edit() as editor:
+            editor.font("Helvetica", 28)
 
-        with pytest.raises(
-            UnsupportedOperation, match="Font and color changes are not supported"
-        ):
-            with text_line.edit() as editor:
-                editor.font("Helvetica", 28)
+        # Verify the text still exists (font changed but text preserved)
+        PDFAssertions(pdf).assert_textline_exists(original_text)
 
 
 def test_text_line_edit_color_only():
-    """Test that text line color-only changes raise UnsupportedOperation"""
+    """Test text line color-only changes"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
+        original_text = text_line.text
 
-        # Color changes on text lines should raise UnsupportedOperation
-        from pdfdancer.types import UnsupportedOperation
+        # Color changes on text lines should work
+        with text_line.edit() as editor:
+            editor.color(Color(0, 255, 0))
 
-        with pytest.raises(
-            UnsupportedOperation, match="Font and color changes are not supported"
-        ):
-            with text_line.edit() as editor:
-                editor.color(Color(0, 255, 0))
+        # Verify the text still exists (color changed but text preserved)
+        PDFAssertions(pdf).assert_textline_exists(original_text)
 
 
 def test_text_line_edit_move_only():
@@ -87,8 +83,7 @@ def test_text_line_edit_move_only():
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
@@ -108,100 +103,105 @@ def test_text_line_edit_move_only():
 
 
 def test_text_line_edit_text_and_font():
-    """Test that text+font changes raise UnsupportedOperation"""
+    """Test text+font changes work together"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
-        # Text + Font changes should raise UnsupportedOperation
-        from pdfdancer.types import UnsupportedOperation
+        # Text + Font changes should work
+        with text_line.edit() as editor:
+            editor.replace("New Text Here")
+            editor.font("Helvetica", 16)
 
-        with pytest.raises(
-            UnsupportedOperation, match="Font and color changes are not supported"
-        ):
-            with text_line.edit() as editor:
-                editor.replace("New Text Here")
-                editor.font("Helvetica", 16)
+        # Verify the new text exists
+        (
+            PDFAssertions(pdf)
+            .assert_textline_exists("New Text Here")
+            .assert_textline_does_not_exist(
+                "This is regular Sans text showing alignment and styles."
+            )
+        )
 
 
 def test_text_line_edit_all_properties():
-    """Test that combined property changes raise UnsupportedOperation"""
+    """Test that combined property changes work (except line spacing)"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
-        # Combined changes including font/color should raise UnsupportedOperation
-        from pdfdancer.types import UnsupportedOperation
+        # Combined changes including font/color/position should work
+        with text_line.edit() as editor:
+            editor.replace("Fully Modified")
+            editor.font("Helvetica", 18)
+            editor.color(Color(255, 0, 0))
+            editor.move_to(100, 200)
 
-        with pytest.raises(
-            UnsupportedOperation, match="Font and color changes are not supported"
-        ):
-            with text_line.edit() as editor:
-                editor.replace("Fully Modified")
-                editor.font("Helvetica", 18)
-                editor.color(Color(255, 0, 0))
-                editor.move_to(100, 200)
+        # Verify the modified text exists
+        PDFAssertions(pdf).assert_textline_exists("Fully Modified")
 
 
-def test_text_line_edit_line_spacing_ignored():
-    """Test that line spacing is ignored for text lines with a warning"""
+def test_text_line_edit_line_spacing_fails():
+    """Test that line spacing changes fail hard for text lines"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
-        # Line spacing should be ignored with a warning
-        with text_line.edit() as editor:
-            editor.line_spacing(2.0)  # This should be ignored
-            editor.replace("Text with ignored spacing")
+        # Line spacing should raise UnsupportedOperation
+        from pdfdancer.types import UnsupportedOperation
 
-        # Verify the text was changed
-        PDFAssertions(pdf).assert_textline_exists("Text with ignored spacing")
+        with pytest.raises(
+                UnsupportedOperation,
+                match="Line spacing changes are not supported for individual text lines",
+        ):
+            with text_line.edit() as editor:
+                editor.line_spacing(2.0)
+                editor.replace("Text with spacing")
 
 
 def test_text_line_edit_chaining():
-    """Test that chained font/color changes raise UnsupportedOperation"""
+    """Test that chained font/color changes work"""
     base_url, token, pdf_path = _require_env_and_fixture("Showcase.pdf")
 
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
-        # Chained font/color changes should raise UnsupportedOperation
-        from pdfdancer.types import UnsupportedOperation
+        # Chained font/color changes should work
+        with text_line.edit() as editor:
+            editor.replace("Chained Edits").font("Helvetica", 15).color(
+                Color(128, 128, 128)
+            )
 
-        with pytest.raises(
-            UnsupportedOperation, match="Font and color changes are not supported"
-        ):
-            with text_line.edit() as editor:
-                editor.replace("Chained Edits").font("Helvetica", 15).color(
-                    Color(128, 128, 128)
-                )
+        # Verify the new text exists
+        (
+            PDFAssertions(pdf)
+            .assert_textline_exists("Chained Edits")
+            .assert_textline_does_not_exist(
+                "This is regular Sans text showing alignment and styles."
+            )
+        )
 
 
 def test_text_line_edit_with_exception_no_apply():
@@ -212,8 +212,7 @@ def test_text_line_edit_with_exception_no_apply():
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
@@ -238,8 +237,7 @@ def test_text_line_edit_multiple_sequential():
         text_lines = pdf.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
         with text_line.edit() as editor:
@@ -270,8 +268,7 @@ def test_text_line_edit_vs_manual_apply():
         text_lines = pdf1.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]
 
@@ -285,8 +282,7 @@ def test_text_line_edit_vs_manual_apply():
         text_lines = pdf2.page(0).select_text_lines_starting_with(
             "This is regular Sans text showing alignment and styles."
         )
-        if not text_lines:
-            pytest.skip("Required text line not found in test PDF")
+        assert len(text_lines) >= 1
 
         text_line = text_lines[0]