meshV2: Create broadcast message locally if it doesn't exist when received from mesh

[takaokouji]

Summary

In the meshV2 extension, when a broadcast message is received from the mesh network, it might not exist in the local Scratch project. Currently, if the message name is not found in the project's variables, the broadcast is ignored, and the message doesn't appear in the block dropdowns.

Current Behavior

• src/extensions/scratch3_mesh_v2/mesh-service.js#broadcastEvent triggers event_broadcast opcode with a message name.
• Scratch3EventBlocks.broadcast (in src/blocks/scratch3_event.js) uses lookupBroadcastMsg.
• If lookupBroadcastMsg returns null (because the message was never created locally), no hats are started.
• The received message is not selectable in "when I receive [message]" or "broadcast [message]" block dropdowns because it doesn't exist in stage.variables.

Expected Behavior

• When a broadcast message is received via meshV2, if it doesn't exist locally, it should be created automatically on the stage.
• The new message should be selectable in the broadcast block dropdowns.
• The event_broadcast opcode should be able to trigger the hats for the newly created message.

Proposed Fix

In MeshV2Service (likely in handleBatchEvent or broadcastEvent), add logic to ensure the broadcast message exists:

1. Get the stage target: const stage = this.runtime.getTargetForStage();
2. Check if the message exists: let broadcastVar = stage.lookupBroadcastMsg(null, event.name);
3. If it doesn't exist:
   • Create it: stage.createVariable(null, event.name, Variable.BROADCAST_MESSAGE_TYPE);
   • Get the newly created variable: broadcastVar = stage.lookupBroadcastMsg(null, event.name);
   • Notify the GUI to refresh blocks: this.runtime.requestBlocksUpdate();
4. Use the id from broadcastVar in the args passed to event_broadcast.

Locations to modify

• src/extensions/scratch3_mesh_v2/mesh-service.js

References

• Broadcast message type: Variable.BROADCAST_MESSAGE_TYPE ('broadcast_msg')

[takaokouji]

調査結果

現在の実装の問題点

PR#85の現在の実装では、MeshV2Service.broadcastEvent が event_broadcast オペコードを直接呼び出しています：

this.blocks.opcodeFunctions.event_broadcast(args, util);

これは以下の問題を引き起こします：

1. BlockUtility依存の問題: BlockUtility.lastInstance() に依存していますが、メッシュからのイベント受信時には適切なインスタンスが存在しない可能性があります
2. 実行コンテキストの不一致: event_broadcast オペコードは、ブロックの実行コンテキストで動作することを想定しています
3. 間接的な呼び出し: 複数の層を経由するため、デバッグが困難です

メッセージマッチングの仕組み

ブロードキャストメッセージのマッチングは以下のように行われます：

1. startHats('event_whenbroadcastreceived', {BROADCAST_OPTION: messageName}) が呼び出される
2. RuntimeScriptCache がハットブロックのフィールド値を大文字化してキャッシュ（field.value.toUpperCase()）
3. startHats 内で optMatchFields の値も大文字化（value.toUpperCase()）
4. 両方を比較してマッチング

重要: 比較には変数の名前のみが使用され、変数IDは使用されません。

問題1: メッセージが重複作成される原因

初期状態で「メッセージ1」がドロップダウンに表示されているにもかかわらず、重複作成される原因：

1. プロジェクトファイルに event_whenbroadcastreceived ブロックが存在し、フィールドに変数IDと名前が保存されている
2. しかし、実際の変数オブジェクトが Stage.variables に存在しない（何らかの理由で削除されている）
3. lookupBroadcastByInputValue が変数を見つけられず、新しい変数が作成される

問題2: イベントハンドラが起動しない原因

メッセージ2のイベントハンドラが起動しない問題の仮説：

1. broadcastEvent が event_broadcast オペコードを経由して startHats を呼び出している
2. BlockUtility.lastInstance() が適切なインスタンスを返していない可能性がある
3. event_broadcast 内の lookupBroadcastMsg(id, name) が何らかの理由で失敗している

修正方針

主要な変更

MeshV2Service.broadcastEvent を以下のように変更します：

broadcastEvent (event) {
    log.info(`Mesh V2: Executing broadcastEvent for: ${event.name}`);
    try {
        const stage = this.runtime.getTargetForStage();
        let broadcastVar = stage.lookupBroadcastByInputValue(event.name);
        if (!broadcastVar) {
            log.info(`Mesh V2: Creating missing broadcast message: ${event.name}`);
            stage.createVariable(null, event.name, Variable.BROADCAST_MESSAGE_TYPE);
            broadcastVar = stage.lookupBroadcastByInputValue(event.name);
            if (broadcastVar) {
                broadcastVar.isPersistent = true;
            }
            this.runtime.requestBlocksUpdate();
        }

        if (broadcastVar) {
            // 直接 startHats を呼び出す（event_broadcast opcode を経由しない）
            log.info(`Mesh V2: Starting hats for broadcast: ${broadcastVar.name}`);
            const startedThreads = this.runtime.startHats('event_whenbroadcastreceived', {
                BROADCAST_OPTION: broadcastVar.name
            });
            log.info(`Mesh V2: Started ${startedThreads ? startedThreads.length : 0} threads for ${broadcastVar.name}`);
        } else {
            log.warn(`Mesh V2: Failed to find or create broadcast variable: ${event.name}`);
        }
    } catch (error) {
        log.error(`Mesh V2: Error in broadcastEvent: ${error}`);
    }
}

この変更により

1. ✅ BlockUtility への依存を削除
2. ✅ より直接的で理解しやすいコードになる
3. ✅ デバッグログが充実し、問題の診断が容易になる
4. ✅ startHats の戻り値を確認できる（何個のスレッドが起動されたか）

さらに確認が必要な項目

1. lookupBroadcastByInputValue が期待通りに動作しているか
2. プロジェクトファイルに変数が正しく保存されているか
3. ハットブロックのフィールド値が正しく設定されているか
4. 実際のログ出力を確認し、どの段階で問題が発生しているかを特定する

テスト計画

修正後、以下をテストする必要があります：

1. 初期状態でメッセージ1が存在する場合、重複作成されないこと
2. 新しいメッセージを作成して送信した場合、受信側でイベントハンドラが正しく起動すること
3. 大文字小文字が異なるメッセージ名でも正しくマッチングすること
4. 複数のノード間で同時にメッセージを送受信した場合も正しく動作すること