chore: remove dead Supabase middleware - #248
Conversation
|
@lie75922-cpu is attempting to deploy a commit to the Diego Demarziani's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks for this, and for being explicit about what you could not verify. That is the right call, and the draft status made it obvious where things stood. CI is unblocked now and the test workflow passes. The Vercel check needs deploy authorization for fork PRs, so you can ignore that one, it is not on you. One thing to fold in before marking this ready:
Worth knowing, that code sample was already stale before your PR. The real Your |
What changes
Deletes
src/lib/supabase/middleware.ts, which exports an unused duplicate of the live session refresh logic insrc/proxy.ts.Why
The dead helper can be mistaken for the active Next.js middleware even though no source file imports it. Removing it leaves
src/proxy.tsas the single implementation.Closes #206
Test plan
rg "supabase/middleware|updateSession" srcreturns no source references after deletiongit diff --checknpm run lintnpx tsc --noEmitnpm testNotes for the reviewer
The full dependency-based checks could not be run locally because two
npm ciattempts reached the 20-minute download limit while waiting on the package registry. CI should run the remaining checks.