isolate-package 1.24.0 → 1.25.0

package/dist/isolate-bin.mjs

@@ -1,18 +1,18 @@
 #!/usr/bin/env node
 
 // src/isolate-bin.ts
-import console2 from "node:console";
+import console2 from "console";
 import sourceMaps from "source-map-support";
 
 // src/isolate.ts
 import fs15 from "fs-extra";
-import assert6 from "node:assert";
-import path20 from "node:path";
+import assert6 from "assert";
+import path20 from "path";
 import { unique as unique2 } from "remeda";
 
 // src/lib/config.ts
 import fs8 from "fs-extra";
-import path6 from "node:path";
+import path6 from "path";
 import { isEmpty } from "remeda";
 
 // src/lib/logger.ts
@@ -83,14 +83,14 @@ function toErrorWithMessage(maybeError) {
 }
 
 // src/lib/utils/inspect-value.ts
-import { inspect } from "node:util";
+import { inspect } from "util";
 function inspectValue(value) {
   return inspect(value, false, 16, true);
 }
 
 // src/lib/utils/is-rush-workspace.ts
-import fs from "node:fs";
-import path from "node:path";
+import fs from "fs";
+import path from "path";
 function isRushWorkspace(workspaceRootDir) {
   return fs.existsSync(path.join(workspaceRootDir, "rush.json"));
 }
@@ -126,7 +126,7 @@ async function readTypedJson(filePath) {
 }
 
 // src/lib/utils/log-paths.ts
