Disable banner by default, for now

Author: pvdz

src/utils/meow-with-subcommands.ts

@@ -7,6 +7,7 @@ import { getSetting } from './settings.ts'
 import { MeowFlags, commonFlags } from '../flags'
 
 import type { Options } from 'meow'
+import { envAsBoolean } from '@socketsecurity/registry/lib/env'
 
 interface CliAlias {
   description: string
@@ -87,7 +88,10 @@ export async function meowWithSubcommands(
   }
   // ...else we provide basic instructions and help
 
-  console.log(getAsciiHeader(name))
+  // Temp disable until we clear the --json and --markdown usage
+  if (envAsBoolean(process.env['SOCKET_CLI_SHOW_BANNER'])) {
+    console.log(getAsciiHeader(name))
+  }
 
   const cli = meow(
     `
@@ -160,7 +164,10 @@ export function meowOrExit({
 
   const help = config.help(command, config)
 
-  console.log(getAsciiHeader(command))
+  // Temp disable until we clear the --json and --markdown usage
+  if (envAsBoolean(process.env['SOCKET_CLI_SHOW_BANNER'])) {
+    console.log(getAsciiHeader(command))
+  }
 
   // This exits if .printHelp() is called either by meow itself or by us.
   const cli = meow({

test/dry-run.test.ts

@@ -11,9 +11,10 @@
 import path from 'node:path'
 
 import spawn from '@npmcli/promise-spawn'
-import { describe, expect, it } from 'vitest'
+import { afterAll, beforeAll, describe, expect, it } from 'vitest'
 
 import constants from '../dist/constants.js'
+import { envAsBoolean } from '@socketsecurity/registry/lib/env'
 
 type PromiseSpawnOptions = Exclude<Parameters<typeof spawn>[2], undefined> & {
   encoding?: BufferEncoding | undefined
@@ -43,6 +44,22 @@ function cmdit(
 }
 
 describe('dry-run on all commands', async () => {
+  let was: unknown
+
+  beforeAll(() => {
+    // Temp: we have to disable the banner by default until we make it work
+    //       properly with --json and --markdown, since otherwise the output
+    //       would be invalid.
+    was = process.env['SOCKET_CLI_SHOW_BANNER']
+    process.env['SOCKET_CLI_SHOW_BANNER'] = '1'
+  })
+
+  afterAll(() => {
+    if (was) {
+      process.env['SOCKET_CLI_SHOW_BANNER'] = was as any
+    }
+  })
+
   // Lazily access constants.rootBinPath.
   const entryPath = path.join(constants.rootBinPath, `${CLI}.js`)