Type error in SSELink Apollo example

[extremegf]

Screenshot
I use code from the example here

Expected Behaviour
The example should work without type errors.

Actual Behaviour
Instead I get the error as shown above.

Further Information
This started showing up when i updated all patch versions of my packages. Likely some typescript instrumentation just caught it, because downgrading back the apollo and graphql packages did not help.

[extremegf]

Out of curiosity, I've checked the dependency update one by one. This is caused by updating @apollo/client 3.10.4 -> 3.11.8

[WKampel]

Any solutions here?

[extremegf]

I just downgraded.

[enisdenjo]

You can always cast to any.

[extremegf]

I assume it will be fixed eventually, if not I'll probably do that.

[RMHonor]

Instead of importing FetchResult from @apollo/client, import ExecutionResult from graphql.

import { ApolloLink, Operation, Observable } from "@apollo/client/core";
import { ExecutionResult, print } from "graphql";
import { createClient, ClientOptions, Client } from "graphql-sse";

export class SSELink extends ApolloLink {
  private client: Client;

  constructor(options: ClientOptions) {
    super();
    this.client = createClient(options);
  }

  public request(operation: Operation): Observable<ExecutionResult> {
    return new Observable((sink) => {
      return this.client.subscribe<ExecutionResult>(
        { ...operation, query: print(operation.query) },
        {
          next: sink.next.bind(sink),
          complete: sink.complete.bind(sink),
          error: sink.error.bind(sink),
        },
      );
    });
  }
}

[extremegf]

I confirm, that @RMHonor suggestion sovles the problem.
🎉

[enisdenjo]

Even though #108 (comment) is a fix, I consider this a bug because there's no reason the FetchResult should not work. I think the issue is caused by the lack/improper usage of the patch message in graphql-sse.

[mbiggs-gresham]

I've just been looking at switching from graphql-ws to graphql-sse and came across this line in the GraphQLWsLink code which also works for this too.

https://github.com/apollographql/apollo-client/blob/77b89318040748b3ffad97938eccacd8b1fbb4bd/src/link/subscriptions/index.ts#L85

@enisdenjo Thanks for the library, are there any plans to add a link based on your recipe to the apollo codebase? I don't know if that was yourself or the apollo guys who originally contributed it.

[enisdenjo]

@mbiggs-gresham a new major release is cooking, and fixing this issue is certainly on the list (along many other important fixes)! Please bear with me, I am quite busy at the moment but it shouldn't be too long.