Skip to content

fix(GameUi): 兼容商店两种招牌图片,修复反复进出商店的问题 - #1977

Merged
runhey merged 1 commit into
runhey:devfrom
leovefy:dev
Oct 29, 2026
Merged

runhey merged 1 commit into
runhey:devfrom
leovefy:dev

Conversation

@leovefy

@leovefy leovefy commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

紫色招牌无法命中旧商店模板,导致任务反复进出商店并触发连续点击保护。保留旧图并新增紫色模板,统一进入和返回商店的识别,匹配阈值保持 0.8。

离线验证:

  • .\toolkit\python.exe -B -X utf8 -m unittest discover -s tests -p test_mall_appearance.py -v:6 项通过,覆盖两种外观、导航返回和超时退出。
  • .\toolkit\python.exe -B -X utf8 -m unittest discover -s tests -p test_hero_test_skill_fallback.py -v:7 项通过。
  • 6 张现场截图全部通过,匹配分数 0.9868~1.0000;旧外观使用原模板构造画面验证。

Sourcery 总结

支持商场招牌的两种外观,并提升进出商场导航的稳定性。

新功能:

  • 支持识别旧版和紫色两种商场招牌外观。

错误修复:

  • 通过统一的页面导航处理返回商场的流程,避免重复尝试进入和离开商场。

改进:

  • 在 DailyTrifles 和 RichMan 导航流程中使用共享的商场页面检测逻辑。

测试:

  • 添加离线测试,覆盖商场招牌的两种外观、导航返回路径以及超时失败处理。
Original summary in English

Summary by Sourcery

Support both mall sign appearances and stabilize navigation into and out of the mall.

New Features:

  • Support recognition of both legacy and purple mall sign appearances.

Bug Fixes:

  • Prevent repeated mall entry and exit attempts by using unified page navigation for mall returns.

Enhancements:

  • Use shared mall page detection for DailyTrifles and RichMan navigation flows.

Tests:

  • Add offline coverage for both mall appearances, navigation return paths, and timeout failure handling.

@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 评估

已批准。


Sourcery 为开源项目免费提供服务——如果你喜欢我们的评审,欢迎分享 ✨
Original comment in English

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

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@runhey
runhey merged commit 65311c5 into runhey:dev Oct 29, 2026
6 of 8 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