Skip to content

fix: 修复稳健刷新模式中,覆盖内容不全导致判定为直接命中#1676

Merged
MistEO merged 1 commit intov2from
fix/creditshopping-prudent-refresh
Mar 28, 2026
Merged

fix: 修复稳健刷新模式中,覆盖内容不全导致判定为直接命中#1676
MistEO merged 1 commit intov2from
fix/creditshopping-prudent-refresh

Conversation

@overflow65537
Copy link
Copy Markdown
Member

@overflow65537 overflow65537 commented Mar 28, 2026

Summary by Sourcery

增强功能:

  • 将表达式求值日志提取到一个可复用的辅助函数中,以在表达式识别服务中实现日志记录的集中化和标准化。
Original summary in English

Summary by Sourcery

Enhancements:

  • Extract expression evaluation logging into a reusable helper function to centralize and standardize logging in the expression recognition service.
- 重构表达式识别的日志记录方式,将评估日志的构建逻辑提取到一个专用的辅助函数中。
Original summary in English

Summary by Sourcery

增强功能:

  • 将表达式求值日志提取到一个可复用的辅助函数中,以在表达式识别服务中实现日志记录的集中化和标准化。
Original summary in English

Summary by Sourcery

Enhancements:

  • Extract expression evaluation logging into a reusable helper function to centralize and standardize logging in the expression recognition service.

@overflow65537 overflow65537 linked an issue Mar 28, 2026 that may be closed by this pull request
@overflow65537 overflow65537 marked this pull request as ready for review March 28, 2026 12:07
Copilot AI review requested due to automatic review settings March 28, 2026 12:07
@overflow65537 overflow65537 marked this pull request as draft March 28, 2026 12:07
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

该 PR 旨在修复“稳健刷新模式”下由于 pipeline/task 覆盖内容不完整导致识别被判定为直接命中(DirectHit)的问题,同时对 ExpressionRecognition 的评估日志输出做了小幅重构,便于统一维护。

Changes:

  • assets/tasks/CreditShopping.json 中补全 CreditShoppingPrudentRefreshrecognition 覆盖:显式指定 type: And、补全 Custom(ExpressionRecognition) 定义,并设置合适的 box_index
  • assets/resource/pipeline/CreditShopping/Shopping.json 中将稳健刷新相关的自定义识别切换为 ExpressionRecognition(与表达式配置一致)。
  • agent/go-service/expressionrecognition/recognition.go 中将表达式评估结果的日志构建提取到独立辅助函数 logEvaluationResult

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
assets/tasks/CreditShopping.json 补全稳健刷新阈值的 pipeline_override 识别配置,避免覆盖不全导致 DirectHit,并补充 box_index。
assets/resource/pipeline/CreditShopping/Shopping.json 将稳健刷新判断用的 custom_recognition 调整为 ExpressionRecognition,以正确评估表达式条件。
agent/go-service/expressionrecognition/recognition.go 抽取表达式评估日志到 helper,降低 Run 函数内的日志样板代码。

@overflow65537 overflow65537 marked this pull request as ready for review March 28, 2026 13:15
@MistEO MistEO merged commit 72cd3ab into v2 Mar 28, 2026
22 checks passed
@MistEO MistEO deleted the fix/creditshopping-prudent-refresh branch March 28, 2026 14:37
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.

信用点购物开启稳健刷新后出现问题

3 participants