oficina / memoria Public

Review: Calma in memoria open approved

On branch theme-calma

Claimed by claude~paul@keymail.dev 2 weeks ago

claude agent, operated by paul@keymail.dev operated by paul@keymail.dev session 5a634b30 · claude-code/2.1.280 · MacBookPro.localdomain

Adopts estilo's Calma (estilo decisions-table row 8, pinned 5b2ce83): own accent via theme.ForApp, Onest, Calma icons. Reviewer: Claude Fable 5.1 (one review across all seven app branches), result to be posted here.

claude agent, operated by paul@keymail.dev operated by paul@keymail.dev session 5a634b30 · claude-code/2.1.280 · MacBookPro.localdomain

Review by Claude Fable 5.1 (claude-fable-5-1), one pass across all seven app branches — this one: approve.

Verified across the set: each app serves theme.ForApp(<its own slug>) and a test pins it; the Onest link equals theme.FontPrefix and the route reaches FontHandler ahead of /static/ (ServeMux longest pattern / chi static-over-wildcard), signed-out; every app's CSP is rastrillo's default (default-src 'self' covers same-origin fonts), nothing inline added; distribute.py --check passes against estilo 5b2ce83 for all seven; go.mod churn is the estilo pin only.

Findings answered in the follow-up commits: a test pinning the layout's font link to theme.FontPrefix (slides, calendar, calls, fichas; memoria/docs/sheets already had equivalents); Slides' committed .pyc removed and Onest added to its credits page test; writetheme comment typos; fichas estilo made a direct requirement. Noted, not changed here: calls' make test doesn't pin GOTOOLCHAIN (pre-existing); docs' empty CI re-run commit.