レビュー要求を内部キューで保持し、レビュー完了後に Codex を更新する

[lynnswap]

目的

「後でアップデート」を選んだ後もレビュー要求を受け付け、CodexReviewKit 内部で待機させる。実行中レビューが終わったら Codex を更新して app-server だけを再起動し、同じ MCP セッションとジョブ ID で待機レビューを実行する。

完了条件

• 新規要求を拒否せず、Kit 側のキューと通常の MCP 完了待機で保持する。更新専用の再送・待機判断を LLM に要求しない。
• 更新を一度だけ実行し、Monitor と MCP サーバーは存続する。失敗時も受付済みジョブを失わない。
• 待機ジョブの状態参照・キャンセル、受付順、履歴保存、既存の並行実行を保つ。
• 起動時と起動から8時間ごとに自動チェックする。設定画面から同じ checker を手動実行し、確認結果を表示する。
• 既存の540秒の MCP 待機上限は維持し、上限に届いた場合は同じジョブを review_await で待てる。
• Preview と回帰テストを追加し、各 Ready PR をローカル・リモートレビューと CI 成功後に main へ統合する。

2026-09-26 に承認された Docs/deferred-codex-update-design.md に従う。依存する実装単位は sub-issue の順で統合する。

処理フロー

sequenceDiagram
    participant L as MCP クライアント
    participant S as CodexReviewStore
    participant C as app-server
    participant U as Codex 更新処理
    Note over S: 「後でアップデート」を選択
    L->>S: review_start(B)
    Note over S: B をキューで保持し、MCP 呼び出しを待機
    C-->>S: 実行中レビュー A の完了
    Note over S,C: A の結果保存と後片付けを完了し、旧プロセスを終了
    S->>U: codex update を一度実行
    U-->>S: 更新結果
    S->>C: 現在の CLI から app-server を起動・初期化
    S->>C: B を開始
    C-->>S: B の最終結果
    S-->>L: review_start(B) の結果

Loading

B の要求は Kit 内部で保持し、更新待ちを理由に LLM へ再送や専用の待機判断を要求しません。既存の540秒の待機上限に達した場合だけ、同じジョブを review_await で待てます。

[lynnswap]

最初の子 #383 の実装を Ready PR #386 に提出しました。Kit 内のキュー、受付と実行開始の時刻分離、履歴移行、待機中のキャンセル、元の MCP 呼び出しを保った完了待機を含みます。

• パッケージ全体: 1,228テスト成功。
• ReviewMonitor: 43テスト成功（既存 CI と同じ -skipMacroValidation を使用）。
• ローカル codex-review: 7A051D69-BFB7-4F90-83E5-705D9AF64C86、main 比較、completed / 指摘なし。

承認済み設計は PR 内の Docs/deferred-codex-update-design.md に保存しました。リモートレビューと CI の通過後に #386 を統合し、次の子 #384、#385 の順に進めます。更新ダイアログと8時間周期・設定画面 UI は後続の子で接続します。

[lynnswap]

PR #386 のリモートレビューで指摘された、実行開始の保存失敗後にも開始時刻・経過時間が残る問題を db182b3 で修正しました。保存失敗時だけ仮の時刻を取り消し、保存後のキャンセルは既存どおり扱います。

回帰テストで修正前の失敗を再現し、修正後はパッケージ1,228件・アプリ42件が成功しました。固定した main に対するローカル再レビュー E7338CAB-0C1F-4A41-97DD-9EBADC8D84F4 は completed / 指摘なしです。指摘へ返信・resolve し、新しい HEAD を指定してリモートレビューを再依頼しました。再レビューと新しい CI の通過を確認してから #383 を統合し、#384 へ進みます。

[lynnswap]

#383 / PR #386 の再レビューで見つかった履歴 E2E のスキーマ期待値を修正しました（1b03667）。本番 DDL で作った一時 DB に実際の検査関数を適用し、修正前の失敗と修正後の成功を確認済みです。

main 比較のローカル Codex review 689B02B6-8B95-4493-8F58-CACCBC10B222 は completed / no findings。最新 HEAD のリモート再レビューと CI を待っています。#386 の統合後、順序どおり #384 に進みます。

[lynnswap]

#383 / PR #386 の診断出力の更新漏れを修正しました（5f274fa）。backend の開始待ち中にも running・開始時刻・表示文言が診断ファイルへ反映されることを回帰テストで確認しました。

パッケージ1,229件・アプリ42件が成功し、main 比較のローカル Codex review 3F771241-88DD-4F1D-B8D2-BE597FD708C2 は completed / no findings。最新 HEAD のリモート再レビューを依頼済みです。#386 統合後に #384 へ進みます。

