[jgiunti-coatue1]

Summary

Add Hermes Agent as a supported platform for pup skills install, following the same shape as the devin target (#b333373). Hermes reads SKILL.md skills from ~/.hermes/skills/ (user-global) and <repo>/.hermes/skills/ (project-local), and exports HERMES_AGENT=true into every shell command it runs, so pup can both install skills for it and auto-detect it.

Changes

• Register hermes in PLATFORMS and add SkillsPlatform::Hermes + as_canonical arm (src/skills.rs:681, src/skills.rs:717, src/skills.rs:736). Both scopes resolve to .hermes/skills; agents fall back to SKILL.md like cursor/codex/devin.
• Add a hermes detector keyed on the HERMES_AGENT truthy marker, placed before generic-agent (src/useragent.rs:109). Source: Hermes exports HERMES_AGENT="${HERMES_AGENT:-true}" in tools/environments/base_session_env.py.
• Surface hermes in the skills help text (src/main.rs:2758), the resolve error hints (src/commands/skills.rs:18-33), the README Agent Skills section, docs/COMMANDS.md, and the agent detection list in docs/ARCHITECTURE.md.

Not included (intentionally, to keep one concern per PR): honoring HERMES_HOME for user-scope installs, analogous to CLAUDE_CONFIG_DIR. Happy to follow up if wanted.

Testing

• cargo fmt --check, cargo clippy --all-targets -- -D warnings: clean.
• New positive tests: test_lookup_platform_hermes, test_skills_dir_hermes_project_scope, test_skills_dir_hermes_user_scope, test_install_path_agent_hermes_as_skill, test_detect_agent_info_hermes; resolve_or_bail_expands_all now also asserts devin and hermes.
• New negative test: test_detect_agent_info_hermes_false_not_detected (HERMES_AGENT=false must not detect).
• Existing platform_enum_matches_table enforces the enum/table sync.
• cargo test: all new tests pass. 5 pre-existing mock-HTTP tests (model_lab, notebooks, security, users, debugger) fail identically on untouched main in my local environment with DNS errors against unused.local, so they are unrelated to this change.

Related Issues

None.

---

🤖 Generated with Claude Code

[datadog-datadog-prod-us1]

More details

Hermes uses the existing SKILL.md installation handling for both scopes, and its detector excludes HERMES_AGENT=false. The completed static review found no reportable defect.

Was this helpful? React 👍 or 👎

_{Open Bits AI session}

_{🤖 Bits Code Review · Commit b1ab1a9}