databricks-tellr-app 0.3.11.dev7__tar.gz → 0.3.11.dev9__tar.gz

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/PKG-INFO

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

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/databricks_tellr_app.egg-info/PKG-INFO

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

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/databricks_tellr_app.egg-info/SOURCES.txt

@@ -14,6 +14,7 @@ setup.py
 ./src/api/middleware/request_logging.py
 ./src/api/routes/__init__.py
 ./src/api/routes/admin.py
+./src/api/routes/admin_usage.py
 ./src/api/routes/agent_config.py
 ./src/api/routes/chat.py
 ./src/api/routes/deck_contributors.py
@@ -33,7 +34,6 @@ setup.py
 ./src/api/routes/settings/__init__.py
 ./src/api/routes/settings/contributors.py
 ./src/api/routes/settings/deck_prompts.py
-./src/api/routes/settings/design_systems.py
 ./src/api/routes/settings/identities.py
 ./src/api/routes/settings/slide_styles.py
 ./src/api/schemas/__init__.py
@@ -52,6 +52,8 @@ setup.py
 ./src/api/services/job_queue.py
 ./src/api/services/session_manager.py
 ./src/api/services/session_naming.py
+./src/api/services/usage_events.py
+./src/api/services/usage_service.py
 ./src/api/utils/__init__.py
 ./src/api/utils/validation.py
 ./src/core/__init__.py
@@ -74,7 +76,6 @@ setup.py
 ./src/database/__init__.py
 ./src/database/models/__init__.py
 ./src/database/models/deck_contributor.py
-./src/database/models/design_system.py
 ./src/database/models/feedback.py
 ./src/database/models/genie_space.py
 ./src/database/models/google_global_credentials.py
@@ -88,6 +89,7 @@ setup.py
 ./src/database/models/session.py
 ./src/database/models/slide_deck_prompt.py
 ./src/database/models/slide_style_library.py
+./src/database/models/usage_event.py
 ./src/database/models/user_preference.py
 ./src/domain/__init__.py
 ./src/domain/slide.py
@@ -97,9 +99,6 @@ setup.py
 ./src/services/agent_factory.py
 ./src/services/config_service.py
 ./src/services/config_validator.py
-./src/services/design_system_compiler.py
-./src/services/design_system_service.py
-./src/services/design_system_templates.py
 ./src/services/drive_uploader.py
 ./src/services/genie_service.py
 ./src/services/google_slides_auth.py
@@ -124,14 +123,12 @@ setup.py
 ./src/services/identity_providers/workspace_provider.py
 ./src/services/tools/__init__.py
 ./src/services/tools/agent_bricks_tool.py
-./src/services/tools/ds_asset_tool.py
 ./src/services/tools/genie_tool.py
 ./src/services/tools/mcp_tool.py
 ./src/services/tools/model_endpoint_tool.py
 ./src/services/tools/vector_tool.py
 ./src/utils/__init__.py
 ./src/utils/css_utils.py
-./src/utils/ds_asset_utils.py
 ./src/utils/error_handling.py
 ./src/utils/html_safety.py
 ./src/utils/html_utils.py
@@ -160,6 +157,7 @@ src/api/middleware/__init__.py
 src/api/middleware/request_logging.py
 src/api/routes/__init__.py
 src/api/routes/admin.py
+src/api/routes/admin_usage.py
 src/api/routes/agent_config.py
 src/api/routes/chat.py
 src/api/routes/deck_contributors.py
@@ -179,7 +177,6 @@ src/api/routes/version.py
 src/api/routes/settings/__init__.py
 src/api/routes/settings/contributors.py
 src/api/routes/settings/deck_prompts.py
-src/api/routes/settings/design_systems.py
 src/api/routes/settings/identities.py
 src/api/routes/settings/slide_styles.py
 src/api/schemas/__init__.py
@@ -198,6 +195,8 @@ src/api/services/feedback_service.py
 src/api/services/job_queue.py
 src/api/services/session_manager.py
 src/api/services/session_naming.py
+src/api/services/usage_events.py
+src/api/services/usage_service.py
 src/api/utils/__init__.py
 src/api/utils/validation.py
 src/core/__init__.py
@@ -220,7 +219,6 @@ src/core/user_context.py
 src/database/__init__.py
 src/database/models/__init__.py
 src/database/models/deck_contributor.py
