@inline-chat/hermes-agent-adapter 0.0.7 → 0.0.8

package/plugin/inline/tools.py

@@ -23,6 +23,8 @@ except Exception:  # pragma: no cover - used by lightweight package tests
 _DEFAULT_SIDECAR_PORT = 8794
 _DEFAULT_SIDECAR_BIND = "127.0.0.1"
 _MAX_HISTORY_LIMIT = 100
+_MAX_MESSAGE_IDS = 100
+_MAX_INLINE_ID = 9_223_372_036_854_775_807
 _DEFAULT_HISTORY_LIMIT = 20
 _MAX_TEXT_CHARS = 4000
 _MAX_QUERY_CHARS = 500
@@ -56,7 +58,11 @@ INLINE_PLATFORM_GUIDANCE = (
     "while keeping the stable user ID in the link target. Link chats as "
     "[title](inline://chat?id=123), and link reply threads as "
     "[title](inline://thread?id=123). In Inline, reply threads are chat ids; "
-    "do not treat thread ids as reply/quote message ids."
+    "do not treat thread ids as reply/quote message ids. Use ordinary Markdown "
+    "tables for tabular data, never fenced-code tables. For collapsible work, "
+    "use <details open>, then <summary>Title</summary>, body Markdown, and "
+    "</details>; put kind=\"progress\" on summary only while work is in progress. "
+    "Use <footer>Attribution or brief metadata</footer> for a short footer."
 )
 
 _sidecar: Dict[str, Any] = {}
@@ -76,6 +82,7 @@ _ACTION_MANIFEST = [
     ("list_pins", "(chat_id?)", "List pinned Inline message IDs for a chat or reply thread."),
     ("create_thread", "(parent_chat_id?, parent_message_id?, title?)", "Create an Inline reply thread."),
     ("create_chat", "(title, space_id?, participant_user_ids?, is_public?)", "Create a top-level Inline thread/chat."),
+    ("create_agent", "(name, skill_key?, instructions?)", "Create a named Inline Agent backed by this bot."),
     ("set_presence", "(chat_id?|user_id?, kind, comment?)", "Set the bot avatar presence/status message."),
 ]
 _ACTIONS = [name for name, _, _ in _ACTION_MANIFEST]
@@ -179,6 +186,7 @@ INLINE_TOOL_SCHEMA = {
             "message_ids": {
                 "type": "array",
                 "items": {"type": "string"},
+                "maxItems": _MAX_MESSAGE_IDS,
                 "description": "Inline message IDs for get_messages.",
             },
             "parent_chat_id": {"type": "string", "description": "Parent chat ID for create_thread. Defaults to the current chat."},
@@ -187,6 +195,10 @@ INLINE_TOOL_SCHEMA = {
                 "description": "Parent message ID for create_thread. Defaults to the triggering message when available.",
             },
             "title": {"type": "string", "description": "Thread title. Required for create_chat and optional for create_thread."},
+            "name": {"type": "string", "description": "Agent name for create_agent."},
+            "handle": {"type": "string", "description": "Optional Agent handle."},
+            "skill_key": {"type": "string", "description": "Optional harness skill key for create_agent."},
+            "instructions": {"type": "string", "description": "Optional specialized Agent instructions."},
             "description": {"type": "string", "description": "Optional thread description for create_thread or create_chat."},
             "emoji": {"type": "string", "description": "Optional thread emoji for create_thread/create_chat, or reaction emoji for reaction actions."},
             "space_id": {"type": "string", "description": "Optional parent space ID for create_chat."},
@@ -356,6 +368,14 @@ def _request_for_action(action: str, args: Dict[str, Any]) -> tuple[str, Dict[st
                 body[key] = _truncate(value, limit)
         return "/create-chat", body
 
+    if action == "create_agent":
+        body = {"name": _required_str(args, "name", max_chars=256)}
+        for key in ("handle", "emoji", "description", "skill_key", "instructions"):
+            value = _str(args.get(key))
+            if value:
+                body[key] = value
+        return "/create-agent", body
+
     if action == "set_presence":
         kind = _str(args.get("kind"))
         if kind not in _PRESENCE_KINDS:
@@ -460,11 +480,21 @@ def _message_ids(args: Dict[str, Any]) -> list[str]:
         values = _str(raw).split(",")
     else:
         values = []
-    ids = [_inline_id(value) for value in values]
-    single = _inline_id(args.get("message_id"))
-    if single:
-        ids.append(single)
-    return [item for item in ids if item]
+    items = list(values)
+    single_value = args.get("message_id")
+    if _str(single_value):
+        items.append(single_value)
+    if len(items) > _MAX_MESSAGE_IDS:
+        raise InlineToolError(f"message_ids supports at most {_MAX_MESSAGE_IDS} items", "bad_format")
+    ids: list[str] = []
+    seen: set[str] = set()
+    for item in items:
+        inline_id = _inline_id(item)
+        if not inline_id or inline_id in seen:
+            continue
+        seen.add(inline_id)
+        ids.append(inline_id)
+    return ids
 
 
 def _message_id_or_current(args: Dict[str, Any]) -> str:
@@ -515,6 +545,8 @@ def _compact_result(action: str, result: Dict[str, Any]) -> Dict[str, Any]:
             "chat": _compact_chat(result.get("chat") if isinstance(result.get("chat"), dict) else {}),
             "dialog": _summarize_value(result.get("dialog")) if result.get("dialog") is not None else None,
         }
+    if action == "create_agent":
+        return {"agent": _summarize_value(result.get("agent"))}
     return result
 
 
@@ -746,13 +778,20 @@ def _inline_id(value: Any) -> str:
         prefix, rest = text.split(":", 1)
         if prefix.lower() in {"chat", "thread", "user", "space", "message", "msg"}:
             text = rest.strip()
-    return text
+    if not text.isdigit():
+        raise InlineToolError("Inline IDs must be positive signed 64-bit integers", "bad_format")
+    parsed = int(text)
+    if parsed <= 0 or parsed > _MAX_INLINE_ID:
+        raise InlineToolError("Inline IDs must be positive signed 64-bit integers", "bad_format")
+    return str(parsed)
 
 
 def _id_list(value: Any, *, max_items: int) -> list[str]:
     if value is None:
         return []
-    items = value if isinstance(value, (list, tuple, set)) else [value]
+    items = list(value) if isinstance(value, (list, tuple)) else [value]
+    if len(items) > max_items:
+        raise InlineToolError(f"too many IDs (max {max_items})", "bad_format")
     ids: list[str] = []
     seen: set[str] = set()
     for item in items:
@@ -761,8 +800,6 @@ def _id_list(value: Any, *, max_items: int) -> list[str]:
             continue
         seen.add(inline_id)
         ids.append(inline_id)
-        if len(ids) > max_items:
-            raise InlineToolError(f"too many IDs (max {max_items})", "bad_format")
     return ids