Skip to content

fix(honcho): surface profile namespace diagnostics#43230

Open
konsisumer wants to merge 1 commit into
NousResearch:mainfrom
konsisumer:fix/honcho-profile-namespace-diagnostics
Open

fix(honcho): surface profile namespace diagnostics#43230
konsisumer wants to merge 1 commit into
NousResearch:mainfrom
konsisumer:fix/honcho-profile-namespace-diagnostics

Conversation

@konsisumer

Copy link
Copy Markdown
Contributor

What changed and why

This adds a diagnostics-first followup to the profile-scoped memory discussion in #4726. hermes honcho status now tells the active profile whether it is using a strict profile-isolated workspace, sharing a workspace with distinct user peers, or sharing the same workspace and user peer with other profiles. hermes honcho status --all now summarizes cross-profile namespace groups and warns when multiple AI peers share the same workspace/user namespace, because gateway session keys are still AI-peer agnostic today.

Per the reporter's followup, this intentionally stops short of changing Honcho routing semantics. The goal is to make the current sharing policy and the gateway-collision risk visible to operators before introducing any new namespace mode or session-prefix behavior.

How to test

  • Run pytest tests/honcho_plugin/test_cli.py -q --timeout=60
  • Configure two Honcho profiles that share the same workspace and peerName but use different aiPeer values, then run hermes honcho status inside one profile and confirm it reports a shared user memory namespace plus the gateway-session warning.
  • Run hermes honcho status --all and confirm it groups shared namespaces and prints the same warning for multi-AI-peer shared-user layouts.

What platforms tested on

  • macOS (local pytest run in the worker sandbox)

Fixes #4726

@alt-glitch alt-glitch added type/feature New feature or request comp/plugins Plugin system and bundled plugins tool/memory Memory tool and memory providers P3 Low — cosmetic, nice to have labels Jun 10, 2026

@tonydwb tonydwb 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.

Code Review Summary

Verdict: Approved

Looks Good:

  • Fixes honcho plugin to surface profile namespace diagnostics. Previously, when the honcho memory plugin encountered namespace issues (missing profile, invalid config), the errors were silently swallowed. Now they surface to the user with actionable diagnostics.
  • 2 files, +190/-9. Clean fix in honcho/cli.py with test coverage in test_cli.py.
  • The print statements for diagnostics are intentional user-facing output, not debug artifacts.

Reviewed by Hermes Agent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have tool/memory Memory tool and memory providers type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Profile-scoped memory namespaces for multi-agent setups

3 participants