pdfdancer-client-python 0.2.3__tar.gz → 0.2.5__tar.gz

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/.gitignore

@@ -6,3 +6,4 @@ dist/
 src/pdfdancer_python.egg-info
 .aider*
 src/pdfdancer_client_python.egg-info
+/logs/

{pdfdancer_client_python-0.2.3/src/pdfdancer_client_python.egg-info → pdfdancer_client_python-0.2.5}/PKG-INFO

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

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/pyproject.toml

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

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/src/pdfdancer/pdfdancer_v1.py

@@ -138,6 +138,12 @@ class PDFDancer:
             env_token = os.getenv("PDFDANCER_TOKEN")
             resolved_token = env_token.strip() if env_token and env_token.strip() else None
 
+        if resolved_token is None:
+            raise ValidationException(
+                "Missing PDFDancer API token. Pass a token via the `token` argument "
+                "or set the PDFDANCER_TOKEN environment variable."
+            )
+
         env_base_url = os.getenv("PDFDANCER_BASE_URL")
         resolved_base_url = base_url or (env_base_url.strip() if env_base_url and env_base_url.strip() else None)
         if resolved_base_url is None:
@@ -259,6 +265,22 @@ class PDFDancer:
             # If JSON parsing fails, return response content or status
             return response.text or f"HTTP {response.status_code}"
 
