Add back getAgentExecPath logic thought to be redundant

Author: jdalton

src/utils/package-environment.mts

@@ -212,6 +212,31 @@ const LOCKS: Record<string, Agent> = {
 
 async function getAgentExecPath(agent: Agent): Promise<string> {
   const binName = binByAgent.get(agent)!
+  if (binName === NPM) {
+    // Try to use constants.npmExecPath first, but verify it exists.
+    const npmPath = constants.npmExecPath
+    if (existsSync(npmPath)) {
+      return npmPath
+    }
+    // If npmExecPath doesn't exist, try common locations.
+    // Check npm in the same directory as node.
+    const nodeDir = path.dirname(process.execPath)
+    const npmInNodeDir = path.join(nodeDir, 'npm')
+    if (existsSync(npmInNodeDir)) {
+      return npmInNodeDir
+    }
+    // Fall back to whichBin.
+    return (await whichBin(binName, { nothrow: true })) ?? binName
+  }
+  if (binName === PNPM) {
+    // Try to use constants.pnpmExecPath first, but verify it exists.
+    const pnpmPath = constants.pnpmExecPath
+    if (existsSync(pnpmPath)) {
+      return pnpmPath
+    }
+    // Fall back to whichBin.
+    return (await whichBin(binName, { nothrow: true })) ?? binName
+  }
   return (await whichBin(binName, { nothrow: true })) ?? binName
 }