Should remove @client and __typename-only fragment and fragment spread if fragment appears before query

[alessbell]

Issue Description

For the following document node:

const query = gql`
  fragment toBeRemoved on Author {
    __typename
    isLoggedIn @client
  }

  query {
    author {
      name
      ...toBeRemoved
    }
  }
`

The expected document returned via removeClientSetsFromDocument(query) is

query {
  author {
    name
  }
}

instead of the current behavior:

query {
  author {
    name
    ...toBeRemoved
  }
}

which keeps a dangling reference to the removed fragment and will cause a server error.

Link to Reproduction

See code example

Reproduction Steps

No response

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
For general questions, we recommend using StackOverflow or our discord server.