athena-python-docx 0.15.4__tar.gz → 0.16.0__tar.gz

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: athena-python-docx
-Version: 0.15.4
+Version: 0.16.0
 Summary: Drop-in replacement for python-docx that connects to Athena's Superdoc/Keryx collaborative document stack
 Project-URL: Homepage, https://athenaintelligence.ai
 Author-email: Athena Intelligence <engineering@athenaintelligence.ai>

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/docx/__init__.py

@@ -6,7 +6,7 @@ See CLAUDE.md for the API parity contract.
 
 from __future__ import annotations
 
-__version__ = "0.15.4"
+__version__ = "0.16.0"
 
 from docx.api import Document
 from docx._buffer import flush_all

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/docx/_buffer.py

@@ -21,9 +21,9 @@ best-effort coalesce — if the loop thread races us we just flush twice
 (or zero times; the next loop-thread call drains it).
 
 `flush_all` is a process-wide hook used by the Daytona sandbox prelude to
-make sure pending writes hit Keryx and server-side edit sessions are
-checkpointed before the sandbox is suspended. It walks a weak-ref registry
-so dead Document instances don't keep their buffers alive.
+make sure pending writes hit Keryx before the sandbox is suspended. It walks
+a weak-ref registry so dead Workbook instances don't keep their buffers
+alive.
 """
 
 from __future__ import annotations
@@ -124,10 +124,10 @@ if TYPE_CHECKING:
 # HTTP POST to agora (see ``_ptc._send``), so emitting one card per low-level
 # mutation produces hundreds of sub-cards per script and pays full network
 # RTT on every one. The allow-list keeps the per-action signal — one card
-# per logical paragraph creation — without the spam or the cumulative
-# latency. Asset-level creation events (``CreateDocument``) are emitted
-# from ``Document.create`` directly, not through this buffer path.
-_PTC_EMIT_TOOLS: frozenset[str] = frozenset({"CreateParagraph"})
+# per logical block creation (paragraph, heading, table) — without the spam
+# or the cumulative latency. Asset-level creation events (``CreateDocument``)
+# are emitted from ``Document.create`` directly, not through this buffer path.
+_PTC_EMIT_TOOLS: frozenset[str] = frozenset({"CreateParagraph", "CreateHeading", "CreateTable"})
 
 
 def _ptc_emit_end_batch(cmds: list[Command], *, is_error: bool) -> None:
@@ -180,12 +180,11 @@ def _unregister(buffer: "CommandBuffer") -> None:
 
 
 def flush_all(*, strict: bool = False) -> None:
-    """Flush and checkpoint every live CommandBuffer in this process.
+    """Flush every live CommandBuffer in this process.
 
     Used by the Daytona sandbox prelude after user code returns, so
-    buffered mutations make it to Keryx and server-side edit sessions are
-    durably released before the sandbox is suspended. Safe to call when no
-    Buffers exist (no-op).
+    buffered mutations make it to Keryx before the sandbox is suspended.
+    Safe to call when no Buffers exist (no-op).
 
     By default, failures are logged and swallowed for backwards
     compatibility with existing local scripts. The executor calls
@@ -217,9 +216,9 @@ def flush_all(*, strict: bool = False) -> None:
                     sys.stderr.write(f"[docx-sdk] flush_all: {msg}\n")
             continue
         try:
-            buf.commit()
+            buf.flush()
         except Exception as e:  # noqa: BLE001
-            msg = f"buffer {buf.asset_id} flush/commit failed: {e}"
+            msg = f"buffer {buf.asset_id} flush failed: {e}"
             if strict:
                 failures.append(msg)
             else:
@@ -500,20 +499,6 @@ class CommandBuffer:
         _apply_proxy_id_rewrites(results, proxy_id_refs)
         return results
 
-    def commit(self) -> None:
-        """Flush pending commands and release the server-side edit session.
-
-        The HTTP command response tells us docx-studio applied the batch to its
-        pooled SuperDoc handle. Releasing the handle closes the collaboration
-        session without discard on the server, which is the durability
-        checkpoint for asset-backed document edits.
-        """
-        self._assert_current_execution()
-        self.flush()
-        release_asset = getattr(self._client, "release_asset", None)
-        if callable(release_asset):
-            release_asset(self._asset_id)
-
     def close(self) -> None:
         """Flush and disable. Idempotent."""
         if self._closed:
@@ -528,7 +513,7 @@ class CommandBuffer:
                     self._proxy_id_refs = {}
                 _ptc_emit_end_batch(pending, is_error=True)
             else:
-                self.commit()
+                self.flush()
         finally:
             self._closed = True
             _unregister(self)

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/docx/_http_doc.py

@@ -699,24 +699,6 @@ class HttpClient:
         except Exception:  # noqa: BLE001
             pass
 
-    def release_asset(self, asset_id: str) -> None:
-        """Ask docx-studio to close its pooled SuperDoc session for ``asset_id``.
-
-        Command POSTs only prove that the server accepted and applied the
-        mutation batch to its live SuperDoc handle. The durable checkpoint for
-        collaborative sessions happens when that handle is closed without
-        discard, so ``Document.save()`` and the Daytona ``flush_all`` cleanup
-        call this endpoint after draining the command buffer.
-        """
-        url: str = f"{self._base_url}/docs/{asset_id}/session/release"
-        _http_post_json(
-            session=self._session,
-            url=url,
-            api_key=self._api_key,
-            body={},
-            timeout=30.0,
-        )
-
     def execute_batch(
         self,
         asset_id: str,

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/docx/client.py

@@ -231,8 +231,8 @@ class Session:
         handle = self._doc_handle
         buffer = getattr(handle, "buffer", None) if handle is not None else None
         if buffer is not None:
-            buffer.commit()
-        _log_info(f"Saved {self._asset_id} (buffer drained and session released)")
+            buffer.flush()
+        _log_info(f"Saved {self._asset_id} (buffer drained)")
 
     async def close(self) -> None:
         """Close the session. Idempotent."""

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "athena-python-docx"
-version = "0.15.4"
+version = "0.16.0"
 description = "Drop-in replacement for python-docx that connects to Athena's Superdoc/Keryx collaborative document stack"
 readme = "README.md"
 license = "MIT"

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/tests/test_buffer.py

@@ -7,9 +7,8 @@ Verifies that:
 - Creates WITH a client id (the transparent-batching path) defer,
 - pending mutations drain in the same batch as an eager call,
 - ``flush()`` drains explicitly,
-- ``commit()`` drains and releases the server-side session,
 - the idle timer auto-flushes after a configurable window,
-- the process-wide ``flush_all`` walks every live buffer and commits it.
+- the process-wide ``flush_all`` walks every live buffer.
 """
 
 from __future__ import annotations
