feat(hooks): SessionStart hook でセッション ID 伝播 + CronCreate セッション問題解決 (#17 再作成) - #18
Merged
Conversation
📝 Walkthrough概要セッション ID の管理とハンドシェイク機能を実装するために、SessionStart フックを導入し、プッシュパイプラインの複数ステップを無効化しました。hooks-post-pr-monitor に主セッション ID の取得と再開ロジックを追加し、hooks-session-start という新しい Rust バイナリでセッション ID の環境変数への書き込みとファイル保存を行うようにしました。 変更一覧
シーケンス図sequenceDiagram
participant User as ユーザー<br/>(pnpm push)
participant SessionStart as SessionStart<br/>フック
participant EnvFile as 環境ファイル
participant SessionFile as .session-id<br/>ファイル
participant PostMonitor as hooks-post-pr-monitor
participant Claude as Claude CLI
User->>SessionStart: セッション ID を含む<br/>JSON を stdin に渡す
SessionStart->>EnvFile: CLAUDE_CODE_SESSION_ID<br/>をエクスポートして追記
SessionStart->>SessionFile: セッション ID<br/>を書き込み (初回のみ)
SessionStart->>PostMonitor: フック出力に<br/>セッション ID を含める
PostMonitor->>PostMonitor: get_main_session_id()<br/>で ID を取得
PostMonitor->>Claude: claude -p --resume <id><br/>で実行を試みる
alt --resume 成功時
Claude-->>PostMonitor: 成功応答
else --resume 失敗時
PostMonitor->>Claude: claude -p --continue<br/>でフォールバック
Claude-->>PostMonitor: 応答
end
推定コードレビュー工数🎯 3 (Moderate) | ⏱️ ~25 minutes 関連する可能性のあるプルリクエスト
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (3)
docs/todo.md (1)
10-12: TODO 項目のステータス確認この PR で SessionStart フックと
--resume方式が実装されているため、この TODO 項目は[x]に更新すべきではないでしょうか？📝 提案される修正
-- [ ] **CronCreate がサブセッションに閉じ込められる**: `pnpm push` 実行時、`review:ai` (`claude -p "/pre-push-review"`) のサブセッションが「最新セッション」となり、後続の `hooks-post-pr-monitor --monitor-only` の `--continue` がサブセッションに接続してしまう。SessionStart hook でメインセッション ID を `.session-id` ファイルに記録し、`--resume <session_id>` で明示指定する方式に修正 +- [x] **CronCreate がサブセッションに閉じ込められる**: `pnpm push` 実行時、`review:ai` (`claude -p "/pre-push-review"`) のサブセッションが「最新セッション」となり、後続の `hooks-post-pr-monitor --monitor-only` の `--continue` がサブセッションに接続してしまう。SessionStart hook でメインセッション ID を `.session-id` ファイルに記録し、`--resume <session_id>` で明示指定する方式に修正🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/todo.md` around lines 10 - 12, Update the TODO checklist entry about "CronCreate セッション問題" to mark it complete by changing the checkbox from `[ ]` to `[x]` since SessionStart hook and the `--resume` mechanism (referenced as SessionStart and `--resume <session_id>`) have been implemented; modify the line under "CronCreate セッション問題 (PR `#16` 調査で発見)" that currently reads "- [ ] **CronCreate がサブセッションに閉じ込められる**" to "- [x] **CronCreate がサブセッションに閉じ込められる**" and optionally add a brief note that SessionStart writes the main session ID to `.session-id` and `--resume` is used to explicitly reconnect..claude/hooks-session-start/src/main.rs (2)
72-73: ファイル書き込み失敗の無視
let _ = std::fs::write(...)は書き込み失敗を無視します。失敗した場合、hooks-post-pr-monitorはセッション ID を取得できず、--continueにフォールバックします。これは設計上許容されていますが、デバッグ用にログを出力することを検討してください。🔍 デバッグ用ログの追加
if should_write { - let _ = std::fs::write(&sid_path, &session_id); + if let Err(e) = std::fs::write(&sid_path, &session_id) { + eprintln!("[hooks-session-start] 警告: .session-id 書き込み失敗: {}", e); + } }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/hooks-session-start/src/main.rs around lines 72 - 73, 現在の let _ = std::fs::write(&sid_path, &session_id); は書き込みエラーを無視しているため、書き込みに失敗した場合にエラーをログ出力するように変更してください — std::fs::write の戻り値 (Result) をマッチまたは ? で処理し、Err の場合には sid_path と session_id を含めた分かりやすいメッセージをログ（既存のロガーがあればそれを使う、なければ eprintln!）しておきます。対象箇所は should_write ブロック内の std::fs::write 呼び出しです。
27-33: パス解決のエッジケース
current_exe()が失敗した場合、unwrap_or_default()が空のPathBufを返し、そのparent()もNoneを返すため、最終的に"./.session-id"になる可能性があります。これは.claude/.session-idではなく、hooks-post-pr-monitorが読み取れない場所になります。実際には
current_exe()が失敗するのは稀ですが、より堅牢にするには明示的なエラーハンドリングを検討してください。♻️ より堅牢なパス解決の提案
fn session_id_file_path() -> std::path::PathBuf { - std::env::current_exe() - .unwrap_or_default() - .parent() - .unwrap_or(Path::new(".")) - .join(".session-id") + std::env::current_exe() + .ok() + .and_then(|p| p.parent().map(|p| p.to_path_buf())) + .unwrap_or_else(|| Path::new(".claude").to_path_buf()) + .join(".session-id") }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/hooks-session-start/src/main.rs around lines 27 - 33, session_id_file_path currently calls std::env::current_exe().unwrap_or_default() which can produce an empty PathBuf and cause the function to resolve to "./.session-id"; change session_id_file_path to explicitly handle the Result from std::env::current_exe() (e.g. match or if let), on Ok use the exe's parent when present, and on Err try a robust fallback such as std::env::current_dir() and/or a dedicated directory like ".claude" before joining ".session-id"; update the logic around parent() to use that explicit fallback so the function reliably returns the intended .claude/.session-id path even when current_exe() fails.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude/hooks-session-start/src/main.rs:
- Around line 72-73: 現在の let _ = std::fs::write(&sid_path, &session_id);
は書き込みエラーを無視しているため、書き込みに失敗した場合にエラーをログ出力するように変更してください — std::fs::write の戻り値
(Result) をマッチまたは ? で処理し、Err の場合には sid_path と session_id
を含めた分かりやすいメッセージをログ（既存のロガーがあればそれを使う、なければ eprintln!）しておきます。対象箇所は should_write
ブロック内の std::fs::write 呼び出しです。
- Around line 27-33: session_id_file_path currently calls
std::env::current_exe().unwrap_or_default() which can produce an empty PathBuf
and cause the function to resolve to "./.session-id"; change
session_id_file_path to explicitly handle the Result from
std::env::current_exe() (e.g. match or if let), on Ok use the exe's parent when
present, and on Err try a robust fallback such as std::env::current_dir() and/or
a dedicated directory like ".claude" before joining ".session-id"; update the
logic around parent() to use that explicit fallback so the function reliably
returns the intended .claude/.session-id path even when current_exe() fails.
In `@docs/todo.md`:
- Around line 10-12: Update the TODO checklist entry about "CronCreate セッション問題"
to mark it complete by changing the checkbox from `[ ]` to `[x]` since
SessionStart hook and the `--resume` mechanism (referenced as SessionStart and
`--resume <session_id>`) have been implemented; modify the line under
"CronCreate セッション問題 (PR `#16` 調査で発見)" that currently reads "- [ ] **CronCreate
がサブセッションに閉じ込められる**" to "- [x] **CronCreate がサブセッションに閉じ込められる**" and optionally
add a brief note that SessionStart writes the main session ID to `.session-id`
and `--resume` is used to explicitly reconnect.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 452156a2-b35d-4e37-b8cd-f0be1d84609f
⛔ Files ignored due to path filters (1)
.claude/hooks-session-start/Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (8)
.claude/hooks-config.toml.claude/hooks-post-pr-monitor/src/main.rs.claude/hooks-session-start/Cargo.toml.claude/hooks-session-start/src/main.rs.claude/settings.local.json.template.gitignoredocs/todo.mdpackage.json
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR #17 の CodeRabbit レビュー上限到達のため再作成。内容は同一。
Summary by CodeRabbit
リリースノート
新機能
その他