Use CResult in findSocketYmlSync

Author: jdalton

src/utils/alerts-map.mts

@@ -11,6 +11,7 @@ import { getPublicApiToken, setupSdk } from './sdk.mts'
 import { addArtifactToAlertsMap } from './socket-package-alert.mts'
 
 import type { CompactSocketArtifact } from './alert/artifact.mts'
+import type { FoundSocketYml } from './config.mts'
 import type { AlertFilter, AlertsByPurl } from './socket-package-alert.mts'
 import type { LockfileObject } from '@pnpm/lockfile.fs'
 import type { Spinner } from '@socketsecurity/registry/lib/spinner'
@@ -83,7 +84,10 @@ export async function getAlertsMapFromPurls(
     throw new Error('Auth error: Run `socket login` first')
   }
   const sockSdk = sockSdkCResult.data
-  const socketYml = findSocketYmlSync()?.parsed
+  const socketYmlResult = findSocketYmlSync()
+  const socketYml = socketYmlResult.ok
+    ? (socketYmlResult.data as FoundSocketYml).parsed
+    : undefined
 
   const alertsMapOptions = {
     consolidate: opts.consolidate,

src/utils/config.mts

@@ -111,7 +111,7 @@ export type FoundSocketYml = {
 
 export function findSocketYmlSync(
   dir = process.cwd(),
-): FoundSocketYml | undefined {
+): CResult<FoundSocketYml | undefined> {
   let prevDir = null
   while (dir !== prevDir) {
     let ymlPath = path.join(dir, 'socket.yml')
@@ -123,18 +123,25 @@ export function findSocketYmlSync(
     if (typeof yml === 'string') {
       try {
         return {
-          path: ymlPath,
-          parsed: config.parseSocketConfig(yml),
+          ok: true,
+          data: {
+            path: ymlPath,
+            parsed: config.parseSocketConfig(yml),
+          },
         }
       } catch (e) {
         debugDir('inspect', { error: e })
-        throw new Error(`Found file but was unable to parse ${ymlPath}`)
+        return {
+          ok: false,
+          message: `Found file but was unable to parse ${ymlPath}`,
+          cause: e instanceof Error ? e.message : String(e),
+        }
       }
     }
     prevDir = dir
     dir = path.join(dir, '..')
   }
-  return undefined
+  return { ok: true, data: undefined }
 }
 
 export function getConfigValue<Key extends keyof LocalConfig>(