Skip to content

定时任务-Command类型,只有第一次成功,后续都失败问题#46

Merged
zds-s merged 1 commit intomineadmin:mainfrom
luckydog-sunny:faq
Mar 13, 2026
Merged

定时任务-Command类型,只有第一次成功,后续都失败问题#46
zds-s merged 1 commit intomineadmin:mainfrom
luckydog-sunny:faq

Conversation

@luckydog-sunny
Copy link
Copy Markdown
Contributor

@luckydog-sunny luckydog-sunny commented Mar 11, 2026

调用目标:{"command":"mine:xxx","--disable-event-dispatcher":true}

Summary by CodeRabbit

文档

  • 新增常见问题解答章节,介绍定时任务的已知问题及解决方案,包含配置示例和相关文档链接。

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 11, 2026

@luckydog-sunny is attempting to deploy a commit to the deathsatan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 11, 2026

Deploy Preview for super-cobbler-53ff65 failed. Why did it fail? →

Name Link
🔨 Latest commit 034fe05
🔍 Latest deploy log https://app.netlify.com/projects/super-cobbler-53ff65/deploys/69b1a92779595a000893b4bf

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 11, 2026

Walkthrough

该PR在中文FAQ文档中添加了一个新的小节,说明Command类型计划任务的一个常见问题:首次运行成功但后续运行失败。文档提供了解决方案示例和相关文档链接。

Changes

Cohort / File(s) Summary
FAQ文档更新
docs/zh/faq/index.md
新增关于计划任务常见陷阱的FAQ小节,描述Command类型任务的重复执行失败问题,并说明需要在crontab配置中设置 --disable-event-dispatcher 为true的解决方案。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 文档添一笔,常见问题解,
计划任务有玄机,禁用事件派发妙,
兔兔文档全了解,再无困扰笑盈盈。✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR标题与变更内容完全相关,准确概括了FAQ中添加的新内容——解决Command类型定时任务只有首次运行成功、后续运行失败的问题。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/zh/faq/index.md`:
- Around line 144-147: Fix the typo "ture" to "true" and reword the sentence to
clarify that "--disable-event-dispatcher" is a JSON key inside the crontab
entry's value rather than a CLI flag; specifically, change the text around
"调用目标,需要加上--disable-event-dispatcher: ture配置。" to something like "在 value 中增加
\"--disable-event-dispatcher\" 并设为 true",and ensure the JSON example uses the
correct boolean form {"command":"mine:xxx","--disable-event-dispatcher":true} so
readers don't confuse it with command-line syntax.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 24838b15-8e85-4cbd-a172-4ab37558fad7

📥 Commits

Reviewing files that changed from the base of the PR and between 888357a and 034fe05.

📒 Files selected for processing (1)
  • docs/zh/faq/index.md

Comment on lines +144 to +147
调用目标,需要加上--disable-event-dispatcher: ture配置。
即crontab表中的value值为:
```json
{"command":"mine:xxx","--disable-event-dispatcher":true}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

修正文案里的布尔值拼写和配置写法。

Line 144 的 ture 是明显笔误;另外这里写成 --disable-event-dispatcher: ture 也容易让人误解成命令行参数语法,但下方示例实际是 JSON 键值。建议直接改成“在 value 中增加 --disable-event-dispatcher 并设为 true”,避免读者照抄错误配置。

✏️ 建议修改
-调用目标,需要加上--disable-event-dispatcher: ture配置。
-即crontab表中的value值为:
+调用目标时,需要在 crontab 表的 `value` 中增加 `--disable-event-dispatcher`,并将其设置为 `true`。
+示例如下:
 ```json
 {"command":"mine:xxx","--disable-event-dispatcher":true}
</details>

<details>
<summary>🤖 Prompt for AI Agents</summary>

Verify each finding against the current code and only fix it if needed.

In @docs/zh/faq/index.md around lines 144 - 147, Fix the typo "ture" to "true"
and reword the sentence to clarify that "--disable-event-dispatcher" is a JSON
key inside the crontab entry's value rather than a CLI flag; specifically,
change the text around "调用目标,需要加上--disable-event-dispatcher: ture配置。" to
something like "在 value 中增加 "--disable-event-dispatcher" 并设为 true",and ensure
the JSON example uses the correct boolean form
{"command":"mine:xxx","--disable-event-dispatcher":true} so readers don't
confuse it with command-line syntax.


</details>

<!-- fingerprinting:phantom:triton:grasshopper -->

<!-- This is an auto-generated comment by CodeRabbit -->

@zds-s zds-s merged commit ee0e92e into mineadmin:main Mar 13, 2026
2 of 8 checks passed
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