Actual fix of compatibility with older QueryVariables API (#140)

Author: mattjohnsonpint

CHANGELOG.md

@@ -1,12 +1,16 @@
 # Change Log
 
+## UNRELEASED
+
+- Actual fix of compatibility with older QueryVariables API [#140](https://github.com/hypermodeAI/functions-as/pull/140)
+
 ## 2024-07-15 - Version 0.10.2
 
 - No changes (re-publish of 0.10.1 to fix npm packaging issue)
 
 ## 2024-07-15 - Version 0.10.1
 
-- Fix compatibility with older QueryVariables API [#137](https://github.com/hypermodeAI/functions-as/pull/137)
+- Attempted fix compatibility with older QueryVariables API [#137](https://github.com/hypermodeAI/functions-as/pull/137)
 
 ## 2024-07-15 - Version 0.10.0
 

src/assembly/compat.ts

@@ -9,26 +9,26 @@ export abstract class connection {
   /**
    * @deprecated Use `graphql.execute` instead.
    */
-  static invokeGraphqlApi<TData>(
+  static invokeGraphqlApi<T>(
     hostName: string,
     statement: string,
-    variables = new graphql.Variables(),
-  ): graphql.Response<TData> {
-    return graphql.execute<TData>(hostName, statement, variables);
+    variables: QueryVariables = new QueryVariables(),
+  ): GQLResponse<T> {
+    const r = graphql.execute<T>(hostName, statement, variables);
+    return <GQLResponse<T>>{
+      errors: r.errors,
+      data: r.data,
+    };
   }
 }
 
 /**
  * @deprecated Import `graphql`, and use `graphql.Variables` instead.
  */
-export class QueryVariables {
-  private qv: graphql.Variables = new graphql.Variables();
+export class QueryVariables extends graphql.Variables {}
 
-  public set<T>(name: string, value: T): void {
-    this.qv.set(name, value);
-  }
-
-  public toJSON(): string {
-    return this.qv.toJSON();
-  }
-}
+/**
+ * @deprecated Import `graphql`, and use `graphql.Response` instead.
+ */
+@json
+export class GQLResponse<T> extends graphql.Response<T> {}