phue2 0.0.1.dev280__tar.gz → 0.0.3__tar.gz

phue2-0.0.3/.cursor/rules/docs.mdc

@@ -0,0 +1,14 @@
+---
+description: style and tone for documentation
+globs: *.mdx
+alwaysApply: false
+---
+
+# tone
+- casual, clear and concise
+- use "you" to refer to the user (e.g. "Foo is great when you want Bar")
+
+# content
+- breathable
+- makes good use of admonitions and mintlify's features like <CodeGroup> and <Tip>
+- interspersed with practical and simple examples with output in <Accordian> where apt

phue2-0.0.3/.cursor/rules/python-style.mdc

@@ -0,0 +1,36 @@
+---
+description: how to write, test and run python
+globs: *.py
+alwaysApply: false
+---
+
+# Python Style
+
+## Design
+- aggressively minimal and elegant
+- prefer functional over OOP
+- keep implementation details "private" (e.g. def _impl)
+
+## Type Hints
+- Marvin supports Python 3.10+
+- Use modern syntax and full type annotations
+- Use | instead of Union
+- Use builtins like list, dict instead of typing.List and typing.Dict
+- use | None instead of Optional
+- Always type optional keyword arguments correctly e.g. def f(x: int | None = None)
+
+## Running Commands 
+`uv` gives us superpowers to resolve deps fast and on the fly. no more pip or fussing with venvs
+
+### Install dependencies
+- install project deps: `uv sync`
+- install project deps with an extra: `uv sync --extra foo`
+
+### Running script
+- run script with existing project deps: `uv run some/script.py`
+- run script additional deps on the fly: `uv run --with pandas script.py`
+
+### Testing
+- test all: `uv run pytest`
+- test all with 3 runners: `uv run pytest -n3`
+- test certain folder, only matching cases: `uv run pytest tests/basic -k some_test_fn_subtr`

{phue2-0.0.1.dev280 → phue2-0.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: phue2
-Version: 0.0.1.dev280
+Version: 0.0.3
 Summary: A Philips Hue Python library
 Author: Nathanaël Lécaudé, Nathan Nowack
 License: MIT

{phue2-0.0.1.dev280 → phue2-0.0.3}/pyproject.toml

@@ -28,20 +28,14 @@ dev = [
 
 
 [build-system]
-requires = ["hatchling", "uv-dynamic-versioning>=0.7.0"]
+requires = ["hatchling", "hatch-vcs"]
 build-backend = "hatchling.build"
 
 [tool.hatch.build.targets.wheel]
 packages = ["src/phue2"]
 
 [tool.hatch.version]
-source = "uv-dynamic-versioning"
-
-[tool.uv-dynamic-versioning]
-vcs = "git"
-style = "pep440"
-bump = true
-metadata = false
+source = "vcs"
 
 
 [tool.pytest.ini_options]

{phue2-0.0.1.dev280 → phue2-0.0.3}/src/phue2/__main__.py

@@ -60,6 +60,12 @@ def parse_args(
     )
     parser.add_argument("--config-file-path", help="Path to the config file")
     parser.add_argument("--debug", action="store_true", help="Enable debug logging")
+    parser.add_argument(
+        "--no-save-config",
+        action="store_false",
+        dest="save_config",
+        help="Do not save connection details to the config file",
+    )
 
     # Command subparsers
     subparsers = parser.add_subparsers(dest="command", help="Command to execute")
@@ -161,7 +167,11 @@ def main(argv: list[str] | None = None) -> int:
         console.info(f"Connecting to bridge at {args.host}...")
         while True:
             try:
-                bridge = Bridge(args.host, config_file_path=args.config_file_path)
+                bridge = Bridge(
+                    args.host,
+                    config_file_path=args.config_file_path,
+                    save_config=args.save_config,
+                )
                 console.success("Successfully connected to the bridge!")
                 break
             except PhueRegistrationException:

{phue2-0.0.1.dev280 → phue2-0.0.3}/src/phue2/bridge.py

@@ -46,6 +46,7 @@ class Bridge:
         username: str | None = None,
         config_file_path: str | None = None,
         timeout: int = 10,
+        save_config: bool = True,
     ):
         """Initialization function.
 
@@ -54,6 +55,7 @@ class Bridge:
             username: Optional username for the bridge
             config_file_path: Optional path to the configuration file
             timeout: Request timeout in seconds (default: 10)
+            save_config: If False, don't save the config file (default: True)
         """
         # Determine config file path
         if config_file_path is not None:
@@ -89,6 +91,7 @@ class Bridge:
         self.ip = ip
         self._username = username
         self.timeout = timeout
+        self.save_config = save_config
         self.lights_by_id: dict[int, Light] = {}
         self.lights_by_name: dict[str, Light] = {}
         self.sensors_by_id: dict[int, Sensor] = {}
@@ -233,12 +236,13 @@ class Bridge:
         for line in response:
             for key in line:
                 if "success" in key:
-                    with open(self.config_file_path, "w") as f:
-                        logger.info(
-                            "Writing configuration file to " + self.config_file_path
-                        )
-                        f.write(json.dumps({self.ip: line["success"]}))
-                        logger.info("Reconnecting to the bridge")
+                    if self.save_config:
+                        with open(self.config_file_path, "w") as f:
+                            logger.info(
+                                "Writing configuration file to " + self.config_file_path
+                            )
+                            f.write(json.dumps({self.ip: line["success"]}))
+                            logger.info("Reconnecting to the bridge")
                     self._username = line["success"]["username"]
                     return
                 if "error" in key:

{phue2-0.0.1.dev280 → phue2-0.0.3}/tests/test_bridge.py

@@ -90,3 +90,30 @@ def test_config_file(tmp_path: Path) -> None:
         # Test loading from config
         bridge2 = phue2.Bridge(config_file_path=str(config_file))
         assert bridge2.username == "testuser"
+
+
+def test_no_save_config(tmp_path: Path) -> None:
+    """Test that the config file is not saved when save_config is False."""
+    config_file = tmp_path / ".python_hue"
+
+    with respx.mock(assert_all_called=True) as mock:
+        mock.post("http://192.168.1.100/api", name="create_user").mock(
+            return_value=httpx.Response(
+                200, json=[{"success": {"username": "nosavetest"}}]
+            )
+        )
+
+        # Create a bridge with save_config=False
+        bridge = phue2.Bridge(
+            ip="192.168.1.100",
+            config_file_path=str(config_file),
+            save_config=False,
+        )
+
+        # Verify config file was NOT created and username is set
+        assert not config_file.exists()
+        assert bridge.username == "nosavetest"
+
+        # Verify that trying to connect again without IP fails (as config wasn't saved)
+        with pytest.raises(phue2.PhueException):
+            phue2.Bridge(config_file_path=str(config_file))