Review fixes for #74: PGRST204 degrade, email-shared bucket, route hardening

↗ view on GitHub · sloth-ninja · 2026-08-05 · 78b04a46

Review should-fixes: isMissingColumnOrTable now accepts PostgREST's
PGRST204 (UPDATE payloads naming a missing column bypass Postgres and are
rejected from the schema cache - the live pre-migration path; DURABLE_
LESSONS 2026-08-05 appended); TemplateList gains the spec'd `shared`
bucket (workflow_shares email match, own/tombstoned/is_system excluded)
and listTemplates takes the caller's email. Nits: uuid front-guard on all
:id routes (uniform 404, no 22P02-as-500, catches the /admin typo),
tombstone predicate on admin revert, >=1 column required, clearer
non-object column message, control characters stripped from column names
and tag values. Suite 725 green (53 seam + 25 route).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Repository Sloth-ninja/JessicaOSS
Author sloth-ninja <ezanahaddis@gmail.com>
Authored
Parents 02522115
Stats 6 files changed , +415 , -87
Part of Firm-shared tabular review templates

Capture this commit into my fork

Download a Markdown prompt that tells Claude how to port this exact commit into your working tree. Run it via claude -p < capture-commit-78b04a46.md from inside the repo you want the change in.

⬇ Download capture-commit-78b04a46.md