@miller-tech/uap 1.84.0 → 1.84.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miller-tech/uap",
-  "version": "1.84.0",
+  "version": "1.84.1",
   "description": "Autonomous AI agent memory system with CLAUDE.md protocol enforcement",
   "type": "module",
   "main": "dist/index.js",

package/tools/agents/scripts/anthropic_proxy.py

@@ -3351,7 +3351,28 @@ def anthropic_to_openai_response(anthropic_resp: dict) -> dict:
     }
 
 
+# A3: the anthropic->openai tool conversion + schema sanitize walks every tool's
+# (often deeply nested) JSON schema. The tool set is IDENTICAL across every turn
+# of a session, so this recomputed the same result each turn (observed: ~1
+# SCHEMA SANITIZE log per turn). Cache by a stable hash of the tool definitions.
+# Downstream only READS the converted dicts and FILTERS the list (narrowing), so
+# returning the cached object directly is safe.
+_TOOL_CONVERT_CACHE: "OrderedDict[str, list]" = OrderedDict()
+_TOOL_CONVERT_CACHE_MAX = 32
+
+
 def _convert_anthropic_tools_to_openai(anthropic_tools: list[dict]) -> list[dict]:
+    cache_key = None
+    try:
+        cache_key = hashlib.sha1(
+            json.dumps(anthropic_tools, sort_keys=True, default=str).encode("utf-8")
+        ).hexdigest()
+    except Exception:
+        cache_key = None
+    if cache_key is not None and cache_key in _TOOL_CONVERT_CACHE:
+        _TOOL_CONVERT_CACHE.move_to_end(cache_key)
+        return _TOOL_CONVERT_CACHE[cache_key]
+
     converted = []
     removed_pattern_fields = 0
     for tool in anthropic_tools:
@@ -3375,6 +3396,10 @@ def _convert_anthropic_tools_to_openai(anthropic_tools: list[dict]) -> list[dict
             removed_pattern_fields,
             len(anthropic_tools),
         )
+    if cache_key is not None:
+        _TOOL_CONVERT_CACHE[cache_key] = converted
+        if len(_TOOL_CONVERT_CACHE) > _TOOL_CONVERT_CACHE_MAX:
+            _TOOL_CONVERT_CACHE.popitem(last=False)
     return converted
 
 

package/tools/agents/tests/test_tool_convert_cache.py

@@ -0,0 +1,49 @@
+"""Tests for A3: per-session tool-conversion cache."""
+import importlib.util
+import unittest
+from pathlib import Path
+
+proxy_path = Path(__file__).resolve().parents[3] / "tools" / "agents" / "scripts" / "anthropic_proxy.py"
+spec = importlib.util.spec_from_file_location("anthropic_proxy", proxy_path)
+ap = importlib.util.module_from_spec(spec)
+spec.loader.exec_module(ap)
+
+TOOLS = [
+    {"name": "Read", "description": "read", "input_schema": {"type": "object",
+        "properties": {"p": {"type": "string", "pattern": "^/.*"}}, "required": ["p"]}},
+    {"name": "Bash", "description": "run", "input_schema": {"type": "object",
+        "properties": {"cmd": {"type": "string"}}}},
+]
+
+
+class ToolConvertCacheTest(unittest.TestCase):
+    def setUp(self):
+        ap._TOOL_CONVERT_CACHE.clear()
+
+    def test_correct_conversion_and_sanitize(self):
+        out = ap._convert_anthropic_tools_to_openai(TOOLS)
+        self.assertEqual(out[0]["function"]["name"], "Read")
+        # regex pattern field stripped by sanitize
+        self.assertNotIn("pattern", out[0]["function"]["parameters"]["properties"]["p"])
+
+    def test_second_call_is_cache_hit_same_object(self):
+        a = ap._convert_anthropic_tools_to_openai(TOOLS)
+        self.assertEqual(len(ap._TOOL_CONVERT_CACHE), 1)
+        b = ap._convert_anthropic_tools_to_openai([dict(t) for t in TOOLS])  # equal-by-value
+        self.assertIs(a, b, "identical tool set must return the cached object")
+
+    def test_different_tools_miss(self):
+        ap._convert_anthropic_tools_to_openai(TOOLS)
+        ap._convert_anthropic_tools_to_openai(TOOLS[:1])
+        self.assertEqual(len(ap._TOOL_CONVERT_CACHE), 2)
+
+    def test_cache_bounded(self):
+        for i in range(ap._TOOL_CONVERT_CACHE_MAX + 5):
+            ap._convert_anthropic_tools_to_openai(
+                [{"name": f"T{i}", "description": "", "input_schema": {"type": "object"}}]
+            )
+        self.assertLessEqual(len(ap._TOOL_CONVERT_CACHE), ap._TOOL_CONVERT_CACHE_MAX)
+
+
+if __name__ == "__main__":
+    unittest.main()