# NousResearch/hermes-agent · Dashboard file-read credential guard security fixes

## Open now

- Nothing open on GitHub.

These items address a security concern where dashboard filesystem preview and git diff routes could read files outside the managed root, potentially exposing credentials on locked deployments. The work involves confining file reads to the managed root and widening the credential guard beyond .env files.

## TL;DR

- 4 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 2 PR(s) closed without merging.
- A fix was merged to main in PR #133869. Verify it addresses your case.
- **Recommended:** read PR #133869 (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-133869`. 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.50`: 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

- #58724: \[Bug\]: dashboard fs preview and git diff routes read credentials outside the managed root on locked deployments (bypasses #57505)

## Prior & competing PRs

- **#133869: fix(dashboard): confine fs preview and git diff reads to the managed root**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `hermes_cli/web_git.py`, `hermes_cli/web_routers/files.py`, `hermes_cli/web_routers/git.py`, `hermes_cli/web_server_files.py`, `tests/hermes_cli/test_web_read_confinement.py`
- **#58726: fix(dashboard): confine fs preview and git diff reads to the managed root**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`
- **#125994: fix(dashboard): apply credential guards to every fs/git file route**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#58724`

## Fix status

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