Make updatePackageJsonFromNode return a boolean (#452)

Author: jdalton

src/utils/arborist-helpers.ts

@@ -294,28 +294,34 @@ export function updatePackageJsonFromNode(
   node: SafeNode,
   targetVersion: string,
   rangeStyle?: RangeStyle | undefined
-) {
+): boolean {
+  let result = false
   if (isTopLevel(tree, node)) {
     const { name } = node
     for (const depField of [
       'dependencies',
       'optionalDependencies',
       'peerDependencies'
     ]) {
-      const oldValue = editablePkgJson.content[depField] as
+      const depObject = editablePkgJson.content[depField] as
         | { [key: string]: string }
         | undefined
-      if (oldValue) {
-        const refRange = oldValue[name]
-        if (refRange) {
-          editablePkgJson.update({
-            [depField]: {
-              ...oldValue,
-              [name]: applyRange(refRange, targetVersion, rangeStyle)
-            }
-          })
+      if (depObject) {
+        const oldRange = depObject[name]
+        if (oldRange) {
+          const newRange = applyRange(oldRange, targetVersion, rangeStyle)
+          if (oldRange !== newRange) {
+            result = true
+            editablePkgJson.update({
+              [depField]: {
+                ...depObject,
+                [name]: newRange
+              }
+            })
+          }
         }
       }
     }
   }
+  return result
 }