mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-15 14:22:43 +00:00
a0a6cd80f5
* fix(agent): persist truthful tool effect dispositions * fix(agent): preserve successful siblings during orphan recovery * fix(agent): narrow effect dispositions to none and unknown
40 lines
1.3 KiB
Python
40 lines
1.3 KiB
Python
"""Shared helpers for classifying tool result payloads."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import json
|
|
from typing import Any
|
|
|
|
|
|
FILE_MUTATING_TOOL_NAMES = frozenset({"write_file", "patch"})
|
|
|
|
|
|
# Tools whose interrupted/dangling execution is safe to discard because they
|
|
# cannot mutate either external state or Hermes session state. Unknown/plugin/
|
|
# MCP tools stay effect-capable by default.
|
|
NO_EFFECT_TOOL_NAMES = frozenset({
|
|
"read_file", "search_files", "session_search", "skill_view", "skills_list",
|
|
"web_extract", "web_search", "vision_analyze", "browser_snapshot",
|
|
"browser_get_images", "browser_console", "read_terminal",
|
|
})
|
|
|
|
|
|
def tool_may_have_side_effect(tool_name: str) -> bool:
|
|
return tool_name not in NO_EFFECT_TOOL_NAMES
|
|
|
|
|
|
def file_mutation_result_landed(tool_name: str, result: Any) -> bool:
|
|
"""Return True when a file mutation result proves the write landed."""
|
|
if tool_name not in FILE_MUTATING_TOOL_NAMES or not isinstance(result, str):
|
|
return False
|
|
try:
|
|
data = json.loads(result.strip())
|
|
except Exception:
|
|
return False
|
|
if not isinstance(data, dict) or data.get("error"):
|
|
return False
|
|
if tool_name == "write_file":
|
|
return "bytes_written" in data
|
|
if tool_name == "patch":
|
|
return data.get("success") is True
|
|
return False
|