Prune pnpm lockfile before writing to disk (#64)

src/isolate.ts

@@ -156,13 +156,14 @@ export async function isolate(
    * Copy the target manifest file to the isolate location and adapt its
    * workspace dependencies to point to the isolated packages.
    */
-  await adaptTargetPackageManifest({
+  const outputManifest = await adaptTargetPackageManifest({
     manifest: targetPackageManifest,
     packagesRegistry,
-    isolateDir,
     workspaceRootDir,
   });
 
+  await writeManifest(isolateDir, outputManifest);
+
   /** Generate an isolated lockfile based on the original one */
   const usedFallbackToNpm = await processLockfile({
     workspaceRootDir,
@@ -171,6 +172,7 @@ export async function isolate(
     internalDepPackageNames: internalPackageNames,
     targetPackageDir,
     targetPackageName: targetPackageManifest.name,
+    targetPackageManifest: outputManifest,
   });
 
   if (usedFallbackToNpm) {

src/lib/lockfile/helpers/generate-pnpm-lockfile.ts

@@ -3,12 +3,13 @@ import {
   readWantedLockfile,
   writeWantedLockfile,
 } from "@pnpm/lockfile-file";
+import { pruneLockfile } from "@pnpm/prune-lockfile";
 import assert from "node:assert";
 import path from "node:path";
 import { pick } from "remeda";
 import { useConfig } from "~/lib/config";
 import { useLogger } from "~/lib/logger";
-import type { PackagesRegistry } from "~/lib/types";
+import type { PackageManifest, PackagesRegistry } from "~/lib/types";
 import { getErrorMessage } from "~/lib/utils";
 import { pnpmMapImporter } from "./pnpm-map-importer";
 
@@ -18,12 +19,14 @@ export async function generatePnpmLockfile({
   isolateDir,
   internalDepPackageNames,
   packagesRegistry,
+  targetPackageManifest,
 }: {
   workspaceRootDir: string;
   targetPackageDir: string;
   isolateDir: string;
   internalDepPackageNames: string[];
   packagesRegistry: PackagesRegistry;
+  targetPackageManifest: PackageManifest;
 }) {
   const { includeDevDependencies, includePatchedDependencies } = useConfig();
   const log = useLogger();
@@ -91,8 +94,15 @@ export async function generatePnpmLockfile({
       )
     );
 
+    log.debug("Pruning the lockfile");
+    const prunedLockfile = await pruneLockfile(
+      lockfile,
+      targetPackageManifest,
+      "."
+    );
+
     await writeWantedLockfile(isolateDir, {
-      ...lockfile,
+      ...prunedLockfile,
       /**
        * Don't know how to map the patched dependencies yet, so we just include
        * them but I don't think it would work like this. The important thing for

src/lib/lockfile/process-lockfile.ts

@@ -1,7 +1,7 @@
 import { useConfig } from "../config";
 import { useLogger } from "../logger";
 import { usePackageManager } from "../package-manager";
-import type { PackagesRegistry } from "../types";
+import type { PackageManifest, PackagesRegistry } from "../types";
 import {
   generateNpmLockfile,
   generatePnpmLockfile,
@@ -21,13 +21,15 @@ export async function processLockfile({
   isolateDir,
   internalDepPackageNames,
   targetPackageDir,
+  targetPackageManifest,
 }: {
   workspaceRootDir: string;
   packagesRegistry: PackagesRegistry;
   isolateDir: string;
   internalDepPackageNames: string[];
   targetPackageDir: string;
   targetPackageName: string;
+  targetPackageManifest: PackageManifest;
 }) {
   const log = useLogger();
 
@@ -86,6 +88,7 @@ export async function processLockfile({
         isolateDir,
         internalDepPackageNames,
         packagesRegistry,
+        targetPackageManifest,
       });
       break;
     }

src/lib/manifest/adapt-target-package-manifest.ts

@@ -3,7 +3,6 @@ import { useConfig } from "../config";
 import { usePackageManager } from "../package-manager";
 import type { PackageManifest, PackagesRegistry } from "../types";
 import { adaptManifestInternalDeps, adoptPnpmFieldsFromRoot } from "./helpers";
-import { writeManifest } from "./io";
 
 /**
  * Adapt the output package manifest, so that:
@@ -15,12 +14,10 @@ import { writeManifest } from "./io";
 export async function adaptTargetPackageManifest({
   manifest,
   packagesRegistry,
-  isolateDir,
   workspaceRootDir,
 }: {
   manifest: PackageManifest;
   packagesRegistry: PackagesRegistry;
-  isolateDir: string;
   workspaceRootDir: string;
 }) {
   const packageManager = usePackageManager();
@@ -46,7 +43,7 @@ export async function adaptTargetPackageManifest({
           packagesRegistry,
         });
 
-  const outputManifest = {
+  return {
     ...adaptedManifest,
     /**
      * Scripts are removed by default if not explicitly picked or omitted via
@@ -58,6 +55,4 @@ export async function adaptTargetPackageManifest({
         ? omit(manifest.scripts ?? {}, omitFromScripts)
         : undefined,
   };
-
-  await writeManifest(isolateDir, outputManifest);
 }