Skip to content

kaizen: docs/org-monitor.md の古い固定 nightly/afternoon スケジュール表を削除しました。 (#61)#66

Merged
s-hiraoku merged 1 commit into
mainfrom
kaizen/issue-61-monitor-clarify-scheduler-job-guidance
Jun 27, 2026
Merged

kaizen: docs/org-monitor.md の古い固定 nightly/afternoon スケジュール表を削除しました。 (#61)#66
s-hiraoku merged 1 commit into
mainfrom
kaizen/issue-61-monitor-clarify-scheduler-job-guidance

Conversation

@s-hiraoku

@s-hiraoku s-hiraoku commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Closes #61

Summary

docs/org-monitor.md の古い固定 nightly/afternoon スケジュール表を削除しました。
各リポジトリの scheduler.jobs を authoritative な契約として読む説明に更新しました。
コミット 53d1aa0 を作成しました。

Builder notes

検証: 指定の test コマンド、git diff --check、scripts/check-daily-dogfood-sync-contract.sh が成功。保護パスの変更なし。

Verification

  • test -f docs/README.md && test -f docs/issue-to-pr-mvp.md && test -f profile/README.md
  • test -f skills/gh-link-issue-pr/SKILL.md && test -f skills/kaizen-bug-router/SKILL.md && test -f skills/pr-guardian/SKILL.md

Kaizen risk policy

Verifier cleared PR: Open PR with 0 should_fix item(s); risk is low.

Changed files: 1
Changed lines: 15

Summary by CodeRabbit

  • Documentation
    • Updated the org monitor guidance to reflect repository-owned job scheduling.
    • Removed references to a fixed, organization-wide schedule and example trigger names.
    • Clarified that monitor reports should use each repository’s configured enabled jobs and settings, without assuming shared job names or run times.

@coderabbitai

coderabbitai Bot commented Jun 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: kaizen-agents-org/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f740004e-ac51-4766-9ea9-e53895bf3e93

📥 Commits

Reviewing files that changed from the base of the PR and between d44ddcf and 53d1aa0.

📒 Files selected for processing (1)
  • docs/org-monitor.md

📝 Walkthrough

Walkthrough

The org monitor documentation now uses each repository’s .kaizen/config.yml scheduler.jobs map as the schedule source and removes the fixed nightly/afternoon trigger examples.

Changes

Org monitor scheduler guidance

Layer / File(s) Summary
Repository-owned schedule model
docs/org-monitor.md
The schedule section now reads enabled jobs from each repository’s .kaizen/config.yml scheduler.jobs map and removes the fixed nightly/afternoon trigger examples and assumptions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

  • builder-agent issue 38 — Matches the same scheduler guidance change by replacing fixed cadence examples with repository-owned scheduler.jobs configuration.
  • renovate-config issue 15 — Concerns the same org-monitor scheduling model update in docs/org-monitor.md, with the old fixed schedule section removed.

Possibly related PRs

  • kaizen-agents-org/.github#41: Migrates scheduler configuration toward scheduler.jobs, which matches the documentation’s updated source-of-truth model.
  • kaizen-agents-org/.github#46: Updates docs/org-monitor.md with the same shift away from fixed nightly/afternoon guidance.
  • kaizen-agents-org/.github#53: Removes legacy scheduler-key assumptions while enforcing the scheduler.jobs structure that the docs now describe.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main doc change: removing the stale fixed nightly/afternoon schedule table.
Linked Issues check ✅ Passed The doc now aligns with #61 by removing fixed trigger guidance and making scheduler.jobs the authoritative source.
Out of Scope Changes check ✅ Passed The changes stay scoped to docs/org-monitor.md and only rewrite the scheduler guidance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kaizen/issue-61-monitor-clarify-scheduler-job-guidance

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

@s-hiraoku

Copy link
Copy Markdown
Contributor Author

PR Guardian pass 1/5 complete.

Final mergeability and checks:

  • PR is non-draft and mergeable: mergeStateStatus=CLEAN, mergeable=MERGEABLE.
  • gh pr checks reports CodeRabbit passing with Review completed.
  • Branch protection on main requires conversation resolution, but does not require PR review approval or required status checks. No unresolved conversations were found.

CI runs watched:

  • No GitHub Actions workflow runs existed for branch kaizen/issue-61-monitor-clarify-scheduler-job-guidance, so there was no gh run watch --exit-status target. The relevant external status was monitored through gh pr checks until completion.

Fixes pushed:

  • None. No CI failure or actionable review feedback required a code change.

Feedback addressed:

  • Fetched paginated GraphQL review threads, PR comments, and reviews through hasNextPage=false.
  • Review threads: none.
  • Reviews: none.
  • PR comments: CodeRabbit reported no actionable comments and all listed pre-merge checks passed.

Unresolved or skipped feedback:

  • None.

Remaining blockers:

  • None. The PR is ready to merge when the repository owner chooses. This comment does not merge the PR.

@s-hiraoku s-hiraoku merged commit f778cfe into main Jun 27, 2026
1 check 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.

[monitor] Clarify scheduler job guidance

1 participant