Skip to content

fix(memory): expose legacy honcho setup alias#25529

Closed
BeliefanX wants to merge 2 commits into
NousResearch:mainfrom
BeliefanX:fix/honcho-memory-cli-docs
Closed

fix(memory): expose legacy honcho setup alias#25529
BeliefanX wants to merge 2 commits into
NousResearch:mainfrom
BeliefanX:fix/honcho-memory-cli-docs

Conversation

@BeliefanX

Copy link
Copy Markdown
Contributor

Summary

  • keep a minimal hermes honcho compatibility entrypoint discoverable before Honcho is the active memory provider
  • redirect legacy hermes honcho setup users into hermes memory setup
  • align bundled docs/skills with the unified memory setup flow

Implementation

  • register a legacy-only Honcho CLI surface when no memory provider is active
  • preserve provider-gated full Honcho CLI behavior once memory.provider=honcho
  • update help text and Honcho skill/docs references to prefer hermes memory setup

Testing

  • pytest -q tests/hermes_cli/test_plugin_cli_registration.py
  • python -m hermes_cli.main --help
  • python -m hermes_cli.main honcho --help
  • python -m hermes_cli.main honcho setup --help

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins comp/cli CLI entry point, hermes_cli/, setup wizard tool/memory Memory tool and memory providers labels May 14, 2026
@BeliefanX

BeliefanX commented May 14, 2026

Copy link
Copy Markdown
Contributor Author

CI note: the PR-specific checks/tests pass locally (tests/hermes_cli/test_plugin_cli_registration.py + tests/hermes_cli/test_startup_plugin_gating.py: 47 passed). The current test workflow failure appears to match current main baseline drift rather than this PR: the failing nodes are in unrelated Bedrock/gateway/provider/transcription areas, and a recent main Tests run shows the same broad failure pattern. I did not fold unrelated baseline fixes into this focused Honcho CLI/docs PR.

@BeliefanX BeliefanX force-pushed the fix/honcho-memory-cli-docs branch from 8f40e34 to ab88e8b Compare May 29, 2026 07:26
@BeliefanX

Copy link
Copy Markdown
Contributor Author

Closing this to reduce the active PR maintenance set.

The remaining value here is a legacy hermes honcho setup compatibility alias, while the current direction is the unified hermes memory setup honcho flow. This branch is also now conflicted, so I'm closing it rather than refreshing it.

@BeliefanX BeliefanX closed this Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have tool/memory Memory tool and memory providers type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants