@joshuafolkken/kit 1.1050.0 → 1.1094.0

package/prompts/collaboration-workflow/wip-cap.md

@@ -1,4 +1,4 @@
-# オープン Issue の WIP 上限（30 件）
+# オープン Issue の WIP 上限
 
 **単一ソースはこのファイルである。**
 
@@ -8,7 +8,7 @@
 
 ## 規則
 
-**オープン Issue が 30 件を超えている状態で新しく起票するときは、先に 1 件閉じる。**
+**オープン Issue が上限を超えている状態で新しく起票するときは、先に 1 件閉じる。**
 
 件数は毎回数える。見積もりでも記憶でもない。
 
@@ -19,7 +19,7 @@ gh api "repos/{owner}/{repo}/issues?state=open&per_page=100" --paginate \
 
 - **数えるのは「起票しようとしているリポジトリ」のオープン Issue 全件**である。`{owner}/{repo}` はカレントリポジトリに展開されるので、**別リポジトリへ起票するときはそのリポジトリのパスを直接書く**（例: `repos/joshuafolkken/app-kit/issues?...`）。自リポジトリの件数で他リポジトリへの起票を判断するのは、別のバックログを見ているのと同じである。
 - epic も子も含める。epic を除外すると、epic を作ることで上限を回避できてしまう。
-- **上限は 30。** 31 件目以降を作る前に 1 件閉じる。
+- **上限は 30。** これを超える 1 件を作る前に 1 件閉じる。エージェントが読む文書（`CLAUDE.md`・`prompts/`・配布 skill）がこの数字を書くのはこの行だけで、一次情報は `scripts/rules/delivered-rules.ts` の `WIP_CAP` である（一致はテストが固定する）。
 - **プルリクエストは数えない。** REST の `issues` エンドポイントは PR も返すため、`select(.pull_request == null)` で落とす。この 1 句を省くと、PR が開いているだけで上限に達する。
 - **`gh issue list` は使わない。** 配布ドキュメントの実行可能ブロックに置ける `gh` は REST のものだけで、`gh issue list` は GraphQL 経由のためクラウドセッションで 403 になる。
 
@@ -27,11 +27,11 @@ gh api "repos/{owner}/{repo}/issues?state=open&per_page=100" --paginate \
 
 測定は `docs/maintainers/wip-cap-rationale.md` →「なぜ上限が要るのか — 測定」。
 
-**上限は、増加を見えるようにするための強制装置である。** 数を減らすこと自体が目的ではない。30 を超えた状態で起票しようとしたときに「本当にこれは 1 件増やす価値があるか」を必ず 1 回考えさせる、そこだけが役割である。
+**上限は、増加を見えるようにするための強制装置である。** 数を減らすこと自体が目的ではない。上限を超えた状態で起票しようとしたときに「本当にこれは 1 件増やす価値があるか」を必ず 1 回考えさせる、そこだけが役割である。
 
 ## 超過時の手順
 
-起票しようとした時点で 30 件を超えていたら、**その起票が裁量によるものか、実行がそれに詰まっているか、割り込みか**の 3 つに分岐する。判断ではなく、起票の種類で決まる。**上限が効くのは裁量側だけである。**
+起票しようとした時点で上限を超えていたら、**その起票が裁量によるものか、実行がそれに詰まっているか、割り込みか**の 3 つに分岐する。判断ではなく、起票の種類で決まる。**上限が効くのは裁量側だけである。**
 
 **判定するのは「起票 1 件」ではなく「起票のひとまとまり」である。** 分割判定が作る子 N 件とその epic、割り込み Issue とその上流バックリンク、起票直後の `epic:bundle` が `create_epic` と答えたときの epic — これらは 1 回の起票行為であり、途中で上限に当たって半分だけ作るのが最悪の結果になる。**ひとまとまりの先頭で 1 回だけ判定し、通ったらまとまり全体を作り切る。**
 
@@ -71,36 +71,39 @@ gh api "repos/{owner}/{repo}/issues?state=open&per_page=100" --paginate \
 
 #### 該当したときの手順
 
-1. **上限に関係なく起票する。** 1 件閉じることを条件にしない。起票は `pnpm josh issue:file` で行う。本文の検査、本文が宣言する分類ラベルの付与、重複探し、`epic:bundle` はこのコマンドが実行する（`docs/josh-commands.md` → `josh issue:file`）。
+1. **上限に関係なく起票する。** 1 件閉じることを条件にしない。起票は `pnpm josh issue:file` で行う。本文の検査、本文が宣言する分類ラベルの付与、重複探し、`epic:bundle` はこのコマンドが実行する（`docs/josh-commands-automation.md` → `josh issue:file`）。
 
    ```bash
    pnpm josh issue:file "<title>" --body-file <body-file> --depth <n> --route interrupt
    ```
 
 2. **超過している事実と、当たった条件の番号を本文に 1 行書く**（例:「オープン 37 件で起票。3 条件のうち 2（文書化された手順が完了できない）に該当」）。**どちらか一方では足りない** — 件数だけでは判定が再現できず、条件だけでは上限の可視化が効かない。
