Skip to content

fix(customs): 对齐 ExpressionRecognition 注册方式与 README 格式 - #9

Merged
kqcoxn merged 1 commit into
MaaXYZ:mainfrom
overflow65537:fix/expression-recognition-docs
Jun 28, 2026
Merged

fix(customs): 对齐 ExpressionRecognition 注册方式与 README 格式#9
kqcoxn merged 1 commit into
MaaXYZ:mainfrom
overflow65537:fix/expression-recognition-docs

Conversation

@overflow65537

@overflow65537 overflow65537 commented Jun 28, 2026

Copy link
Copy Markdown
Member

Summary by Sourcery

使 ExpressionRecognition 自定义识别器的注册方式与文档中描述的用法保持一致,并更新 README 以匹配当前的管道配置和表达式语义。

增强内容:

  • 使用 AgentServer 的 custom_recognition 装饰器,直接在类上注册 ExpressionRecognition,以符合预期用法。

文档更新:

  • 重写 ExpressionRecognition 的 README,重点放在面向使用场景的示例上,澄清表达式语法和参数,并使注册方式与管道示例对齐,以匹配当前实现。
Original summary in English

Summary by Sourcery

Align the ExpressionRecognition custom recognizer’s registration with the documented usage and refresh the README to match the current pipeline configuration and expression semantics.

Enhancements:

  • Register ExpressionRecognition directly on the class using the AgentServer custom_recognition decorator to match expected usage.

Documentation:

  • Rewrite the ExpressionRecognition README to focus on usage-oriented examples, clarify expression syntax and parameters, and align registration and pipeline samples with the current implementation.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗨——我已经审查了你的改动,一切看起来都很棒!


Sourcery 对开源项目是免费的——如果你喜欢我们的代码审查,请考虑分享 ✨
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈来改进代码审查。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@kqcoxn
kqcoxn merged commit 0d3d8c8 into MaaXYZ:main Jun 28, 2026
3 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