Skip draft PRs in review scan

- Drop draft PRs in list_open_prs_by_authors before building OpenPR
- Add draft property to the _PullRequest protocol
- Add draft field to _FakePull and cover the skip with a new test

Draft PRs are still in progress and not ready for the reviewer
workflow, so the trusted-author scan filters them out alongside its
existing author check, avoiding dispatch and fingerprinting on PRs the
author hasn't marked ready for review.

Signed-off-by: Jean Schmidt <contato@jschmidt.me>

Author: jeanschmidt

greenlight/src/greenlight/github_client.py

@@ -44,6 +44,8 @@ def head(self) -> _PRBase: ...
         def updated_at(self) -> datetime | None: ...
         @property
         def labels(self) -> Iterable[_Label]: ...
+        @property
+        def draft(self) -> bool: ...
 
     class _Repo(Protocol):
         def get_pulls(self, state: str) -> Iterable[_PullRequest]: ...
@@ -172,6 +174,8 @@ def list_open_prs_by_authors(client: _RepoClient, repo: str, authors: Iterable[s
             continue
         login = user.login
         if login and login.lower() in trusted:
+            if pr.draft:
+                continue
             updated_at = pr.updated_at
             prs.append(
                 OpenPR(

greenlight/tests/test_github_client.py

@@ -30,6 +30,7 @@ def __init__(
         head_sha: str = "head-sha",
         updated_at: datetime | None = None,
         labels: list[str] | None = None,
+        draft: bool = False,
     ) -> None:
         self.number = number
         self.user = _FakeUser(login) if login is not None else None
@@ -38,6 +39,7 @@ def __init__(
         self.head = _FakeBase(head_sha)
         self.updated_at = updated_at
         self.labels = [_FakeLabel(name) for name in (labels or [])]
+        self.draft = draft
 
 
 class _FakeRepo:
@@ -263,6 +265,20 @@ def test_list_open_prs_by_authors_skips_pulls_with_no_user():
     assert [pr.author for pr in prs] == ["alice"]
 
 
+def test_list_open_prs_by_authors_skips_draft_pulls():
+    client = _client_with_pulls(
+        [
+            _FakePull(1, "alice", "draft in progress", "https://example.test/1", draft=True),
+            _FakePull(2, "alice", "ready for review", "https://example.test/2", draft=False),
+        ]
+    )
+
+    prs = github_client.list_open_prs_by_authors(client, "pytorch/pytorch", ["alice"])
+
+    assert [pr.number for pr in prs] == [2]
+    assert [pr.author for pr in prs] == ["alice"]
+
+
 def test_list_open_prs_by_authors_sorts_output_by_pr_number():
     client = _client_with_pulls(
         [