Greptile Summary

This PR fixes a false-negative in Feishu bot mention detection by removing the strict display-name equality guard (botName check) from checkBotMentioned, relying solely on the stable open_id to identify the bot. This correctly handles cases where Feishu surfaces a different alias or app-context display name for the same bot identity.

Key changes:

• checkBotMentioned now only uses open_id for matching — the botName && m.name !== botName guard is removed.
• botName parameter removed from checkBotMentioned and parseFeishuMessageEvent signatures.
• botName is no longer destructured in handleFeishuMessage (line 865), but remains in the params type declaration (line 860) as a leftover that should be cleaned up.
• A regression test is added covering the alias display-name mismatch case.

The logic change is correct and well-tested. The only remaining issue is the dead botName parameter in the public API type that misleads callers.

Confidence Score: 4/5

• Safe to merge — the core fix is correct and well-tested. The dead botName parameter is a minor API surface issue that does not impact runtime behavior.
• This PR safely fixes the reported issue by removing the strict display-name check from bot mention detection. The logic change is correct and is backed by a new regression test. The only finding is a minor cleanup: the unused botName parameter in the type declaration (line 860) should be removed to avoid misleading callers. This is a style/API clarity issue rather than a functional problem, so the score reflects high confidence in the core changes with a minor remaining cleanup.
• extensions/feishu/src/bot.ts line 860 — Remove unused botName parameter from the handleFeishuMessage type declaration.

_{Last reviewed commit: e6a3cab}

Follow-up done per review suggestion:

• removed the unused botName field from handleFeishuMessage param type in extensions/feishu/src/bot.ts

Validation:

• pnpm -s vitest run extensions/feishu/src/bot.checkBotMentioned.test.ts
• Result: 15/15 passing

Commit: 7adf8c6.