Remove cli stacktrace from cli error output

packages/cli/cli.js

@@ -22,7 +22,9 @@ const path = require('path')
 const { exit } = require('process')
 
 // remove the `npx cli` prefix
-const index = process.argv.findIndex((arg) => arg.includes('@alephium/cli') || arg.includes('cli.js') || arg.includes('cli'))
+const index = process.argv.findIndex(
+  (arg) => arg.includes('@alephium/cli') || arg.includes('cli.js') || arg.includes('cli')
+)
 if (index === -1) {
   console.log('Please run "npx @alephium/cli@latest <command>"')
   exit(-1)
@@ -31,8 +33,12 @@ const argString = process.argv.slice(index + 1).join(' ')
 const cliRootPath = path.resolve(__dirname)
 const cliInternalPath = path.join(cliRootPath, 'cli_internal.ts')
 const command = `npx --yes ts-node --transpile-only ${cliInternalPath} ${argString}`
-execSync(command, {
-  stdio: 'inherit',
-  cwd: process.cwd(),
-  env: process.env
-})
+try {
+  execSync(command, {
+    stdio: 'inherit',
+    cwd: process.cwd(),
+    env: process.env
+  })
+} catch (err) {
+  exit(err.status)
+}

packages/cli/cli_internal.ts

@@ -26,11 +26,11 @@
 import { generateImagesWithOpenAI, uploadImagesAndMetadataToIPFS } from './scripts/pre-designed-nft'
 import { codegen, getConfigFile, isNetworkLive, loadConfig } from './src'
 
 function getConfig(options: any): Configuration {
   const configFile = options.config ? (options.config as string) : getConfigFile()
   console.log(`Loading alephium config file: ${configFile}`)
   const config = loadConfig(configFile)
-  if (config.enableDebugMode) enableDebugMode()
+  if (config.enableDebugMode || options.debug) enableDebugMode()
   return config
 }
 
@@ -44,6 +44,11 @@
   return networkId as NetworkId
 }
 
+function buildErrorOutput(error: Error, isDebug: boolean): string {
+  const debugMsg = error.stack ?? error.toString()
+  return isDebug ? debugMsg : error.message
+}
+
 const templateTypes = ['base', 'react', 'nextjs']
 
 program
@@ -73,14 +78,14 @@
   .option('-c, --config <config-file>', 'project config file (default: alephium.config.{ts|js})')
   .option('-n, --network <network-type>', 'network type')
   .option('--skipGenerate', 'skip generate typescript code by contract artifacts')
+  .option('--debug', 'show detailed debug information such as error stack traces')
   .action(async (options) => {
     try {
       const config = getConfig(options)
       const networkId = checkAndGetNetworkId(options.network)
       const nodeUrl = config.networks[networkId].nodeUrl
       if (!(await isNetworkLive(nodeUrl))) {
-        console.log(`${networkId} is not live`)
-        process.exit(1)
+        throw new Error(`${networkId} is not live`)
       }
       web3.setCurrentNodeProvider(nodeUrl)
       const fullNodeVersion = (await web3.getCurrentNodeProvider().infos.getInfosVersion()).version
@@ -91,11 +96,11 @@
       if (options.skipGenerate) {
         return
       }
       const artifactDir = config.artifactDir! // there is a default value always
       codegen(artifactDir)
       console.log('✅ Codegen completed!')
     } catch (error) {
-      program.error(`Failed to compile, error: ${(error as Error).stack}`)
+      program.error(`✘ Failed to compile, error: ${buildErrorOutput(error, options.debug)}`)
     }
   })
 
@@ -160,6 +165,7 @@
     '-t, --to <number>',
     'run scripts to a specific index(inclusive), the number refers to the prefix of the script file'
   )
+  .option('--debug', 'show detailed debug information such as error stack traces')
   .action(async (options) => {
     try {
       const config = getConfig(options)
@@ -168,7 +174,7 @@
       const toIndex = tryGetScriptIndex(options.to)
       await deployAndSaveProgress(config, networkId, fromIndex, toIndex)
     } catch (error) {
-      program.error(`Failed to deploy contracts, error: ${(error as Error).stack}`)
+      program.error(`✘ Failed to deploy contracts, error: ${buildErrorOutput(error, options.debug)}`)
     }
   })
 
@@ -182,13 +188,14 @@
   .option('-d, --dir <directory-of-stored-images>', 'directory where to store the images')
   .option('-n, --number <number-of-images>', 'number of images to generate', '1')
   .option('-s, --size <size-of-image>', 'size of the image to generate', '512x512')
+  .option('--debug', 'show detailed debug information such as error stack traces')
   .action(async (options, args) => {
     try {
       const config = getConfig(options)
       const networkId = checkAndGetNetworkId(options.network)
       const openaiAPIKey = config.networks[networkId].settings.openaiAPIKey
       if (!openaiAPIKey) {
-        program.error('OpenAI API key not specified')
+        throw new Error('OpenAI API key not specified')
       }
       const numberOfImages = Number(options.number)
       const imageSize = options.size as CreateImageRequestSizeEnum
@@ -197,7 +204,7 @@
 
       await generateImagesWithOpenAI(openaiAPIKey, prompt, numberOfImages, imageSize, storedDir)
     } catch (error) {
-      program.error(`Failed to generate images, error: ${(error as Error).stack}`)
+      program.error(`✘ Failed to generate images, error: ${buildErrorOutput(error, options.debug)}`)
     }
   })
 
@@ -209,6 +216,7 @@
   .option('-d, --localDir <directory-of-local-images>', 'directory of local images to be uploaded')
   .option('-i, --ipfsDir <ipfs-directory-of-uploaded-images>', 'IPFS directory to upload the images')
   .option('-m, --metadataFile <metadata-file>', 'file to store the metadata of the uploaded images')
+  .option('--debug', 'show detailed debug information such as error stack traces')
   .action(async (options) => {
     try {
       const localDir = options.localDir as string
@@ -216,18 +224,18 @@
       const metadataFile = options.metadataFile as string
       const config = getConfig(options)
       const networkId = checkAndGetNetworkId(options.network)
       const settings = config.networks[networkId].settings
       const projectId = settings.ipfs.infura.projectId
       const projectSecret = settings.ipfs.infura.projectSecret
       if (!projectId || !projectSecret) {
-        program.error('Infura project id or secret not specified')
+        throw new Error('Infura project id or secret not specified')
       }
 
       const result = await uploadImagesAndMetadataToIPFS(localDir, ipfsDir, metadataFile, projectId, projectSecret)
       console.log('NFTBaseUri:')
       console.log(result)
     } catch (error) {
-      program.error(`Failed to upload images, error: ${(error as Error).stack}`)
+      program.error(`✘ Failed to upload images, error: ${buildErrorOutput(error, options.debug)}`)
     }
   })
 
@@ -236,6 +244,7 @@
   .description('validate nft base uri for pre-designed collection')
   .option('-n, --nftBaseUri <nft-base-uri>', 'NFT base uri')
   .option('-m, --maxSupply <max-supply-of-the-pre-designed-collection>', 'max supply of the NFT collection')
+  .option('--debug', 'show detailed debug information such as error stack traces')
   .action(async (options) => {
     try {
       const nftBaseUri = options.nftBaseUri as string
@@ -244,7 +253,7 @@
       console.log('Token Metadataz:')
       console.log(result)
     } catch (error) {
-      program.error(`Failed to upload images metadata, error: ${(error as Error).stack} `)
+      program.error(`✘ Failed to upload images metadata, error: ${buildErrorOutput(error, options.debug)}`)
     }
   })