Turbopack: always log HMR rebuild times (vercel#60908)

wbinnssmith · ijjk · web-flow · commit 498f342b3552 · 2024-01-19T17:07:22.000-08:00
This removes the need for `NEXT_HMR_TIMING` to do this.


Closes PACK-2264

Co-authored-by: JJ Kasper &lt;jj@jjsweb.site&gt;
diff --git a/packages/next/src/server/lib/router-utils/setup-dev-bundler.ts b/packages/next/src/server/lib/router-utils/setup-dev-bundler.ts
@@ -1560,10 +1560,8 @@ async function startWatcher(opts: SetupOpts) {
                   false,
                   route.dataEndpoint,
                   (pageName) => {
-                    if (process.env.NEXT_HMR_TIMING) {
-                      // Report the next compilation again
-                      readyIds.delete(page)
-                    }
+                    // Report the next compilation again
+                    readyIds.delete(page)
                     return {
                       event: HMR_ACTIONS_SENT_TO_BROWSER.SERVER_ONLY_CHANGES,
                       pages: [pageName],
@@ -1643,10 +1641,8 @@ async function startWatcher(opts: SetupOpts) {
                     // There will be another update without errors eventually
                     return
                   }
-                  if (process.env.NEXT_HMR_TIMING) {
-                    // Report the next compilation again
-                    readyIds.delete(page)
-                  }
+                  // Report the next compilation again
+                  readyIds.delete(page)
                   return {
                     action:
                       HMR_ACTIONS_SENT_TO_BROWSER.SERVER_COMPONENT_CHANGES,
@@ -1739,7 +1735,7 @@ async function startWatcher(opts: SetupOpts) {
               warnings: [],
             })
 
-            if (process.env.NEXT_HMR_TIMING && hmrEventHappened) {
+            if (hmrEventHappened) {
               const time = updateMessage.value.duration
               const timeMessage =
                 time > 2000 ? `${Math.round(time / 100) / 10}s` : `${time}ms`
