fix(align-deps): reduce diff output (#1979)

microsoft · Oct 27, 2022 · 8a8c548 · 8a8c548
1 parent e823ec9
commit 8a8c548
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/.changeset/nervous-bags-kiss.md b/.changeset/nervous-bags-kiss.md
@@ -0,0 +1,2 @@
+---
+---
diff --git a/packages/align-deps/src/commands/check.ts b/packages/align-deps/src/commands/check.ts
@@ -1,4 +1,6 @@
 import { info } from "@rnx-kit/console";
+import { pickValues } from "@rnx-kit/tools-language";
+import type { PackageManifest } from "@rnx-kit/tools-node/package";
 import { readPackage } from "@rnx-kit/tools-node/package";
 import chalk from "chalk";
 import { diffLinesUnified } from "jest-diff";
@@ -12,6 +14,18 @@ import { resolve } from "../preset";
 import type { Command, ErrorCode, Options } from "../types";
 import { checkPackageManifestUnconfigured } from "./vigilant";
 
+const visibleKeys = [
+  "name",
+  "version",
+  "dependencies",
+  "peerDependencies",
+  "devDependencies",
+];
+
+function stringify(manifest: PackageManifest): string {
+  return JSON.stringify(pickValues(manifest, visibleKeys), undefined, 2);
+}
+
 export function checkPackageManifest(
   manifestPath: string,
   options: Options,
@@ -58,8 +72,8 @@ export function checkPackageManifest(
   );
 
   // Don't fail when manifests only have whitespace differences.
-  const updatedManifestJson = JSON.stringify(updatedManifest, undefined, 2);
-  const normalizedManifestJson = JSON.stringify(manifest, undefined, 2);
+  const updatedManifestJson = stringify(updatedManifest);
+  const normalizedManifestJson = stringify(manifest);
 
   if (updatedManifestJson !== normalizedManifestJson) {
     if (options.write) {