feat(proxy-state-tree): don't allow nesting proxies in state tree

Author: FWeinb

packages/node_modules/proxy-state-tree/src/index.test.ts

@@ -66,6 +66,19 @@ describe('OBJECTS', () => {
         tree.get().foo = 'bar2'
       }).toThrow()
     })
+    test('should throw if parts of state are nested', () => {
+      const state = {
+        foo: {
+          nested: 'bar',
+        },
+      }
+      const tree = new ProxyStateTree(state)
+      expect(() => {
+        tree.startMutationTracking()
+        tree.get().foo = tree.get().foo
+        tree.clearMutationTracking()
+      }).toThrowError(/You tried to nest parts of the state/i)
+    })
     test('should track SET mutations', () => {
       const state = {
         foo: 'bar',
@@ -228,6 +241,28 @@ describe('ARRAYS', () => {
         tree.get().foo.push('foo')
       }).toThrow()
     })
+    test('should throw if parts of state are nested', () => {
+      const state = {
+        foo: [undefined],
+        bar: {
+          nested: 'baz',
+        },
+      }
+      const tree = new ProxyStateTree(state)
+      tree.startMutationTracking()
+
+      // Property mutation
+      expect(() => {
+        tree.get().foo[0] = tree.get().bar
+      }).toThrowError(/You tried to nest parts of the state/i)
+
+      // PUSH mutation
+      expect(() => {
+        tree.get().foo.push(tree.get().bar)
+      }).toThrowError(/You tried to nest parts of the state/i)
+
+      tree.clearMutationTracking()
+    })
     test('should track PUSH mutations', () => {
       const state = {
         foo: [],

packages/node_modules/proxy-state-tree/src/proxify.ts

@@ -60,6 +60,12 @@ function createArrayProxy(tree, value, path) {
           )
         }
         return (...args) => {
+          if (args.some((value) => value[IS_PROXY] === true)) {
+            throw new Error(
+              `proxy-state-tree - You tried to nest parts of the state, but it is not allowed`
+            )
+          }
+
           tree.currentMutations.push({
             method: prop,
             path: path,
@@ -84,7 +90,11 @@ function createArrayProxy(tree, value, path) {
           `proxy-state-tree - You are mutating the path "${nestedPath}", but it is not allowed`
         )
       }
-
+      if (value[IS_PROXY] === true) {
+        throw new Error(
+          `proxy-state-tree - You tried to nest parts of the state, but it is not allowed`
+        )
+      }
       tree.currentMutations.push({
         method: 'set',
         path: nestedPath,
@@ -131,6 +141,11 @@ function createObjectProxy(tree, value, path) {
           `proxy-state-tree - You are mutating the path "${nestedPath}", but it is not allowed`
         )
       }
+      if (value[IS_PROXY] === true) {
+        throw new Error(
+          `proxy-state-tree - You tried to nest parts of the state, but it is not allowed`
+        )
+      }
       if (shouldTrackMutations(tree, nestedPath)) {
         if (!(prop in target)) {
           tree.objectChanges.add(path)