Conversation
- 统计信息分支不再自己判渠道、直调传输层，改为 dispatch + statistic_targets： 补上系统通知、Koishi、微信（iLink）、QQ 官方机器人四类渠道，并按全局 + 用户 目标发送；同时修复会向已禁用的自定义 Webhook 推送（中央 _webhooks 过滤 Info.Enabled，原先遍历 MultipleConfig.values() 未过滤） - 代理结果分支改用 notify_core.push_proxy_result，删掉内联重复实现 - 渠道正文分流保持原样：邮件 HTML 完整版；ServerChan 完整版、超 30KB 回退简略版； 聊天机器人 Webhook 简略版。_signed() 只承载这两处渠道级覆盖 - 前端通知区改用共享组件 UserNotifyConfig，与其余 11 个专项一致 - notify_core 收敛名单补上 BetterGI
- 数据源为 BGI「奖励识别」的逐轮日志（自动秘境/自动首领讨伐：本轮奖励识别结果 …）， 新增 drop_statistics 解析模块，按物品跨轮、跨来源累加，日志文案属上游对外契约 - 用户级开关 Notify.IfSendDropStatistics 默认开启；开启后运行前强制打开识别： 写全局 autoDomainConfig.rewardRecognitionEnabled（秘境 Param 的 SetDefault() 从该段 拷贝，一条龙与执行层两条路径全覆盖）+ 槽位 AutoBossRewardRecognitionEnabled（首领 原生路径）+ Plan.settings（首领执行层）；该叶子已在运行时快照集合内，结束自动还原 - 统计通知正文与 general_statistics.html 新增「掉落物品 / 数量」两列表格（条件渲染， 不影响共用该模板的其它专项）；聊天机器人 Webhook 仍用简略版，不塞表 - 前端复用共享组件 UserNotifyConfig 的开关（与 MAA 六星同款 showSixStar 模式），补三语词条 - 补单测 tests/task/test_bettergi_drop_statistics.py（5 例）
BGI 把「本轮奖励识别结果」的明细段整体包在一对 ASCII 双引号里（实测 2026-09-14 日志，首领那行的来源名也带引号），解析时未剥引号，导致： - 每行第一条物品名带上前导引号，通知里会显示成 `"好感经验`； - 同一物品处于首位与非首位时被拆成两个键（`"好感经验` 与 `好感经验`）， 数量合不到一起、统计偏小。 改为对解析出的物品名统一剥掉两端引号类字符；不把「」计入剥离集合，避免破坏 `「久雨莲」的种子` 这类以「开头的名字。测试采样行同步改为实测的带引号格式， 并补充「首条剥引号」「跨来源同名合并」「方括号不被误剥」三条用例。
按 tests/AGENTS.md「功能边界或 bug 边界的测试不提交」，撤下上一提交新增的三条 回归用例（首条剥引号、跨来源同名合并、方括号不被误剥），本地验证结果记入 PR 正文。 采样行仍保留实测的带引号格式：原样例是无引号的臆测格式，不改则测试验证的是 现实中并不存在的输入。
- main.js 新增战斗步骤必填项前置校验：缺 bossName 时不进 BGI，改打 MAS_STEP_MISSING_CONFIG 标记并跳过该步，避免落回 AutoBossParam 无参构造 读取 BGI 全局 autoBossConfig 的旧首领 - AutoProxy 识别该标记判负，并把「自动首领讨伐：未选择首领」推到调度台与日志
- main.js 四项战斗步骤的队伍改为无条件赋值：空即明确传空串，由 BGI 侧 「空 = 不切换队伍」的原生守卫生效；此前用 if (x) 守卫跳过赋值，会留下 Param 构造时 SetDefault() 从全局 config.json 读来的旧队伍 - 地脉花好感队随战斗队伍一起清空，避免空战斗队伍下仍切好感队 - 通用战斗策略留空时解析为「根据队伍自动选择」并写入全局叶子（新增 autoBossConfig.strategyName，补齐首领执行层无通用策略来源的缺口）， 不再跳过写入而让 BGI 沿用配置里的旧策略
- main.js 地脉花在 Param 无 setCombatStrategyPath 的版本改直接写 FightConfig.StrategyName，使右栏单独为地脉花选择的策略真正生效 （此前只打 MAS_LEYLINE_STRATEGY_UNSUPPORTED 日志，设置被静默忽略） - _apply_leaves 不再跳过空值；apply_global_battle_team 在通用队伍为空时 把 autoLeyLineOutcropConfig.Team / autoStygianOnslaughtConfig.fightTeamName 清成空串，避免原生路径（地脉花/幽境危战直读全局段）沿用 BGI 旧队伍； 写入仍受快照保护，运行结束由 restore_global_battle_config 还原 - CHANGELOG 条目合并为一条，覆盖本分支两组修复
原先战斗 4 项（MAS一条龙）与自定义配置组（MAS-{短id}-自定义配置组N）分两次
启动 BetterGI，各自杀进程、启动、监控、收尾；队列顺序也无法跨越这两批。
现改为「切段 + 一次 --startGroups」：
- one_dragon_bridge.build_execution_segments 按左栏队列顺序把执行层切成若干「段」，
write_execution_groups 每段写一个配置组文件，再由一次
`--startGroups <段1 段2 ...>` 顺序跑完（BGI 的 OnStartMultiScriptGroupWithNamesAsync
按传入顺序 foreach 执行）。
- 战斗 4 项整体一段（内部仍由 main.js 按 Plan 编排，保留树脂耗尽短路等批次语义），
插在队列中第一个被接管的战斗项处；js/pathing/keymouse 类与相邻项合并成段
（其组级 config 由 MAS 合成、无用户语义）；
scriptgroup 类独立成段 —— BGI 的 Pathing/Shell 项目只读**组级** config
（ScriptGroupProject.Run 用 GroupInfo.Config.PathingConfig），合并会丢掉
用户配置组的队伍/自动拾取等设置，故必须保真。
- one_dragon 新增 resolve_custom_group，把「自定义项 → 可执行配置组」的解析
统一到一处，materialize_user_script_groups 与切段共用，消除重复分支。
- 执行层接管时不再逐项物化，改为把这些项的名字并进 exclude_task_names 从原生
副本剔除；段组文件与历史残留由 _restore_one_dragon_config /
cleanup_leftover_execution_groups 清理。
- 删除已被取代的 write_one_dragon_group / remove_one_dragon_group 与
_run_plan_custom，_run_plan_combat 泛化为 _run_execution_layer（等待所有段的
执行结束标记）。
- 原生副本剔除战斗组的门控从 plan_mode（是否存在启用项）改为 use_execution_layer + Plan 归属：全部战斗行都关掉时 plan_mode 为假，原来战斗项会整体落回原生副本照跑， 与界面「已关」不符（2026-09-12 实机排障） - 新增 one_dragon_plan.plan_combat_bases()：只看归属（Plan 中配过实例）不看启停， 与 build_combat_steps() 的分工是「谁负责」与「这次跑谁」 - Plan 中没有实例的战斗组仍留在原生副本，宁可多跑一次也不静默丢任务 - 同步 step-enabled 接口与前端服务层文档：enabled=false 的语义是「本次不跑」， 不是「退回原生一条龙跑」 - 补单测 tests/task/test_bettergi_combat_scope.py（4 例）
_run_execution_layer 跑完 BetterGI 后不写 log_record：战斗/自定义项全部由执行层完成、 原生一条龙又无启用任务时，整轮结果显示「未开始运行」（UserItem.result 对空记录的兜底）， 并且不写历史日志、不发「统计信息」通知、分步报告与掉落统计无数据可解析。 改为执行层启动时新建 log_record 并把执行层日志写进去，收尾按成败写入状态（成功用项目 契约 "Success!"，失败写具体原因：命中致命日志 / 进程在结束标记前退出 / 空闲超时 / 执行异常 / 必填配置缺失）；同时把 log_start_time / cur_user_log 指向它，使执行层阶段的 on_crash 异常写入与「运行异常」通知也能生效。没有需要接管的段时不建记录。
本 PR 重建到当前 dev 后按新规范登记：碎片固定用 changelog.d/，不再改 CHANGELOG.md 与 res/version.json（重做时已从提交里剔除这两处生成物改动）。
本分支为自洽而包含了 AUTO-MAS-Project#766 的提交（执行层判负逻辑依赖它引入的 _missing_config_reasons）， 但「一条 PR 恰好一个碎片」，重复登记会被 CI 判失败。故此处移除该碎片：它在 AUTO-MAS-Project#766 里保留。 待 AUTO-MAS-Project#766 合入后，本分支 rebase 到新的 dev 即可让这段重复代码一并消失。
…tatistics # Conflicts: # app/task/BetterGI/AutoProxy.py # app/task/BetterGI/tools/one_dragon.py
本分支为自洽而包含 AUTO-MAS-Project#756 的提交（掉落统计经由统一通知编排推送），但「一条 PR 恰好一个 碎片」，重复登记会被 CI 判失败，故移除该碎片；它在 AUTO-MAS-Project#756 里保留。 待 AUTO-MAS-Project#756 合入后需在本分支恢复该碎片：届时 dev 已有它，本分支缺失会被显示成「删除他人 碎片」，恢复后本分支与 dev 完全一致、diff 归零。
直控（脚本直控）的语义是「用 BGI 所选原生配置启动、MAS 不接管」，此前被快速配置 耦合破坏：use_mas_config / masConfigEnabled 都写成「来源 ≠ 直控 或 快速配置」，而快速 配置默认开，于是直控被算成仍由 MAS 接管——前端不切到原生控件、「一条龙名称」被锁成 MAS 槽位名，后端也改用 MAS 槽位启动。 改回按来源判定（快速配置不参与「用哪份配置启动」这件事）： - AutoProxy / ScriptConfig：use_mas_config = 来源 ≠ 直控 → 直控下一律用 Task.OneDragonConfigName 指向的 BGI 原生配置启动 - BetterGIUserEdit：masConfigEnabled = 来源 ≠ 直控 → 直控下显示「一条龙名称（可选原生配置）+ 配置 BetterGI」、隐藏 MAS 面板， 并禁用快速配置开关（其语义见下一步）、直控说明常显 验证：py_compile 通过；pytest tests/task/test_direct_control_gates.py tests/task/test_quick_config_runtime_gates.py tests/task/test_bettergi_one_dragon_read_failure.py 共 38 passed。 后续（同一 PR）：直控 + 快速配置开启时，把面板值写入该原生配置、结束恢复现场。
第二步：直控来源下的快速配置不再是没有效果的开关——开启时把面板值（组开关、队列 顺序、领取奖励队伍、战斗队伍/策略）写入 Task.OneDragonConfigName 指向的那份 BGI 原生配置，并把通用的队伍/策略全局叶子一并补写；运行前对整份原生配置做快照， 运行/异常结束还原，且还原前比对内容——运行期间被外部改过的配置保留外部改动。 - one_dragon 新增 read_native_one_dragon / write_native_one_dragon：种子是该原生配置 自身（保留用户的一条龙结构与自定义组定义），写回同一个文件，不建 MAS 槽位、 不物化 MAS- 前缀配置组 - AutoProxy 新增 writes_native_config 判定与 _write_native_one_dragon； _backup/_restore_one_dragon_config 在直控分支下接管原生配置的快照与还原 - 前端：直控下快速配置开关恢复可用（语义即「把面板值写入该原生配置」） 验证：py_compile 通过；pytest tests/task/test_direct_control_gates.py tests/task/test_quick_config_runtime_gates.py tests/task/test_bettergi_one_dragon_read_failure.py 共 38 passed； scripts/changelog.py check 通过。未做实机验证。
用户编辑页此前有两个指向同一个 Info.IfQuickConfig 的入口：模式选择器里的 「是否启用快速配置」与一个裸开关旁的「快速配置」，分不清哪个才是真实开关。 现统一走 GeneralConfigModeSelector 的表单项（带说明 tooltip，与 Okww 等专项一致）， 删除裸开关与「快速配置」文案；新增 handleQuickConfigChange 负责保存该字段。
第三步：四项战斗组的 per-任务设置（树脂模式与次数、奖励识别、秘境名、首领名、队伍/ 策略等可对齐字段）此前只存在于 Plan、仅由执行层消费，直控下不生效。 - one_dragon_plan 新增 plan_steps_to_native_settings()：按 RIGHTBAR_TO_PLAN 反向映射 Plan settings → 原生键，只收白名单键、且只收非空值（留空保持原生配置现有值） - write_native_one_dragon 增加 native_step_settings：首领讨伐/地脉花合并进一条龙文件， 秘境/幽境分别经 write_global_domain_settings / write_global_stygian_settings 落全局段 （两者自带白名单过滤；叶子快照已由 snapshot_global_battle_config 覆盖） - AutoProxy._write_native_one_dragon 传入 plan_combat_steps 的映射结果 结构化/全局共享字段（秘境每周表、地脉花每工作日 country/type、maxArtifactStar） 按 one_dragon_plan 的约定留在原生存储，不参与对齐。 验证：py_compile 通过；映射函数抽样正确；pytest 直控门控 / 快速配置运行时门控 / 读配置失败 共 38 passed。未做实机验证。
锁住 plan_steps_to_native_settings 的对外行为（8 例，实测通过）： - 四项战斗组各自的原生键名（AutoBossName / AutoBossRunCount / AutoBossSpecifyRunCount / AutoBossReviveRetryCount / AutoBossRewardRecognitionEnabled / specifyResinUse / condensedResinUseCount / DomainName / SundayEverySelectedValue / LeyLineRunCount / country / bossNum / fightTeamName） - 空值过滤：留空字段不得进入原生配置（否则会灌空值） - 白名单过滤：未登记的键（如 weeklyDomain）不产出 - 同名多实例（自动秘境-副本A）按基名归集，原生单值语义下后写者胜 - 未知组、非 dict settings、非 dict 步骤、None 输入一律忽略
apply_groups 在队列模式下对战斗 4 项（秘境/地脉花/幽境危战/首领讨伐）仅按 Groups 纳入判定启停，忽略队列条目的 enabled。前端关闭战斗组开关只把队列条目 enabled 置 false、基名仍保留在 Groups，导致「界面关了、原生一条龙仍跑」。 改为战斗 4 项启停由「Groups 纳入」与「队列条目 enabled」共同决定，并补回归 测试。 Co-Authored-By: Claude <noreply@anthropic.com>
- controlTypeOf 按 BetterGI 官方 type 枚举补全别名归一与结构兜底，避免开关/下拉/多选项被误识别为文本框 - a-select 增加 getPopupContainer，避免被父容器 overflow 裁剪导致选项列表不可见
# Conflicts: # app/task/BetterGI/tools/notify.py
# Conflicts: # app/task/BetterGI/AutoProxy.py # app/task/BetterGI/tools/one_dragon.py # res/templates/BetterGI/MASOneDragon/main.js
# Conflicts: # app/task/BetterGI/AutoProxy.py
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
TCddddd
changed the title
审查者指南整合了九个围绕队列顺序、单次启动执行层的 BetterGI PR；分离原生配置与 MAS 配置的所有权，并通过安全的临时写入机制进行管理；新增必填字段验证、掉落统计报告、统一通知、UI 修复和回归测试。 BetterGI 队列顺序执行时序图sequenceDiagram
participant AutoProxyTask
participant one_dragon_bridge
participant BetterGI
participant LogMonitor
participant LogRecord
participant NativeOneDragon
AutoProxyTask->>one_dragon_bridge: build_execution_segments()
one_dragon_bridge-->>AutoProxyTask: exec_group_names
AutoProxyTask->>one_dragon_bridge: write_execution_groups()
AutoProxyTask->>BetterGI: open_process(--startGroups, ...group_names)
BetterGI-->>LogMonitor: ordered execution logs
LogMonitor->>LogRecord: update content and status
LogMonitor-->>AutoProxyTask: group completion or failure markers
AutoProxyTask->>NativeOneDragon: run remaining enabled tasks
AutoProxyTask->>one_dragon_bridge: remove_execution_groups()
BetterGI 配置所有权与恢复时序图sequenceDiagram
participant AutoProxyTask
participant NativeConfig
participant one_dragon
participant BetterGI
AutoProxyTask->>AutoProxyTask: read_config_source()
alt MAS configuration source
AutoProxyTask->>one_dragon: write_user_one_dragon()
one_dragon-->>BetterGI: temporary MAS slot and execution groups
else Direct configuration with quick config
AutoProxyTask->>NativeConfig: read_native_one_dragon()
AutoProxyTask->>one_dragon: write_native_one_dragon()
one_dragon-->>NativeConfig: temporary panel values
end
AutoProxyTask->>BetterGI: execute configured tasks
AutoProxyTask->>NativeConfig: read_file()
alt configuration unchanged
AutoProxyTask->>NativeConfig: write_file(snapshot)
else externally modified
AutoProxyTask->>NativeConfig: retain external changes
end
BetterGI 掉落统计通知时序图sequenceDiagram
participant BetterGI
participant LogMonitor
participant AutoProxyTask
participant parse_drop_lines
participant push_notification
participant dispatch
participant NotificationChannels
BetterGI-->>LogMonitor: reward recognition log lines
LogMonitor->>AutoProxyTask: execution LogRecord
AutoProxyTask->>parse_drop_lines: parse_drop_lines(log lines)
parse_drop_lines-->>AutoProxyTask: drop_statistics
AutoProxyTask->>push_notification: push_notification(统计信息)
push_notification->>dispatch: dispatch(NotifyPayload, statistic_targets)
dispatch->>NotificationChannels: deliver unified notification
BetterGI 必填配置验证流程图flowchart TD
A[Plan step is scheduled today] --> B{Required fields present?}
B -->|No| C[masLog MAS_STEP_MISSING_CONFIG]
C --> D[Skip step]
D --> E[Execution layer reports failure and reason]
B -->|Yes| F[dispatchCombat]
F --> G[BetterGI executes step]
G --> H{Runtime failure?}
H -->|Yes| I[Skip failed step and continue]
H -->|No| J[Mark step complete]
文件级变更
可能关联的问题
提示和命令与 Sourcery 交互
自定义使用体验访问你的控制面板以：
获取帮助Original review guide in EnglishReviewer's GuideConsolidates nine BetterGI PRs around a queue-ordered, single-launch execution layer; separates native versus MAS configuration ownership with safe temporary writes; adds required-field validation, drop-statistics reporting, unified notifications, UI fixes, and regression tests. Sequence diagram for BetterGI queue-ordered executionsequenceDiagram
participant AutoProxyTask
participant one_dragon_bridge
participant BetterGI
participant LogMonitor
participant LogRecord
participant NativeOneDragon
AutoProxyTask->>one_dragon_bridge: build_execution_segments()
one_dragon_bridge-->>AutoProxyTask: exec_group_names
AutoProxyTask->>one_dragon_bridge: write_execution_groups()
AutoProxyTask->>BetterGI: open_process(--startGroups, ...group_names)
BetterGI-->>LogMonitor: ordered execution logs
LogMonitor->>LogRecord: update content and status
LogMonitor-->>AutoProxyTask: group completion or failure markers
AutoProxyTask->>NativeOneDragon: run remaining enabled tasks
AutoProxyTask->>one_dragon_bridge: remove_execution_groups()
Sequence diagram for BetterGI configuration ownership and restorationsequenceDiagram
participant AutoProxyTask
participant NativeConfig
participant one_dragon
participant BetterGI
AutoProxyTask->>AutoProxyTask: read_config_source()
alt MAS configuration source
AutoProxyTask->>one_dragon: write_user_one_dragon()
one_dragon-->>BetterGI: temporary MAS slot and execution groups
else Direct configuration with quick config
AutoProxyTask->>NativeConfig: read_native_one_dragon()
AutoProxyTask->>one_dragon: write_native_one_dragon()
one_dragon-->>NativeConfig: temporary panel values
end
AutoProxyTask->>BetterGI: execute configured tasks
AutoProxyTask->>NativeConfig: read_file()
alt configuration unchanged
AutoProxyTask->>NativeConfig: write_file(snapshot)
else externally modified
AutoProxyTask->>NativeConfig: retain external changes
end
Sequence diagram for BetterGI drop statistics notificationsequenceDiagram
participant BetterGI
participant LogMonitor
participant AutoProxyTask
participant parse_drop_lines
participant push_notification
participant dispatch
participant NotificationChannels
BetterGI-->>LogMonitor: reward recognition log lines
LogMonitor->>AutoProxyTask: execution LogRecord
AutoProxyTask->>parse_drop_lines: parse_drop_lines(log lines)
parse_drop_lines-->>AutoProxyTask: drop_statistics
AutoProxyTask->>push_notification: push_notification(统计信息)
push_notification->>dispatch: dispatch(NotifyPayload, statistic_targets)
dispatch->>NotificationChannels: deliver unified notification
Flow diagram for BetterGI required configuration validationflowchart TD
A[Plan step is scheduled today] --> B{Required fields present?}
B -->|No| C[masLog MAS_STEP_MISSING_CONFIG]
C --> D[Skip step]
D --> E[Execution layer reports failure and reason]
B -->|Yes| F[dispatchCombat]
F --> G[BetterGI executes step]
G --> H{Runtime failure?}
H -->|Yes| I[Skip failed step and continue]
H -->|No| J[Mark step complete]
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概述
将以下 9 个 BetterGI 相关 PR 合并为一个分支统一提交：
#756 #759 #765 #766 #767 #768 #771 #779 #780
注：#768 的提交历史已内含 #765 与 #766，故以 #768 一次性带入三者，避免重复应用。
主要改动（用户可见）
Sourcery 摘要
统一 BetterGI 的执行、配置、通知和报告行为，同时修复任务启用和验证相关问题。
新功能：
错误修复：
改进：
测试：
杂项：
Original summary in English
Summary by Sourcery
Unify BetterGI execution, configuration, notification, and reporting behavior while fixing task enablement and validation issues.
New Features:
Bug Fixes:
Enhancements:
Tests:
Chores: