From f4934e8778f2fe1d3e71e6b8e49d8f16c191cfef Mon Sep 17 00:00:00 2001
From: patak <matias.capeletto@gmail.com>
Date: Thu, 24 Mar 2022 09:59:57 +0100
Subject: [PATCH] feat: hide optimized deps found during scan phase logs
 (#7419)

---
 packages/vite/src/node/optimizer/index.ts     |  2 +
 .../src/node/optimizer/registerMissing.ts     | 59 ++++++++++---------
 2 files changed, 34 insertions(+), 27 deletions(-)

diff --git a/packages/vite/src/node/optimizer/index.ts b/packages/vite/src/node/optimizer/index.ts
index cefa24133db529..37294aea459c15 100644
--- a/packages/vite/src/node/optimizer/index.ts
+++ b/packages/vite/src/node/optimizer/index.ts
@@ -253,6 +253,8 @@ export function loadCachedDepOptimizationMetadata(
       // need to resolve the processing promise so requests can move on
       return cachedMetadata
     }
+  } else {
+    config.logger.info('Forced re-optimization of dependencies')
   }
 
   // Start with a fresh cache
diff --git a/packages/vite/src/node/optimizer/registerMissing.ts b/packages/vite/src/node/optimizer/registerMissing.ts
index 65508ac5ae5155..ee4824389c202b 100644
--- a/packages/vite/src/node/optimizer/registerMissing.ts
+++ b/packages/vite/src/node/optimizer/registerMissing.ts
@@ -47,14 +47,18 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
 
   let newDepsToLog: string[] = []
   let newDepsToLogHandle: NodeJS.Timeout | undefined
-  const logNewDeps = () => {
-    config.logger.info(
-      colors.green(`✨ dependencies optimized: ${depsLogString(newDepsToLog)}`),
-      {
-        timestamp: true
-      }
-    )
-    newDepsToLog = []
+  const logNewlyDiscoveredDeps = () => {
+    if (newDepsToLog.length) {
+      config.logger.info(
+        colors.green(
+          `✨ new dependencies optimized: ${depsLogString(newDepsToLog)}`
+        ),
+        {
+          timestamp: true
+        }
+      )
+      newDepsToLog = []
+    }
   }
 
   let depOptimizationProcessing = newDepOptimizationProcessing()
@@ -124,7 +128,7 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
     setTimeout(warmUp, 0)
   }
 
-  async function runOptimizer() {
+  async function runOptimizer(isRerun = false) {
     // Ensure that rerun is called sequentially
     enqueuedRerun = undefined
     currentlyProcessing = true
@@ -223,11 +227,13 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
           }
         }
 
-        newDepsToLog.push(
-          ...Object.keys(newData.optimized).filter(
-            (dep) => !metadata.optimized[dep]
+        if (isRerun) {
+          newDepsToLog.push(
+            ...Object.keys(newData.optimized).filter(
+              (dep) => !metadata.optimized[dep]
+            )
           )
-        )
+        }
 
         metadata = optimizedDeps.metadata = newData
         resolveEnqueuedProcessingPromises()
@@ -236,17 +242,16 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
       if (!needsReload) {
         commitProcessing()
 
-        if (isDebugEnabled) {
-          logNewDeps()
-          debug(colors.green(`✨ previous optimized dependencies unchanged`), {
-            timestamp: true
-          })
-        } else {
+        if (!isDebugEnabled) {
           if (newDepsToLogHandle) clearTimeout(newDepsToLogHandle)
           newDepsToLogHandle = setTimeout(() => {
             newDepsToLogHandle = undefined
-            logNewDeps()
+            logNewlyDiscoveredDeps()
           }, 2 * debounceMs)
+        } else {
+          debug(colors.green(`✨ optimized dependencies unchanged`), {
+            timestamp: true
+          })
         }
       } else {
         if (newDepsDiscovered) {
@@ -267,14 +272,14 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
         } else {
           commitProcessing()
 
-          if (newDepsToLogHandle) clearTimeout(newDepsToLogHandle)
-          newDepsToLogHandle = undefined
-          logNewDeps()
+          if (!isDebugEnabled) {
+            if (newDepsToLogHandle) clearTimeout(newDepsToLogHandle)
+            newDepsToLogHandle = undefined
+            logNewlyDiscoveredDeps()
+          }
 
           logger.info(
-            colors.green(
-              `✨ previous optimized dependencies have changed, reloading page`
-            ),
+            colors.green(`✨ optimized dependencies changed. reloading`),
             {
               timestamp: true
             }
@@ -320,7 +325,7 @@ export function createOptimizedDeps(server: ViteDevServer): OptimizedDeps {
     debug(colors.green(`new dependencies found: ${depsString}`), {
       timestamp: true
     })
-    runOptimizer()
+    runOptimizer(true)
   }
 
   function getDiscoveredBrowserHash(