fix(patch-remove): check if the parameters are valid (#9809)

btea · web-flow · commit 81b8a0eacb5c · 2025-07-31T14:02:46.000+02:00
diff --git a/.changeset/wicked-deer-drive.md b/.changeset/wicked-deer-drive.md
@@ -0,0 +1,5 @@
+---
+"@pnpm/plugin-commands-patching": patch
+---
+
+When executing the `pnpm patch-remove` command, verify whether the passed parameter is a valid patch package name.
diff --git a/patching/plugin-commands-patching/src/patchRemove.ts b/patching/plugin-commands-patching/src/patchRemove.ts
@@ -54,6 +54,12 @@ export async function handler (opts: PatchRemoveCommandOptions, params: string[]
     throw new PnpmError('NO_PATCHES_TO_REMOVE', 'There are no patches that need to be removed')
   }
 
+  for (const patch of patchesToRemove) {
+    if (!Object.hasOwn(patchedDependencies, patch)) {
+      throw new PnpmError('PATCH_NOT_FOUND', `Patch "${patch}" not found in patched dependencies`)
+    }
+  }
+
   const patchesDirs = new Set<string>()
   await Promise.all(patchesToRemove.map(async (patch) => {
     if (Object.hasOwn(patchedDependencies, patch)) {

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@pnpm/plugin-commands-patching": patch
 +---
++
 +When executing the `pnpm patch-remove` command, verify whether the passed parameter is a valid patch package name.