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

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

@@ -2,6 +2,77 @@
 
 All notable changes to `athena-python-pptx` are documented in this file.
 
+## 0.1.76
+
+**Table cell paragraph alignment now round-trips to OOXML (baseline-parity Gap A2).**
+
+`cell.text_frame.paragraphs[0].alignment = PP_ALIGN.X` previously stored
+the value locally for python-pptx parity but never reached the server.
+The setter now translates `PP_PARAGRAPH_ALIGNMENT` enum members (and the
+python-pptx lowercase string aliases) to the OOXML short form
+(`l` / `ctr` / `r` / `just` / `dist` / `thaiDist` / `justLow`) and emits
+a `SetTableCell` command carrying `paragraphAlignment`. The applier
+writes it to every paragraph in the cell's text body; the export worker
+emits a `SetTableCellText` intent whose paragraph alignment lands on
+`<a:pPr algn="…">` in the exported PPTX.
+
+Closes ~52 baseline-parity XML diffs against the 33 fixtures in
+`pptx-studio/tests/baseline-parity/`.
+
+## 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.76}/PKG-INFO

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

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

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.76}/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.76}/pptx/commands.py

@@ -476,6 +476,12 @@ class SetTableCell(Command):
         col: Column index (0-based)
         text: Cell text content
         fill_color_hex: Cell background color
+        paragraph_alignment: Horizontal alignment of the cell's text body.
+            Mirrors python-pptx ``cell.text_frame.paragraphs[0].alignment``.
+            Accepts the OOXML short forms ``l``, ``ctr``, ``r``, ``just``,
+            ``dist``, ``thaiDist``. Translated from ``PP_ALIGN`` /
+            ``PP_PARAGRAPH_ALIGNMENT`` by the SDK before emit. Closes
+            baseline-parity Gap A2 (~52 diffs across table fixtures).
     """
 
     shape_id: ShapeId
@@ -486,6 +492,7 @@ class SetTableCell(Command):
     font_size_centipoints: Optional[int] = None
     bold: Optional[bool] = None
     font_color_hex: Optional[str] = None
+    paragraph_alignment: Optional[str] = None
 
     @property
     def command_type(self) -> str:

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.76}/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:
@@ -56,6 +56,57 @@ _ELEMENT_TYPE_TO_MSO_SHAPE_TYPE: dict[str, MSO_SHAPE_TYPE] = {
 }
 
 
+# Map PP_PARAGRAPH_ALIGNMENT enum members and python-pptx string aliases
+# to OOXML <a:pPr algn="..."> short forms. Accepts the IntEnum int values
+# directly so callers can pass ``PP_ALIGN.CENTER`` without importing the
+# enum class. ``None`` clears the attribute.
+_PP_ALIGN_INT_TO_OOXML: dict[int, str] = {
+    1: "l",
+    2: "ctr",
+    3: "r",
+    4: "just",
+    5: "dist",
+    6: "thaiDist",
+    7: "justLow",
+}
+_PP_ALIGN_NAME_TO_OOXML: dict[str, str] = {
+    "left": "l",
+    "center": "ctr",
+    "right": "r",
+    "justify": "just",
+    "distribute": "dist",
+    "thai_distribute": "thaiDist",
+    "justify_low": "justLow",
+    # Already-short forms pass through.
+    "l": "l", "ctr": "ctr", "r": "r",
+    "just": "just", "dist": "dist", "thaiDist": "thaiDist", "justLow": "justLow",
+}
+
+
+def _normalize_pp_align(value: Any) -> Optional[str]:
+    """Translate a PP_ALIGN value to the OOXML short form used by SetTableCell.
+
+    Accepts ``PP_PARAGRAPH_ALIGNMENT`` (or any ``int``) and the lowercased
+    python-pptx string aliases used by ``pptx.text.PP_ALIGN``. Returns
+    ``None`` for ``None`` input so the setter can clear the attribute.
+    """
+    if value is None:
+        return None
+    if isinstance(value, int) and not isinstance(value, bool):
+        try:
+            return _PP_ALIGN_INT_TO_OOXML[int(value)]
+        except KeyError:
+            raise ValueError(f"Unsupported paragraph alignment value: {value!r}")
+    if isinstance(value, str):
+        try:
+            return _PP_ALIGN_NAME_TO_OOXML[value]
+        except KeyError:
+            raise ValueError(f"Unsupported paragraph alignment value: {value!r}")
+    raise TypeError(
+        f"alignment must be a PP_ALIGN enum, int, or string; got {type(value).__name__}"
+    )
+
+
 # Map XL_CHART_TYPE → (server chart_type, grouping). Only the chart types
 # author.ts supports today are listed here; anything else raises
 # UnsupportedFeatureError. Surface area mirrors python-pptx's accepted enum
@@ -2516,6 +2567,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 +2588,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 +5212,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 +5224,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 +5236,7 @@ class Shapes:
                 properties=elem.properties,
                 placeholder=elem.placeholder,
                 source=elem.source,
+                text_frame_properties=elem.text_frame_properties,
             )
         else:
             shape = Shape(
@@ -5193,6 +5249,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
@@ -5808,15 +5865,13 @@ class _TableCellParagraph:
     For API parity with python-pptx, ``add_run()`` and ``clear()`` are
     supported but operate on the single underlying run — multiple
     ``add_run()`` calls return the *same* ``_TableCellRun`` object, and
-    only the last text/style assignment wins. ``alignment`` is accepted
-    for compatibility but doesn't yet round-trip to OOXML (no per-cell
-    paragraph-alignment command on the applier).
+    only the last text/style assignment wins. ``alignment`` round-trips
+    to ``<a:pPr algn="...">`` on the cell's text body via SetTableCell.
     """
 
     def __init__(self, cell: "TableCell"):
         self._cell = cell
         self._runs = [_TableCellRun(cell)]
