Add unstable_useRefreshRoot #32342

huozhi · 2021-12-09T22:19:06Z

Feature

Resolves #32332

Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
Related issues linked using fixes #number
Integration tests added
Documentation added
Telemetry added. In case of a feature if it's used or not.
Errors have helpful link attached, see contributing.md

ijjk · 2021-12-10T01:29:29Z

Failing test suites

Commit: bec7d7f

test/integration/react-streaming-and-server-components/test/index.test.js

concurrentFeatures - dev > should support next/link

Expand output

● concurrentFeatures - dev › should support next/link

expect(received).toBe(expected) // Object.is equality

Expected: 1
Received: undefined

  257 |       context.appPort,
  258 |       '/routes/dynamic1'
> 259 |     )
      |      ^
  260 |     const dynamicRouteHTML2 = await renderViaHTTP(
  261 |       context.appPort,
  262 |       '/routes/dynamic2'

  at Object.<anonymous> (integration/react-streaming-and-server-components/test/index.test.js:259:56)

ijjk · 2021-12-10T10:04:09Z

Stats from current PR

Default Build (Increase detected ⚠️)

General Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
buildDuration	20.2s	20.1s	-121ms
buildDurationCached	3.8s	3.8s	⚠️ +30ms
nodeModulesSize	351 MB	351 MB	⚠️ +3.46 kB

Page Load Tests Overall increase ✓

	vercel/next.js canary	huozhi/next.js use-refresh	Change
/ failed reqs	0	0	✓
/ total time (seconds)	3.52	3.267	-0.25
/ avg req/sec	710.3	765.13	+54.83
/error-in-render failed reqs	0	0	✓
/error-in-render total time (seconds)	1.622	1.525	-0.1
/error-in-render avg req/sec	1541.24	1639.54	+98.3

Client Bundles (main, webpack, commons) Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
450.HASH.js gzip	179 B	179 B	✓
framework-HASH.js gzip	42.2 kB	42.2 kB	✓
main-HASH.js gzip	28.8 kB	28.9 kB	⚠️ +143 B
webpack-HASH.js gzip	1.45 kB	1.45 kB	✓
Overall change	72.7 kB	72.8 kB	⚠️ +143 B

Legacy Client Bundles (polyfills)

	vercel/next.js canary	huozhi/next.js use-refresh	Change
polyfills-HASH.js gzip	31 kB	31 kB	✓
Overall change	31 kB	31 kB	✓

Client Pages

	vercel/next.js canary	huozhi/next.js use-refresh	Change
_app-HASH.js gzip	1.37 kB	1.37 kB	✓
_error-HASH.js gzip	194 B	194 B	✓
amp-HASH.js gzip	312 B	312 B	✓
css-HASH.js gzip	326 B	326 B	✓
dynamic-HASH.js gzip	2.39 kB	2.39 kB	✓
head-HASH.js gzip	350 B	350 B	✓
hooks-HASH.js gzip	919 B	919 B	✓
image-HASH.js gzip	4.73 kB	4.73 kB	✓
index-HASH.js gzip	263 B	263 B	✓
link-HASH.js gzip	2.13 kB	2.13 kB	✓
routerDirect..HASH.js gzip	321 B	321 B	✓
script-HASH.js gzip	383 B	383 B	✓
withRouter-HASH.js gzip	318 B	318 B	✓
85e02e95b279..7e3.css gzip	107 B	107 B	✓
Overall change	14.1 kB	14.1 kB	✓

Client Build Manifests

	vercel/next.js canary	huozhi/next.js use-refresh	Change
_buildManifest.js gzip	459 B	459 B	✓
Overall change	459 B	459 B	✓

Rendered Page Sizes Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
index.html gzip	531 B	531 B	✓
link.html gzip	544 B	545 B	⚠️ +1 B
withRouter.html gzip	525 B	525 B	✓
Overall change	1.6 kB	1.6 kB	⚠️ +1 B

Diffs

Diff for main-HASH.js

@@ -465,7 +465,7 @@
       exports.renderError = renderError;
       exports.emitter = exports.router = exports.version = void 0;
       __webpack_require__(400);
-      var _react = _interopRequireDefault1(__webpack_require__(7294));
+      var _react = _interopRequireWildcard1(__webpack_require__(7294));
       var _reactDom = _interopRequireDefault1(__webpack_require__(3935));
       var _styledJsx = __webpack_require__(8771);
       var _headManagerContext = __webpack_require__(8404);
@@ -486,6 +486,7 @@
       var _router1 = __webpack_require__(387);
       var _isError = _interopRequireDefault1(__webpack_require__(676));
       var _vitals = __webpack_require__(7813);
+      var _rsc = __webpack_require__(4692);
       function asyncGeneratorStep(
         gen,
         resolve,
@@ -1245,7 +1246,12 @@
       };
       var RSCComponent;
       if (false) {
-        var RSCWrapper, rscCache, createResponseCache;
+        var RSCWrapper,
+          getHref,
+          rscCache,
+          createFromFetch,
+          fetchFlight,
+          createResponseCache;
       }
       var lastAppProps;
       function doRender(input) {
@@ -2746,6 +2752,28 @@
       /***/
     },
 
+    /***/ 4692: /***/ function(
+      __unused_webpack_module,
+      exports,
+      __webpack_require__
+    ) {
+      "use strict";
+
+      Object.defineProperty(exports, "__esModule", {
+        value: true
+      });
+      exports.useRefreshRoot = useRefreshRoot;
+      exports.RefreshContext = void 0;
+      var _react = __webpack_require__(7294);
+      var RefreshContext = (0, _react).createContext(function(_) {});
+      exports.RefreshContext = RefreshContext;
+      function useRefreshRoot() {
+        return (0, _react).useContext(RefreshContext);
+      } //# sourceMappingURL=rsc.js.map
+
+      /***/
+    },
+
     /***/ 699: /***/ function(
       __unused_webpack_module,
       exports,

Diff for index.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Diff for link.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Diff for withRouter.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Default Build with SWC (Increase detected ⚠️)

General Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
buildDuration	22.2s	22.1s	-32ms
buildDurationCached	3.9s	3.7s	-175ms
nodeModulesSize	351 MB	351 MB	⚠️ +3.46 kB

Page Load Tests Overall decrease ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
/ failed reqs	0	0	✓
/ total time (seconds)	3.435	3.54	⚠️ +0.1
/ avg req/sec	727.75	706.15	⚠️ -21.6
/error-in-render failed reqs	0	0	✓
/error-in-render total time (seconds)	1.582	1.647	⚠️ +0.06
/error-in-render avg req/sec	1579.83	1518	⚠️ -61.83

Client Bundles (main, webpack, commons) Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
450.HASH.js gzip	179 B	179 B	✓
framework-HASH.js gzip	42.3 kB	42.3 kB	✓
main-HASH.js gzip	29 kB	29.1 kB	⚠️ +49 B
webpack-HASH.js gzip	1.44 kB	1.44 kB	✓
Overall change	73 kB	73 kB	⚠️ +49 B

Legacy Client Bundles (polyfills)

	vercel/next.js canary	huozhi/next.js use-refresh	Change
polyfills-HASH.js gzip	31 kB	31 kB	✓
Overall change	31 kB	31 kB	✓

Client Pages

	vercel/next.js canary	huozhi/next.js use-refresh	Change
_app-HASH.js gzip	1.35 kB	1.35 kB	✓
_error-HASH.js gzip	180 B	180 B	✓
amp-HASH.js gzip	305 B	305 B	✓
css-HASH.js gzip	321 B	321 B	✓
dynamic-HASH.js gzip	2.39 kB	2.39 kB	✓
head-HASH.js gzip	342 B	342 B	✓
hooks-HASH.js gzip	906 B	906 B	✓
image-HASH.js gzip	4.75 kB	4.75 kB	✓
index-HASH.js gzip	256 B	256 B	✓
link-HASH.js gzip	2.19 kB	2.19 kB	✓
routerDirect..HASH.js gzip	314 B	314 B	✓
script-HASH.js gzip	375 B	375 B	✓
withRouter-HASH.js gzip	309 B	309 B	✓
85e02e95b279..7e3.css gzip	107 B	107 B	✓
Overall change	14.1 kB	14.1 kB	✓

Client Build Manifests

	vercel/next.js canary	huozhi/next.js use-refresh	Change
_buildManifest.js gzip	458 B	458 B	✓
Overall change	458 B	458 B	✓

Rendered Page Sizes Overall increase ⚠️

	vercel/next.js canary	huozhi/next.js use-refresh	Change
index.html gzip	531 B	532 B	⚠️ +1 B
link.html gzip	544 B	545 B	⚠️ +1 B
withRouter.html gzip	525 B	526 B	⚠️ +1 B
Overall change	1.6 kB	1.6 kB	⚠️ +3 B

Diffs

Diff for main-HASH.js

@@ -465,7 +465,7 @@
       exports.renderError = renderError;
       exports.emitter = exports.router = exports.version = void 0;
       __webpack_require__(400);
-      var _react = _interopRequireDefault1(__webpack_require__(7294));
+      var _react = _interopRequireWildcard1(__webpack_require__(7294));
       var _reactDom = _interopRequireDefault1(__webpack_require__(3935));
       var _styledJsx = __webpack_require__(8771);
       var _headManagerContext = __webpack_require__(8404);
@@ -486,6 +486,7 @@
       var _router1 = __webpack_require__(387);
       var _isError = _interopRequireDefault1(__webpack_require__(676));
       var _vitals = __webpack_require__(7813);
+      var _rsc = __webpack_require__(4692);
       function asyncGeneratorStep(
         gen,
         resolve,
@@ -1245,7 +1246,12 @@
       };
       var RSCComponent;
       if (false) {
-        var RSCWrapper, rscCache, createResponseCache;
+        var RSCWrapper,
+          getHref,
+          rscCache,
+          createFromFetch,
+          fetchFlight,
+          createResponseCache;
       }
       var lastAppProps;
       function doRender(input) {
@@ -2746,6 +2752,28 @@
       /***/
     },
 
+    /***/ 4692: /***/ function(
+      __unused_webpack_module,
+      exports,
+      __webpack_require__
+    ) {
+      "use strict";
+
+      Object.defineProperty(exports, "__esModule", {
+        value: true
+      });
+      exports.useRefreshRoot = useRefreshRoot;
+      exports.RefreshContext = void 0;
+      var _react = __webpack_require__(7294);
+      var RefreshContext = (0, _react).createContext(function(_) {});
+      exports.RefreshContext = RefreshContext;
+      function useRefreshRoot() {
+        return (0, _react).useContext(RefreshContext);
+      } //# sourceMappingURL=rsc.js.map
+
+      /***/
+    },
+
     /***/ 699: /***/ function(
       __unused_webpack_module,
       exports,

Diff for index.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Diff for link.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Diff for withRouter.html

@@ -19,7 +19,7 @@
       defer=""
     ></script>
     <script
-      src="/_next/static/chunks/main-a6d927878c4a570b.js"
+      src="/_next/static/chunks/main-f53bd6d15edae6e3.js"
       defer=""
     ></script>
     <script

Commit: 8c68210

shuding

LGTM as an initial implementation.

ijjk added created-by: Next.js team PRs by the Next.js team type: next labels Dec 9, 2021

This comment has been minimized.

Sign in to view

huozhi added 4 commits December 10, 2021 10:44

Add unstable_useRefreshRoot

c01ec1e

fix lint

fd35776

refresh with props

70c1625

rexports

8c68210

huozhi force-pushed the use-refresh branch from bec7d7f to 8c68210 Compare December 10, 2021 09:45

huozhi marked this pull request as ready for review December 10, 2021 10:13

huozhi requested review from ijjk, padmaia, shuding and timneutkens as code owners December 10, 2021 10:13

shuding approved these changes Dec 10, 2021

View reviewed changes

kodiakhq bot merged commit 6f8cf67 into vercel:canary Dec 10, 2021

huozhi deleted the use-refresh branch December 10, 2021 10:23

vercel locked as resolved and limited conversation to collaborators Jan 27, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add unstable_useRefreshRoot #32342

Add unstable_useRefreshRoot #32342

huozhi commented Dec 9, 2021

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

ijjk commented Dec 10, 2021

ijjk commented Dec 10, 2021

Diffs

Diffs

shuding left a comment

Add unstable_useRefreshRoot #32342

Add unstable_useRefreshRoot #32342

Conversation

huozhi commented Dec 9, 2021

Feature

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

ijjk commented Dec 10, 2021

Failing test suites

ijjk commented Dec 10, 2021

Stats from current PR

Diffs

Diffs

shuding left a comment

Choose a reason for hiding this comment