pnpm/action-setup
1
0
Fork 0
mirror of https://github.com/pnpm/action-setup.git synced 2026-04-03 19:00:10 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: add @pnpm/exe dir to PATH on Windows instead of .bin shims

Browse Source 
On Windows, npm's .bin shims can't properly execute the extensionless
native binaries from @pnpm/exe. Instead of trying to fix the shims,
add the @pnpm/exe directory directly to PATH where pnpm.exe lives.
This commit is contained in:
Zoltan Kochan
2026-03-27 19:30:36 +01:00
parent cd98fe77eb
commit 5a859cc8cb
2 changed files with 118 additions and 130 deletions
Download Patch File Download Diff File Expand all files Collapse all files

216
dist/index.js vendored
View File

File diff suppressed because one or more lines are too long

32
src/install-pnpm/run.ts
View File

@@ -1,6 +1,6 @@
import { addPath, exportVariable } from '@actions/core'
import { spawn } from 'child_process'
import { rm, writeFile, mkdir, symlink, link, copyFile } from 'fs/promises'
import { rm, writeFile, mkdir, symlink } from 'fs/promises'
import { readFileSync, existsSync } from 'fs'
import path from 'path'
import util from 'util'
@@ -34,33 +34,21 @@ export async function runSelfInstaller(inputs: Inputs): Promise<number> {
return npmExitCode
}
const pnpmHome = path.join(dest, 'node_modules', '.bin')
// On Windows with standalone mode, npm's .bin shims can't properly
// execute the extensionless @pnpm/exe native binaries. Add the
// @pnpm/exe directory directly to PATH so pnpm.exe is found natively.
const pnpmHome = standalone && process.platform === 'win32'
? path.join(dest, 'node_modules', '@pnpm', 'exe')
: path.join(dest, 'node_modules', '.bin')
addPath(pnpmHome)
addPath(path.join(pnpmHome, 'bin'))
exportVariable('PNPM_HOME', pnpmHome)
// On Windows, npm creates .cmd wrappers that invoke bin entries through
// `node`, but @pnpm/exe bins are native executables — not JavaScript.
// Place pnpm.exe directly in .bin/ so that PATHEXT resolution finds the
// .exe before the broken .cmd wrapper.
if (process.platform === 'win32' && standalone) {
const exeDir = path.join(dest, 'node_modules', '@pnpm', 'exe')
for (const name of ['pnpm', 'pnpx']) {
const exe = path.join(exeDir, `${name}.exe`)
const cmd = path.join(exeDir, `${name}.cmd`)
if (existsSync(exe)) {
await copyFile(exe, path.join(pnpmHome, `${name}.exe`))
} else if (existsSync(cmd)) {
await copyFile(cmd, path.join(pnpmHome, `${name}.cmd`))
}
}
}
// Ensure pnpm bin link exists — npm ci sometimes doesn't create it
if (process.platform !== 'win32') {
const pnpmBinLink = path.join(pnpmHome, 'pnpm')
addPath(path.join(pnpmHome, 'bin'))
const pnpmBinLink = path.join(dest, 'node_modules', '.bin', 'pnpm')
if (!existsSync(pnpmBinLink)) {
await mkdir(pnpmHome, { recursive: true })
await mkdir(path.join(dest, 'node_modules', '.bin'), { recursive: true })
const target = standalone
? path.join('..', '@pnpm', 'exe', 'pnpm')
: path.join('..', 'pnpm', 'bin', 'pnpm.mjs')