chore(cli): collapse redundant setNoLogMode reset+set pairs

Combine the `setNoLogMode(false)` followed by conditional
`setNoLogMode(true)` into a single `setNoLogMode(!!…)` call — same
effect, one line, and the intent reads directly. Trim near-duplicate
comments at the two call sites to just the WHY (vitest worker state
leak) that isn't obvious from the code.

Author: jdalton

packages/cli/src/utils/cli/with-subcommands.mts

@@ -525,16 +525,10 @@ export async function meowWithSubcommands(
   const compactMode = !!compactHeaderFlag || !!(getCI() && !VITEST)
   const noSpinner = spinnerFlag === false || isDebug()
 
-  // Reset first so prior test runs (module-level state is shared across
-  // vitest cases in the same worker) can't leak their noLogMode setting
-  // into this invocation. Then engage as early as possible so subsequent
-  // informational output in this function and downstream commands knows
-  // to stay on stderr. Keeps stdout clean for `--json | jq` style
-  // pipelines.
-  setNoLogMode(false)
-  if (noLogFlag) {
-    setNoLogMode(true)
-  }
+  // Reset unconditionally: module-level state is shared across vitest
+  // cases in the same worker, so a prior run with --no-log could leak
+  // into this invocation.
+  setNoLogMode(!!noLogFlag)
 
   // Use CI spinner style when --no-spinner is passed or debug mode is enabled.
   // This prevents the spinner from interfering with debug output.
@@ -934,11 +928,6 @@ export function meowOrExit(
   const command = `${parentName} ${cliConfig.commandName}`
   lastSeenCommand = command
 
-  // Reset no-log mode for each command invocation so state doesn't leak
-  // across unit tests that exercise multiple commands in sequence. The
-  // flag is re-engaged below if the parsed flags call for it.
-  setNoLogMode(false)
-
   // This exits if .printHelp() is called either by meow itself or by us.
   const cli = meow({
     argv,
@@ -976,12 +965,10 @@ export function meowOrExit(
   const compactMode = !!compactHeaderFlag || !!(getCI() && !VITEST)
   const noSpinner = spinnerFlag === false || isDebug()
 
-  // Engage no-log mode when the user asked for it directly, or when
-  // `--json` / `--markdown` is in effect — in both cases stdout belongs
-  // to the primary payload and informational output should go to stderr.
-  if (noLogFlag || jsonFlag || markdownFlag) {
-    setNoLogMode(true)
-  }
+  // --json / --markdown imply --no-log: their stdout belongs to the
+  // primary payload, so informational output must go to stderr. Reset
+  // unconditionally to clear any prior in-worker vitest state.
+  setNoLogMode(!!(noLogFlag || jsonFlag || markdownFlag))
 
   // Use CI spinner style when --no-spinner is passed.
   // This prevents the spinner from interfering with debug output.