You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This issue is a cleanup map for the current open Honcho PR cluster: pick one canonical PR per bug cluster, merge/rebase that, then close duplicates with a Superseded by #NNN link.
This is not meant to be an exhaustive audit of every Honcho search hit. It is a maintainer path through the pile.
Canonical: #27979, adopted into local #30077 work with authorship preserved.
#17124, #31240, and #27979 all address the same direct target peer-card fallback cluster. Prefer #27979 because it includes both:
direct target peer-card fallback, and
peer-card read/write path alignment.
#31240 is narrower and includes .paperclip_artifacts/candidate-skip-ledger.jsonl; do not adopt it directly. #17124 includes merge commits and an older fallback shape.
#23485 is also a good fit: it aligns honcho_context(peer="user") perspective with the same observer/target resolution model.
Recommendation: keep #24763 and #31664 narrow. Treat #24847/#19102 as startup-latency/fail-open work; verify against the current lazy/session-init path.
Some of these are broader memory-provider/prompt-assembly hygiene, not Honcho-only. Pick one canonical boundary rule for memory-context handling, then close narrower duplicates.
Honcho PR Map: Cleanup Pass
Context
This issue is a cleanup map for the current open Honcho PR cluster: pick one canonical PR per bug cluster, merge/rebase that, then close duplicates with a
Superseded by #NNNlink.This is not meant to be an exhaustive audit of every Honcho search hit. It is a maintainer path through the pile.
Current highest-priority path
1. Security / config hygiene
0600permissions to memory-plugin config files containing API keys.observe_me=True: Honcho integration runs third-party inference on user conversations without disclosure #4074: cloud disclosure/security issue. fix(honcho): disclose cloud data flow during setup #24458 attempts setup disclosure, but needs freshness/rebase review.2. Runtime identity / peer mapping
Canonical: #30077.
#30077 now carries the runtime peer mapping path and should be treated as the canonical identity/perspective correctness PR.
It covers:
pinPeerNameinhermes memory setupwizard for multi-instance Honcho setups #28283: setup/status UX forpinPeerName/pinUserPeer.pinPeerNamesurface, superseded by fix(honcho): runtime peer mapping — correctness follow-ups + setup wizard + docs #30077's broader setup/status/docs work.Do not fold these into #30077:
userPeerAliasesdocs/examples if needed; do not addpeerByLidas a parallel Honcho config surface.peerIdentityStrategy/peerIdentitiesmodel. Do not mix with fix(honcho): runtime peer mapping — correctness follow-ups + setup wizard + docs #30077's resolver surface.3. Peer cards /
No profile factsCanonical: #27979, adopted into local #30077 work with authorship preserved.
#17124, #31240, and #27979 all address the same direct target peer-card fallback cluster. Prefer #27979 because it includes both:
#31240 is narrower and includes
.paperclip_artifacts/candidate-skip-ledger.jsonl; do not adopt it directly. #17124 includes merge commits and an older fallback shape.#23485 is also a good fit: it aligns
honcho_context(peer="user")perspective with the same observer/target resolution model.Local validation performed against #30077 + #27979 + #23485:
scripts/run_tests.sh tests/honcho_plugin/test_session.py tests/test_honcho_session_context.py tests/honcho_plugin/test_pin_peer_name.py tests/honcho_plugin/test_cli.py tests/run_agent/test_memory_provider_init.py tests/gateway/test_agent_cache.pyAfter #27979/#23485 are pushed into #30077, #30077 should also close/supersede:
honcho_profilereturnsNo profile factson self-hosted Honcho v3-style direct peer-card lookup.4. Multimodal sync crash
Canonical: #32030.
Recommendation: prefer #32030; close #22768/#30264/#30294 if #32030 covers their tests.
5. Self-hosted / base URL / local auth
Keep these as separate sub-bugs unless a clean self-hosted umbrella PR appears:
baseUrland local session links.6. Session naming / path normalization
These are small enough to merge independently after freshness checks.
Secondary clusters
Startup/lifecycle reliability
Recommendation: keep #24763 and #31664 narrow. Treat #24847/#19102 as startup-latency/fail-open work; verify against the current lazy/session-init path.
Tool correctness / budgets / schemas
honcho_search/honcho_reasoningqueries.honcho_concludecannot retry after initialization failure.platform_toolsetsis configured.Prioritize #29403 and #4523 because they affect runtime correctness/tool availability.
Memory-context leakage / prompt hygiene
<memory-context>leaks.Some of these are broader memory-provider/prompt-assembly hygiene, not Honcho-only. Pick one canonical boundary rule for memory-context handling, then close narrower duplicates.
Session writes / metadata / write frequency
writeFrequencyinsync_turn.sessionStrategy.Compare #18536, #19650, and #9422 before merging; they may touch the same write path from different angles.
Plugin migration / setup compatibility
plugins/memory/honcho/, so this is migration/activation cleanup, not the original move.Only merge if current setup flows still break for users upgrading from old Honcho config paths. Otherwise close as stale migration cleanup.
Optional/session-sharing features
HONCHO_SESSIONenv var for per-launch session pinning.Defer until #30077 settles the core identity mapping semantics.
Suggested cleanup order
At a glance
Status note: PR mergeability and cherry-pick viability must be rechecked immediately before merging/closing anything.