fix: set NODE_PATH when is turned on

packages/config/src/Config.ts

@@ -19,6 +19,7 @@ export interface Config {
   cliOptions: Record<string, any>, // eslint-disable-line
   useBetaCli: boolean
   extraBinPaths: string[]
+  extraEnv: Record<string, string>
   filter: string[]
   filterProd: string[]
   rawLocalConfig: Record<string, any>, // eslint-disable-line

packages/config/src/index.ts

@@ -402,6 +402,18 @@ export default async (
   } else {
     pnpmConfig.extraBinPaths = []
   }
+
+  if (pnpmConfig.preferSymlinkedExecutables && pnpmConfig.virtualStoreDir) {
+    pnpmConfig.extraEnv = {
+      NODE_PATH: path.join(
+        pnpmConfig.lockfileDir ?? pnpmConfig.dir,
+        'node_modules',
+        pnpmConfig.virtualStoreDir,
+        'node_modules'
+      ),
+    }
+  }
+
   if (pnpmConfig['shamefullyFlatten']) {
     warnings.push('The "shamefully-flatten" setting has been renamed to "shamefully-hoist". Also, in most cases you won\'t need "shamefully-hoist". Since v4, a semistrict node_modules structure is on by default (via hoist-pattern=[*]).')
     pnpmConfig.shamefullyHoist = true

packages/core/src/install/extendInstallOptions.ts

@@ -24,6 +24,7 @@ export interface StrictInstallOptions {
   frozenLockfileIfExists: boolean
   enablePnp: boolean
   extraBinPaths: string[]
+  extraEnv: Record<string, string>
   hoistingLimits?: HoistingLimits
   useLockfile: boolean
   saveLockfile: boolean

packages/core/src/install/index.ts

@@ -228,6 +228,7 @@ export async function mutateModules (
   async function _install (): Promise<UpdatedProject[]> {
     const scriptsOpts: RunLifecycleHooksConcurrentlyOptions = {
       extraBinPaths: opts.extraBinPaths,
+      extraEnv: opts.extraEnv,
       rawConfig: opts.rawConfig,
       scriptsPrependNodePath: opts.scriptsPrependNodePath,
       scriptShell: opts.scriptShell,
@@ -961,9 +962,12 @@ const _installInContext: InstallFunction = async (projects, ctx, opts) => {
         const depPaths = Object.keys(dependenciesGraph)
         const rootNodes = depPaths.filter((depPath) => dependenciesGraph[depPath].depth === 0)
 
-        let extraEnv: Record<string, string> | undefined
+        let extraEnv: Record<string, string> | undefined = opts.scriptsOpts.extraEnv
         if (opts.enablePnp) {
-          extraEnv = makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs'))
+          extraEnv = {
+            ...extraEnv,
+            ...makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs')),
+          }
         }
         await buildModules(dependenciesGraph, rootNodes, {
           childConcurrency: opts.childConcurrency,
@@ -1107,7 +1111,10 @@ const _installInContext: InstallFunction = async (projects, ctx, opts) => {
     ])
     if (!opts.ignoreScripts) {
       if (opts.enablePnp) {
-        opts.scriptsOpts.extraEnv = makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs'))
+        opts.scriptsOpts.extraEnv = {
+          ...opts.scriptsOpts.extraEnv,
+          ...makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs')),
+        }
       }
       const projectsToBeBuilt = projectsWithTargetDirs.filter(({ mutation }) => mutation === 'install') as ProjectToBeInstalled[]
       await runLifecycleHooksConcurrently(['preinstall', 'install', 'postinstall', 'prepare'],

packages/headless/src/index.ts

@@ -100,6 +100,7 @@ export interface HeadlessOptions {
   enablePnp?: boolean
   engineStrict: boolean
   extraBinPaths?: string[]
+  extraEnv?: Record<string, string>
   extraNodePaths?: string[]
   preferSymlinkedExecutables?: boolean
   hoistingLimits?: HoistingLimits
@@ -186,6 +187,7 @@ export default async (opts: HeadlessOptions) => {
   const scriptsOpts = {
     optional: false,
     extraBinPaths: opts.extraBinPaths,
+    extraEnv: opts.extraEnv,
     rawConfig: opts.rawConfig,
     scriptsPrependNodePath: opts.scriptsPrependNodePath,
     scriptShell: opts.scriptShell,
@@ -427,9 +429,12 @@ export default async (opts: HeadlessOptions) => {
     if (opts.hoistPattern != null) {
       extraBinPaths.unshift(path.join(virtualStoreDir, 'node_modules/.bin'))
     }
-    let extraEnv: Record<string, string> | undefined
+    let extraEnv: Record<string, string> | undefined = opts.extraEnv
     if (opts.enablePnp) {
-      extraEnv = makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs'))
+      extraEnv = {
+        ...extraEnv,
+        ...makeNodeRequireOption(path.join(opts.lockfileDir, '.pnp.cjs')),
+      }
     }
     await buildModules(graph, Array.from(directNodes), {
       childConcurrency: opts.childConcurrency,

packages/plugin-commands-installation/src/install.ts

@@ -279,6 +279,7 @@ export type InstallCommandOptions = Pick<Config,
 | 'virtualStoreDir'
 | 'workspaceConcurrency'
 | 'workspaceDir'
+| 'extraEnv'
 > & CreateStoreControllerOptions & {
   argv: {
     original: string[]

packages/plugin-commands-installation/src/installDeps.ts

@@ -75,6 +75,7 @@ export type InstallDepsOptions = Pick<Config,
 | 'optional'
 | 'workspaceConcurrency'
 | 'workspaceDir'
+| 'extraEnv'
 > & CreateStoreControllerOptions & {
   argv: {
     original: string[]

packages/plugin-commands-publishing/src/publish.ts

@@ -171,6 +171,7 @@ Do you want to continue?`,
   const _runScriptsIfPresent = runScriptsIfPresent.bind(null, {
     depPath: dir,
     extraBinPaths: opts.extraBinPaths,
+    extraEnv: opts.extraEnv,
     pkgRoot: dir,
     rawConfig: opts.rawConfig,
     rootModulesDir: await realpathMissing(path.join(dir, 'node_modules')),

packages/plugin-commands-publishing/src/recursivePublish.ts

@@ -27,6 +27,7 @@ Partial<Pick<Config,
 | 'force'
 | 'dryRun'
 | 'extraBinPaths'
+| 'extraEnv'
 | 'fetchRetries'
 | 'fetchRetryFactor'
 | 'fetchRetryMaxtimeout'

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

@@ -9,6 +9,7 @@ export interface StrictRebuildOptions {
   cacheDir: string
   childConcurrency: number
   extraBinPaths: string[]
+  extraEnv: Record<string, string>
   lockfileDir: string
   nodeLinker: 'isolated' | 'hoisted' | 'pnp'
   scriptShell?: string

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

@@ -154,6 +154,7 @@ export async function rebuildProjects (
   const store = await createOrConnectStoreController(opts)
   const scriptsOpts = {
     extraBinPaths: ctx.extraBinPaths,
+    extraEnv: opts.extraEnv,
     rawConfig: opts.rawConfig,
     scriptsPrependNodePath: opts.scriptsPrependNodePath,
     scriptShell: opts.scriptShell,
@@ -278,6 +279,7 @@ async function _rebuild (
         await runPostinstallHooks({
           depPath,
           extraBinPaths: ctx.extraBinPaths,
+          extraEnv: opts.extraEnv,
           optional: pkgSnapshot.optional === true,
           pkgRoot,
           rawConfig: opts.rawConfig,

packages/plugin-commands-script-runners/src/run.ts

@@ -116,7 +116,7 @@ For options that may be used with `-r`, see "pnpm help recursive"',
 export type RunOpts =
   & Omit<RecursiveRunOpts, 'allProjects' | 'selectedProjectsGraph' | 'workspaceDir'>
   & { recursive?: boolean }
-  & Pick<Config, 'dir' | 'engineStrict' | 'extraBinPaths' | 'reporter' | 'scriptsPrependNodePath' | 'scriptShell' | 'shellEmulator' | 'enablePrePostScripts' | 'userAgent'>
+  & Pick<Config, 'dir' | 'engineStrict' | 'extraBinPaths' | 'reporter' | 'scriptsPrependNodePath' | 'scriptShell' | 'shellEmulator' | 'enablePrePostScripts' | 'userAgent' | 'extraEnv'>
   & (
     & { recursive?: false }
     & Partial<Pick<Config, 'allProjects' | 'selectedProjectsGraph' | 'workspaceDir'>>
@@ -134,6 +134,8 @@ export async function handler (
   opts: RunOpts,
   params: string[]
 ) {
+  console.log(opts.extraEnv)
+
   let dir: string
   const [scriptName, ...passedThruArgs] = params
   if (opts.recursive) {
@@ -174,6 +176,7 @@ so you may run "pnpm -w run ${scriptName}"`,
   const lifecycleOpts: RunLifecycleHookOptions = {
     depPath: dir,
     extraBinPaths: opts.extraBinPaths,
+    extraEnv: opts.extraEnv,
     pkgRoot: dir,
     rawConfig: opts.rawConfig,
     rootModulesDir: await realpathMissing(path.join(dir, 'node_modules')),
@@ -188,7 +191,10 @@ so you may run "pnpm -w run ${scriptName}"`,
   const pnpPath = (opts.workspaceDir && await existsPnp(opts.workspaceDir)) ??
     await existsPnp(dir)
   if (pnpPath) {
-    lifecycleOpts.extraEnv = makeNodeRequireOption(pnpPath)
+    lifecycleOpts.extraEnv = {
+      ...lifecycleOpts.extraEnv,
+      ...makeNodeRequireOption(pnpPath),
+    }
   }
   try {
     if (

packages/plugin-commands-script-runners/src/runRecursive.ts

@@ -21,7 +21,7 @@ export type RecursiveRunOpts = Pick<Config,
 | 'shellEmulator'
 | 'stream'
 > & Required<Pick<Config, 'allProjects' | 'selectedProjectsGraph' | 'workspaceDir'>> &
-Partial<Pick<Config, 'extraBinPaths' | 'bail' | 'reverse' | 'sort' | 'workspaceConcurrency'>> &
+Partial<Pick<Config, 'extraBinPaths' | 'extraEnv' | 'bail' | 'reverse' | 'sort' | 'workspaceConcurrency'>> &
 {
   ifPresent?: boolean
 }
@@ -72,6 +72,7 @@ export default async (
           const lifecycleOpts: RunLifecycleHookOptions = {
             depPath: prefix,
             extraBinPaths: opts.extraBinPaths,
+            extraEnv: opts.extraEnv,
             pkgRoot: prefix,
             rawConfig: opts.rawConfig,
             rootModulesDir: await realpathMissing(path.join(prefix, 'node_modules')),
@@ -83,7 +84,10 @@ export default async (
           }
           const pnpPath = workspacePnpPath ?? await existsPnp(prefix)
           if (pnpPath) {
-            lifecycleOpts.extraEnv = makeNodeRequireOption(pnpPath)
+            lifecycleOpts.extraEnv = {
+              ...lifecycleOpts.extraEnv,
+              ...makeNodeRequireOption(pnpPath),
+            }
           }
           if (
             opts.enablePrePostScripts &&