databricks-tellr-app 0.4.2.dev16__tar.gz → 0.4.2.dev18__tar.gz

{databricks_tellr_app-0.4.2.dev16 → databricks_tellr_app-0.4.2.dev18}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: databricks-tellr-app
-Version: 0.4.2.dev16
+Version: 0.4.2.dev18
 Summary: Tellr application package for Databricks Apps
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown

{databricks_tellr_app-0.4.2.dev16 → databricks_tellr_app-0.4.2.dev18}/databricks_tellr_app.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: databricks-tellr-app
-Version: 0.4.2.dev16
+Version: 0.4.2.dev18
 Summary: Tellr application package for Databricks Apps
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown

{databricks_tellr_app-0.4.2.dev16 → databricks_tellr_app-0.4.2.dev18}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "databricks-tellr-app"
-version = "0.4.2.dev16"
+version = "0.4.2.dev18"
 description = "Tellr application package for Databricks Apps"
 readme = "README.md"
 requires-python = ">=3.10"

{databricks_tellr_app-0.4.2.dev16 → databricks_tellr_app-0.4.2.dev18}/src/api/routes/export.py

@@ -181,12 +181,47 @@ def build_slide_html(slide: dict, slide_deck: dict) -> str:
     # see buildStandaloneDeckDocument in frontend/src/services/slideDocument.ts.
     #
     # Deliberately a `#` comment and NOT part of the f-string. Anything inside the
-    # f-string is EMITTED into every exported document, where it (a) displaces
-    # useful deck CSS from the ~15,000-char budget `_truncate_html` hands the LLM,
-    # and (b) eats headroom under the 2000-char length threshold a sheet-role
-    # classifier uses to tell this reset sheet from the deck's own. Keep the
-    # rendered comment to one short line and put the reasoning here instead.
+    # f-string is EMITTED into every exported document, where it displaces useful
+    # deck CSS from the ~15,000-char budget `_truncate_html` hands the LLM — an
+    # oversize injected stylesheet driving that budget negative is what produced
+    # the all-placeholder export bug (WF-01). Keep every rendered comment to one
+    # short line and put the reasoning here instead.
+    #
+    # There is NO 2000-char sheet-role classifier, despite what this comment used
+    # to claim. Nothing in src/ or tests/ keys off a stylesheet's length; the tests
+    # identify sheets POSITIONALLY (`blocks[0..2]`, `len(blocks) == 3`) and by
+    # content (the deck's sheet is the one equal to the deck's CSS verbatim), so
+    # budgeting against that supposed cap would be budgeting against nothing. The
+    # real cost is the LLM budget above.
     # Pinned by tests/unit/test_preview_box_model_parity.py.
+    #
+    # NO SLIDE-HOST FRAME CONTRACT HERE, DELIBERATELY. A contract framing `body`
+    # and stretching its CHILD to `position: absolute !important; inset: 0
+    # !important; width/height: 100% !important` was injected into this sheet at
+    # 0.4.2.dev17 and is REVERTED, because on the huashu path it destroys tables.
+    #
+    # `services/pptx-emit-huashu/preprocess.mjs::flattenTables()` reads every
+    # `<td>/<th>`'s bounding rect and re-emits each cell as an absolutely
+    # positioned <div> carrying its own `left/top/width/height` as NON-important
+    # INLINE styles, then calls `document.body.appendChild(div)`. Every cell is
+    # therefore a DIRECT CHILD of `body` by the time the walker runs, so a
+    # stylesheet `!important` rule on `body > *` outranks each cell's inline
+    # position and collapses all of them onto one rect. Measured on a real
+    # 21-cell deck: distinct cell rects in ppt/slides/slide1.xml went 42 -> 11,
+    # with 12 shapes stacked at (12.0, 349.8) and 12 more at (0.0, 719.5).
+    #
+    # The document itself is unchanged by this — `body` has exactly one rendered
+    # child before scripts run — so a document-level geometry probe reads 0.00 px
+    # and is structurally blind to the defect. It is an EMIT-TIME regression; only
+    # diffing the emitted slide XML can see it.
+    #
+    # What the contract was for (a section-wrapped design-system deck whose ground
+    # never paints) is handled on the capture surfaces by aiming at the slide ROOT
+    # instead — see `findSlideRoot` in frontend/src/services/slideDocument.ts. That
+    # locator injects no CSS, so it cannot perturb huashu geometry. This path never
+    # needed the contract regardless: the sidecar reads
+    # `getComputedStyle(root).backgroundColor`, and a 1280x0 element still computes
+    # the right colour, so the .pptx was immune here BY CONSTRUCTION.
     complete_html = f"""<!DOCTYPE html>
 <html lang="en">
 <head>