@miller-tech/uap 1.20.44 → 1.20.45

package/package.json

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

package/tools/agents/scripts/anthropic_proxy.py

@@ -2472,8 +2472,21 @@ def _completion_blockers(
 def _sanitize_tool_schema_for_llama(schema):
     """Remove JSON Schema keywords that generate unsupported regex grammar.
 
-    llama.cpp's tool grammar generator can fail on regex-heavy schema fields
-    such as "pattern" and "patternProperties" (for example "\\w").
+    llama.cpp's tool grammar generator can fail on regex-heavy schema fields:
+
+    - "pattern" / "patternProperties" — regex strings (e.g. "\\w").
+    - "format" — string formats. llama.cpp's json-schema-to-grammar turns
+      "format": "date" / "date-time" / "time" / "uuid" into grammar rules
+      built from `\\d`, which its own GBNF parser then rejects with
+      `error parsing grammar: unknown escape at \\d...` → `failed to parse
+      grammar`. Observed on MCP tools with date fields (Atlassian
+      getJiraIssue, tempo bulkCreateWorklogs). "format" is an advisory
+      annotation — dropping it just leaves the field as an unconstrained
+      string in the tool-call grammar, which is correct behaviour.
+
+    All three are stripped only when they appear as schema *keywords*, not
+    when they are property *names* (a tool may legitimately have a parameter
+    literally called "pattern" or "format").
     """
 
     removed = 0
@@ -2486,7 +2499,7 @@ def _sanitize_tool_schema_for_llama(schema):
             for key, value in node.items():
                 key_is_property_name = parent_key in property_map_keys
                 if (
-                    key == "pattern"
+                    key in ("pattern", "format")
                     and isinstance(value, str)
                     and not key_is_property_name
                 ):

package/tools/agents/tests/test_anthropic_proxy_streaming.py

@@ -284,6 +284,79 @@ class TestToolSchemaSanitization(unittest.TestCase):
         self.assertIn("pattern", params["required"])
         self.assertEqual(params["properties"]["pattern"]["type"], "string")
 
+    def test_convert_tools_strips_format_fields(self):
+        """A string field with "format": "date" must have format stripped.
+        llama.cpp's json-schema-to-grammar turns format:date/date-time/etc.
+        into `\\d`-based grammar rules that its own GBNF parser then rejects
+        ('unknown escape at \\d' -> 'failed to parse grammar'). Observed on
+        MCP tools like tempo bulkCreateWorklogs (a worklogEntries[].date
+        field) and Atlassian getJiraIssue."""
+        anthropic_tools = [
+            {
+                "name": "bulkCreateWorklogs",
+                "description": "test",
+                "input_schema": {
+                    "type": "object",
+                    "properties": {
+                        "worklogEntries": {
+                            "type": "array",
+                            "items": {
+                                "type": "object",
+                                "properties": {
+                                    "date": {
+                                        "type": "string",
+                                        "format": "date",
+                                    },
+                                    "started": {
+                                        "type": "string",
+                                        "format": "date-time",
+                                    },
+                                },
+                            },
+                        }
+                    },
+                },
+            }
+        ]
+
+        converted = proxy._convert_anthropic_tools_to_openai(anthropic_tools)
+        item = converted[0]["function"]["parameters"]["properties"][
+            "worklogEntries"
+        ]["items"]
+        self.assertNotIn("format", item["properties"]["date"])
+        self.assertNotIn("format", item["properties"]["started"])
+        # The field itself and its type survive — only the format hint goes.
+        self.assertEqual(item["properties"]["date"]["type"], "string")
+
+    def test_convert_tools_keeps_property_named_format(self):
+        """A tool parameter literally named "format" (e.g. an output-format
+        selector) must NOT be stripped — only the format *keyword* is."""
+        anthropic_tools = [
+            {
+                "name": "ExportTool",
+                "description": "test",
+                "input_schema": {
+                    "type": "object",
+                    "required": ["format"],
+                    "properties": {
+                        "format": {
+                            "type": "string",
+                            "enum": ["json", "csv", "yaml"],
+                            "description": "Output format",
+                        },
+                    },
+                },
+            }
+        ]
+
+        converted = proxy._convert_anthropic_tools_to_openai(anthropic_tools)
+        params = converted[0]["function"]["parameters"]
+        self.assertIn("format", params["required"])
+        self.assertEqual(params["properties"]["format"]["type"], "string")
+        self.assertEqual(
+            params["properties"]["format"]["enum"], ["json", "csv", "yaml"]
+        )
+
 
 class TestStreamGuardedPathSelection(unittest.TestCase):
     def test_required_tool_turn_uses_guarded_non_stream(self):