Operation name graphql

[jwallet]

This PR contains:

• A NEW FEATURE
• breaking the helper function

Describe the problem you have without this PR

GraphQL operationName is required with some tools to valid the gQL request

We used patch-package to change this in rxdb but we will like to be able to upgrade without it.

[pubkey]

Hi @jwallet
Do you think this is a breaking change?

[jwallet]

I can remove the changes made to the helper functions to make it non breaking, but it looks like these functions does not follow the required graphql recommandations found in apollo or hotchocolate documentation OperationName can stay an optional param

…

On Fri., Dec. 22, 2023, 8:28 a.m. Daniel Meyer, ***@***.***> wrote: Hi @jwallet <https://github.com/jwallet> Do you think this is a breaking change? — Reply to this email directly, view it on GitHub <#5418 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFQEZMICJC6TZYC43P2NI4DYKWDGXAVCNFSM6AAAAABA7CULCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRXGY4DSMRZGY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[jwallet]

I changed the helper functions queryBuilderFromRxSchema to prevent a breaking change with the pullStream.

[jwallet]

Also, if someone uses a query like the one below, he can just ignore the "operationName" param, but the operationName must match the name after query/mutation keyword, and if after the merge no one provide an operationName, it will just stay like before the merge since the param is optional.

query($checkpoint: CheckpointInput, $limit: Int!)
                    {
                        collectionFeedForRxDBReplication(checkpoint: $checkpoint, limit: $limit) {
                            collection {
                                documents {
                                    id
                                    name
                                    age
                                    updatedAt
                                    deleted
                                }
                                checkpoint {
                                    id
                                    updatedAt
                                }
                            }
                        }
                    }

[pubkey]

Thank you, merged.