Skip to content

Commit 7f93cb4

Browse files
authored
[DOM] Infer react-server entries bundles if not explicitly configured (#28795)
When packaging we want to infer that a bundle exists for a `react-server` file even if it isn't explicitly configured. This is useful in particular for the react-server entrypoints that error on import that were recently added to `react-dom` This change also cleans up a wayward comment left behind in a prior PR
1 parent f613165 commit 7f93cb4

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

packages/react-dom/src/ReactDOMSharedInternals.js

-4
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,4 @@ const Internals: ReactDOMInternals = {
4747
usingClientEntryPoint: false,
4848
};
4949

50-
// if (__DEV__) {
51-
// (Internals: any).usingClientEntryPoint = false;
52-
// }
53-
5450
export default Internals;

scripts/rollup/packaging.js

+12
Original file line numberDiff line numberDiff line change
@@ -213,6 +213,18 @@ function filterOutEntrypoints(name) {
213213
hasBundle =
214214
entryPointsToHasBundle.get(entry + '.node') ||
215215
entryPointsToHasBundle.get(entry + '.browser');
216+
217+
// The .react-server and .rsc suffixes may not have a bundle representation but
218+
// should infer their bundle status from the non-suffixed entry point.
219+
if (entry.endsWith('.react-server')) {
220+
hasBundle = entryPointsToHasBundle.get(
221+
entry.slice(0, '.react-server'.length * -1)
222+
);
223+
} else if (entry.endsWith('.rsc')) {
224+
hasBundle = entryPointsToHasBundle.get(
225+
entry.slice(0, '.rsc'.length * -1)
226+
);
227+
}
216228
}
217229
if (hasBundle === undefined) {
218230
// This doesn't exist in the bundles. It's an extra file.

0 commit comments

Comments
 (0)