test(patch): use Promise.allSettled for cleanup operations

Convert all patch test cleanup functions to use Promise.allSettled
to ensure all node_modules and patches directories are removed even
if one deletion fails. This prevents test pollution from failed
cleanup operations.

Author: jdalton

packages/cli/src/commands/patch/cmd-patch-cleanup.test.mts

@@ -14,7 +14,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch-discover.test.mts

@@ -15,7 +15,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch-download.test.mts

@@ -14,7 +14,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch-get.test.mts

@@ -16,7 +16,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  await Promise.all([
+  await Promise.allSettled([
     safeDelete(path.join(pnpmFixtureDir, 'node_modules')),
     safeDelete(path.join(fixtureBaseDir, 'npm/node_modules')),
     safeDelete(path.join(fixtureBaseDir, 'yarn/node_modules')),
@@ -25,7 +25,7 @@ async function cleanupNodeModules() {
 
 async function cleanupPatchesDir() {
   // Clean up generated patches directories.
-  await Promise.all([
+  await Promise.allSettled([
     safeDelete(path.join(pnpmFixtureDir, 'patches')),
     safeDelete(path.join(process.cwd(), 'patches')),
   ])

packages/cli/src/commands/patch/cmd-patch-info.test.mts

@@ -14,7 +14,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch-list.test.mts

@@ -14,7 +14,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch-rm.test.mts

@@ -42,7 +42,7 @@ const originalManifest = {
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  Promise.all([
+  Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,

packages/cli/src/commands/patch/cmd-patch.test.mts

@@ -14,7 +14,7 @@ const pnpmFixtureDir = path.join(fixtureBaseDir, 'pnpm')
 
 async function cleanupNodeModules() {
   // Clean up node_modules from all package manager directories.
-  await Promise.all([
+  await Promise.allSettled([
     fs.rm(path.join(pnpmFixtureDir, 'node_modules'), {
       force: true,
       recursive: true,