factory/ のリファクタリングの提案 20 件が時間で見送られ、子課題が同じ重複を写して広げる #2199

依頼（原文）

#2199。人へ問わずに進め、決められない点は前提か未決として課題の本文へ書く。

（書き手: AI の起動指示、日付: 2026-10-31。利用者の指示の原文としては扱わない）

課題の本文（下の「起票時の記録」）は、#2170 の cross-refactoring（PR #2192）の後に起票者（AI）が書いた記録である。利用者が書いた文は含まれていない（書き手: 起票者、日付: 2026-10-30）。

起票時の記録

起票者の本文をそのまま引用する（書き換えない）。

何を見つけたか

#2170 の検査（cross-refactoring、PR #2192）で、factory/ の構造改善の提案 20 件が時間（budget）で見送られ、どの課題にも残っていない。リファクタリング計画（PR #2192 のコメントの「見送った提案」）にだけある。

主なもの（見積りの大きい順）:

対象	兆候	見積り
factory/api/src/accounts/store.ts#DynamoAccountStore	duplication	10.3 分
factory/api/src/routes/teams.ts#registerTeams	duplication	9.2 分
factory/api/src/accounts/store.ts#seatItem	duplication	9.2 分
factory/api/src/handlers/github-oidc.ts#requireEnv	duplication	9.2 分
factory/api/src/accounts/service.ts#AccountService.hist	long_parameter_list	4.7 分
factory/api/src/accounts/store.ts#AccountStore.updateAccount	data_clump	4.7 分
factory/api/src/index.ts#depsFromEnv	circular_dependency	2.0 分
factory/api/src/accounts/store.ts#AccountStore	divergent_change	2.0 分

