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

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

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

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

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

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

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

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/chat.py

@@ -384,7 +384,7 @@ async def send_message(
         logger.error(f"Chat request failed: {e}", exc_info=True)
         raise HTTPException(
             status_code=500,
-            detail="Failed to process message",
+            detail=f"Failed to process message: {str(e)}",
         ) from e
     finally:
         await asyncio.to_thread(
@@ -662,7 +662,7 @@ async def submit_chat_async(
             session_manager.release_session_lock, request.session_id
         )
         logger.error(f"Failed to submit async chat: {e}")
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.get("/chat/poll/{request_id}")

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/export.py

@@ -754,13 +754,13 @@ async def export_to_pptx(request: ExportPPTXRequest):
                     temp_dir.rmdir()
             except Exception:
                 pass
-            raise HTTPException(status_code=500, detail="Export failed")
+            raise HTTPException(status_code=500, detail=f"Export failed: {str(e)}")
         
     except HTTPException:
         raise
     except Exception as e:
         logger.error("PPTX export failed", exc_info=True, extra={"error": str(e)})
-        raise HTTPException(status_code=500, detail="Export failed")
+        raise HTTPException(status_code=500, detail=f"Export failed: {str(e)}")
 
 
 # =============================================================================
@@ -893,7 +893,7 @@ async def start_pptx_export_async(request: ExportPPTXRequest):
     except Exception as e:
         total_time = time.time() - start_time
         logger.error(f"Failed to start async export after {total_time:.2f}s: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to start export")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.get("/pptx/poll/{job_id}", response_model=ExportJobResponse)
@@ -1038,7 +1038,7 @@ async def export_pptx_editable_from_records(request: ExportPPTXFromRecordsReques
         pptx_bytes = build_pptx(title, request.slides, font_mode=font_mode)
     except Exception as e:
         logger.exception(f"Editable PPTX emission failed: {e}")
-        raise HTTPException(status_code=500, detail="Editable PPTX export failed")
+        raise HTTPException(status_code=500, detail=str(e))
 
     safe = "".join(c if c.isalnum() or c in "-_" else "_" for c in title).strip("_") or "slides"
     from fastapi.responses import Response

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/google_slides.py

@@ -10,10 +10,9 @@ user tokens are stored per-user in the ``google_oauth_tokens`` table.
 import json
 import logging
 import os
-import secrets
 from typing import Any, Optional
 
-from fastapi import APIRouter, Depends, HTTPException, Query, Request, Response
+from fastapi import APIRouter, Depends, HTTPException, Query, Request
 from fastapi.responses import HTMLResponse
 from pydantic import BaseModel
 from sqlalchemy.orm import Session
@@ -34,12 +33,6 @@ logger = logging.getLogger(__name__)
 
 router = APIRouter(prefix="/api/export/google-slides", tags=["google-slides"])
 
-# F-CR-3 (SDR-4437): the OAuth "state" carries an unguessable nonce that is also
-# stored in this short-lived cookie; the callback requires them to match before
-# exchanging the authorization code (anti-CSRF / anti-login-CSRF double-submit).
-_OAUTH_STATE_COOKIE = "g_oauth_state"
-_OAUTH_COOKIE_PATH = "/api/export/google-slides"
-
 
 # -------------------------------------------------------------------------
 # Helpers
@@ -105,76 +98,32 @@ class AuthUrlResponse(BaseModel):
 # -------------------------------------------------------------------------
 
 
-def _build_app_origin(request: Request) -> str:
-    """Return the public origin (``scheme://host``) serving the app.
+def _build_redirect_uri(request: Request) -> str:
+    """Build the OAuth callback redirect URI from the current request.
+
+    Google requires redirect URIs to match *exactly* what's registered in GCP,
+    with no query parameters.
 
     Behind a reverse proxy (e.g. Databricks Apps), ``request.base_url``
     returns the internal address (``http://localhost:8000``).  We use the
     ``X-Forwarded-Host`` / ``X-Forwarded-Proto`` headers set by the proxy
-    to reconstruct the public URL instead. For localhost, force ``http``.
+    to reconstruct the public URL instead.
+
+    For localhost, Google only allows ``http``.
     """
     forwarded_host = request.headers.get("x-forwarded-host")
     forwarded_proto = request.headers.get("x-forwarded-proto")
 
     if forwarded_host:
         scheme = forwarded_proto or "https"
-        return f"{scheme}://{forwarded_host.split(',')[0].strip()}"
-
-    base = str(request.base_url).rstrip("/")
-    if "://localhost" in base or "://127.0.0.1" in base:
-        base = base.replace("https://", "http://")
-    return base
-
-
-def _build_redirect_uri(request: Request) -> str:
-    """Build the OAuth callback redirect URI from the current request.
-
-    Google requires redirect URIs to match *exactly* what's registered in GCP,
-    with no query parameters.
-    """
-    return f"{_build_app_origin(request)}/api/export/google-slides/auth/callback"
-
-
-# Callback page template. The JS object braces are intentionally literal; the
-# %TOKENS% are substituted below so the postMessage targets the explicit app
-# origin (F-CR-3) instead of the wildcard '*'.
-_CALLBACK_TEMPLATE = """
-<html><body>
-    <h2>%TITLE%</h2>
-    <p>%MESSAGE%</p>
-    <script>
-        if (window.opener) {
-            window.opener.postMessage({ type: 'google-slides-auth', success: %SUCCESS% }, %ORIGIN%);
-        }
-        setTimeout(() => window.close(), %DELAY%);
-    </script>
-</body></html>
-"""
-
-
-def _callback_response(app_origin: str, *, success: bool) -> HTMLResponse:
-    """Render the popup callback page and clear the anti-CSRF state cookie.
-
-    ``postMessage`` targets ``app_origin`` explicitly (F-CR-3); ``json.dumps``
-    yields a safely-quoted JS string literal for the origin.
-    """
-    if success:
-        title, message, delay = "Authorization Successful", "You can close this window.", "1500"
+        base = f"{scheme}://{forwarded_host.split(',')[0].strip()}"
     else:
-        title = "Authorization Failed"
-        message = "Authorization could not be completed. Please close this window and try again."
-        delay = "3000"
-    body = (
-        _CALLBACK_TEMPLATE
-        .replace("%TITLE%", title)
-        .replace("%MESSAGE%", message)
-        .replace("%SUCCESS%", "true" if success else "false")
-        .replace("%ORIGIN%", json.dumps(app_origin))
-        .replace("%DELAY%", delay)
-    )
-    resp = HTMLResponse(content=body, status_code=200)
-    resp.delete_cookie(_OAUTH_STATE_COOKIE, path=_OAUTH_COOKIE_PATH)
-    return resp
+        base = str(request.base_url).rstrip("/")
+        # Force http for localhost (Google rejects https://localhost)
+        if "://localhost" in base or "://127.0.0.1" in base:
+            base = base.replace("https://", "http://")
+
+    return f"{base}/api/export/google-slides/auth/callback"
 
 
 @router.get("/auth/status", response_model=AuthStatusResponse)
@@ -192,7 +141,6 @@ async def auth_status(db: Session = Depends(get_db)):
 @router.get("/auth/url", response_model=AuthUrlResponse)
 async def auth_url(
     request: Request,
-    response: Response,
     db: Session = Depends(get_db),
 ):
     """Generate the Google OAuth consent URL.
@@ -201,26 +149,13 @@ async def auth_url(
     passed via the OAuth ``state`` parameter so the callback can persist the
     token for the correct user — the redirect URI itself stays clean
     (no query params) to satisfy Google's exact-match requirement.
-
-    F-CR-3: an unguessable nonce is bound into both the ``state`` and a
-    short-lived cookie; the callback requires them to match.
     """
     try:
         auth = _get_auth(db)
         redirect_uri = _build_redirect_uri(request)
-        nonce = secrets.token_urlsafe(32)
-        state = json.dumps({"user": _get_user_identity(), "nonce": nonce})
+        state = json.dumps({"user": _get_user_identity()})
         url = auth.get_auth_url(redirect_uri=redirect_uri, state=state)
 
-        response.set_cookie(
-            key=_OAUTH_STATE_COOKIE,
-            value=nonce,
-            max_age=600,
-            httponly=True,
-            secure=True,
-            samesite="lax",  # sent on Google's top-level redirect back to us
-            path=_OAUTH_COOKIE_PATH,
-        )
         return AuthUrlResponse(url=url)
     except GoogleSlidesAuthError as exc:
         raise HTTPException(status_code=400, detail=str(exc)) from exc
@@ -249,22 +184,12 @@ async def auth_callback(
     them in the ``google_oauth_tokens`` table.  Returns a small HTML page
     that notifies the opener window and closes itself.
     """
-    app_origin = _build_app_origin(request)
-    cookie_nonce = request.cookies.get(_OAUTH_STATE_COOKIE)
     try:
         state_data = json.loads(state) if state else {}
         user = state_data.get("user")
         if not user:
             raise ValueError("Missing user in OAuth state")
 
-        # F-CR-3: verify the anti-CSRF nonce BEFORE exchanging the code. The
-        # state nonce must match the cookie set when the auth URL was issued.
-        state_nonce = state_data.get("nonce")
-        if not cookie_nonce or not state_nonce or not secrets.compare_digest(
-            str(cookie_nonce), str(state_nonce)
-        ):
-            raise ValueError("OAuth state nonce mismatch")
-
         auth = _get_auth(db)
         redirect_uri = _build_redirect_uri(request)
         code_verifier = state_data.get("code_verifier")
@@ -273,13 +198,39 @@ async def auth_callback(
             "Google Slides OAuth callback successful",
             extra={"user": _get_user_identity()},
         )
-    except Exception:
-        # F-CR-2 (SDR-4437): never reflect the raw exception into the HTML page —
-        # it is logged server-side above. Show a static, generic failure message.
+    except Exception as exc:
         logger.error("OAuth callback failed", exc_info=True)
-        return _callback_response(app_origin, success=False)
+        return HTMLResponse(
+            content=f"""
+            <html><body>
+                <h2>Authorization Failed</h2>
+                <p>{exc}</p>
+                <script>
+                    if (window.opener) {{
+                        window.opener.postMessage({{ type: 'google-slides-auth', success: false }}, '*');
+                    }}
+                    setTimeout(() => window.close(), 3000);
+                </script>
+            </body></html>
+            """,
+            status_code=200,
+        )
 
-    return _callback_response(app_origin, success=True)
+    return HTMLResponse(
+        content="""
+        <html><body>
+            <h2>Authorization Successful</h2>
+            <p>You can close this window.</p>
+            <script>
+                if (window.opener) {
+                    window.opener.postMessage({ type: 'google-slides-auth', success: true }, '*');
+                }
+                setTimeout(() => window.close(), 1500);
+            </script>
+        </body></html>
+        """,
+        status_code=200,
+    )
 
 
 # -------------------------------------------------------------------------
@@ -333,7 +284,7 @@ async def start_google_slides_export(
         slide_deck = chat_service.get_slides(request_body.session_id)
     except Exception as exc:
         logger.error("Failed to fetch slides", exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to fetch slides") from exc
+        raise HTTPException(status_code=500, detail=f"Failed to fetch slides: {exc}") from exc
 
     if not slide_deck or not slide_deck.get("slides"):
         raise HTTPException(status_code=404, detail="No slides available")
@@ -497,7 +448,7 @@ async def export_google_slides_from_records(
             )
     except Exception as exc:
         logger.error("Drive upload failed", exc_info=True)
-        raise HTTPException(status_code=502, detail="Drive upload failed") from exc
+        raise HTTPException(status_code=502, detail=f"Drive upload failed: {exc}") from exc
 
     session_manager.set_google_slides_info(
         session_id=request.session_id,
@@ -647,7 +598,7 @@ async def export_google_slides_from_huashu(
         logger.error("Drive upload failed (huashu path)", exc_info=True)
         raise HTTPException(
             status_code=502,
-            detail="Drive upload failed",
+            detail=f"Drive upload failed: {exc}",
         ) from exc
 
     session_manager.set_google_slides_info(

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/sessions.py

@@ -553,7 +553,7 @@ async def duplicate_session(
         logger.error(f"Failed to duplicate session: {e}", exc_info=True)
         raise HTTPException(
             status_code=500,
-            detail="Failed to duplicate session",
+            detail=f"Failed to duplicate session: {str(e)}",
         ) from e
 
 

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/settings/identities.py

@@ -109,7 +109,7 @@ def list_users(
         logger.error(f"Failed to list users: {e}", exc_info=True)
         raise HTTPException(
             status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail="Failed to list users",
+            detail=f"Failed to list users: {e}",
         )
 
 
@@ -149,7 +149,7 @@ def list_groups(
         logger.error(f"Failed to list groups: {e}", exc_info=True)
         raise HTTPException(
             status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail="Failed to list groups",
+            detail=f"Failed to list groups: {e}",
         )
 
 
@@ -203,5 +203,5 @@ def search_identities(
         logger.error(f"Failed to search identities: {e}", exc_info=True)
         raise HTTPException(
             status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail="Failed to search identities",
+            detail=f"Failed to search identities: {e}",
         )

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/setup.py

@@ -123,7 +123,7 @@ async def configure_workspace(request: ConfigureWorkspaceRequest):
         logger.error(f"Failed to configure workspace: {e}")
         raise HTTPException(
             status_code=500,
-            detail="Failed to save configuration"
+            detail=f"Failed to save configuration: {str(e)}"
         )
 
 
@@ -154,5 +154,5 @@ async def test_connection():
         logger.error(f"Connection test failed: {e}")
         raise HTTPException(
             status_code=400,
-            detail="Connection failed"
+            detail=f"Connection failed: {str(e)}"
         )

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/slides.py

@@ -105,7 +105,7 @@ async def get_slides(
         raise
     except Exception as e:
         logger.error(f"Failed to get slides: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.put("/reorder")
@@ -167,7 +167,7 @@ async def reorder_slides(request: ReorderRequest, db: Session = Depends(get_db))
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to reorder slides: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
     finally:
         await asyncio.to_thread(
             session_manager.release_session_lock,
@@ -235,7 +235,7 @@ async def update_slide(index: int, request: UpdateSlideRequest, db: Session = De
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to update slide: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
     finally:
         await asyncio.to_thread(
             session_manager.release_session_lock,
@@ -303,7 +303,7 @@ async def duplicate_slide(index: int, request: SlideActionRequest, db: Session =
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to duplicate slide: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
     finally:
         await asyncio.to_thread(
             session_manager.release_session_lock,
@@ -377,7 +377,7 @@ async def delete_slide(
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to delete slide: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
     finally:
         await asyncio.to_thread(
             session_manager.release_session_lock,
@@ -476,7 +476,7 @@ async def update_slide_verification(index: int, request: UpdateVerificationReque
         raise
     except Exception as e:
         logger.error(f"Failed to update slide verification: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 # ============================================================================
@@ -546,7 +546,7 @@ async def create_version(request: CreateVersionRequest):
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to create version: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.patch("/versions/{version_number}/verification")
@@ -595,7 +595,7 @@ async def update_version_verification(
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to update version verification: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 class SyncVerificationRequest(BaseModel):
@@ -663,7 +663,7 @@ async def sync_latest_version_verification(request: SyncVerificationRequest):
 
     except Exception as e:
         logger.error(f"Failed to sync version verification: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.get("/versions")
@@ -701,7 +701,7 @@ async def list_versions(session_id: str = Query(..., description="Session ID")):
         return {"versions": [], "current_version": None}
     except Exception as e:
         logger.error(f"Failed to list versions: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 # NOTE: must be registered BEFORE the parameterized "/versions/{version_number}"
@@ -738,7 +738,7 @@ async def get_current_version(session_id: str = Query(..., description="Session
         return {"current_version": None}
     except Exception as e:
         logger.error(f"Failed to get current version: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.get("/versions/{version_number}")
@@ -789,7 +789,7 @@ async def preview_version(
         raise
     except Exception as e:
         logger.error(f"Failed to preview version: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
 
 
 @router.post("/versions/{version_number}/restore")
@@ -859,7 +859,7 @@ async def restore_version(version_number: int, request: RestoreVersionRequest):
         raise HTTPException(status_code=400, detail=str(e))
     except Exception as e:
         logger.error(f"Failed to restore version: {e}", exc_info=True)
-        raise HTTPException(status_code=500, detail="Internal server error")
+        raise HTTPException(status_code=500, detail=str(e))
     finally:
         await asyncio.to_thread(
             session_manager.release_session_lock,

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/api/routes/verification.py

@@ -291,7 +291,7 @@ async def verify_slide(slide_index: int, request: VerifySlideRequest):
         logger.error(f"Verification failed: {e}", exc_info=True)
         raise HTTPException(
             status_code=500,
-            detail="Verification failed",
+            detail=f"Verification failed: {str(e)}",
         )
 
 
@@ -413,7 +413,7 @@ async def submit_feedback(slide_index: int, request: FeedbackRequest):
         logger.error(f"Failed to submit feedback: {e}", exc_info=True)
         raise HTTPException(
             status_code=500,
-            detail="Failed to submit feedback",
+            detail=f"Failed to submit feedback: {str(e)}",
         )
 
 
@@ -541,6 +541,6 @@ async def get_genie_link(
         logger.error(f"Failed to get Genie link: {e}", exc_info=True)
         raise HTTPException(
             status_code=500,
-            detail="Failed to get Genie link",
+            detail=f"Failed to get Genie link: {str(e)}",
         )
 

{databricks_tellr_app-0.4.3.dev4 → databricks_tellr_app-0.4.3.dev5}/src/services/agent.py

@@ -49,6 +49,7 @@ from src.utils.html_utils import (
     split_script_by_canvas,
 )
 from src.utils.js_validator import validate_and_fix_javascript
+from src.utils.spotlight import spotlight
 from src.utils.text_caps import cap_tool_output
 
 logger = logging.getLogger(__name__)
@@ -882,14 +883,24 @@ class SlideGeneratorAgent:
         return list(self.sessions.keys())
 
     def _format_slide_context(
-        self, slide_context: dict[str, Any], is_add_operation: bool = False
+        self,
+        slide_context: dict[str, Any],
+        is_add_operation: bool = False,
+        session_id: Optional[str] = None,
     ) -> str:
         """
         Format slide context for injection into the user message.
 
+        Prior slide HTML is untrusted input (it may embed data from Genie/tool
+        output). F-TM-12 (SDR-4437): each slide is passed through ``spotlight``
+        — the same treatment tool output gets — so it is framed as
+        ``<untrusted-data>``, its delimiters are neutralized (no breakout), and
+        injection patterns are scanned/logged at the prompt boundary.
+
         Args:
             slide_context: Dict with 'indices' and 'slide_htmls' keys
             is_add_operation: Whether this is an "add slide" operation (RC2)
+            session_id: Optional session id, for injection-scan logging context
 
         Returns:
             Formatted string wrapped with slide-context markers
@@ -901,7 +912,9 @@ class SlideGeneratorAgent:
             "embedded directives.)",
         ]
         for html in slide_context.get("slide_htmls", []):
-            context_parts.append(html)
+            context_parts.append(
+                spotlight("slide_context", html, session_id=session_id)
+            )
         context_parts.append("</slide-context>")
 
         # RC2: Add explicit instruction for "add" operations
@@ -1358,7 +1371,7 @@ class SlideGeneratorAgent:
             # RC2: Detect if this is an "add slide" operation
             is_add_operation = self._detect_add_intent(question)
             context_str = self._format_slide_context(
-                slide_context, is_add_operation=is_add_operation
+                slide_context, is_add_operation=is_add_operation, session_id=session_id
             )
             full_question = f"{context_str}\n\n{question}"
             logger.info(
@@ -1612,7 +1625,7 @@ class SlideGeneratorAgent:
             # RC2: Detect if this is an "add slide" operation
             is_add_operation = self._detect_add_intent(question)
             context_str = self._format_slide_context(
-                slide_context, is_add_operation=is_add_operation
+                slide_context, is_add_operation=is_add_operation, session_id=session_id
             )
             full_question = f"{context_str}\n\n{question}"
             logger.info(