test(blocks-screenshot): align mock workspace with scratch-blocks v2 API

[takaokouji]

Summary

feat/upstream-merge-2026-05 で CI を 9 件失敗させていた blocks-screenshot.test.js のモックを scratch-blocks v2 の API に追従させる。本体 blocks-screenshot.js は既に workspace.getCanvas() / workspace.getBubbleCanvas() を使う v2 形式に移行済みで、テストモックだけが v1 の直接プロパティアクセス (svgBlockCanvas_ / svgBubbleCanvas_) のままだった。

Changes Made

• makeMockWorkspace の戻り値を v2 メソッド形式に変更 (getCanvas / getBubbleCanvas を jest.fn で公開)
• テスト内で workspace.svgBubbleCanvas_ を直接参照していた 2 箇所を workspace.getBubbleCanvas() 経由 / workspace.getBubbleCanvas = jest.fn(() => null) に書き換え

Test Coverage

Test Suites: 1 passed, 1 total
Tests:       21 passed, 21 total

ローカルで全 21 テスト緑、Prettier も pass。

Related Issues

Closes #636

[github-actions]

🚀 Preview deployed: https://smalruby.jp/smalruby3-editor/fix/blocks-screenshot-getbubblecanvas/

[takaokouji]

手動動作確認中に追加の bug を発見したため、本 issue / PR の scope に含めました。

追加で修正した問題

blocks-screenshot が @ruby:* コメントを持つブロック (= Ruby converter が生成するすべての Smalruby ブロック) で実行すると SecurityError: Tainted canvases may not be exported. で失敗していた。

原因

3 つの canvas-taint 源があった:

1. block canvas に残る <foreignObject>: scratch-blocks v2 ではブロック付属のコメントが g.blocklyBlockCanvas 内に配置され、その中に編集 UI 用の <foreignObject> を持つ。既存コードは bubble canvas のみ strip していたため、block canvas の foreignObject が残って blob 描画時に canvas を taint していた。
2. インジェクト CSS の url() 参照: <style> の blocklyText 系ルールには cursor / sprite 用の url("./static/blocks-media/...") も含まれている。これら相対パスが blob:// コンテキストで解決できず taint。
3. 空 href の hidden <image>: dynamic block 内のドロップダウンには display:none の <image href=""> プレースホルダがあり、空 href は document URL (= blob URL) にフォールバックして taint。

修正

packages/scratch-gui/src/lib/blocks-screenshot.js:

• <foreignObject> の strip を block canvas にも適用
• stripExternalCssUrls ヘルパーで <style> から外部 url() を除去 (同一文書参照の url(#filter) と url(data:) は保持)
• inlineImageHrefs で空 href の <image> を remove()

検証

• ユニットテスト 29/29 緑 (foreignObject strip / stripExternalCssUrls / 空 href 除去 をそれぞれカバー)
• ブラウザ実機で t = [3,1,2]; t.sort! 等の typical script に対し PNG 出力成功 (0 console errors)

dynamic block 実行確認

merge の影響を受ける各 dynamic block を実機で flag → 実行 → 状態確認:

機能	検証	結果
Array#sort!	[3,1,2].sort! → list 値	[1,2,3] ✅
Array#each	[10,20,30].each { total += n }	total = 60 ✅
Hash#keys + each	{a:1,b:2,c:3}.keys.each { say k }	keys list [:a,:b,:c], errors 0 ✅
String#reverse!	"hello".reverse!	_s_1_ = "olleh" ✅

[takaokouji]

さらに重要なバグ発見 — 配列メソッドの実行結果が壊れていた

「実行時にエラーがないことは当然として、結果が正しいことを確認してください」 という指示で実機検証したところ、新たな本番バグを発見したため scope に含めました。

現象

arr = [3, 1, 4, 1, 5, 9, 2, 6] のような単桁数値配列に対して:

式	期待値	修正前	修正後
arr.max	"9"	"31415926"	"9" ✅
arr.min	"1"	"31415926"	"1" ✅
arr.first	"3"	"31415926"	"3" ✅
arr.last	"6"	"31415926"	"6" ✅
arr.sort.join(",")	"1,1,2,3,4,5,6,9"	"31415926"	"1,1,2,3,4,5,6,9" ✅
arr.join("-")	"1-1-2-3-4-5-6-9"	"11234569"	"1-1-2-3-4-5-6-9" ✅

原因

scratch-vm/src/extensions/smalruby_ruby/method-executors.js の executeArrayMethod (非 bang) が args.RECEIVER を space-split で items に戻していた。Scratch の data_listcontents は 要素が全て 1 文字なら区切り無し で結合する仕様 ([3,1,4,1,5,9,2,6] → "31415926")。空白で split しても 1 要素しか復元できないため、max/min/sort/join/first/last すべてが結合文字列をそのまま返していた。

修正

util.thread.peekStack() からカレントブロック ID を取得 → target.blocks.getBlock(id) で RECEIVER 入力先のブロックを取得 → data_listcontents なら target.lookupVariableById で list を直接ルックアップして list.value で操作。リストでない受信者 (リテラル文字列など) は従来通り space-split にフォールバック。

ハッシュ・文字列メソッドは多くが多文字なので影響を受けない (検証済み)。

動作確認

実機 Playwright で arr = [3,1,4,1,5,9,2,6] を含むスクリプトを実行し、グローバル変数に各結果を格納してチェック。テーブル全項目で正しい値を確認済み。

文字列・ハッシュ・bang 含む追加 18 ケースも全て期待値:

• "hello world".reverse → "dlrow olleh"
• "hello world".gsub("world", "smalruby") → "hello smalruby"
• "abcd".reverse! (s2) → "dcba"
• {a:10, b:20, c:30}.values.each { total += v } → total = 60

既存 unit test (42 件) も全て pass。