python-openevse-http 1.0.0__tar.gz → 1.0.2__tar.gz

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/.github/workflows/links.yml

@@ -19,7 +19,7 @@ jobs:
         with:
           egress-policy: audit
 
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10  # v6
 
       - name: Link Checker
         uses: lycheeverse/lychee-action@8646ba30535128ac92d33dfc9133794bfdd9b411  # v2

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/.github/workflows/publish-to-pypi.yml

@@ -25,13 +25,13 @@ jobs:
         with:
           egress-policy: audit
 
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10  # v6
         with:
           ref: ${{ inputs.tag || github.ref }}
           fetch-depth: 0
 
       - name: Install uv
-        uses: astral-sh/setup-uv@08807647e7069bb48b6ef5acd8ec9567f424441b  # v8.1.0
+        uses: astral-sh/setup-uv@fac544c07dec837d0ccb6301d7b5580bf5edae39  # v8.2.0
         with:
           enable-cache: true
           version: "0.10.9"

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/.github/workflows/test.yml

@@ -22,7 +22,7 @@ jobs:
         with:
           egress-policy: audit
 
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10  # v6
       - name: Set up Python
         uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405  # v6
         with:
@@ -45,7 +45,7 @@ jobs:
         with:
           egress-policy: audit
 
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10  # v6
         with:
           fetch-depth: 2
       - name: Set up Python ${{ matrix.python-version }}
@@ -74,7 +74,7 @@ jobs:
           egress-policy: audit
 
       - name: Check out the repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6
+        uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10  # v6
         with:
           fetch-depth: 2
       - name: Download coverage data
@@ -82,6 +82,6 @@ jobs:
         with:
           name: coverage-data
       - name: Upload coverage report
-        uses: codecov/codecov-action@e79a6962e0d4c0c17b229090214935d2e33f8354  # v6
+        uses: codecov/codecov-action@fb8b3582c8e4def4969c97caa2f19720cb33a72f  # v6
         with:
           token: ${{ secrets.CODECOV_TOKEN }}