-src/database/models/design_system.py
 src/database/models/feedback.py
 src/database/models/genie_space.py
 src/database/models/google_global_credentials.py
@@ -234,6 +232,7 @@ src/database/models/request_log.py
 src/database/models/session.py
 src/database/models/slide_deck_prompt.py
 src/database/models/slide_style_library.py
+src/database/models/usage_event.py
 src/database/models/user_preference.py
 src/domain/__init__.py
 src/domain/slide.py
@@ -243,9 +242,6 @@ src/services/agent.py
 src/services/agent_factory.py
 src/services/config_service.py
 src/services/config_validator.py
-src/services/design_system_compiler.py
-src/services/design_system_service.py
-src/services/design_system_templates.py
 src/services/drive_uploader.py
 src/services/genie_service.py
 src/services/google_slides_auth.py
@@ -270,14 +266,12 @@ src/services/identity_providers/local_provider.py
 src/services/identity_providers/workspace_provider.py
 src/services/tools/__init__.py
 src/services/tools/agent_bricks_tool.py
-src/services/tools/ds_asset_tool.py
 src/services/tools/genie_tool.py
 src/services/tools/mcp_tool.py
 src/services/tools/model_endpoint_tool.py
 src/services/tools/vector_tool.py
 src/utils/__init__.py
 src/utils/css_utils.py
-src/utils/ds_asset_utils.py
 src/utils/error_handling.py
 src/utils/html_safety.py
 src/utils/html_utils.py

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/pyproject.toml

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

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/main.py

@@ -16,7 +16,7 @@ from fastapi.middleware.cors import CORSMiddleware
 from fastapi.responses import FileResponse
 from fastapi.staticfiles import StaticFiles
 
-from src.api.routes import admin, agent_config, chat, export, feedback, images, profiles, sessions, slides, tools, tour, verification, version, google_slides, setup, local_version
+from src.api.routes import admin, admin_usage, agent_config, chat, export, feedback, images, profiles, sessions, slides, tools, tour, verification, version, google_slides, setup, local_version
 from src.api.routes.deck_contributors import router as deck_contributors_router
 from src.core.databricks_client import get_or_create_user_client, set_user_client
 from src.core.user_context import get_current_user as get_ctx_user, set_current_user
@@ -27,7 +27,6 @@ from src.core.permission_context import (
 from src.api.routes.settings import (
     contributors_router,
     deck_prompts_router,
-    design_systems_router,
     identities_router,
     slide_styles_router,
 )
@@ -382,6 +381,14 @@ async def user_auth_middleware(request: Request, call_next):
     )
     set_permission_context(permission_ctx)
 
+    # Record login usage-event (visit-deduped, non-blocking, all envs)
+    if user_name:
+        try:
+            from src.api.services.usage_events import record_login
+            record_login(user_name)
+        except Exception as e:
+            logger.debug(f"Failed to record login usage event: {e}")
+
     # Record user login for local identity table (non-blocking)
     if user_id and user_name and IS_PRODUCTION:
         try:
@@ -412,6 +419,7 @@ app.add_middleware(RequestLoggingMiddleware)
 
 # Include API routers
 app.include_router(admin.router)
+app.include_router(admin_usage.router)
 app.include_router(agent_config.router)
 app.include_router(chat.router)
 app.include_router(feedback.router)
