feat(feishu): add support for merge_forward message parsing by tsu-builds · Pull Request #28707 · openclaw/openclaw

tsu-builds · 2026-02-27T13:00:18Z

OpenClaw PR: Support Feishu merge_forward message parsing

Summary

This PR adds support for parsing Feishu merge_forward (合并转发) messages. Previously, these messages would only show a placeholder text "Merged and Forwarded Message" without the actual content.

Problem

When a user sends a merged/forwarded message in Feishu:

The websocket event's message.content field only contains the placeholder string "Merged and Forwarded Message"
The actual sub-messages are not included in the websocket payload
This makes it impossible for the agent to see the forwarded content

Solution

Detect merge_forward message type in handleFeishuMessage()
Call the Feishu im.message.get API with the message_id
The API returns all sub-messages in the items array, with upper_message_id pointing to the merge_forward container
Parse and format the sub-messages into readable text

Changes

`extensions/feishu/src/bot.ts`

Modified parseMessageContent(): Return a loading placeholder for merge_forward type
Added parseMergeForwardContent(): Async function to parse the API response and format sub-messages
Added formatSubMessageContent(): Helper to format individual sub-message content based on type
Modified handleFeishuMessage(): Added logic to fetch and expand merge_forward messages via API

Example Output

Before:

[Merged and Forwarded Message]

After:

[Merged and Forwarded Messages]
- First message content
- Second message content
- [Image]
- Third message content
... and 5 more messages

Testing

Tested with real Feishu merge_forward messages containing:

Text messages
Rich text (post) messages
Images
Nested replies

Notes

Sub-messages are limited to 50 to prevent excessive API calls
Nested merge_forward messages show as [Nested Merged Forward] (no recursive expansion)
The API call adds slight latency (~200-500ms) but is necessary since websocket doesn't include sub-message content

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e03338e18d

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

chatgpt-codex-connector · 2026-02-27T13:05:27Z

extensions/feishu/src/bot.ts

+  }
+
+  // Filter to only sub-messages (those with upper_message_id, skip the merge_forward container itself)
+  const subMessages = items.filter((item) => item.upper_message_id);


Restrict merge-forward expansion to direct child items

The current filter keeps every item with a truthy upper_message_id, which means nested descendants are treated as top-level entries whenever Feishu returns recursive items for nested merge_forward content. In that case, grandchildren from nested forwards are expanded as regular bullets (despite the "[Nested Merged Forward]" placeholder intent), producing duplicated or out-of-scope content; this should filter by the current container message id instead of any non-empty parent id.

Useful? React with 👍 / 👎.

greptile-apps · 2026-02-27T13:06:03Z

Greptile Summary

Added support for parsing Feishu merge_forward (合并转发) messages by fetching full message content via API and expanding sub-messages into readable text.

Key changes:

Added parseMessageContent() placeholder for merge_forward type
Added parseMergeForwardContent() to fetch and format sub-messages from API response (limits to 50 messages)
Added formatSubMessageContent() to handle different message types (text, post, image, file, etc.)
Modified handleFeishuMessage() to detect merge_forward messages and fetch full content via client.im.message.get()

Issues found:

Logic bug in API response handling - see inline comment for fix

Confidence Score: 4/5

Safe to merge with one minor logic fix in error handling
Found one logic bug in line 682 where empty items array produces misleading error message, but this is a non-critical edge case. The feature implementation is sound with proper error handling, type definitions match existing patterns, and the API integration follows established conventions from send.ts
Line 682 in extensions/feishu/src/bot.ts needs the length > 0 check removed

_{Last reviewed commit: e03338e}

greptile-apps

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-02-27T13:06:07Z

extensions/feishu/src/bot.ts

