From c2627e53b721adef0b52e07f4d4e99bea9dc8a5c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Eduardo=20Bou=C3=A7as?= <mail@eduardoboucas.com>
Date: Wed, 21 Dec 2022 20:28:51 +0000
Subject: [PATCH 1/2] feat: add `quiet` flag

---
 packages/build/src/core/build.ts              |  15 +++-
 packages/build/src/core/config.js             |  16 +++-
 packages/build/src/core/normalize_flags.ts    |   5 +-
 packages/build/src/core/types.ts              |   5 ++
 packages/build/src/plugins/spawn.ts           |   9 +-
 packages/build/src/steps/return.js            |   7 +-
 packages/build/src/steps/run_step.ts          |   6 +-
 packages/build/src/steps/run_steps.js         |   2 +
 .../fixtures/dev_with_error/manifest.yml      |   2 +
 .../fixtures/dev_with_error/netlify.toml      |   2 +
 .../fixtures/dev_with_error/plugin.js         |  33 +++++++
 .../plugins_events/snapshots/tests.js.md      |  85 +++++++++++++++++-
 .../plugins_events/snapshots/tests.js.snap    | Bin 2560 -> 2714 bytes
 packages/build/tests/plugins_events/tests.js  |  22 +++++
 14 files changed, 198 insertions(+), 11 deletions(-)
 create mode 100644 packages/build/tests/plugins_events/fixtures/dev_with_error/manifest.yml
 create mode 100644 packages/build/tests/plugins_events/fixtures/dev_with_error/netlify.toml
 create mode 100644 packages/build/tests/plugins_events/fixtures/dev_with_error/plugin.js

