Fix linting

src/array.js

@@ -36,6 +36,7 @@ export const recurseArray = ({
     })
 
     if (state.value !== undefined) {
+      // eslint-disable-next-line fp/no-mutating-methods
       newArray.push(state.value)
     }
   }

src/cycle.js

@@ -18,6 +18,7 @@ export const checkCycleThenRecurse = ({
   }
 
   if (ancestors.has(value)) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: value,

src/get.js

@@ -16,6 +16,7 @@ export const safeGetProp = ({ parent, key, changes, path }) => {
     const prop = getProp({ parent, key, changes, path })
     return { prop, safe: true }
   } catch (error) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: undefined,
@@ -40,6 +41,7 @@ const getProp = ({ parent, key, changes, path }) => {
 // When `parent[key]` was a getter and|or setter
 const addGetterChange = ({ changes, path, prop, descriptor: { get, set } }) => {
   if (get !== undefined || set !== undefined) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: get,
@@ -57,6 +59,7 @@ const addDescriptorChange = ({
   descriptor: { writable, configurable },
 }) => {
   if (writable === false) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: prop,
@@ -66,6 +69,7 @@ const addDescriptorChange = ({
   }
 
   if (configurable === false) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: prop,

src/get.test.js

@@ -16,6 +16,7 @@ each(
   ],
   ({ title }, { descriptor, reason }) => {
     test(`Make properties configurable and writable | ${title}`, (t) => {
+      // eslint-disable-next-line fp/no-mutating-methods
       const input = Object.defineProperty({}, 'prop', {
         value: true,
         enumerable: true,
@@ -60,7 +61,7 @@ each(
       input: {
         // eslint-disable-next-line fp/no-get-set
         get prop() {
-          // eslint-disable-next-line fp/no-this
+          // eslint-disable-next-line fp/no-mutating-methods, fp/no-this
           Object.defineProperty(this, 'prop', {
             value: true,
             enumerable: true,
@@ -106,6 +107,7 @@ test('Resolve setters without getters', (t) => {
 
 test('Omit getters that throw', (t) => {
   const error = new Error('test')
+  // eslint-disable-next-line fp/no-mutating-methods
   const input = Object.defineProperty({}, 'prop', {
     get: () => {
       throw error.message

src/indices.js

@@ -14,6 +14,7 @@ export const addNotArrayIndexChanges = (array, changes, path) => {
 
   for (const key of Reflect.ownKeys(array)) {
     if (!arrayProps.has(key)) {
+      // eslint-disable-next-line fp/no-mutating-methods
       changes.push({
         path: [...path, key],
         oldValue: safeGetChangeProp({ parent: array, key }),

src/indices.test.js

@@ -20,6 +20,7 @@ each(
   // eslint-disable-next-line max-params
   ({ title }, key, enumerable, { descriptor, oldValue }) => {
     test(`Omit array properties that are not indices | ${title}`, (t) => {
+      // eslint-disable-next-line fp/no-mutating-methods
       const array = Object.defineProperty([true], key, {
         ...descriptor,
         enumerable,

src/key.js

@@ -3,6 +3,7 @@
 //  - Non-enumerable properties, except in arrays
 export const omitInvalidKey = ({ parent, key, prop, changes, path }) => {
   if (typeof key === 'symbol') {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: prop,
@@ -13,6 +14,7 @@ export const omitInvalidKey = ({ parent, key, prop, changes, path }) => {
   }
 
   if (!isEnum.call(parent, key) && !Array.isArray(parent)) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: prop,

src/key.test.js

@@ -19,6 +19,7 @@ each(
       ],
     },
     {
+      // eslint-disable-next-line fp/no-mutating-methods
       input: Object.defineProperty({}, 'prop', {
         value: true,
         enumerable: false,
@@ -36,6 +37,7 @@ each(
       ],
     },
     {
+      // eslint-disable-next-line fp/no-mutating-methods
       input: Object.defineProperty([], '0', {
         value: true,
         enumerable: false,

src/main.test.js

@@ -12,6 +12,7 @@ test('Is deep by default on arrays', (t) => {
 
 test('Can be shallow on objects', (t) => {
   const value = { one: 0n }
+  // eslint-disable-next-line fp/no-mutating-methods
   Object.defineProperty(value, 'two', {
     value: true,
     enumerable: false,

src/object.js

@@ -57,6 +57,7 @@ const getNewObject = (object) =>
 //  - This mimics `JSON.stringify()` behavior
 const addClassChange = ({ object, newObject, changes, path }) => {
   if (!isPlainObj(object)) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: object,

src/size.js

@@ -36,6 +36,7 @@ export const addSize = ({ type, size, maxSize, changes, path, context }) => {
     return { size: newSize, stop }
   }
 
+  // eslint-disable-next-line fp/no-mutating-methods
   changes.push({
     path,
     oldValue: getOldValue(context),

src/size.test.js

@@ -168,6 +168,7 @@ each(
       change: { reason: 'ignoredUndefined' },
     },
     {
+      // eslint-disable-next-line fp/no-mutating-methods
       input: Object.defineProperty({}, 'prop', {
         get: () => {
           throw error

src/to_json.js

@@ -18,6 +18,7 @@ export const callToJSON = (value, changes, path) => {
 
   try {
     const toJSONResult = triggerToJSON(value)
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: value,
@@ -26,6 +27,7 @@ export const callToJSON = (value, changes, path) => {
     })
     return toJSONResult
   } catch (error) {
+    // eslint-disable-next-line fp/no-mutating-methods
     changes.push({
       path,
       oldValue: value,

src/type.js

@@ -9,6 +9,7 @@ export const omitInvalidTypes = (value, changes, path) => {
     return value
   }
 
+  // eslint-disable-next-line fp/no-mutating-methods
   changes.push({ path, oldValue: value, newValue: undefined, reason })
 }
 

src/uncaught.js

@@ -20,6 +20,7 @@ export const handleUnsafeException = ({
   error,
   size,
 }) => {
+  // eslint-disable-next-line fp/no-mutating-methods
   changes.push({
     path,
     oldValue: value,