Skip to content

fix: register IM read tools when api.config is undefined (#415)#430

Open
Hamzaa6296 wants to merge 1 commit into
larksuite:mainfrom
Hamzaa6296:fix/im-tools-not-registering
Open

fix: register IM read tools when api.config is undefined (#415)#430
Hamzaa6296 wants to merge 1 commit into
larksuite:mainfrom
Hamzaa6296:fix/im-tools-not-registering

Conversation

@Hamzaa6296
Copy link
Copy Markdown

Fixes #415

The registerGetMessages function in src/tools/oapi/im/message-read.js
returned false silently when api.config was undefined, preventing
feishu_im_user_get_messages, feishu_im_user_get_thread_messages, and
feishu_im_user_search_messages from being registered.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 18, 2026

CLA assistant check
All committers have signed the CLA.

@Hamzaa6296
Copy link
Copy Markdown
Author

Hi team!
After digging into the source, I traced the root cause to src/tools/oapi/im/message-read.js — the registerGetMessages function has a silent early return when api.config is undefined:
This causes feishu_im_user_get_messages, feishu_im_user_get_thread_messages, and feishu_im_user_search_messages to be skipped with no error or warning in the logs, while all other tool groups register fine.
I've submitted a PR with this fix. Happy to adjust if the maintainers prefer a different approach
Thanks for the great plugin!

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.

feishu_im_* tools not registered (api.config is undefined in register callback)

2 participants