Cache per-request transaction reads (badge reloads full JSON every page) #33

New Issue

tonym · 2026-06-06T19:58:15Z

tonym commented

2026-06-06 19:58:15 +00:00

Severity: medium
Category: tech-debt
Location: app.py:166-170

Problem

inject_unreviewed_count context processor parses the entire transactions.json on every request, including pages without the badge, and duplicates loads done elsewhere in the same request.

Suggested fix

Memoize on flask.g per request.

Filed by automated code review.

**Severity:** medium **Category:** tech-debt **Location:** `app.py:166-170` ### Problem `inject_unreviewed_count` context processor parses the entire transactions.json on every request, including pages without the badge, and duplicates loads done elsewhere in the same request. ### Suggested fix Memoize on flask.g per request. _Filed by automated code review._

tonym added the

tech-debt

severity:medium

labels 2026-06-06 19:58:15 +00:00

tonym referenced this issue from a commit

2026-06-06 21:43:06 +00:00

Migrate transactions to SQLite (self-contained, on the volume)

Sign in to join this conversation.