@pnpm/config.package-is-installable 1100.0.16 → 1100.1.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @pnpm/package-is-installable
 
+## 1100.1.0
+
+### Minor Changes
+
+- Fixed an installed optional dependency being left without one of its own required dependencies. When a package reached through `optionalDependencies` is installable on the current system but one of its regular `dependencies` is not, a lockfile-based install skipped that dependency and installed the parent anyway, so importing the parent failed with `MODULE_NOT_FOUND`. The dependency is now installed, and an install-check warning reports the incompatibility. A dependency is still only skipped when every path to it is optional, or when the package that pulls it in was itself skipped [#13286](https://github.com/pnpm/pnpm/issues/13286).
+
+### Patch Changes
+
+- Updated dependencies:
+  - @pnpm/cli.meta@1100.0.12
+  - @pnpm/core-loggers@1100.3.0
+  - @pnpm/engine.runtime.system-version@1100.0.7
+  - @pnpm/types@1101.7.0
+
 ## 1100.0.16
 
 ### Patch Changes

package/lib/index.d.ts

@@ -5,14 +5,28 @@ export type { Engine } from './checkEngine.js';
 export type { Platform, WantedPlatform } from './checkPlatform.js';
 export { inferPlatformFromPackageName } from './inferPlatformFromPackageName.js';
 export { UnsupportedEngineError, UnsupportedPlatformError, type WantedEngine, };
-export declare function packageIsInstallable(pkgId: string, pkg: {
+export interface InstallabilityManifest {
     name: string;
     version: string;
     engines?: WantedEngine;
     cpu?: string[];
     os?: string[];
     libc?: string[];
-}, options: {
+}
+/**
+ * The installability verdict on its own — `null` when the package is
+ * installable — without the logging and the `engineStrict` throw that
+ * [packageIsInstallable] wraps around it. Callers that have to classify a
+ * package before they know how to report it use this, so the report is
+ * emitted exactly once.
+ */
+export declare function checkPackageInstallability(pkgId: string, pkg: InstallabilityManifest, options: {
+    nodeVersion?: string;
+    optional: boolean;
+    pnpmVersion?: string;
+    supportedArchitectures?: SupportedArchitectures;
+}): null | UnsupportedEngineError | UnsupportedPlatformError;
+export declare function packageIsInstallable(pkgId: string, pkg: InstallabilityManifest, options: {
     engineStrict?: boolean;
     nodeVersion?: string;
     optional: boolean;

package/lib/index.js

@@ -5,8 +5,18 @@ import { checkPlatform, UnsupportedPlatformError } from './checkPlatform.js';
 import { inferPlatformFromPackageName } from './inferPlatformFromPackageName.js';
 export { inferPlatformFromPackageName } from './inferPlatformFromPackageName.js';
 export { UnsupportedEngineError, UnsupportedPlatformError, };
+/**
+ * The installability verdict on its own — `null` when the package is
+ * installable — without the logging and the `engineStrict` throw that
+ * [packageIsInstallable] wraps around it. Callers that have to classify a
+ * package before they know how to report it use this, so the report is
+ * emitted exactly once.
+ */
+export function checkPackageInstallability(pkgId, pkg, options) {
+    return checkPackage(pkgId, { engines: pkg.engines, ...effectivePlatform(pkg, options.optional) }, options);
+}
 export function packageIsInstallable(pkgId, pkg, options) {
-    const warn = checkPackage(pkgId, { engines: pkg.engines, ...effectivePlatform(pkg, options.optional) }, options);
+    const warn = checkPackageInstallability(pkgId, pkg, options);
     if (warn == null)
         return true;
     installCheckLogger.warn({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/config.package-is-installable",
-  "version": "1100.0.16",
+  "version": "1100.1.0",
   "description": "Checks if a package is installable on the current system",
   "keywords": [
     "pnpm",
@@ -27,22 +27,22 @@
     "!*.map"
   ],
   "dependencies": {
-    "@pnpm/cli.meta": "1100.0.11",
-    "@pnpm/core-loggers": "1100.2.5",
-    "@pnpm/engine.runtime.system-version": "1100.0.6",
+    "@pnpm/cli.meta": "1100.0.12",
+    "@pnpm/core-loggers": "1100.3.0",
+    "@pnpm/engine.runtime.system-version": "1100.0.7",
     "@pnpm/error": "1100.1.0",
-    "@pnpm/types": "1101.6.0",
+    "@pnpm/types": "1101.7.0",
     "detect-libc": "^2.1.2",
     "execa": "npm:safe-execa@0.3.0",
     "memoize": "^11.0.0",
-    "semver": "^7.8.4"
+    "semver": "^7.8.5"
   },
   "peerDependencies": {
     "@pnpm/logger": "^1100.0.0"
   },
   "devDependencies": {
     "@jest/globals": "30.4.1",
-    "@pnpm/config.package-is-installable": "1100.0.16",
+    "@pnpm/config.package-is-installable": "1100.1.0",
     "@pnpm/logger": "1100.0.0",
     "@types/semver": "7.7.1"
   },