Skip to content

FeedEcho v1.10.1 — code review fixes

Choose a tag to compare

View all tags
@jcrabapple jcrabapple released this 22 Aug 21:08
· 58 commits to master since this release
v1.10.1
9943442

Fixes from a Kimi K3 code review of v1.10.0.

  • Legacy date:iso / date:short normalization now only rewrites tokens inside {{ }} expressions, so literal template prose is left untouched
  • Stored templates are revalidated at startup; echoes whose legacy templates no longer parse log a warning instead of failing silently at render time
  • Render failures now record the Jinja2 error in the post history (gave_up rows show the real reason)
  • README documents safe list access (| first, | default) — indexing into empty/missing lists raises at render time
  • /api/preview now runs as a sync route so blocking feed fetches use the threadpool instead of stalling the event loop
  • Preview button alerts when a form has no feed selector instead of silently doing nothing
  • Tests hardened against ambient FEEDCHO_AUTH_TOKEN; echo-form tests seed real accounts
  • 275 tests (was 269)
Assets 2
Loading