fix(ts): show user and token in session callback

[ajstars1]

☕️ Reasoning

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

Before code updation 👇

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

Type Intersection: The original code incorrectly used a union type (|), potentially leading to type mismatches. The updated code employs an intersection type (&) to guarantee that both user and token properties are always available within the session callback, regardless of the authentication strategy.

After code updation 👇

🧢 Checklist

• Documentation
• Tests
• Ready to be merged

🎫 Affected issues

Fixed: #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	❌ Failed (Inspect)			Jan 24, 2024 3:32pm

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
next-auth-docs	⬜️ Ignored (Inspect)	Visit Preview		Jan 24, 2024 3:32pm

[balazsorban44]

Type Intersection: The original code incorrectly used a union type (|), potentially leading to type mismatches. The updated code employs an intersection type (&) to guarantee that both user and token properties are always available within the session callback, regardless of the authentication strategy.

This is also incorrect since it usually is either user or token, based on the session strategy!

But I think previously we had the same as what this tries to fix, so for backwards compact, we can probably merge this. This needs to be revisited, maybe as part of #9702

Thanks!