docs: ADR-045 に jj workspace の PR 運用知見を追記 + 改善タスク 3 件登録 (順位 233-235)

[aloekun]

概要

PR #227（jj workspace での初 PR 運用ケース）の dogfood で得た知見を docs に反映する。独立した 2 つの変更を含む docs-only PR。

変更内容

1. ADR-045 に PR 運用知見を追記 (docs/adr/adr-045-jj-workspace-parallel-sessions.md)

jj workspace の初 PR 運用で判明した、ADR-045 に未記載だった運用上の制約を追記:

• secondary workspace の .git 不在: jj workspace add で作った workspace は colocated にならず .git を持たないため、gh ベースのコマンド（pnpm create-pr / pnpm merge-pr / cli-pr-monitor --monitor-only）が fatal: not a git repository で失敗する。pnpm push（jj git push backend）だけが動く非対称が初回検証（push 中心）で見落とされていた
• GIT_DIR 回避策: メインリポジトリの .git を GIT_DIR で参照する（Bash / PowerShell のコマンド例を併記）。jj は GIT_DIR を無視するため gh だけを制御し、jj 操作・monitor state 分離には影響しない
• merge-pipeline の bookmark 誤検出: lib-jj-helpers の BOOKMARK_SEARCH_REVSETS（@/@-/@--）の近接性衝突の注意（PR head 以外の bookmark を近接 revset に置くと誤検出）
• 恒久対策の候補: cli-* exe での GIT_DIR 自動注入 / direnv / colocated 化

2. 改善タスク 3 件を todo に登録 (docs/todo-summary.md / docs/todo13.md、順位 233-235)

• 順位 233: post-pr-review (takt) の docs-only 誤判定の修正 — PR 全体でなく @ コミットのみ見て誤分類するバグ（PR fix(cli-pr-monitor): 並列テスト flaky を tempfile 一意化と state path DI で構造的に解消 #227 観測）
• 順位 234: memory feedback-di-over-ambient-global-tests に serialization primitive 例外境界 + PR fix(cli-pr-monitor): 並列テスト flaky を tempfile 一意化と state path DI で構造的に解消 #227 具体例を追記（PR fix(cli-pr-monitor): 並列テスト flaky を tempfile 一意化と state path DI で構造的に解消 #227 post-merge-feedback T3-1 採用）
• 順位 235: ADR-022 に Serialization Primitive Single-Instance Rule の Appendix 追加（PR fix(cli-pr-monitor): 並列テスト flaky を tempfile 一意化と state path DI で構造的に解消 #227 post-merge-feedback T3-2 採用）

種別

docs-only（ADR / todo ドキュメントの更新のみ、コード変更なし）。

Summary by CodeRabbit

• Documentation
  • 並列セッション運用時の注意点を追記し、secondary workspace で .git がないケースや、PR 操作で必要になる環境変数設定例を明確化しました。
  • PR 監視・マージ時の挙動差や、誤検出を避けるための運用上の回避策を追加しました。
  • 今後対応予定の改善項目を TODO に追加し、実行順序の見直しや設計ルールの補足を整理しました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 19c891d6-492f-4517-a38f-4f4387ddd182

📥 Commits

Reviewing files that changed from the base of the PR and between 3d8e2aa and 0a60c54.

📒 Files selected for processing (3)

• docs/adr/adr-045-jj-workspace-parallel-sessions.md
• docs/todo-summary.md
• docs/todo13.md

---

📝 Walkthrough

Walkthrough

ADR-045 に secondary workspace で .git が存在しない問題を追記し、GIT_DIR の必要性・bookmark 誤検出の回避策・follow-up 候補を文書化した。あわせて docs/todo13.md と docs/todo-summary.md に PR #227 観測に基づく新規 TODO エントリ 3 件を追加した。

Changes

ADR-045 更新と関連 TODO 追加

Layer / File(s)	Summary
ADR-045: GIT_DIR 要件・bookmark 誤検出の追記 docs/adr/adr-045-jj-workspace-parallel-sessions.md	ステータス欄に 2026-06-30 初回 PR 運用で判明した .git 不在問題を反映。検証結果 caveat・選択肢 C 説明を更新し、PR 操作手順に GIT_DIR 設定例（Bash/PowerShell）、コマンド別要否表、bookmark 誤検出（BOOKMARK_SEARCH_REVSETS 近接探索）の回避策と follow-up 候補を追加した。
TODO エントリ追加 docs/todo13.md, docs/todo-summary.md	post-pr-review の diff scope を PR 全体へ変更する計画、feedback-di-over-ambient-global-tests memory への serialization primitive 例外境界追記、ADR-022 への "Serialization Primitive Single-Instance Rule" Appendix 追加計画を todo13.md に新規記載。todo-summary.md の推奨実行順序テーブルに順位 233〜235 として対応エントリを追加した。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#226: 同じ docs/adr/adr-045-jj-workspace-parallel-sessions.md を変更しており、本 PR はその上に運用 caveat を追記している。
• aloekun/claude-code-hook-test#225: docs/todo-summary.md と docs/todo13.md への ADR-022 関連 TODO 追記が重複領域にある。
• aloekun/claude-code-hook-test#64: ADR-022 の Principle 1・自動化責任境界を改訂しており、本 PR の ADR-022 Appendix 追加計画と直接関連する。

🚥 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	ADR-045 への運用知見追記と改善タスク3件の登録という変更内容を正確に要約しています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

---

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.}