fix(sessions): resolve transcript paths with explicit agent context#16288
Merged
gumadeiras merged 5 commits intoFeb 14, 2026
Merged
Conversation
openclaw-barnacle
Bot
added
channel: discord
robbyczgw-cla
force-pushed
the
fix/16271-16278-session-path-agentid
branch
from
February 14, 2026 16:11
b0d6b99 to
4539408
Compare
openclaw-barnacle
Bot
added
size: XS
and removed
channel: discord
robbyczgw-cla
force-pushed
the
fix/16271-16278-session-path-agentid
branch
2 times, most recently
from
February 14, 2026 16:44
14df9c2 to
55d9345
Compare
gumadeiras
force-pushed
the
fix/16271-16278-session-path-agentid
branch
3 times, most recently
from
February 14, 2026 18:44
258ac8d to
fa117c4
Compare
… paths Sub-agent and implicit-main-agent sessions failed with 'Session file path must be within sessions directory' because resolveSessionFilePath was called without agentId in several reply dispatch paths. This caused transcript paths to resolve against the main agent's directory instead of the correct agent's directory. Fixes openclaw#16271 Fixes openclaw#16278
gumadeiras
force-pushed
the
fix/16271-16278-session-path-agentid
branch
from
February 14, 2026 18:44
fa117c4 to
7cbe9de
Compare
Member
|
Merged via squash. Thanks @robbyczgw-cla! |
akoscz
pushed a commit
to akoscz/openclaw
that referenced
this pull request
Feb 15, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
GwonHyeok
pushed a commit
to learners-superpumped/openclaw
that referenced
this pull request
Feb 15, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
vincentkoc
pushed a commit
to vincentkoc/openclaw
that referenced
this pull request
Feb 15, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
This was referenced Feb 18, 2026
lovewanwan
pushed a commit
to lovewanwan/openclaw
that referenced
this pull request
Apr 28, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
ogt-redknie
pushed a commit
to ogt-redknie/OPENX
that referenced
this pull request
May 2, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 9, 2026
…penclaw#16288) Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 7cbe9de Co-authored-by: robbyczgw-cla <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #16271 and #16278 — transcript path resolution used wrong agent context in multi-agent setups.
Root cause
resolveSessionFilePath()was called without explicitagentIdin several dispatch paths. When agent context was missing or mismatched, resolution fell back to the wrong sessions directory and failed withSession file path must be within sessions directory.Changes
Pass
agentIdthrough session-file resolution call sites:src/gateway/server-methods/chat.tssrc/gateway/session-utils.fs.tssrc/agents/subagent-announce.tssrc/agents/tools/sessions-list-tool.tssrc/config/sessions/transcript.tsHarden
resolveSessionFilePathOptions():src/config/sessions/paths.ts— if bothstorePathandagentIdare provided and the resolved dir looks like a default path for a different agent, prefer the explicitagentIdImpact
AI-assisted (Claude). Reviewed by human.