fix(msteams): suppress reasoning-only text in outbound rendering#25511
fix(msteams): suppress reasoning-only text in outbound rendering#25511chilu18 wants to merge 1 commit intoopenclaw:mainfrom
Conversation
|
This pull request has been automatically marked as stale due to inactivity. |
|
Hi @chilu18 — thanks for the submission. I’m the new Microsoft Teams maintainer for OpenClaw. Please give me a day or two to work through the open Teams backlog. Also, join the Twitter community for daily MS Teams feedback + updates: https://x.com/i/communities/2031170403607974228 |
1 similar comment
|
Hi @chilu18 — thanks for the submission. I’m the new Microsoft Teams maintainer for OpenClaw. Please give me a day or two to work through the open Teams backlog. Also, join the Twitter community for daily MS Teams feedback + updates: https://x.com/i/communities/2031170403607974228 |
|
Closing this stale Microsoft-tracker item for cleanup. If this is still an issue or still worth pursuing, please re-open it. We now have dedicated Microsoft maintainers watching this area. |
Summary
Reasoning:blocks and think-tag payloads)extensions/msteams/src/messenger.test.tsWhy
MS Teams does not have a dedicated reasoning lane. When reasoning text is emitted as channel payload text, users can see internal thinking output instead of user-facing answers. This aligns MS Teams behavior with other channels that suppress reasoning-only output.
Tests
./node_modules/.bin/vitest extensions/msteams/src/messenger.test.tsFixes #25458
Greptile Summary
Adds reasoning-only text suppression to MS Teams channel rendering, preventing internal thinking traces from leaking to users. Follows the established pattern from Matrix (1298bd4) and Discord (e8a4d5d) channels.
isReasoningOnlyText()helper that detectsReasoning:\nprefix and thinking tagsConfidence Score: 5/5
Last reviewed commit: 591470c