Type graph resolve result as Promise<T> instead of Query

IlyaSemenov · Jun 21, 2024 · f3f5f31 · f3f5f31
1 parent e802bea
commit f3f5f31
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/.changeset/selfish-drinks-flow.md b/.changeset/selfish-drinks-flow.md
@@ -0,0 +1,6 @@
+---
+"objection-graphql-resolver": minor
+"graphql-orm": minor
+---
+
+Type graph resolve result as `Promise<T>` instead of `Query`.
diff --git a/packages/graphql-orm/src/resolvers/graph.ts b/packages/graphql-orm/src/resolvers/graph.ts
@@ -33,10 +33,10 @@ export class GraphResolver<Orm extends OrmAdapter, Context> {
 		readonly options: GraphResolverOptions<Orm, Context> = {},
 	) {}
 
-	resolve(
+	resolve<T>(
 		query: Orm["Query"],
 		{ info, ...context }: GraphResolveOptions<Context>,
-	) {
+	): Promise<T> {
 		const tree = this._get_resolve_tree(info)
 		return this._resolve_type(query, { ...context, tree })
 	}

diff --git a/packages/objection-graphql-resolver/src/resolvers/graph.ts b/packages/objection-graphql-resolver/src/resolvers/graph.ts
@@ -30,8 +30,8 @@ class ObjectionGraphResolver<Context> extends GraphResolver<
 	resolve<Query extends AnyQueryBuilder>(
 		query: Query,
 		options: GraphResolveOptions<Context>,
-	): Query {
-		return super.resolve(query, options) as Query
+	): Promise<Query["ResultType"]> {
+		return super.resolve(query, options)
 	}
 
 	resolvePage<M extends Model, Query extends QueryBuilder<M, M[]>>(