Skip to content

Commit

Permalink
change default module id strategy back to dev (#71052)
Browse files Browse the repository at this point in the history
### What?

Disable deterministic module ids by default until the performance issue
is solved.
  • Loading branch information
sokra authored Oct 10, 2024
1 parent 58ab93b commit b8db5e2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion crates/next-api/src/project.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1281,7 +1281,7 @@ impl Project {
}
None => match *self.next_mode().await? {
NextMode::Development => Ok(Vc::upcast(DevModuleIdStrategy::new())),
NextMode::Build => Ok(Vc::upcast(GlobalModuleIdStrategyBuilder::build(self))),
NextMode::Build => Ok(Vc::upcast(DevModuleIdStrategy::new())),
},
}
}
Expand Down
6 changes: 6 additions & 0 deletions test/integration/module-id-strategies/next.config.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
module.exports = {
bundlePagesRouterDependencies: true,
serverExternalPackages: ['opted-out-external-package'],
experimental: {
turbo: {
moduleIdStrategy:
process.env.NODE_ENV === 'production' ? 'deterministic' : undefined,
},
},
}

0 comments on commit b8db5e2

Please sign in to comment.