-3. **次に実行されるものとして挿入する。** epic の下にあるなら `pnpm josh epic --add <E> <N>` で、**位置を指定せずに**追加する（`<E>` が epic、`<N>` が起票した割り込み）。**`--before` / `--after` を使ってはならない。** それらは `blocked-by` 関係を実際に書き込むため（`docs/josh-commands.md` →「`josh epic --add`」）、「発見したランをブロックしていない欠陥」という割り込みの定義と矛盾する**偽の依存関係**になり、`epic:next` が実装中の子を「割り込みにブロックされている」として出さなくなる。位置を指定しない追加は依存を一切宣言しない（joshuafolkken/kit#1253）ので、**順序は実行側が持つ** — 次に取り上げるのは実行側の選択であって、宣言された関係ではない（経緯は `docs/maintainers/wip-cap-rationale.md` →「位置を指定しない追加の経緯」）。**先に提示させたいときは `--order-before <M>` を使う**（joshuafolkken/kit#1738）— これはタスクリストの行だけを動かし、依存宣言も `blocked-by` も一切書かないので、上の禁止には当たらない。**禁止しているのは依存を書き込む `--before` / `--after` であって、順序そのものではない。** `epic:next` は runnable な子をタスクリスト順に提示する（joshuafolkken/kit#1583）ため、位置を指定しない追加は「末尾に提示される」を意味する — 割り込みを次に取り上げてほしいなら、行だけを前に出すこの指定がその手段である。**epic の下に無いなら挿入先そのものが無い**ので、この手順は飛ばして 4 だけを行う。
+3. **次に実行されるものとして挿入する。** epic の下にあるなら `pnpm josh epic --add <E> <N>` で、**位置を指定せずに**追加する（`<E>` が epic、`<N>` が起票した割り込み）。**`--before` / `--after` を使ってはならない。** それらは `blocked-by` 関係を実際に書き込むため（`docs/josh-commands-automation.md` →「`josh epic --add`」）、「発見したランをブロックしていない欠陥」という割り込みの定義と矛盾する**偽の依存関係**になり、`epic:next` が実装中の子を「割り込みにブロックされている」として出さなくなる。位置を指定しない追加は依存を一切宣言しない（joshuafolkken/kit#1253）ので、**順序は実行側が持つ** — 次に取り上げるのは実行側の選択であって、宣言された関係ではない（経緯は `docs/maintainers/wip-cap-rationale.md` →「位置を指定しない追加の経緯」）。**先に提示させたいときは `--order-before <M>` を使う**（joshuafolkken/kit#1738）— これはタスクリストの行だけを動かし、依存宣言も `blocked-by` も一切書かないので、上の禁止には当たらない。**禁止しているのは依存を書き込む `--before` / `--after` であって、順序そのものではない。** `epic:next` は runnable な子をタスクリスト順に提示する（joshuafolkken/kit#1583）ため、位置を指定しない追加は「末尾に提示される」を意味する — 割り込みを次に取り上げてほしいなら、行だけを前に出すこの指定がその手段である。**epic の下に無いなら挿入先そのものが無い**ので、この手順は飛ばして 4 だけを行う。
 4. **着手は、既にある承認の内側でのみ、指示を待たずに行う。** `backlogrun` の中なら、打たれたキーワードがバッチを承認しているので、割り込みは次の子として着手してよい。**バッチの外では `CLAUDE.md` →「Explicit invocation required」が変わらず効く** — 起票と挿入まで済ませたうえで、利用者に打つべきコマンド（`fullrun #<N>`）を報告して止まる。**割り込みは上限の例外であって、明示起動規則の例外ではない。**
 
 **起票と、実行されるようにするところまでが 1 つのまとまりである。** epic の下にある割り込みを起票だけして追加しなければ、`epic:next` はそれを候補にすら出さない — Issue は残るが実行されないという、コメントに埋もれるのと大差ない状態になる。**epic の外にある割り込みでは、まとまりの後半は 4 の報告そのもの**であり、打つべきコマンドを利用者に渡すまでが 1 回の行為である。上の「ひとまとまりで判定する」がここにも効く。
 
-#### 実行のしかた — 検証経路そのものの欠陥は単独で走らせる
+#### 実行のしかた — `main` で誤答している検証の欠陥は単独で走らせる
 
-**割り込みの対象が検証経路そのものの欠陥であるとき、その Issue は単独で実行し、バッチを再開するのはそれが `main` へマージされてからとする。** 「着地」はマージであって、PR を開いた時点でもレビューが収束した時点でもない。
+**kit 自身の検証が今の `main` で無関係な PR の判定まで誤らせている欠陥を直す Issue は単独で実行し、バッチを再開するのはそれが `main` へマージされてからとする。** 「着地」はマージであって、PR を開いた時点でもレビューが収束した時点でもない。
 
-**該当するかは判断ではなく、影響先の列挙で決まる。** 欠陥が次のいずれかに影響するか:
+**該当するかは判断ではなく、次の 3 条件で決まる。**
 
-- **検証ゲート**（lint / 型チェック / スペルチェック / 単体テスト）
-- **コードレビュー**
-- **push 時のフック**
-- **マージ時のチェック**
+1. **欠陥である**（`- 種別: 不具合`）。改善・リファクタ・削除・機能追加は当たらない。
+2. **kit 自身の検証の欠陥である。** 検証とは次の 4 つを指し、配布先リポジトリ向けの CI・テンプレートの不具合は当たらない。
+   - **検証ゲート**（lint / 型チェック / スペルチェック / 単体テスト）
+   - **コードレビュー**
+   - **push 時のフック**
+   - **マージ時のチェック**
+3. **今の `main` で、その Issue と無関係な PR の判定まで誤らせている**（嘘の緑 / 嘘の赤）。効率の問題や、特定の条件でしか起きないものは当たらない。
 
