[PR #184] [CLOSED] Add show and hide reason for thinking tag and clean no used component #191

Closed
opened 2026-02-16 10:16:23 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/webapp-conversation/pull/184
Author: @Eric-Guo
Created: 9/16/2025
Status: Closed

Base: mainHead: main


📝 Commits (5)

  • c787d77 Remove no need react-tooltip and react-markdown
  • 14a8a56 Add show and hide reason for thinking tag.
  • 06ce697 Extract ThinkTag, we still need segments way.
  • 816bd79 Implement internationalization for ThinkTag component with translations for show/hide reasoning in multiple languages.
  • 20f7412 pnpm upgrade

📊 Changes

13 files changed (+737 additions, -1307 deletions)

View changed files

.cursorignore (+6 -0)
app/components/base/markdown.tsx (+0 -45)
📝 app/components/base/streamdown-markdown.tsx (+61 -2)
app/components/base/think-tag/index.tsx (+29 -0)
📝 i18n/lang/common.en.ts (+4 -0)
📝 i18n/lang/common.es.ts (+4 -0)
📝 i18n/lang/common.fr.ts (+4 -0)
📝 i18n/lang/common.ja.ts (+4 -0)
📝 i18n/lang/common.vi.ts (+4 -0)
📝 i18n/lang/common.zh.ts (+4 -0)
📝 package.json (+40 -47)
📝 pnpm-lock.yaml (+573 -1213)
pnpm-workspace.yaml (+4 -0)

📄 Description

Screen

image

Done list

  • Remove no used react-markdown and react-tooltip
  • Add think tag support for Streamdown (it's can not support by simply define as component, (or I'm wrong) unless change Streamdown source code.

commits

  • Add show and hide reason for thinking tag.
  • Extract ThinkTag, we still need segments way.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/webapp-conversation/pull/184 **Author:** [@Eric-Guo](https://github.com/Eric-Guo) **Created:** 9/16/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (5) - [`c787d77`](https://github.com/langgenius/webapp-conversation/commit/c787d77bb2c71930906f996f570483057cba232f) Remove no need react-tooltip and react-markdown - [`14a8a56`](https://github.com/langgenius/webapp-conversation/commit/14a8a564df65682230a4efcd344d62bb03cbca27) Add show and hide reason for thinking tag. - [`06ce697`](https://github.com/langgenius/webapp-conversation/commit/06ce697db1d652dffc387776fe76334705ae5dfb) Extract ThinkTag, we still need segments way. - [`816bd79`](https://github.com/langgenius/webapp-conversation/commit/816bd79a6873f8001d8e6b1bf2052d3e4b71ee97) Implement internationalization for ThinkTag component with translations for show/hide reasoning in multiple languages. - [`20f7412`](https://github.com/langgenius/webapp-conversation/commit/20f7412d5db9bbd18e575b67023139b726036492) pnpm upgrade ### 📊 Changes **13 files changed** (+737 additions, -1307 deletions) <details> <summary>View changed files</summary> ➕ `.cursorignore` (+6 -0) ➖ `app/components/base/markdown.tsx` (+0 -45) 📝 `app/components/base/streamdown-markdown.tsx` (+61 -2) ➕ `app/components/base/think-tag/index.tsx` (+29 -0) 📝 `i18n/lang/common.en.ts` (+4 -0) 📝 `i18n/lang/common.es.ts` (+4 -0) 📝 `i18n/lang/common.fr.ts` (+4 -0) 📝 `i18n/lang/common.ja.ts` (+4 -0) 📝 `i18n/lang/common.vi.ts` (+4 -0) 📝 `i18n/lang/common.zh.ts` (+4 -0) 📝 `package.json` (+40 -47) 📝 `pnpm-lock.yaml` (+573 -1213) ➕ `pnpm-workspace.yaml` (+4 -0) </details> ### 📄 Description ## Screen <img width="1097" height="721" alt="image" src="https://github.com/user-attachments/assets/a9f6240a-0d25-44eb-a09b-0c2b56bb59d7" /> ## Done list - Remove no used `react-markdown` and `react-tooltip` - Add `think` tag support for `Streamdown` (it's can not support by simply define as <think> component, ~~(or I'm wrong)~~ unless change `Streamdown` source code. ## commits - **Add show and hide reason for thinking tag.** - **Extract ThinkTag, we still need segments way.** --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 10:16:23 -05:00
yindo closed this issue 2026-02-16 10:16:23 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/webapp-conversation#191