+        path: { message_id: event.message.message_id },
+      })) as { code?: number; data?: { items?: unknown[] } };
+
+      if (response.code === 0 && response.data?.items && response.data.items.length > 0) {


Remove length > 0 check - causes misleading "could not fetch" error when API returns empty items array. parseMergeForwardContent already handles empty arrays correctly (returns "no sub-messages").

Suggested change

if (response.code === 0 && response.data?.items && response.data.items.length > 0) {

if (response.code === 0 && response.data?.items) {

Prompt To Fix With AI

This is a comment left during a code review. Path: extensions/feishu/src/bot.ts Line: 682 Comment: Remove `length > 0` check - causes misleading "could not fetch" error when API returns empty items array. `parseMergeForwardContent` already handles empty arrays correctly (returns "no sub-messages"). ```suggestion if (response.code === 0 && response.data?.items) { ``` How can I resolve this? If you propose a fix, please make it concise.

- Detect merge_forward message type in handleFeishuMessage() - Fetch sub-messages via im.message.get API (websocket event doesn't include them) - Parse and format sub-messages into readable text - Support text, post, image, file, audio, video, sticker message types - Limit to 50 sub-messages to prevent excessive API calls - Handle nested merge_forward as placeholder (no recursive expansion) Fixes: Feishu merged/forwarded messages now display actual content instead of just 'Merged and Forwarded Message' placeholder.

Takhoffman · 2026-02-28T02:57:43Z

PR #28707 - feat(feishu): add support for merge_forward message parsing (#28707)

Merged via squash.

Merge commit: f53ef73
Verified: pnpm build, pnpm check, pnpm test:macmini
Changes made:
M CHANGELOG.md
M extensions/feishu/src/bot.test.ts
M extensions/feishu/src/bot.ts
Why these changes were made:
Deep review found the feature direction was sound, but merge-forward parsing needed explicit regression coverage and small implementation hardening before landing.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @tsu-builds!

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e23aeca69f

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

chatgpt-codex-connector · 2026-02-28T03:01:30Z

extensions/feishu/src/bot.ts

+  }
+
+  // Filter to only sub-messages (those with upper_message_id, skip the merge_forward container itself)
+  const subMessages = items.filter((item) => item.upper_message_id);


Filter merge-forward items by container message ID

items.filter((item) => item.upper_message_id) treats every descendant as a top-level sub-message, so when Feishu returns recursive items (for nested merged forwards or nested replies) grandchildren get flattened into this message body and can duplicate/out-of-scope content. This breaks the intended [Nested Merged Forward] boundary and can feed unrelated nested text to the agent; restrict the expansion to items whose upper_message_id matches the current merge-forward container message ID.

Useful? React with 👍 / 👎.

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit ed8e6f7)

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit ed8e6f7)

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit ed8e6f7)

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit f53ef73)

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit f53ef73)

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

@tsu-builds

…#28707) thanks @tsu-builds Verified: - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]> (cherry picked from commit f53ef73)

@tsu-builds

…#28707) thanks @tsu-builds (#1461) Verified: - pnpm build - pnpm check - pnpm test:macmini (cherry picked from commit f53ef73) Co-authored-by: tsu-builds <[email protected]> Co-authored-by: tsu-builds <[email protected]> Co-authored-by: Tak Hoffman <[email protected]>

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

chatgpt-codex-connector bot reviewed Feb 27, 2026

View reviewed changes

greptile-apps bot reviewed Feb 27, 2026

View reviewed changes

Takhoffman added a commit to tsu-builds/openclaw that referenced this pull request Feb 28, 2026

fix(feishu): harden merge_forward parsing coverage for openclaw#28707

09da56d

openclaw-barnacle bot added size: M and removed size: S labels Feb 28, 2026

tsu-builds and others added 2 commits February 27, 2026 20:42

fix(feishu): harden merge_forward parsing coverage for openclaw#28707

e23aeca

Takhoffman force-pushed the feat/feishu-merge-forward-support branch from 09da56d to e23aeca Compare February 28, 2026 02:56

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

chatgpt-codex-connector bot reviewed Feb 28, 2026

View reviewed changes

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

fix(failover): trigger failover for network_error stop reason #29437

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

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 f53ef73a2: feat(feishu): add support for merge_forward message parsing (open remoteclaw/remoteclaw#1461

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(feishu): add support for merge_forward message parsing#28707

feat(feishu): add support for merge_forward message parsing#28707
Takhoffman merged 2 commits intoopenclaw:mainfrom
tsu-builds:feat/feishu-merge-forward-support

tsu-builds commented Feb 27, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Feb 27, 2026

Uh oh!

greptile-apps bot commented Feb 27, 2026

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot Feb 27, 2026

Uh oh!

Uh oh!

Takhoffman commented Feb 28, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	if (response.code === 0 && response.data?.items && response.data.items.length > 0) {
	if (response.code === 0 && response.data?.items) {

Uh oh!

Conversation

tsu-builds commented Feb 27, 2026

OpenClaw PR: Support Feishu merge_forward message parsing

Summary

Problem

Solution

Changes

extensions/feishu/src/bot.ts

Example Output

Testing

Notes

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot commented Feb 27, 2026

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 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Takhoffman commented Feb 28, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

`extensions/feishu/src/bot.ts`