Repository navigation
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…qa-dev.csv の試料(#375 の 1 パス目) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…の 2 パス目) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 27, 2026
This branch has not been deployed
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.
Fixes #375
Fixes #367
Fixes #372
cruise #474 の周(不在の周。
/dev-loop:cruise 474 --absent)。変更
check-signoff.pyが比べる範囲を、2 つの承認が見た中身に揃える。shared/・requirements/・spec/(内部の Q&A 表spec/qa-dev.csvを除く)とremoved.csv(v1 のものも)removed.csvは内部の層の Q&A の行(D-)以外を全部比べる(範囲の外のファイルの行も——mergeが拒否するので)feasibility/・proposal/など範囲の外だけの違いは今までどおり 0references/signoff-tags.md(比べる場所・空かを見る場所・0.27.0 の移行の節)・tree.md・README.md・CLAUDE.mdsignoff-tags.mdの「範囲を広げた(spec-loop 0.27.0)」)仕様の穴(不在の線で推奨により決めた)
Issue の「決めること」A・B・C を推奨で決めた——確認の Issue: #478(承認するなら閉じる、直すなら実装の Issue を #474 に足す)。
requirements/・spec/を範囲に入れ、違えば 1(警告にとどめる案・別の終了コードの案は採らなかった)removed.csvと、範囲の外のファイルの行も比べる(予算側に配置の検査を当てる案は採らなかった)確かめたこと
scripts/test-check-signoff.py:OK: 試料 74 件・変異 65 件(spec-loop: check-signoff が requirements/・spec/ を比べないので、見積りの根拠が承認と違う中身になっても検出しない #367・spec-loop: 予算の合意のタグの時点にだけ v1 の removed.csv があると check-signoff が 0 を返す #372 の試料、範囲の外の陽性対照、前の版のrequirements/・spec/・qa-dev.csv)scripts/test-build-proposal.py:OK: 試料 82 件・変異 90 件(2 つのタグの間に要求を足すと作らない・比較は req のタグの時点)scripts/check-all.py: 52/52関門
直す線:
既定(cruise の確かめた線)。/code-review(high)1 パス目で、私の誤りが 2 つ見つかった——(1)
test-build-proposal.pyの変異「比較を予算のタグの時点から作る」を「等価変異」として外したが、removed.csvの理由の列だけ違う組で殺せた(戻した)。(2) 計画の受入基準 I3 に「
removed.csvの範囲の外のファイルの行 → 0」を入れていたが、Issue の受入基準 B(予算側だけのmergeが拒否する配置を 1 か 2)と食い違っていた(Issue に合わせて直した)。採否の表は
docs/plans/issue-375.md。残課題(直していない指摘)
INTERNAL_QA・SCOPE_DIRSがmerge-versions.pyのINTERNAL_QA_PATH・AREASの写しreferences/qa-csv.mdの「shared/は比べる範囲なので内部の表を置かない」の理由が古いsignoff-tags.mdの突き合わせがDOCSを見なくなった(走らせていない)requirements/・spec/のモードだけの変更(chmod)も 1 になる(走らせていない)removed.csvの同じ行が片側だけ 2 回あるのを見る試料が無い(この周より前から)Requirements/が予算側だけにあっても 0(shared/から同じ穴。spec-loop: macOS で Removed.csv・Shared/ の扱いがスキルの差分と提案書で食い違う #276 が cruise spec-loop: 残課題を cruise で回す(親 Issue) #474 の台帳にある)spec/spec_ir.jsonを 2 つのタグの間で commit すると 1(走らせていない。要確認: #375 で、照合の範囲を shared/・requirements/・spec/(qa-dev.csv を除く)と removed.csv に広げた(推奨で決めた) #478 に書き足した)core.quotePath)docs/plugins/index.md)と description の「共有の範囲(shared/ ほか)」未証明
なし(照合は git の出力だけで決まり、試料で当てた)。
マージ
不在の cruise はここで止まる(このリポジトリのマージはサイトを再デプロイするので、確かめた線
マージ = 関門を通ればマージが効かない。#476)。#477(#440)と同じく spec-loop の版を上げている——先にどちらかをマージしたら、もう片方は main を取り込んで版を上げ直す。
🤖 Generated with Claude Code