+    def _handle_authentication_error(self, response: Optional[requests.Response]) -> None:
+        """
+        Translate authentication failures into a clear, actionable validation error.
+        """
+        if response is None:
+            return
+
+        if response.status_code in (401, 403):
+            details = self._extract_error_message(response)
+            raise ValidationException(
+                "Authentication with the PDFDancer API failed. "
+                "Confirm that your API token is valid, has not expired, and is supplied via "
+                "the `token` argument or the PDFDANCER_TOKEN environment variable. "
+                f"Server response: {details}"
+            )
+
     def _create_session(self) -> str:
         """
         Creates a new PDF processing session by uploading the PDF data.
@@ -274,6 +296,7 @@ class PDFDancer:
                 timeout=self._read_timeout if self._read_timeout > 0 else None
             )
 
+            self._handle_authentication_error(response)
             response.raise_for_status()
             session_id = response.text.strip()
 
@@ -283,6 +306,7 @@ class PDFDancer:
             return session_id
 
         except requests.exceptions.RequestException as e:
+            self._handle_authentication_error(getattr(e, 'response', None))
             error_message = self._extract_error_message(getattr(e, 'response', None))
             raise HttpClientException(f"Failed to create session: {error_message}",
                                       response=getattr(e, 'response', None), cause=e) from None
@@ -316,10 +340,12 @@ class PDFDancer:
                 except (json.JSONDecodeError, KeyError):
                     pass
 
+            self._handle_authentication_error(response)
             response.raise_for_status()
             return response
 
         except requests.exceptions.RequestException as e:
+            self._handle_authentication_error(getattr(e, 'response', None))
             error_message = self._extract_error_message(getattr(e, 'response', None))
             raise HttpClientException(f"API request failed: {error_message}", response=getattr(e, 'response', None),
                                       cause=e) from None

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/src/pdfdancer/types.py

@@ -31,21 +31,6 @@ class PDFObjectBase:
         self.internal_id = internal_id
         self.object_type = object_type
 
-    # --------------------------------------------------------------
-    # Core properties
-    # --------------------------------------------------------------
-    def internal_id(self) -> str:
-        """Internal PDFDancer object identifier, e.g. 'PATH_000023'."""
-        return self.internal_id
-
-    def type(self) -> ObjectType:
-        """Enum value representing the PDF object type."""
-        return self.object_type
-
-    def position(self) -> Position:
-        """The geometric position of the object on its page."""
-        return self.position
-
     @property
     def page_index(self) -> int:
         """Page index where this object resides."""

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5/src/pdfdancer_client_python.egg-info}/PKG-INFO

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

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/src/pdfdancer_client_python.egg-info/SOURCES.txt

@@ -20,6 +20,7 @@ src/pdfdancer_client_python.egg-info/dependency_links.txt
 src/pdfdancer_client_python.egg-info/requires.txt
 src/pdfdancer_client_python.egg-info/top_level.txt
 tests/__init__.py
+tests/test_authentication.py
 tests/test_models.py
 tests/test_openapi_compliance.py
 tests/e2e/__init__.py

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/tests/e2e/__init__.py

@@ -6,7 +6,7 @@ import requests
 
 
 def _get_base_url():
-    return os.getenv('PDFDANCER_BASE_URL', 'http://localhost:8080')
+    return os.getenv('PDFDANCER_BASE_URL', 'https://api.pdfdancer.com')
 
 
 def _read_token() -> str | None:

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/tests/e2e/test_acroform.py

@@ -9,9 +9,9 @@ def test_find_form_fields():
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         form_fields = pdf.select_form_fields()
         assert len(form_fields) == 10
-        assert form_fields[0].type() == ObjectType.TEXT_FIELD
-        assert form_fields[4].type() == ObjectType.CHECK_BOX
-        assert form_fields[6].type() == ObjectType.RADIO_BUTTON
+        assert form_fields[0].object_type == ObjectType.TEXT_FIELD
+        assert form_fields[4].object_type == ObjectType.CHECK_BOX
+        assert form_fields[6].object_type == ObjectType.RADIO_BUTTON
 
         # Verify not all fields at origin
         all_at_origin = all(
@@ -26,7 +26,7 @@ def test_find_form_fields():
         first_form = pdf.page(0).select_form_fields_at(290, 460)
         assert len(first_form) == 1
         f = first_form[0]
-        assert f.type() == ObjectType.RADIO_BUTTON
+        assert f.object_type == ObjectType.RADIO_BUTTON
         assert f.internal_id == "FORM_FIELD_000008"
 
 
@@ -73,7 +73,7 @@ def test_edit_form_fields():
         f = fields[0]
         assert f.name == "firstName"
         assert f.value is None
-        assert f.type() == ObjectType.TEXT_FIELD
+        assert f.object_type == ObjectType.TEXT_FIELD
         assert f.internal_id == "FORM_FIELD_000001"
 
         f.edit().value("Donald Duck").apply()

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/tests/e2e/test_form_x_objects.py

@@ -11,7 +11,7 @@ def test_delete_form(tmp_path: Path):
     with PDFDancer.open(pdf_path, token=token, base_url=base_url) as pdf:
         forms = pdf.select_forms()
         assert len(forms) == 17
-        assert forms[0].type() == ObjectType.FORM_X_OBJECT
+        assert forms[0].object_type == ObjectType.FORM_X_OBJECT
 
         # Delete all form XObjects
         for form in forms:

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/tests/e2e/test_image.py

@@ -13,7 +13,7 @@ def test_find_images():
     with PDFDancer.open(pdf_path, token=token, base_url=base_url, timeout=30.0) as pdf:
         images = pdf.select_images()
         assert len(images) == 3
-        assert images[0].type() == ObjectType.IMAGE
+        assert images[0].object_type == ObjectType.IMAGE
 
         images_page0 = pdf.page(0).select_images()
         assert len(images_page0) == 2

{pdfdancer_client_python-0.2.3 → pdfdancer_client_python-0.2.5}/tests/e2e/test_page.py

@@ -9,7 +9,7 @@ def test_get_pages():
         pages = pdf.pages()
         assert pages is not None
         assert len(pages) == 12
-        assert pages[0].type() == ObjectType.PAGE
+        assert pages[0].object_type == ObjectType.PAGE
 
 
 def test_get_page():

pdfdancer_client_python-0.2.5/tests/test_authentication.py

@@ -0,0 +1,36 @@
+import requests
+import pytest
+
+from pdfdancer import ValidationException
+from pdfdancer.pdfdancer_v1 import PDFDancer
+
+
+def test_open_without_token_reports_actionable_message():
+    with pytest.raises(ValidationException) as exc_info:
+        PDFDancer.open(pdf_data=b"%PDF", token="")
+
+    message = str(exc_info.value)
+    assert "Missing PDFDancer API token" in message
+    assert "PDFDANCER_TOKEN" in message
+
+
+def test_create_session_unauthorized_reports_guidance(monkeypatch):
+    class FakeResponse:
+        status_code = 401
+        text = "Unauthorized"
+
+        def json(self):
+            return {"message": "Unauthorized"}
+
+    def fake_post(self, *args, **kwargs):
+        return FakeResponse()
+
+    monkeypatch.setattr(requests.Session, "post", fake_post)
+
+    with pytest.raises(ValidationException) as exc_info:
+        PDFDancer(token="bad-token", pdf_data=b"%PDF", base_url="https://api.example.com")
+
+    message = str(exc_info.value)
+    assert "Authentication with the PDFDancer API failed" in message
+    assert "Server response: Unauthorized" in message
+    assert "PDFDANCER_TOKEN" in message