refactor(error): route uncaught CLI errors through typed-error display

The typed-error classes in `utils/error/errors.mts` (AuthError, InputError,
ConfigError, NetworkError, FileSystemError, RateLimitError) plus the
matching `formatErrorForDisplay`/`formatErrorForTerminal`/`formatErrorForJson`
helpers were only ever exercised by tests — the CLI entry point used its
own inline `if/else` dispatch that handled AuthError/InputError/Error.

- Replace the inline dispatch in `cli-entry.mts` with calls to
  `formatErrorForTerminal` and `formatErrorForJson`, so any typed error
  now thrown in production gets consistent titles, recovery suggestions,
  and JSON/text formatting.
- Convert `queryApi`'s "Socket API base URL is not configured" throw from
  a generic Error to a ConfigError keyed on CONFIG_KEY_API_BASE_URL. Users
  now see recovery suggestions directing them to `socket config set`
  instead of a bare stack trace.

The remaining typed classes (NetworkError, FileSystemError, RateLimitError)
stay parked. Forcing them into current CResult-returning sites would be a
larger refactor and isn't needed for the display wiring itself to work —
they slot in automatically the moment any throw site adopts them.

Author: jdalton

packages/cli/src/cli-entry.mts

@@ -28,7 +28,6 @@ process.emitWarning = function (warning, ...args) {
   return Reflect.apply(originalEmitWarning, this, [warning, ...args])
 }
 
-import { messageWithCauses, stackWithCauses } from 'pony-cause'
 import lookupRegistryAuthToken from 'registry-auth-token'
 import lookupRegistryUrl from 'registry-url'
 
@@ -53,11 +52,10 @@ import { VITEST } from './env/vitest.mts'
 import meow from './meow.mts'
 import { meowWithSubcommands } from './utils/cli/with-subcommands.mts'
 import {
-  AuthError,
-  captureException,
-  InputError,
-} from './utils/error/errors.mts'
-import { failMsgWithBadge } from './utils/error/fail-msg-with-badge.mts'
+  formatErrorForJson,
+  formatErrorForTerminal,
+} from './utils/error/display.mts'
+import { captureException } from './utils/error/errors.mts'
 import { serializeResultJson } from './utils/output/result-json.mts'
 import { runPreflightDownloads } from './utils/preflight/downloads.mts'
 import { isSeaBinary } from './utils/sea/detect.mts'
@@ -181,24 +179,6 @@ void (async () => {
     debug('CLI uncaught error')
     debugDir(e)
 
-    let errorBody: string | undefined
-    let errorTitle: string
-    let errorMessage = ''
-    if (e instanceof AuthError) {
-      errorTitle = 'Authentication error'
-      errorMessage = e.message
-    } else if (e instanceof InputError) {
-      errorTitle = 'Invalid input'
-      errorMessage = e.message
-      errorBody = e.body
-    } else if (e instanceof Error) {
-      errorTitle = 'Unexpected error'
-      errorMessage = messageWithCauses(e)
-      errorBody = stackWithCauses(e)
-    } else {
-      errorTitle = 'Unexpected error with no details'
-    }
-
     // Try to parse the flags, find out if --json is set.
     const isJson = (() => {
       const cli = meow({
@@ -213,20 +193,12 @@ void (async () => {
     })()
 
     if (isJson) {
-      logger.log(
-        serializeResultJson({
-          ok: false,
-          message: errorTitle,
-          cause: errorMessage,
-        }),
-      )
+      logger.log(serializeResultJson(formatErrorForJson(e)))
     } else {
       // Add 2 newlines in stderr to bump below any spinner.
       logger.error('\n')
-      logger.fail(failMsgWithBadge(errorTitle, errorMessage))
-      if (errorBody) {
-        debugDirNs('inspect', { errorBody })
-      }
+      logger.error(formatErrorForTerminal(e))
+      debugDirNs('inspect', { error: e })
     }
 
     await captureException(e)

packages/cli/src/utils/socket/api.mts

@@ -55,6 +55,7 @@ import {
 } from '../ecosystem/requirements.mts'
 import {
   buildErrorCause,
+  ConfigError,
   getNetworkErrorDiagnostics,
 } from '../error/errors.mts'
 
@@ -383,7 +384,10 @@ export async function handleApiCallNoSpinner<T extends SocketSdkOperations>(
 export async function queryApi(path: string, apiToken: string) {
   const baseUrl = getDefaultApiBaseUrl()
   if (!baseUrl) {
-    throw new Error('Socket API base URL is not configured.')
+    throw new ConfigError(
+      'Socket API base URL is not configured.',
+      CONFIG_KEY_API_BASE_URL,
+    )
   }
 
   return await socketHttpRequest(