diff --git a/packages/build/src/core/build.ts b/packages/build/src/core/build.ts
index e45e9de7c2..3b84ac34af 100644
--- a/packages/build/src/core/build.ts
+++ b/packages/build/src/core/build.ts
@@ -27,7 +27,10 @@ export const startBuild = function (flags: Partial<BuildCLIFlags>) {
   const timers = initTimers()
 
   const logs = getBufferLogs(flags)
-  logBuildStart(logs)
+
+  if (!flags.quiet) {
+    logBuildStart(logs)
+  }
 
   const { bugsnagKey, ...flagsA } = normalizeFlags(flags, logs)
   const errorMonitor = startErrorMonitor({ flags: flagsA, logs, bugsnagKey })
@@ -72,6 +75,7 @@ const tExecBuild = async function ({
   featureFlags,
   timeline,
   devCommand,
+  quiet,
 }) {
   const configOpts = getConfigOpts({
     config,
@@ -117,6 +121,7 @@ const tExecBuild = async function ({
     logs,
     nodePath,
     timers,
+    quiet,
   })
   const constants = await getConstants({
     configPath,
@@ -177,6 +182,7 @@ const tExecBuild = async function ({
     featureFlags,
     timeline,
     devCommand,
+    quiet,
   })
   return {
     pluginsOptions: pluginsOptionsA,
@@ -227,6 +233,7 @@ export const runAndReportBuild = async function ({
   featureFlags,
   timeline,
   devCommand,
+  quiet,
 }) {
   try {
     const {
@@ -272,6 +279,7 @@ export const runAndReportBuild = async function ({
       featureFlags,
       timeline,
       devCommand,
+      quiet,
     })
     await Promise.all([
       reportStatuses({
@@ -367,6 +375,7 @@ const initAndRunBuild = async function ({
   featureFlags,
   timeline,
   devCommand,
+  quiet,
 }) {
   const { pluginsOptions: pluginsOptionsA, timers: timersA } = await getPluginsOptions({
     pluginsOptions,
@@ -396,6 +405,7 @@ const initAndRunBuild = async function ({
     debug,
     timers: timersA,
     featureFlags,
+    quiet,
   })
 
   try {
@@ -439,6 +449,7 @@ const initAndRunBuild = async function ({
       featureFlags,
       timeline,
       devCommand,
+      quiet,
     })
 
     await Promise.all([
@@ -500,6 +511,7 @@ const runBuild = async function ({
   featureFlags,
   timeline,
   devCommand,
+  quiet,
 }) {
   const { pluginsSteps, timers: timersA } = await loadPlugins({
     pluginsOptions,
@@ -554,6 +566,7 @@ const runBuild = async function ({
     timers: timersA,
     testOpts,
     featureFlags,
+    quiet,
   })
 
   return { stepsCount, netlifyConfig: netlifyConfigA, statuses, failedPlugins, timers: timersB, configMutations }
diff --git a/packages/build/src/core/config.js b/packages/build/src/core/config.js
index fe20cc6f05..fe75bc46f6 100644
--- a/packages/build/src/core/config.js
+++ b/packages/build/src/core/config.js
@@ -59,7 +59,16 @@ export const getConfigOpts = function ({
 }
 
 // Retrieve configuration object
-const tLoadConfig = async function ({ configOpts, cachedConfig, cachedConfigPath, envOpt, debug, logs, nodePath }) {
+const tLoadConfig = async function ({
+  configOpts,
+  cachedConfig,
+  cachedConfigPath,
+  envOpt,
+  debug,
+  logs,
+  nodePath,
+  quiet,
+}) {
   const {
     configPath,
     headersPath,
@@ -74,7 +83,10 @@ const tLoadConfig = async function ({ configOpts, cachedConfig, cachedConfigPath
     siteInfo,
     env,
   } = await resolveInitialConfig(configOpts, cachedConfig, cachedConfigPath)
-  logConfigInfo({ logs, configPath, buildDir, netlifyConfig, context: contextA, debug })
+
+  if (!quiet) {
+    logConfigInfo({ logs, configPath, buildDir, netlifyConfig, context: contextA, debug })
+  }
 
   const apiA = addApiErrorHandlers(api)
   const envValues = mapObj(env, (key, { value }) => [key, value])
diff --git a/packages/build/src/core/normalize_flags.ts b/packages/build/src/core/normalize_flags.ts
index 552cb8ea4f..ff4b3070b4 100644
--- a/packages/build/src/core/normalize_flags.ts
+++ b/packages/build/src/core/normalize_flags.ts
@@ -61,7 +61,9 @@ export const normalizeFlags = function (flags: Partial<BuildCLIFlags>, logs): Re
   }
   const normalizedFlags = removeFalsy(mergedFlags) as any
 
-  logFlags(logs, rawFlags, normalizedFlags)
+  if (!flags.quiet) {
+    logFlags(logs, rawFlags, normalizedFlags)
+  }
 
   return normalizedFlags
 }
@@ -90,6 +92,7 @@ const getDefaultFlags = function ({ env: envOpt = {} }, combinedEnv) {
     featureFlags: DEFAULT_FEATURE_FLAGS,
     statsd: { port: DEFAULT_STATSD_PORT },
     timeline: 'build',
+    quiet: false,
   }
 }
 
diff --git a/packages/build/src/core/types.ts b/packages/build/src/core/types.ts
index 2502a5e011..e4d4839976 100644
--- a/packages/build/src/core/types.ts
+++ b/packages/build/src/core/types.ts
@@ -28,6 +28,11 @@ export type BuildCLIFlags = {
   cwd?: string
   /** A list of all the feature flags passed to netlify/build */
   featureFlags: Record<string, boolean>
+  /**
+   * Print only essential/error output
+   * @default false
+   */
+  quiet?: boolean
 }
 
 export type BuildResult = {
diff --git a/packages/build/src/plugins/spawn.ts b/packages/build/src/plugins/spawn.ts
index 7fd076acf0..30e7320991 100644
--- a/packages/build/src/plugins/spawn.ts
+++ b/packages/build/src/plugins/spawn.ts
@@ -22,9 +22,12 @@ const CHILD_MAIN_FILE = fileURLToPath(new URL('child/main.js', import.meta.url))
 //    (for both security and safety reasons)
 //  - logs can be buffered which allows manipulating them for log shipping,
 //    transforming and parallel plugins
-const tStartPlugins = async function ({ pluginsOptions, buildDir, childEnv, logs, debug, featureFlags }) {
-  logRuntime(logs, pluginsOptions)
-  logLoadingPlugins(logs, pluginsOptions, debug)
+const tStartPlugins = async function ({ pluginsOptions, buildDir, childEnv, logs, debug, featureFlags, quiet }) {
+  if (!quiet) {
+    logRuntime(logs, pluginsOptions)
+    logLoadingPlugins(logs, pluginsOptions, debug)
+  }
+
   logOutdatedPlugins(logs, pluginsOptions)
   logIncompatiblePlugins(logs, pluginsOptions)
 
diff --git a/packages/build/src/steps/return.js b/packages/build/src/steps/return.js
index 90eb485d31..a1ea9fd1cf 100644
--- a/packages/build/src/steps/return.js
+++ b/packages/build/src/steps/return.js
@@ -27,6 +27,7 @@ export const getStepReturn = function ({
   durationNs,
   testOpts,
   systemLog,
+  quiet,
 }) {
   if (newError !== undefined) {
     return handleStepError({
@@ -45,9 +46,11 @@ export const getStepReturn = function ({
     })
   }
 
-  logStepSuccess(logs)
+  if (!quiet) {
+    logStepSuccess(logs)
 
-  logTimer(logs, durationNs, timerName, systemLog)
+    logTimer(logs, durationNs, timerName, systemLog)
+  }
 
   return { newEnvChanges, netlifyConfig, configMutations, headersPath, redirectsPath, newStatus, timers }
 }
diff --git a/packages/build/src/steps/run_step.ts b/packages/build/src/steps/run_step.ts
index 1e91ba3228..fd6c0b76a9 100644
--- a/packages/build/src/steps/run_step.ts
+++ b/packages/build/src/steps/run_step.ts
@@ -54,6 +54,7 @@ export const runStep = async function ({
   timers,
   testOpts,
   featureFlags,
+  quiet,
 }) {
   const constantsA = await addMutableConstants({ constants, buildDir, netlifyConfig })
 
@@ -73,7 +74,9 @@ export const runStep = async function ({
     return {}
   }
 
-  logStepStart({ logs, event, packageName, coreStepDescription, index, error, netlifyConfig })
+  if (!quiet) {
+    logStepStart({ logs, event, packageName, coreStepDescription, index, error, netlifyConfig })
+  }
 
   const fireStep = getFireStep(packageName, coreStepId, event)
   const {
@@ -147,6 +150,7 @@ export const runStep = async function ({
     durationNs,
     testOpts,
     systemLog,
+    quiet,
   })
   return { ...newValues, newIndex: index + 1 }
 }
diff --git a/packages/build/src/steps/run_steps.js b/packages/build/src/steps/run_steps.js
index 46a5bf8e9e..9543fc9fde 100644
--- a/packages/build/src/steps/run_steps.js
+++ b/packages/build/src/steps/run_steps.js
@@ -39,6 +39,7 @@ export const runSteps = async function ({
   timers,
   testOpts,
   featureFlags,
+  quiet,
 }) {
   const {
     index: stepsCount,
@@ -133,6 +134,7 @@ export const runSteps = async function ({
         timers: timersA,
         testOpts,
         featureFlags,
+        quiet,
       })
       const statusesA = addStatus({ newStatus, statuses, event, packageName, pluginPackageJson })
       return {
diff --git a/packages/build/tests/plugins_events/fixtures/dev_with_error/manifest.yml b/packages/build/tests/plugins_events/fixtures/dev_with_error/manifest.yml
new file mode 100644
index 0000000000..a3512f0259
--- /dev/null
+++ b/packages/build/tests/plugins_events/fixtures/dev_with_error/manifest.yml
@@ -0,0 +1,2 @@
+name: test
+inputs: []
diff --git a/packages/build/tests/plugins_events/fixtures/dev_with_error/netlify.toml b/packages/build/tests/plugins_events/fixtures/dev_with_error/netlify.toml
new file mode 100644
index 0000000000..4b06556c85
--- /dev/null
+++ b/packages/build/tests/plugins_events/fixtures/dev_with_error/netlify.toml
@@ -0,0 +1,2 @@
+[[plugins]]
+package = "./plugin.js"
diff --git a/packages/build/tests/plugins_events/fixtures/dev_with_error/plugin.js b/packages/build/tests/plugins_events/fixtures/dev_with_error/plugin.js
new file mode 100644
index 0000000000..476bdd8a96
--- /dev/null
+++ b/packages/build/tests/plugins_events/fixtures/dev_with_error/plugin.js
@@ -0,0 +1,33 @@
+export const onPreDev = function ({ constants }) {
+  console.log('onPreDev:', constants)
+
+  throw new Error('Houston, we have a problem.')
+}
+
+export const onDev = function ({ constants }) {
+  console.log('onDev:', constants)
+}
+
+export const onPreBuild = function ({ constants }) {
+  console.log('onPreBuild:', constants)
+}
+
+export const onBuild = function ({ constants }) {
+  console.log('onBuild:', constants)
+}
+
+export const onPostBuild = function ({ constants }) {
+  console.log('onPostBuild:', constants)
+}
+
+export const onSuccess = function ({ constants }) {
+  console.log('onSuccess:', constants)
+}
+
+export const onError = function ({ constants }) {
+  console.log('onError:', constants)
+}
+
+export const onEnd = function ({ constants }) {
+  console.log('onEnd:', constants)
+}
diff --git a/packages/build/tests/plugins_events/snapshots/tests.js.md b/packages/build/tests/plugins_events/snapshots/tests.js.md
index bc56db0304..41c54f6942 100644
--- a/packages/build/tests/plugins_events/snapshots/tests.js.md
+++ b/packages/build/tests/plugins_events/snapshots/tests.js.md
@@ -1,4 +1,4 @@
-# Snapshot report for `tests/plugins_events/tests.js`
+# Snapshot report for `packages/build/tests/plugins_events/tests.js`
 
 The actual snapshot is saved in `tests.js.snap`.
 
@@ -1640,3 +1640,86 @@ Generated by [AVA](https://avajs.dev).
     ␊
     ␊
     (dev1.0.0command completed in 1ms)`
+
+## Keeps output to a minimum in the `startDev` entrypoint when `quiet: true`
+
+> Snapshot 1
+
+    `onPreDev: {␊
+      CONFIG_PATH: 'netlify.toml',␊
+      FUNCTIONS_DIST: '.netlify/functions/',␊
+      EDGE_FUNCTIONS_DIST: '.netlify/edge-functions-dist/',␊
+      CACHE_DIR: '.netlify/cache',␊
+      IS_LOCAL: true,␊
+      NETLIFY_BUILD_VERSION: '1.0.0',␊
+      SITE_ID: undefined,␊
+      NETLIFY_API_TOKEN: undefined,␊
+      NETLIFY_API_HOST: 'api.netlify.com',␊
+      INTERNAL_FUNCTIONS_SRC: '.netlify/functions-internal',␊
+      INTERNAL_EDGE_FUNCTIONS_SRC: '.netlify/edge-functions',␊
+      PUBLISH_DIR: '.',␊
+      FUNCTIONS_SRC: undefined,␊
+      EDGE_FUNCTIONS_SRC: undefined␊
+    }␊
+    onDev: {␊
+      CONFIG_PATH: 'netlify.toml',␊
+      FUNCTIONS_DIST: '.netlify/functions/',␊
+      EDGE_FUNCTIONS_DIST: '.netlify/edge-functions-dist/',␊
+      CACHE_DIR: '.netlify/cache',␊
+      IS_LOCAL: true,␊
+      NETLIFY_BUILD_VERSION: '1.0.0',␊
+      SITE_ID: undefined,␊
+      NETLIFY_API_TOKEN: undefined,␊
+      NETLIFY_API_HOST: 'api.netlify.com',␊
+      INTERNAL_FUNCTIONS_SRC: '.netlify/functions-internal',␊
+      INTERNAL_EDGE_FUNCTIONS_SRC: '.netlify/edge-functions',␊
+      PUBLISH_DIR: '.',␊
+      FUNCTIONS_SRC: undefined,␊
+      EDGE_FUNCTIONS_SRC: undefined␊
+    }`
+
+## Shows error information in the `startDev` entrypoint even when `quiet: true`
+
+> Snapshot 1
+
+    `onPreDev: {␊
+      CONFIG_PATH: 'netlify.toml',␊
+      FUNCTIONS_DIST: '.netlify/functions/',␊
+      EDGE_FUNCTIONS_DIST: '.netlify/edge-functions-dist/',␊
+      CACHE_DIR: '.netlify/cache',␊
+      IS_LOCAL: true,␊
+      NETLIFY_BUILD_VERSION: '1.0.0',␊
+      SITE_ID: undefined,␊
+      NETLIFY_API_TOKEN: undefined,␊
+      NETLIFY_API_HOST: 'api.netlify.com',␊
+      INTERNAL_FUNCTIONS_SRC: '.netlify/functions-internal',␊
+      INTERNAL_EDGE_FUNCTIONS_SRC: '.netlify/edge-functions',␊
+      PUBLISH_DIR: '.',␊
+      FUNCTIONS_SRC: undefined,␊
+      EDGE_FUNCTIONS_SRC: undefined␊
+    }␊
+    ␊
+      Plugin "./plugin.js" internal error                           ␊
+    ────────────────────────────────────────────────────────────────␊
+    ␊
+      Error message␊
+      Error: Houston, we have a problem.␊
+    ␊
+      Plugin details␊
+      Package:        ./plugin.js␊
+      Version:        1.0.0␊
+      Repository:     git+https://github.com/netlify/build.git␊
+      Report issues:  https://github.com/netlify/build/issues␊
+    ␊
+      Error location␊
+      In "onPreDev" event in "./plugin.js" from netlify.toml␊
+    STACK TRACE␊
+    ␊
+      Resolved config␊
+      build:␊
+        publish: packages/build/tests/plugins_events/fixtures/dev_with_error␊
+        publishOrigin: default␊
+      plugins:␊
+        - inputs: {}␊
+          origin: config␊
+          package: ./plugin.js`
diff --git a/packages/build/tests/plugins_events/snapshots/tests.js.snap b/packages/build/tests/plugins_events/snapshots/tests.js.snap
index ec4ce5dea02f1ea5aaf830dc6d00759c56a2c58d..f044ed32903f05879b7b0c9542f89bf5aea042e5 100644
GIT binary patch
literal 2714
zcmV;L3T5>{RzV<BZeeh9Xm4~Nb~6eC0GMmF&$u!F^|5L}g&Y0qb$P;dxn{-W<L?gO
zmc4)GSRacB00000000B+T~BY@$Q4h!>txMsyC`z$9tuo@wsEqtlx%hn70|^xvb<KK
z+J>bp5MTj?7Do~{6sdBCc7kGo_B#~qc?-0+9(yRz572&tUV7=d_X0f>eVid>jYui9
zEe|Q`8MDDkoEgsi=KcB2<KyV-{)yxDUb5f)o^qQz^wrkb2?n-X@!WRM>ruvt%?S7H
z!GQYIBA!c_{N84+Ic7tTvtNI0Std81-}=W}H*Vbs|Gu$t<GbJd`z?cz1Iit{|C&4r
zY{w!O{WJdcn_sPTF^o^h=hSDm=K>g?x)P1*i9q8~<zeMvSYyvI2Mn%T^duP60PTRn
zC!dZzW^>Pfee8L>M#g6Er8%HXwytu@IIGIlVqJPlUHI6yU-7_)I?*C^y_e8aSoLts
zS<MJRjheJ<#-I64O<WZhnC(C(E!%~qhJTKHFYJP{Xhbm3df@v|g;=&vd*R5S-DQUr
zcIUZ$dqDbd-^+JYjX<q%Z{&o%9#iHyry#36(LMAmI*cYCoH#ZcUIoobCx@UFwp)W`
z>zjcSVi>iTx7sGQI}QMK^2@IbDbHTiVlpz(n;c1vRHFM=zGM+PoW2s>`koc^cql~h
zeakZ~Sf2Q1hPc&K+fRjDk-qPZh>8sXk(;hbwmerVrkL=(HBiTB!iKRmjYAAW&l`;$
z%0YH)mpmG=dxrd;aXKbe;G5wZC<`^WX^8fR3AAIIng#DtpdC#(fp-kVr4XOj*hn~>
zdQ@YL9L|=CKt_NUH;T!@gEVsBdR!$4ebaWpY^D%|KPv>`k4O;mCkU{5MI{N*(3V3K
zWM?|b!US0)3@clV5Q8HzeR5|Ch&!-UocgZm5bFD$Uv3NtX^2UVK&(OV;?FfQVN(;3
zEXu+9hcb2)OKJ2sEvzKAd6TQs)Dh6f*rv%#1Dk&^<b2F()hhh|FgU4zbE+zKvH};P
zralLc#DdTqkvBD~%9`iXgN?B8hVVrbmOGl<9VzQ1ApuE!X?J$&Pf6!^r`|BcfXZQ8
zV(<Sbv-g+6uyrp<0XFX{6ySTCNC9%E0O5*tWu-#$fa&oqhYU>hsFM+>KqD<!*)pFN
zgiB1b7_kyyfV7}!v>*=ygCUn|4Y_=niFEZ$w?`e(QyS~qn0Yd@fvhWc)+Lv$E1DFN
zb}0i~4)2=oQYZ16!5W#@$`--AB8w>&73AgQwF1c+U_d4mYrdNRcmx{-EdpcUQRy`p
z_r_AMQXb$Nr;3{xB%M3Di99yvg>(;MV9VQ%>)wZ>8(8In5AyY*jZZ-qhC3!%4&@AZ
zMAIv5$OB^?`n2FWV;%P=BvuMyTBKuc_UX+p6!QA>n@C=BBd@|66fLb}GN`MO*0>uT
zgmnr>B&#c1G?P_yQP=3Aew=hs*>WE@-&fp|fr$@!yQczXJ}LuShMUrt^*Dnmx~r8f
zMta0U+Pk&EAewN(WlI<?<+9V8Vr&IdNeUEK$-$1t+^U-7Ru|1K{k}r7zJnwyZ<1Bi
z+|uOemO-~P=ar^GjKphY3ucuf6(}ARz?{-t)IiT`yy-!#H)NCE`b1$_ReX&%Z<bZq
zT+)=`ErVz2&Ld4l8JX6~77TG|4{>46?rhHP*^C#C6Lymh_uW4z4)?e3qQfmfSfhZB
zb_qGT=N;*dV4y=?<$?|svt%U(3&J-M+bC!FMtoh+@Quc}4az@;FV|K6<M7&#W{CZO
zaz=pMOX)ZLK`?@mraZlDDK!75`1CSwnjf!K0jd6EXv?7cacBA{{}i%F`&YIYh7f$@
z{)W*3ASA&RkW4;Cwn1@sm{O);L6Sslc1@6pX=P%XO@7SAwo}5aCzJ$Cstbd``k)cf
zeUq&0y`Lzo?EZVm%5r99(UKLAl*J=k1}BU9(nrb8Vb#LOF4@7*bS;PaUC*UU=!Kto
z*ToK_it2c?)|TT!=DENG#iUaO_yWwc92lJQJQav+WUmAmEGgieuq!LU^-`kFvtp1R
z;ag{iLdfGv;_LM?PSY<v&){(3J#WVLxbej&5QW?KuCE!4x8OfK--5f=1%6NPugcE*
zPnc(%<IdZ)3iq<S9MTo{+M*LJo9q}teBQCLCHG?oL;INx>yn@i6HY!aaW9E<T0?x<
z=$c<PTI~GipO>?#IRF2?gU&x!=U>Fra?Xc!brPWQq?~CqkqoSCnV0j7F8Uf=G)8qW
zs#BDxj&A#%lF1;M)J0FnxvfyDw=usoUrJTjvvFp|c6D;4`&^s^xJa{Bw&1Ba=>Ci5
z{_#wlT$Di5Zx5t|sor!V&O?P+{Q#d-<;$#!dmhg0;I7WCw4a5O#24As$`%Y_X$)fF
z890gGt$d`oUFA0`?W{jr`Od!;@B2@DRFr>Qqi|k#8Aszp-RKNql+L@ZnP<UA7mG9W
z-O@%bU}}YoTVTLKpX6U>z~Uo?IhpX^f)`!6H=uk74mSCl-@Zbgdia~)`I0B$4+_Vg
z<fyJrpF}^p_)`pJxd;@7C)Vz45v>RHK3;dqju}mQaK=ZPMcNwZbe1uN{U8ziyU4*T
z-@2(V*Z<(D*!eQoLLSR9HLk0(SDi<)D1ffpi=%iT%gU972eM=%kR`xC&J|{^oO%!;
zaX}K9sr90hCe{pfnE9NuCN6{e!q&*&k-ry=zppP@h<6hYcjIgUKXA#5`@8h?g<Pdj
zjR_x8i8SH%h&r~L^3v(;=gOAiXSZ=naj7ju-*nh=2)sp4yRh0z$(B?4N|1Ox(c2@R
zW~PNDke%}hgL(rX^}~a`=4aiboz8xZ+?|H&?gO~I_w1nFX&xT5ySvSH2WnJedFuzR
zSP@ng6&t&sHM;X_Q)@uCCr!33*m}x#^_}{D1F9cS*Y26UAr+UK?QZL^zSELB(D0js
zMyJ)>d)|HWtl8S_e%?54!wsRaSai{*-Rv~F&D|OaTrjj?R;=lEJ4elK=kRIcV8QkM
zLos@DY{x?i-I5&8L8oziu+y5FYx}sK3e0xQJ=BIXNX@iXff6l`o;_(b+xsy{GwUMS
zD>F&E!K8xmRU(i&(SRf+OO5rf*)A%YFscix%ff?YQlHcud5mY(ms04;WWuN~s4vpJ
zOjICMXFf{kOj{m5m&wDFLT920qsE}d2#tvqAXQ&J#%utXFzO2G%7RdGB1(j*t_hIh
zDBa$1;4%@6QzLX?0;@RBL|hS_%tr4ldb%KXxA`|EcJ~)N-2`KIOOM@w#Ab7TC6JwX
z!sz_a`NfgDQ2sN<?J}!MS+oUJ1yv<hmAQKIxI%ych1W(Q<LO5an4OT^P{$jGKaf{y
zJu$Xmrb4h{4CYeB=2J??fM&qQ0T4rvm}F$T_9z&IH3d;*-1NDmkuLXNk3EnNayFzc
zdGU+Drd-PSizHio``Zdz{NG#X?bcj{(SWb<>c=Lh)gF3hEXv-pT`<ohk^HmR;Dj_M
zlhxVSQmoGV8)$XbTkbHCa8V-Rz85g=xev%0B}4O+0s(;i{fR?Im9i!mN^?-C?7}eg
Uat0I9`ijv10ftnT2AJ9a05FVTlK=n!

literal 2560
zcmV+b3jg&%RzV<BZeeh9Xm4~Nb~6eC0A~#8?-2mw?j-=@kRJ}jO~EPvS%pC79v&7-
zX3&UaCLfCk00000000B+T~CkO#uaz}*v>LRgTR;cP+)4L@!H8scAOq6ph4DJZLCJ!
zU998)1_CHtainF<B?S(-u@MAFzC+u43beN#b12XckbHw&dg;0M0zLI{hQsBEl+4}L
z4k_v_CPqZg4Bz}_-n`$uc^rN-K61R#Gxn#y(7+BH`h0WhgcI9scy2!&jVNQpW+d?K
z$%OjUBA!c_xNbAooU&stU|)Y@S!Q59z4^mWuHC#A{r%|rwKsnJ{Y`_AJsLRn_yu_s
z+Kxre`)B<7k6*2HGK>$%C)8)Q=K>fXx&n>n5l7=e<9_3Qlw;R1Ck*ae^eCLP0PT>%
zFP~05W(S`C;?VPg7MYr(XXb=5QMwsW7O<x1EjFa5)P<j8`*{%hkSFe>q4x}Gin8uc
z1J*JkP~##!n*~pNr^WB`8_af~lAi5CSHnLCz86(NSu7C<+75jmvJlJm=_ry6%3V~f
zs5;Lb+Y>T|=U%*`W(;a0@J^hl)<eoX=M={3h*u9a^9tj}hewXhjxU2|QprAOh3&SW
z+s0<-L>R{9#iO=}?M_2LoqX}7A;z;8mq;YzwFyaDq!B;A@dp;86VT_pTHmw6Q4kFx
zxZd+j3%W->%;1komHm*9D>C-I6QW{6V92F&l1<MQipd*%cMa4rZm?l&rg4xkjJ%Vn
zLjxE)wo4wIu)Bu1&H_3mR_L2iA1EC)w`hd+dl|IloLU6$TA&>_ID@wY;zEeeD{N#8
zPCL%AMh0h7MW7(Siwj9|@NOPCaJ@h!2xHTBVA{+f2;V9M;crL~$|nfWdsQU~v1rR7
z3ZgQdWFbKo3B$@JBf{W-H=o>20dX6;Dxkh=I)wVZ=hqtpLOQ&WCorsG;K|=DlCY@+
zB#Q<x`$uE!K#tP*FFja^ob!^qVyYvck8(~Ew<dP*&T$Y-S*zKE|340o8ep87$~@VC
z8*x!T0E@)JXgVS<E7lY_&zc7dVQvlKLKnI_Zrp7#)=6drWc8&#*lvGJ28Y}2j==@0
z$J`>V|AWHTUkrxLyIBgbaYvy5Z*Cw3D4haCJvJ1XD#-)V@-2r9q-xa32vnev7OZTV
zPYa?hrbQ;P8eo95plY<Bj0px^E?4Sud9M)Z8kz2hI=rSl)^&YmXXZMxuF_eT=&~U%
zQc2pS2)G>Hm9A1J@tVOJnb*oD&b;ColXohNm!lU7Bx`^HNhsEQGXd~$HVRq<lYvKt
z)nI`)6?&Dkfp45DW@3^x?r0{;Se#eVJaEC5w;b2K4o5Sv$_Y2}^`ectAPe0cNt8o9
z9UgJ>D(mt<Uxz*|xJqBgy@bSyftcs%m>Ye1<5Pva{{9A%*V4!<w+2-UD@la99BGx+
z=pd{q9FeT9Y|>0t(L`OLiF!M0qKd^nZv0#^PX-1)lx?0W82P9OwhS|+FY0jyQ#4mA
zn~Z#qhnVly27|c48IvtwxRA-tFN(eu3?-?MU!{b3Tym>smRp@Sy7U(c$@&?Rtg=Z~
zRbxxD(k+8-X^tz+gBXd|$|j5|MJiA|Du6MirKo|P*Ld@tSTBhtz4?K{vYPlBZ`mxX
zvazH&;w^(`>5e1KMH!ja$|iJiX?Jm9%<gQ=?&mXBSO)B74eqOdRSfQ*Uqypk0k=j4
z4eb(Qa?dK#>%l;Sy2=R+Dn`i)3>LU=VzyCF_YJ>XQTL5TzYUImbYHIP_?ParU(FEv
z2@MzlaxX@|=}*EFkThravZv7e-{S6N*)(7FRRyVjBHA+OzN}0i<xe4tw0~ujVeo;E
z+}|(;0E8g89FpW`JU1xj4nxW`bV!zn&Grd0F|ABY^OP^y*jA34^@x&?33Xv0tPdIy
zKR3(DUjL24%I>|6tgK{K7I#?%Ntu*v8JsMxOCKdWgH;P7yI=*!rfWIWA9^lbLM_4P
z-c_-~IHM|W(cD5VWS$GAAW1q^fX~4^>w&=;&r^XYM)qoe!IB)#nR#U;xL%Ied3G3-
zNBGv+pb+w~k-5BHlT-So=NTN%z2+@g9v8m215vqk@A^)IatZ#{*%I8f&hb6Le<~~Q
zzhj(ni7Ri{s@%)+LZnNswRt647TFR(-0xV~RQj=l(fnBq>zbeq4bJYDxEDk^uOPl`
zbj2?lEjIr1&&%0RjQ{^$LE~Sl@vq`(IcH*Bo&;z-DJP94l7W>?^J1RSL|=i4Mz0Qf
zb*kdk(LI0XL^22_b>7o)ZYh-Nrx;&aE~TpM**G(@U7lR&J{KngF4C-(O?WB}n*XYq
ze>@YX6eZB~?SULO)yq!Axvwy*U*b+xxy-7%=i$r>cX@85{Vbd;zR0dtHlY(sqZ13y
zz{z~K@_}M@m2XzsML%2l%6}B=`!C!oD&MbBIjg&jqjBPDbh<EVXI<CKvf$Rm;s|}W
zww?=US|$A!=&;Z?@~_fi@xH>GO!#lXi>`q;p}{d2*z7mIeT6)Y@SES|k|%BtD#xCL
zRF|huydGWrNkUmK0)_60wL4ox>p{H_*PXIsM3WwzaZ9sGO9P$8Qc~D=GS0vA9L)0C
z4TZV>7f;15m$_E*SeBGrmuIg!k7Q8*UAGrU@j#Z9OA8NVDMlb`fPrW$v|Ks$AV%Vx
zB(hNHMJG+H8S2pTIcH5=1oefbk-sB<uNZ${U$P)q6Zdyyw16MF<g<G_^z<{)OOYCr
z;Ft=eNnoE)$98jGI=%H&SyKG&7A`3+w4@lD4qFa^x9I5*dV49+a!Oqp60gU5d*IW;
zw6F$a=d8h?-T+8@e{Z+@(ePk<@VG_pq~W^rF5KRIvezDT_xJk4oo;^sIT~`jjYF6B
z2y60;ot=+5!}+<XHKAKcku3|Bo}yfPyZyKW*$>mXN9O35^4s11u(#jd?uiv>bY-tI
z=yi9W4j(<~_I8G!bPoIQKq$;RotNo%2c2Per$s^+CR&(QtaQ2UgYIy!|8Zw;!Trbk
zTzYeAOOYb8Bm~+UbPo5nd#See58JuGY)S5+7Mww9rnCwaFM06fQLo#7EJ2#-7hYaz
zN!|mJ491rkM{1%0SxT1c>tC}~RNP=x7gU#p2g~H{)Esy$D5@{D(3PaYs4u85!n`CZ
zkgGHAXLP17j-M-JVQQf>af4A~P-D2pNCn8%mtSKv05lkN1$AYCD>>sO!cf-?NYYEU
zdl<TmJL6OjT@=8|&okk7cqOymJByw!@ZD{Er}*yvfv20G?{4Y7I~cLWj9(4Lj%+X*
zKQw;Qa~F;Of_}Tgs!|tiK~+Ijk*YFRPaZZ15TNMZNyK=@@dmRKksIoG)93?vwbm13
W6I#j#mc$@~)Bgc+u-{gc&;S6n+~J@A

diff --git a/packages/build/tests/plugins_events/tests.js b/packages/build/tests/plugins_events/tests.js
index d22baa47da..b82a7f7a8e 100644
--- a/packages/build/tests/plugins_events/tests.js
+++ b/packages/build/tests/plugins_events/tests.js
@@ -112,3 +112,25 @@ test('Runs the `*Dev` events and not the `*Build` events on the dev timeline', a
 
   t.is(devCommand.callCount, 1)
 })
+
+test('Keeps output to a minimum in the `startDev` entrypoint when `quiet: true`', async (t) => {
+  const devCommand = sinon.stub().resolves()
+
+  const output = await new Fixture('./fixtures/dev_and_build')
+    .withFlags({ debug: false, quiet: true, timeline: 'dev' })
+    .runDev(devCommand)
+  t.snapshot(normalizeOutput(output))
+
+  t.is(devCommand.callCount, 1)
+})
+
+test('Shows error information in the `startDev` entrypoint even when `quiet: true`', async (t) => {
+  const devCommand = sinon.stub().resolves()
+
+  const output = await new Fixture('./fixtures/dev_with_error')
+    .withFlags({ debug: false, quiet: true, timeline: 'dev' })
+    .runDev(devCommand)
+  t.snapshot(normalizeOutput(output))
+
+  t.is(devCommand.callCount, 0)
+})

From fd471016c071bcc204dba575dc9342cb2e31a5b0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Eduardo=20Bou=C3=A7as?= <mail@eduardoboucas.com>
Date: Wed, 21 Dec 2022 20:40:15 +0000
Subject: [PATCH 2/2] chore: update snapshot

---
 packages/build/tests/plugins_events/snapshots/tests.js.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/build/tests/plugins_events/snapshots/tests.js.md b/packages/build/tests/plugins_events/snapshots/tests.js.md
index 41c54f6942..2740293b7b 100644
--- a/packages/build/tests/plugins_events/snapshots/tests.js.md
+++ b/packages/build/tests/plugins_events/snapshots/tests.js.md
@@ -1,4 +1,4 @@
-# Snapshot report for `packages/build/tests/plugins_events/tests.js`
+# Snapshot report for `tests/plugins_events/tests.js`
 
 The actual snapshot is saved in `tests.js.snap`.