Skip to content

Fix Next.js 15 compatibility: async params and eslint-config-next upgrade - #103

Merged
dcotelo merged 2 commits into
snyk-fix-85006cc8859b9e120bfbad5d9f37457cfrom
copilot/sub-pr-98
Feb 9, 2026
Merged

dcotelo merged 2 commits into
snyk-fix-85006cc8859b9e120bfbad5d9f37457cfrom
copilot/sub-pr-98

Conversation

Copilot AI commented Feb 9, 2026 •

Copy link
Copy Markdown
Contributor

CI build was failing due to Next.js 15 breaking changes in dynamic route parameter handling. The params prop in page components is now an async Promise.

Changes

  • Refactored /app/analysis/[id]/page.tsx: Split into async server component (page) and client component (StoredAnalysisContent) to properly await params
  • Updated eslint-config-next: v14.0.4 → v15.5.10 for Next.js 15 compatibility

Before/After

// Before (Next.js 14)
export default function Page({ params }: { params: { id: string } }) {
  return <Component id={params.id} />
}

// After (Next.js 15)
export default async function Page({ params }: { params: Promise<{ id: string }> }) {
  const { id } = await params;
  return <Component id={id} />
}

The separation into server/client components maintains the existing client-side hooks (useRouter, useState) while allowing the page to handle async params correctly.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ade eslint-config-next

Co-authored-by: dcotelo <50673035+dcotelo@users.noreply.github.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Feb 9, 2026 •

Copy link
Copy Markdown

Deploying chartimpact with  Cloudflare Pages  Cloudflare Pages

Latest commit: b4e34c7
Status: ✅  Deploy successful!
Preview URL: https://72b1a3fb.chartimpact.pages.dev
Branch Preview URL: https://copilot-sub-pr-98.chartimpact.pages.dev

View logs

@sonarqubecloud

sonarqubecloud Bot commented Feb 9, 2026

Copy link
Copy Markdown

Copilot AI changed the title [WIP] Update next from 14.2.35 to 15.5.10 for security fixes Fix Next.js 15 compatibility: async params and eslint-config-next upgrade Feb 9, 2026
Copilot AI requested a review from dcotelo February 9, 2026 14:51
@dcotelo
dcotelo marked this pull request as ready for review February 9, 2026 14:55
@dcotelo
dcotelo merged commit 8478c05 into snyk-fix-85006cc8859b9e120bfbad5d9f37457c Feb 9, 2026
12 checks passed
@dcotelo
dcotelo deleted the copilot/sub-pr-98 branch February 9, 2026 14:55
dcotelo added a commit that referenced this pull request Feb 9, 2026
* fix: frontend/package.json & frontend/package-lock.json to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-NEXT-15104645

* Fix Next.js 15 compatibility: async params and eslint-config-next upgrade (#103)

* Initial plan

* Fix Next.js 15 compatibility issues - update params to async and upgrade eslint-config-next

Co-authored-by: dcotelo <50673035+dcotelo@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: dcotelo <50673035+dcotelo@users.noreply.github.com>

---------

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants