fix: bug on push events

Author: lhoupert

src/python_security_auditing/settings.py

@@ -4,6 +4,7 @@
 
 from typing import Literal
 
+from pydantic import field_validator
 from pydantic_settings import BaseSettings, SettingsConfigDict
 
 
@@ -36,6 +37,14 @@ class Settings(BaseSettings):
     github_run_id: str = ""
     pr_number: int | None = None
     github_event_name: str = ""
+
+    @field_validator("pr_number", mode="before")
+    @classmethod
+    def _empty_str_to_none(cls, v: object) -> object:
+        if v == "":
+            return None
+        return v
+
     github_head_ref: str = ""  # Branch name for PRs
     github_step_summary: str = ""  # Path to step summary file
 

tests/test_settings.py

@@ -68,6 +68,12 @@ def test_pr_number_from_env(monkeypatch: pytest.MonkeyPatch) -> None:
     assert s.pr_number == 42
 
 
+def test_pr_number_empty_string(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setenv("PR_NUMBER", "")
+    s = Settings()
+    assert s.pr_number is None
+
+
 def test_post_pr_comment_false(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setenv("POST_PR_COMMENT", "false")
     s = Settings()