Skip to content

Commit 82f3684

Browse files
authored
Revert Node Web Streams (#33472)
Reverts #33457, #33456 and #33442. There are too many issues with wrappers, lazy init, stateful modules, duplicate instantiation of async_hooks and duplication of code. Instead, we'll just do a wrapper polyfill that uses Node Streams internally. I kept the client indirection files that I added for consistency with the server though.
1 parent 142aa07 commit 82f3684

33 files changed

+19
-977
lines changed

packages/react-client/src/forks/ReactFlightClientConfig.dom-node-webstreams-parcel.js

Lines changed: 0 additions & 18 deletions
This file was deleted.

packages/react-client/src/forks/ReactFlightClientConfig.dom-node-webstreams-turbopack.js

Lines changed: 0 additions & 19 deletions
This file was deleted.

packages/react-client/src/forks/ReactFlightClientConfig.dom-node-webstreams-webpack.js

Lines changed: 0 additions & 19 deletions
This file was deleted.

packages/react-client/src/forks/ReactFlightClientConfig.dom-node-webstreams.js

Lines changed: 0 additions & 18 deletions
This file was deleted.

packages/react-dom/npm/server.node.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
11
'use strict';
22

3-
var l, s, w;
3+
var l, s;
44
if (process.env.NODE_ENV === 'production') {
55
l = require('./cjs/react-dom-server-legacy.node.production.js');
66
s = require('./cjs/react-dom-server.node.production.js');
7-
w = require('./cjs/react-dom-server.node-webstreams.production.js');
87
} else {
98
l = require('./cjs/react-dom-server-legacy.node.development.js');
109
s = require('./cjs/react-dom-server.node.development.js');
11-
w = require('./cjs/react-dom-server.node-webstreams.development.js');
1210
}
1311

1412
exports.version = l.version;
@@ -18,7 +16,3 @@ exports.renderToPipeableStream = s.renderToPipeableStream;
1816
if (s.resumeToPipeableStream) {
1917
exports.resumeToPipeableStream = s.resumeToPipeableStream;
2018
}
21-
exports.renderToReadableStream = w.renderToReadableStream;
22-
if (w.resume) {
23-
exports.resume = w.resume;
24-
}

packages/react-dom/npm/static.node.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,12 @@
11
'use strict';
22

3-
var s, w;
3+
var s;
44
if (process.env.NODE_ENV === 'production') {
55
s = require('./cjs/react-dom-server.node.production.js');
6-
w = require('./cjs/react-dom-server.node-webstreams.production.js');
76
} else {
87
s = require('./cjs/react-dom-server.node.development.js');
9-
w = require('./cjs/react-dom-server.node-webstreams.development.js');
108
}
119

1210
exports.version = s.version;
1311
exports.prerenderToNodeStream = s.prerenderToNodeStream;
1412
exports.resumeAndPrerenderToNodeStream = s.resumeAndPrerenderToNodeStream;
15-
exports.prerender = w.prerender;
16-
exports.resumeAndPrerender = w.resumeAndPrerender;

packages/react-dom/server.node.js

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -37,17 +37,3 @@ export function resumeToPipeableStream() {
3737
arguments,
3838
);
3939
}
40-
41-
export function renderToReadableStream() {
42-
return require('./src/server/react-dom-server.node-webstreams').renderToReadableStream.apply(
43-
this,
44-
arguments,
45-
);
46-
}
47-
48-
export function resume() {
49-
return require('./src/server/react-dom-server.node-webstreams').resume.apply(
50-
this,
51-
arguments,
52-
);
53-
}

packages/react-dom/src/__tests__/ReactDOMFizzServerNodeWebStreams-test.js

Lines changed: 0 additions & 43 deletions
This file was deleted.

packages/react-dom/src/__tests__/ReactDOMFizzStaticNodeWebStreams-test.js

Lines changed: 0 additions & 177 deletions
This file was deleted.

packages/react-dom/src/server/react-dom-server.node-webstreams.js

Lines changed: 0 additions & 11 deletions
This file was deleted.

0 commit comments

Comments
 (0)