fix(ts): revert session callback type changes

[juliusmarminge]

Closes #7132
Fixes t3-oss/create-t3-app#1328

Seems like #7056 changed this type which breaks module augmentation.

NOTE:

• It's a good idea to open an issue first to discuss potential changes.
• Please make sure that you are NOT opening a PR to fix a potential security vulnerability. Instead, please follow the Security guidelines to disclose the issue to us confidentially.

☕️ Reasoning

🧢 Checklist

• Documentation
• Tests
• Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: INSERT_ISSUE_LINK_HERE

📌 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)
next-auth-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 3, 2023 6:53am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
auth-docs	⬜️ Ignored (Inspect)			Apr 3, 2023 6:53am

[vercel]

@juliusmarminge is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

[balazsorban44]

See #7132 (comment), happy to discuss on that issue if we should revert or keep it, not sure. 🙏

[balazsorban44]

Thank you! Based on the discussion, this is fair!

[LegendNihal]

I found the main problem here. When you hover over user on the right side one session.user.id = user.id; you can see user is adapterUser which is only available if strategy is set to database. Since we are using Prisma and Prisma doesn't run on Edge runtime, we had to set strategy to jwt. Thats why user.id is null.