# NousResearch/hermes-agent · 1Password service account vault flag missing in item reads

## Open now

- Nothing open on GitHub.

These issues and PRs concern a bug where the --vault flag is not passed when reading 1Password items under service account authentication. Multiple fix attempts address including the vault parameter in item retrieval calls.

## TL;DR

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

- #108335: \[Bug\]: 1Password browser-vault fill omits --vault with service-account authentication

## Prior & competing PRs

- **#139203: 1Password browser logins fill again under a service account (#108335, salvage #109456)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `agent/vault_backends/onepassword.py`, `contributors/emails/[redacted:email]`, `tests/agent/test_vault_backends.py`
- **#108339: fix(vault): pass --vault to op item get for service-account 1Password auth**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `7520c4e8b37`, `#139203`, `#108335`, `#109456`
- **#108343: fix(vault): include vault in 1Password item reads**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `7520c4e8b37`, `#139203`, `#108335`, `#109456`
- **#116393: fix(vault): resolve 1Password items with their own vault for service accounts**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#129464`
- **#129464: fix(vault): scope service-account item reads to their vault**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `7520c4e8b37`, `#139203`, `#108335`, `#109456`

## Fix status

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