fix(studio-bridge): sync action modules before execute in process run

StudioBridgeServer.executeAsync() sent the execute message without first
registering the execute.luau action handler with the plugin, causing
UNKNOWN_REQUEST errors. Add _ensureActionsAsync() to sync dynamic action
modules (via syncActions + registerAction) before the first execute call,
matching the pattern already used by BridgeSession.

Author: Quenty

.github/workflows/studio-linux-e2e.yml

@@ -44,16 +44,9 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Setup Aftman
-        run: |
-          mkdir -p ~/.aftman/bin
-          curl -fsSL https://github.com/LPGhatguy/aftman/releases/download/v0.3.0/aftman-0.3.0-linux-x86_64.zip -o /tmp/aftman.zip
-          unzip -o /tmp/aftman.zip -d /tmp/aftman
-          install -m 755 /tmp/aftman/aftman ~/.aftman/bin/aftman
-          echo "$HOME/.aftman/bin" >> "$GITHUB_PATH"
-          export PATH="$HOME/.aftman/bin:$PATH"
-          aftman install --no-trust-check
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        uses: ok-nick/setup-aftman@v0.4.2
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Build all tools
         run: pnpm -r --filter './tools/**' run build
@@ -78,33 +71,21 @@ jobs:
         env:
           ROBLOSECURITY: ${{ secrets.ROBLOSECURITY }}
 
-      - name: Collect Studio logs
+      - name: Print logs
         if: always()
         run: |
-          mkdir -p /tmp/studio-logs
-          cp /tmp/studio-bridge-wine.log /tmp/studio-logs/wine.log 2>/dev/null || true
-          # Roblox Studio internal logs (inside Wine prefix)
-          find ~/.wine/drive_c/users/ -path "*/Roblox/logs/*" -name "*.log" -exec cp {} /tmp/studio-logs/ \; 2>/dev/null || true
-          # Also check for crash dumps or error logs
-          find ~/.wine/drive_c/users/ -path "*/Roblox/*.log" -exec cp {} /tmp/studio-logs/ \; 2>/dev/null || true
-          # List what we found
-          echo "=== Collected log files ==="
-          ls -la /tmp/studio-logs/ 2>/dev/null || echo "No logs found"
-          echo ""
           echo "=== Wine log (last 100 lines) ==="
-          tail -100 /tmp/studio-logs/wine.log 2>/dev/null || echo "No Wine log"
+          tail -100 /tmp/studio-bridge-wine.log 2>/dev/null || echo "No Wine log"
           echo ""
           echo "=== Studio logs ==="
-          for f in /tmp/studio-logs/*.log; do
-            [ "$f" = "/tmp/studio-logs/wine.log" ] && continue
-            echo "--- $f ---"
-            tail -50 "$f" 2>/dev/null
-          done
+          tail -50 ~/.wine/drive_c/users/*/AppData/Local/Roblox/logs/*.log 2>/dev/null || echo "No Studio logs"
 
       - name: Upload logs
         if: always()
         uses: actions/upload-artifact@v4
         with:
           name: studio-bridge-logs
-          path: /tmp/studio-logs/
+          path: |
+            /tmp/studio-bridge-wine.log
+            ~/.wine/drive_c/users/*/AppData/Local/Roblox/logs/
           if-no-files-found: ignore

tools/studio-bridge/docker/Dockerfile

