fix check run billing gate

Author: suejungshin

fixtures/github.py

@@ -3560,7 +3560,7 @@
         "external_id": "4663713",
         "html_url": "https://github.com/test/repo/runs/4"
     },
-    "user": {
+    "sender": {
         "id": 12345678,
         "login": "test-user"
     }
@@ -3574,7 +3574,7 @@
         "full_name": "getsentry/sentry",
         "html_url": "https://github.com/getsentry/sentry"
     },
-    "user": {
+    "sender": {
         "id": 12345678,
         "login": "test-user"
     }

src/sentry/seer/code_review/preflight.py

@@ -9,8 +9,6 @@
 from sentry.models.repository import Repository
 from sentry.models.repositorysettings import CodeReviewSettings, RepositorySettings
 
-from .billing import passes_code_review_billing_check
-
 DenialReason = str | None
 
 
@@ -91,6 +89,10 @@ def _check_repo_feature_enablement(self) -> DenialReason:
             return "repo_code_review_disabled"
 
     def _check_billing(self) -> DenialReason:
+        return None
+
+        # TODO: Once we're ready to actually gate billing (when it's time for GA), uncomment this
+        """
         if self.integration_id is None or self.pr_author_external_id is None:
             return "billing_missing_contributor_info"
 
@@ -103,6 +105,7 @@ def _check_billing(self) -> DenialReason:
             return "billing_quota_exceeded"
 
         return None
+        """
 
     # -------------------------------------------------------------------------
     # Org type helpers

src/sentry/seer/code_review/utils.py

@@ -220,4 +220,8 @@ def get_pr_author_id(event: Mapping[str, Any]) -> str | None:
     if (user_id := event.get("user", {}).get("id")) is not None:
         return str(user_id)
 
+    # Check sender.id (for check_run events). Sender is the user who triggered the event
+    if (user_id := event.get("sender", {}).get("id")) is not None:
+        return str(user_id)
+
     return None