Client review fixes: bug fix, dead code, deduplication, CSS vars

Fork: 0

root / navi-1

Browse code Client review fixes: bug fix, dead code, deduplication, CSS vars - renderPreviewStrip: fix index-capture bug (splice by value, not index) - api.js: remove dead sendMessage (messaging goes via WebSocket only) - utils.js: extract shared esc() and timeLabel() from chat.js + sidebar.js - enrichSession: remove unnecessary async/Promise.all wrapper - style.css: thinking-card colors moved to CSS variables in :root - WsClient: remove unused #sessionId field Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> feature/navi-code master vmkdemo
1 parent aa34e55 commit 92ef26f9eede10771999d10f3ea155780c792fca Eugene Sukhodolskiy authored on 8 Apr

Browse code

- renderPreviewStrip: fix index-capture bug (splice by value, not index)
- api.js: remove dead sendMessage (messaging goes via WebSocket only)
- utils.js: extract shared esc() and timeLabel() from chat.js + sidebar.js
- enrichSession: remove unnecessary async/Promise.all wrapper
- style.css: thinking-card colors moved to CSS variables in :root
- WsClient: remove unused #sessionId field

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

feature/navi-code master vmkdemo

1 parent aa34e55 commit 92ef26f9eede10771999d10f3ea155780c792fca

Eugene Sukhodolskiy authored on 8 Apr

Patch

Unified Split

Showing 7 changed files

Ignore Space Show notes View client/js/api.js

Ignore Space Show notes View client/js/app.js

Ignore Space Show notes View client/js/chat.js

Ignore Space Show notes View client/js/sidebar.js

Ignore Space Show notes View client/js/utils.js 0 → 100644

Ignore Space Show notes View client/js/ws.js

Ignore Space Show notes View client/style.css

Show line notes below