Docs: 使える者だけで始める収束ループと、結果なしの取り込みを確定仕様にする(#727 #728 の束) - #798
Merged
Merged
Conversation
実装が済んだ 2 つのまとまりの計画・設計・要求を、現行の実装と一致する確定仕様へ 書き直した。 - cross-review-participants-and-seats.md: 使える者の解決・止めない確認・毎ラウンド 2 席の埋め方・席の名前・再開で渡した引数の反映(#727 #687 #478 #648) - cross-refactoring-apply-intake.md: 結果なしを 3 つの取り込みが同じ手順で受けること、 適用ラウンドの開き直しと試行の上限、項目の無いラウンドを作らないこと、帰属の段落の 後ろから記名を読むこと(#728 #647 #592 #553 #674) - cross-review-launch-outcome.md: 読み取りの契約が「まだ実装に入っていない」と書いて いた箇所を、実装済みの仕様への参照へ直した - README.md: 索引に 2 行を足した Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MGCedPTy818Zw7VYdmE4GB
takemi-ohama
commented
Sep 22, 2026
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | kiro | APPROVE
実装 2 本(#793 / #796)と突き合わせて確認した確定仕様の追加。定数(MAX_APPLY_ATTEMPTS=2 / IMPL_STALL_MARGIN=900)、関数(resolve_participants / review_seats / impl_assign / seat_runtime / group_reopening / impl_for_seq)、置き場所、参照テスト、SEAT_PATTERN が実装と一致。impl_for_seq が assignment.assign を包む形(差し替えは #664 に留保)も両仕様の「書かなかったこと」と整合。link / line-limit / staleness の 3 検査は exit=0。launch-outcome 仕様の「未実装」記述も実装済み仕様への参照へ正しく更新済み。索引・既存仕様との重複なし。修正を要する点は見つからなかった。
takemi-ohama
commented
Sep 22, 2026
This was referenced Sep 22, 2026
Merged
Merged
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.
Summary
実装が済んだ 2 つのまとまりの計画・設計・要求を、現行の実装と一致する確定仕様へ書き直した。実装は
#793(#727 #687 #478 #648)と #796(#728 #647 #592 #553 #674)で
developへ入っている。docs/specifications/cross-review-participants-and-seats.md(401 行)docs/specifications/cross-refactoring-apply-intake.md(411 行)あわせて 2 件を直した。
cross-review-launch-outcome.mdが「この契約の実装はまだ入っていない」と書いていた箇所を、実装済みの仕様への参照へ直したdocs/specifications/README.mdの索引へ 2 行を足した書かなかったこと
従来の割り当てを包む形で動いている。差し替えは別の変更(cross-refactoring: 担当を外す引数が無く、外せても使える者が 2 者だとレビュー担当が 1 者になる #664)が行う
取り込みでまとめて行う
Test plan
作業ツリーで実行した。合否は終了コードで見る。
python3 scripts/check-doc-line-limit.py --root .→Markdown line limits are satisfied (272 files, limit 500)exit=0python3 scripts/check-markdown-links.py --root .→Markdown local links are validexit=0python3 scripts/check-doc-staleness.py --root .→documented skill counts and versions are up to dateexit=0文書の検査
使える者だけで始める収束ループ(401 行)
結果なしの取り込みと開き直し(411 行)
超過の 2 件は、分けない方を採った。
図とその要素の表が離れる
個別の節へ入るため、節の粒度が実質の単位になる。ファイルを分ける基準(501 行)にも届かない
再構成の前後の対は取れない。どちらの文書も組み直しの元になる下書きを持たず、最初から読み手の
問う順(目的 / 用語 / 対象範囲 / 背景 / 決定と理由 / 仕様 / データ・設定 / テスト観点 / 関連リンク)で
書いたためである。
🤖 Generated with Claude Code
https://claude.ai/code/session_01MGCedPTy818Zw7VYdmE4GB