Fix Gemini thinking blocks, resync reliability, and startup UX - #58

Merged

wesm merged 45 commits into

mainfrom

nicer-gemini-thinking

Feb 26, 2026

Commits on Feb 26, 2026

fix: render Gemini thinking blocks as collapsible blocks

wesm
and
claude
committed
fix: ResyncAll now replaces message content instead of append-only

wesm
and
claude
committed
fix: avoid full FTS5 rewrite for unchanged sessions during resync

wesm
and
claude
committed
fix: drop+rebuild FTS index during resync instead of per-row heuristic

wesm
and
claude
committed
fix: remove opencode log that polluted progress line output

wesm
and
claude
committed
fix: serialize SyncPaths and SyncSingleSession with syncMu

wesm
and
claude
committed
Surface FTS rebuild failure in resync response, verify search after resync

wesm
and
claude
committed
fix: order Gemini message parts chronologically (thinking → content → tools)

wesm
and
claude
committed
fix: update syncMu comment, persist resync warnings to lastSyncStats

wesm
and
claude
committed
fix: bulk-delete messages per batch during resync instead of per-session

wesm
and
claude
committed
feat: add debug log file and DB operation timing

wesm
and
claude
committed
style: go fmt whitespace fixes in test files

wesm
and
claude
committed
fix: fall back to per-session replace on bulk delete failure, quiet normal sync

wesm
and
claude
committed
fix: truncate debug.log on startup when >10MB, add log tests

wesm
and
claude
committed
feat: resync via fresh database build instead of in-place mutation

wesm
and
claude
committed
fix: skip symlinks in log truncation, fix test portability

wesm
and
claude
committed
fix: harden resync DB swap against failures

wesm
and
claude
committed
fix: check errors in symlink test, skip when unsupported

wesm
and
claude
committed
fix: track sync failures, abort resync when failures exceed synced

wesm
and
claude
committed
fix: only skip symlink test for permission errors, fatal otherwise

wesm
and
claude
committed
fix: use file-level counters in resync abort guard, broaden skip errnos

wesm
and
claude
committed
fix: add fork+failure abort test, fix SyncStats comment accuracy

wesm
and
claude
committed
fix: log to file only, simplify sync summaries

wesm
and
claude
committed
fix: keep fatal errors on stderr, fix test fixtures

wesm
and
claude
committed
fix: stop thinking regex from swallowing Gemini response text

wesm
and
claude
committed
fix: add [/Thinking] end markers, show session ID for all agents

wesm
and
claude
committed
fix: two-pass thinking regex to avoid premature termination

wesm
and
claude
committed
fix: use atomic.Pointer for DB connections, close before rename

wesm
and
claude
committed
test: add HTTP integration tests for resync DB swap

wesm
and
claude
committed
fix: abort ResyncAll on CloseConnections failure, harden tests

wesm
and
claude
committed
fix: correct assertion logic and reader barrier in engine tests

wesm
and
claude
committed
fix: prevent insight loss during resync, persist stats on all paths

wesm
and
claude
committed
fix: close newDB on failure path, detect FTS regression in test

wesm
and
claude
committed
fix: align session counts across analytics, stats, and session list

wesm
and
claude
committed
test: verify session count consistency across all three endpoints

wesm
and
claude
committed
test: e2e test for session count consistency across all views

wesm
and
claude
committed
fix: assert exact fixture count in e2e session consistency test

wesm
and
claude
committed
fix: abort resync when discovery returns zero files

wesm
and
claude
committed
fix: abort resync on insights copy failure, harden empty-discovery guard

wesm
and
claude
committed
fix: Windows log file cleanup, e2e thinking block locator

wesm
and
claude
committed
fix: emptyDiscovery guard must account for OpenCode sessions

wesm
and
claude
committed
fix: use file-backed session count for empty-discovery guard

wesm
and
claude
committed
fix: pin connection for ATTACH DATABASE in CopyInsightsFrom

wesm
and
claude
committed
docs: add review guidelines for TOCTOU, schema nulls, control flow

wesm
and
claude
committed
fix: wait for filtered results before checking scroll position

wesm
and
claude
committed