{python_openevse_http-1.0.0/python_openevse_http.egg-info → python_openevse_http-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python_openevse_http
-Version: 1.0.0
+Version: 1.0.2
 Summary: Python wrapper for OpenEVSE HTTP API
 Home-page: https://github.com/firstof9/python-openevse-http
 Download-URL: https://github.com/firstof9/python-openevse-http

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/openevsehttp/client.py

@@ -50,13 +50,13 @@ class OpenEVSE(CommandsMixin, ManagersMixin, SensorsMixin, PropertiesMixin):
     def __init__(
         self,
         host: str,
-        user: str = "",
-        pwd: str = "",
+        user: str | None = None,
+        pwd: str | None = None,
         session: aiohttp.ClientSession | None = None,
     ) -> None:
         """Connect to an OpenEVSE charger equipped with wifi or ethernet."""
-        self._user = user
-        self._pwd = pwd
+        self._user = user or ""
+        self._pwd = pwd or ""
         self.url = f"http://{host}/"
         self._status: dict[str, Any] = {}
         self._config: dict[str, Any] = {}

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/openevsehttp/properties.py

@@ -84,9 +84,12 @@ class PropertiesMixin:
         return bool(self._config.get("relayt", False))
 
     @property
-    def service_level(self) -> int | str | None:
-        """Return the service level (1, 2, or 'A')."""
-        return self._config.get("service")
+    def service_level(self) -> str | None:
+        """Return the service level ('1', '2', or 'A')."""
+        value = self._config.get("service")
+        if value is None:
+            return value
+        return str(value)
 
     @property
     def openevse_firmware(self) -> str | None:

python_openevse_http-1.0.2/openevsehttp/utils.py

@@ -0,0 +1,55 @@
+"""Utility functions for python-openevse-http."""
+
+import logging
+import re
+
+from awesomeversion import AwesomeVersion
+
+_LOGGER = logging.getLogger(__name__)
+
+
+def normalize_version(version: str) -> str:
+    """Normalize the version string to strip 'dev' tag."""
+    if "dev" in version:
+        _LOGGER.debug("Stripping 'dev' from version.")
+        value = version.split(".")
+        return ".".join(value[0:3])
+    return version
+
+
+def get_awesome_version(version: str) -> AwesomeVersion:
+    """Parse and normalize the version string, returning an AwesomeVersion.
+
+    Development and custom firmware versions (e.g. branch builds ending in a commit hash,
+    or standalone 'main'/'master' builds) are normalized to 'dev' to bypass feature flags
+    and version-checks.
+
+    Non-dev version strings are normalized by extracting the first stable 'X.Y.Z' match,
+    meaning pre-release, build, or other trailing tags (such as '_rc1' or '_alpha') are
+    removed and not preserved in the returned version.
+    """
+    # Match non-numeric git branch names (e.g. 'main', 'master', 'develop', or 'feature-x_abc123456')
+    # We use custom word boundary checks to avoid false positives like 'domain' matching 'main'
+    # or 'webmaster' matching 'master'.
+    is_dev = False
+    if re.search(
+        r"(?:^|[^a-zA-Z0-9])(master|main)(?:[^a-zA-Z0-9]|$)", version, re.IGNORECASE
+    ):
+        is_dev = True
+    elif re.search(r"_[a-fA-F0-9]{7,40}$", version):
+        is_dev = True
+    elif re.search(
+        r"(?:^|[^a-zA-Z0-9])(dev|feature|fix|main|master)(?:[^a-zA-Z0-9].*?)?_[a-fA-F0-9]{6}$",
+        version,
+        re.IGNORECASE,
+    ):
+        is_dev = True
+
+    if is_dev:
+        version = "dev"
+    value = normalize_version(version)
+    if "dev" not in version:
+        firmware_search = re.search(r"\d+\.\d+\.\d+", value)
+        if firmware_search:
+            value = firmware_search.group(0)
+    return AwesomeVersion(value)

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2/python_openevse_http.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python_openevse_http
-Version: 1.0.0
+Version: 1.0.2
 Summary: Python wrapper for OpenEVSE HTTP API
 Home-page: https://github.com/firstof9/python-openevse-http
 Download-URL: https://github.com/firstof9/python-openevse-http

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/requirements_lint.txt

@@ -1,4 +1,4 @@
 -r requirements.txt
-prek==0.4.3
-ruff==0.15.15
+prek==0.4.4
+ruff==0.15.16
 mypy==2.1.0

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/requirements_test.txt

@@ -1,10 +1,10 @@
 -r requirements.txt
-prek==0.4.3
+prek==0.4.4
 pytest==9.0.3
 pytest-cov==7.1.0
 pytest-timeout==2.4.0
 pytest-asyncio
 requests_mock
 aiohttp
-tox==4.55.0
+tox==4.55.1
 freezegun

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/tests/fixtures/v4_json/config-unknown-semver.json

@@ -3,7 +3,7 @@
     "protocol": "-",
     "espflash": 4194304,
     "wifi_serial": "1234567890AB",
-    "version": "random_a4f11e",
+    "version": "random_g4f11e",
     "diodet": 0,
     "gfcit": 0,
     "groundt": 0,

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/tests/test_client.py

@@ -424,7 +424,8 @@ async def test_firmware_check(
     assert firmware["latest_version"] == "4.1.4"
 
     await test_charger_unknown_semver.update()
-    assert test_charger_unknown_semver.wifi_firmware == "random_a4f11e"
+    # Changed from 'random_a4f11e' to 'random_g4f11e' to avoid false positive matches on the 6-character hex hash regex
+    assert test_charger_unknown_semver.wifi_firmware == "random_g4f11e"
     mock_aioclient.get(
         TEST_URL_GITHUB_v4,
         status=200,
@@ -432,7 +433,7 @@ async def test_firmware_check(
     )
     with caplog.at_level(logging.DEBUG):
         firmware = await test_charger_unknown_semver.firmware_check()
-        assert "Using version: random_a4f11e" in caplog.text
+        assert "Using version: random_g4f11e" in caplog.text
         assert "Non-semver firmware version detected" in caplog.text
         assert firmware is None
 
@@ -634,6 +635,59 @@ async def test_version_check_limit():
     assert charger.version_check("2.0.0") is True
 
 
+async def test_version_check_dev_branches():
+    """Test _version_check with dev branches.
+
+    Test that dev branch version strings bypass version checks and are correctly
+    treated as development builds. Also tests false-positive avoidance and
+    pre-release handling.
+    """
+    charger = OpenEVSE(SERVER_URL, session=MagicMock())
+
+    # Standalone dev branches - treated as dev, returns True
+    charger._config = {"version": "main"}
+    assert charger._version_check("2.0.0") is True
+
+    charger._config = {"version": "master"}
+    assert charger._version_check("2.0.0") is True
+
+    # 'main' branch with hash - treated as dev, returns True
+    charger._config = {"version": "main_abc1234"}
+    assert charger._version_check("2.0.0") is True
+
+    # 'master' branch with hash - treated as dev, returns True
+    charger._config = {"version": "master_abc1234"}
+    assert charger._version_check("2.0.0") is True
+
+    # Custom branch with 7-char hash - treated as dev, returns True
+    charger._config = {"version": "feature-ui_2b4ad2c"}
+    assert charger._version_check("2.0.0") is True
+
+    # Custom branch with 6-char hash - treated as dev, returns True
+    charger._config = {"version": "feature-ui_2b4ad2"}
+    assert charger._version_check("2.0.0") is True
+
+    # Case-insensitive hex hash matching (uppercase) - treated as dev, returns True
+    charger._config = {"version": "feature_1A2B3C"}
+    assert charger._version_check("2.0.0") is True
+
+    # False positives containing 'main' or 'master' but not at word boundaries,
+    # or ending in 6-char hashes without dev keywords — should fail version check
+    charger._config = {"version": "domain_abc123"}
+    assert charger._version_check("2.0.0") is False
+
+    charger._config = {"version": "webmaster_def456"}
+    assert charger._version_check("2.0.0") is False
+
+    # Pre-release (rc) version — should fail when checking against a newer target
+    charger._config = {"version": "4.1.2_rc1"}
+    assert charger._version_check("5.0.0") is False
+
+    # Pre-release (alpha) version — should also fail when checking against a newer target
+    charger._config = {"version": "4.1.0_alpha"}
+    assert charger._version_check("5.0.0") is False
+
+
 # ── websocket lifecycle ──────────────────────────────────────────────
 
 

{python_openevse_http-1.0.0 → python_openevse_http-1.0.2}/tests/test_properties.py

@@ -56,8 +56,8 @@ SERVER_URL = "openevse.test.tld"
         ("test_charger_v2", "ammeter_offset", 0),
         ("test_charger", "ammeter_scale_factor", 220),
         ("test_charger_v2", "ammeter_scale_factor", 220),
-        ("test_charger", "service_level", 2),
-        ("test_charger_v2", "service_level", 2),
+        ("test_charger", "service_level", "2"),
+        ("test_charger_v2", "service_level", "2"),
         # safety checks
         ("test_charger", "temp_check_enabled", False),
         ("test_charger_v2", "temp_check_enabled", False),

python_openevse_http-1.0.0/openevsehttp/utils.py

@@ -1,29 +0,0 @@
-"""Utility functions for python-openevse-http."""
-
-import logging
-import re
-
-from awesomeversion import AwesomeVersion
-
-_LOGGER = logging.getLogger(__name__)
-
-
-def normalize_version(version: str) -> str:
-    """Normalize the version string to strip 'dev' tag."""
-    if "dev" in version:
-        _LOGGER.debug("Stripping 'dev' from version.")
-        value = version.split(".")
-        return ".".join(value[0:3])
-    return version
-
-
-def get_awesome_version(version: str) -> AwesomeVersion:
-    """Parse and normalize the version string, returning an AwesomeVersion."""
-    if "master" in version:
-        version = "dev"
-    value = normalize_version(version)
-    if "dev" not in version:
-        firmware_search = re.search(r"\d+\.\d+\.\d+", value)
-        if firmware_search:
-            value = firmware_search.group(0)
-    return AwesomeVersion(value)