@@ -45,7 +44,6 @@ class _FakeClient:
         # Envelope-level fields recorded per batch — used by
         # track-changes tests to verify changeMode / user propagation.
         self.envelopes: list[dict[str, Any]] = []
-        self.releases: list[str] = []
         self._lock = threading.Lock()
 
     def execute_batch(
@@ -64,10 +62,6 @@ class _FakeClient:
         # Mimic per-command result envelope.
         return [{"index": i, "ok": True} for i, _ in enumerate(commands)]
 
-    def release_asset(self, asset_id: str) -> None:
-        with self._lock:
-            self.releases.append(asset_id)
-
 
 class _FakeDocHandle:
     def __init__(self, buffer: CommandBuffer) -> None:
@@ -195,19 +189,6 @@ def test_flush_with_empty_queue_is_noop() -> None:
     buf.close()
 
 
-def test_commit_drains_pending_and_releases_server_session() -> None:
-    fake = _FakeClient()
-    buf = CommandBuffer(fake, "asset_1", auto_flush_seconds=0.0)
-
-    buf.call(FormatApply(target={"kind": "block"}, inline={"bold": True}))
-    buf.commit()
-
-    assert len(fake.batches) == 1
-    assert _types(fake.batches[0]) == ["FormatApply"]
-    assert fake.releases == ["asset_1"]
-    buf.close()
-
-
 def test_idle_timer_auto_flushes() -> None:
     """A non-zero idle window should flush after inactivity.
 
@@ -301,8 +282,6 @@ def test_flush_all_walks_every_live_buffer() -> None:
     flush_all()
     assert len(fake_a.batches) == 1
     assert len(fake_b.batches) == 1
-    assert fake_a.releases == ["asset_a"]
-    assert fake_b.releases == ["asset_b"]
     buf_a.close()
     buf_b.close()
 

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/tests/test_http_transport.py

@@ -145,32 +145,6 @@ def test_table_set_cell_batch_uses_long_running_http_timeout() -> None:
     assert captured["timeout"] == LONG_RUNNING_COMMAND_TIMEOUT_SECONDS
 
 
-def test_release_asset_posts_to_session_release_endpoint() -> None:
-    captured: dict[str, Any] = {}
-
-    def _capture(url: str, **kwargs: Any) -> _FakeResponse:
-        captured["url"] = url
-        captured["body"] = json.loads(kwargs["data"].decode("utf-8"))
-        captured["auth"] = kwargs["headers"].get("Authorization")
-        captured["timeout"] = kwargs["timeout"]
-        return _FakeResponse(
-            status_code=200,
-            body=b'{"assetId":"asset_test","released":true}',
-        )
-
-    client = HttpClient(base_url="https://example.com", api_key="bg_key")
-
-    with patch.object(client._session, "post", side_effect=_capture):
-        client.release_asset("asset_test")
-
-    assert captured == {
-        "url": "https://example.com/docs/asset_test/session/release",
-        "body": {},
-        "auth": "Bearer bg_key",
-        "timeout": 30.0,
-    }
-
-
 def test_deeply_nested_paths() -> None:
     """``doc.format.paragraph.set_alignment(p)`` walks three levels and
     serializes as ``SetParagraphAlignment``. It's a pure mutation, so it

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/tests/test_ptc.py

@@ -465,16 +465,20 @@ def test_buffer_emits_begin_on_call_and_end_on_flush(monkeypatch: Any) -> None:
     client = _FakeHttpClient()
     buf = CommandBuffer(client, asset_id="asset-1", auto_flush_seconds=0)
 
-    buf.call(CreateParagraph(
-        text="hello",
-        at={"kind": "documentEnd"},
-        client_node_id="p_first",
-    ))
-    buf.call(CreateParagraph(
-        text="world",
-        at={"kind": "documentEnd"},
-        client_node_id="p_second",
-    ))
+    buf.call(
+        CreateParagraph(
+            text="hello",
+            at={"kind": "documentEnd"},
+            client_node_id="p_first",
+        )
+    )
+    buf.call(
+        CreateParagraph(
+            text="world",
+            at={"kind": "documentEnd"},
+            client_node_id="p_second",
+        )
+    )
 
     assert len(calls["begin"]) == 2
     assert calls["begin"][0][0] == "CreateParagraph"
@@ -490,6 +494,36 @@ def test_buffer_emits_begin_on_call_and_end_on_flush(monkeypatch: Any) -> None:
     assert all(c["result"] == {"ok": True} for c in calls["end"])
 
 
+def test_buffer_emits_for_create_table(monkeypatch: Any) -> None:
+    """``CreateTable`` is allow-listed too, so a table create surfaces as
+    its own PTC sub-tool-card (so the frontend can offer a scroll-to-table
+    affordance). The begin fires on call and the end on flush."""
+    from docx._buffer import CommandBuffer
+    from docx.commands import CreateTable
+
+    calls = _stub_ptc(monkeypatch)
+    client = _FakeHttpClient()
+    buf = CommandBuffer(client, asset_id="asset-1", auto_flush_seconds=0)
+
+    buf.call(
+        CreateTable(
+            rows=3,
+            columns=4,
+            at={"kind": "documentEnd"},
+            client_node_id="t_first",
+        )
+    )
+
+    assert len(calls["begin"]) == 1
+    assert calls["begin"][0][0] == "CreateTable"
+    assert calls["begin"][0][3] == "asset-1"
+    assert len(calls["end"]) == 0
+
+    buf.flush()
+    assert len(calls["end"]) == 1
+    assert calls["end"][0]["is_error"] is False
+
+
 def test_buffer_emits_end_with_error_on_flush_failure(monkeypatch: Any) -> None:
     """End events fire with ``is_error=True`` when the batch raises.
 
@@ -505,11 +539,13 @@ def test_buffer_emits_end_with_error_on_flush_failure(monkeypatch: Any) -> None:
     client = _FakeHttpClient(raise_on_call=True)
     buf = CommandBuffer(client, asset_id="asset-1", auto_flush_seconds=0)
 
-    buf.call(CreateParagraph(
-        text="hello",
-        at={"kind": "documentEnd"},
-        client_node_id="p_first",
-    ))
+    buf.call(
+        CreateParagraph(
+            text="hello",
+            at={"kind": "documentEnd"},
+            client_node_id="p_first",
+        )
+    )
     assert len(calls["begin"]) == 1
 
     import pytest as _pytest

{athena_python_docx-0.15.4 → athena_python_docx-0.16.0}/uv.lock

@@ -21,7 +21,7 @@ wheels = [
 
 [[package]]
 name = "athena-python-docx"
-version = "0.15.3"
+version = "0.15.2"
 source = { editable = "." }
 dependencies = [
     { name = "requests" },