Skip to content

Conversation

@joaquimds
Copy link
Member

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request upgrades multiple packages and makes related configuration changes. However, it contains a critical breaking change in the middleware function naming that will prevent the application from functioning correctly.

Changes:

  • Upgraded Next.js from 15.5.7 to 16.1.4, React from 19.1.1 to 19.2.3, and various other dependencies including tRPC, Sanity, and react-email packages
  • Updated TypeScript configuration to use the modern JSX transform and include Next.js dev types
  • Renamed the middleware function from middleware to proxy in src/proxy.ts
  • Removed deprecated Sentry configuration options from next.config.ts

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

File Description
tsconfig.json Updated JSX transform to react-jsx, reformatted arrays for consistency, and added .next/dev/types/**/*.ts to includes
src/proxy.ts Renamed exported function from middleware to proxy (breaking change)
package.json Upgraded Next.js, React, tRPC, Sanity, and other dependencies; changed email dev server port from 3001 to 3002; added React type overrides
next.config.ts Removed deprecated Sentry configuration options (disableLogger, automaticVercelMonitors)
Comments suppressed due to low confidence (1)

src/proxy.ts:7

  • The exported function name has been changed from 'middleware' to 'proxy'. According to Next.js conventions, middleware must export a function named 'middleware' (or the file must be named 'middleware.ts'). This change will prevent Next.js from recognizing this file as middleware, causing the request handling logic to be bypassed. Unless there's a separate middleware.ts file that re-exports this function as middleware, this is a breaking change that will cause the application to malfunction.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@joaquimds joaquimds merged commit ad2b7f5 into main Jan 20, 2026
6 of 7 checks passed
@joaquimds joaquimds deleted the feat/upgrade-packages branch January 20, 2026 15:21
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