[IMP] Memory: Improve perf but user can not use Object.defineProperty

Author: Gorash

packages/core/src/Memory/VersionableArray.ts

@@ -45,7 +45,7 @@ const proxyArrayHandler = {
         }
         const params = array[memoryProxyPramsKey];
         if (!params.memory) {
-            return proxyObjectHandler.get(array, prop, proxy);
+            return array[prop];
         }
         switch (prop) {
             case 'indexOf':
@@ -69,7 +69,7 @@ const proxyArrayHandler = {
             case 'filter':
                 return filter.bind(proxy, params);
             default:
-                return proxyObjectHandler.get(array, prop, proxy);
+                return array[prop];
         }
     },
     set(

packages/core/src/Memory/VersionableObject.ts

@@ -71,24 +71,6 @@ export interface ProxyObjectHandler {
 }
 
 export const proxyObjectHandler = {
-    get(obj: object, prop: keyType, proxy: object): MemoryAllowedType {
-        if (prop === memoryProxyPramsKey || prop === 'toString' || prop === 'constructor') {
-            return obj[prop];
-        }
-
-        const desc = Object.getOwnPropertyDescriptor(obj, prop);
-        if (desc?.get) {
-            return desc.get.call(proxy);
-        }
-
-        const params = obj[memoryProxyPramsKey];
-        const protoMethod = params.proto.getters[prop as string];
-        if (protoMethod) {
-            return protoMethod.call(proxy);
-        }
-
-        return obj[prop];
-    },
     set(obj: object, prop: keyType, value: MemoryAllowedType, proxy: object): boolean {
         // Object.assign might try to set the value of the paramsKey. We
         // obviously don't want that. Let it think it succeeded (returning false

packages/core/src/Memory/test/memory.test.ts

@@ -7,7 +7,7 @@ import { VersionableSet } from '../VersionableSet';
 import { makeVersionable, markNotVersionable, proxifyVersionable } from '../Versionable';
 import { memoryProxyPramsKey } from '../const';
 
-describe('core', () => {
+describe.only('core', () => {
     describe('state', () => {
         describe('memory', () => {
             describe('throw', () => {
@@ -723,7 +723,7 @@ describe('core', () => {
                     obj = new CustomCustom();
                     expect(obj.getConstructor()).to.equal('CustomCustom');
                 });
-                it('array getter should keep proxy for "this" (use defineProperty)', () => {
+                it.skip('array getter should keep proxy for "this" (use defineProperty)', () => {
                     // eslint-disable-next-line @typescript-eslint/no-explicit-any
                     const obj = new VersionableArray() as any;
                     let test = 0;