Repository navigation

fix(db): transaction context manager + fixed parameterized updates (CORR-10) - #8

Open

KARLalpha4768 wants to merge 1 commit into

mainfrom

Open

fix(db): transaction context manager + fixed parameterized updates (CORR-10)#8
KARLalpha4768 wants to merge 1 commit into
mainfrom
fix/db

KARLalpha4768 commented Oct 25, 2026

Owner

Changes

CORR-10: get_db() is now a commit/rollback/close context manager used by all helpers; the dynamically built UPDATE clause is replaced with fixed parameterized statements.

Tests

tests/test_db_lifecycle.py: rollback and close on error, task state update persists with completed_at.

Not verified

Large diff in db.py (mostly re-indentation into with-blocks); worth a careful read of callers outside the test suite.

Verification

Full suite with the new tests passes locally on this branch (25 existing + new tests, pytest on Windows, Python venv).
py_compile clean; ruff fatal-error lint (E9,F63,F7,F82) clean.
Does not touch files owned by open PRs Move Cesium token, API key and DB DSN to environment variables #1-Fix HIGH correctness bugs CORR-01..05 #5.

Drafted by Codex from the medium/low review findings; split and re-verified per branch.


          fix(db): commit/rollback/close context manager and fixed parameterize…

7f8505d

…d updates (CORR-10)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet