Skip to content

fix:修复线基建奖励和信用点商店bug#1750

Merged
MistEO merged 8 commits intov2from
fix/dr
Mar 30, 2026
Merged

fix:修复线基建奖励和信用点商店bug#1750
MistEO merged 8 commits intov2from
fix/dr

Conversation

@overflow65537
Copy link
Copy Markdown
Member

@overflow65537 overflow65537 commented Mar 30, 2026

Summary by Sourcery

修复积分商店解析和地脊奖励配置相关问题。

Bug 修复:

  • 通过新增单独的 Priority2 和 Priority3 配置,处理无法负担的物品在积分商店 OCR 解析中的问题。
  • 修正地脊奖励中孵化室(growth chamber)及相关场景/任务配置,以正确发放线路基础设施类奖励。

增强改进:

  • 更新积分购物和地脊奖励的资源流水线及任务定义,使其与修正后的行为保持一致。
Original summary in English

Summary by Sourcery

Fix credit shop parsing and Dijiang reward configuration issues.

Bug Fixes:

  • Handle credit shop OCR parsing for items that cannot be afforded by adding separate Priority2 and Priority3 configurations.
  • Correct DijiangRewards growth chamber and related scene/task configurations for line infrastructure rewards.

Enhancements:

  • Update credit shopping and Dijiang rewards resource pipelines and task definitions to align with the corrected behaviors.
- 更正 DijiangRewards 任务配置中的线路基础设施奖励数据。
Original summary in English

Summary by Sourcery

修复积分商店解析和地脊奖励配置相关问题。

Bug 修复:

  • 通过新增单独的 Priority2 和 Priority3 配置,处理无法负担的物品在积分商店 OCR 解析中的问题。
  • 修正地脊奖励中孵化室(growth chamber)及相关场景/任务配置,以正确发放线路基础设施类奖励。

增强改进:

  • 更新积分购物和地脊奖励的资源流水线及任务定义,使其与修正后的行为保持一致。
Original summary in English

Summary by Sourcery

Fix credit shop parsing and Dijiang reward configuration issues.

Bug Fixes:

  • Handle credit shop OCR parsing for items that cannot be afforded by adding separate Priority2 and Priority3 configurations.
  • Correct DijiangRewards growth chamber and related scene/task configurations for line infrastructure rewards.

Enhancements:

  • Update credit shopping and Dijiang rewards resource pipelines and task definitions to align with the corrected behaviors.

@overflow65537 overflow65537 linked an issue Mar 30, 2026 that may be closed by this pull request
@overflow65537 overflow65537 linked an issue Mar 30, 2026 that may be closed by this pull request
@overflow65537 overflow65537 linked an issue Mar 30, 2026 that may be closed by this pull request
@overflow65537 overflow65537 changed the title fix:修复线基建奖励bug fix:修复线基建奖励和信用点商店bug Mar 30, 2026
@overflow65537 overflow65537 marked this pull request as ready for review March 30, 2026 14:32
Copilot AI review requested due to automatic review settings March 30, 2026 14:32
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.

Hey - 我给出了一些整体性的反馈:

  • 新增的 Priority2OCR_CanNotAffordPriority3OCR_CanNotAfford 键与普通 OCR 条目使用了相同的 expected 值;建议将其重构为一个辅助函数或共享常量,以避免重复并让配置映射更易于维护。
给 AI Agent 的提示
Please address the comments from this code review:

## Overall Comments
- The new `Priority2OCR_CanNotAfford` and `Priority3OCR_CanNotAfford` keys duplicate the same `expected` value as the normal OCR entries; consider refactoring this into a helper or shared constant to avoid repetition and keep the configuration map more maintainable.

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

Hey - I've left some high level feedback:

  • The new Priority2OCR_CanNotAfford and Priority3OCR_CanNotAfford keys duplicate the same expected value as the normal OCR entries; consider refactoring this into a helper or shared constant to avoid repetition and keep the configuration map more maintainable.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The new `Priority2OCR_CanNotAfford` and `Priority3OCR_CanNotAfford` keys duplicate the same `expected` value as the normal OCR entries; consider refactoring this into a helper or shared constant to avoid repetition and keep the configuration map more maintainable.

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 旨在修复帝江线基建奖励流程与信用点商店(CreditShopping)相关的识别/购买逻辑问题,并同步更新界面选项文案与后端解析覆盖参数,以提升自动化流程的稳定性与可配置性。

Changes:

  • 调整 DijiangRewards 任务中的线索库存阈值识别规则,并清理部分培养舱种子检查的覆盖配置。
  • 重构 CreditShopping 的优先级购买节点命名与执行顺序,补齐“买不起”场景下 Priority2/3 的识别与折扣匹配链路。
  • 为 CreditShopping 新增/更新多语言选项描述,并在 go-service 中扩展 pipeline override 覆盖到新增的 *_CanNotAfford OCR 节点。

Reviewed changes

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

Show a summary per file
File Description
assets/tasks/DijiangRewards.json 更新线索库存计数 expected 正则;移除一段培养舱种子非空检查的 pipeline_override。
assets/tasks/CreditShopping.json Priority1 选项增加 description;将开关覆盖节点从 BuyFirst 重命名为 BuyPriority1;更新折扣识别覆盖并补齐 Priority2/3 “买不起”分支覆盖。
assets/resource/pipeline/SceneManager/SceneDijiang.json 在进入总控中枢入口动作前增加 pre_wait_freezes 等待静止。
assets/resource/pipeline/DijiangRewards/GrowthChamber.json 调整“再次种植”按钮 OCR 的 ROI 区域。
assets/resource/pipeline/CreditShopping/Shopping.json 将主购买节点改为 CreditShoppingBuyPriority1;调整 next 顺序(ReserveCredit 前置到 Priority2/3 之前);新增 AutoGetCreditsBuyPriority2/3 识别组合。
assets/resource/pipeline/CreditShopping/Item.json 新增 Priority2/3 的 *_CanNotAfford OCR 与对应折扣匹配节点。
assets/locales/interface/zh_tw.json 更新 CreditShopping Priority 文案,并新增 description 文案键。
assets/locales/interface/zh_cn.json 更新 CreditShopping Priority 文案,并新增 description 文案键。
assets/locales/interface/ko_kr.json 更新 CreditShopping Priority 文案,并新增 description 文案键。
assets/locales/interface/ja_jp.json 更新 CreditShopping Priority 文案,并新增 description 文案键。
assets/locales/interface/en_us.json 更新 CreditShopping Priority 文案,并新增 description 文案键。
agent/go-service/creditshopping/creditshopping.go pipeline override 覆盖范围扩展到 Priority2/3 的 *_CanNotAfford OCR 节点。

@MistEO MistEO merged commit 32712ad into v2 Mar 30, 2026
18 checks passed
@MistEO MistEO deleted the fix/dr branch March 30, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants