Skip to content

fix(offload): recover after-tool-call messages#249

Open
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/after-tool-call-message-recovery
Open

fix(offload): recover after-tool-call messages#249
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/after-tool-call-message-recovery

Conversation

@YOMXXX

@YOMXXX YOMXXX commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • recover after_tool_call conversation messages from compatible hook context locations when event.messages is missing or empty
  • backfill event.messages before patch-effectiveness detection so existing MMD injection and L3 compression continue through the same path
  • add focused coverage for event, ctx, history, and session-param message sources

Verification

  • npm test
  • npm run build

Fixes #118
Related: #184

Signed-off-by: 李冠辰 <liguanchen@xiaomi.com>
@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

Awesome work! 13 fixes in one batch covering Gateway hardening, embedding compatibility, Hermes memory injection, offload stability, and pipeline optimization — a very productive cleanup session 👏

We'll review and merge them one by one 🚀

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.

after_tool_call hook: event.messages undefined, L3 compression broken in OpenClaw 5.27

2 participants