fix: allow free-tier LLM in dev mode, improve error visibility (#3)

↗ view on GitHub · Archibald312 · 2026-05-14 · 15421457

* fix: allow free-tier LLM in dev mode and improve error visibility

- freeTierGuard: skip allowlist check when no documents are attached -
  document-free calls (e.g. column prompt generation) carry no
  customer-data risk so blocking them on a missing allowlist was wrong
- tabular route: surface the real exception message in the 502 response
  instead of swallowing it in a bare catch, making LLM failures debuggable
- documents/projects upload routes: expand AggregateError to show all
  sub-errors and log the full exception server-side

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: validate allowlist before skipping document check in freeTierGuard

The early return for no-document calls must come after the allowlist
presence check, not before. Missing FREE_TIER_FIXTURE_ALLOWLIST should
still throw even when no documents are attached (config validation), but
once the allowlist is confirmed present, doc-free calls can pass through.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Repository Archibald312/GordonOSS
Author Archibald312 <129438073+Archibald312@users.noreply.github.com>
Authored
Parents eb461a39
Stats 4 files changed , +22 , -5
Part of Free-tier LLM guard usability fixes

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-15421457.md from inside the repo you want the change in.

⬇ Download capture-commit-15421457.md