fix(types): Session callback params does not include user and token

[nbifrye]

☕️ Reasoning

In session callback, argument type does not contain user and token

callbacks: {
  session: async ({ session, user }) => {
    if(session.user){
      session.user.id = user.id;
    }
    return session;
  }
},

Property 'user' does not exist on type '({ session: Session; user: AdapterUser; } | { session: Session; token: JWT; }) & { newSession: any; trigger?: "update" | undefined; }'.

This seems to be related to microsoft/TypeScript#14094. Look at this TS Playground example.

type Obj1 = { a: string, b: string }
type Obj2 = { a: string, c: string }

const func = (args: Obj1 | Obj2) => {
    args.b = ""
         ^
         Property 'b' does not exist on type 'Obj1 | Obj2'.
           Property 'b' does not exist on type 'Obj2'.
}

Using { session, user?, token? } instead of { session, user } | { session, token } seems to fix this.

🧢 Checklist

• Documentation
• Tests
• Ready to be merged

🎫 Affected issues

Fixes: #9633

📌 Resources

• Security guidelines
• Contributing guidelines
• Code of conduct
• Contributing to Open Source

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
auth-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 17, 2024 0:02am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
next-auth-docs	⬜️ Ignored (Inspect)	Visit Preview		Jan 17, 2024 0:02am

[aakash14goplani]

Same case with SvelteKitAuth.

cc: @ndom91 and @ThangHuuVu

[balazsorban44]

Closed via #9756. Needs more work in the future. See #9756 (review)