fix(bettergi): #523 评审修复——切号脚本恢复重做、日志监控交由 #510 - #1
Open
qiyinxi wants to merge 4 commits into
Open
Conversation
…t#498) - log monitor: pass callable log path so daily-rolling logs switch across midnight - ScriptConfig: remove runtime slot after config session (final_task & on_crash) - account_switch: only rebuild repo when previous checkout also failed - open BetterGI with elevated only when MAS itself is not elevated - step report: pick success run when retries happened (no duplicated 1/N) - notify: add one day when step duration crosses midnight - frontend/backend: unify enabled default for custom groups
AUTO-MAS-Project#510 已把 LogMonitor 入参统一为 resolver、将 _build_log_path 改名为 _resolve_log_file_path 并删除 script_log_path，还补了切换后刷新时间戳基准 日期的修复；这两处 hunk 与其冲突且缺该修复，先合入时会引用不存在的方法。 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
- 标记改为在切号结束、BGI 确已启动并退出后，脚本仍缺失时才写入，并放到
MAS 自有的 data/{script_id}/ 目录，不再进 BGI 的 User/JsScript
- 是否 rmtree 仓库改在杀掉旧 BGI 进程之后单独判断，首次启用与用户误删
都不会在第一轮删除整个 Repos/bettergi-scripts-list
- 调度台文案与 docstring 同步为新流程
- 分步报告无成功轮时回退取最后一个能解析出步骤的轮次，避免整块省略
- notify._step_duration 按 ruff format 排版
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
1w1w11w1
force-pushed
the
fix/bettergi-498-issues
branch
from
September 3, 2026 13:25
c288722 to
0d0b724
Compare
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
2 participants
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.
对应 AUTO-MAS-Project#523 的评审意见，直接提到你的分支上，合并后 AUTO-MAS-Project#523 会自动带上这些改动。三个提交各自独立，可以只取一部分。
data/{script_id}/目录；是否rmtree仓库改在杀掉旧 BGI 进程之后单独判断。首次启用、用户误删、BGI 启动失败三种情况都不会在第一轮删掉Repos/bettergi-scripts-list；只有上一轮 BGI 跑完仍没把脚本补回来，下一轮才强制重建。调度台文案与 docstring 同步。script_log_path已删）且缺 fix(log): BetterGI 跨日后监控当日日志 AUTO-MAS-Project/AUTO-MAS#510 补的时间戳基准日期刷新，交由 fix(log): BetterGI 跨日后监控当日日志 AUTO-MAS-Project/AUTO-MAS#510 处理。notify._step_duration按 ruff format 排版；main.py的is_admin()复用IS_ELEVATED；changelog 精简为一句。验证（独立 venv，Python 3.12，Windows）：模拟脚本跑通首轮不删库 → 记录标记 → 次轮重建 → 脚本就绪后清标记 → 误删不删库五个阶段，BGI 目录树内无 MAS 残留文件；
pytest tests --collect-only -q退出码 0；全量tests360 passed / 1 failed，失败项在 dev 上同样失败；改动文件ruff check通过。另外 AUTO-MAS-Project#523 正文的
Closes #498建议改成「关联 AUTO-MAS-Project#498」，清单里还有未处理项。