chore(deps): update gql to v4 (#598)

* chore(deps): update gql to v4.0.0

* fix: use non-deprecated gql calls

It's worth mentioning that the documentation suggests[^1] setting `variable_values`
directly on the (in our case, global) query object, and doesn't really talk about race conditions,
but I would assume they can happen, especially in async code.
It might be fine if you call `gql.gql(<str>)` every time, which might be what the documentation suggests,
but that seems like a performance nightmare.
Therefore, we just construct a new `GraphQLRequest` from the existing globals,
which automatically copies over the parsed `Document` node, so it should have little to no overhead.

[^1]: https://gql.readthedocs.io/en/latest/usage/variables.html

* feat: improve gql logging level overrides

transports now log with `DEBUG` instead of `INFO`,
so we can go back up to `INFO` without getting too spammy.

* fix: remove now unnecessary ssl param for gql transport

Author: shiftinv

monty/exts/info/github_info.py

@@ -221,9 +221,7 @@ class GithubInfo(
     def __init__(self, bot: Monty) -> None:
         self.bot = bot
 
-        transport = AIOHTTPTransport(
-            url="https://api.github.com/graphql", timeout=20, headers=GITHUB_REQUEST_HEADERS, ssl=True
-        )
+        transport = AIOHTTPTransport(url="https://api.github.com/graphql", timeout=20, headers=GITHUB_REQUEST_HEADERS)
 
         self.gql = gql.Client(transport=transport, fetch_schema_from_transport=True)
 
@@ -575,12 +573,14 @@ async def fetch_issues(
 
             try:
                 json_data = await self.gql.execute_async(
-                    DISCUSSION_GRAPHQL_QUERY,
-                    variable_values={
-                        "user": user,
-                        "repository": repository,
-                        "number": number,
-                    },
+                    gql.GraphQLRequest(
+                        DISCUSSION_GRAPHQL_QUERY,
+                        variable_values={
+                            "user": user,
+                            "repository": repository,
+                            "number": number,
+                        },
+                    )
                 )
             except (TransportError, TransportQueryError):
                 return FetchError(-1, "Issue not found.")
@@ -1010,10 +1010,12 @@ async def handle_issue_comment(
 
                 try:
                     json_data = await self.gql.execute_async(
-                        DISCUSSION_COMMENT_GRAPHQL_QUERY,
-                        variable_values={
-                            "id": global_id,
-                        },
+                        gql.GraphQLRequest(
+                            DISCUSSION_COMMENT_GRAPHQL_QUERY,
+                            variable_values={
+                                "id": global_id,
+                            },
+                        )
                     )
                 except (TransportError, TransportQueryError) as e:
                     log.warning("encountered error fetching discussion comment: %s", e)

monty/log.py

@@ -102,7 +102,8 @@ def setup() -> None:
     logging.getLogger("cachingutils").setLevel(logging.INFO)
     logging.getLogger("sqlalchemy.engine").setLevel(logging.DEBUG)
     logging.getLogger("sqlalchemy.pool").setLevel(logging.INFO)
-    logging.getLogger("gql.transport.aiohttp").setLevel(logging.WARNING)
+    logging.getLogger("gql.dsl").setLevel(logging.INFO)
+    logging.getLogger("gql.transport.aiohttp").setLevel(logging.INFO)
     _set_trace_loggers()
 
     root_logger.info("Logging initialization complete")

pyproject.toml

@@ -22,7 +22,7 @@ dependencies = [
     "colorama~=0.4.5; sys_platform == \"win32\"",
     "disnake @ git+https://github.com/DisnakeDev/disnake.git@master",
     "orjson<4.0.0,>=3.10.18",
-    "gql<4.0.0,>=3.5.2",
+    "gql~=4.0",
     "lxml<6.0.0,>=5.4.0",
     "markdownify==0.11.6",
     "mistune<3.0.0,>=2.0.4",