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

{databricks_tellr_app-0.4.2.dev17 → 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.dev17
+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.dev17 → 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.dev17
+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.dev17 → 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.dev17"
+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.dev17 → databricks_tellr_app-0.4.2.dev18}/src/api/routes/export.py

@@ -22,7 +22,6 @@ from src.utils.html_safety import (
     SLIDE_CSP_META,
     SLIDE_ROOT_RESET_STYLE,
     scan_html_for_unsafe_patterns,
-    slide_host_frame_style,
 )
 
 logger = logging.getLogger(__name__)
@@ -191,27 +190,38 @@ def build_slide_html(slide: dict, slide_deck: dict) -> str:
     # 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). This
-    # sheet is already over that supposed cap at 2,098 chars, so budgeting against
-    # it would be budgeting against nothing. The real cost is the LLM budget above.
+    # 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.
     #
-    # THE FRAME CONTRACT GOES IN THIS SHEET ONLY, NEVER SHEET 2 (WF-03). Sheet 2
-    # must stay byte-equal to the deck's own CSS, or a deck whose CSS opens with
-    # `@import url(fonts.googleapis…)` loses its webfont: `@import` is only valid
-    # before every other rule of ITS OWN stylesheet. Pinned by
-    # tests/unit/test_export_deck_stylesheet.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.
     #
-    # WHY THE CONTRACT IS HERE AT ALL. A design-system-pinned deck nests the slide
-    # two levels deep: a `<section>` carries the slide ground via a bare TYPE
-    # selector, and inside it `.slide` is `position: absolute; inset: 0`. The
-    # wrapper holds no in-flow content, collapses to height 0, and the ground it
-    # carries never paints — the deck's own dark `html, body` shows through at
-    # contrast 1.3025. Five preview surfaces injected this contract; no export
-    # builder did. Latent on this path today (the huashu sidecar reads
+    # `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 BY CONSTRUCTION) — but the
-    # divergence is real and this is where it stops being one.
+    # the right colour, so the .pptx was immune here BY CONSTRUCTION.
     complete_html = f"""<!DOCTYPE html>
 <html lang="en">
 <head>
@@ -263,8 +273,6 @@ def build_slide_html(slide: dict, slide_deck: dict) -> str:
        shifts content past the clip and truncates the export's bottom edge
        (same neutralization as every other surface). */
     {SLIDE_ROOT_RESET_STYLE}
-    /* After deck CSS: the shared slide-host frame contract. */
-    {slide_host_frame_style("body")}
   </style>
 </head>
 <body>

{databricks_tellr_app-0.4.2.dev17 → databricks_tellr_app-0.4.2.dev18}/src/utils/html_safety.py

@@ -52,47 +52,6 @@ SLIDE_ROOT_RESET_STYLE = """
   }
 """
 
-# The fixed 16:9 frame every slide surface renders into. Mirrors the frontend's
-# SLIDE_FRAME_W / SLIDE_FRAME_H; tests/unit/test_export_csp.py pins both.
-SLIDE_FRAME_W = 1280
-SLIDE_FRAME_H = 720
-
-
-def slide_host_frame_style(host_selector: str) -> str:
-    """The slide-host frame contract: frame ``host_selector``, STRETCH ITS CHILD.
-
-    Byte-identical mirror of the frontend builder's ``slideHostFrameStyle``
-    (frontend/src/services/slideDocument.ts), which carries the full rationale and
-    the per-declaration measurements. ``tests/unit/test_export_csp.py`` asserts
-    this renders exactly what the TS function renders, so the two cannot drift.
-
-    Why any of it is needed: a design-system-pinned deck nests the slide two
-    levels deep — a ``<section>`` carries the slide ground via a bare TYPE
-    selector, and inside it ``.slide`` is ``position: absolute; inset: 0``. The
-    wrapper therefore holds no in-flow content and collapses to ``height: 0``, so
-    the ground it carries never paints and the deck's own dark ``html, body``
-    shows through instead. Colour and font still inherit, which is why the result
-    is readable DARK-ON-DARK rather than an obviously blank slide. Sizing the ROOT
-    does not fix it; only stretching the host's CHILD gives the
-    background-carrying element area.
-    """
-    return f"""
-  {host_selector} {{
-    position: relative !important;
-    width: {SLIDE_FRAME_W}px !important;
-    height: {SLIDE_FRAME_H}px !important;
-    overflow: hidden;
-  }}
-  {host_selector} > :not(#tellr-host-frame-boost):not(.slide-wrapper) {{
-    position: absolute !important;
-    inset: 0 !important;
-    width: 100% !important;
-    height: 100% !important;
-    box-sizing: border-box !important;
-    overflow: hidden;
-  }}
-"""
-
 # Script sources allowed in slides (Chart.js + Tailwind Play CDN + Google Fonts).
 # Google Fonts <link>s are allowed by the slide CSP `style-src`/`font-src` at
 # runtime, so the scanner must not flag them as external resources.