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

fix: return empty array if to-many relation is empty instead of removing the property altogether

.changeset/thick-spiders-admire.md

@@ -0,0 +1,5 @@
+---
+"@supabase-cache-helpers/postgrest-core": patch
+---
+
+fix: return empty array if to-many relation is empty instead of removing the property altogether

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

@@ -45,4 +45,33 @@ describe('denormalize', () => {
       assignee: null,
     });
   });
+
+  it('should set empty array if relation is empty array', () => {
+    expect(
+      denormalize(
+        [
+          {
+            declaration: 'tags:tag.id',
+            alias: 'tags.id',
+            path: 'tag.id',
+          },
+          {
+            declaration: 'tags:tag.name',
+            alias: 'tags.name',
+            path: 'tag.name',
+          },
+          {
+            declaration: 'tags:tag.color',
+            alias: 'tags.color',
+            path: 'tag.color',
+          },
+        ],
+        {
+          tag: [],
+        }
+      )
+    ).toEqual({
+      tags: [],
+    });
+  });
 });

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

@@ -41,11 +41,11 @@ 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) {
+    // if value is null or an empty array, the relation is not set and we can return the "empty" value
+    if (value === null || (Array.isArray(value) && value.length === 0)) {
       return {
         ...prev,
-        [curr.alias || curr.path]: null,
+        [curr.alias || curr.path]: value,
       };
     }