Skip to content

feat: add transcript upload and reverse-proxy base path support - #34

Open
PangYiMing wants to merge 2 commits into
S40911120:mainfrom
PangYiMing:feat/upload-jsonl-and-base-path
Open

PangYiMing wants to merge 2 commits into
S40911120:mainfrom
PangYiMing:feat/upload-jsonl-and-base-path

Conversation

@PangYiMing

Copy link
Copy Markdown

Summary

  • Add a streaming multipart endpoint for uploading one .jsonl transcript into the uploads project.
  • Sanitize uploaded filenames, write atomically, index the new session, and return its session ID.
  • Add an Upload action to the session list that waits for indexing and opens the uploaded session.
  • Make frontend API, WebSocket, favicon, and logo URLs honor Vite base paths for reverse-proxy deployments.
  • Document the upload behavior and add coverage for filename handling, multipart submission, and base paths.

Behavior

Uploaded files are limited by RECENSA_MAX_UPLOAD_BYTES (256 MiB by default), stored under RECENSA_PROJECTS_DIR/uploads, and remain available in the session history alongside source sessions.

Validation

  • cd backend && npm run check — 37 tests passing
  • cd frontend && npm run check — 31 tests passing
  • VITE_BASE_PATH=/claude/review-jsonl/ npm run build — production build passing

Signed-off-by: ympang <ympang@trip.com>
Signed-off-by: ympang <ympang@trip.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants