PR #14561: fix(agent): load all subdirectory hint files

• Repository: NousResearch/hermes-agent
• Author: LeonSGP43
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/14561

Description (problem / solution / changelog)

Summary

• load every supported hint file found in a newly visited subdirectory instead of stopping at the first match
• preserve existing formatting and truncation behavior for each discovered hint file
• add a regression test covering a directory that contains both AGENTS.md and .cursorrules

Testing

• python3 -m pytest -o addopts='' tests/agent/test_subdirectory_hints.py -q

Issue

• closes #14537

Changed files

• agent/subdirectory_hints.py (modified, +0/-2)
• tests/agent/test_subdirectory_hints.py (modified, +18/-0)

PR #14593: fix(hints): load all hint files per directory, not just the first

• Repository: NousResearch/hermes-agent
• Author: draix
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/14593

Description (problem / solution / changelog)

Problem

Closes #14537

SubdirectoryHintTracker._load_hints_for_directory() contains an unconditional break at the end of the per-filename try block. This stops the loop after the first matching file, silently dropping any other hint files in the same directory (e.g. AGENTS.md + .cursorrules).

The module docstring and issue description both state that all matching hint files should be loaded.

Root Cause

# agent/subdirectory_hints.py  (before)
found_hints.append((rel_path, content))
# First match wins per directory (like startup loading)
break  # ← stops after first file

Fix

Remove the break. The found_hints list and the sections-building code below the loop already handle multiple entries correctly — only the early exit was wrong.

# after
found_hints.append((rel_path, content))
# (no break — continue to next filename)

Minimal Reproduction (from issue)

import tempfile
from pathlib import Path
from agent.subdirectory_hints import SubdirectoryHintTracker

with tempfile.TemporaryDirectory() as td:
    wd = Path(td)
    sub = wd / 'pkg'
    sub.mkdir()
    (sub / 'AGENTS.md').write_text('agents rules')
    (sub / '.cursorrules').write_text('cursor rules')

    tracker = SubdirectoryHintTracker(str(wd))
    result = tracker._load_hints_for_directory(sub)
    print(result)

Before: only agents rules (cursor rules silently dropped) After: both agents rules and cursor rules

Tests Added

• test_loads_all_hint_files_in_directory — AGENTS.md + .cursorrules both appear in result
• test_first_hit_does_not_suppress_subsequent_files — two distinct hint files both loaded
• test_single_hint_file_still_works — regression guard for single-file directories

Changed files

• agent/subdirectory_hints.py (modified, +0/-2)
• tests/agent/test_subdirectory_hints.py (modified, +52/-0)