Greptile Summary

Fixed token-wasting double-reply bug in Feishu integration where permission errors triggered two separate agent turns. The fix consolidates both the permission error notification and the user message into a single agent turn by appending the error notice directly to messageBody before dispatch.

Key changes:

• Removed separate permission error dispatch logic (52 lines)
• Appended permission error notice to main message body using consistent [System: ...] format
• Single dispatch now handles both user message and permission error notification
• Preserved cooldown mechanism to prevent error notification spam

Code quality:

• Clean, minimal change that directly addresses [Bug]: Feishu channel may send duplicate replies when sender-name permission lookup fails (99991672) #27372
• Well-commented with clear rationale
• Follows existing patterns for system notices
• Reduces complexity and improves maintainability

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change is simple, focused, and well-documented. It removes 52 lines of duplicate logic, follows existing code patterns, and directly addresses the reported issue without introducing side effects or breaking changes. The fix maintains all existing functionality while eliminating wasteful double-dispatch behavior.
• No files require special attention

_{Last reviewed commit: 2a6a2ee}

Landed via temp rebase onto main.

• Gate: pnpm test extensions/feishu/src/bot.test.ts (pass)
• Full gate: blocked by unrelated existing failure in pnpm check (src/telegram/bot-message-context.test-harness.ts missing sendChatActionHandler)
• Land commit: 6a54fc9
• Merge commit: cf4853e

Thanks @byungsker!