feat(feishu): extract embedded video/media from post (rich text) messages by laopuhuluwa · Pull Request #21786 · openclaw/openclaw

laopuhuluwa · 2026-02-20T11:05:16Z

Summary

Feishu post (rich text) messages can contain embedded videos and audio via <media> tags, but parsePostContent() only extracted <img> tags. This meant embedded videos in rich text posts were silently dropped — the agent never received them.

Problem

When a user sends a rich text message with an embedded video (common in Feishu — the editor allows dragging videos into posts), the message is received as message_type: 'post' with content like:

{"content": [[{"tag": "media", "file_key": "file_v3_xxx", "file_name": "video.mov"}]]}

The existing parsePostContent() only looked for img tags, so this media was ignored.

Changes

`parsePostContent()`

Now also extracts media tags (embedded video/audio) with their file_key and optional file_name
Returns new mediaKeys: Array<{ fileKey: string; fileName?: string }> alongside existing imageKeys

`resolveFeishuMediaList()`

After downloading embedded images, now also downloads embedded media files from posts
Uses messageResource API with type: 'file' for media downloads
Adds logging for media discovery and download

Testing

Build passes (pnpm build)
Lint passes (pnpm lint, 0 warnings 0 errors)
Tested with real Feishu post messages containing embedded .MOV video files

Greptile Summary

Adds support for extracting embedded video/audio files from Feishu rich text (post) messages.

Previously, parsePostContent() only extracted <img> tags, silently dropping <media> tags containing videos/audio. This change:

Extends parsePostContent() to collect both image keys and media keys (file_key + optional file_name) from post content
Updates resolveFeishuMediaList() to download embedded media files using the messageResource API with type: 'file'
Adds appropriate logging for media discovery and downloads

The implementation follows the existing pattern for embedded images and correctly uses the messageResource API for downloading files from messages.

Confidence Score: 4/5

Safe to merge with one minor type inconsistency
The implementation correctly mirrors the existing embedded image handling pattern and properly uses the Feishu messageResource API. The change is well-tested according to the PR description. One minor type inconsistency exists where fileName is added to the media info object but not defined in the FeishuMediaInfo type - however, this doesn't affect runtime behavior since the property is not used downstream.
No files require special attention

_{Last reviewed commit: c3ef915}

