ref(github): Address review nits on paginated repos

Document cache consistency caveat in get_repositories_paginated
docstring: cache expiry between pages can cause duplicates/skips,
acceptable for infinite-scroll.

Make _parse_cursor a staticmethod. Add comment clarifying per_page
and offset are guaranteed set when paginated is not None.

Refs VDY-46

Author: jaydgoss

src/sentry/integrations/api/endpoints/organization_integration_repos.py

@@ -129,6 +129,7 @@ def get(
                 {"repos": serialized_repositories, "searchable": install.repo_search}
             )
 
+            # per_page and offset are guaranteed set when paginated is not None
             if paginated is not None and (has_next or offset > 0):
                 # CursorResult only used for Link header generation
                 cursor_result: CursorResult[IntegrationRepository] = CursorResult(
@@ -142,7 +143,8 @@ def get(
 
         return self.respond({"detail": "Repositories not supported"}, status=400)
 
-    def _parse_cursor(self, request: Request) -> Cursor:
+    @staticmethod
+    def _parse_cursor(request: Request) -> Cursor:
         cursor_param = request.GET.get("cursor", "")
         if not cursor_param:
             return Cursor(0, 0, False)

src/sentry/integrations/github/integration.py

@@ -379,7 +379,10 @@ def get_repositories_paginated(
         """Paginate over the cached accessible-repos list.
 
         Always serves from ``get_accessible_repos_cached()`` so every
-        page is fast after the initial cache warm-up.
+        page is fast after the initial cache warm-up. If the cache
+        expires between page fetches the underlying list can change,
+        which may cause duplicates or skipped repos across pages.
+        This is acceptable for infinite-scroll consumers.
         """
         client = self.get_client()
         all_repos = client.get_accessible_repos_cached()