Merge pull request #35 from StauroDEV/fix-gw-tty

fix: disable colors for non tty in ping command
StauroDEV · May 28, 2024 · 6db81f0 · 6db81f0
2 parents 4208f74 + 1a85fd6
commit 6db81f0
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/actions/ping.ts b/src/actions/ping.ts
@@ -1,8 +1,9 @@
 import * as colors from 'colorette'
+import { isTTY } from '../constants'
 
 let retryCount = 0
 
-const gwOfflineMessage = `😞 Max retries exceeded. Gateway is ${colors.bold(colors.red('Offline'))}.`
+const gwOfflineMessage = `😞 Max retries exceeded. Gateway is ${isTTY ? colors.bold(colors.red('Offline')) : 'Offline'}.`
 
 export const pingAction = async (
   { cid, endpoint, options }: {
@@ -15,7 +16,7 @@ export const pingAction = async (
 
   retryCount++
   const url = `https://${cid}.ipfs.${endpoint}`
-  console.log(`${colors.bold(`[${retryCount}]`)}: Requesting content at ${url}`)
+  console.log(`${isTTY ? colors.bold(`[${retryCount}]`) : `[${retryCount}]`}: Requesting content at ${url}`)
   try {
     const response = await fetch(url, { signal: AbortSignal.timeout(timeout), redirect: 'follow' })
     if (response.status === 504) {
@@ -31,7 +32,7 @@ export const pingAction = async (
     else {
       return console.log(`Gateway status: ${
         response.status >= 200 && response.status < 400
-        ? colors.bold(colors.green(`Online ${response.status}`))
+        ? (isTTY ? colors.bold(colors.green(`Online ${response.status}`)) : `Online ${response.status}`)
         : response.status
       }`)
     }