Skip to content

fix(maa): 活动关优先不再清零普通理智作战的理智药额度 - #421

Merged
1w1w11w1 merged 1 commit into
AUTO-MAS-Project:devfrom
1w1w11w1:codex/maa-activity-first-medicine
Aug 26, 2026
Merged

fix(maa): 活动关优先不再清零普通理智作战的理智药额度#421
1w1w11w1 merged 1 commit into
AUTO-MAS-Project:devfrom
1w1w11w1:codex/maa-activity-first-medicine

Conversation

@1w1w11w1

@1w1w11w1 1w1w11w1 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

变更摘要

  • 移除开启活动关优先后对普通理智作战理智药额度的静默清零,活动关优先任务与理智作战各自使用独立的理智药额度(Task.ActivityMedicineNumb 与计划表 MedicineNumb)。
  • 保留活动关优先、库存保持、普通理智、剩余理智的分段执行语义,不合并为单个 FightTask。

Closes #420

Summary by Sourcery

修复活动关优先模式下普通理智作战理智药额度被清零的问题。

Bug Fixes:

  • 保留活动关优先任务的理智药额度,不再因启用活动关优先而静默清零普通理智作战的理智药额度。

Enhancements:

  • 将活动关优先与普通理智作战配置为相互独立的执行任务,并抽取活动关优先任务构建逻辑。
Original summary in English

Summary by Sourcery

修复活动关优先模式下普通理智作战理智药额度被清零的问题。

Bug Fixes:

  • 保留活动关优先任务的理智药额度,不再因启用活动关优先而静默清零普通理智作战的理智药额度。

Enhancements:

  • 将活动关优先与普通理智作战配置为相互独立的执行任务,并抽取活动关优先任务构建逻辑。

@sourcery-ai

sourcery-ai Bot commented Aug 26, 2026

Copy link
Copy Markdown

审查者指南

修复启用活动关优先时,普通理智作战的理智药额度被静默清零的问题:活动关优先任务继续使用 Task.ActivityMedicineNumb,普通理智作战保留计划表中的 MedicineNumb,并通过辅助函数独立生成活动任务配置。

文件级变更

变更 详情 文件
将活动关优先任务构建逻辑抽取为独立辅助函数,并让其与普通理智作战分别保留各自的理智药额度。
  • 新增活动关优先 FightTask 构建函数,集中设置活动关优先的阶段、掉落和药剂配置。
  • 移除对普通 FightTask 的 UseMedicine、MedicineCount 及相关过期药配置的强制清零。
  • 保持活动关优先与普通理智作战分段执行、分别消耗各自额度的语义。
app/task/MAA/AutoProxy.py
更新应用版本信息以反映本次修复。
  • 调整版本号配置。
res/version.json

可能关联的问题


提示与命令

与 Sourcery 交互

  • 触发新的审查: 在 pull request 中评论 @sourcery-ai review
  • 继续讨论: 直接回复 Sourcery 的审查评论。
  • 根据审查评论生成 GitHub issue: 回复审查评论,请求 Sourcery 根据该评论创建 issue。你也可以使用 @sourcery-ai issue 回复审查评论,以根据该评论创建 issue。
  • 生成 pull request 标题: 在 pull request 标题的任意位置写入 @sourcery-ai,即可随时生成标题。你也可以在 pull request 中评论 @sourcery-ai title,以随时生成或重新生成标题。
  • 生成 pull request 摘要: 在 pull request 正文的任意位置写入 @sourcery-ai summary,即可在指定位置随时生成 PR 摘要。你也可以在 pull request 中评论 @sourcery-ai summary,以随时生成或重新生成摘要。
  • 生成审查者指南: 在 pull request 中评论 @sourcery-ai guide,即可随时生成或重新生成审查者指南。
  • 解决所有 Sourcery 评论: 在 pull request 中评论 @sourcery-ai resolve,即可解决所有 Sourcery 评论。如果你已经处理完所有评论且不想再看到它们,这会非常有用。
  • 忽略所有 Sourcery 审查: 在 pull request 中评论 @sourcery-ai dismiss,即可忽略所有现有的 Sourcery 审查。如果你希望重新开始一次新的审查,这尤其有用——别忘了评论 @sourcery-ai review 来触发新的审查!

自定义使用体验

访问你的控制面板以:

  • 启用或禁用审查功能,例如 Sourcery 生成的 pull request 摘要、审查者指南等。
  • 更改审查语言。
  • 添加、移除或编辑自定义审查说明。
  • 调整其他审查设置。

获取帮助

Original review guide in English

Reviewer's Guide

修复开启活动关优先时普通理智作战理智药额度被静默清零的问题:活动关优先任务继续使用 Task.ActivityMedicineNumb,普通理智作战保留计划表 MedicineNumb,并通过辅助函数独立生成活动任务配置。

File-Level Changes

Change Details Files
将活动关优先任务构建逻辑抽取为独立辅助函数,并让其与普通理智作战保留各自的理智药额度。
  • 新增活动关优先 FightTask 构建函数,集中设置活动关优先的阶段、掉落和药剂配置。
  • 移除对普通 FightTask 的 UseMedicine、MedicineCount 及相关过期药配置的强制清零。
  • 保持活动关优先与普通理智作战分段执行、分别消耗各自额度的语义。
app/task/MAA/AutoProxy.py
更新应用版本信息以反映本次修复。
  • 调整版本号配置。
res/version.json

Possibly linked issues


Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@1w1w11w1
1w1w11w1 merged commit 874b6cd into AUTO-MAS-Project:dev Aug 26, 2026
2 checks passed
@1w1w11w1
1w1w11w1 deleted the codex/maa-activity-first-medicine branch August 26, 2026 06:20

@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

Needs a human reviewer. If the change is wrong, the generated ordinary-fight task can consume sanity medicines using its configured quota in addition to the activity-priority task. Those consumable items are depleted and reverting the code cannot restore them, although the amount is bounded by the configured quotas and inventory.


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.

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.

1 participant