[P2.13] `get_batch`/`list_batches`/`get_run`/`list_runs` leak SQLite connections on query failure

[frankbria]

Severity: low | Category: correctness | Phase: P2.13

Problem

These hot-path readers conn.close() only on the happy path; any OperationalError (e.g. "database is locked" under parallel load) leaks the connection.

Evidence

codeframe/core/conductor.py:750

Acceptance criteria

Wrap connection use in try/finally (or a contextmanager), matching the tasks.py pattern.

Dependencies

None

---

_{Filed from the SaaS launch-readiness audit. Atomic: one developer, one session. Work order: strictly P0.1 → P3.12 (no forward dependencies).}

[coderabbitai]

🔗 Related PRs

#686 - fix(core): add busy_timeout + WAL to workspace SQLite connections (#648) [closed]
#688 - fix(core): release tasks.py DB connections on exception + retain fire-and-forget auto-close task (#650) [closed]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!