feat: pass all tests

Author: avallete

src/PostgrestClient.ts

@@ -54,7 +54,13 @@ export type GetRpcFunctionFilterBuilderByArgs<
           : Fn['Returns'] extends Record<string, unknown>
           ? Fn['Returns']
           : never
-        Result: Fn['Returns']
+        Result: Fn['SetofOptions'] extends GenericSetofOption
+          ? Fn['SetofOptions']['isSetofReturn'] extends true
+            ? Fn['SetofOptions']['isOneToOne'] extends true
+              ? Fn['Returns'][]
+              : Fn['Returns']
+            : Fn['Returns']
+          : Fn['Returns']
         RelationName: Fn['SetofOptions'] extends GenericSetofOption
           ? Fn['SetofOptions']['to']
           : FnName

src/PostgrestTransformBuilder.ts

@@ -35,7 +35,11 @@ export default class PostgrestTransformBuilder<
     ClientOptions,
     Schema,
     Row,
-    NewResultOne[],
+    Method extends 'RPC'
+      ? Result extends unknown[]
+        ? NewResultOne[]
+        : NewResultOne
+      : NewResultOne[],
     RelationName,
     Relationships,
     Method
@@ -60,7 +64,11 @@ export default class PostgrestTransformBuilder<
       ClientOptions,
       Schema,
       Row,
-      NewResultOne[],
+      Method extends 'RPC'
+        ? Result extends unknown[]
+          ? NewResultOne[]
+          : NewResultOne
+        : NewResultOne[],
       RelationName,
       Relationships,
       Method

src/select-query-parser/result.ts

@@ -377,6 +377,7 @@ type ProcessEmbeddedResourceResult<
       match: 'refrel' | 'col' | 'fkname' | 'func'
       isNotNullable?: boolean
       referencedRelation: string
+      isSetofReturn?: boolean
     }
     direction: string
   },
@@ -407,7 +408,14 @@ type ProcessEmbeddedResourceResult<
           : Resolved['relation']['isOneToOne'] extends true
           ? Resolved['relation']['match'] extends 'func'
             ? Resolved['relation']['isNotNullable'] extends true
-              ? ProcessedChildren
+              ? Resolved['relation']['isSetofReturn'] extends true
+                ? ProcessedChildren
+                : // TODO: This shouldn't be necessary but is due in an inconsitency in PostgREST v12/13 where if a function
+                  // is declared with RETURNS <table-name> instead of RETURNS SETOF <table-name> ROWS 1
+                  // In case where there is no object matching the relations, the object will be returned with all the properties within it
+                  // set to null, we mimic this buggy behavior for type safety an issue is opened on postgREST here:
+                  // https://github.com/PostgREST/postgrest/issues/4234
+                  { [P in keyof ProcessedChildren]: ProcessedChildren[P] | null }
               : ProcessedChildren | null
             : ProcessedChildren | null
           : ProcessedChildren[]

src/select-query-parser/utils.ts

@@ -473,7 +473,10 @@ export type ResolveForwardRelationship<
                 match: 'func'
                 isNotNullable: FoundEmbededFunctionJoinTableRelation['isNotNullable'] extends true
                   ? true
-                  : false
+                  : FoundEmbededFunctionJoinTableRelation['isSetofReturn'] extends true
+                  ? false
+                  : true
+                isSetofReturn: FoundEmbededFunctionJoinTableRelation['isSetofReturn']
               }
               direction: 'forward'
               from: CurrentTableOrView

src/types.ts

@@ -61,6 +61,7 @@ export type GenericNonUpdatableView = {
 export type GenericView = GenericUpdatableView | GenericNonUpdatableView
 
 export type GenericSetofOption = {
+  isSetofReturn?: boolean | undefined
   isOneToOne?: boolean | undefined
   isNotNullable?: boolean | undefined
   to: string