# NousResearch/hermes-agent · Bound recursion depth in browser output redaction walkers

## Open now

- Nothing open on GitHub.

These members share one theme: a bug report says deeply nested page values can trigger RecursionError in the browser tool's output redaction functions (\`_redact_browser_output\` / \`_redact_cdp_output\`), and three pull requests propose bounding or capping redaction depth. The titles do not show whether any change was merged or whether the problem is still open or has been closed.

## TL;DR

- 5 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 3 PR(s) closed without merging.
- A fix was merged to main in PR #140562. Verify it addresses your case.
- **Recommended:** read PR #140562 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `resolved` · deciding PR(s): `pr-140562`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.40`: the share of member 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.

## Closed / duplicate issues

- #137067: \[Bug\]: Deeply nested page value crashes the browser tool output redaction walkers with RecursionError (no depth budget on \`_redact_browser_output\` / \`_redact_cdp_output\`)

## Prior & competing PRs

- **#140562: Browser tools no longer crash on deeply nested page data (#137067, salvage #137138)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `agent/redact.py`, `contributors/emails/[redacted:email]`, `tests/tools/test_browser_redact_depth.py`, `tools/browser_cdp_tool.py`, `tools/browser_tool_snapshot.py`
- **#137070: fix(browser): bound output redaction depth**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `18c525b669`, `870633cba780`, `#140562`
- **#137075: fix(browser): bound the redaction walkers so deep page values cannot crash them**. closed without merge
  - maintainer direction: category `uncategorized`; implication `needs_llm`; refs `#137070`, `#137067`
- **#137138: fix(browser): cap redaction walks so nested page data cannot crash**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `18c525b669`, `440c9b61dfd1`, `#140562`

## Fix status

- Merged to main in PR #140562 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-10-17`.
- Subsystem: `agent`