athena-python-pptx 0.3.1__tar.gz → 0.4.0__tar.gz

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: athena-python-pptx
-Version: 0.3.1
+Version: 0.4.0
 Summary: Drop-in replacement for python-pptx that connects to PPTX Studio for real-time collaboration
 Project-URL: Homepage, https://github.com/pptx-studio/python-sdk
 Project-URL: Documentation, https://docs.pptx-studio.com/sdk/python

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/pptx/__init__.py

@@ -132,7 +132,7 @@ def flush_all() -> None:
     _active_buffers[:] = alive
 
 
-__version__ = "0.3.1"
+__version__ = "0.4.0"
 
 __all__ = [
     # Main entry point

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/pptx/client.py

@@ -72,7 +72,11 @@ class Client:
             base_url: Base URL of the API (e.g., "https://api.pptx-studio.com").
                      If not provided, uses ATHENA_PPTX_BASE_URL environment variable.
             api_key: Optional API key for authentication.
-                    If not provided, uses ATHENA_PPTX_API_KEY environment variable.
+                    If not provided, falls back to the ATHENA_PPTX_API_KEY
+                    environment variable, then to ATHENA_API_KEY (the canonical
+                    Athena user API key — same value, accepted as a fallback so
+                    code running in sandboxes that only inject ATHENA_API_KEY
+                    still authenticates).
             timeout: Request timeout in seconds
 
         Raises:
@@ -83,11 +87,13 @@ class Client:
             base_url = os.environ.get("ATHENA_PPTX_BASE_URL")
             if base_url is None:
                 raise ValueError(
-                    "base_url must be provided or ATHENA_PPTX_BASE_URL environment variable must be set"
+                    "base_url must be provided or the ATHENA_PPTX_BASE_URL "
+                    "environment variable must be set. Example: "
+                    'ATHENA_PPTX_BASE_URL="https://pptx-studio.prd.athenaintel.com".'
                 )
 
         if api_key is None:
-            api_key = os.environ.get("ATHENA_PPTX_API_KEY")
+            api_key = os.environ.get("ATHENA_PPTX_API_KEY") or os.environ.get("ATHENA_API_KEY")
 
         self.base_url = base_url.rstrip("/")
         self.api_key = api_key
@@ -129,7 +135,18 @@ class Client:
     def _handle_response(self, response: requests.Response) -> Any:
         """Handle API response, raising appropriate errors."""
         if response.status_code == 401:
-            raise AuthenticationError("Invalid or expired API key")
+            if not self.api_key:
+                raise AuthenticationError(
+                    "No API key was sent. Pass api_key= to Presentation(...) "
+                    "or set ATHENA_PPTX_API_KEY (or ATHENA_API_KEY as a "
+                    "fallback) in the environment."
+                )
+            raise AuthenticationError(
+                "Invalid or expired API key. The pptx-studio API rejected the "
+                "credentials. Verify ATHENA_PPTX_API_KEY (or the ATHENA_API_KEY "
+                "fallback) is set to a valid Athena user API key for the "
+                "current workspace."
+            )
 
         if response.status_code == 409:
             data = response.json() if response.text else {}
@@ -416,7 +433,7 @@ class Client:
         if name is None:
             name = os.path.basename(file_path)
             # Remove extension
-            if name.endswith('.pptx') or name.endswith('.potx'):
+            if name.endswith(".pptx") or name.endswith(".potx"):
                 name = name[:-5]
 
         # Step 1: Create deck to get presigned URL
@@ -496,7 +513,7 @@ class Client:
         # Use filename as name if not provided
         if name is None:
             name = os.path.basename(file_path)
-            if name.endswith('.pptx') or name.endswith('.potx'):
+            if name.endswith(".pptx") or name.endswith(".potx"):
                 name = name[:-5]
 
         # Step 1: Create deck to get presigned URL
@@ -800,9 +817,7 @@ class Client:
                 return self._download(download_url)
 
             if status["status"] in ("failed", "error"):
-                raise ExportError(
-                    status.get("error", "Export failed"), job_id
-                )
+                raise ExportError(status.get("error", "Export failed"), job_id)
 
             time.sleep(poll_interval)
 
@@ -870,9 +885,7 @@ class Client:
                 return self._download(image_url)
 
             if status["status"] in ("failed", "error"):
-                raise RenderError(
-                    status.get("error", "Render failed"), job_id
-                )
+                raise RenderError(status.get("error", "Render failed"), job_id)
 
             time.sleep(poll_interval)
 

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/pptx/commands.py

@@ -1464,6 +1464,23 @@ class SetGradientFill(Command):
                         "stops"
                     )
 
