iii-sdk 0.19.0.dev4__tar.gz → 0.19.2__tar.gz

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iii-sdk
-Version: 0.19.0.dev4
+Version: 0.19.2
 Summary: III SDK for Python
 Project-URL: Homepage, https://github.com/iii-hq/iii
 Project-URL: Repository, https://github.com/iii-hq/iii
@@ -14,7 +14,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.10
-Requires-Dist: iii-observability==0.19.0.dev4
+Requires-Dist: iii-observability==0.19.2
 Requires-Dist: opentelemetry-api>=1.25
 Requires-Dist: pydantic>=2.0
 Requires-Dist: websockets>=12.0

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "iii-sdk"
-version = "0.19.0.dev4"
+version = "0.19.2"
 description = "III SDK for Python"
 authors = [{ name = "III" }]
 license = { text = "Apache-2.0" }
@@ -23,7 +23,7 @@ dependencies = [
     "websockets>=12.0",
     "pydantic>=2.0",
     "opentelemetry-api>=1.25",
-    "iii-observability==0.19.0.dev4",
+    "iii-observability==0.19.2",
 ]
 
 [project.urls]

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/src/iii/iii.py

@@ -1235,7 +1235,7 @@ class III:
             ),
         }
 
-        return {
+        metadata: dict[str, Any] = {
             "runtime": "python",
             "version": sdk_version,
             "name": worker_name,
@@ -1244,6 +1244,11 @@ class III:
             "isolation": os.environ.get("III_ISOLATION") or None,
             "telemetry": telemetry,
         }
+        # Optional, like the other SDKs: only send `description` when set so the
+        # engine's `#[serde(default)]` field stays absent otherwise.
+        if self._options.worker_description:
+            metadata["description"] = self._options.worker_description
+        return metadata
 
     def _register_worker_metadata(self) -> None:
         msg = InvokeFunctionMessage(

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/src/iii/iii_constants.py

@@ -47,6 +47,8 @@ class InitOptions:
 
     Attributes:
         worker_name: Display name for this worker. Defaults to ``hostname:pid``.
+        worker_description: One-line, human/LLM-readable summary of what this
+            worker does. Surfaces in ``engine::workers::list`` / ``engine::workers::info``.
         enable_metrics_reporting: Enable worker metrics via OpenTelemetry. Default ``True``.
         invocation_timeout_ms: Default timeout for ``trigger()`` in milliseconds. Default ``30000``.
         reconnection_config: WebSocket reconnection behavior.
@@ -56,6 +58,7 @@ class InitOptions:
     """
 
     worker_name: str | None = None
+    worker_description: str | None = None
     enable_metrics_reporting: bool = True
     invocation_timeout_ms: int = DEFAULT_INVOCATION_TIMEOUT_MS
     reconnection_config: ReconnectionConfig | None = None

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/tests/test_api_triggers.py

@@ -136,6 +136,64 @@ async def test_raw_json_request_body(engine_http_url, iii_client: III):
     trigger.unregister()
 
 
+@pytest.mark.asyncio
+async def test_conflicting_route_structure_is_rejected(engine_http_url, iii_client: III, caplog):
+    """Two routes with identical structure but swapped path-param names must not
+    crash the engine: the first keeps serving and the second is rejected with a
+    logged registration error."""
+    caplog.set_level("ERROR", logger="iii")
+
+    def handler(input_data):
+        return {"status_code": 200, "body": {"ok": True}}
+
+    # First route registers normally.
+    fn_a = iii_client.register_function("test.api.conflict.a.py", handler)
+    trig_a = iii_client.register_trigger(
+        {
+            "type": "http",
+            "function_id": "test.api.conflict.a.py",
+            "config": {
+                "api_path": "test/py/conflict/:listId/:userId",
+                "http_method": "GET",
+            },
+        }
+    )
+
+    # Second route has the same axum shape with swapped param names -> conflict.
+    fn_b = iii_client.register_function("test.api.conflict.b.py", handler)
+    trig_b = iii_client.register_trigger(
+        {
+            "type": "http",
+            "function_id": "test.api.conflict.b.py",
+            "config": {
+                "api_path": "test/py/conflict/:userId/:listId",
+                "http_method": "GET",
+            },
+        }
+    )
+
+    # Give the engine time to process both registrations and reply.
+    time.sleep(0.5)
+
+    # Engine stayed alive and the first route still serves — no panic.
+    async with aiohttp.ClientSession() as session:
+        async with session.get(f"{engine_http_url}/test/py/conflict/list1/user1") as resp:
+            assert resp.status == 200
+            data = await resp.json()
+            assert data["ok"] is True
+
+    # The conflicting registration was surfaced as an error. The engine rejects
+    # whichever route it processes second (wire order is not guaranteed), so assert on
+    # the conflict message rather than a specific route or the random trigger id.
+    messages = [record.getMessage() for record in caplog.records]
+    assert any("conflicts with already-registered route" in m for m in messages), messages
+
+    fn_a.unregister()
+    trig_a.unregister()
+    fn_b.unregister()
+    trig_b.unregister()
+
+
 @pytest.mark.asyncio
 async def test_path_parameters(engine_http_url, iii_client: III):
     """Verify path parameters are extracted correctly."""

{iii_sdk-0.19.0.dev4 → iii_sdk-0.19.2}/tests/test_worker_metadata.py

@@ -47,6 +47,18 @@ def test_get_worker_metadata_forwards_iii_isolation_env_var(
     assert metadata["isolation"] == "docker"
 
 
+def test_get_worker_metadata_omits_description_when_unset() -> None:
+    metadata = _call_metadata_method()
+
+    assert "description" not in metadata
+
+
+def test_get_worker_metadata_includes_worker_description_when_set() -> None:
+    metadata = _call_metadata_method(InitOptions(worker_description="resizes images"))
+
+    assert metadata["description"] == "resizes images"
+
+
 @requires_tomllib
 def test_detect_project_name_reads_pyproject_name(tmp_path: Path) -> None:
     (tmp_path / "pyproject.toml").write_text('[project]\nname = "my-pkg"\n')