全体リファクタリング(挙動維持): 重複解消・inbox view 切り出し・widget コア分割

[kosako]

背景

アーキテクチャレビュー(2026-07-02)の G-2/G-3/G-4 と監査(2026-06-20)の L-31/L-32/L-34/L-36/L-37 で指摘済みの構造負債を、挙動を一切変えないリファクタリングとして解消する。receive.js は 2083 行(export ゼロ)、widget/src/index.js は 1729 行(state 直接参照 180 箇所)まで肥大している。

スライス(PR 分割)

R-1: DRY 小粒 + build ガード(refactor/dry-core)

• safeFilePart の重複解消(receive.js:1499 と widget index.js:970 にバイト同一)→ shared/format.js へ移設。まず server 側を切替(widget 側の切替は R-3 で実施)
• status 定数の単一ソース化: FEEDBACK_STATUSES(receive.js:98)と VALID_STATUSES(store.js:41)→ store.js から export し receiver が require。正規化の丸め挙動(不正値→"new")は不変のまま(拒否化は L-12 で別 issue)
• build.js に複数宣言子 export(export const A = 1, B = 2;)の検出 fail を追加(L-34。現 src 該当 0 件なので dist 不変)

R-2: inbox view の切り出し(refactor/inbox-view、R-1 の上に stack)

• renderInbox 系 7 関数(receive.js:1904-2083)+ renderLoginPage を server/inbox-view.js(CJS)へ。G-2 で「最も安全・いつでも切れる」と合意済みの境界
• レスポンスヘッダを書く responder(respondLoginPage / handleGetInbox 等)は receive.js に残す。HTML 出力はバイト単位で不変

R-3: widget コアの分割(refactor/widget-split)

