-
-
Notifications
You must be signed in to change notification settings - Fork 652
飞书回复新增思考过程,https://github.com/AstrBotDevs/AstrBot/issues/1415 #1421
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Original review guide in EnglishReviewer's Guide by SourceryThis pull request implements a feature for Lark platform integration to improve user experience during LLM processing. It introduces a 'thinking' message that is sent immediately after receiving a user's query and is updated with the final LLM response once available. This is achieved by sending an initial interactive card message and subsequently updating it. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
嘿 @mio4kon - 我已经查看了你的更改 - 这里有一些反馈:
总体评论:
- 仅使用提取的文本在更新的卡片中发送最终响应可能无法完全代表最初包含图像或其他富媒体的回复。
以下是我在审查期间查看的内容
- 🟡 一般问题:发现 1 个问题
- 🟢 安全性:一切看起来都很好
- 🟢 测试:一切看起来都很好
- 🟢 复杂性:一切看起来都很好
- 🟢 文档:一切看起来都很好
帮助我更有用!请点击每个评论上的 👍 或 👎,我将使用反馈来改进你的评论。
Original comment in English
Hey @mio4kon - I've reviewed your changes - here's some feedback:
Overall Comments:
- Sending the final response within an updated card using only extracted text might not fully represent replies originally containing images or other rich media.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个更改是否会导致在开启了分段回复或者插件用这个 Event 发送多条消息的情况下(即触发多次 send())会一直编辑同一条消息?
3c3d4ce
to
58f6c84
Compare
解决了 #1415
Motivation
提高飞书交互体验。
requirements.txt
和pyproject.toml
文件相应位置。好的,这是翻译成中文的 pull request 总结:
Sourcery 总结
通过在 Lark (飞书) 上显示“思考中”指示器来改善用户体验,以提示正在处理请求。
新功能:
Original summary in English
Summary by Sourcery
Improve the user experience on Lark (Feishu) by showing a "thinking" indicator while processing requests.
New Features: