Skip to content

fix: disable empty reasoning expansion#98

Merged
weareoutman merged 1 commit into
mainfrom
fix/empty-reasoning-rendering
Jun 23, 2026
Merged

fix: disable empty reasoning expansion#98
weareoutman merged 1 commit into
mainfrom
fix/empty-reasoning-rendering

Conversation

@easyops-eve

Copy link
Copy Markdown

Summary

Elevo Messenger now treats vip.elevo.reasoning.empty: true as an explicit empty reasoning marker. For these fallback reasoning messages, the UI shows only the duration summary and does not offer the expand interaction.

Implementation Plan

  • Add a small reasoning metadata helper that preserves legacy empty-body behavior.
  • Extend Matrix reasoning metadata parsing with empty?: boolean.
  • Use the helper in MsgTypeRenderers so explicit empty fallback messages are non-interactive even when the Matrix body contains Thought for N seconds.
  • Add focused tests for explicit empty metadata, legacy non-empty reasoning, and legacy empty-body reasoning.

Verification

  • npm test -- src/app/components/message/MsgTypeRenderers.test.ts --run
  • npm test -- --run
  • npm run build
  • git diff --check

npm run typecheck is still blocked by existing default-branch issues unrelated to this change: html-dom-parser package export typings and existing TUploadContent / Blob.name errors.

Co-authored-by: Codex Agent <codex-agent@elevo.vip>
@github-actions

Copy link
Copy Markdown

🚀 Deployed on https://pr-98--elevo-messenger.netlify.app

@github-actions
github-actions Bot temporarily deployed to pull request June 23, 2026 09:50 Inactive
@weareoutman
weareoutman merged commit 58a235b into main Jun 23, 2026
2 checks passed
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.

2 participants