[lynnswap]

#383 を PR #386 で main に統合しました（6c352bb）。最新 HEAD のリモート Codex レビュー完了・指摘なし、未解決スレッドなし、全5 CI 成功を確認済みです。

次は #384 の Store 所有の更新操作に進みます。実行中レビューの終了と後片付けを待ち、MCP と待機ジョブを維持して app-server の更新・再起動・キュー再開を行います。

[lynnswap]

#384 の Ready PR #387 を作成しました。Store が新規要求をキューに保持し、実行中レビューの完了後に一度だけ更新・app-server 再起動・キュー再開を行います。MCP の同じ呼び出しが更新をまたいで完了する統合テストも追加しています。

パッケージ1,248件・アプリ42件が成功し、ローカル Codex review 238A5CDC-0203-488F-A756-96437B50D987 は main 比較で completed / no findings でした。リモートレビューと CI の通過後に統合し、#385 の UI と起動時・8時間周期・手動チェックへ進みます。

[lynnswap]

PR #387 のリモート指摘2件を修正し、停止時に更新の依存待機から抜けられること、停止要求後にキューを再開しないことを確認しました。追加レビューで確認した終了失敗後の復旧も修正し、記録済みの終了エラーと現在のプロセス終了状態を分けて扱います。

最新 HEAD は 8ff02db。パッケージ1,253件・アプリ42件が成功し、ローカル Codex review E0323A94-F76D-4516-BA29-B429863ADE89 は completed / no findings。リモートの両スレッドに返信・resolve し、この HEAD の再レビューを依頼済みです。

[lynnswap]

#384 を PR #387 で main に統合しました（a5f31e2）。最新 HEAD のリモート Codex レビュー完了・指摘なし、未解決スレッドなし、全5 CI 成功を確認済みです。

最後の #385 に進み、後回し更新の UI、起動時・8時間周期と設定画面の手動確認、Preview を接続し、Monitor 全体を再起動する旧更新経路を削除します。

[lynnswap]

最後の #385 を Ready PR #388 に提出しました。後回し・即時更新を Store API に接続し、サイドバーの待機・更新・再試行表示、起動時と起動から8時間ごとの確認、Settings → Updates の手動確認を実装しました。旧アプリ再起動ヘルパーは削除しています。

パッケージ1,254件・アプリ44件が成功し、ローカルレビュー AA30B898-C740-4EF4-8C60-AD0FB71DF9D4 は completed / no findings。更新状態と設定画面の Preview を追加済みで、Store を通した UI シナリオも検証済みです。Canvas 自体の描画は Xcode の StructuredQueries マクロ再承認待ちです。

リモートレビューと CI の成功後に統合し、main の最終検証と親 Issue の完了確認へ進みます。

[lynnswap]

#385 / PR #388 を main に統合しました（15bc72add2c570c65bf800cec28674dabe092765）。対象 HEAD 4d7c3da のリモート Codex レビュー完了・指摘なし・未解決スレッドなしと、全5件の CI 成功を確認済みです。

3つの子 Issue の実装が揃いました。統合後の main でパッケージ・アプリの全テストと、同一 MCP セッションのキュー受付→更新1回→元の呼び出し完了のシナリオを最終確認します。

[lynnswap]

#386・#387・#388 をローカル／リモート Codex レビューと全 CI 成功後に main へ統合し、子 Issue #383・#384・#385 はすべて closed になりました。

最終 main 15bc72add2c570c65bf800cec28674dabe092765 で確認しました。

• パッケージテスト 1,254件成功。実際の MCP HTTP を通した A 実行→B/C キュー受付→更新1回→同一セッション・ジョブ ID の元の呼び出し完了のシナリオも成功。
• ReviewMonitor アプリテスト 44件成功（macOS arm64、CI と同じ -skipMacroValidation）。起動時・8時間周期、手動確認による周期維持、スリープ復帰、重複確認の合流、設定画面と checker の共有を確認。
• キューの履歴・キャンセル・失敗時の保持と復旧、後回し／即時更新、Store を通す更新 UI の回帰テストが成功。
• ローカル main は origin/main と一致し、作業ツリーは clean。

Preview ソースは追加・コンパイル済みです。Xcode Canvas の描画は StructuredQueries マクロの再承認待ちで未確認です。更新処理・transport を差し替えて本番の data flow を検証しており、実際の Codex CLI 更新や稼働中 Monitor の置換・停止は行っていません。

承認済み設計の完了条件を満たしたため、この親 Issue とフォローアップを終了します。