Repository navigation
docs: Web の 3 本を必須へ昇格し、文書を実測に合わせる - #75
Merged
Merged
Conversation
`Web wasm32 (cross-build)` / `Web browser E2E` / `Package web-wasm` を 必須チェックへ昇格させた（21 → 24 本）。根拠は #63 / #69 / #70 / #72 / #73 の 5 本すべてで success になったことで、必須外にしていた当初の理由 （「緑の実績が浅い」）は失効していた。モバイルの 2 本が M4 の直後は必須外で 2026-08-31 に昇格したのと同じ手順である。 設定は required_status_checks だけを PATCH した（/protection への PUT は 他の保護設定を消しうる）。変更後に strict / enforce_admins / 直線履歴 / force push 禁止 / 削除禁止がいずれも元のままであることを確認した。 CLAUDE.md の列挙 24 件が API の 24 件と過不足なく一致することを機械で 突き合わせてある（両向きとも差分なし）。CLAUDE.md 自身が「増減したら README の 2 段落も一緒に直せ」と書いているので、両 README の該当段落も 同じ commit で直した —— 片方だけ直すと翻訳同士が食い違う。 **帰結: Web のレーンは、いまは赤ければ merge を止める。** ついでに、両 README がリンク済み module を 8 つと書いていたのを 9 つに 直した（`dnn` が抜けていた。正本は cmake/FindOpenCvUnityDeps.cmake:84）。
review-1: docs/roadmap.md の M6「満たしていないもの」に「必須にするかは まだ決着していない」「同じ手順を踏む」が現在形で残っていた。昇格で偽に なったのに前の commit が触っていなかった。解消した項として書き直す。 review-2（自分で作り込んだ欠陥）: CLAUDE.md:416 で Web 3 job を挿すときに 開き ** を足したため、対応がずれて ** が奇数（9 個）になっていた。結果、 足した文は太字にならず、代わりに履歴の部分が太字になり、生の ** が本文に 出ていた。開き記号を落として 1 つの太字が 3 群を覆う形に戻した。416/417 の 両行が偶数であること、ファイル全体で「** が奇数の行」が main と同じ 35 行の ままであることを機械で確認した。 review-3: 根拠の呼び名が集合と合っていなかった。「直近 5 本」「five consecutive」はどちらも #63/#69/#70/#72/#73 を指さない（#64-#68 を飛ばし #74 を含まない）。実測すると根拠はもっと強く、#63 以降に merge された 11 本すべてで 3 本とも success だったので、そう書き直した。あわせて CLAUDE.md の同じ根拠が 2 箇所にあったのを 1 箇所にし、他方は上を指す。 review-4: 必須でないものを 6 本と書いていたが、PR #74 の 33 checks から 必須 24 本を引くと 9 本である。CodeQL の集約 check と、PR では skip される Unity の結果公開 job 2 本が漏れていた。3 文書とも 9 本に直した。 （旧「9 本」も同じ穴を持っていたので、今回の混入ではない。）
必須外を 9 本に直したとき、両 README に「残る 3 本は…」/「The other three unrequired checks are…」と書いたが、9 - 5 = 4 で残余は 4 本である。名指し していたのは集約 CodeQL と Unity の結果公開 2 本の計 3 本で、4 本目の Publish the release は段落末尾にしか出てこない。「残る」/「other」は残余を 宣言する語なので、読者が数えると 8 本で止まる。 前の文（"Three cover Web/Wasm"）は残余を主張していなかったので、この閉じ 方は前の commit で入ったものである。4 本と書き、Publish the release が 後述であることを明示した。EN と JA は同じ形で間違っていたので、翻訳の ずれは無い。 あわせて roadmap の M6 の項の頭に「（解消済み）」を置いた。見出しが 「満たしていないもの」なので、走り読みで取り違えないようにする。
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
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 はそれに文書を合わせるものである。
Web wasm32 (cross-build)/Web browser E2E/Package web-wasmの 3 本を必須チェックへ昇格させた（21 → 24 本）。帰結: Web のレーンは、いまは赤ければ merge を止める。
根拠
必須外にしていた理由は「緑の実績が浅い」だったが、失効していた。実測（
gh pr checks）:当初は「直近 5 本」と書いたが、レビューで指摘されて数え直したところ根拠はもっと強かった ——
#63以降に merge された PR は 11 本あり、そのすべてで 3 本とも success だった。（旧記載の
#63 / #69 / #70 / #72 / #73という並びは「直近 5 本」でも「連続 5 本」でもなく、何も指していなかった。）
モバイルの 2 本が M4 の直後は必須外で 2026-08-31 に昇格したのと同じ手順である。
設定の変え方
/protectionへの PUT ではなく、required_status_checksだけを PATCH した ——前者は送らなかった項目を消しうる。変更後に
strict/enforce_admins/required_linear_history/ force push 禁止 / 削除禁止 / レビュー必須数がいずれも元のままであることを確認した（レビュアーも独立に確認済み）。
文書
CLAUDE.mdは「必須チェックの正本はこの表ではなく GitHub 側の設定である。増やしたら、この表も一緒に直すこと」と書き、さらに「リポジトリ内で二重に書かれているのは
両 README の 2 段落だけ、という状態を保つ」と書いている。その 3 箇所を同じ commit で直した。
CLAUDE.mdの列挙 24 件が API の 24 件と過不足なく一致することを機械で突き合わせた（両向き差分ゼロ）CodeQLの集約 check と、PR では skip される Unity の結果公開 job 2 本が漏れていた）。PR docs: M7 の完了を反映し、追い越された文書・skill・hook を直す #74 の 33 checks から
必須 24 本を引いて確かめた。旧記載の「9 本」も同じ穴を持っていた（実際は 12）
docs/roadmap.mdの M6 節が「必須にするかはまだ決着していない」のまま残っていたので、解消した項として書き直した
（
dnnが抜けていた。正本はcmake/FindOpenCvUnityDeps.cmake:84）AI レビュー（PR 前、別エージェント）
1 回目: REQUEST CHANGES（要修正 2 / Minor 2）
docs/roadmap.mdの M6 節が現在形のままCLAUDE.md:416の**が奇数（9 個）**が本文に出ていた2 回目（スコープを絞って）: APPROVE WITH FINDINGS —— 新規欠陥が 1 件出た。これも作り込み ——
必須外を 9 本に直したとき「残る 3 本は…」と書いたが 9 − 5 = 4 で、4 本目の
Publish the releaseは段落末尾にしか出てこない。「残る」は残余を宣言する語なので、読者が数えると 8 本で止まる。前の文は残余を主張していなかったので、この閉じ方は
前の commit で入ったものである。 4 本と書き直した。
レビュアーは「偶数か」ではなく記号の対応を追った —— 416 の太字区間が
release6 job / モバイル 2 job / Web 3 job を 1 つの span で覆っていること、417 の 9 区間すべてが意図どおりに落ちることを確認している。行ごとの偶奇は
必要条件でしかないためで、こちらの検算より強い。
確かめていないこと
gh pr checksが返すのは各 check の最終結果なので、**「一度も赤くならなかった」ではなく「最終的に緑だった」**である
dev.ps1 test-toolsは私が回して exit 0。Unity のレーンは回していない（この差分は文書のみ）