-**いずれかに当たれば単独実行。** その Issue に `run:solo` を付け、強制はそれを読む `backlog:next` と `epic:next --lanes` が行う（joshuafolkken/kit#2776）。当たらなければ（機能追加、文書修正など）`run:lane` を付け、並列バッチの一員として走らせてよい。どちらもなければ `triage` が返り何も始まらない（joshuafolkken/kit#2779）ので、run が起票する Issue（割り込み・分割・前提）は起票時にどちらかを付ける。**「重大だと感じるか」は判定条件ではない**（3 条件と同じ理由）。
+**3 条件すべてに当たれば単独実行、1 つでも外れれば `run:lane`。** 単独実行なら `run:solo` を付け、強制はそれを読む `backlog:next` と `epic:next --lanes` が行う（joshuafolkken/kit#2776）。外れれば（改善、機能追加、文書修正、配布先 CI の修正など）`run:lane` を付け、並列バッチの一員として走らせてよい。どちらもなければ `triage` が返り何も始まらない（joshuafolkken/kit#2779）ので、run が起票する Issue（割り込み・分割・前提）は起票時にどちらかを付ける。**「重大だと感じるか」は判定条件ではない**（起票側の 3 条件と同じ理由）。
 
-**理由は 2 つあり、どちらか一方だけでも単独実行の根拠として十分である。** その 2 つと、判定を列挙にした由来は `docs/maintainers/wip-cap-rationale.md` →「単独実行の理由と由来」。
+**他レーンを止める根拠は 1 つ — 壊れた検証の上でバッチを回すと、全員の結果が信用できないこと。** 割り込み自身の検証が欠陥の影響下にあることはその Issue 自身の扱い（修正後の検証で確かめる）であって、他レーンを止める根拠にはしない。根拠と、判定を条件にした由来・狭めた経緯は `docs/maintainers/wip-cap-rationale.md` →「単独実行の理由と由来」。
 
 ## 上限そのものを変えるとき
 
-**30 という数字は Tier C ではない** — 引き上げも引き下げも、測定を添えて提案してよい。ただし**変更は必ずこのファイルとマーカーテストの両方を同じコミットで動かす**。片方だけ動かすと、常駐側が存在しない数字を指す。
+**上限の数字は Tier C ではない** — 引き上げも引き下げも、測定を添えて提案してよい。ただし**変更は必ず `WIP_CAP` と「規則」節の 1 行を同じコミットで動かす**。片方だけ動かすと一致を固定するテストが落ちる。
 
 **引き上げる条件は「窮屈だから」ではない。** オープン件数が 7 日移動平均で減り続けていて、なお上限が起票を止めているとき — つまり装置が目的を果たし終えたときだけである。
 
@@ -108,8 +111,8 @@ gh api "repos/{owner}/{repo}/issues?state=open&per_page=100" --paginate \
 
 上限は単独では効かない。**製造側を止めないまま上限だけ入れると、製造は続いたまま実行だけが止まる。** 経緯は `docs/maintainers/wip-cap-rationale.md` →「3 つを 1 コミットで着地させた経緯」。
 
-| 製造元                    | 変更                                              | 単一ソース                                                              |
-| ------------------------- | ------------------------------------------------- | ----------------------------------------------------------------------- |
-| 分割判定                  | 既定を「分割しない」に。目安 10 ファイル / 400 行 | `.claude/skills/workflow-commands/split-assessment.md` → "The question" |
-| レビュー round 2 の残指摘 | 既定を branch 3（PR に 1 行残して落とす）に       | `prompts/review.md` → "Three-way disposition after the cap"             |
-| オープン件数              | 30 件の WIP 上限                                  | このファイル                                                            |
+| 製造元                    | 変更                                           | 単一ソース                                                              |
+| ------------------------- | ---------------------------------------------- | ----------------------------------------------------------------------- |
+| 分割判定                  | 既定を「分割しない」に。目安は単一ソースにある | `.claude/skills/workflow-commands/split-assessment.md` → "The question" |
+| レビュー round 2 の残指摘 | 既定を branch 3（PR に 1 行残して落とす）に    | `prompts/review.md` → "Three-way disposition after the cap"             |
+| オープン件数              | WIP 上限                                       | このファイル                                                            |

package/prompts/collaboration-workflow.md

@@ -2,11 +2,11 @@
 
 <!-- cspell:words coderabbit -->
 
-このドキュメントは、Claude / Cursor / Gemini を含む複数の AI ツールで共通に使う Issue 駆動の共同作業ワークフローの**正典**である。
+このドキュメントは、Claude / Cursor / Gemini を含む複数の AI ツールで共通に使う Issue 駆動の共同作業ワークフローの**索引**である。
 
 ## この索引の使い方
 
-**これは人間が読むための正典であり、実行中の参照先ではない。** 実行中の操作手順は `.claude/skills/workflow-commands/` にあり、規則そのものは `CLAUDE.md` にある。両者とこの正典は一致していなければならない。実行中にこの正典を開く必要があるのは、skill と規則の言い分が食い違ったときだけである。
+**これは人間が読むための索引であり、規則を定める場所ではない。** 常駐の引き金は `CLAUDE.md` に、実行中の操作手順は `.claude/skills/workflow-commands/` に、各話題の規則は下の表のファイルにあり、それぞれが自分の規則の一次情報である。この索引はそれらを言い直さない。
 
 **話題ごとにファイルが分かれている。** 以前はこの内容が 1 本の 169KB のファイルにあり、1 つの節を確かめるだけでも全体を読む必要があった。読んだものはそのセッションの残り全ターンで積み上がった前置きとして課金され続けるため、確認 1 回の費用が会話の長さに比例して効いていた（joshuafolkken/kit#965）。**必要な 1 本だけを開くこと。この索引を入口に、下の表から選ぶ。**
 
@@ -17,11 +17,8 @@
 | Step 2: 提案依頼（AI 共通）                                                                       | [`proposal-request.md`](./collaboration-workflow/proposal-request.md)     |
 | Step 3: 計画コメントを記録して通知する                                                            | [`plan-comment.md`](./collaboration-workflow/plan-comment.md)             |
 | 報告フォーマット（平易な概要 ＋ 技術詳細）                                                        | [`report-format.md`](./collaboration-workflow/report-format.md)           |
-| セッション向け出力で Issue はリンク＋短い日本語要約で参照する                                     | [`issue-citation.md`](./collaboration-workflow/issue-citation.md)         |
-| 後から関連が判明した Issue を epic に束ねる（本文は `epic-commands` skill）                       | [`epic-bundle.md`](./collaboration-workflow/epic-bundle.md)               |
-| オープン Issue の WIP 上限（30 件）                                                               | [`wip-cap.md`](./collaboration-workflow/wip-cap.md)                       |
-| リポジトリをまたぐ EPIC（本文は `epic-commands` skill）                                           | [`cross-repo-epic.md`](./collaboration-workflow/cross-repo-epic.md)       |
-| `josh epic:audit` — 子 Issue 群を横断して矛盾を検出する（本文は `epic-commands` skill）           | [`epic-audit.md`](./collaboration-workflow/epic-audit.md)                 |
+| セッション向け出力で Issue はリンク＋セッション言語の短い要約で参照する                           | [`issue-citation.md`](./collaboration-workflow/issue-citation.md)         |
+| オープン Issue の WIP 上限                                                                        | [`wip-cap.md`](./collaboration-workflow/wip-cap.md)                       |
 | 別パッケージ起因の問題は割り込み Issue で対応する                                                 | [`upstream-interrupt.md`](./collaboration-workflow/upstream-interrupt.md) |
 | 原則 — クローン禁止・シンプル第一・相談と実行の区別・最新優先・恒久ルールと配布物と規則の置き場所 | [`principles.md`](./collaboration-workflow/principles.md)                 |
 | ファイル編集はコマンド本文に本文を載せない                                                        | [`file-edits.md`](./collaboration-workflow/file-edits.md)                 |
@@ -37,7 +34,7 @@
 
 ## 参照の書き方
 
-`CLAUDE.md` と `.claude/skills/` からこの正典を指すときは、**本文があるファイルを直接指す**（話題ファイル、skill へ単一ソース化済みならその skill。[`residency.md`](./collaboration-workflow/residency.md)）。索引や指し先を経由させる形（`prompts/collaboration-workflow.md` → 「節名」）は、読み手に 2 回読みを強いるうえ、節名が変わると黙って行き止まりになる。
+`CLAUDE.md` と `.claude/skills/` からこれらの話題を指すときは、**本文があるファイルを直接指す**（話題ファイル、skill へ単一ソース化済みならその skill。[`residency.md`](./collaboration-workflow/residency.md)）。索引や指し先を経由させる形（`prompts/collaboration-workflow.md` → 「節名」）は、読み手に 2 回読みを強いるうえ、節名が変わると黙って行き止まりになる。
 
 ```md
 See `prompts/collaboration-workflow/upstream-interrupt.md`.

package/prompts/review-rubric.md

@@ -74,7 +74,7 @@ code changed, because returning none is what the instruction rules out. So a Med
 the second round even where the fixes were sufficient. The second round is narrowed to remove that.
 
 **What changes is the question, never the standard.** A confirmed High still blocks the merge
-whatever the round count, the cap is still two rounds, and every rule here applies to this pass
+whatever the round count, the cap in `prompts/review.md` → "Review round cap" still holds, and every rule here applies to this pass
 unchanged.
 
 |                | The first round      | The second round                                                                                                                                                               |

package/prompts/review.md

@@ -27,8 +27,8 @@ points to them rather than restating them:
 The workflow review uses the `reviewer` role profile printed by `pnpm josh review:brief`. Pass its
 model and effort explicitly to the review subagent; do not substitute the review level for effort.
 
-Re-run after applying fixes until **no high or medium findings remain — or until two reviews have run
-in total, the first included — whichever comes first.** The cap below is not optional. The second round
+Re-run after applying fixes until **no high or medium findings remain — or until the round cap below is
+reached, the first review included — whichever comes first.** The cap below is not optional. The second round
 is a verification pass over the fixes, not the first review again
 (`prompts/review-rubric.md` → "The second round is a verification pass, not a second full review").
 
@@ -197,7 +197,7 @@ Only branch 2 files an Issue. What follows applies to that branch.
   1. File the follow-up Issue referencing the current one, tagged `route:review-cap` — **before the current Issue closes.**
      `pnpm josh issue:file "<title>" --body-file <body-file> --depth <n> --route review-cap`.
      The command lints the body, applies the classification labels it declares, runs the duplicate scan
-     and then `epic:bundle` on the new Issue (`docs/josh-commands.md` → `josh issue:file`). The
+     and then `epic:bundle` on the new Issue (`docs/josh-commands-automation.md` → `josh issue:file`). The
      `epic:bundle` candidate search reads open issues only, so once the parent has closed it answers
      `none` permanently.
   2. Act on the `epic:bundle` answer it prints. **`epic:bundle` recommends and writes nothing**, so acting means running the

package/prompts/sonar-hotspot-handling.md

@@ -9,9 +9,13 @@ AI tools cannot perform OAuth browser login to SonarCloud, so this guide assumes
 ## Quality Gate enforcement (CI blocks PRs)
 
 The distributed `.github/workflows/sonar-qube.yml` runs the SonarCloud scan with
-`-Dsonar.qualitygate.wait=true`. The scanner waits for SonarCloud to evaluate the project's
-Quality Gate and **fails the `SonarQube` job — a required PR check — when the gate is red**, so a
-PR that introduces a finding cannot be merged green. This is the single enforcement point:
+`-Dsonar.qualitygate.wait=true`, so the `SonarQube` job — a required PR check — fails when the
+Quality Gate is red. **The gate alone is not enough**: `Sonar way` judges ratings and ratios, so a
+few MINOR code smells keep the maintainability rating at A and a small copied block stays under the
+duplication threshold. On a pull request the job therefore ends with `pnpm josh sonar:new-code <PR>`,
+which **fails on a single new issue of any type or severity, or a single new duplicated block**
+(joshuafolkken/kit#3045). An issue resolved in SonarCloud as accepted or a false positive is not
+counted — that resolution is the route for a deliberate exception. This job is the single enforcement point:
 `josh followup` does **not** scan SonarCloud findings (it only scans CodeRabbit / Claude Review
 comments), but it **does** wait on the required `SonarQube` CI check, so a red gate also blocks the
 `followup` merge. Consumers pick the workflow up verbatim via `josh sync`; do not re-implement the
@@ -25,6 +29,10 @@ run without the secret in the Dependabot store), so a repository that relies on
 
 ### New Code vs Overall Code semantics
 
+The scan passes `package.json`'s `version` as `sonar.projectVersion`, so New Code — "since the
+previous version" — restarts at every `josh bump` that reaches main instead of growing from the
+first analysis on.
+
 SonarCloud's default **Sonar way** Quality Gate evaluates **New Code only** — conditions
 (new bugs, new code smells above the Maintainability rating, new coverage/duplication) apply to
 lines changed in the PR's New Code period, not the whole codebase. Consequences:

package/prompts/testing-guide.md

@@ -264,9 +264,9 @@ a repository whose CI reports no `E2E` context at all resolves the required entr
 which is never `pass`, so the evaluation stays `pending` until the 32-minute wait times out with
 nothing actually wrong. What guarantees the job runs is the detection rule above, not the required
 list. A project that wants it required opts in through `JOSH_REQUIRED_CHECKS`, which
-`scripts/git/git-pr-checks-eval.test.ts` covers; the decision itself — `E2E` off the default list,
+`scripts/gh/git-pr-checks-eval.test.ts` covers; the decision itself — `E2E` off the default list,
 and a repository reporting no `E2E` check still mergeable — is pinned in
-`scripts/git/git-pr-checks-e2e-gate.test.ts`.
+`scripts/gh/git-pr-checks-e2e-gate.test.ts`.
 
 **A red E2E is reported as soon as it goes red, and by name.** `E2E` is not on the required list, so
 until joshuafolkken/kit#990 nothing ended the wait when it failed: the rollup read as pending,
@@ -275,4 +275,4 @@ to complete.` — a message naming neither the job nor the cause. Any failing ch
 on the poll that sees it, and the error names it (`PR checks failed (failed checks: E2E).`).
 The poll loop keeps CodeRabbit exempt under the temporary kit#753 policy. **This is a change to how fast the gate
 reports, never to what it lets through** — no failing check gained a path to `success`, which
-`scripts/git/git-pr-checks-e2e-gate.test.ts` continues to assert.
+`scripts/gh/git-pr-checks-e2e-gate.test.ts` continues to assert.

package/scripts/adopt/adopt-toolkits.ts

@@ -167,7 +167,6 @@ const adopt_toolkits = {
 	TOOLKIT_SCOPE,
 	LATEST_VERSION,
 	is_toolkit_package,
-	compare_toolkits,
 	declared_toolkits,
 	misplaced_toolkits,
 	pick_bin_name,
@@ -175,7 +174,6 @@ const adopt_toolkits = {
 	read_bin_name,
 	all_declared_toolkits,
 	is_unreachable_toolkit,
-	resolve_toolkit,
 	discover_toolkits,
 	unresolved_toolkits,
 }

package/scripts/adopt/adopt.ts

@@ -156,13 +156,11 @@ function main(argv: ReadonlyArray<string>): void {
 }
 
 const adopt = {
-	DRY_RUN_REASON,
 	KNOWN_FLAGS,
 	NOTHING_TO_ADOPT,
 	parse_options,
 	run_here,
 	run,
-	main,
 }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) main(process.argv.slice(ARGV_OFFSET))

package/scripts/agent/agent-argv.ts

