feat(apply): add DEV check

Author: unadlib

src/apply.ts

@@ -33,7 +33,7 @@ import { create } from './create';
 export function apply<
   T extends object,
   F extends boolean = false,
-  A extends ApplyOptions<F> = ApplyOptions<F>
+  A extends ApplyOptions<F> = ApplyOptions<F>,
 >(state: T, patches: Patches, applyOptions?: A): ApplyResult<T, F, A> {
   let i: number;
   for (i = patches.length - 1; i >= 0; i -= 1) {
@@ -124,6 +124,15 @@ export function apply<
     });
   };
   if ((applyOptions as ApplyMutableOptions)?.mutable) {
+    if (__DEV__) {
+      if (
+        Object.keys(applyOptions!).filter((key) => key !== 'mutable').length
+      ) {
+        console.warn(
+          'The "mutable" option is not allowed to be used with other options.'
+        );
+      }
+    }
     mutate(state);
     return undefined as ApplyResult<T, F, A>;
   }

test/dev.test.ts

@@ -1,7 +1,7 @@
 /* eslint-disable consistent-return */
 /* eslint-disable no-param-reassign */
 /* eslint-disable @typescript-eslint/ban-ts-comment */
-import { create } from '../src';
+import { apply, create } from '../src';
 
 test('custom shallow copy without checking in prod mode', () => {
   global.__DEV__ = false;
@@ -88,3 +88,53 @@ test('custom shallow copy with checking in dev mode', () => {
     `"You can't use mark and patches or auto freeze together."`
   );
 });
+
+test('check warn when apply patches with other options', () => {
+  {
+    global.__DEV__ = true;
+    const baseState = { foo: { bar: 'test' } };
+    const warn = console.warn;
+    jest.spyOn(console, 'warn').mockImplementation(() => {});
+    apply(
+      baseState,
+      [
+        {
+          op: 'replace',
+          path: ['foo', 'bar'],
+          value: 'test2',
+        },
+      ],
+      {
+        mutable: true,
+        enableAutoFreeze: true,
+      }
+    );
+    expect(console.warn).toHaveBeenCalledWith(
+      'The "mutable" option is not allowed to be used with other options.'
+    );
+  }
+  {
+    global.__DEV__ = true;
+    const baseState = { foo: { bar: 'test' } };
+    const warn = console.warn;
+    jest.spyOn(console, 'warn').mockImplementation(() => {});
+    apply(
+      baseState,
+      [
+        {
+          op: 'replace',
+          path: ['foo', 'bar'],
+          value: 'test2',
+        },
+      ],
+      {
+        mutable: true,
+        enableAutoFreeze: true,
+        mark: () => {},
+      }
+    );
+    expect(console.warn).toHaveBeenCalledWith(
+      'The "mutable" option is not allowed to be used with other options.'
+    );
+  }
+});