Skip to content

fix: Add async message handler to prevent IM platform timeout#372

Closed
mjnhmd wants to merge 8 commits intonetease-youdao:mainfrom
mjnhmd:fix/im-timeout-only
Closed

fix: Add async message handler to prevent IM platform timeout#372
mjnhmd wants to merge 8 commits intonetease-youdao:mainfrom
mjnhmd:fix/im-timeout-only

Conversation

@mjnhmd
Copy link
Copy Markdown

@mjnhmd mjnhmd commented Mar 11, 2026

Summary

  • Add async message handler that processes messages in background and sends notifications
  • Add isQuickQuery detection to distinguish simple queries from long-running tasks
  • Add progress update interval (30s) for tasks exceeding timeout threshold
  • Add getCurrentProgress and getSessionIdByConversation methods for progress tracking
  • For quick queries (greetings, short messages), use sync mode for immediate response

Test plan

  • Test IM bot with short queries (should respond immediately)
  • Test IM bot with long-running tasks (should receive progress updates every 30s)

🤖 Generated with Claude Code

zhouzheng and others added 8 commits March 10, 2026 10:25
The PolyBool union was returning area > 100% due to coordinate issues.
Added intersect with bounds and clamp the result to valid range.

Co-Authored-By: Claude <noreply@anthropic.com>
The previous CDN link used wrong package name 'polybool' instead of 'polybooljs'

Co-Authored-By: Claude <noreply@anthropic.com>
The PolyBool.intersect with bounds was causing incorrect area calculation.
Now just union the zones and clamp the result to 0-100 range.

Co-Authored-By: Claude <noreply@anthropic.com>
…atform timeout

- Add async message handler that processes messages in background and sends notifications
- Add isQuickQuery detection to distinguish simple queries from long-running tasks
- Add progress update interval (30s) for tasks exceeding timeout threshold
- Add getCurrentProgress and getSessionIdByConversation methods for progress tracking
- For quick queries (greetings, short messages), use sync mode for immediate response

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant