Hygiene backlog: P3 leftovers from the #26 audit #29

New issue

Open

opened 2026-07-27 11:07:12 -04:00 by cmoriarty · 0 comments

cmoriarty commented

2026-07-27 11:07:12 -04:00

Owner

Low-priority leftovers documented in the #26 audit, parked here when that ticket closed:

assetq.submit_oneoff daily-ref race: count(*)+1 inside the transaction without a lock can mint duplicate #N handles on concurrent submits (cosmetic today; matters for the multi-user studio, v3).
Postgres connection per call in assetq/ledger + 1s SSE polls = connection churn; fine single-host, a small pool is the eventual fix.
cli _watch_poll loops forever if the orchestrator disappears permanently mid-watch — deserves a give-up timeout with a resume hint.
audio-server ships /tts, /vad, /transcribe that trog never calls (vendored surface) — document as extras or prune from the image to slim it.
Optional CI coverage: the job image lacks ffmpeg, so the 4 sheet-assembly tests always skip in CI (they run in make test locally). One apt line (~25s/run) buys them.
pixel_post._cleanup_orphans is a per-pixel Python loop — fine at 96px, vectorize only if grids ever grow.

Low-priority leftovers documented in the #26 audit, parked here when that ticket closed: - [ ] `assetq.submit_oneoff` daily-ref race: `count(*)+1` inside the transaction without a lock can mint duplicate #N handles on concurrent submits (cosmetic today; matters for the multi-user studio, v3). - [ ] Postgres connection per call in `assetq`/`ledger` + 1s SSE polls = connection churn; fine single-host, a small pool is the eventual fix. - [ ] `cli _watch_poll` loops forever if the orchestrator disappears permanently mid-watch — deserves a give-up timeout with a resume hint. - [ ] audio-server ships /tts, /vad, /transcribe that trog never calls (vendored surface) — document as extras or prune from the image to slim it. - [ ] Optional CI coverage: the job image lacks ffmpeg, so the 4 sheet-assembly tests always skip in CI (they run in `make test` locally). One apt line (~25s/run) buys them. - [ ] `pixel_post._cleanup_orphans` is a per-pixel Python loop — fine at 96px, vectorize only if grids ever grow.

cmoriarty referenced this issue

2026-07-27 11:07:49 -04:00

Clean up project to professional software engineering standards #26

cmoriarty added this to the v1 - single user autonomous milestone

2026-07-27 11:10:18 -04:00