-        self._alignment: Optional[Any] = None
 
     @property
     def runs(self) -> list[_TableCellRun]:
@@ -5832,12 +5887,22 @@ class _TableCellParagraph:
 
     @property
     def alignment(self) -> Optional[Any]:
-        return self._alignment
+        """Return the cell's paragraph alignment as the original input value.
+
+        Stored on the parent cell so that text-body re-creation (which
+        rebuilds the paragraph wrapper) doesn't lose the assignment.
+        """
+        return self._cell._paragraph_alignment_input
 
     @alignment.setter
     def alignment(self, value: Any) -> None:
-        # Accepted for python-pptx parity; not yet round-tripped to OOXML.
-        self._alignment = value
+        # Translate PP_ALIGN enum / string to OOXML short form. Stored on
+        # the cell so we can re-emit it inside SetTableCell. Keep the
+        # original input around for the getter to return per python-pptx
+        # semantics (callers expect the same value type back).
+        self._cell._paragraph_alignment_input = value
+        self._cell._paragraph_alignment = _normalize_pp_align(value)
+        self._cell._emit_cell_change()
 
     def add_run(self) -> _TableCellRun:
         """Return the single underlying run.
@@ -5937,6 +6002,10 @@ class TableCell:
         self._margin_top = margin_top
         self._margin_bottom = margin_bottom
         self._vertical_anchor = vertical_anchor
+        # OOXML short form (``l``/``ctr``/``r``/...) sent over the wire and
+        # the caller's original input retained for the python-pptx getter.
+        self._paragraph_alignment: Optional[str] = None
+        self._paragraph_alignment_input: Any = None
 
     @property
     def text(self) -> str:
@@ -6312,6 +6381,7 @@ class TableCell:
                 font_size_centipoints=int(self._font_size_pt * 100) if self._font_size_pt is not None else None,
                 bold=self._bold,
                 font_color_hex=self._font_color_hex,
+                paragraph_alignment=self._paragraph_alignment,
             )
             self._table._buffer.add(cmd)
 

{athena_python_pptx-0.1.74 → athena_python_pptx-0.1.76}/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.76}/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.76}/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.76}/pyproject.toml

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