fix: Static props prevents selectors recall (#101)

* add failing test for #100

* fix it!!!

* update test desc

* update CHANGELOG

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## [Unreleased]
 
+### Changed
+
+- fix: Static props prevents selectors recall #101
+
 ## [3.0.0] - 2024-05-02
 
 ### Changed

src/memoize.ts

@@ -48,7 +48,6 @@ const touchAffected = (dst: unknown, src: unknown, affected: Affected) => {
   if (!isObject(dst) || !isObject(src)) return;
   if (trackMemoOriginalObjSet.has(getUntracked(src) as never)) {
     trackMemo(dst);
-    return;
   }
   const used = affected.get(getUntracked(src) || src);
   if (!used) return;

tests/issue_100.spec.ts

@@ -0,0 +1,44 @@
+import { describe, expect, it } from 'vitest';
+import { memoize } from 'proxy-memoize';
+
+describe('Static props prevents selectors recall (#100)', () => {
+  it('should select correctly with nested selectors', () => {
+    type State = {
+      book1: {
+        staticProp: string;
+        priceString: string;
+      };
+    };
+
+    const state1: State = {
+      book1: {
+        staticProp: '5',
+        priceString: '10',
+      },
+    };
+
+    const state2: State = {
+      book1: {
+        staticProp: '5',
+        priceString: '20',
+      },
+    };
+
+    const selectAllBooks = memoize((state: State) => Object.values(state));
+
+    const selectPriceString = memoize(
+      (state: State) => state.book1.priceString,
+    );
+
+    const selectAdjustedPriceString = memoize((state: State) => {
+      const priceString = selectPriceString(state);
+      state.book1.staticProp; // touch the prop
+      return priceString;
+    });
+
+    selectAllBooks(state1);
+
+    expect(selectAdjustedPriceString(state1)).toBe('10');
+    expect(selectAdjustedPriceString(state2)).toBe('20');
+  });
+});