fix: pass agentDir to /compact command for agent-specific auth#24133

Merged

gumadeiras merged 3 commits intoopenclaw:mainfrom

Glucksberg:fix/issue-23812

Feb 23, 2026

Contributor

Glucksberg commented Feb 23, 2026 •

edited by greptile-apps bot

Loading

Summary

The /compact command was not passing the agentDir parameter when calling compactEmbeddedPiSession(), causing it to ignore agent-specific authentication profiles
Added agentDir?: string to HandleCommandsParams interface and threaded it through handleCommands() and commands-compact.ts

Test plan

TypeScript compilation passes
All 40 command tests pass
Verified agentDir flows from inline actions through to compaction call

Fixes #23812

🤖 Generated with Claude Code

Greptile Summary

Threaded agentDir parameter through the /compact command flow to enable agent-specific authentication profiles during compaction. The change adds the optional agentDir field to HandleCommandsParams, passes it from handleInlineActions to handleCommands, and finally includes it when calling compactEmbeddedPiSession() in the compact command handler.

Confidence Score: 5/5

This PR is safe to merge with minimal risk
The change is a straightforward parameter threading fix with minimal code surface area - adding an optional field to an interface and passing it through two function calls. The parameter was already being used correctly in the target function (compactEmbeddedPiSession), it just wasn't being passed through. Type safety is maintained throughout, and the PR description indicates tests pass.
No files require special attention

_{Last reviewed commit: 4ae1e1a}

openclaw-barnacle bot added size: XS experienced-contributor labels

gumadeiras self-assigned this

gumadeiras force-pushed the fix/issue-23812 branch 2 times, most recently from f146aa9 to d4c6f1b Compare

February 23, 2026 07:15

openclaw-barnacle bot added size: S and removed size: XS labels

gumadeiras force-pushed the fix/issue-23812 branch from e8a3a17 to 1cf266f Compare

February 23, 2026 07:25

Glucksberg and others added 3 commits

February 23, 2026 02:32


          fix: pass agentDir to /compact command for agent-specific auth

19d1fd3

The /compact command was not forwarding agentDir to
compactEmbeddedPiSession, causing it to fall back to the default
"main" agent directory for authentication. This broke compaction
for non-default agents that have their own auth-profiles.json.

Fixes openclaw#23812

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>


          test: cover /compact agentDir forwarding

086380f


          test: assert inline commands forward agentDir

4bb10ca

gumadeiras force-pushed the fix/issue-23812 branch from 1cf266f to 4bb10ca Compare

February 23, 2026 07:32

gumadeiras merged commit 36400df into openclaw:main

13 checks passed

Member

gumadeiras commented Feb 23, 2026

Merged via squash.

Prepared head SHA: 4bb10ca
Merge commit: 36400df

Thanks @Glucksberg!

github-actions bot mentioned this pull request

📡 Upstream Digest — 2026-02-23 08:39 UTC curtismercier/openclaw-mods#108

Open

vabole mentioned this pull request

ops: decision needed for upstream sync (3 disputed files) vabole/openclaw#3

Merged

3 tasks

obviyus pushed a commit to jd316/openclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

d3771ea

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

jaydiamond42 pushed a commit to jaydiamond42/bloomtbot that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

f4b4fc6

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

carlosrivera pushed a commit to myascendai/meshiclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

3ee8c35

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

gabrielkoo pushed a commit to gabrielkoo/openclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

81c4ce7

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

mreedr pushed a commit to mreedr/openclaw-custom that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

7b80096

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

margulans pushed a commit to margulans/Neiron-AI-assistant that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

cf15990

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

brianleach pushed a commit to brianleach/openclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

c1bf955

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

mylukin pushed a commit to mylukin/openclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

64b0806

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

r4jiv007 pushed a commit to r4jiv007/openclaw that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

feb4fc7

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

github-actions bot mentioned this pull request

cherry-pick: upstream bugfix commits (2026-03-01-0443) hughdidit/DAISy-Agency#140

Closed

6 tasks

hughdidit pushed a commit to hughdidit/DAISy-Agency that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

3e135e9

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

(cherry picked from commit 36400df)

# Conflicts:
#	CHANGELOG.md
#	src/auto-reply/reply/commands.test.ts
#	src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts

hughdidit pushed a commit to hughdidit/DAISy-Agency that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

d3633e8

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

(cherry picked from commit 36400df)

# Conflicts:
#	CHANGELOG.md
#	src/auto-reply/reply/commands.test.ts
#	src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts

zooqueen pushed a commit to hanzoai/bot that referenced this pull request


          fix: pass agentDir to /compact command for agent-specific auth (openc…

509c8c7

…law#24133)

Merged via /review-pr -> /prepare-pr -> /merge-pr.

Prepared head SHA: 4bb10ca
Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: gumadeiras <5599352+gumadeiras@users.noreply.github.com>
Reviewed-by: @gumadeiras

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

Labels