Skip to content

Handle active Abyss Shadows map on Shenshe entry#1641

Merged
runhey merged 1 commit into
runhey:devfrom
leovefy:codex/cobra-03-abyss-shadows-map
Jun 27, 2026
Merged

Handle active Abyss Shadows map on Shenshe entry#1641
runhey merged 1 commit into
runhey:devfrom
leovefy:codex/cobra-03-abyss-shadows-map

Conversation

@leovefy

@leovefy leovefy commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Split from leovefy/cobra.

This PR contains a single commit: 1bb9350.
Base: runhey/dev.

Summary by Sourcery

在进入神社时处理已经处于激活状态的深渊暗影运行,并调整深渊暗影地图界面的检测方式。

Bug 修复:

  • 在进入神社后,如果深渊暗影导航界面已处于激活状态,则进行检测并提前短路以避免重复操作。

增强:

  • 扩展深渊暗影地图的检测区域,以更稳健地识别地图处于激活状态的情况。
Original summary in English

Summary by Sourcery

Handle the case where entering Shenshe resumes an already active Abyss Shadows run and adjust detection of the Abyss Shadows map UI.

Bug Fixes:

  • Detect and short-circuit when the Abyss Shadows navigation UI is already active after entering Shenshe to avoid redundant actions.

Enhancements:

  • Expand the Abyss Shadows map detection region to more robustly recognize the active map state.

@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.

嗨——我已经 review 了你的改动,看起来非常棒!


Sourcery 对开源项目是免费的——如果你喜欢我们的 review,欢迎分享 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的 review。
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.

@runhey runhey merged commit 9ce255c into runhey:dev Jun 27, 2026
9 of 11 checks passed
@leovefy leovefy deleted the codex/cobra-03-abyss-shadows-map branch June 27, 2026 17:22
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