PR #64820: fix(feishu): break circular module init causing ReferenceError on group mention

• Repository: openclaw/openclaw
• Author: Ricardo-M-L
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/64820

Description (problem / solution / changelog)

Summary

Fixes #64783

When mentioning (@) a bot in a Feishu group chat with multi-agent configuration, the gateway throws:

ReferenceError: Cannot access 'utils_1' before initialization

Root cause: monitor.account.ts statically imports handleFeishuMessage and parseFeishuMessageEvent from bot.ts, which pulls in a deep transitive dependency tree (bot-content -> post -> comment-shared -> plugin-sdk/text-runtime, plus policy, reply-dispatcher, dedup, etc.). When tsdown/esbuild bundles these into a single output chunk, the shared utility bindings can violate the Temporal Dead Zone — the bundler evaluates one module scope before the utility variable it references has been initialized.

Fix (3 changes):

• Extract parseFeishuMessageEvent into a new lightweight message-parser.ts module that only depends on shallow modules (bot-content, mention, event-types, types) — no heavy transitive deps
• Lazy-load handleFeishuMessage and handleFeishuCardAction in monitor.account.ts via dynamic import() instead of static imports, deferring evaluation of bot.ts and card-action.ts until the first event is dispatched
• Update sequential-key.ts to import from message-parser.ts instead of bot.ts
• bot.ts re-exports parseFeishuMessageEvent for full backward compatibility

Test plan

• Added message-parser.test.ts with:
  • Unit tests for parseFeishuMessageEvent (basic parsing, bot mention detection, mention forward detection, sender fields, thread IDs)
  • Backward compatibility test verifying bot.ts re-export is the same function reference
  • Module isolation tests verifying message-parser.ts does not import from bot.ts, and monitor.account.ts uses dynamic import for bot.ts
• Existing tests (bot.checkBotMentioned.test.ts, bot.stripBotMention.test.ts, sequential-key.test.ts, etc.) continue to pass since bot.ts re-exports the function
• Manual test: configure multi-agent Feishu setup, mention a bot in a group chat — no more ReferenceError

🤖 Generated with Claude Code

Changed files

• extensions/feishu/src/bot.ts (modified, +2/-54)
• extensions/feishu/src/message-parser.test.ts (added, +203/-0)
• extensions/feishu/src/message-parser.ts (added, +87/-0)
• extensions/feishu/src/monitor.account.ts (modified, +45/-12)
• extensions/feishu/src/sequential-key.ts (modified, +2/-1)