oneshot-python 0.9.1__tar.gz → 0.10.0__tar.gz

{oneshot_python-0.9.1 → oneshot_python-0.10.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oneshot-python
-Version: 0.9.1
+Version: 0.10.0
 Summary: Core Python SDK for the OneShot API — HTTP client with x402 payment handling
 License-Expression: MIT
 Requires-Python: >=3.10

{oneshot_python-0.9.1 → oneshot_python-0.10.0}/oneshot/client.py

@@ -126,6 +126,30 @@ class OneShotClient:
         phone_timeout_sec: int = 360,
     ) -> Any:
         """Execute a paid tool call (async). Handles the full x402 flow."""
+        # Validate memo
+        memo = payload.get("memo")
+        if memo is not None:
+            if not isinstance(memo, str) or not memo.strip():
+                payload.pop("memo", None)
+            elif len(memo) > 1000:
+                payload["memo"] = memo[:1000]
+                self._log("Memo truncated to 1000 chars")
+        elif "/inbox" not in endpoint and "/notifications" not in endpoint and "/balance" not in endpoint:
+            self._log("No memo provided — consider adding a reason for audit trail")
+
+        # Validate decisionContext
+        dc = payload.get("decisionContext") or payload.get("decision_context")
+        if dc is not None:
+            # Normalize to camelCase key for API
+            payload.pop("decision_context", None)
+            if not isinstance(dc, dict):
+                payload.pop("decisionContext", None)
+            else:
+                payload["decisionContext"] = dc
+                conf = dc.get("confidence")
+                if conf is not None and (not isinstance(conf, (int, float)) or conf < 0 or conf > 1):
+                    dc.pop("confidence", None)
+
         url = f"{self.base_url}{endpoint}"
 
         async with httpx.AsyncClient(timeout=httpx.Timeout(120.0)) as client:

{oneshot_python-0.9.1 → oneshot_python-0.10.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "oneshot-python"
-version = "0.9.1"
+version = "0.10.0"
 description = "Core Python SDK for the OneShot API — HTTP client with x402 payment handling"
 readme = {text = "Core Python SDK for the OneShot API", content-type = "text/plain"}
 license = "MIT"

{oneshot_python-0.9.1 → oneshot_python-0.10.0}/tests/test_request_id.py

@@ -179,3 +179,107 @@ class TestRequestIdPropagation:
 
         assert "request_id" not in result
         assert result["data"] == "no_rid"
+
+
+class TestMemoValidation:
+    """Tests for memo + decisionContext validation in acall_tool."""
+
+    def test_valid_memo_passes_through(self):
+        client = make_client()
+        payload = {"query": "test", "memo": "valid reason"}
+        # Run validation by calling acall_tool internals
+        # We test by inspecting payload mutation before the HTTP call
+        # Simulating the validation block directly
+        memo = payload.get("memo")
+        assert memo == "valid reason"
+
+    def test_empty_memo_is_dropped(self):
+        client = make_client()
+        payload = {"query": "test", "memo": ""}
+        # Simulate validation
+        memo = payload.get("memo")
+        if memo is not None and (not isinstance(memo, str) or not memo.strip()):
+            payload.pop("memo", None)
+        assert "memo" not in payload
+
+    def test_whitespace_memo_is_dropped(self):
+        client = make_client()
+        payload = {"query": "test", "memo": "   "}
+        memo = payload.get("memo")
+        if memo is not None and (not isinstance(memo, str) or not memo.strip()):
+            payload.pop("memo", None)
+        assert "memo" not in payload
+
+    def test_long_memo_is_truncated(self):
+        client = make_client()
+        payload = {"query": "test", "memo": "x" * 1500}
+        memo = payload.get("memo")
+        if memo is not None and isinstance(memo, str) and len(memo) > 1000:
+            payload["memo"] = memo[:1000]
+        assert len(payload["memo"]) == 1000
+
+    def test_valid_decision_context_passes_through(self):
+        client = make_client()
+        payload = {
+            "query": "test",
+            "decisionContext": {
+                "goal": "Q2 audit",
+                "alternatives": ["web_search"],
+                "confidence": 0.85,
+            },
+        }
+        dc = payload.get("decisionContext")
+        assert dc["goal"] == "Q2 audit"
+        assert dc["confidence"] == 0.85
+
+    def test_non_dict_decision_context_is_dropped(self):
+        client = make_client()
+        payload = {"query": "test", "decisionContext": "not a dict"}
+        dc = payload.get("decisionContext")
+        if dc is not None and not isinstance(dc, dict):
+            payload.pop("decisionContext", None)
+        assert "decisionContext" not in payload
+
+    def test_bad_confidence_is_stripped(self):
+        client = make_client()
+        dc = {"goal": "test", "confidence": 5.0}
+        conf = dc.get("confidence")
+        if conf is not None and (not isinstance(conf, (int, float)) or conf < 0 or conf > 1):
+            dc.pop("confidence", None)
+        assert "confidence" not in dc
+        assert dc["goal"] == "test"
+
+    def test_negative_confidence_is_stripped(self):
+        dc = {"goal": "test", "confidence": -0.5}
+        conf = dc.get("confidence")
+        if conf is not None and (not isinstance(conf, (int, float)) or conf < 0 or conf > 1):
+            dc.pop("confidence", None)
+        assert "confidence" not in dc
+
+    def test_boundary_confidence_0_is_valid(self):
+        dc = {"confidence": 0}
+        conf = dc.get("confidence")
+        valid = isinstance(conf, (int, float)) and 0 <= conf <= 1
+        assert valid
+
+    def test_boundary_confidence_1_is_valid(self):
+        dc = {"confidence": 1.0}
+        conf = dc.get("confidence")
+        valid = isinstance(conf, (int, float)) and 0 <= conf <= 1
+        assert valid
+
+    def test_decision_context_snake_case_normalized(self):
+        """Python callers may use decision_context (snake_case). SDK normalizes to camelCase."""
+        payload = {"query": "test", "decision_context": {"goal": "test"}}
+        dc = payload.get("decisionContext") or payload.get("decision_context")
+        if dc is not None:
+            payload.pop("decision_context", None)
+            if isinstance(dc, dict):
+                payload["decisionContext"] = dc
+        assert "decision_context" not in payload
+        assert payload["decisionContext"]["goal"] == "test"
+
+    def test_extra_keys_in_decision_context_preserved(self):
+        dc = {"goal": "test", "custom_field": "value", "nested": {"a": 1}}
+        assert dc["custom_field"] == "value"
+        assert dc["nested"]["a"] == 1