Skip to content

fix(feishu): prevent [99992402] field validation failed in thread replies - #65484

Closed
leowzz wants to merge 6 commits into
NousResearch:mainfrom
leowzz:feature/leo/feishu-reply-in-thread
Closed

fix(feishu): prevent [99992402] field validation failed in thread replies#65484
leowzz wants to merge 6 commits into
NousResearch:mainfrom
leowzz:feature/leo/feishu-reply-in-thread

Conversation

@leowzz

@leowzz leowzz commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • add feishu.reply_in_thread so replies to top-level Feishu/Lark messages can start a topic
  • preserve the existing default (false) for backward compatibility while keeping replies inside existing topics
  • fix an issue where jobs created inside a Feishu thread could deliver to the parent chat or fail delivery, causing users to miss the job response in the originating thread
  • avoid unsupported receive_id_type=thread_id create requests, which Feishu rejects with 99992402 field validation failed
  • preserve the triggering Feishu message ID when a cron job is created inside a topic, then use it as reply_to_message_id for both live and standalone delivery
  • only fall back to the parent chat when a topic delivery genuinely no longer has a valid reply anchor
  • avoid treating Feishu SDK response objects as dictionaries during live-delivery confirmation
  • document the setting in the English and Chinese Feishu guides

Testing

  • scripts/run_tests.sh tests/cron/test_scheduler.py tests/tools/test_cronjob_tools.py tests/tools/test_send_message_tool.py tests/gateway/test_feishu.py tests/gateway/test_stream_consumer_thread_routing.py tests/gateway/test_run_progress_topics.py -q --tb=short
  • 707 tests passed
  • git diff --check

@alt-glitch alt-glitch added type/feature New feature or request comp/plugins Plugin system and bundled plugins platform/feishu Feishu / Lark adapter P3 Low — cosmetic, nice to have sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades labels Jul 16, 2026
@leowzz leowzz changed the title feat(feishu): support configurable replies in topics fix(feishu): prevent [99992402] field validation failed in thread replies Jul 16, 2026
@leowzz

leowzz commented Jul 16, 2026

Copy link
Copy Markdown
Author

This mixed feature/fix PR has been split into two focused PRs based on the latest main:

Closing this superseded PR to keep review scope and commit history focused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have platform/feishu Feishu / Lark adapter sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants