Skip to content

Commit 7cffb67

Browse files
committed
fix(nextjs): add browser/node condition for client/server bundles
When setting default conditions, include 'browser' for client bundles and 'node' for server bundles. This is required for @sentry/browser to resolve correctly on the client side.
1 parent d4662e5 commit 7cffb67

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

packages/nextjs/src/config/webpack.ts

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,17 @@ export function constructWebpackConfigFunction({
106106
// Prepend 'development' to existing conditions to preserve Next.js's ESM/CJS resolution
107107
newConfig.resolve.conditionNames = ['development', ...existingConditions];
108108
} else if (!existingConditions) {
109-
// Set default conditions with 'development' first (webpack defaults + development)
110-
newConfig.resolve.conditionNames = ['development', 'webpack', 'module', 'import', 'require', 'default'];
109+
// Set default conditions with 'development' first
110+
// Include 'browser' for client bundles, 'node' for server bundles
111+
const platformCondition = isServer ? 'node' : 'browser';
112+
newConfig.resolve.conditionNames = [
113+
'development',
114+
platformCondition,
115+
'module',
116+
'import',
117+
'require',
118+
'default',
119+
];
111120
}
112121
}
113122

0 commit comments

Comments
 (0)