_{(3/5) Reply to the agent's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!}

greptile-apps

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-02-20T11:08:51Z

extensions/feishu/src/bot.ts

+        out.push({
+          path: saved.path,
+          contentType: saved.contentType,
+          fileName: media.fileName,


fileName property not defined in FeishuMediaInfo type (see types.ts:62-66)

Suggested change

fileName: media.fileName,

fileName: media.fileName,

Prompt To Fix With AI

This is a comment left during a code review. Path: extensions/feishu/src/bot.ts Line: 442 Comment: `fileName` property not defined in `FeishuMediaInfo` type (see `types.ts:62-66`) ```suggestion fileName: media.fileName, ``` How can I resolve this? If you propose a fix, please make it concise.

laopuhuluwa · 2026-02-20T11:19:44Z

Updated: also added "media" to the allowed message types whitelist and parseMediaKeys() switch. Feishu uses message_type: "media" (not "video") for video files sent as standalone messages, which caused them to be silently dropped.

openclaw-barnacle · 2026-02-28T04:10:03Z

This pull request has been automatically marked as stale due to inactivity.
Please add updates or it will be closed.

…ages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download

Takhoffman · 2026-02-28T05:39:45Z

PR #21786 - feat(feishu): extract embedded video/media from post (rich text) messages (#21786)

Merged via squash.

Merge commit: 53a2e72
Verified:
- pnpm vitest run --config vitest.extensions.config.ts extensions/feishu/src/bot.test.ts
- pnpm build
- pnpm check
- pnpm test:macmini
Changes made:
- extensions/feishu/src/post.ts: parse rich-text post payload media tags and return normalized embedded media keys.
- extensions/feishu/src/bot.ts: download embedded post media via messageResource with type: "file" and include them in inbound media payload.
- extensions/feishu/src/bot.test.ts: add regression coverage for embedded post media download path.
- CHANGELOG.md: add Feishu post embedded-media note for feat(feishu): extract embedded video/media from post (rich text) messages #21786.
Why these changes were made:
- Main already had rich-text markdown parsing and embedded image extraction; embedded media tags were still dropped for post messages. This keeps behavior focused and merge-safe by adding only the missing media extraction/download path with tests.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @laopuhuluwa!

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit 7adb044)

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit 53a2e72) # Conflicts: # CHANGELOG.md

…ages (openclaw#21786) Cherry-pick of upstream 53a2e72.

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit 53a2e72) # Conflicts: # CHANGELOG.md

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

…ages (openclaw#21786) Cherry-pick of upstream 53a2e72.

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

…ages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit 53a2e72)

…test fix (#1481) * feat(feishu): extract embedded video/media from post (rich text) messages (openclaw#21786) * feat(feishu): extract embedded video/media from post (rich text) messages Previously, parsePostContent() only extracted embedded images (img tags) from rich text posts, ignoring embedded video/audio (media tags). Users sending post messages with embedded videos would not have the media downloaded or forwarded to the agent. Changes: - Extend parsePostContent() to also collect media tags with file_key - Return new mediaKeys array alongside existing imageKeys - Update resolveFeishuMediaList() to download embedded media files from post messages using the messageResource API - Add appropriate logging for embedded media discovery and download * Feishu: keep embedded post media payloads type-safe * Feishu: format post parser after media tag extraction --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit 53a2e72) * Feishu: fix locale-wrapper post parser test (openclaw#29576) (cherry picked from commit f810932) --------- Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: laopuhuluwa <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> Co-authored-by: Vincent Koc <[email protected]>

openclaw-barnacle bot added channel: feishu Channel integration: feishu size: S labels Feb 20, 2026

greptile-apps bot reviewed Feb 20, 2026

View reviewed changes

laopuhuluwa force-pushed the feat/feishu-post-embedded-media branch from c3ef915 to 378e873 Compare February 20, 2026 11:19

laopuhuluwa force-pushed the feat/feishu-post-embedded-media branch from 378e873 to 3617e25 Compare February 20, 2026 11:22

openclaw-barnacle bot added the stale Marked as stale due to inactivity label Feb 28, 2026

Takhoffman force-pushed the feat/feishu-post-embedded-media branch from 3617e25 to 19c61bc Compare February 28, 2026 05:36

laopuhuluwa and others added 3 commits February 27, 2026 23:38

Feishu: keep embedded post media payloads type-safe

a1d83ac

Feishu: format post parser after media tag extraction

08e3bfe

Takhoffman force-pushed the feat/feishu-post-embedded-media branch from 19c61bc to 08e3bfe Compare February 28, 2026 05:39

Takhoffman merged commit 53a2e72 into openclaw:main Feb 28, 2026
3 checks passed

github-actions bot mentioned this pull request Feb 28, 2026

📡 Upstream Digest — 2026-02-28 06:28 UTC curtismercier/openclaw-mods#143

Open

greptile-apps bot mentioned this pull request Feb 28, 2026

fix(unittest): add missing property to failing unit test #29574

Closed

18 tasks

Takhoffman mentioned this pull request Feb 28, 2026

feat(feishu): comprehensive enhancements for Feishu channel #8975

Closed

github-actions bot mentioned this pull request Mar 1, 2026

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

Closed

6 tasks

gemini-code-assist bot mentioned this pull request Mar 1, 2026

chore(sync): replay rollup fixes on upstream/main MillionthOdin16/openclaw#111

Merged

zooqueen added a commit to hanzoai/bot that referenced this pull request Mar 1, 2026

feat(feishu): extract embedded video/media from post (rich text) mess…

ffdfcc2

…ages (openclaw#21786) Cherry-pick of upstream 53a2e72.

zooqueen added a commit to hanzoai/bot that referenced this pull request Mar 6, 2026

feat(feishu): extract embedded video/media from post (rich text) mess…

035d6f2

…ages (openclaw#21786) Cherry-pick of upstream 53a2e72.

alexey-pelykh mentioned this pull request Mar 10, 2026

Cherry-pick: Feishu adapter — community PRs and fixes remoteclaw/remoteclaw#678

Closed

alexey-pelykh mentioned this pull request Mar 15, 2026

Cherry-pick 53a2e72fc: feat(feishu): extract embedded video/media from post messages (#21786) remoteclaw/remoteclaw#1479

Closed

alexey-pelykh mentioned this pull request Mar 16, 2026

Cherry-pick 53a2e72fc+f81093285: feishu post video extraction + locale test fix remoteclaw/remoteclaw#1481

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(feishu): extract embedded video/media from post (rich text) messages#21786

feat(feishu): extract embedded video/media from post (rich text) messages#21786
Takhoffman merged 3 commits intoopenclaw:mainfrom
laopuhuluwa:feat/feishu-post-embedded-media

laopuhuluwa commented Feb 20, 2026 •

edited by greptile-apps bot

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot Feb 20, 2026

Uh oh!

laopuhuluwa commented Feb 20, 2026

Uh oh!

openclaw-barnacle bot commented Feb 28, 2026

Uh oh!

Uh oh!

Takhoffman commented Feb 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

laopuhuluwa commented Feb 20, 2026 • edited by greptile-apps bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Problem

Changes

parsePostContent()

resolveFeishuMediaList()

Testing

Greptile Summary

Confidence Score: 4/5

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

laopuhuluwa commented Feb 20, 2026

Uh oh!

openclaw-barnacle bot commented Feb 28, 2026

Uh oh!

Uh oh!

Takhoffman commented Feb 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

laopuhuluwa commented Feb 20, 2026 •

edited by greptile-apps bot

Loading

`parsePostContent()`

`resolveFeishuMediaList()`