fix(nextjs): Update params type with Promise as expected in Next.js 15

[ggallon]

☕️ Reasoning

Follow #12002, fix type error with auth() wapprer, update params type with Promise in handler.
Ref: Next.js Doc

🧢 Checklist

• Documentation
• Tests
• Ready to be merged

🎫 Affected issues

📌 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	Oct 31, 2024 10:27pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
next-auth-docs	⬜️ Ignored (Inspect)	Visit Preview		Oct 31, 2024 10:27pm

[vercel]

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

A member of the Team first needs to authorize it.

[balazsorban44]

Thanks! I believe we should use our Awaitable here from @auth/core/types, to be compatible with Next.js 14.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 39.21%. Comparing base (ad9ed04) to head (87af2a5).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/next-auth/src/lib/types.ts	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12114      +/-   ##
==========================================
- Coverage   39.22%   39.21%   -0.01%     
==========================================
  Files         191      191              
  Lines       29928    29929       +1     
  Branches     1291     1297       +6     
==========================================
  Hits        11738    11738              
- Misses      18190    18191       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ggallon]

@balazsorban44, It's done

[github-actions]

Broken Link Checker

0 broken links found. Links organised below by source page, or page where they were found.

[ggallon]

@balazsorban44 sorry, that's not the way. pnpm tsc --noEmit in locale not always cause an error (??) but every time with pnpm next build:

Type "AppRouteHandlerFnContext" is not a valid type for the function's second argument.
    Expected "Promise<any>", got "Awaitable<Record<string, string | string[]>> | undefined".
      Expected "Promise<any>", got "undefined".

[Aralf]

@balazsorban44 sorry, that's not the way. pnpm tsc --noEmit in locale not always cause an error (??) but every time with pnpm next build:

Type "AppRouteHandlerFnContext" is not a valid type for the function's second argument.
    Expected "Promise<any>", got "Awaitable<Record<string, string | string[]>> | undefined".
      Expected "Promise<any>", got "undefined".

@ggallon as the error mentions, undefined is not valid, that's because params is set as optional, it shouldn't be.

[rpavlovs]

Suggested change

	params?: Awaitable<Record<string, string | string[]>>
	params: Awaitable<Record<string, string | string[]>>

[rpavlovs]

@balazsorban44 could you please bump the npm package version?