fix: don't relink injected packages if they were not built (#5792)

pnpm · Dec 13, 2022 · 6b00a83 · 6b00a83
1 parent 90ec7fc
commit 6b00a83
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.changeset/soft-peaches-boil.md b/.changeset/soft-peaches-boil.md
@@ -0,0 +1,6 @@
+---
+"@pnpm/lifecycle": patch
+"pnpm": patch
+---
+
+Don't relink injected directories if they were not built.
diff --git a/exec/lifecycle/src/runLifecycleHooksConcurrently.ts b/exec/lifecycle/src/runLifecycleHooksConcurrently.ts
@@ -49,11 +49,13 @@ export async function runLifecycleHooksConcurrently (
           pkgRoot: rootDir,
           rootModulesDir: modulesDir,
         }
+        let isBuilt = false
         for (const stage of (importerStages ?? stages)) {
           if ((manifest.scripts == null) || !manifest.scripts[stage]) continue
           await runLifecycleHook(stage, manifest, runLifecycleHookOpts)
+          isBuilt = true
         }
-        if (targetDirs == null || targetDirs.length === 0) return
+        if (targetDirs == null || targetDirs.length === 0 || !isBuilt) return
         const filesResponse = await fetchFromDir(rootDir, { resolveSymlinks: opts.resolveSymlinksInInjectedDirs })
         await Promise.all(
           targetDirs.map(async (targetDir) => {