-import { join } from "node:path";
+import { join } from "path";
 function getRootRelativeLogPath(path21, rootPath) {
   const strippedPath = path21.replace(rootPath, "");
   return join("(root)", strippedPath);
@@ -137,18 +137,18 @@ function getIsolateRelativeLogPath(path21, isolatePath) {
 }
 
 // src/lib/utils/pack.ts
-import assert2 from "node:assert";
-import { exec } from "node:child_process";
-import fs5 from "node:fs";
-import path5 from "node:path";
+import assert2 from "assert";
+import { exec } from "child_process";
+import fs5 from "fs";
+import path5 from "path";
 
 // src/lib/package-manager/index.ts
-import path4 from "node:path";
+import path4 from "path";
 
 // src/lib/package-manager/helpers/infer-from-files.ts
 import fs3 from "fs-extra";
-import { execSync } from "node:child_process";
-import path2 from "node:path";
+import { execSync } from "child_process";
+import path2 from "path";
 
 // src/lib/utils/get-major-version.ts
 function getMajorVersion(version) {
@@ -203,8 +203,8 @@ function getVersion(packageManagerName) {
 
 // src/lib/package-manager/helpers/infer-from-manifest.ts
 import fs4 from "fs-extra";
-import assert from "node:assert";
-import path3 from "node:path";
+import assert from "assert";
+import path3 from "path";
 function inferFromManifest(workspaceRoot) {
   const log = useLogger();
   const { packageManager: packageManagerString } = readTypedJsonSync(
@@ -388,7 +388,7 @@ function resolveConfig(initialConfig) {
 // src/lib/lockfile/helpers/generate-npm-lockfile.ts
 import Arborist from "@npmcli/arborist";
 import fs9 from "fs-extra";
-import path7 from "node:path";
+import path7 from "path";
 
 // src/lib/lockfile/helpers/load-npm-config.ts
 import Config from "@npmcli/config";
@@ -433,8 +433,8 @@ async function generateNpmLockfile({
 }
 
 // src/lib/lockfile/helpers/generate-pnpm-lockfile.ts
-import assert3 from "node:assert";
-import path9 from "node:path";
+import assert3 from "assert";
+import path9 from "path";
 import {
   getLockfileImporterId as getLockfileImporterId_v8,
   readWantedLockfile as readWantedLockfile_v8,
@@ -450,7 +450,7 @@ import { pruneLockfile as pruneLockfile_v9 } from "pnpm_prune_lockfile_v9";
 import { pick } from "remeda";
 
 // src/lib/lockfile/helpers/pnpm-map-importer.ts
-import path8 from "node:path";
+import path8 from "path";
 import { mapValues } from "remeda";
 function pnpmMapImporter(importerPath, { dependencies, devDependencies, ...rest }, {
   includeDevDependencies,
@@ -559,7 +559,8 @@ async function generatePnpmLockfile({
         return [
           importerId,
           pnpmMapImporter(importerId, importer, {
-            includeDevDependencies,
+            includeDevDependencies: false,
+            // Only include dev deps for target package
             includePatchedDependencies,
             directoryByPackageName
           })
@@ -588,8 +589,8 @@ async function generatePnpmLockfile({
 
 // src/lib/lockfile/helpers/generate-yarn-lockfile.ts
 import fs10 from "fs-extra";
-import { execSync as execSync2 } from "node:child_process";
-import path10 from "node:path";
+import { execSync as execSync2 } from "child_process";
+import path10 from "path";
 async function generateYarnLockfile({
   workspaceRootDir,
   isolateDir
@@ -700,12 +701,12 @@ async function processLockfile({
 import { omit as omit2, pick as pick2 } from "remeda";
 
 // src/lib/manifest/helpers/adapt-internal-package-manifests.ts
-import path13 from "node:path";
+import path13 from "path";
 import { omit } from "remeda";
 
 // src/lib/manifest/io.ts
 import fs11 from "fs-extra";
-import path11 from "node:path";
+import path11 from "path";
 async function readManifest(packageDir) {
   return readTypedJson(path11.join(packageDir, "package.json"));
 }
@@ -717,7 +718,7 @@ async function writeManifest(outputDir, manifest) {
 }
 
 // src/lib/manifest/helpers/patch-internal-entries.ts
-import path12 from "node:path";
+import path12 from "path";
 function patchInternalEntries(dependencies, packagesRegistry, parentRootRelativeDir) {
   const log = useLogger();
   const allWorkspacePackageNames = Object.keys(packagesRegistry);
@@ -868,17 +869,17 @@ async function adaptTargetPackageManifest({
 }
 
 // src/lib/manifest/validate-manifest.ts
-function validateManifestMandatoryFields(manifest, packagePath) {
+function validateManifestMandatoryFields(manifest, packagePath, requireFilesField = true) {
   const log = useLogger();
   const missingFields = [];
   if (!manifest.version) {
     missingFields.push("version");
   }
-  if (!manifest.files || !Array.isArray(manifest.files) || manifest.files.length === 0) {
+  if (requireFilesField && (!manifest.files || !Array.isArray(manifest.files) || manifest.files.length === 0)) {
     missingFields.push("files");
   }
   if (missingFields.length > 0) {
-    const errorMessage = `Package at ${packagePath} is missing mandatory fields: ${missingFields.join(", ")}. The "version" field is required for pack to execute, and the "files" field is required to declare what files should be included in the output. See the documentation for more details.`;
+    const errorMessage = `Package at ${packagePath} is missing mandatory fields: ${missingFields.join(", ")}. See the documentation for more details.`;
     log.error(errorMessage);
     throw new Error(errorMessage);
   }
@@ -887,7 +888,7 @@ function validateManifestMandatoryFields(manifest, packagePath) {
 
 // src/lib/output/get-build-output-dir.ts
 import { getTsconfig } from "get-tsconfig";
-import path15 from "node:path";
+import path15 from "path";
 import outdent from "outdent";
 async function getBuildOutputDir({
   targetPackageDir,
@@ -920,7 +921,7 @@ async function getBuildOutputDir({
 }
 
 // src/lib/output/pack-dependencies.ts
-import assert4 from "node:assert";
+import assert4 from "assert";
 async function packDependencies({
   /** All packages found in the monorepo by workspaces declaration */
   packagesRegistry,
@@ -951,7 +952,7 @@ async function packDependencies({
 
 // src/lib/output/process-build-output-files.ts
 import fs12 from "fs-extra";
-import path16 from "node:path";
+import path16 from "path";
 var TIMEOUT_MS = 5e3;
 async function processBuildOutputFiles({
   targetPackageDir,
@@ -976,7 +977,7 @@ async function processBuildOutputFiles({
 
 // src/lib/output/unpack-dependencies.ts
 import fs13 from "fs-extra";
-import path17, { join as join2 } from "node:path";
+import path17, { join as join2 } from "path";
 async function unpackDependencies(packedFilesByName, packagesRegistry, tmpDir, isolateDir) {
   const log = useLogger();
   await Promise.all(
@@ -1003,11 +1004,11 @@ async function unpackDependencies(packedFilesByName, packagesRegistry, tmpDir, i
 // src/lib/registry/create-packages-registry.ts
 import fs14 from "fs-extra";
 import { globSync } from "glob";
-import path19 from "node:path";
+import path19 from "path";
 
 // src/lib/registry/helpers/find-packages-globs.ts
-import assert5 from "node:assert";
-import path18 from "node:path";
+import assert5 from "assert";
+import path18 from "path";
 function findPackagesGlobs(workspaceRootDir) {
   const log = useLogger();
   const packageManager2 = usePackageManager();
@@ -1200,11 +1201,20 @@ function createIsolator(config) {
         includeDevDependencies: config2.includeDevDependencies
       }
     );
+    const productionInternalPackageNames = listInternalPackages(
+      targetPackageManifest,
+      packagesRegistry,
+      {
+        includeDevDependencies: false
+      }
+    );
     for (const packageName of internalPackageNames) {
       const packageDef = packagesRegistry[packageName];
+      const isProductionDependency = productionInternalPackageNames.includes(packageName);
       validateManifestMandatoryFields(
         packageDef.manifest,
-        getRootRelativeLogPath(packageDef.absoluteDir, workspaceRootDir)
+        getRootRelativeLogPath(packageDef.absoluteDir, workspaceRootDir),
+        isProductionDependency
       );
     }
     const packedFilesByName = await packDependencies({