fix(feishu): keep quoted DMs out of topic routing

[JingZed]

Summary

Fix Feishu quoted replies in direct messages being routed as topic/thread messages and disappearing from the main conversation.

Feishu sets root_id on a quoted/replied DM to identify the root of the reply chain. The adapter previously promoted that value to source.thread_id whenever a real message.thread_id was absent. Downstream reply routing then treated an ordinary DM quote chain as a topic/thread and anchored the bot response to the quoted message instead of the new triggering message.

Root cause

thread_id = message.thread_id or message.root_id

In a p2p chat, root_id is reply-chain context, not topic routing metadata. It already remains available through reply_to_message_id; it should not also become source.thread_id.

Fix

• In Feishu p2p chats, populate normalized source.thread_id only from an actual message.thread_id.
• Preserve the existing root_id fallback for non-p2p chats, because Feishu group topics may use it as routing context.
• Preserve root_id as the reply_to_message_id fallback in all chats.
• Add regression coverage that verifies:
  • a quoted DM has no normalized thread when message.thread_id is absent;
  • the quote root remains available as reply context;
  • the outbound reply anchor resolves to the current triggering message;
  • group-topic root_id routing remains unchanged.

Reproduction

On current main, the DM regression test fails with:

AssertionError: 'om_quoted_root' is not None

The normalized DM event incorrectly receives source.thread_id = root_id.

Test plan

• Verified the new DM regression test fails against the old adapter behavior.
• scripts/run_tests.sh tests/gateway/test_feishu.py -q
  • 78 passed, 0 failed
• ruff check plugins/platforms/feishu/adapter.py tests/gateway/test_feishu.py
• Python bytecode compilation for both changed files
• git diff --check
• Live Feishu validation: quoting another message now produces a visible bot reply in the main DM conversation.

[alt-glitch]

This was generated by AI during triage.

Duplicate of #30164: both preserve root_id for non-p2p routing while preventing a Feishu DM quote root from becoming thread metadata.

[JingZed]

Confirmed — thanks for the pointer. I compared the production change and regression coverage directly: #30164 already implements the same p2p-only root_id guard while preserving non-p2p topic routing and reply context. Closing this PR as a duplicate in favor of #30164.

[JingZed]

Closing as duplicate of #30164.