Skip to content

Conversation

@Rezakarimzadeh98
Copy link

What

Fix flakiness on Windows when tests create an isolated Next.js install under %TEMP%.
Long temp directory names can exceed Windows path limitations and cause Turbopack to fail resolving next/package.json / workspace root.

How

Shorten the random suffix used for isolated temp dirs on Windows (keep existing behavior on non-Windows).

Why

This makes the new dev test(s) reliable on Windows for both Webpack and Turbopack.

Test Plan

  • Windows: test/development/pages-dir/http-methods/http-methods.test.ts
    • Webpack dev mode: PASS
    • Turbopack dev mode: PASS

Copilot AI review requested due to automatic review settings December 15, 2025 15:38
@nextjs-bot
Copy link
Collaborator

Allow CI Workflow Run

  • approve CI run for commit: a37fefd

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Dec 15, 2025

Allow CI Workflow Run

  • approve CI run for commit: 8cd79d3

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@Rezakarimzadeh98 Rezakarimzadeh98 changed the title test: shorten isolated install dir names on Windows to prevent Turbopack root resolution failures test: avoid Windows long-path failures in isolated Next.js installs Dec 15, 2025
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants