refactor: use Promise.allSettled to prevent memory leaks

Convert Promise.all to Promise.allSettled in scripts and tests to
ensure all operations complete even if one fails. This prevents memory
leaks from short-circuit behavior and provides better error handling
for parallel operations.

Updated result handling to extract values from PromiseSettledResult.

Author: jdalton

scripts/build.mjs

@@ -344,7 +344,7 @@ async function main() {
       }
 
       // Run source and types builds in parallel
-      const [srcResult, typesExitCode] = await Promise.all([
+      const results = await Promise.allSettled([
         buildSource({
           quiet,
           verbose,
@@ -353,6 +353,8 @@ async function main() {
         }),
         buildTypes({ quiet, verbose, skipClean: true }),
       ])
+      const srcResult = results[0].status === 'fulfilled' ? results[0].value : undefined
+      const typesExitCode = results[1].status === 'fulfilled' ? results[1].value : 1
 
       // Log completion messages
       if (!quiet) {

scripts/claude.mjs

@@ -3414,12 +3414,16 @@ async function runAudit(claudeCmd, options = {}) {
   log.step('Gathering project information')
 
   // Run various checks.
-  const [npmAudit, depCheck, licenseCheck] = await Promise.all([
+  const results = await Promise.allSettled([
     runCommandWithOutput('npm', ['audit', '--json']),
     runCommandWithOutput('pnpm', ['licenses', 'list', '--json']),
     fs.readFile(path.join(rootPath, 'package.json'), 'utf8'),
   ])
 
+  const npmAudit = results[0].status === 'fulfilled' ? results[0].value : ''
+  const depCheck = results[1].status === 'fulfilled' ? results[1].value : ''
+  const licenseCheck = results[2].status === 'fulfilled' ? results[2].value : '{}'
+
   const packageJson = JSON.parse(licenseCheck)
 
   const prompt = `Perform a comprehensive audit of the project:

scripts/utils/run-command.mjs

@@ -81,7 +81,8 @@ export async function runParallel(commands) {
   const promises = commands.map(({ args = [], command, options = {} }) =>
     runCommand(command, args, options),
   )
-  return Promise.all(promises)
+  const results = await Promise.allSettled(promises)
+  return results.map(r => r.status === 'fulfilled' ? r.value : 1)
 }
 
 /**

test/entitlements.test.mts

@@ -369,7 +369,8 @@ describe('Entitlements API', () => {
           client.getEnabledEntitlements('concurrent-org'),
       )
 
-      const results = await Promise.all(promises)
+      const settled = await Promise.allSettled(promises)
+      const results = settled.filter(r => r.status === 'fulfilled').map(r => r.value)
 
       results.forEach((result: string[]) => {
         expect(result).toEqual(['firewall'])
@@ -392,7 +393,8 @@ describe('Entitlements API', () => {
           client.getEnabledEntitlements(`org-${i}`),
       )
 
-      const results = await Promise.all(promises)
+      const settled = await Promise.allSettled(promises)
+      const results = settled.filter(r => r.status === 'fulfilled').map(r => r.value)
 
       results.forEach((result: string[], i: number) => {
         expect(result).toEqual([`product-${i}`])