mikeboone Claude Sonnet 4.6 commited on
Commit
4e1b8cd
·
1 Parent(s): 01e294d

docs: update sprint doc — mark state isolation fix complete

Browse files

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Files changed (1) hide show
  1. sprint_2026_03.md +6 -2
sprint_2026_03.md CHANGED
@@ -77,9 +77,13 @@ This sprint covers hardening, settings, and new capabilities before that happens
77
  - `ts_username` in SETTINGS_SCHEMA → pre-fills on load via `load_settings_on_startup`
78
  - `liveboard_name`, `default_use_case`, `default_llm` all pre-fill on startup
79
  - company no longer pre-filled (chat-driven) — working as intended
80
- - [~] **State Isolation Fix** ⚠️ PARTIAL session logger fixed, prompt logger still broken
81
  - Session logger: stored on `self._session_logger` (per controller instance) ✅
82
- - Prompt logger: `reset_prompt_logger(session_id)` is called but `_prompt_logger` is still a module-level singleton — Session 2's reset overwrites Session 1's logger; **NOT fixed**
 
 
 
 
83
  - `inject_admin_settings_to_env()` still in use (deferred — requires cdw_connector refactor)
84
 
85
  ---
 
77
  - `ts_username` in SETTINGS_SCHEMA → pre-fills on load via `load_settings_on_startup`
78
  - `liveboard_name`, `default_use_case`, `default_llm` all pre-fill on startup
79
  - company no longer pre-filled (chat-driven) — working as intended
80
+ - [x] **State Isolation Fix** both loggers now fully per-session
81
  - Session logger: stored on `self._session_logger` (per controller instance) ✅
82
+ - Prompt logger: global singleton removed; `PromptLogger` instantiated directly per controller
83
+ - `logged_completion()` and `log_researcher_call()` accept `logger=` param; if None, skips log
84
+ - `ThoughtSpotDeployer.prompt_logger` set from controller after construction
85
+ - `SmartDataAdjuster` accepts `prompt_logger=` constructor param
86
+ - Prompt log tab timer reads from `controller._prompt_logger` via `gr.State`
87
  - `inject_admin_settings_to_env()` still in use (deferred — requires cdw_connector refactor)
88
 
89
  ---