@@ -126,8 +126,6 @@ function resolve_in(
 }
 
 const agent_argv = {
-	build,
-	build_resume,
 	resolve,
 	resolve_in,
 	resume_argv,

package/scripts/agent/agent-diagnostics.ts

@@ -115,7 +115,7 @@ function check(profile: AgentProfile, ports: DiagnosticPorts = default_ports): D
 	return check_auth(cli, ports)
 }
 
-const agent_diagnostics = { AUTH_ARGS, MODEL_CLI_FLOORS, PROVIDER_CLI_TABLE, VERSION_ARGS, check }
+const agent_diagnostics = { check }
 
 export type { DiagnosticPorts, DiagnosticResult, ProcessOutcome }
 export { agent_diagnostics }

package/scripts/agent/agent-role-profile.ts

@@ -302,17 +302,11 @@ function parse(value: unknown): AgentProfile | undefined {
 
 const agent_role_profile = {
 	DEFAULT_PROFILES,
-	EFFORT_SCHEMA,
-	ENV_KEYS,
 	HANDED_PROVIDER_KEY,
 	IMPLEMENTATION_PHASE,
-	LEGACY_WORKER_KEYS,
 	MAX_VALUE_LENGTH,
 	OPENAI_PROFILES,
-	PHASE_EFFORT,
-	PHASE_SCHEMA,
 	PRE_GATE_PHASE,
-	PROVIDER_SCHEMA,
 	PROFILE_SCHEMA,
 	REVIEWER,
 	SCHEDULER,
@@ -322,7 +316,6 @@ const agent_role_profile = {
 	has_completion_callback,
 	is_safe_value,
 	parse,
-	phase_effort,
 	resolve,
 	resolve_provider,
 	with_phase_effort,

package/scripts/agent/claude-agent-argv.ts

@@ -143,8 +143,6 @@ const claude_agent_argv = {
 	build,
 	build_resume,
 	resolve,
-	tool_flags,
-	with_profile,
 }
 
 export type { ClaudeArgv, ClaudeArgvResult }

package/scripts/agent/codex-agent-argv.ts

@@ -88,7 +88,7 @@ function build(invocation: string, profile: AgentProfile, cwd?: string): CodexAr
 	}
 }
 
-const codex_agent_argv = { AGENT_COMMAND, AGENT_FLAGS, build }
+const codex_agent_argv = { build }
 
 export type { CodexArgv }
 export { codex_agent_argv }

package/scripts/agent/codex-hooks.ts

@@ -0,0 +1,92 @@
+import { hook_launch } from '#scripts/init/hook-launch'
+
+// `.codex/hooks.json` is derived from `.claude/settings.json`, never written by hand
+// (joshuafolkken/kit#2997). The hand copy carried every hook command twice, so a change to one side
+// left the other silently stale. Codex differs from Claude Code in three places only, and each is a
+// rule applied here rather than a line maintained in the copy:
+//
+// 1. The pretool and posttool handlers run through the Codex input adapter, which translates the
+//    `apply_patch` payload into the Edit/Write shape the Claude-side guards read.
+// 2. The PreToolUse and PostToolUse matchers also select `apply_patch`, Codex's edit tool.
+// 3. UserPromptSubmit carries no matcher — Codex documents that the event ignores it.
+
+interface HookHandler {
+	type: string
+	command: string
+	timeout?: number
+}
+
+interface MatcherGroup {
+	matcher?: string
+	hooks: Array<HookHandler>
+}
+
+type HookEvents = Record<string, Array<MatcherGroup>>
+
+interface HookConfig {
+	hooks: HookEvents
+}
+
+const CODEX_EDIT_TOOL = 'apply_patch'
+const TOOL_EVENTS: ReadonlySet<string> = new Set(['PreToolUse', 'PostToolUse'])
+const UNMATCHED_EVENTS: ReadonlySet<string> = new Set(['UserPromptSubmit'])
+const JSON_INDENT = '\t'
+
+function adapter_command(mode: string): string {
+	return hook_launch.bundle_launch_command(
+		`codex-hook-adapter.js ${mode}`,
+		`pnpm exec tsx scripts/hooks/codex-hook-adapter.ts ${mode}`,
+	)
+}
+
+// Each Claude-side guard command the adapter stands in for, keyed by that command verbatim.
+const ADAPTED_COMMANDS: ReadonlyMap<string, string> = new Map([
+	[
+		hook_launch.hook_launch_command('pretool-guard.js', 'pretool:guard'),
+		adapter_command('pretool'),
+	],
+	[
+		hook_launch.hook_launch_command('format-edited.js', 'format:edited'),
+		adapter_command('posttool'),
+	],
+])
+
+function codex_handler(handler: HookHandler): HookHandler {
+	return { ...handler, command: ADAPTED_COMMANDS.get(handler.command) ?? handler.command }
+}
+
+function codex_matcher(event: string, matcher: string | undefined): string | undefined {
+	if (UNMATCHED_EVENTS.has(event)) return undefined
+	if (matcher !== undefined && TOOL_EVENTS.has(event)) return `${matcher}|${CODEX_EDIT_TOOL}`
+
+	return matcher
+}
+
+function codex_group(event: string, group: MatcherGroup): MatcherGroup {
+	const matcher = codex_matcher(event, group.matcher)
+	const hooks = group.hooks.map((handler) => codex_handler(handler))
+
+	return matcher === undefined ? { hooks } : { matcher, hooks }
+}
+
+function codex_hook_config(claude: HookConfig): HookConfig {
+	const events = Object.entries(claude.hooks).map(([event, groups]) => [
+		event,
+		groups.map((group) => codex_group(event, group)),
+	])
+
+	return { hooks: Object.fromEntries(events) as HookEvents }
+}
+
+// The file text `.codex/hooks.json` holds — the formatter's own output for this JSON, so a write
+// followed by `pnpm josh format:edited` leaves it byte-identical.
+function codex_hooks_text(claude_settings_text: string): string {
+	const claude = JSON.parse(claude_settings_text) as HookConfig
+
+	return `${JSON.stringify(codex_hook_config(claude), undefined, JSON_INDENT)}\n`
+}
+
+const codex_hooks = { adapter_command, codex_hook_config, codex_hooks_text }
+
+export { codex_hooks }
+export type { HookConfig }

package/scripts/agent/interactive-ask.ts

@@ -86,6 +86,6 @@ function refused_ask_of(permission_denials: unknown): string | undefined {
 	return questions.map((question) => render_question(question)).join(QUESTION_SEPARATOR)
 }
 
-const interactive_ask = { INTERACTIVE_TOOLS, is_interactive_tool, refused_ask_of }
+const interactive_ask = { is_interactive_tool, refused_ask_of }
 
 export { interactive_ask }

package/scripts/auto-ok/auto-ok-cli.ts

@@ -3,19 +3,19 @@ import { fileURLToPath } from 'node:url'
 import { epic_bundle_gaps } from '#scripts/epic/epic-bundle-gaps'
 import { epic_index, type FetchedEpics } from '#scripts/epic/epic-index'
 import { epic_issue } from '#scripts/epic/epic-issue'
-import { git_gh_command } from '#scripts/git/git-gh-command'
-import { SUMMARY_FIELDS } from '#scripts/git/git-gh-issue'
-import { git_next_issues } from '#scripts/git/git-next-issues'
-import { AUTO_OK_LABEL } from '#scripts/git/issue-labels'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
+import { SUMMARY_FIELDS } from '#scripts/gh/git-gh-issue'
 import { cutoff_cause, cutoff_of, type ScanCutoff } from '#scripts/git/listing-cutoff'
 import { read_json_listing } from '#scripts/git/parse-json-array'
 import { open_issue_schema, type OpenIssueData } from '#scripts/git/schemas'
+import { git_next_issues } from '#scripts/issue/git-next-issues'
+import { AUTO_OK_LABEL } from '#scripts/issue/issue-labels'
 
 // `josh auto-ok:next` — which issue outside the epic an unattended run picks up next
 // (joshuafolkken/kit#906).
 //
 // A command rather than a documented `gh` invocation, for the reason the label constant exists: the
-// name `auto-ok` is single-sourced in `scripts/git/issue-labels.ts`, and a procedure that told an
+// name `auto-ok` is single-sourced in `scripts/issue/issue-labels.ts`, and a procedure that told an
 // agent to type `gh issue list --label auto-ok` would put a second copy of it in prose, where
 // nothing checks it. The contract is `epic:next`'s: one token on standard output, every explanation
 // on standard error, so `answer=$(pnpm josh auto-ok:next)` reads it.
@@ -48,7 +48,7 @@ const LISTING_LIMIT = 200
 // CLI's version cannot be what changed the row shape — the same misdirection the message below
 // already refuses to repeat for the blocker relations, left standing one constant above it.
 const UNREADABLE_MESSAGE = `Could not read the \`${AUTO_OK_LABEL}\` listing. That is not "nothing is opted in" — check \`gh auth status\` and ask again.`
-const UNEXPECTED_SHAPE_MESSAGE = `Read the \`${AUTO_OK_LABEL}\` listing but could not parse it. That is not "nothing is opted in" — the rows came back in a shape this command does not recognize, so check the fields it asks for and the REST field mapping in \`scripts/git/git-gh-issue-rest.ts\` rather than your authentication.`
+const UNEXPECTED_SHAPE_MESSAGE = `Read the \`${AUTO_OK_LABEL}\` listing but could not parse it. That is not "nothing is opted in" — the rows came back in a shape this command does not recognize, so check the fields it asks for and the REST field mapping in \`scripts/gh/git-gh-issue-rest.ts\` rather than your authentication.`
 // The blocker relations failing takes the whole listing with them, and `issue_list` swallows
 // the error — so the read looks exactly like an access failure and sends the reader to
 // `gh auth status`, which is green. That is the misdirection joshuafolkken/kit#996 added the message
@@ -491,23 +491,13 @@ const auto_ok_cli = {
 	TRUNCATED_WITHOUT_ANSWER,
 	NONE_OPTED_IN_MESSAGE,
 	truncated_cause,
-	parse_issue_number,
-	parse_exclude,
-	parse_pair,
 	parse_options,
 	fetch_opted_in,
-	parse_listing,
-	is_page_complete,
-	is_closed,
-	is_unblocked,
 	fetch_tracking,
 	is_runnable,
 	pick_next,
 	truncation_note,
-	none_reason,
-	report,
 	run,
-	main,
 }
 
 if (process.argv[1] === fileURLToPath(import.meta.url)) await main(process.argv.slice(ARGV_OFFSET))

package/scripts/backlog/backlog-budget.ts

@@ -268,7 +268,6 @@ function decide(input: BudgetInput): BudgetDecision {
 
 const backlog_budget = {
 	ANSWERS,
-	ANSWER_STOP_REASONS,
 	BLOCKED_REASON,
 	DEFAULT_IDLE_MINUTES,
 	DEFAULT_IDLE_MS,
@@ -283,7 +282,6 @@ const backlog_budget = {
 	UNREADABLE_REASON,
 	WATCH_VERDICT,
 	WHOLE_RUN_BUDGET_HOURS,
-	WHOLE_RUN_BUDGET_MINUTES,
 	WHOLE_RUN_BUDGET_MS,
 	WHOLE_RUN_REASON,
 	decide,

package/scripts/backlog/backlog-defect-priority.ts

@@ -1,6 +1,6 @@
 import type { EpicChild } from '#scripts/epic/epic-graph'
 import type { EpicNextResult, RepoCandidates } from '#scripts/epic/epic-report'
-import { git_gh_command } from '#scripts/git/git-gh-command'
+import { git_gh_command } from '#scripts/gh/git-gh-command'
 import { defect_rate, type DefectRate, type IssueKind } from '#scripts/issue/defect-rate'
 import { defect_rate_cli } from '#scripts/issue/defect-rate-cli'
 
@@ -72,6 +72,6 @@ async function prioritize(
 	return await reorder(result, repo)
 }
 
-const backlog_defect_priority = { UNMEASURED_MESSAGE, KIND_RANK, prioritize }
+const backlog_defect_priority = { UNMEASURED_MESSAGE, prioritize }
 
 export { backlog_defect_priority }

package/scripts/backlog/backlog-drive-cli.ts

@@ -5,13 +5,13 @@ import { CONTEXT_CUT_THRESHOLD } from '#scripts/cost-runtime/context-cut-thresho
 import { hook_decision } from '#scripts/josh/hook-decision'
 import { josh_command } from '#scripts/josh/josh-run'
 import { lane_await, type AwaitState } from '#scripts/lane/lane-await'
-import { lane_launch_cli } from '#scripts/lane/lane-launch-cli'
+import { lane_launch_cli, type LaunchOutcome } from '#scripts/lane/lane-launch-cli'
 import { lane_registry } from '#scripts/lane/lane-registry'
 import { cli_flags } from '#scripts/lib/cli-flags'
 import { error_text } from '#scripts/lib/error-message'
 import { run_carry, type RunCarry } from '#scripts/run/run-carry'
 import { run_event_stream_emit } from '#scripts/run/run-event-stream-emit'
-import { run_merge_cli } from '#scripts/run/run-merge-cli'
+import { run_merge_cli, type MergeResult } from '#scripts/run/run-merge-cli'
 import { z } from 'zod'
 import {
 	backlog_drive,
@@ -28,6 +28,7 @@ import { backlog_drive_owner } from './backlog-drive-owner'
 import { backlog_drive_restore } from './backlog-drive-restore'
 import { backlog_drive_retrospective } from './backlog-drive-retrospective'
 import { backlog_offer_cli } from './backlog-offer-cli'
+import { backlog_ready } from './backlog-ready'
 
 // `josh backlog:drive` — the backlogrun parent's loop as one resident wait (joshuafolkken/kit#2499).
 // The parent issues it once, in the background, and is woken only when it exits — with a token the model
@@ -217,7 +218,7 @@ function merge_token(out: string, code: number): string {
 
 // The child's transcript, where its lane recorded one, so `run:merge` can tell an API outage from a
 // failure exactly as it does for the parent that passes `--output` by hand.
-async function merge(issue: string, owner: string): Promise<string> {
+async function merge(issue: string, owner: string): Promise<MergeResult> {
 	await backlog_drive_owner.assert_current(owner)
 	const lane = await lane_registry.find_open_lane(issue)
 	const result = await run_merge_cli.merge_child({
@@ -231,13 +232,15 @@ async function merge(issue: string, owner: string): Promise<string> {
 
 	await backlog_drive_named.mark_done(issue, result, owner)
 
-	return result.token
+	return result
 }
 
-async function launch(issue: string, owner: string): Promise<boolean> {
+async function launch(issue: string, owner: string): Promise<LaunchOutcome['kind']> {
 	await backlog_drive_owner.assert_current(owner)
 
-	return (await lane_launch_cli.launch_lane({ issue, stash: undefined })) !== undefined
+	const outcome = await lane_launch_cli.launch_lane({ issue, stash: undefined })
+
+	return outcome.kind
 }
 
 // `lane:await`'s re-confirmed check, one state per child. A child already gone when the loop starts —
@@ -304,6 +307,7 @@ function ports_of(
 		merge: async (issue) => await merge(issue, context.owner),
 		offer: async (state) => await read_offer(state, context),
 		launch: async (issue) => await launch(issue, context.owner),
+		free_lanes: backlog_ready.drive_free_lane_count,
 		now: () => new Date(),
 		sleep: async (milliseconds) => {
 			await sleep(milliseconds)
@@ -375,14 +379,12 @@ async function main(argv: ReadonlyArray<string>): Promise<void> {
 }
 
 const backlog_drive_cli = {
-	USAGE,
 	end_line,
 	finish: backlog_drive_finish.finish,
 	merge_token,
 	offer_argv: backlog_drive_offer_argv.offer_argv,
 	parse,
 	resume_line,
-	run,
 	to_offer,
 }