Archibald312 stops the privacy guard from crying wolf

A small fix unblocks legitimate requests that carried no documents at all, and makes upload failures actually legible.

intakeinfrastructure

Archibald312's fork ran a data-privacy check on every request - sensible, except it was also firing when a request had no documents attached, blocking perfectly safe calls for no reason. The fix is the obvious one: no documents, no privacy risk, so let it through. That clears a class of false alarms that had been getting in the way during testing.

The rest of the commit is about seeing failures clearly. When document uploads or an AI prompt step failed, the fork was collapsing the error into a vague one-line message and throwing away the detail underneath. Now the full error - including the buried sub-errors you get when several uploads fail at once - lands in the server logs and the response. Less guessing when something breaks.

So what Worth a look for anyone running batches of document uploads, where failures tend to pile up and the real cause is easy to lose.

View this fork on GitHub →

Spotted something wrong? Or know the PR text has fresher detail than the writeup above?

Commits in this thread

1 commit from Archibald312/GordonOSS, oldest first. Source extracted verbatim from the harvested git log.

SHA Subject Author Date
15421457 fix: allow free-tier LLM in dev mode, improve error visibility (#3) Archibald312 2026-05-14 ↗ GitHub
commit body
* 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>

Capture this thread into my fork

Download a single Markdown prompt that tells Claude how to port every commit above into your working tree — adapting paths and structure to match your repo. Run it via claude -p < capture-thread-422.md from inside the repo you want the changes in.

⬇ Download capture-thread-422.md