pr review feedback

Author: srest2021

src/sentry/seer/code_review/contributor_seats.py

@@ -90,6 +90,14 @@ def seed_contributor(
     user_id: str | int,
     user_username: str,
 ) -> OrganizationContributors:
+    """
+    Create the OrganizationContributors row without incrementing the number
+    of actions.
+
+    Seeding happens upstream when the PR is created; the action is incremented
+    separately only after the PR clears the code-review preflight check and
+    other checks on the webhook's properties.
+    """
     contributor, _ = OrganizationContributors.objects.get_or_create(
         organization_id=organization.id,
         integration_id=integration_id,

src/sentry/seer/code_review/webhooks/pull_request.py

@@ -39,6 +39,7 @@ class Log(enum.StrEnum):
     MISSING_PULL_REQUEST = "github.webhook.pull_request.missing-pull-request"
     MISSING_ACTION = "github.webhook.pull_request.missing-action"
     UNSUPPORTED_ACTION = "github.webhook.pull_request.unsupported-action"
+    MISSING_AUTHOR_ID = "github.webhook.pull_request.missing-author-id"
 
 
 class PullRequestAction(enum.StrEnum):
@@ -174,6 +175,17 @@ def handle_pull_request_event(
                 user_id=author_id,
                 provider="github",
             )
+        else:
+            # A missing user id means we can't identify which contributor's actions to increment.
+            # This should never happen.
+            logger.warning(
+                Log.MISSING_AUTHOR_ID.value,
+                extra={
+                    "organization_id": organization.id,
+                    "repo_id": repo.id,
+                    "integration_id": integration.id,
+                },
+            )
 
     pr_number = pull_request.get("number")
     if pr_number and action in ACTIONS_ELIGIBLE_FOR_EYES_REACTION: