improve Coana error logging and add e2e test retries

- Include exit code and signal in Coana failure messages
- Log Coana version, target, and cwd on reachability analysis failure
- Add retry: 2 to all scan reach e2e tests for transient CI failures

Author: mtorp

src/commands/scan/cmd-scan-reach.e2e.test.mts

@@ -394,7 +394,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: longTestTimeout },
+      { retry: 2, timeout: longTestTimeout },
     )
 
     cmdit(
@@ -491,7 +491,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
   })
 
@@ -603,7 +603,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
 
     cmdit(
@@ -681,7 +681,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
 
     cmdit(
@@ -779,7 +779,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
 
     cmdit(
@@ -836,7 +836,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
 
     cmdit(
@@ -923,7 +923,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
   })
 
@@ -1027,7 +1027,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
 
     cmdit(
@@ -1127,7 +1127,7 @@ describe('socket scan reach (E2E tests)', async () => {
           await tempFixture.cleanup()
         }
       },
-      { timeout: testTimeout },
+      { retry: 2, timeout: testTimeout },
     )
   })
 })

src/commands/scan/perform-reachability-analysis.mts

@@ -1,5 +1,7 @@
 import path from 'node:path'
 
+import { logger } from '@socketsecurity/registry/lib/logger'
+
 import constants from '../../constants.mts'
 import { handleApiCall } from '../../utils/api.mts'
 import { extractTier1ReachabilityScanId } from '../../utils/coana.mts'
@@ -227,15 +229,25 @@ export async function performReachabilityAnalysis(
     spinner.start()
   }
 
-  return coanaResult.ok
-    ? {
-        ok: true,
-        data: {
-          // Use the actual output filename for the scan.
-          reachabilityReport: outputFilePath,
-          tier1ReachabilityScanId:
-            extractTier1ReachabilityScanId(outputFilePath),
-        },
-      }
-    : coanaResult
+  if (!coanaResult.ok) {
+    const coanaVersion =
+      reachabilityOptions.reachVersion ||
+      constants.ENV.INLINED_SOCKET_CLI_COANA_TECH_CLI_VERSION
+    logger.error(
+      `Coana reachability analysis failed. Version: ${coanaVersion}, target: ${analysisTarget}, cwd: ${cwd}`,
+    )
+    if (coanaResult.message) {
+      logger.error(`Details: ${coanaResult.message}`)
+    }
+    return coanaResult
+  }
+
+  return {
+    ok: true,
+    data: {
+      // Use the actual output filename for the scan.
+      reachabilityReport: outputFilePath,
+      tier1ReachabilityScanId: extractTier1ReachabilityScanId(outputFilePath),
+    },
+  }
 }

src/utils/dlx.mts

@@ -278,8 +278,21 @@ export async function spawnCoanaDlx(
     return { ok: true, data: output.stdout }
   } catch (e) {
     const stderr = (e as any)?.stderr
+    const exitCode = (e as any)?.code
+    const signal = (e as any)?.signal
     const cause = getErrorCause(e)
-    const message = stderr || cause
+    // Build a descriptive error message with exit code and signal details.
+    const details: string[] = []
+    if (typeof exitCode === 'number') {
+      details.push(`exit code ${exitCode}`)
+    }
+    if (signal) {
+      details.push(`signal ${signal}`)
+    }
+    const detailSuffix = details.length ? ` (${details.join(', ')})` : ''
+    const message = stderr
+      ? `Coana command failed${detailSuffix}: ${stderr}`
+      : `Coana command failed${detailSuffix}: ${cause}`
     return {
       ok: false,
       data: e,