Skip to content

fix(honcho): scope gateway sessions by runtime user id#11434

Closed
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/honcho-gateway-user-scope
Closed

fix(honcho): scope gateway sessions by runtime user id#11434
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/honcho-gateway-user-scope

Conversation

@LeonSGP43

Copy link
Copy Markdown
Contributor

Summary

  • keep gateway user_id separate from HonchoClientConfig.peer_name during session setup
  • pass the runtime gateway identity into HonchoSessionManager and prefer it when choosing the user peer
  • update Honcho user-scoping regressions to assert runtime scoping without mutating config

Testing

  • python3 -m pytest -o addopts= tests/agent/test_memory_user_id.py -q
  • python3 -m pytest -o addopts= tests/agent/test_memory_provider.py -q
  • python3 -m pytest -o addopts= tests/honcho_plugin/test_session.py -q

Fixes #11199

@erosika

erosika commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

@LeonSGP43 — cherry-picked your commit into #12160 preserving authorship. It sits alongside a broader dialectic-lifecycle pass I was working on, and the per-user scoping fix here slots in cleanly. The runtime_user_peer_name plumbing is carried forward verbatim.

If #12160 lands, this can close. Thanks for the clean fix.

kshitijk4poor added a commit that referenced this pull request Apr 19, 2026
The cherry-picked commit from #11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
kshitijk4poor added a commit that referenced this pull request Apr 19, 2026
The cherry-picked commit from #11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
@kshitijk4poor

Copy link
Copy Markdown
Collaborator

Merged via PR #12419 as part of #12160's salvage. Your commit was cherry-picked with authorship preserved — runtime_user_peer_name approach is cleaner than config mutation. Thanks @LeonSGP43!

ulasbilgen pushed a commit to ulasbilgen/hermes-adhd-agent that referenced this pull request May 1, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
aj-nt pushed a commit to aj-nt/hermes-agent that referenced this pull request May 1, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
Luminet2023 pushed a commit to Luminet2023/hermes-agent that referenced this pull request May 1, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
The cherry-picked commit from NousResearch#11434 uses the 154585401+ prefixed
noreply format. Add it alongside the existing bare entry so the
contributor audit passes.
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.

[Bug]: Honcho memory scoping ignores gateway user_id when peer_name is configured

3 participants