• state + DEFAULTS を widget/src/state.js へ(state への再代入はゼロ・全 180 箇所がプロパティ読み書きなので参照共有で挙動不変)
• state 非依存の純関数クラスタを分割: screenshot SVG 生成系 / 永続化 envelope 系 / payload 組み立て。orchestrator(submitComment / renderShell / renderFeedbackList / restorePersistedFeedback 等の相互再帰群)は index.js に残す(build.js は循環 import で hard fail)
• 切り出したモジュールに単体テストを追加(テスト負債: widget コア単体 / Slack 配信経路 / import 失敗系分岐の欠落 #86 の widget コア単体テスト不在と同方向のスライス)
• widget 側 safeFilePart を shared 参照へ切替(L-31 完了)

触らない境界(挙動変更 or 進行中 issue との衝突)

• Slack delivery 抽出(receive.js:1533-1902)→ Slack 送信を POST /feedback の応答パスから分離する（遅延ブロック解消・再送導線） #85 と同時(G-2/G-5 合意)
• Slack Block Kit の widget/server 共通化・widget 直送モード(index.js:977-1073)→ Slack 経路の縮約: widget 直送モード削除と SLACK_IMAGE_MODE の 2 種化(link/off) #97 で削除予定。移動もしない
• gitHubIssueBody 系(receive.js:847-955)→ payload に sourceContext(repo/branch/commit)を追加し、GitHub Issue body を agent-ready に再設計する(路線 1 の前提) #96 PR-B の再設計領域
• バグ修正系(L-1/2/4/12/15/30/35、M-1 等)→ 挙動が変わるため対象外
• ROUTES の shape・auth kind 名 → 公開デプロイ時の receiver ハードニング（認証・CORS 制限） #43 系後続の前提

受け入れ条件(各 PR 共通)

1. npm run check(lint + dist バイト一致 + 全テスト)green。既存テストの期待値は 1 件も変更しない
2. widget を触る PR は dist 再生成をコミットし、dist の diff で関数本体のテキストが不変(モジュール境界コメント・IIFE ラッパ以外に差分がない)ことを確認
3. R-3 は実ブラウザ E2E(コメント投稿 → receiver 保存)で回帰がないことを確認
4. AI 相互レビュー(author ≠ reviewer)で must 0

実装分担: R-1/R-2 は Codex、R-3 は Claude(レビューは各々反対側)。

[kosako]

3 スライスすべて実装・相互レビュー完了(いずれも must 0)。merge 待ち。

• R-1 PR server/shared: safeFilePart と status 定数を単一ソース化し build.js に export ガードを足す (#109 R-1) #110(base: main)— safeFilePart の shared 移設(server 側)+ status 定数の単一ソース化 + build.js 複数宣言子 export ガード。実装 Codex / レビュー Claude ✅
• R-2 PR server: inbox / login の HTML 生成を inbox-view.js へ切り出す (#109 R-2) #111(base: refactor/dry-core)— inbox / login の HTML 生成 8 関数を server/inbox-view.js へ。実装 Codex / レビュー Claude ✅(should 1 件は 9eb42c5 で対応済み)。新旧 receiver を同一 DB で起動し inbox / login / 認証付き inbox の HTML がバイト単位で一致することを実測確認
• R-3 PR widget: state と純関数クラスタを 4 モジュールに分割する (#109 R-3) #112(base: refactor/inbox-view)— widget/src/index.js(1729→1311 行)を state.js / screenshot.js / persistence.js / payload.js に分割 + 単体テスト 11 件 + widget 側 safeFilePart 切替(L-31 完了)。実装 Claude / レビュー Codex ✅。dist の機械照合で関数本体の不変を確認、実ブラウザ E2E(投稿→保存→reload 復元、旧ビルド envelope のクロスバージョン復元含む)green

merge 順: #110 → #111(retarget)→ #112(retarget)。各 merge 後に stack の rebase 不要なら squash のみで進む想定。

スコープ外として残した項目(挙動変更 or 進行中 issue と衝突するため不実施): Slack delivery 抽出(#85 と同時)/ Block Kit 共通化・直送モード(#97)/ gitHubIssueBody 系(#96 PR-B)/ バグ修正系 L 項目。

[kosako]

3 スライスすべて merge 完了(main 1fa4e1b)。

• R-1: PR server/shared: safeFilePart と status 定数を単一ソース化し build.js に export ガードを足す (#109 R-1) #110(safeFilePart shared 移設・status 定数単一ソース化・build.js export ガード)
• R-2: PR server: inbox / login の HTML 生成を inbox-view.js へ切り出す (#109 R-2) #111 → server: inbox / login の HTML 生成を inbox-view.js へ切り出す (#109 R-2) #113 で merge(server/shared: safeFilePart と status 定数を単一ソース化し build.js に export ガードを足す (#109 R-1) #110 の squash merge に伴う base ブランチ削除で GitHub が server: inbox / login の HTML 生成を inbox-view.js へ切り出す (#109 R-2) #111 を自動 close し、base 消失のため reopen 不可だったため、rebase 済み同内容ブランチから再作成。レビュー履歴は server: inbox / login の HTML 生成を inbox-view.js へ切り出す (#109 R-2) #111 参照)
• R-3: PR widget: state と純関数クラスタを 4 モジュールに分割する (#109 R-3) #112(widget を state.js / screenshot.js / persistence.js / payload.js に分割 + 単体テスト 11 件 + widget 側 safeFilePart 切替)

最終状態: receive.js 2083→1873 行 / widget index.js 1729→1318 行 / 全 124 テスト green。全 PR で相互レビュー must 0(挙動不変は HTML バイト比較・dist 関数本体照合・実ブラウザ E2E で実証)。

スコープ外として残した項目は各 issue で追跡: Slack delivery 抽出(#85)/ Block Kit 共通化・直送モード(#97)/ gitHubIssueBody 系(#96 PR-B)/ 残テスト負債(#86)。

運用メモ: stacked PR を squash merge する際、親 PR を --delete-branch で merge すると子 PR が自動 close され reopen 不可になる(base 消失)。次回は「親 merge 時は branch を残す → 子を main に retarget → rebase & force-push → 子 merge → branch 削除」の順で行う(#112 はこの手順で無事故)。