Skip to content

feat(hermes): 1Password integration for routine credentials - #88

Merged
salekseev merged 2 commits into
masterfrom
hermes-1password-integration
Jul 2, 2026
Merged

feat(hermes): 1Password integration for routine credentials#88
salekseev merged 2 commits into
masterfrom
hermes-1password-integration

Conversation

@salekseev

Copy link
Copy Markdown
Owner

Summary

  • Adds design spec for wiring 1Password into the Hermes gateway stack so routine credentials (Gixen, future services) live in 1Password instead of plaintext Portainer env vars
  • Single repo-side change: add OP_SERVICE_ACCOUNT_TOKEN env var to hermes-gateway; op CLI is installed at container runtime by the official security-1password Hermes skill via install-and-remember

What's in this PR

  • docs/superpowers/specs/2026-07-02-hermes-1password-integration-design.md — full design spec

Architecture

OP_SERVICE_ACCOUNT_TOKEN (Portainer env var) → hermes-gateway container → security-1password skill installs op CLI + forwards token to terminal sessions → routines call op read "op://hermes/<item>/<field>" on demand.

Infrastructure secrets (ANTHROPIC_TOKEN, CAMOFOX_SHARED_KEY, dashboard keys) remain in Portainer unchanged.

Phase 2

Documents migration path to native startup resolution when PR NousResearch/hermes-agent#36896 merges.

Test plan

  • Review spec for correctness and completeness
  • Approve spec before implementation plan is written

🤖 Generated with Claude Code

salekseev and others added 2 commits July 2, 2026 12:44
Phase-1: op CLI via install-and-remember + security-1password skill.
Phase-2: native secret source backend (pending PR #36896 merge).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…UUID

- Remove ANTHROPIC_TOKEN, HERMES_DASHBOARD_BASIC_AUTH_PASSWORD/SECRET,
  HINDSIGHT_API_KEY from compose env block — they move to /opt/data/.env
  on silverstone (not visible in Portainer UI, no redeploy to rotate)
- Remove HERMES_TUNNEL_ID interpolation; hardcode UUID inline like media.yaml
- Add OP_SERVICE_ACCOUNT_TOKEN to .env tier (1Password integration)
- Only cross-container secrets remain in Portainer: CAMOFOX_SHARED_KEY,
  HERMES_API_SERVER_KEY
- Update spec to match two-tier secret split

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@salekseev
salekseev marked this pull request as ready for review July 2, 2026 17:13
@salekseev
salekseev merged commit ae77aaf into master Jul 2, 2026
2 checks passed
@salekseev
salekseev deleted the hermes-1password-integration branch July 2, 2026 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant