athena-python-pptx 0.1.74__tar.gz → 0.1.75__tar.gz

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/CHANGELOG.md

@@ -2,6 +2,60 @@
 
 All notable changes to `athena-python-pptx` are documented in this file.
 
+## 0.1.75
+
+**Critical bug fix — invisible text from `font.size = Pt(X)`.**
+
+A user surfaced that agent-generated decks rendered as blank
+rectangles in the Olympus editor. Investigation traced the symptom to
+`fontSizePt: 355600` (= 28 × 12700) landing on the wire instead of
+`fontSizePt: 28` — making the renderer treat text as 355,600-point
+glyphs and rendering them effectively invisible.
+
+### Root cause
+
+`Length.__truediv__` returns a `Length` subclass — e.g.
+`Pt(28) / EMU_PER_PT` returns a `Pt` instance with raw int value `28`
+(meaning 28 EMU, semantically "28 points worth"). The SDK's font-size
+serializer relies on this to convert EMU → points:
+
+```python
+style["fontSizePt"] = self._size / EMU_PER_PT  # Pt(355600) / 12700
+```
+
+The trouble: `Command.to_dict()` calls `dataclasses.asdict()` which
+**deep-copies all nested values**. `copy.deepcopy(Pt(28))` reconstructs
+via `Pt.__new__(28)`, and `Pt.__new__` interprets its argument as
+*points* (not EMU) and multiplies by 12700. So a Pt instance whose
+internal int was 28 EMU came back out as 28 × 12700 = 355600 EMU.
+
+By the time the JSON went over the wire, `fontSizePt` was 355600.
+Server stored it, renderer rendered text at 355,600pt → invisible.
+
+### Fix
+
+Override `Length.__reduce_ex__` so `copy.deepcopy` (and `pickle`)
+reconstruct via the new module-level `_length_reconstruct`, which uses
+`Length._from_emu` (preserves raw EMU value). Three lines in
+`pptx/units.py`; behaviour-preserving for every existing call site.
+
+### Regression test
+
+New `test_length_round_trips_through_deepcopy` in `tests/test_units.py`
+asserts that `copy.deepcopy(Pt(28))` and `asdict({...Pt(28) / 12700...})`
+both preserve the raw EMU value. Would have caught this bug on day one.
+
+### Impact
+
+Every agent-authored deck where the agent set `run.font.size = Pt(X)`
+through the SDK was silently shipping `fontSizePt: X * 12700` to the
+server. Existing decks may need to be regenerated by the agent or
+manually corrected. Future agent runs against v0.1.75+ will render
+text correctly.
+
+The new SDK was published to PyPI as 0.1.75 and the Daytona
+`presentation-exec` snapshot bumped to v38.
+
 ## 0.1.74
 
 **True drop-in import-path parity.** A behavioural-fidelity audit found

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: athena-python-pptx
-Version: 0.1.74
+Version: 0.1.75
 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.1.74 → athena_python_pptx-0.1.75}/pptx/__init__.py

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

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pptx/client.py

@@ -542,6 +542,7 @@ class Client:
             MasterSnapshot,
             PlaceholderSnapshot,
             SlideSnapshot,
+            TextFramePropertiesSnapshot,
             ThemeHierarchySnapshot,
             Transform,
         )
@@ -577,6 +578,21 @@ class Client:
                     local_transform_override=placeholder_data.get("localTransformOverride"),
                 )
 
+            # Server stores margins under `marginLeftEmu` after any SDK update
+            # and under `marginLeft` immediately after ingest. Read both.
+            tfp_data = elem_data.get("textFrameProperties")
+            text_frame_properties: Optional[TextFramePropertiesSnapshot] = None
+            if isinstance(tfp_data, dict):
+                text_frame_properties = TextFramePropertiesSnapshot(
+                    word_wrap=tfp_data.get("wordWrap"),
+                    auto_size=tfp_data.get("autoSize"),
+                    vertical_anchor=tfp_data.get("verticalAnchor"),
+                    margin_left=tfp_data.get("marginLeftEmu", tfp_data.get("marginLeft")),
+                    margin_right=tfp_data.get("marginRightEmu", tfp_data.get("marginRight")),
+                    margin_top=tfp_data.get("marginTopEmu", tfp_data.get("marginTop")),
+                    margin_bottom=tfp_data.get("marginBottomEmu", tfp_data.get("marginBottom")),
+                )
+
             elements[elem_id] = ElementSnapshot(
                 id=elem_data["id"],
                 type=elem_data["type"],
@@ -596,6 +612,7 @@ class Client:
                 source=elem_data.get("source"),
                 table_data=elem_data.get("tableData"),
                 asset_id=elem_data.get("assetId"),
+                text_frame_properties=text_frame_properties,
             )
 
         theme_hierarchy_raw = data.get("themeHierarchy")

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pptx/shapes/__init__.py

