fix(github): Only use cache when search query is present

Avoid caching the initial no-query load when accessibleOnly is set.
Cache is only useful for debounced keystroke searches, not the first
page load which should always return fresh data.

Author: jaydgoss

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

@@ -72,7 +72,9 @@ def get(
 
             try:
                 repositories = install.get_repositories(
-                    search, accessible_only=accessible_only, use_cache=accessible_only
+                    search,
+                    accessible_only=accessible_only,
+                    use_cache=accessible_only and bool(search),
                 )
             except (IntegrationError, IdentityNotValid) as e:
                 return self.respond({"detail": str(e)}, status=400)

tests/sentry/integrations/api/endpoints/test_organization_integration_repos.py

@@ -224,7 +224,7 @@ def test_accessible_only_without_search(self, get_repositories: MagicMock) -> No
         response = self.client.get(self.path, format="json", data={"accessibleOnly": "true"})
 
         assert response.status_code == 200, response.content
-        get_repositories.assert_called_once_with(None, accessible_only=True, use_cache=True)
+        get_repositories.assert_called_once_with(None, accessible_only=True, use_cache=False)
 
     @patch(
         "sentry.integrations.github.integration.GitHubIntegration.get_repositories", return_value=[]