+    def to_dict(self) -> dict[str, Any]:
+        """Serialize, camelCasing each stop's ``color_hex`` to ``colorHex`` for the server schema."""
+        result = super().to_dict()
+        if self.stops is not None:
+            camel_stops: list[dict[str, Any]] = []
+            for stop in self.stops:
+                camel: dict[str, Any] = {
+                    "position": stop.get("position"),
+                    "colorHex": stop.get("color_hex"),
+                }
+                transparency = stop.get("transparency")
+                if transparency is not None:
+                    camel["transparency"] = transparency
+                camel_stops.append(camel)
+            result["stops"] = camel_stops
+        return result
+
 
 @dataclass
 class SetShapeZOrder(Command):

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/pptx/shapes/__init__.py

@@ -2963,6 +2963,43 @@ class _ChartPlotCollection:
 # (rather than in their own file) so they're visible from the chart
 # adapters above.
 
+# python-pptx ``XL_DATA_LABEL_POSITION`` (IntEnum) -> the OOXML ``dLblPos``
+# string symbols the server accepts. Assigning the enum member directly
+# (``data_labels.position = XL_LABEL_POSITION.OUTSIDE_END``) must serialize to
+# ``'outEnd'`` rather than the raw int ``2``, otherwise the commands endpoint
+# rejects the patch with a 400. The accepted set is mirrored from
+# ``packages/chart-core/src/schema.ts`` (``'ctr'|'inEnd'|'inBase'|'outEnd'|
+# 'bestFit'|'t'|'b'|'l'|'r'``). Mirrors the ``Legend._POSITION_INT_TO_STR``
+# pattern.
+_DATA_LABEL_POSITION_INT_TO_STR = {
+    -4108: 'ctr',     # CENTER
+    3: 'inEnd',       # INSIDE_END
+    4: 'inBase',      # INSIDE_BASE
+    2: 'outEnd',      # OUTSIDE_END
+    5: 'bestFit',     # BEST_FIT
+    0: 't',           # ABOVE
+    1: 'b',           # BELOW
+    -4131: 'l',       # LEFT
+    -4152: 'r',       # RIGHT
+}
+
+
+def _coerce_data_label_position(value: Any) -> Optional[str]:
+    """Normalize a data-label position to its OOXML ``dLblPos`` string.
+
+    Accepts an ``XL_DATA_LABEL_POSITION`` member (or raw int), in which case
+    it maps to the OOXML symbol, or an already-valid string, which passes
+    through unchanged. Unmappable ints resolve to ``None`` so no invalid
+    patch is emitted (matching ``Legend.position``'s behavior)."""
+    if value is None:
+        return None
+    if isinstance(value, bool):
+        return None
+    if isinstance(value, int):
+        return _DATA_LABEL_POSITION_INT_TO_STR.get(int(value))
+    return str(value)
+
+
 class DataLabel:
     """python-pptx parity for a single data label on a chart series.
 
@@ -3047,9 +3084,10 @@ class DataLabel:
 
     @position.setter
     def position(self, value: Optional[str]) -> None:
-        self._position = value
-        if value is not None:
-            self._emit(position=str(value))
+        coerced = _coerce_data_label_position(value)
+        self._position = coerced
+        if coerced is not None:
+            self._emit(position=coerced)
 
     @property
     @athena_extension(
@@ -3357,9 +3395,10 @@ class DataLabels:
 
     @position.setter
     def position(self, value: Optional[str]) -> None:
-        self._position = value
-        if value is not None:
-            self._emit_style(position=value)
+        coerced = _coerce_data_label_position(value)
+        self._position = coerced
+        if coerced is not None:
+            self._emit_style(position=coerced)
 
     @property
     def show_category_name(self) -> bool:

{athena_python_pptx-0.3.1 → athena_python_pptx-0.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "athena-python-pptx"
-version = "0.3.1"
+version = "0.4.0"
 description = "Drop-in replacement for python-pptx that connects to PPTX Studio for real-time collaboration"
 readme = "README.md"
 license = "MIT"