test(session-start): stale_check_enabled の TOML パース経路をテストで固定する

[aloekun]

概要

夜間 todo 消化ループ (順位 284) が 2026-08-17 に実装したが、PR が作られないまま
ブランチだけが残っていたものを拾い上げる。実装内容は当時のまま、追加変更はしていない。

背景 — なぜ PR が無かったか

claude/nightly-284 は 2026-08-17 に push されたが、対応する PR が存在しなかった。
同日の GitHub 障害で PR 作成まで到達しなかった可能性が高い (プロジェクト側の不具合とは
断定していない)。

放置すると静かな行き止まりになる: 夜間ループの着手済み判定は open PR ではなく
リモートブランチの存在で行う (nightly-todo.yml)。ブランチが残る限り順位 284 は
選定対象から除外され続け、台帳の行は ✅ のまま誰も気づかない。

変更内容 (順位 284)

StalenessConfig.stale_check_enabled のパース経路にテストが無く、silent degrade
(機能が黙って無効化される) を検出できない状態だった。既存テスト
hooks_config_parses_session_start_staleness_section の fixture に 1 行足し、
assert を 1 行足す。

• src/hooks-session-start/src/hooks_config.rs (+2 行)
• 台帳の後始末: docs/todo15.md の節削除、docs/todo-summary2.md / docs/claude-code-web-tasks.md の 284 行削除

拾い上げ時の確認

• 削除対象の 3 箇所がいずれも master に現存することを確認済み
• stale_check_enabled フィールドと対象テストが master に現存し、変更が今も適用可能であることを確認済み
• master より 8 commit 遅れているが、変更行が重ならないため rebase なしで merge できる見込み

検証

CI (rust ubuntu/windows) の結果による。ローカルでの再検証は PR 作成後に実施する。

Summary by CodeRabbit

• テスト

  • 設定ファイルで stale_check_enabled = true を指定した場合、値が正しく読み込まれることを確認するテストを追加しました。

• ドキュメント

  • 完了済みの設定パーステストに関するタスク、作業計画、完了基準をタスク一覧から削除しました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5ca1ce14-74b5-4acf-8e81-2f73564a5f88

📥 Commits

Reviewing files that changed from the base of the PR and between 8aeeffd and 1dcb9a4.

📒 Files selected for processing (4)

• docs/claude-code-web-tasks.md
• docs/todo-summary2.md
• docs/todo15.md
• src/hooks-session-start/src/hooks_config.rs

💤 Files with no reviewable changes (3)

• docs/todo-summary2.md
• docs/todo15.md
• docs/claude-code-web-tasks.md

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

---

📝 Walkthrough

Walkthrough

stale_check_enabled = true の TOML パーステストを追加しました。パース結果の検証を追加し、関連するタスク台帳と TODO 項目を削除しました。

Changes

stale_check_enabled テスト更新

Layer / File(s)	Summary
TOML パーステストと TODO 更新 src/hooks-session-start/src/hooks_config.rs, docs/claude-code-web-tasks.md, docs/todo-summary2.md, docs/todo15.md	stale_check_enabled = true の TOML 入力と、パース結果が Some(true) になることの検証を追加しました。関連するタスク台帳と TODO 項目を削除しました。

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

Merge Risk: ⚪ Minimal · up to 1dcb9

This localized change adds coverage for the stale-check configuration parsing path and cleans up related tracking documentation; no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related PRs

• aloekun/claude-code-hook-test#320: 同じ TOML パーステストと TODO 更新を扱っています。
• aloekun/claude-code-hook-test#324: hooks_config.rs の stale_check_enabled パース検証を扱っています。
• aloekun/claude-code-hook-test#162: staleness 設定と SessionStart フックの TOML パーステストを扱っています。

🚥 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	タイトルは、主変更であるstale_check_enabledのTOMLパース経路のテスト追加を明確かつ簡潔に説明しています。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/nightly-284

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

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

• トリガー: issue_comment (created) / 実行 run
• CI: pending (rust (ubuntu-latest) pending, rust (windows-latest) pending, request skipping)
• レビュー状況: 未実施 (陽性証拠なし) — human/bot 問わずレビュー 0 件（pulls/426/reviews は空、インライン指摘も空）。CodeRabbit は「fewer than 10 stars のため自動レビュー対象外」と明示し、手動トリガー待ちのまま未実施
• Verdict: user_decision

Applicable Findings (Critical / High / Major)

該当なし（レビュー指摘が1件も存在しないため）

Filtered (not applicable)

該当なし

差分概要 (軽量サマリー)

• src/hooks-session-start/src/hooks_config.rs: 既存テストの TOML fixture に stale_check_enabled = true を追加し、対応する assert_eq! を1行追加（+2 行）。PR タイトル通り、未テストだった stale_check_enabled (Option) のパース経路を固定するテスト追加のみで、本体ロジック変更なし。
• docs/todo15.md / docs/todo-summary2.md / docs/claude-code-web-tasks.md: 消化済みタスク（順位284「stale_check_enabled の TOML パーステスト追加」）の台帳エントリ削除（純ドキュメント整理、-24行前後）。

次のアクション

• CodeRabbit の手動レビュートリガー（コメント内チェックボックス）を人間側で起票するか、CI 完了後に人間レビューのみで進めるか判断してください。
• CI (rust (ubuntu-latest) / rust (windows-latest)) の完了を待ってから merge 判断をしてください（現時点では pending）。