@@ -21,9 +21,14 @@ RUN dpkg --add-architecture i386 \
     && apt-get install -y --no-install-recommends winehq-stable \
     && apt-get clean && rm -rf /var/lib/apt/lists/*
 
-# --- Node.js 22 LTS ---
+# --- Node.js 22 LTS + GitHub CLI (needed by setup-aftman action) ---
 RUN curl -fsSL https://deb.nodesource.com/setup_22.x | bash - \
-    && apt-get install -y --no-install-recommends nodejs \
+    && curl -fsSL https://cli.github.com/packages/githubcli-archive-keyring.gpg \
+        -o /usr/share/keyrings/githubcli-archive-keyring.gpg \
+    && echo "deb [arch=$(dpkg --print-architecture) signed-by=/usr/share/keyrings/githubcli-archive-keyring.gpg] https://cli.github.com/packages stable main" \
+        > /etc/apt/sources.list.d/github-cli.list \
+    && apt-get update \
+    && apt-get install -y --no-install-recommends nodejs gh \
     && corepack enable pnpm \
     && apt-get clean && rm -rf /var/lib/apt/lists/*
 

tools/studio-bridge/src/server/studio-bridge-server.ts

@@ -30,6 +30,10 @@ import {
   decodePluginMessage,
 } from './web-socket-protocol.js';
 import { ActionDispatcher } from './action-dispatcher.js';
+import {
+  loadActionSourcesAsync,
+  type ActionSource,
+} from '../commands/framework/action-loader.js';
 import {
   injectPluginAsync,
   type InjectedPlugin,
@@ -218,6 +222,8 @@ export class StudioBridgeServer {
   private _negotiatedCapabilities: Capability[] = ['execute'];
   private _lastHeartbeatTimestamp: number | undefined;
   private _actionDispatcher = new ActionDispatcher();
+  private _actionsReady = false;
+  private _actionSources: ActionSource[] | undefined;
 
   constructor(options: StudioBridgeServerOptions = {}) {
     this._options = options;
@@ -420,6 +426,10 @@ export class StudioBridgeServer {
       throw new Error('Cannot execute: no connected client');
     }
 
+    // Sync action modules before first execute (state must still be 'ready'
+    // because performActionAsync checks for it).
+    await this._ensureActionsAsync();
+
     this._state = 'executing';
     this._onPhase?.('executing');
 
@@ -483,6 +493,75 @@ export class StudioBridgeServer {
     this._state = 'stopped';
   }
 
+  // -----------------------------------------------------------------------
+  // Private: _ensureActionsAsync
+  // -----------------------------------------------------------------------
+
+  /**
+   * Ensure action modules (like `execute.luau`) are synced to the plugin
+   * before first use. Uses `syncActions` to check which actions the plugin
+   * is missing, then registers them via `registerAction`.
+   *
+   * Only works with v2 plugins. v1 plugins skip this step (they would need
+   * actions baked in, which the current plugin architecture does not support).
+   */
+  private async _ensureActionsAsync(): Promise<void> {
+    if (this._actionsReady) return;
+
+    if (this._negotiatedProtocolVersion < 2) {
+      this._actionsReady = true;
+      return;
+    }
+
+    if (!this._actionSources) {
+      this._actionSources = await loadActionSourcesAsync();
+      OutputHelper.verbose(
+        `[StudioBridge] Loaded ${this._actionSources.length} action source(s): ${this._actionSources.map((a) => a.name).join(', ') || '(none)'}`,
+      );
+    }
+
+    if (this._actionSources.length === 0) {
+      this._actionsReady = true;
+      return;
+    }
+
+    const actions: Record<string, string> = {};
+    for (const action of this._actionSources) {
+      actions[action.name] = action.hash;
+    }
+
+    OutputHelper.verbose('[StudioBridge] Syncing actions with plugin');
+    const syncResult = await this.performActionAsync<PluginMessage>({
+      type: 'syncActions',
+      payload: { actions },
+    }, 10_000);
+
+    if (syncResult.type === 'syncActionsResult') {
+      const needed = (syncResult.payload as Record<string, unknown>).needed as string[];
+      OutputHelper.verbose(
+        `[StudioBridge] ${needed.length} action(s) need registering${needed.length > 0 ? ': ' + needed.join(', ') : ''}`,
+      );
+
+      for (const actionName of needed) {
+        const action = this._actionSources.find((a) => a.name === actionName);
+        if (!action) continue;
+
+        OutputHelper.verbose(`[StudioBridge] Registering action: ${actionName}`);
+        await this.performActionAsync<PluginMessage>({
+          type: 'registerAction',
+          payload: {
+            name: action.name,
+            source: action.source,
+            hash: action.hash,
+          },
+        }, 10_000);
+      }
+    }
+
+    this._actionsReady = true;
+    OutputHelper.verbose('[StudioBridge] Action sync complete');
+  }
+
   // -----------------------------------------------------------------------
   // Private: _injectPluginAsync
   // -----------------------------------------------------------------------