[voidstackloop]

Summary

• close failed read-only probes before recovery snapshots or renames the database
• close the primary SQLite connection when initialization throws before singleton ownership
• upgrade legacy call_logs columns before schema indexes reference them

Validation

• tests/unit/db-core-init.test.ts: 14/14 passed on Windows
• npm run typecheck:core
• npm run check:cycles
• prettier --check src/lib/db/core.ts
• ESLint reports only the three pre-existing unused-symbol errors in core.ts

⚠️ base-red inherited: #13319

Fixes #13303

[diegosouzapw]

Thanks for tracking down the Windows handle-leak in getDbInstance() — closing
the probe/primary connections on every failure path (not just the happy path) is
exactly right for #13303. Ran your changed test file at this PR's head:
tests/unit/db-core-init.test.ts — 14/14 pass. One ask before merge: could you
add a changelog.d/fixes/ fragment for #13303 (repo convention, missing from
this diff)? Everything else looks solid.

[diegosouzapw]

Thanks @voidstackloop — merging via the release merge-train. Validated in local merge-train /tmp/mt-train3b.log on 192.168.0.113 @ train tip 408e2128791696a18966681953136fc96aeb99b0 (32 PRs boarded): static gates green; full test:unit 40694 tests, 17 failing — every one reproduces on the pure release tip (base-red sweep list), zero new reds. Merged --admin per merge-gates §4/§7.