Skip to content

docs(ledger): マージ済み順位 324/412/457 の残骸を後始末する - #467

Merged
aloekun merged 1 commit into
masterfrom
docs/ledger-stale-rank-cleanup
Sep 2, 2026
Merged

docs(ledger): マージ済み順位 324/412/457 の残骸を後始末する#467
aloekun merged 1 commit into
masterfrom
docs/ledger-stale-rank-cleanup

Conversation

@aloekun

@aloekun aloekun commented Sep 2, 2026

Copy link
Copy Markdown
Owner

何を直すか

2026-08-30 のリベース (jj rebase -r で先端のみ移動) が、夜間 PR 3 本 (#427 / #459 / #461) の台帳削除コミット (親) を置き去りにした。実装はマージ済みなのに台帳行・順位 table 行・詳細エントリが残り、2026-09-01 の夜間 run が順位 324 を再選択して空 diff red になった (run 90894308468)。

何をしたか

  • cli-ledger-cleanup --apply を順位ごとに実行し、324 / 412 / 457 の 3 点セット (台帳行 + 順位 table 行 + 詳細エントリ) を削除 (計 9 エントリ)
  • 経緯を台帳の棚卸し履歴 2026-09-02 に記録

この PR に含めないもの

  • marker claude/nightly-324 の削除 — 本 PR マージ後に実施 (master に残骸行がある間に消すと今夜再選択されるため)
  • 再発防止 — (1) claude/nightly-<順位> PR が当該順位の台帳行削除を diff に含むことの CI 検査、(2) merged PR × 台帳の照合を weekly-review / 夜間 preflight へ組み込み。別 PR で実装する

Summary by CodeRabbit

  • ドキュメント
    • 採用タスク一覧から、マージ済みまたは他計画へ移管された項目を整理しました。
    • 棚卸し履歴に2026年9月2日の記録を追加し、削除経緯と再発防止策を記載しました。
    • 推奨実行順序および関連するTODO一覧から、順位324・412・457の項目を削除しました。
    • 残存するタスク項目や一覧の構成は維持されています。

2026-08-30 のリベース (jj rebase -r で先端のみ移動) が夜間 PR 3 本
(#427/#459/#461) の台帳削除コミット (親) を置き去りにし、実装は
マージ済みなのに台帳行・順位 table 行・詳細エントリが残っていた。
2026-09-01 の夜間 run が順位 324 を再選択して空 diff red になった原因。

削除は cli-ledger-cleanup --apply を順位ごとに実行 (3 点セット x 3)。
経緯は台帳の棚卸し履歴 2026-09-02 に記録した。再発防止 (CI 検査 +
merged PR 照合) は別 PR で実装する。
@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 8049e5ee-83d6-489b-a112-1d694350f48f

📥 Commits

Reviewing files that changed from the base of the PR and between df50e67 and 91e24b4.

📒 Files selected for processing (5)
  • docs/claude-code-web-tasks.md
  • docs/todo-summary2.md
  • docs/todo17.md
  • docs/todo21.md
  • docs/todo23.md
💤 Files with no reviewable changes (4)
  • docs/todo17.md
  • docs/todo23.md
  • docs/todo21.md
  • docs/todo-summary2.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

マージ済みまたは他計画へ移管された採用タスクを複数の文書から削除した。残存タスクを維持し、2026-09-02 の棚卸し履歴と削除手順を追加した。

Changes

採用タスク台帳の整理

Layer / File(s) Summary
台帳と TODO の整理
docs/claude-code-web-tasks.md, docs/todo-summary2.md, docs/todo17.md, docs/todo21.md, docs/todo23.md
マージ済みまたは移管済みの順位を一覧と詳細 TODO から削除した。残存する順位は維持した。docs/claude-code-web-tasks.md に 2026-09-02 の棚卸し履歴、cli-ledger-cleanup --apply による削除手順、再発防止策の予定を追加した。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 91e24

This PR removes stale ledger and ranking entries and records the cleanup history without changing product behavior; no actionable merge-blocking risk remains beyond normal checks and review.

🚥 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 タイトルは、マージ済みの順位324、412、457に関する台帳残骸の後始末という変更内容を正確かつ簡潔に示しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/ledger-stale-rank-cleanup

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.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

🤖 PR Monitor 分析 (GitHub Actions バックストップ)

  • トリガー: issue_comment (created) / 実行 run
  • CI: pending — rust (ubuntu-latest) / rust (windows-latest) は pending、request は skipping、CodeRabbit チェックは pass 表示だが実体は「Review skipped: manual review required for this OSS repository」(レビュー未実施の意)。mergeStateStatus は BLOCKED (pending チェック起因とみられる)。
  • レビュー状況: 未実施 (陽性証拠なし) — reviews API は空配列、インライン指摘も空。会話コメントは CodeRabbit の自動 skip 通知 (Review skipped... fewer than 10 stars) のみで、レビュー本体ではない。人間レビューも無し。
  • Verdict: user_decision

Applicable Findings (Critical / High / Major)

該当なし (指摘 0 件)

Applicable Findings (Medium 以下)

該当なし

Filtered (not applicable)

該当なし

軽量サマリー (レビュー指摘 0 件のため)

  • 変更範囲: docs/ 配下 5 ファイルのみ (claude-code-web-tasks.md, todo-summary2.md, todo17.md, todo21.md, todo23.md)、純削除中心 (PR タイトル通り、マージ済み順位 324/412/457 の台帳残骸削除)。コード (src/) への変更なし。
  • 内容は台帳の 3 順位 (324・412・457) について「採用タスク行」「summary 行」「詳細エントリ」を削除し、claude-code-web-tasks.md に削除理由の履歴 (2026-09-02 セクション) を追記する形で一貫しており、diff 内で自己矛盾は見られない。
  • CodeRabbit は本リポジトリの star 数不足によりレビュー自体を行っていない (定型 skip 通知のみ)。人間レビューも未着手。

次のアクション

  • CI (rust ubuntu/windows) が pending のため、完了を待って green を確認する。
  • docs-only 変更で自動レビューが機能しない状況のため、マージ前に人間による目視確認 (削除対象の順位・行が意図通りか) を推奨する。
  • 本コメント以降、実質的なレビュー (人間 or CodeRabbit の実分析) が付いた時点で再分析が必要。

@aloekun
aloekun merged commit dc39b70 into master Sep 2, 2026
4 checks passed
@aloekun
aloekun deleted the docs/ledger-stale-rank-cleanup branch September 2, 2026 07:10
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.

1 participant