@@ -26,7 +26,7 @@ from ..chart.category import Categories
 from ..dml.color import RGBColor, ColorFormat
 from ..errors import UnsupportedFeatureError
 from ..text import TextFrame
-from ..typing import ElementSnapshot, ShapeId, Transform, PlaceholderSnapshot, PP_PLACEHOLDER
+from ..typing import ElementSnapshot, ShapeId, Transform, PlaceholderSnapshot, PP_PLACEHOLDER, TextFramePropertiesSnapshot
 from ..units import Emu, Length, ensure_emu
 
 if TYPE_CHECKING:
@@ -2516,6 +2516,7 @@ class Shape:
         properties: Optional[dict[str, Any]] = None,
         placeholder: Optional[PlaceholderSnapshot] = None,
         source: Optional[str] = None,
+        text_frame_properties: Optional[TextFramePropertiesSnapshot] = None,
     ):
         self._shape_id = shape_id
         self._slide = slide
@@ -2536,6 +2537,7 @@ class Shape:
                 buffer=buffer,
                 preview_text=preview_text,
                 rich_content=rich_content,
+                text_frame_properties=text_frame_properties,
             )
 
     # Known misspelled/non-existent attributes with specific fix guidance
@@ -5159,6 +5161,7 @@ class Shapes:
                 properties=elem.properties,
                 placeholder=elem.placeholder,
                 source=elem.source,
+                text_frame_properties=elem.text_frame_properties,
             )
         elif elem.type == "connector":
             shape = Connector(
@@ -5170,6 +5173,7 @@ class Shapes:
                 properties=elem.properties,
                 placeholder=elem.placeholder,
                 source=elem.source,
+                text_frame_properties=elem.text_frame_properties,
             )
         elif elem.type == "group":
             shape = GroupShape(
@@ -5181,6 +5185,7 @@ class Shapes:
                 properties=elem.properties,
                 placeholder=elem.placeholder,
                 source=elem.source,
+                text_frame_properties=elem.text_frame_properties,
             )
         else:
             shape = Shape(
@@ -5193,6 +5198,7 @@ class Shapes:
                 properties=elem.properties,
                 placeholder=elem.placeholder,
                 source=elem.source,
+                text_frame_properties=elem.text_frame_properties,
             )
         self._shapes.append(shape)
         self._shapes_by_id[elem.id] = shape

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pptx/text/__init__.py

@@ -10,7 +10,7 @@ from typing import TYPE_CHECKING, Any, Iterator, Optional, Union
 from ..commands import SetText, SetRunStyle, SetParagraphStyle, SetTextFrameProperties, FitText
 from ..dml.color import RGBColor, ColorFormat
 from ..errors import UnsupportedFeatureError
-from ..typing import ShapeId, TextStyle
+from ..typing import ShapeId, TextStyle, TextFramePropertiesSnapshot
 
 if TYPE_CHECKING:
     from ..batching import CommandBuffer
@@ -1328,12 +1328,36 @@ class TextFrame:
         buffer: Optional[CommandBuffer],
         preview_text: Optional[str] = None,
         rich_content: Optional[dict[str, Any]] = None,
+        text_frame_properties: Optional[TextFramePropertiesSnapshot] = None,
     ):
         self._shape_id = shape_id
         self._buffer = buffer
         self._preview_text = preview_text or ""
         self._paragraphs: list[Paragraph] = []
 
