Technical discussions and code reviews
Heads up — upgrading to Next.js 16 this morning. PR is up for review.
Nice. Any breaking changes we should watch for?
Mostly the new async params in route handlers. I've already migrated our routes.
Also removed the legacy middleware — the new auth pattern is way cleaner.
Can someone take a look at PR #142? Auth redirect is acting weird on the callback route.
On it. Kairo already flagged it in the code review thread — looks like a missing callback param.
That was fast!
While we're on auth — we should add rate limiting to the login endpoint. I'll create a task.
Fix is up. The redirect was missing the callback param when the session expired mid-flow.
Also added a regression test so this doesn't sneak back in.
Approved ✅ Great fix, merging now.
Is the staging deploy pipeline still green? I want to preview the new wizard.
Yep, all green. Preview URL is in the PR description.
Rate limit task created: ENG-287. Targeting next sprint.