fix: ruff violations in api/server.py and test_db_lock_guard.py

- Remove unused get_decision_log import from server.py (F401)
- Remove unused open_db import from test_db_lock_guard.py (F401)
- Fix import sort order in test_db_lock_guard.py (I001)
- Remove unused original_monotonic variable (F841)
- Tighten pytest.raises(Exception) to pytest.raises(duckdb.Error) (B017)

Author: kbichave

packages/quant_pod/api/server.py

@@ -46,7 +46,6 @@
 
 from quant_pod.audit.decision_log import (
     extract_indicator_attributions,
-    get_decision_log,
     get_decision_log_readonly,
 )
 from quant_pod.audit.models import AuditQuery

tests/quant_pod/test_db_lock_guard.py

@@ -17,11 +17,9 @@
 
 import duckdb
 import pytest
-
 from quant_pod.db import (
     _connect_with_lock_guard,
     _is_process_alive,
-    open_db,
     open_db_readonly,
     reset_connection,
     reset_connection_readonly,
@@ -123,10 +121,6 @@ def test_stale_lock_deadline_exceeded_raises(self):
         def always_lock(_path, **_kw):
             raise _lock_exc(dead_pid)
 
-        import time as _time
-
-        # Make monotonic always return a value past the deadline on first check
-        original_monotonic = _time.monotonic
         with (
             patch("quant_pod.db.duckdb.connect", side_effect=always_lock),
             patch("quant_pod.db._is_process_alive", return_value=False),
@@ -194,8 +188,8 @@ def test_succeeds_when_file_exists(self, tmp_path):
         result = ro_conn.execute("SELECT COUNT(*) FROM foo").fetchone()
         assert result is not None
 
-        # Writes must fail
-        with pytest.raises(Exception):
+        # Writes must fail — DuckDB raises an error on read-only connections
+        with pytest.raises(duckdb.Error):
             ro_conn.execute("INSERT INTO foo VALUES (1)")
 
     def test_memory_path_returns_writable_connection(self):