ほか 12 件（github-oidc/app.ts・auth/verify.ts・routes/accounts.ts・web/src/pages/*.tsx・scripts/tests/test_check_markdown_links.py）。全件の表は PR #2192 の cross-refactoring のコメントにある。

どこで見つけたか

PR #2192（#2192 ）の cross-refactoring のコメント「見送った提案」。件数の行は「採用 3・未確認 0・取り消し 0・見送り 20（うち時間 20）」。

なぜこの変更の範囲外なのか

構造改善の工程の予算（--budget-minutes）を超える分で、#2170 の受け入れ条件はすべて満たしている。Value 10 は、上限を超えてまで行いたい品質の向上を別の Issue へ起こして後へ回すと定める。根拠: Value 10・Value 6（MVV 版 3）

直さないと何が起きるか

#2171・#2186・#2173・#2172・#2174 が同じ factory/api/ の store.ts・service.ts・routes/*.ts に足していくため、重複（席の行の組み立て・トランザクション・ルートの登録）が子課題ごとに写されて広がる。後の子課題の cross-refactoring が同じ提案を毎回出し直し、その都度予算で見送る。

由来

PR #2192

見送った提案の全件（PR #2192 のコメントから写した）

起票の後に重複が広がっている。2026-10-31 の develop（ee32e92f）では、requireEnv が 4 か所（factory/api/src/handlers/github-oidc.ts・handlers/api.ts・handlers/ingest.ts・runs/deps.ts）にある。PR #2192 の時点では runs/ と handlers/ingest.ts はまだ無かった。

起票者の推定

目的

• factory/ で見送ったリファクタリングの提案 20 件を片付け、後の子課題（NDF Factory: contracts と進捗の受け口 #2171・NDF Factory: スプリントの一覧（フェーズ）と詳細（工程・PR・CI・承認ゲート） #2172・NDF Factory: 承認ゲートの通知（Slack を含む）と Web からの承認 #2173・NDF Factory: サブスクリプションと Bedrock の token 課金 #2174）が重複を写して広げないようにする
• 後の子課題の cross-refactoring が、同じ提案を出し直して時間で見送ることを止める

前提

• 前提 1: 20 件の対象の一部は、PR スプリント m2170 #2192 の後の変更（NDF Factory: スプリントの実行の制御とランナー #2186 などのマージ）で形が変わっているか、すでに直っている。実装の最初に、20 件のそれぞれが今の develop に残っているかを確かめる
• 前提 2: R07（createGithubOidcApp の POST /token）は PR スプリント m2170 #2192 の I-001 で同じ箇所を抽出済みで、残りがあるかは前提 1 の確かめで決める
• 前提 3: 振る舞いの基準は、実装を始める時点の develop の既存テスト（factory/ の vitest と scripts/tests/test_check_markdown_links.py）である。テストの期待値は変えない（テストの重複を直す R16 は、テストの関数の数と確かめる入力・期待値の組を減らさない）
• 前提 4: 外部 AI の提案ごとの手法（extract_method など）は手がかりであり、同じ兆候をより小さい変更で消せるならその手法を採ってよい

対象範囲

含む:

• 上の表の R01〜R20 の 20 件
• R01〜R20 と同じ役割の写しのうち、PR スプリント m2170 #2192 の後に足されたファイル（factory/api/src/runs/・handlers/ingest.ts など）にあるもの（S3）

含まない:

• 振る舞いの変更（API の応答・エラーの文言・OpenAPI の契約・DynamoDB のキーと属性の形・画面の表示）
• R01〜R20 と同じ役割の写しでない、新しく見つかったリファクタリングの提案（見つけたら out-of-scope で別の課題へ起こす）
• factory/runner/・factory/contracts/ のリファクタリング
• 着手の順位を決めること（S2。棚卸しが決める）
• 依存パッケージの追加

ドメインイベント

用語

受け入れ条件

• 1. 実装の Pull Request の本文に、R01〜R20 の 20 行がそろった表があり、各行の扱いが「適用した」か「採らない」のどちらかである
• 2. 表の「採らない」の行には理由があり、理由は次のどれかである: 対象が今の develop に無い / 兆候がすでに無い / 適用するとテストが落ちる / 適用すると振る舞いが変わる。「時間（budget）」を理由にした行は 0 行である
• 3. factory/api/src で function requireEnv の定義は 1 か所だけである（grep -rn "function requireEnv" factory/api/src が 1 行を返す）。R01〜R20 のほかの「適用した」重複についても、同じ役割の関数の定義が factory/ の中で 1 か所にまとまり、表の行にその場所が書かれている
• 4. npm --prefix factory ci --no-audit --no-fund --silent && npm --prefix factory run check が終了コード 0 で終わる（契約の lint・型検査・vitest を含む）
• 5. uv run --frozen --project . --all-extras pytest scripts/tests/test_check_markdown_links.py -q が終了コード 0 で終わり、通ったテストの件数が変更の前と同じかそれ以上である
• 6. 実装の差分に、factory/ の既存のテストファイル（*.test.ts / *.test.tsx）の期待値（expect(...) の右辺）を変える行が無い。テストを動かした・まとめた行があれば、表の行に理由が書かれている
• 7. 実装の差分に、factory/contracts/ の変更が無い
• 8. 起きてはいけないこと: factory/ の外のコード（scripts/tests/test_check_markdown_links.py を除く）と、plugins/ に差分が無い

非機能の条件

影響

検証手段

前提とする取り決め

境界

未決

進行

モード: — / 進め方: auto

• 要求と受け入れ条件 — 2026-10-31 08:26
• 作業場所の用意 — 2026-11-05 02:04
• 設計 — 2026-10-31 08:28
• 素材の収集と出典の確定
• ドキュメント再構成
• ドキュメントレビュー
• 計画 — 2026-11-05 02:04
• 実装 — 2026-11-05 02:04
• 構造改善 — 2026-11-05 02:41
• 実装レビュー — 2026-11-05 03:07
• 完了判定 — 2026-11-05 02:13
• Pull Request — 2026-11-05 02:26
• 確定仕様化 — 2026-11-05 11:04
• 後片付け — 2026-11-05 11:20
• 配布 — 2026-11-05 03:24
• 体裁レビュー
• リリース後テスト
• 振り返り — 2026-11-05 11:15
• 棚卸し — 2026-11-05 11:21

人の判断待ち（開発ループ l20261031）

• 判定: follow（承認ゲート 1）
• 理由: Value 6・Mission: 見送った提案 20 件を「適用する」か「採らない（理由つき）」に決めます。requireEnv の 4 つの写しや、6 画面にあるログインの確認などを 1 か所の定義へ寄せます。共通の層で根本から直す設計で、R01〜R20 の後に足された写しも対象に含めています／Value 7・Value 3: 決定 1〜10 のそれぞれに「何を決めたか・採らなかった案とその理由」があります。判定表の今の状態は、2026-10-31 の develop（ee32e92f）で grep の行数と pytest 46 passed を実測して書いています／Value 1・Value 10: 新しい現状固定テストは足しません（決定 9）。振る舞いは既存の vitest・pytest の期待値で縛り、確定仕様の 2 行の直しは確定仕様化へ回します（決定 10）。費用の範囲の中で品質を保つ形です／C3・I2: DynamoDB のキー・属性・TransactItems の並び・プレースホルダは変えないと不変条件に明記し、store.test.ts で縛ります。利用者のデータを変える・移す変更は含みません。配布物（plugins/）にも触れません（P1・C5 に当たりません）／Vision・Value 2: テストの requirements-check が exit 2 で止まっています。issues/issue-2199-requirements.md を検査の時点で読めず、要求の写しとの整合を機械で確かめられていません。判定材料が 1 つ欠けていることとして、人へ示す必要があります
• 根拠の項目: Mission, Vision, Value 1, Value 2, Value 3, Value 6, Value 7, Value 10, C2, C3, C5, P1
• 設計 Pull Request: factory/: 見送ったリファクタリングの提案 20 件を片付け、同じ役割の関数を 1 か所へ寄せる（#2199） #2277
• 答え: approved
• 答え方: python3 /persistent/ai/.claude/plugins/cache/ai-plugins/ndf/10.27.7/scripts/sprint-state.py loop answer l20261031 2199 --approved|--rejected

振り返り: #2199 (comment)