Cairn

Fuzzy match strategy bugs and patch tool fixes

NousResearch/hermes-agent · cluster 14497 · agent

Data as of

8 open items. Fix merged in #137288. Check it covers your case.

The short answer

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

These issues and PRs address two related areas: dead/unreachable code in fuzzy_match.py due to incorrect strategy ordering, and patch tool bugs including wrong-region edits and duplicate-content loops from repeated application.

Open issues4

  • Issue#14497[Bug]: Redundancy in fuzzy match: indentation_flexible (strategy 4) is a strict subset of line_trimmed (strategy 2)
  • Issue#14781[Bug]: fuzzy_match.py strategy ordering makes indentation_flexible dead code
  • Issue#18426patch tool can create duplicate content loops when used repeatedly
  • Issue#54572patch tool (replace mode) can edit the wrong region when old_string is not an exact match

Open PRs4

  • PR#14522fix: reorder fuzzy match strategies so indentation_flexible is not dead code
  • PR#14777fix(tools): reorder fuzzy match strategies so indentation_flexible is no longer dead code
  • PR#54575fix(patch): reject wrong-region matches in the fuzzy edit fallback
  • PR#56570fix(tools): patch idempotency guard prevents duplicate-content loops on re-edit (#18426)

Fixes on record

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

  • #137288 fix(tools): patch matching fixes from NVIDIA batch (#128006, #128020, #128138)

    merged to main

    files: tests/tools/test_escape_drift_autocorrect.py, tests/tools/test_file_tools_live.py, tests/tools/test_fuzzy_match.py, tools/escape_drift_autocorrect.py, tools/file_operations.py, tools/fuzzy_match.py, tools/patch_parser.py, tools/skill_manager_tool.py

3 PRs closed without merging

From the close comments: duplicate 1 · merged via another change 1 · stale or abandoned 1

  • #15011 fix(tools): reorder fuzzy_match strategies so indentation_flexible is reachable · duplicate → #14522
  • #84278 fix(tools): guard replayed JSON value patches · stale or abandoned
  • #128138 fix(tools): patch writefile verification warning · merged via another change → #137288
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.15: 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
#137288
Closed issues
0
Attempts
3 PRs closed without merging
Fix link checked
12 Oct 2026

For your agent

The same facts, typed.

cluster/14497.json
cluster/14497.md