explicitly closes SQLite connections used by database dump, comparison, and restore so Windows can replace the live database immediately
sweeps legacy orphaned message vectors at store startup while retaining main's existing single-statement delete_conversation cleanup (the redundant _delete_message_vectors helper is intentionally omitted)
separates Ollama's server bind address from the client URL, normalizing wildcard binds such as 0.0.0.0:11434 to a connectable loopback URL
includes Ollama's error response in raised HTTP failures and strips inline <think> reasoning from complete and streamed replies
fix(ollama): normalize hosts, errors, and reasoning output
Validation
focused runtime suites: 45 passed, 1 skipped
broad Python suite on Windows: 84 passed, 1 skipped, with 3 current-main Windows portability assertions deselected (two POSIX file-mode assertions and one path-separator assertion)
frontend lint and production build passed
Review context
This is the runtime half split out of the closed preview PR #4. It is intentionally independent of the preview feature and is intended to land first.
## Summary
- explicitly closes SQLite connections used by database dump, comparison, and restore so Windows can replace the live database immediately
- sweeps legacy orphaned message vectors at store startup while retaining `main`'s existing single-statement `delete_conversation` cleanup (the redundant `_delete_message_vectors` helper is intentionally omitted)
- separates Ollama's server bind address from the client URL, normalizing wildcard binds such as `0.0.0.0:11434` to a connectable loopback URL
- includes Ollama's error response in raised HTTP failures and strips inline `<think>` reasoning from complete and streamed replies
## Commit structure
1. `fix(sync): close SQLite handles before restore`
2. `fix(memory): sweep legacy orphaned message vectors`
3. `fix(ollama): normalize hosts, errors, and reasoning output`
## Validation
- focused runtime suites: 45 passed, 1 skipped
- broad Python suite on Windows: 84 passed, 1 skipped, with 3 current-`main` Windows portability assertions deselected (two POSIX file-mode assertions and one path-separator assertion)
- frontend lint and production build passed
## Review context
This is the runtime half split out of the closed preview PR #4. It is intentionally independent of the preview feature and is intended to land first.
Repair vector rows left behind by older databases at store startup. Keep the existing single-statement delete path from main and avoid reintroducing the redundant batched helper.
Separate bind and client addresses, include Ollama's response body in HTTP failures, and strip inline <think> blocks from complete and streamed replies.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
main's existing single-statementdelete_conversationcleanup (the redundant_delete_message_vectorshelper is intentionally omitted)0.0.0.0:11434to a connectable loopback URL<think>reasoning from complete and streamed repliesCommit structure
fix(sync): close SQLite handles before restorefix(memory): sweep legacy orphaned message vectorsfix(ollama): normalize hosts, errors, and reasoning outputValidation
mainWindows portability assertions deselected (two POSIX file-mode assertions and one path-separator assertion)Review context
This is the runtime half split out of the closed preview PR #4. It is intentionally independent of the preview feature and is intended to land first.