MC-3579

Luci dispatching this ticket to Larry (background worker via SSH).

Larry: Planning — Tier 1 fix, replacing 14 print() calls with debug_log() in app.pyPROGRESS: Building — replacing 14 print() calls with debug_log()PROGRESS: Testing — verifying no print() calls remain and syntax is validPROGRESS: Building — committing changesPROGRESS: Done — replaced 14 print() error logs with debug_log() in app.py

Completed on Larry: Replaced all 14 `print()` calls in error paths with `debug_log()`. Twelve were redundant prints already paired with a `debug_log()` above (removed), and two standalone prints (line 580 Dropbox URL check, line 2083 AI generation failure) were converted to `debug_log()`. Syntax verified, committed as e2a6b62.

Larry: Triggering post-completion code review...

Code review (Larry reviewer via claude/opus): mid-line at `get_precomputed_scores` (`return dat`); I'm reviewing only the visible portion. If the rema ining hunks follow the same pattern (drop duplicate ` print`, or convert standalone `print` to `debug_log(..., level="error")`), no concern. MINOR: app.py: 2078 — `debug_log(f"AI generation failed: {content}", level="error")` interpolates full AI response into logs; fine but could be noisy. Not wor th fixing here. All visible changes are consistent with the ticket: route error output through the centralized `debug_log()` system instead of bypassing it with `print()`. No behavior change on happy paths, no security/data conc erns, no test breakage. VERDICT: PASS Replaces redundant `print()` calls with `debug_log()` (or removes them when `debug_log()` was already called above). MINOR: app.py — diff is truncated mid-line at `get_precomputed_scores` (`return dat`); I'm reviewing only the visible portion. If the remaining hunks follow the same pattern (drop duplicate `print`, or convert standalone `print` to `debug_log(..., level="error")`), no concern. MINOR: app.py:2078 — `debug_log(f"AI generation failed: {content}", level="error")` interpolates full AI response into logs; fine but could be noisy. Not worth fixing here. All visible changes are consistent with the ticket: route error output through the centralized `debug_log()` system instead of bypassing it with `print()`. No behavior change on happy paths, no security/data concerns, no test breakage. VERDICT: PASS

Larry shipped with clean code review. Auto-dispatching Luci-reviewer (verdict: clean). Next dispatch tick will spawn a single-session reviewer worker to verify, run Tessa subagent inline if UI-touching, merge the PR, and close.

Completed: DONE: Verified Larry's MC-3579 fix on SafairBru host — commit e2a6b62 replaces all 14 `print()` error logs in app.py with `debug_log()` (12 redundant prints removed, 2 standalone converted). No `print()` calls remain, `py_compile` passes syntax check, code review verdict PASS. Pushed e2a6b62 to origin/main (was ahead 1, unpushed). Pure logging change, no UI — Tessa not required.