Implemented part of the suggested fixes

1- Updated debug logging env var name to DEBUG_MODE
2- Updated sigil check for selecting the correct settings to be lower cased check
3- Used enum for the keys that handle the state in logging utils

Author: banginji

README.md

@@ -40,7 +40,7 @@ _Coming soon_
 
 ### Debug logging
 
-Set the `DIFF_POETRY_LOCK_DEBUG` environment variable to `true` (or `1`, `yes`, `on`) to enable verbose debug logging.
+Set the `DEBUG_MODE` environment variable to `true` (or `1`, `yes`, `on`) to enable verbose debug logging.
 When unset, only informational and higher level log messages are emitted, reducing noise in CI logs.
 
 ## History

diff_poetry_lock/github.py

@@ -30,6 +30,7 @@ def __init__(self, repo: str, branch: str) -> None:
         super().__init__(msg)
 
 
+
 class GithubApi:
     def __init__(self, settings: Settings) -> None:
         self.s = settings

diff_poetry_lock/logging_utils.py

@@ -1,20 +1,27 @@
 import logging
 import os
+from enum import Enum
 from typing import Final
 
-DEBUG_ENV_VAR = "DIFF_POETRY_LOCK_DEBUG"
+DEBUG_ENV_VAR = "DEBUG_MODE"
 
-_STATE: Final[dict[str, bool]] = {"configured": False}
+
+class _StateKey(Enum):
+    """Enum for state dictionary keys."""
+    CONFIGURED = "configured"
+
+
+_STATE: Final[dict[str, bool]] = {_StateKey.CONFIGURED.value: False}
 
 
 def configure_logging() -> None:
     """Configure root logging once, honoring the debug flag env var."""
-    if _STATE["configured"] or logging.getLogger().handlers:
+    if _STATE[_StateKey.CONFIGURED.value] or logging.getLogger().handlers:
         return
 
     level = logging.DEBUG if _is_debug_enabled() else logging.INFO
     logging.basicConfig(level=level, format="%(levelname)s %(name)s - %(funcName)s: %(message)s")
-    _STATE["configured"] = True
+    _STATE[_StateKey.CONFIGURED.value] = True
 
 
 def _is_debug_enabled() -> bool:

diff_poetry_lock/settings.py

@@ -37,7 +37,7 @@ class Settings(ABC):
     @classmethod
     def matches_env(cls, env: dict[str, str]) -> bool:
         """Check whether this CI's identifying env var is present."""
-        return any(key == cls.sigil_envvar for key in env)
+        return any(key.lower() == cls.sigil_envvar.lower() for key in env)
 
 
 class VelaSettings(BaseSettings, Settings):

diff_poetry_lock/test/test_poetry_diff.py

@@ -46,7 +46,7 @@ def test_settings(monkeypatch: MonkeyPatch) -> None:
 def test_settings_not_pr(monkeypatch: MonkeyPatch) -> None:
     monkeypatch.setenv("GITHUB_EVENT_NAME", "push")
     monkeypatch.setenv("GITHUB_REF", "refs/pull/1/merge")
-    monkeypatch.setenv("github_repository", "account/repo")
+    monkeypatch.setenv("GITHUB_REPOSITORY", "account/repo")
     monkeypatch.setenv("INPUT_GITHUB_TOKEN", "foobar")
     monkeypatch.setenv("GITHUB_BASE_REF", "main")