+        # Seed internal text-frame-properties state from the source snapshot so
+        # subsequent setter emits (which always rebroadcast the full state)
+        # preserve the props the user hasn't touched. Without this, setting
+        # `text_frame.margin_left = 0` would emit a SetTextFrameProperties
+        # command without `autoSize`, the server's existing autoSize would be
+        # the only thing keeping the box auto-fit — and it isn't there for
+        # shapes created via AddTextBox, so the box renders without auto-fit.
+        if text_frame_properties is not None:
+            if text_frame_properties.word_wrap is not None:
+                self._word_wrap = text_frame_properties.word_wrap
+            if text_frame_properties.auto_size is not None:
+                self._auto_size = text_frame_properties.auto_size
+            if text_frame_properties.vertical_anchor is not None:
+                self._vertical_anchor = text_frame_properties.vertical_anchor
+            if text_frame_properties.margin_left is not None:
+                self._margin_left = text_frame_properties.margin_left
+            if text_frame_properties.margin_right is not None:
+                self._margin_right = text_frame_properties.margin_right
+            if text_frame_properties.margin_top is not None:
+                self._margin_top = text_frame_properties.margin_top
+            if text_frame_properties.margin_bottom is not None:
+                self._margin_bottom = text_frame_properties.margin_bottom
+
         # Initialize paragraphs from rich content or plain text
         if rich_content and rich_content.get("paragraphs"):
             for i, para_data in enumerate(rich_content["paragraphs"]):

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pptx/typing.py

@@ -162,6 +162,26 @@ class PlaceholderSnapshot:
     local_transform_override: Optional[bool] = None
 
 
+@dataclass
+class TextFramePropertiesSnapshot:
+    """Snapshot of a text frame's structural properties (auto-size, margins, etc.).
+
+    Carries the source PPTX's <a:bodyPr> state through `from_url` so the SDK's
+    TextFrame can populate its internal `_auto_size`/`_word_wrap`/`_margin_*`
+    fields. Without this, a user setter (`text_frame.margin_left = 0`) emits a
+    SetTextFrameProperties command that omits the un-touched props, the server
+    keeps whatever it has, and round-trip is lossy.
+    """
+
+    word_wrap: Optional[bool] = None
+    auto_size: Optional[Literal["none", "shape_to_fit_text", "text_to_fit_shape"]] = None
+    vertical_anchor: Optional[Literal["top", "middle", "bottom"]] = None
+    margin_left: Optional[int] = None
+    margin_right: Optional[int] = None
+    margin_top: Optional[int] = None
+    margin_bottom: Optional[int] = None
+
+
 @dataclass
 class ElementSnapshot:
     """Snapshot of an element's state."""
@@ -176,6 +196,7 @@ class ElementSnapshot:
     source: Optional[Literal["ingested", "sdk", "layout", "master"]] = None
     table_data: Optional[dict[str, Any]] = None
     asset_id: Optional[str] = None
+    text_frame_properties: Optional[TextFramePropertiesSnapshot] = None
 
 
 @dataclass

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pptx/units.py

@@ -31,6 +31,17 @@ EMU_PER_CENTIPOINT = 127
 EMU_PER_PX = 9525  # At 96 DPI
 
 
+def _length_reconstruct(cls: type, emu_value: int):  # noqa: ANN201
+    """Top-level reconstructor for :class:`Length` subclasses.
+
+    Used by :meth:`Length.__reduce_ex__` so that ``copy.deepcopy`` (and
+    ``pickle``) round-trip a :class:`Length` subclass without going
+    through the unit-converting ``__new__``. Must be a module-level
+    function so pickle can locate it by qualified name.
+    """
+    return cls._from_emu(emu_value)
+
+
 class Length(int):
     """
     Length expressed in English Metric Units (EMU) — base unit class.
@@ -45,6 +56,15 @@ class Length(int):
     def __new__(cls, value: Union[int, float, "Length"]) -> "Length":
         return super().__new__(cls, int(round(value)))
 
+    # ``copy.deepcopy`` (used by ``dataclasses.asdict``) reconstructs ints
+    # by invoking ``Type(int_value)``. For our subclasses that means
+    # ``Pt(355600)`` etc., which interpret the value as user-units and
+    # multiply by the unit's EMU factor → ``355600 → 4,516,120,000``.
+    # Override the pickle/reduce protocol so deep-copy round-trips via
+    # ``_from_emu`` (treats the value as raw EMU), preserving identity.
+    def __reduce_ex__(self, protocol: int):  # type: ignore[override]
+        return (_length_reconstruct, (type(self), int(self)))
+
     @property
     def inches(self) -> float:
         """Return value in inches."""

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.75}/pyproject.toml

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