databricks-tellr-app 0.3.11.dev4__tar.gz → 0.3.11.dev5__tar.gz

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/PKG-INFO

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

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/databricks_tellr_app.egg-info/PKG-INFO

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

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/pyproject.toml

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

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/src/api/services/chat_service.py

@@ -113,17 +113,26 @@ class ChatService:
             and deck_dict.get("html_content")
             and "{{ds-asset:" in deck_dict.get("html_content", "")
         )
+        # @font-face src url() fonts and background-image url() live in the deck's
+        # top-level ``css``, not in slide html — gate on it too, else a deck whose
+        # only brand-asset reference is in css skips the resolver entirely
+        # (substitute_deck_dict_ds_assets covers the css field).
+        ds_needs_deck_css = bool(
+            deck_dict
+            and deck_dict.get("css")
+            and "{{ds-asset:" in deck_dict.get("css", "")
+        )
 
         if (
             needs_deck or needs_html or needs_deck_html
-            or ds_needs_deck or ds_needs_html or ds_needs_deck_html
+            or ds_needs_deck or ds_needs_html or ds_needs_deck_html or ds_needs_deck_css
         ):
             with get_db_session() as db:
                 if needs_deck or needs_deck_html:
                     substitute_deck_dict_images(deck_dict, db)
                 if needs_html:
                     raw_html = substitute_image_placeholders(raw_html, db)
-                if ds_needs_deck or ds_needs_deck_html:
+                if ds_needs_deck or ds_needs_deck_html or ds_needs_deck_css:
                     substitute_deck_dict_ds_assets(deck_dict, db)
                 if ds_needs_html:
                     raw_html = substitute_ds_asset_placeholders(raw_html, db)

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/src/database/models/design_system.py

@@ -45,8 +45,13 @@ from src.core.database import Base
 # analogous to ``image_service.MAX_FILE_SIZE``. Kept here next to the models so
 # every writer shares one source of truth. Bytes are only ever persisted in the
 # dedicated ``design_system_asset`` table.
-MAX_ASSET_SIZE_BYTES = 10 * 1024 * 1024  # 10 MB per individual asset
-MAX_BUNDLE_SIZE_BYTES = 50 * 1024 * 1024  # 50 MB per uploaded design-system bundle
+MAX_ASSET_SIZE_BYTES = 50 * 1024 * 1024  # 50 MB per individual asset
+MAX_BUNDLE_SIZE_BYTES = 200 * 1024 * 1024  # 200 MB per uploaded design-system bundle
+# NOTE: bytes are persisted in-row in ``design_system_asset`` (Lakebase Postgres).
+# Raising the bundle cap to 200 MB means a single import can add up to ~200 MB of
+# BLOB rows; large blobs bloat the row store and every copy-on-write branch fork.
+# This is a deliberate limit bump, not a storage re-architecture — revisit
+# out-of-row/object-store offloading if bundles routinely approach this size.
 
 # JSON on SQLite (tests); JSONB on PostgreSQL/Lakebase so the parsed manifest can
 # later be introspected/indexed natively. Mirrors the ImageAsset.tags convention.

{databricks_tellr_app-0.3.11.dev4 → databricks_tellr_app-0.3.11.dev5}/src/utils/ds_asset_utils.py

@@ -44,15 +44,33 @@ def substitute_ds_asset_placeholders(html: str, db: Session) -> str:
     return DS_ASSET_PLACEHOLDER_PATTERN.sub(replace_match, html)
 
 
+# Deck-level string fields (siblings of per-slide ``html``) that can carry a
+# ``{{ds-asset:ID}}`` reference. Per the deck_json schema — "slides array, css,
+# external_scripts, scripts" (see ``database/models/session.py``) — these are the
+# only asset-bearing fields: ``css`` holds @font-face ``src: url()`` fonts and
+# ``background-image: url()`` backgrounds, and ``html_content`` is the full
+# knitted HTML. ``scripts``/``external_scripts`` are JavaScript and never
+# reference brand assets, so they are intentionally excluded.
+_DECK_DS_ASSET_FIELDS = ("html_content", "css")
+
+
 def substitute_deck_dict_ds_assets(deck_dict: dict, db: Session) -> dict:
-    """Substitute {{ds-asset:ID}} placeholders in all slides of a deck dict."""
-    if not deck_dict or not deck_dict.get("slides"):
+    """Substitute {{ds-asset:ID}} placeholders across a deck dict.
+
+    Covers every field that can carry the placeholder: each slide's ``html``,
+    the deck's ``html_content`` (full knitted HTML) and its top-level ``css``
+    (fonts + backgrounds). Deck-level fields are resolved independently of the
+    slides array so a deck with no slides still has its ``css``/``html_content``
+    substituted.
+    """
+    if not deck_dict:
         return deck_dict
-    for slide in deck_dict["slides"]:
+    for slide in deck_dict.get("slides") or []:
         html = slide.get("html", "")
         if "{{ds-asset:" in html:
             slide["html"] = substitute_ds_asset_placeholders(html, db)
-    # Also handle html_content if present (full knitted HTML)
-    if deck_dict.get("html_content") and "{{ds-asset:" in deck_dict["html_content"]:
-        deck_dict["html_content"] = substitute_ds_asset_placeholders(deck_dict["html_content"], db)
+    for field in _DECK_DS_ASSET_FIELDS:
+        value = deck_dict.get(field)
+        if value and "{{ds-asset:" in value:
+            deck_dict[field] = substitute_ds_asset_placeholders(value, db)
     return deck_dict