fix: merge PR #1327 #1319 #1318 #1313#1559
Closed
xuwei-xy wants to merge 11 commits intosipeed:mainfrom
Closed
Conversation
When users send text+image together in Feishu, the message type is "post" (rich text) instead of "image". The image download logic only handled MsgTypeImage, so images embedded in post messages were silently ignored — the agent received raw JSON instead of actual image content. - Add extractPostImageKeys() to parse img tags from post content - Add extractPostText() to extract plain text from post content - Handle MsgTypePost in downloadInboundMedia to download embedded images - Handle MsgTypePost in appendMediaTags to add [image: photo] tag
When the message tool sends to a different chat_id (e.g. a newly created Feishu group), the final reply to the original chat was incorrectly suppressed. Track the target chat_id in MessageTool and only skip the outbound reply when it matches the inbound message's chat_id.
…sion Replace single sentChatID (atomic.Value) with a set of chat_ids so that when the message tool sends to multiple chats in one round, the final reply is correctly suppressed for any chat that was already sent to.
…and fix some lint errors
…covery The Lark SDK v3's built-in token retry loop does not clear stale tokens from cache when the server returns error 99991663 (tenant_access_token invalid), causing all API calls to fail until the token naturally expires (~2 hours). - Add tokenCache struct (implementing larkcore.Cache) with Get/Set/InvalidateAll methods and proper expired-entry cleanup - Wire custom cache into lark.NewClient via WithTokenCache() - Add invalidateTokenOnAuthError helper called in all API methods
|
OpenClaw-User seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
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
5 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
Merges fixes from open PRs #1327 #1319 #1318 #1313