chore(upstream): pre-spork 再整合の残債を v13.7.2 へ追従する (follow-up of #749/#751)

[takaokouji]

Goal

PR #751 (Issue #749 の Phase 1) で pre-spork 巻き戻しの取り残しのうち color/blockJSON/hat 系を upstream v13.7.2 へ再整合した。残りの pre-spork 債務ファイルは まだ v13.7.2 と乖離した状態であり、次の upstream マージ時に「reverted 状態を基準にしてしまう」リスクが残る。本 Issue でそれらを段階的に v13.7.2 へ再整合する。

ベースライン・develop-trap・分類は docs/upstream-tracking/pre-spork-realignment-audit.md を参照。

残債（v13.7.2 と乖離したままのファイル）

• packages/scratch-vm/src/engine/blocks.js — コメントイベントの v1 comment_* dual-path を modern block_comment_* のみへ整理（v1 経路が dead であることの確認が必要）
• packages/scratch-vm/src/engine/adapter.js + packages/scratch-vm/src/serialization/sb3.js + packages/scratch-vm/src/engine/comment.js — コメント id 契約（XML xmlChild.attribs.id vs JSON ${blockId}_comment）の整合
• packages/scratch-gui/src/lib/make-toolbox-xml.js — カテゴリ名 %{BKY_CATEGORY_*} → ScratchMsgs.translate(...)（v2.1.19 でも動作する cosmetic 債務・低優先）
• packages/scratch-vm/src/engine/runtime.js — BEFORE_STEP / EXTENSION_DATA_LOADING など hat と無関係な小乖離の確認・整合
• band-aid の除去可否検証と削除: scratch-blocks-auto-style-patch.js、blocks.jsx の v2-bridge guard（forceRerender try/catch、componentWillUnmount optional-chaining、My Blocks 再構築）

重要

• これらは 意図的な「再整合の取り残し」であり、upstream の修正を取りこぼしたものではない。次の upstream マージ時に bin/upstream-divergence-audit がこれらの DIFF を報告するが、lost-fix と誤認しないこと。
• 各ファイルの A/B/C 分類とハンク単位の方針は監査ドキュメント参照。
• .upstream-merge-history.json の postMergeReverts にも登録済み（次マージで自動的に警告される）。

Related

• Audit/plan: chore(upstream): pre-spork 巻き戻し (#270) の取り残しを監査し v13.7.2 ベースラインへ再整合する計画 #749
• Phase 1 実装: fix: re-align pre-spork-reverted files to upstream v13.7.2 (#749) #751

[takaokouji]

追加の残債（敵対的レビューで判明）: packages/scratch-gui/src/lib/define-dynamic-block.js の dynamic-block HAT/reporter 経路が this.setCheckboxInFlyout(true) を呼ぶが、このメソッドは scratch-blocks v2.1.19 に存在しない（upstream v13.7.2 と同一・dormant: 現状の Smalruby dynamic reporter は全て disableMonitor のため発火しない）。v13.7.2 追従の一環として、モニタ可能な dynamic reporter を導入する際は monitor_block 方式へ寄せること。

[takaokouji]

残作業の確認 → actionable なし。close します。

#752 の各残債を Step 2/3 + #757 で解消・確定しました。

解消（v13.7.2 へ再整合）

• ✅ blocks.js コメントイベント dual-path → v13.7.2 の modern-only（fix(scratch-vm): re-align comment handling to upstream v13.7.2 (Step 2, #752) #753 / Step 2）
• ✅ adapter.js + sb3.js + comment.js のコメント id 契約 → ${block.id}_comment に統一・collapsed= 出力（fix(scratch-vm): re-align comment handling to upstream v13.7.2 (Step 2, #752) #753 / Step 2、Ruby コメント往復で実機検証）
• ✅ runtime.js EXTENSION_DATA_LOADING 再追加（upstream 一致） + BEFORE_STEP をマーカー化（chore(upstream): finish #752 — runtime alignment + reclassify v2-bridge guards as legitimate (not band-aids) #757）

調査の結果「除去すべきでない」と確定（intentional keep）

• ✅ scratch-blocks-auto-style-patch.js は唯一の真の pre-spork band-aid → 削除済み（refactor(scratch-gui): retire auto-style-patch band-aid (Step 3, #752) #754 / Step 3）
• 🔒 blocks.jsx の v2-bridge guard（componentWillUnmount focus guard = scratch-blocks#3460、forceRerender try/catch = Ruby list の断続フリーズ防止）は scratch-blocks v2.1.19 自体の実バグ修正であり、除去するとクラッシュ再発（empirical 検証済み）。維持
• 🔒 make-toolbox-xml.js の %{BKY_CATEGORY_*} は v2.1.19 の replaceMessageReferences で upstream の ScratchMsgs.translate と同一の翻訳名に解決する benign な簡略表記。機能差なし。維持
• 🔒 runtime.js BEFORE_STEP は Mesh v2 依存（upstream は削除）。維持・マーカー化

補足

• dynamic-block (define-dynamic-block.js) の setCheckboxInFlyout は upstream v13.7.2 と同一かつ現状 dormant（モニタ可能な dynamic reporter が無いため発火しない）。pre-spork 債務でも Smalruby 乖離でもないため本 Issue の対象外。
• 上記の intentional keep は .upstream-merge-history.json の postMergeReverts と .claude/rules/upstream-tracking.md に記録済み（次の upstream マージで bin/upstream-divergence-audit が DIFF を出しても lost-fix と誤認しないこと）。

全 Step は CI green + ホスト Chrome 実機検証 + mesh 回帰（unit 391 / integration 34 / mesh-v2-classroom-binding.mjs）pass。topic ブランチに集約し #755 で develop へ取り込み中。

→ actionable な残作業なし。close。