Repository navigation
検査のプラン: 想定最大時間を省いてもリファクタリングが下限の時間で通り、収まらないときはコードレビューへ進む（#2047） - #2301
Merged
Merged
Conversation
リファクタリングの想定最大時間を省いたときは下限まで上げ、着手前のテストの 初回の上限は既知の所要から決め、時間の枠に収まらないときは abort せずに コードレビューへ進む、を受け入れ条件にした。用語集に 2 語を足した。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
takemi-ohama
commented
Oct 31, 2026
takemi-ohama
left a comment
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
修正が必要な指摘は major 2 件です。
takemi-ohama
commented
Oct 31, 2026
takemi-ohama
left a comment
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | kiro | REQUEST_CHANGES
ドメインモデルの節と用語集の差分を確認した。用語集に加えた機械の鍵 window_skip / step_extend が、設計のデータ構造・入出力の契約のどの識別子とも一致しない。設計本体が使う名前（SKIP / metrics.skipped / findings.refactor_skipped / NDF_STEP_EXTEND_FILE / cur.extended_timeout）へ合わせるか、実在しない鍵なら code を空にすること。
- 着手前のテストの上限が 0.10·B を超えるとき、init がテストの前に延長のファイルを書く契約を足す（親の 1,980 秒で先に切られない）。延長は今の締め切りより前へ戻さない - I8 を「飛ばしたことだけを理由に failed にしない」へ限り、後の工程の失敗は failed を記録する - 用語集の時間の枠の不足・ステップの延長から、設計に無い機械の鍵を外す Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Author
🔧 /ndf:fix サマリ | round 1 | commit 7f8d5e9対応件数: critical=0 / major=4 / minor=0（合計 4 件） glossary-check の失敗は issues/issue-2047-requirements.md の 1・7 行の廃止した語『構造改善』（『リファクタリング』と書く）。この文書は Issue #2047 の本文の写しで手で直さないため、この修正では直していない。Issue の本文（題と 7 行目）を直してから spec-copy.py write で写しを作り直す必要がある |
Contributor
Author
🔧 /ndf:fix サマリ | commit (なし)対応件数: critical=0 / major=0 / minor=0（合計 0 件） |
takemi-ohama
marked this pull request as ready for review
October 31, 2026 13:45
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
10 of 27 tasks
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.
#2047 の要求と受け入れ条件（
issues/issue-2047-requirements.md）と設計（issues/issue-2047-design.md）を書きました。あわせて用語集（docs/glossary.md・docs/glossary/glossary.json）を更新しています。スプリント l20261031-4 の設計の PR です。利用者向けの変化
移行の手順
変更の要点
issues/issue-2047-requirements.mdを新しく作りました（213 行）。検査のプランの構造改善が既定の予算で下限に届かず、33 行の差分でも refactor が落ちて abort へ進む #2047 の要求と受け入れ条件を書いています。issues/issue-2047-design.mdを新しく作りました（381 行）。検査のプランの構造改善が既定の予算で下限に届かず、33 行の差分でも refactor が落ちて abort へ進む #2047 の設計を書いています。docs/glossary.md: 7 行を変更docs/glossary/glossary.json: 44 行を追加・変更テスト
supervise.py の run のステップの結果です。
issues/issue-2047-requirements.mdを読めない（[Errno 2] No such file o…）requirements-check が exit 2 で止まりました。
issues/issue-2047-requirements.mdが見つからないという内容です。このファイルは変更の統計では追加されています。それでもチェックが止まったので、マージの前に原因を確かめる必要があります。モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー
🤖 Generated with Claude Code
決めたこと
issues/issue-2047-design.md