[codex] Remove unused Supabase request auth helper
From the PR description
Summary
Removes the unused getUserIdFromRequest helper from backend/src/lib/supabase.ts.
Why
The helper is not referenced anywhere in the repository. Authenticated backend routes use the existing Express middleware flow instead, so keeping this helper adds dead code and an extra request-style auth path that is no longer used.
Changes
- Deleted the unused request JWT helper from backend/src/lib/supabase.ts
- Left createServerSupabase unchanged
Testing
- rg "getUserIdFromRequest" . returns no matches
- git diff --check passes
Note: I could not run npm run build --prefix backend in this local shell because the checkout does not have dependencies installed.
Our analysis
Remove unused Supabase request-auth helper — read the full analysis →
Think the analysis missed something the PR description covers?
Commits in this PR (3)
| SHA | Subject | Author | Date | |
|---|---|---|---|---|
a8cf33ae | Remove unused Supabase request auth helper | mahaprasad nanda | 2026-06-19 | ↗ GitHub |
3bea53ff | Merge remote-tracking branch 'origin/main' into codex/remove-dead-supabase-auth-helper | willchen96 | 2026-07-26 | ↗ GitHub |
80d14b86 | test: drop mocks of removed getUserIdFromRequest helper | willchen96 | 2026-07-26 | ↗ GitHub |
commit bodyThe integration tests landed on main after this branch was cut and mocked the helper this PR deletes. Vitest factory mocks don't validate against the real module shape, so the extra key was harmless but misleading. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> | ||||
Capture this PR into my fork
Download a Markdown prompt that tells Claude how to port every
commit in this PR into your working tree. Run it via
claude -p < capture-pull-186.md from
inside the repo you want the changes in.