Cairn

Patch tool required params, error recovery, verification fixes

NousResearch/hermes-agent · cluster 15524 · agent

Data as of

Nothing open on GitHub. Fix merged in #22143. Check it covers your case.

The short answer

0open issues
0open PRs
12items grouped
Known?
Yes. 12 related items grouped here, 2 issues already closed.
Fix event
A fix was merged to main in PR #22143. Check it covers your case. No release data is tracked.
Tracker
The maintainers' tracker marks this partially resolved, deciding PR #22143. That is separate from the fix event.
Still open
Nothing open on GitHub.

These issues and PRs address bugs where conditionally required patch tool parameters were systematically omitted, add per-mode parameter hints in schema descriptions, and improve recoverability from malformed patch calls. A per-turn file-mutation verifier footer is also introduced.

Open issues0

No open issues.

Open PRs0

No open pull requests.

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

  • #22143 fix(patch-tool): advertise per-mode required params in schema descriptions

    merged to main

    files: tests/tools/test_file_tools.py, tools/file_tools.py

  • #97295 feat(skill_manage): operations[] is the call — each op names its skill; atomic with cross-skill rollback

    merged to main

    files: tests/tools/test_skill_manage_batch.py, tests/tools/test_skill_manage_schema_diet.py, tools/skill_manager_tool.py

  • #97650 fix(skill_manage): missing-old_string errors teach recovery instead of dead-ending into write_file clobbers (#33064, salvage #63879)

    merged to main

    files: tests/tools/test_skill_manager_tool.py, tools/skill_manager_tool.py

  • #24498 feat(agent): per-turn file-mutation verifier footer

    merged to main

    files: hermes_cli/config.py, run_agent.py, tests/run_agent/test_file_mutation_verifier.py, website/docs/reference/environment-variables.md, website/docs/user-guide/configuration.md

6 PRs closed without merging

From the close comments: merged via another change 3 · done elsewhere 1 · superseded by another PR 1 · other 1

  • #15673 fix(patch-tool): advertise per-mode required params in schema descriptions · merged via another change → #22143
  • #22149 fix(patch-tool): per-mode required-param hints + per-turn file-mutation verifier footer · merged via another change → 3adcc64, #24498, #15673
  • #33625 fix(skills): support V4A skill_manage patches · other
  • #33652 fix(skill_manage): support V4A patch format in action='patch' · superseded by another PR → #33625, #33064
  • #63876 fix(patch): make a half-formed replace call recoverable instead of a dead end · done elsewhere → #507, #54572
Show 1 more
  • #63879 fix(skills): make skill_manage patch failures recoverable instead of a dead end · merged via another change → dea2b02, 0af77d1, b8203b9, #97650, #97295
Closed issues · 2
  • Closed#15524[Bug]: `patch` tool conditionally required parameters are systematically omitted
  • Closed#33064[Bug]: skill_manage with action='patch' consistently requires old_string parameter and fails to apply patches when updating google-workspace skill
How this cluster was grouped
  • Link evidence (partly structural links): some members are connected by structural evidence (shared changed files, or a closing/superseded reference); the rest are joined by an automated relevance judgement over their text.
  • Members outside that structural core may belong to a different underlying problem.
  • Pairwise link density 0.16: the share of item pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. 1.00 means every pair is linked.

Pick up a Lab task

Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.

Open the Lab

At a glance

Subsystem
agent
Merged PRs
#22143, #97295, #97650, #24498
Closed issues
2
Attempts
6 PRs closed without merging
Fix link checked
29 Aug 2026

For your agent

The same facts, typed.

cluster/15524.json
cluster/15524.md