@pnpm/installing.dedupe.check 1100.0.10 → 1100.1.0

package/lib/dedupeDiffCheck.d.ts

@@ -1,4 +1,20 @@
-import type { SnapshotsChanges } from '@pnpm/installing.dedupe.types';
+import type { DedupeCheckIssues, SnapshotsChanges } from '@pnpm/installing.dedupe.types';
 import type { LockfileObject } from '@pnpm/lockfile.types';
+/**
+ * Compute the changes between two lockfiles, as added/removed/updated
+ * importer and package snapshots. Unlike {@link dedupeDiffCheck} this never
+ * throws — callers that only want to report the diff (e.g. `install
+ * --dry-run`) consume the result directly.
+ *
+ * `includeImporterSpecifiers` also diffs each importer's direct-dependency
+ * `specifier`s, not just their resolved versions. `pnpm install --dry-run`
+ * sets it so a specifier-only manifest edit (which a real install would
+ * persist to the lockfile) is reported; `dedupe --check` leaves it off
+ * because a specifier change is irrelevant to deduplication.
+ */
+export declare function calcDedupeCheckIssues(prev: LockfileObject, next: LockfileObject, opts?: {
+    includeImporterSpecifiers?: boolean;
+}): DedupeCheckIssues;
+export declare function countDedupeCheckIssues(issues: DedupeCheckIssues): number;
 export declare function dedupeDiffCheck(prev: LockfileObject, next: LockfileObject): void;
 export declare function countChangedSnapshots(snapshotChanges: SnapshotsChanges): number;

package/lib/dedupeDiffCheck.js

@@ -1,14 +1,41 @@
 import { DEPENDENCIES_FIELDS } from '@pnpm/types';
 import { DedupeCheckIssuesError } from './DedupeCheckIssuesError.js';
 const PACKAGE_SNAPSHOT_DEP_FIELDS = ['dependencies', 'optionalDependencies'];
-export function dedupeDiffCheck(prev, next) {
-    const issues = {
-        importerIssuesByImporterId: diffSnapshots(prev.importers, next.importers, DEPENDENCIES_FIELDS),
+// Dry-run diffs importers by both the per-group dependency fields and the
+// flat `specifiers` map, so it catches every importer rewrite a real install
+// would persist: the per-group fields surface a dependency moving between
+// `dependencies`/`devDependencies`/`optionalDependencies`, while `specifiers`
+// surfaces a specifier-only edit (same resolved version). `specifiers` is
+// last so that on a per-alias collision its update wins — rendering the
+// specifier delta rather than the re-resolved version, which is cleared in
+// memory for a specifier-mismatched dep.
+const IMPORTER_DRY_RUN_FIELDS = [...DEPENDENCIES_FIELDS, 'specifiers'];
+/**
+ * Compute the changes between two lockfiles, as added/removed/updated
+ * importer and package snapshots. Unlike {@link dedupeDiffCheck} this never
+ * throws — callers that only want to report the diff (e.g. `install
+ * --dry-run`) consume the result directly.
+ *
+ * `includeImporterSpecifiers` also diffs each importer's direct-dependency
+ * `specifier`s, not just their resolved versions. `pnpm install --dry-run`
+ * sets it so a specifier-only manifest edit (which a real install would
+ * persist to the lockfile) is reported; `dedupe --check` leaves it off
+ * because a specifier change is irrelevant to deduplication.
+ */
+export function calcDedupeCheckIssues(prev, next, opts) {
+    const importerFields = opts?.includeImporterSpecifiers ? IMPORTER_DRY_RUN_FIELDS : DEPENDENCIES_FIELDS;
+    return {
+        importerIssuesByImporterId: diffSnapshots(prev.importers, next.importers, importerFields),
         packageIssuesByDepPath: diffSnapshots(prev.packages ?? {}, next.packages ?? {}, PACKAGE_SNAPSHOT_DEP_FIELDS),
     };
-    const changesCount = countChangedSnapshots(issues.importerIssuesByImporterId) +
-        countChangedSnapshots(issues.packageIssuesByDepPath);
-    if (changesCount > 0) {
+}
+export function countDedupeCheckIssues(issues) {
+    return (countChangedSnapshots(issues.importerIssuesByImporterId) +
+        countChangedSnapshots(issues.packageIssuesByDepPath));
+}
+export function dedupeDiffCheck(prev, next) {
+    const issues = calcDedupeCheckIssues(prev, next);
+    if (countDedupeCheckIssues(issues) > 0) {
         throw new DedupeCheckIssuesError(issues);
     }
 }

package/lib/index.d.ts

@@ -1,2 +1,2 @@
 export { DedupeCheckIssuesError } from './DedupeCheckIssuesError.js';
-export { countChangedSnapshots, dedupeDiffCheck } from './dedupeDiffCheck.js';
+export { calcDedupeCheckIssues, countChangedSnapshots, countDedupeCheckIssues, dedupeDiffCheck } from './dedupeDiffCheck.js';

package/lib/index.js

@@ -1,3 +1,3 @@
 export { DedupeCheckIssuesError } from './DedupeCheckIssuesError.js';
-export { countChangedSnapshots, dedupeDiffCheck } from './dedupeDiffCheck.js';
+export { calcDedupeCheckIssues, countChangedSnapshots, countDedupeCheckIssues, dedupeDiffCheck } from './dedupeDiffCheck.js';
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/installing.dedupe.check",
-  "version": "1100.0.10",
+  "version": "1100.1.0",
   "description": "Visualize pnpm dedupe --check issues.",
   "keywords": [
     "pnpm",
@@ -29,12 +29,12 @@
   "dependencies": {
     "@pnpm/error": "1100.0.0",
     "@pnpm/installing.dedupe.types": "1100.0.1",
-    "@pnpm/types": "1101.3.1",
-    "@pnpm/lockfile.types": "1100.0.10"
+    "@pnpm/lockfile.types": "1100.0.11",
+    "@pnpm/types": "1101.3.2"
   },
   "devDependencies": {
-    "@jest/globals": "30.3.0",
-    "@pnpm/installing.dedupe.check": "1100.0.10"
+    "@jest/globals": "30.4.1",
+    "@pnpm/installing.dedupe.check": "1100.1.0"
   },
   "engines": {
     "node": ">=22.13"