fix(feishu): handle message_type "media" for video downloads#25502
Merged
Takhoffman merged 2 commits intoopenclaw:mainfrom Feb 28, 2026
Merged
fix(feishu): handle message_type "media" for video downloads#25502Takhoffman merged 2 commits intoopenclaw:mainfrom
Takhoffman merged 2 commits intoopenclaw:mainfrom
Conversation
Feishu mobile app sends video messages with message_type="media" instead of "video". Add "media" as a fallback case in three locations: 1. resolveFeishuMediaList() - mediaTypes array 2. parseMediaKeys() - switch/case for file_key extraction 3. inferPlaceholder() - switch/case for placeholder text Fixes openclaw#25499
4c73e33 to
d4055c8
Compare
Contributor
|
PR #25502 - fix(feishu): handle message_type "media" for video downloads (#25502) Merged via squash.
Thanks @4ier! |
newtontech
pushed a commit
to newtontech/openclaw-fork
that referenced
this pull request
Feb 28, 2026
6 tasks
hughdidit
pushed a commit
to hughdidit/DAISy-Agency
that referenced
this pull request
Mar 1, 2026
…w#25502) thanks @4ier Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: 4ier <5648066+4ier@users.noreply.github.com> Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com> (cherry picked from commit e4cb6a8) # Conflicts: # CHANGELOG.md # extensions/feishu/src/bot.ts
robertchang-ga
pushed a commit
to robertchang-ga/openclaw
that referenced
this pull request
Mar 2, 2026
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
Feishu mobile app sends video messages with
message_type: "media"instead of"video". The plugin currently only checks for"video", causing video messages to be silently dropped — the agent receives raw JSON metadata instead of the downloaded video file.Changes
Add
"media"as a fallback case alongside"video"in three locations inextensions/feishu/src/bot.ts:resolveFeishuMediaList()— add"media"tomediaTypesarrayparseMediaKeys()— addcase "media":to extractfile_keyandimage_keyinferPlaceholder()— addcase "media":to return"<media:video>"Test Report
Environment: OpenClaw v2026.2.23, @openclaw/feishu v2026.2.23, Feishu mobile (Android)
{"file_key":"...","file_name":"...mp4","duration":15016}msgType=media(not in mediaTypes → skipped)msgType=media→ download triggeredReproduction steps:
Fixes #25499
Greptile Summary
Adds support for Feishu mobile app's
message_type: "media"for video messages. The mobile app uses"media"instead of"video", causing videos to be dropped silently. The fix adds"media"as a fallback alongside"video"in three functions (resolveFeishuMediaList,parseMediaKeys,inferPlaceholder), treating it identically to"video"messages with bothfile_keyandimage_key.The implementation is straightforward and follows existing patterns:
mediaTypesarray inresolveFeishuMediaList(line 349)parseMediaKeysswitch statement (line 246)inferPlaceholderswitch statement (line 324)The changes are minimal, consistent, and preserve backward compatibility with existing
"video"messages.Confidence Score: 5/5
"video"messages, uses case fallthrough correctly, and has been tested by the author. The fix addresses a real user-facing bug where video messages from Feishu mobile were being dropped. No breaking changes, no risky refactoring, and the implementation is consistent across all three modified locations.Last reviewed commit: 4c73e33
(3/5) Reply to the agent's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!