Skip to content

fix(hermes): expose plugins to launchd gateway - #79

Merged
furedea merged 4 commits into
mainfrom
fix/hermes-launchd-plugin-discovery
Aug 31, 2026
Merged

fix(hermes): expose plugins to launchd gateway#79
furedea merged 4 commits into
mainfrom
fix/hermes-launchd-plugin-discovery

Conversation

@furedea

@furedea furedea commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • leave mutable secretary credentials untouched by Home Manager
  • backport the macOS launchd portion of fix(gateway): propagate HERMES_BUNDLED_* into launchd/systemd units NousResearch/hermes-agent#85365 at head commit 9c938204ba5f5bdf379ef9bb389a72841951447b
  • propagate only allowlisted packaged resource paths into the generated launchd plist
  • verify that a direct Python process using the generated launchd environment discovers the bundled Slack plugin
  • keep the Slack gateway dependency import check

No credentials are copied into launchd, and Nix does not read or rewrite the local credential file.

Testing

  • bats --print-output-on-failure tests/nix/hermes_secretary.bats (9 tests passed)
  • nix flake check --print-build-logs
  • nixfmt --check flake.nix nix/darwin/default.nix nix/home/default.nix
  • shfmt -d tests/nix/hermes_secretary.bats
  • statix check flake.nix
  • statix check nix/home/default.nix
  • deadnix --fail flake.nix nix/home/default.nix
  • Apple plutil validation of the generated plist

Copilot AI lite review requested due to automatic review settings August 31, 2026 15:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@furedea
furedea merged commit 001738a into main Aug 31, 2026
11 checks passed
@furedea
furedea deleted the fix/hermes-launchd-plugin-discovery branch August 31, 2026 18:18
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.

2 participants