Skip to content

Update next.js dependencies#195

Open
dividedmind wants to merge 4 commits intomainfrom
chore/next-deps
Open

Update next.js dependencies#195
dividedmind wants to merge 4 commits intomainfrom
chore/next-deps

Conversation

@dividedmind
Copy link
Collaborator

This updates next.js dependencies in the test projects and removes the toplevel dependency (which was only used for types — these are now minimally declared).

Copy link

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 PR updates the Next.js versions used by the repository’s Next.js-based integration test workspaces and removes the root-level next dependency, replacing the previously-imported next/webpack types in the Next hook with minimal local type definitions.

Changes:

  • Bump test/next to next@^14.2.35 and test/next16 to next@^16.1.7 (with corresponding yarn.lock updates).
  • Remove the root devDependencies.next entry (it was only needed for type imports).
  • Update the Next.js integration snapshot to match the new framework behavior/headers.

Reviewed changes

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

Show a summary per file
File Description
yarn.lock Updates locked Next.js-related packages and removes the root next entry’s lock impact.
package.json Drops the root-level next devDependency.
test/next/package.json Bumps the Next.js version used by the Next.js test workspace.
test/next16/package.json Bumps the Next.js version used by the Next.js 16 test workspace.
test/snapshots/next.test.ts.snap Updates snapshot expectations due to Next.js response/header changes.
src/hooks/next.ts Replaces imported next/webpack types with minimal local types used by injectConfig.

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 18, 2026

Open in StackBlitz

npm i https://pkg.pr.new/appmap-node@195

commit: ffd5597

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