fix: Improved graphql query for get a last commit date

Author: GermanMT

app/apis/github_service.py

@@ -1,33 +1,24 @@
-from asyncio import sleep
+from asyncio import TimeoutError, sleep
 from datetime import datetime
 
 from aiohttp import ClientConnectorError, ClientSession
 
 from app.config import settings
-from app.exceptions import InvalidRepositoryException
+from app.exceptions.exceptions import InvalidRepositoryException
 
 headers_github = {
     "Accept": "application/vnd.github.hawkgirl-preview+json",
     "Authorization": f"Bearer {settings.GITHUB_GRAPHQL_API_KEY}",
 }
 
-
-async def get_last_commit_date_github(owner: str, name: str) -> datetime:
+async def get_last_commit_date_github(owner: str, name: str) -> datetime | bool:
     query = f"""
     {{
         repository(owner: "{owner}", name: "{name}") {{
             defaultBranchRef {{
                 target {{
                     ... on Commit {{
-                        history(first: 1) {{
-                            edges {{
-                                node {{
-                                    author {{
-                                        date
-                                    }}
-                                }}
-                            }}
-                        }}
+                        committedDate
                     }}
                 }}
             }}
@@ -46,18 +37,14 @@ async def get_last_commit_date_github(owner: str, name: str) -> datetime:
                     break
             except (ClientConnectorError, TimeoutError):
                 await sleep(5)
-    date = (
-        response.get("data", {})
-            .get("repository", {})
-            .get("defaultBranchRef", {})
-            .get("target", {})
-            .get("history", {})
-            .get("edges", [{}])[0]
-            .get("node", {})
-            .get("author", {})
-            .get("date")
+    repo = response.get("data", {}).get("repository")
+    if not repo or not repo.get("defaultBranchRef"):
+        raise InvalidRepositoryException()
+    date_str = (
+        repo["defaultBranchRef"]
+        .get("target", {})
+        .get("committedDate")
     )
-    if date:
-        return datetime.fromisoformat(date)
-    else:
+    if not date_str:
         raise InvalidRepositoryException()
+    return datetime.fromisoformat(date_str)