fix(Dokan): 允许在任意日期建立尚未开启的道馆

[huiyuhang1]

Sourcery 总结

新功能：

• 允许在任意日期创建道馆，原始逻辑只在周一创建道馆，会导致其他时间有时候无法打道馆。

[sourcery-ai]

嗨——我已审阅你的更改，整体看起来很棒！

Sourcery 评估

需要人工审阅。 符合任务条件时，现在会在任何一个工作日调用 creat_dokan()，因此错误的排期变更可能会导致道场比预期更早创建。即使回滚，这些已创建的状态也可能继续存在，不过影响范围仅限于该道场，可以另行处理。

---

Sourcery 对开源项目免费——如果你喜欢我们的审查结果，请考虑分享一下 ✨

• X
• Mastodon
• LinkedIn
• Facebook

Original comment in English

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

Sourcery assessment

Needs a human reviewer. When the task qualifies, it now calls creat_dokan() on any weekday, so a mistaken scheduling change could create a dojo earlier than intended. That created state may outlive a revert, though the impact is bounded to the dojo and can be addressed separately.

---

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

• X
• Mastodon
• LinkedIn
• Facebook

[github-actions]

Findings

• C1 [不确定]
  位置:.github/workflows/shared/review-checklist.md;PR 文件 tasks/Dokan/script_task.py
  原因:指定 base SHA 的清单无法读取,无法确认 C1 的判定要求;当前改动仅移除 weekday 限制并无对应新增测试。
  修改:恢复可访问的清单内容后,按 C1 要求补充或核验针对任意日期、道馆未开启状态的测试。
• C2 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:143
  原因:无法确认 C2 的规则;改动将 creat_dokan() 从周一条件中移出,行为覆盖范围扩大但缺少验证证据。
  修改:补充非周一且道馆未开启场景的自动化测试,并确认周一行为不回归。
• C3 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:143
  原因:无法确认 C3 的规则;当前 diff 未展示 creat_dokan() 对重复调用、失败或已开启状态的保护。
  修改:核验并补充状态判断或调用失败处理,确保任意日期调用不会造成重复建立或异常循环。
• C4 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:140-147
  原因:无法确认 C4 的规则;PR 检查中 Run 已失败,当前 head 尚无成功状态可证明兼容性。
  修改:先修复或说明失败检查,再验证非目标日期、已开启道馆和找不到道馆等既有路径。
• C5 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:143
  原因:无法确认 C5 的规则;每次循环都会调用 creat_dokan(),新增调用频率的边界未在 diff 中说明。
  修改:确认循环退出条件和调用次数上限,必要时增加防重复调用或日志。
• C6 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:140-147
  原因:无法确认 C6 的规则;缺少清单与测试证据,无法判断该行为变化是否影响配置约束或外部状态。
  修改:补充对配置、登录状态及服务端返回状态的边界验证。
• C7 [不确定]
  位置:.github/workflows/shared/review-checklist.md;tasks/Dokan/script_task.py:143
  原因:无法确认 C7 的规则;删除日期判断后,非周一也会进入建立流程,资源与请求成本未被验证。
  修改:提供调用次数或运行耗时对比,并确认仅在确有必要时发起建立请求。
• C8 [不确定]
  位置:.github/workflows/shared/review-checklist.md;PR fix(Dokan): 允许在任意日期建立尚未开启的道馆 #1950
  原因:无法确认 C8 的规则;PR 描述与代码意图一致,但清单要求和验证材料均不可得。
  修改:在 PR 中补充变更前后行为、适用日期范围及测试结果。

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• ab.chatgpt.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "ab.chatgpt.com"

See Network Configuration for more information.

Generated by PR Review Checklist for #1950 · codex · gpt56 · 8.89 AIC · ⌖ 2.01 AIC · ⊞ 12.1K · ◷