fix: resolve 15 code review findings (5 critical, 10 warnings)#12

Merged

W00DSRULES merged 18 commits into

mainfrom

fix/code-review-findings

May 9, 2026

Commits on May 8, 2026

docs: generate and verify full project documentation suite

imer
and
claude
committed
fix: CR-01 add memory_guidance to allowed blackboard entry types
imer
committed
fix: CR-02 WR-01 WR-02 fix blackboard race, log write failures, remove dead try/except
imer
committed
fix: CR-03 cancel in-flight asyncio task on DELETE /tasks/{task_id}
imer
committed
fix: CR-04 switch RedisSessionStore to redis.asyncio for non-blocking I/O
imer
committed
fix: CR-05 remove duplicate timeout kwarg from OpenAI client call
imer
committed
fix: WR-03 use get_settings() for embedding cache dir, reset in tests
imer
committed
fix: WR-04 replace redis.keys() with scan_iter to avoid O(N) blocking scan
imer
committed
fix: WR-05 include input token cost in _estimate_cost
imer
committed
fix: WR-06 remove dead activate_synthesizer guard in budget_tight branch
imer
committed
fix: WR-07 add bounded LRU eviction to _tasks dict
imer
committed
fix: WR-08 replace datetime.utcnow() with datetime.now(timezone.utc)
imer
committed
fix: WR-09 remove dead _is_sentence_terminator function
imer
committed
fix: WR-10 replace print to stderr with structured logger in corpus ingest
imer
committed
fix: update test mock and await for aioredis migration in CR-04
imer
committed

Commits on May 9, 2026

chore: add code review report and uv lockfile

imer
and
claude
committed
fix(lint): move logger assignment below imports in thesis_flow.py

imer
and
claude
committed
fix(lint): apply black formatting to sessions/store.py and tools/mcp/rag.py

imer
and
claude
committed