Merge pull request #300 from psteinroe/fix/null-relation

feat: fix: set relation to null if it is null instead of removing the property altogether when denormalizing

.changeset/witty-teachers-wave.md

@@ -0,0 +1,5 @@
+---
+"@supabase-cache-helpers/postgrest-core": patch
+---
+
+fix: set relation to null if it is null instead of removing the property altogether when denormalizing

packages/postgrest-core/__tests__/filter/denormalize.spec.ts

@@ -26,4 +26,23 @@ describe('denormalize', () => {
       },
     });
   });
+
+  it('should set null if relation is null', () => {
+    expect(
+      denormalize(
+        [
+          {
+            declaration: 'assignee:assignee_id.id',
+            alias: 'assignee.id',
+            path: 'assignee_id.id',
+          },
+        ],
+        {
+          assignee_id: null,
+        }
+      )
+    ).toEqual({
+      assignee: null,
+    });
+  });
 });

packages/postgrest-core/src/filter/denormalize.ts

@@ -41,6 +41,14 @@ export const denormalize = <R extends Record<string, unknown>>(
       };
     }
 
+    // if value is null, the relation is not set and we can return null
+    if (value === null) {
+      return {
+        ...prev,
+        [curr.alias || curr.path]: null,
+      };
+    }
+
     let isArray = false;
     const flatNestedObjectOrArray = Object.entries(obj).reduce<
       Record<string, Record<string, unknown>> | Record<string, unknown>
@@ -66,6 +74,7 @@ export const denormalize = <R extends Record<string, unknown>>(
         [flatKey]: v,
       };
     }, {});
+
     if (Object.keys(flatNestedObjectOrArray).length === 0) return prev;
     if (isArray && isFlatNestedArray(flatNestedObjectOrArray)) {
       return {