docs(todo): 順位251 完了削除 + PR #239 feedback T3-1/T3-2 採用 (順位252/253 登録)

[aloekun]

概要

順位 251 (非 colocated jj workspace 対応) の完了削除と、PR #239 post-merge-feedback の採用 2 件 (T3-1/T3-2 = 順位 252/253) の todo 登録。docs-only PR。

変更内容

順位 251 の完了削除 (todo13.md エントリ + todo-summary.md 行)

• PR feat(jj-workspace): 非 colocated workspace の gh 全損を GIT_DIR 自動注入で解消 + GH_REPO guard (順位251) #239 で GIT_DIR 自動注入 / gh-repo-env-guard preset / stdout 分離パース / skip 時 marker として land 済み
• 完了基準 (非 colocated workspace からの監視完走 + post-merge feedback 自動発火) を PR feat(jj-workspace): 非 colocated workspace の gh 全損を GIT_DIR 自動注入で解消 + GH_REPO guard (順位251) #239 自身の運用サイクルで実確認済み (feedback report .claude/feedback-reports/239.md 生成を確認)

順位 252 登録 — 部分効果 env var anti-pattern の文書化 (feedback T3-1 採用)

• サブコマンドで効果が異なる env var (GH_REPO 型) の危険性を新 ADR に codify する task。順位 135 の placeholder policy 適用 (番号は land 時確定)

順位 253 登録 — ADR-030 への feedback silent skip 実装記録追記 (feedback T3-2 採用)

• owner_repo 検出失敗 → marker 未書込 → L2 recovery 未発動シナリオと AiStepContext::SkipWithMarker による対処の記録 task

検証

• markdownlint 83 files 0 errors / pnpm lint:docs OK (cross-ref + priority-inversion)
• pre-push review: simplicity-review / security-review ともに approved

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • タスク一覧を更新し、既存の1件を整理したうえで、関連する2件の新しい記録タスクを追加しました。
  • 新たに、環境変数の使い方に関する注意点の文書化と、既存ADRへの実装記録の追記が計画に含まれました。
  • 進行順の見直しにより、後続作業の整理が行われました。

[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: 1f4c92fc-c339-493f-a39f-9f7bb9e7b931

📥 Commits

Reviewing files that changed from the base of the PR and between c063787 and f41d5e8.

📒 Files selected for processing (2)

• docs/todo-summary.md
• docs/todo13.md

---

📝 Walkthrough

Walkthrough

docs/todo-summary.md と docs/todo13.md のタスク管理ドキュメントを更新。PR #238関連の非colocated jj workspace対応の記録セクションを削除し、PR #239対応として部分効果env var anti-patternのADR文書化タスクとADR-030へのfeedback silent skip実装記録追記タスクを新規追加した。

Changes

ドキュメントタスク更新

Layer / File(s)	Summary
推奨実行順序サマリーの更新 docs/todo-summary.md	順位251のタスク行を削除し、PR #239対応の順位252（部分効果env var anti-pattern文書化）・253（ADR-030へのfeedback silent skip記録追記）を追加。
Part 13タスク詳細の更新 docs/todo13.md	PR #238のnon colocated jj workspace対応記録セクションを削除し、部分効果env var anti-patternのADR化タスクとADR-030実装記録追記タスク（AiStepContext::SkipWithMarker言及含む）の作業計画・完了基準を追加。

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

Possibly related PRs

• aloekun/claude-code-hook-test#133: 同じ docs/todo-summary.md の「推奨実行順序サマリー」table を対象に変更を加えている。
• aloekun/claude-code-hook-test#183: 本PRの「ADR-030へfeedback silent skip記録を追記」タスクと同じADR-030ファイルを直接編集している。

🚥 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	PR #239で完了した251番の削除と、252/253番の新規登録を具体的に示しており、変更内容と一致しています。
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.}