if newPatch is more steps than original patch, just return original

Author: jaydp17

src/optimize.js

@@ -15,7 +15,9 @@ const jsonPatch = require('json8-patch');
 function optimize(doc, patches) {
   const copyDoc = JSON.parse(JSON.stringify(doc));
   const { doc: outputDoc } = jsonPatch.apply(copyDoc, patches);
-  return jsonPatch.diff(doc, outputDoc);
+  const newPatches = jsonPatch.diff(doc, outputDoc);
+  if (newPatches.length >= patches.length) return patches;
+  return newPatches;
 }
 
 module.exports = optimize;

src/optimize.test.js

@@ -40,6 +40,12 @@ const cases = [
     ],
     expected: [{ op: 'add', path: '/b', value: 43 }]
   },
+  {
+    name: 'replace an object',
+    doc: { foo: { abc: { def: 34 } } },
+    patches: [{ op: 'replace', path: '/foo/abc', value: { xyz: 64 } }],
+    expected: [{ op: 'replace', path: '/foo/abc', value: { xyz: 64 } }]
+  },
   {
     name: 'add element to end',
     doc: { foo: ['bar', 'baz'] },