Address PR review feedback

- Remove unnecessary workspace member from pyproject.toml
- Replace manual env var handling with pytest monkeypatch fixtures
- Revert unnecessary inline style change in worker.py

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: zzstoatzz

pyproject.toml

@@ -97,8 +97,3 @@ include = ["src", "tests"]
 typeCheckingMode = "strict"
 venvPath = "."
 venv = ".venv"
-
-[tool.uv.workspace]
-members = [
-    "sandbox/memory-streams",
-]

src/docket/worker.py

@@ -361,7 +361,10 @@ async def ack_message(redis: Redis, message_id: RedisMessageID) -> None:
                             if not message:  # pragma: no cover
                                 continue
 
-                            if not start_task(message_id, message, is_redelivery):
+                            task_started = start_task(
+                                message_id, message, is_redelivery
+                            )
+                            if not task_started:
                                 # Other errors - delete and ack
                                 await self._delete_known_task(redis, message)
                                 await ack_message(redis, message_id)

tests/test_fakeredis_backend.py

@@ -1,54 +1,39 @@
-import os
 from unittest.mock import patch
 
 import pytest
+from pytest import MonkeyPatch
 
 from docket import Docket
 
 
-async def test_docket_backend_fake():
+async def test_docket_backend_fake(monkeypatch: MonkeyPatch):
     """Test using fakeredis backend via DOCKET_BACKEND env var."""
-    original_value = os.environ.get("DOCKET_BACKEND")
-    try:
-        os.environ["DOCKET_BACKEND"] = "fake"
+    monkeypatch.setenv("DOCKET_BACKEND", "fake")
 
-        async with Docket(name="test-fake-docket") as docket:
+    async with Docket(name="test-fake-docket") as docket:
 
-            async def simple_task(value: str) -> str:
-                return value
+        async def simple_task(value: str) -> str:
+            return value
 
-            docket.register(simple_task)
+        docket.register(simple_task)
 
-            # Add and run a task
-            execution = await docket.add(simple_task)("test-value")
-            assert execution.key
+        # Add and run a task
+        execution = await docket.add(simple_task)("test-value")
+        assert execution.key
 
-            # Verify snapshot works
-            snapshot = await docket.snapshot()
-            assert snapshot.total_tasks > 0
-    finally:
-        if original_value is None:
-            os.environ.pop("DOCKET_BACKEND", None)
-        else:
-            os.environ["DOCKET_BACKEND"] = original_value
+        # Verify snapshot works
+        snapshot = await docket.snapshot()
+        assert snapshot.total_tasks > 0
 
 
-async def test_docket_backend_fake_missing_dependency():
+async def test_docket_backend_fake_missing_dependency(monkeypatch: MonkeyPatch):
     """Test that using fake backend without fakeredis installed raises helpful error."""
-    original_value = os.environ.get("DOCKET_BACKEND")
-
-    try:
-        os.environ["DOCKET_BACKEND"] = "fake"
-
-        # Mock the import to simulate fakeredis not being installed
-        with patch.dict("sys.modules", {"fakeredis.aioredis": None}):
-            with pytest.raises(
-                ImportError, match="fakeredis is required for DOCKET_BACKEND=fake"
-            ):
-                async with Docket(name="test-fake-docket"):
-                    pass
-    finally:
-        if original_value is None:
-            os.environ.pop("DOCKET_BACKEND", None)
-        else:
-            os.environ["DOCKET_BACKEND"] = original_value
+    monkeypatch.setenv("DOCKET_BACKEND", "fake")
+
+    # Mock the import to simulate fakeredis not being installed
+    with patch.dict("sys.modules", {"fakeredis.aioredis": None}):
+        with pytest.raises(
+            ImportError, match="fakeredis is required for DOCKET_BACKEND=fake"
+        ):
+            async with Docket(name="test-fake-docket"):
+                pass