Perf/disable extended thinking

[yonadavGit]

No description provided.

[gitvelocity-reviewer]

📊 Code Quality Score: 31/100

52 × 0.6 = 31.2 → 31

Category	Score	Factors
🔭 Scope	12/20	8+ files across frontend/backend/config; new SSE event type as public API; config cleanup cross-cutting
🏗️ Architecture	10/20	Clean callback protocol extension through agent stack; thinking disabled is behavioral change; SSE event routing added
⚙️ Implementation	11/20	Streaming delta extraction with dual-path logic; closure-based timing with nonlocal; Svelte 5 streaming state; SSE event type routing
⚠️ Risk	8/20	Model version unpinned (was date-pinned); thinking explicitly disabled; XSS surface with streaming HTML; new SSE event additive/backward-compatible
✅ Quality	9/15	Good unit tests for sdk_runner and metrics_mapper; missing orchestrator integration tests; no frontend tests; config cleanup well-executed
🔒 Perf / Security	2/5	scrollToBottom on every delta may cause perf issues; XSS risk with streaming HTML not addressed; no sanitization verification

Was this score accurate? 👍 Yes · 👎 No

_{Scored by GitVelocity · How are scores calculated?}

[coolify-sefaria-github]

The preview deployment for sefaria/ai-chatbot:server is ready. 🟢

Open app | Open Build Logs | Open Application Logs

Last updated at: 2026-07-07 14:09:01 CET