fix: link local bins created by postinstall scripts

close #1801

exec/lifecycle/package.json

@@ -38,6 +38,7 @@
     "@pnpm/core-loggers": "workspace:*",
     "@pnpm/directory-fetcher": "workspace:*",
     "@pnpm/error": "workspace:*",
+    "@pnpm/link-bins": "workspace:*",
     "@pnpm/npm-lifecycle": "^2.0.1",
     "@pnpm/read-package-json": "workspace:*",
     "@pnpm/store-controller-types": "workspace:*",

exec/lifecycle/src/runLifecycleHooksConcurrently.ts

@@ -1,4 +1,6 @@
 import fs from 'fs'
+import { linkBins } from '@pnpm/link-bins'
+import { logger } from '@pnpm/logger'
 import path from 'path'
 import { fetchFromDir } from '@pnpm/directory-fetcher'
 import { type StoreController } from '@pnpm/store-controller-types'
@@ -13,6 +15,8 @@ export type RunLifecycleHooksConcurrentlyOptions = Omit<RunLifecycleHookOptions,
 > & {
   resolveSymlinksInInjectedDirs?: boolean
   storeController: StoreController
+  extraNodePaths?: string[]
+  preferSymlinkedExecutables?: boolean
 }
 
 export interface Importer {
@@ -43,6 +47,16 @@ export async function runLifecycleHooksConcurrently (
     const importers = importersByBuildIndex.get(buildIndex)!
     return importers.map(({ manifest, modulesDir, rootDir, stages: importerStages, targetDirs }) =>
       async () => {
+        // We are linking the bin files, in case they were created by lifecycle scripts of other workspace packages.
+        await linkBins(modulesDir, path.join(modulesDir, '.bin'), {
+          extraNodePaths: opts.extraNodePaths,
+          allowExoticManifests: true,
+          preferSymlinkedExecutables: opts.preferSymlinkedExecutables,
+          projectManifest: manifest,
+          warn: (message: string) => {
+            logger.warn({ message, prefix: rootDir })
+          },
+        })
         const runLifecycleHookOpts = {
           ...opts,
           depPath: rootDir,

exec/lifecycle/tsconfig.json

@@ -24,6 +24,9 @@
     {
       "path": "../../packages/types"
     },
+    {
+      "path": "../../pkg-manager/link-bins"
+    },
     {
       "path": "../../pkg-manifest/read-package-json"
     },

exec/plugin-commands-rebuild/src/implementation/extendRebuildOptions.ts

@@ -14,6 +14,7 @@ export interface StrictRebuildOptions {
   extraEnv: Record<string, string>
   lockfileDir: string
   nodeLinker: 'isolated' | 'hoisted' | 'pnp'
+  preferSymlinkedExecutables?: boolean
   scriptShell?: string
   sideEffectsCacheRead: boolean
   scriptsPrependNodePath: boolean | 'warn-only'

exec/plugin-commands-rebuild/src/implementation/index.ts

@@ -157,7 +157,9 @@ export async function rebuildProjects (
   const store = await createOrConnectStoreController(opts)
   const scriptsOpts = {
     extraBinPaths: ctx.extraBinPaths,
+    extraNodePaths: ctx.extraNodePaths,
     extraEnv: opts.extraEnv,
+    preferSymlinkedExecutables: opts.preferSymlinkedExecutables,
     rawConfig: opts.rawConfig,
     scriptsPrependNodePath: opts.scriptsPrependNodePath,
     scriptShell: opts.scriptShell,

pkg-manager/core/src/install/index.ts

@@ -293,7 +293,9 @@ export async function mutateModules (
   async function _install (): Promise<{ updatedProjects: UpdatedProject[], stats?: InstallationResultStats }> {
     const scriptsOpts: RunLifecycleHooksConcurrentlyOptions = {
       extraBinPaths: opts.extraBinPaths,
+      extraNodePaths: ctx.extraNodePaths,
       extraEnv: opts.extraEnv,
+      preferSymlinkedExecutables: opts.preferSymlinkedExecutables,
       rawConfig: opts.rawConfig,
       resolveSymlinksInInjectedDirs: opts.resolveSymlinksInInjectedDirs,
       scriptsPrependNodePath: opts.scriptsPrependNodePath,

pkg-manager/headless/src/index.ts

@@ -195,6 +195,8 @@ export async function headlessInstall (opts: HeadlessOptions): Promise<Installat
   const scriptsOpts = {
     optional: false,
     extraBinPaths: opts.extraBinPaths,
+    extraNodePaths: opts.extraNodePaths,
+    preferSymlinkedExecutables: opts.preferSymlinkedExecutables,
     extraEnv: opts.extraEnv,
     rawConfig: opts.rawConfig,
     resolveSymlinksInInjectedDirs: opts.resolveSymlinksInInjectedDirs,