@@ -435,7 +443,6 @@ app.include_router(contributors_router, prefix="/api/settings", tags=["settings"
 app.include_router(deck_prompts_router, prefix="/api/settings", tags=["settings"])
 app.include_router(identities_router, prefix="/api/settings", tags=["settings"])
 app.include_router(slide_styles_router, prefix="/api/settings", tags=["settings"])
-app.include_router(design_systems_router, prefix="/api/settings", tags=["settings"])
 
 # MCP server — mount the FastMCP streamable-HTTP ASGI app at /mcp.
 # Must be registered before the SPA catch-all (which is added lazily by

databricks_tellr_app-0.3.11.dev9/src/api/routes/admin_usage.py

@@ -0,0 +1,145 @@
+"""Admin usage-analytics endpoints (ungated, like the rest of /admin)."""
+
+import logging
+from datetime import date as dt_date
+from datetime import datetime
+
+from fastapi import APIRouter, Depends, HTTPException, Query, status
+from sqlalchemy.orm import Session
+
+from src.api.services.usage_service import MAX_WINDOW_DAYS, UsageService
+from src.core.database import get_db
+
+logger = logging.getLogger(__name__)
+router = APIRouter(prefix="/api/admin/usage", tags=["admin-usage"])
+
+
+def _parse_date(value: str, name: str) -> dt_date:
+    try:
+        return datetime.strptime(value, "%Y-%m-%d").date()
+    except ValueError:
+        raise HTTPException(
+            status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
+            detail=f"{name} must be a YYYY-MM-DD date",
+        )
+
+
+def _window_kwargs(
+    days: int | None,
+    start: str | None,
+    end: str | None,
+    all_data: bool,
+) -> dict:
+    """Validate window query params into UsageService kwargs.
+
+    Precedence: all > start/end range > days > default (7, applied by the
+    service). ``start`` and ``end`` must be provided together.
+    """
+    if all_data:
+        return {"all_data": True}
+    if start is not None or end is not None:
+        if start is None or end is None:
+            raise HTTPException(
+                status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
+                detail="start and end must be provided together",
+            )
+        start_date = _parse_date(start, "start")
+        end_date = _parse_date(end, "end")
+        if start_date > end_date:
+            raise HTTPException(
+                status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
+                detail="start must be on or before end",
+            )
+        return {"start": start_date, "end": end_date}
+    if days is not None:
+        if not 1 <= days <= MAX_WINDOW_DAYS:
+            raise HTTPException(
+                status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
+                detail=f"days must be between 1 and {MAX_WINDOW_DAYS}",
+            )
+        return {"days": days}
+    return {}
+
+
+def _handle(callable_, *args, **kwargs):
+    try:
+        return callable_(*args, **kwargs)
+    except HTTPException:
+        raise
+    except Exception as e:
+        logger.error(f"Usage analytics error: {e}", exc_info=True)
+        raise HTTPException(
+            status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
+            detail="Failed to compute usage analytics",
+        )
+
+
+_DAYS = Query(default=None, description=f"Rolling window, 1..{MAX_WINDOW_DAYS}")
+_START = Query(default=None, description="Range start (YYYY-MM-DD, inclusive)")
+_END = Query(default=None, description="Range end (YYYY-MM-DD, inclusive)")
+_ALL = Query(default=False, alias="all", description="Report over all data")
+
+
+@router.get("/summary")
+def get_summary(
+    days: int | None = _DAYS,
+    start: str | None = _START,
+    end: str | None = _END,
+    all_data: bool = _ALL,
+    db: Session = Depends(get_db),
+):
+    kwargs = _window_kwargs(days, start, end, all_data)
+    return _handle(UsageService().get_summary, db, **kwargs)
+
+
+@router.get("/daily")
+def get_daily(
+    days: int | None = _DAYS,
+    start: str | None = _START,
+    end: str | None = _END,
+    all_data: bool = _ALL,
+    db: Session = Depends(get_db),
+):
+    kwargs = _window_kwargs(days, start, end, all_data)
+    return _handle(UsageService().get_daily, db, **kwargs)
+
+
+@router.get("/top-users")
+def get_top_users(
+    days: int | None = _DAYS,
+    start: str | None = _START,
+    end: str | None = _END,
+    all_data: bool = _ALL,
+    db: Session = Depends(get_db),
+):
+    kwargs = _window_kwargs(days, start, end, all_data)
+    return _handle(UsageService().get_top_users, db, **kwargs)
+
+
+@router.get("/funnel")
+def get_funnel(
+    days: int | None = _DAYS,
+    start: str | None = _START,
+    end: str | None = _END,
+    all_data: bool = _ALL,
+    db: Session = Depends(get_db),
+):
+    kwargs = _window_kwargs(days, start, end, all_data)
+    return _handle(UsageService().get_funnel, db, **kwargs)
+
+
+@router.get("/retention")
+def get_retention(db: Session = Depends(get_db)):
+    return _handle(UsageService().get_retention, db)
+
+
+@router.get("/heatmap")
+def get_heatmap(
+    days: int | None = _DAYS,
+    start: str | None = _START,
+    end: str | None = _END,
+    all_data: bool = _ALL,
+    db: Session = Depends(get_db),
+):
+    kwargs = _window_kwargs(days, start, end, all_data)
+    return _handle(UsageService().get_heatmap, db, **kwargs)

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/agent_config.py

@@ -21,61 +21,10 @@ router = APIRouter(prefix="/api/sessions/{session_id}/agent-config", tags=["agen
 
 
 def _validate_references(config: AgentConfig) -> None:
-    """Validate the library references and SANITIZE the template pin, in place.
-
-    ``design_system_id``, ``slide_style_id`` and ``deck_prompt_id`` stay STRICT
-    (422 on a missing/inactive entry — the existing convention). The template
-    pin is SELF-HEALING instead: template ids are re-minted by the sanctioned
-    delete+re-upload workflow, so a persisted config can legitimately hold a
-    stale pin — rejecting it would wedge EVERY later config update. When
-    ``template_id`` does not resolve to a template belonging to the selected
-    design system (pin with no design system, another system's template, or a
-    missing/re-materialized template), it is cleared in place with a warning;
-    the caller persists and returns the sanitized config so the frontend state
-    syncs. This mirrors the deliberately lenient generation path.
-    """
-    from src.database.models import (
-        DesignSystem,
-        DesignSystemTemplate,
-        SlideDeckPromptLibrary,
-        SlideStyleLibrary,
-    )
+    """Validate that slide_style_id and deck_prompt_id reference existing library entries."""
+    from src.database.models import SlideStyleLibrary, SlideDeckPromptLibrary
 
     with get_db_session() as db:
-        if config.design_system_id is not None:
-            design_system = (
-                db.query(DesignSystem)
-                .filter(
-                    DesignSystem.id == config.design_system_id,
-                    DesignSystem.is_active == True,  # noqa: E712
-                )
-                .first()
-            )
-            if not design_system:
-                raise HTTPException(
-                    status_code=422,
-                    detail=f"design_system_id {config.design_system_id} not found",
-                )
-        if config.template_id is not None:
-            template = None
-            if config.design_system_id is not None:
-                template = (
-                    db.query(DesignSystemTemplate)
-                    .filter(
-                        DesignSystemTemplate.id == config.template_id,
-                        DesignSystemTemplate.design_system_id == config.design_system_id,
-                    )
-                    .first()
-                )
-            if template is None:
-                logger.warning(
-                    "Clearing stale template pin %s (design_system_id=%s): the "
-                    "template does not exist or does not belong to the selected "
-                    "design system; saving the config without it",
-                    config.template_id,
-                    config.design_system_id,
-                )
-                config.template_id = None
         if config.slide_style_id is not None:
             style = (
                 db.query(SlideStyleLibrary)
@@ -151,9 +100,7 @@ async def get_agent_config(session_id: str):
 async def put_agent_config(session_id: str, config: AgentConfig):
     """Replace the full agent config for a session."""
     # Pydantic already validated duplicates via model_validator.
-    # Now validate foreign-key references; a stale template pin is sanitized
-    # (cleared in place) rather than rejected, so what is persisted and
-    # returned below is the effective config.
+    # Now validate foreign-key references.
     _validate_references(config)
 
     try:

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/export.py

@@ -405,14 +405,12 @@ async def export_to_pptx(request: ExportPPTXRequest):
         if not slide_deck or not slide_deck.get("slides"):
             raise HTTPException(status_code=404, detail="No slides available")
 
-        # Substitute {{image:ID}} + {{ds-asset:ID}} placeholders with base64 data URIs
-        # so the PPTX converter can extract and embed the actual images/brand assets
-        from src.utils.ds_asset_utils import substitute_deck_dict_ds_assets
+        # Substitute {{image:ID}} placeholders with base64 data URIs
+        # so the PPTX converter can extract and embed the actual images
         from src.utils.image_utils import substitute_deck_dict_images
         from src.core.database import get_db_session
         with get_db_session() as db:
             substitute_deck_dict_images(slide_deck, db)
-            substitute_deck_dict_ds_assets(slide_deck, db)
         
         slide_count = len(slide_deck.get("slides", []))
         log_msg = (

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/feedback.py

@@ -88,6 +88,33 @@ def get_stats_report(weeks: int = Query(default=12, ge=1, le=52), db: Session =
         )
 
 
+@router.get("/list")
+def list_feedback(
+    weeks: int = Query(default=12, ge=1, le=52),
+    category: str | None = Query(default=None),
+    severity: str | None = Query(default=None),
+    page: int = Query(default=1, ge=1),
+    page_size: int = Query(default=20, ge=1, le=100),
+    db: Session = Depends(get_db),
+):
+    try:
+        service = FeedbackService()
+        return service.list_feedback(
+            db=db,
+            weeks=weeks,
+            category=category,
+            severity=severity,
+            page=page,
+            page_size=page_size,
+        )
+    except Exception as e:
+        logger.error(f"Feedback list error: {e}", exc_info=True)
+        raise HTTPException(
+            status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
+            detail="Failed to list feedback",
+        )
+
+
 @router.get("/report/summary")
 def get_feedback_summary(weeks: int = Query(default=4, ge=1, le=52), db: Session = Depends(get_db)):
     try:

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/google_slides.py

@@ -276,13 +276,11 @@ async def start_google_slides_export(
     if not slide_deck or not slide_deck.get("slides"):
         raise HTTPException(status_code=404, detail="No slides available")
 
-    # Substitute {{image:ID}} + {{ds-asset:ID}} placeholders with base64 data URIs
-    from src.utils.ds_asset_utils import substitute_deck_dict_ds_assets
+    # Substitute {{image:ID}} placeholders with base64 data URIs
     from src.utils.image_utils import substitute_deck_dict_images
     from src.core.database import get_db_session
     with get_db_session() as db:
         substitute_deck_dict_images(slide_deck, db)
-        substitute_deck_dict_ds_assets(slide_deck, db)
 
     slides_data = slide_deck.get("slides", [])
     total = len(slides_data)
@@ -504,11 +502,9 @@ async def export_google_slides_from_huashu(
     # legacy /export route). Use a fresh session because the request's `db`
     # is held by the route's transaction context.
     from src.core.database import get_db_session
-    from src.utils.ds_asset_utils import substitute_deck_dict_ds_assets
     from src.utils.image_utils import substitute_deck_dict_images
     with get_db_session() as imdb:
         substitute_deck_dict_images(slide_deck, imdb)
-        substitute_deck_dict_ds_assets(slide_deck, imdb)
 
     title = slide_deck.get("title") or "Presentation"
     slides_data = slide_deck.get("slides") or []

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/sessions.py

@@ -25,6 +25,7 @@ from src.api.services.session_manager import (
     SessionNotFoundError,
     get_session_manager,
 )
+from src.api.services.usage_events import record_deck_retrieved
 from src.core.database import get_db, get_db_session
 from src.core.permission_context import get_permission_context
 from src.core.user_context import get_current_user
@@ -149,12 +150,10 @@ def _check_deck_permission_for_session(
 def _substitute_deck_images(deck_dict: dict) -> None:
     """Substitute {{image:ID}} placeholders in a deck dict with base64 data URIs."""
     from src.core.database import get_db_session
-    from src.utils.ds_asset_utils import substitute_deck_dict_ds_assets
     from src.utils.image_utils import substitute_deck_dict_images
 
     with get_db_session() as db:
         substitute_deck_dict_images(deck_dict, db)
-        substitute_deck_dict_ds_assets(deck_dict, db)
 
 
 @router.post("")
@@ -469,6 +468,10 @@ async def get_session(session_id: str, db: Session = Depends(get_db)):
         # Check permission
         permission = _require_session_access(session, db, PermissionLevel.CAN_VIEW)
 
+        # Record deck-open usage event (deduped per user+deck, non-blocking)
+        if current_user:
+            record_deck_retrieved(current_user, session.get("id"))
+
         # Only the session creator sees chat messages — conversations are private
         is_creator = session.get("created_by") == current_user
         messages = []

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/routes/settings/__init__.py

@@ -1,14 +1,12 @@
 """Configuration API routes."""
 from src.api.routes.settings.contributors import router as contributors_router
 from src.api.routes.settings.deck_prompts import router as deck_prompts_router
-from src.api.routes.settings.design_systems import router as design_systems_router
 from src.api.routes.settings.identities import router as identities_router
 from src.api.routes.settings.slide_styles import router as slide_styles_router
 
 __all__ = [
     "contributors_router",
     "deck_prompts_router",
-    "design_systems_router",
     "identities_router",
     "slide_styles_router",
 ]

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/schemas/agent_config.py

@@ -59,14 +59,6 @@ class AgentConfig(BaseModel):
     """Agent configuration stored as JSON on sessions and profiles."""
     tools: list[ToolEntry] = Field(default_factory=list)
     slide_style_id: Optional[int] = None
-    # A selected design system compiles to prompt text and, when set, takes
-    # precedence over slide_style_id (see agent_factory._get_prompt_content).
-    design_system_id: Optional[int] = None
-    # Optionally pins ONE of the selected design system's templates
-    # (design_system_template.id). Meaningful only alongside design_system_id;
-    # generation IGNORES it (with a log) unless the template exists and belongs
-    # to that design system, so a stale pin can never fail a request.
-    template_id: Optional[int] = None
     deck_prompt_id: Optional[int] = None
     system_prompt: Optional[str] = None
     slide_editing_instructions: Optional[str] = None

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/services/chat_service.py

@@ -35,10 +35,6 @@ from src.utils.html_utils import (
     extract_canvas_ids_from_html,
     split_script_by_canvas,
 )
-from src.utils.ds_asset_utils import (
-    substitute_deck_dict_ds_assets,
-    substitute_ds_asset_placeholders,
-)
 from src.utils.image_utils import substitute_deck_dict_images, substitute_image_placeholders
 
 logger = logging.getLogger(__name__)
@@ -87,13 +83,11 @@ class ChatService:
         logger.info("ChatService initialized successfully")
 
     def _substitute_images_for_response(self, deck_dict, raw_html=None):
-        """Apply image + design-system asset substitution before sending to client.
+        """Apply image placeholder substitution before sending to client.
 
-        Converts {{image:ID}} placeholders (image_assets) and {{ds-asset:ID}}
-        placeholders (design_system_asset) to base64 data URIs in deck dicts and
-        raw HTML. Called at API response boundaries so that stored/cached HTML
-        keeps lightweight placeholders (avoiding LLM context bloat). The two
-        namespaces are resolved independently.
+        Converts {{image:ID}} placeholders to base64 data URIs in deck dicts
+        and raw HTML. Called at API response boundaries so that stored/cached
+        HTML keeps lightweight placeholders (avoiding LLM context bloat).
         """
         from src.core.database import get_db_session
 
@@ -103,39 +97,12 @@ class ChatService:
         needs_html = raw_html and "{{image:" in raw_html
         needs_deck_html = deck_dict and deck_dict.get("html_content") and "{{image:" in deck_dict.get("html_content", "")
 
-        # Design-system brand assets ({{ds-asset:ID}}) — parallel, orthogonal namespace.
-        ds_needs_deck = deck_dict and any(
-            "{{ds-asset:" in s.get("html", "") for s in deck_dict.get("slides", [])
-        )
-        ds_needs_html = raw_html and "{{ds-asset:" in raw_html
-        ds_needs_deck_html = bool(
-            deck_dict
-            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_css
-        ):
+        if needs_deck or needs_html or needs_deck_html:
             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 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)
         return deck_dict, raw_html
 
     def _build_agent_for_session(

{databricks_tellr_app-0.3.11.dev7 → databricks_tellr_app-0.3.11.dev9}/src/api/services/export_job_queue.py

@@ -203,14 +203,12 @@ async def process_export_job(job_id: str, payload: dict) -> None:
         if not slide_deck or not slide_deck.get("slides"):
             raise ValueError("No slides available")
 
-        # Substitute {{image:ID}} + {{ds-asset:ID}} placeholders with base64 data URIs
-        # so the PPTX converter can extract and embed the actual images/brand assets
-        from src.utils.ds_asset_utils import substitute_deck_dict_ds_assets
+        # Substitute {{image:ID}} placeholders with base64 data URIs
+        # so the PPTX converter can extract and embed the actual images
         from src.utils.image_utils import substitute_deck_dict_images
         from src.core.database import get_db_session
         with get_db_session() as db:
             substitute_deck_dict_images(slide_deck, db)
-            substitute_deck_dict_ds_assets(slide_deck, db)
 
         slides = slide_deck.get("slides", [])
         total_slides = len(slides)