Feature/oauth profile completion

[codeunia-dev]

Summary by CodeRabbit

• New Features
  • Post-auth redirects now preserve the user’s original destination via a returnUrl parameter, improving continuity after login and profile completion.
  • Profile completion flow respects an optional returnUrl, sending users back to where they intended to go once complete.
  • Safe URL encoding applied to returnUrl to prevent malformed redirects.
  • Sensible default remains in place when no returnUrl is provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
codeunia	Building	Preview	Comment	Sep 15, 2025 7:59am

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Redirects from the auth callback to the complete-profile page now include a returnUrl query param. The complete-profile page reads this param and uses it for subsequent redirects, replacing hard-coded destinations. Initialization adjustments introduce searchParams and remove a duplicate router instance. Exported/public entities remain unchanged.

Changes

Cohort / File(s)	Summary of Changes
OAuth Callback Redirect Propagation app/auth/callback/page.tsx	Updated all redirects to /complete-profile to append ?returnUrl=${encodeURIComponent(returnUrl)}. No other navigation or error handling logic changed.
Complete Profile Dynamic Return app/complete-profile/page.tsx	Added useSearchParams; initialized returnUrl (default /protected/dashboard). Replaced hard-coded redirects with navigation to returnUrl. Removed duplicate router initialization. No API/signature changes.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor U as User
    participant OP as OAuth Provider
    participant AC as Auth Callback Page
    participant CP as Complete Profile Page
    participant APP as App (Protected Area)

    U->>OP: Authenticate
    OP-->>AC: Redirect with code/state (+ optional returnUrl)
    AC->>AC: Process OAuth + profile checks
    alt Profile incomplete or needs retry
        AC->>CP: router.replace(/complete-profile?returnUrl=ENC(returnUrl))
        note right of AC: returnUrl preserved via query param
    else Profile complete
        AC->>APP: Continue normal flow (unchanged)
    end

    CP->>CP: Read searchParams, derive returnUrl (default /protected/dashboard)
    alt Existing profile already complete
        CP->>APP: router.replace(returnUrl)
    else User completes profile
        CP->>APP: router.replace(returnUrl)
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: Add OAuth profile completion flow #241 — Introduced redirects to /complete-profile and the Complete Profile page; this PR builds on that by propagating and honoring returnUrl across the same flows.

Poem

A rabbit hops through routes so bright,
With returnUrl tucked snug and tight.
From callback burrow to profile glen,
We bounce back home, right where we’ve been.
Encode, replace—then off we dash,
To dashboards warm, in one quick flash. 🐇✨

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/oauth-profile-completion

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 36dad0f and 97d81ad.

📒 Files selected for processing (2)

• app/auth/callback/page.tsx (5 hunks)
• app/complete-profile/page.tsx (4 hunks)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}