refactor: one handleDocumentUpload for projects, documents, library

↗ view on GitHub · Eli Ziff · 2026-07-27 · 103b8b3d

routes/projects.ts carried a 140-line near-verbatim twin of the
documents.ts upload handler. The documents.ts version is a superset:
library_kind defaults to 'file' in both (explicit vs column default)
and the only response delta is an additive folder_id: null. Projects
now imports the shared handler; four upload-only imports pruned.

Verified: tsc clean; projects.routes suite green. The
localMatter.routes failure in the same run predates this change - it
expects the old attached-document message format while another
session's uncommitted contextBuilders.ts WIP emits the new one.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QUsMXzBsEJ27DjJVPSfW5o
Repository eliziff/Beaver
Author Eli Ziff <eliasziff@gmail.com>
Authored
Parents fb4d47c1
Stats 1 file changed , +3 , -147
Part of Internal cleanup, dead-code removal, and the canonical route builder

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

⬇ Download capture-commit-103b8b3d.md