@uzuhq/code-cli 0.3.19 → 0.3.20

package/dist/dev-server/game-room.js

@@ -19,7 +19,10 @@ import { SeededRandomImpl } from './random.js';
  * dev-server は SDK を import できないため実装を持つ。
  */
 function isServerOnlyAction(handler) {
-    return '__serverOnly' in handler && handler.__serverOnly === true;
+    // handler は logic.actions[name] の結果なので undefined になりうる。型上は
+    // undefined を含まない (noUncheckedIndexedAccess 無効) ため tsc では気付けない。
+    // typeof ガードが無いと `'__serverOnly' in undefined` で TypeError になる。
+    return (typeof handler === 'function' && '__serverOnly' in handler && handler.__serverOnly === true);
 }
 export class GameRoom {
     logic;
@@ -331,24 +334,45 @@ export class GameRoom {
         }
     }
     async dispatchAction(actionName, payload, senderId, ackSeq) {
-        const handler = this.logic.actions[actionName];
-        if (!handler) {
+        const plain = this.logic.actions[actionName];
+        // 移行期の互換: 旧 serverOnly() を actions に入れたままの logic も動かす。
+        const legacyServerOnly = isServerOnlyAction(plain) ? plain : null;
+        const serverHandler = this.logic.serverActions?.[actionName] ?? legacyServerOnly;
+        if (!plain && !serverHandler) {
             throw new Error(`Unknown action: ${actionName}`);
         }
         if (!this.gameState) {
             throw new Error('Game not started');
         }
+        // actions 由来は送信者側で先読み時に発火済みなので ack で skip されるが、
+        // serverActions 由来は先読みで走っていないため別枠で常に配信する。
         const events = [];
+        const serverEvents = [];
         const emit = (name, data) => events.push({ name, data: data ?? {} });
-        // serverOnly handler はサーバーでしか走らないので tick を渡せる。素の handler は
-        // クライアント先読みでも同じコードが走るため、そこで再現できない値は渡さない。
-        if (isServerOnlyAction(handler)) {
-            await handler(this.gameState, payload ?? {}, senderId, emit, { tick: this.tickCount });
+        const serverEmit = (name, data) => serverEvents.push({ name, data: data ?? {} });
+        // action は「全部成功か全部失敗か」にする。plain が state を変更したあと
+        // serverActions が throw すると、caller の catch が __action_error を返して
+        // broadcast されないまま変更が gameState に残り、次の無関係な broadcast の diff に
+        // 紛れて漏れる。クライアントは __action_error で予測を rollback するので、
+        // 巻き戻さないとサーバー真実と表示が食い違ったままになる。
+        const snapshot = structuredClone(this.gameState);
+        try {
+            // 決定的な部分を先に走らせ、serverActions がその結果を見られるようにする。
+            if (plain && !legacyServerOnly) {
+                plain(this.gameState, payload ?? {}, senderId, emit, {});
+            }
+            if (serverHandler) {
+                await serverHandler(this.gameState, payload ?? {}, senderId, serverEmit, {
+                    tick: this.tickCount,
+                    random: this.random ?? new SeededRandomImpl(this.seed),
+                });
+            }
         }
-        else {
-            handler(this.gameState, payload ?? {}, senderId, emit, {});
+        catch (err) {
+            this.gameState = snapshot;
+            throw err;
         }
-        this.broadcastStateDelta(events, { ack: ackSeq, from: senderId });
+        this.broadcastStateDelta(events, { ack: ackSeq, from: senderId, serverEvents });
     }
     handleClose(ws) {
         const attachment = this.attachments.get(ws);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uzuhq/code-cli",
-  "version": "0.3.19",
+  "version": "0.3.20",
   "description": "UZU ゲーム開発 CLI - ビルド・パブリッシュ・プロジェクト作成ツール",
   "type": "module",
   "bin": {