#75 wip: xboxビルドテスト v56

Author: ienaga

src/xbox.ts

@@ -6,6 +6,7 @@ import fs from "fs";
 import os from "os";
 import path from "path";
 import cp from "child_process";
+import { minifySync } from "vite";
 import { ctx } from "./context.js";
 import { getTemplateDir } from "./utils.js";
 import {
@@ -194,6 +195,35 @@ export const renderAssetsRc = (pakAbsPath: string): string =>
     return `N2DASSETS RCDATA "${forward}"\n`;
 };
 
+/**
+ * @description ホストスクリプト(js/bootstrap.js 等)を minify + 難読化する純関数。
+ *              Oxc(vite 同梱)でコメント除去・空白圧縮・ローカル識別子マングルを行う。
+ *              `globalThis.*` 等のグローバル名はゲームが参照するため保持される。
+ *              失敗時は元コードをそのまま返す(埋め込み自体は継続)。
+ *              Minify/obfuscate a host script; returns the original on failure.
+ *
+ * @param  {string} name  ファイル名 (拡張子で構文判定)
+ * @param  {string} code  元の JavaScript
+ * @return {string}
+ * @method
+ * @public
+ */
+export const minifyJs = (name: string, code: string): string =>
+{
+    try {
+        const result = minifySync(name, code);
+        if (result.errors && result.errors.length > 0) {
+            const msg = (result.errors[0] as any)?.message ?? String(result.errors[0]);
+            console.log(pc.yellow(`Xbox: minify skipped for ${name} (${msg}); embedding as-is.`));
+            return code;
+        }
+        return result.code;
+    } catch (error) {
+        console.log(pc.yellow(`Xbox: minify failed for ${name}; embedding as-is. ${error}`));
+        return code;
+    }
+};
+
 /**
  * @description Xbox ホストの assets/app とホストスクリプト(js/bootstrap.js 等)を
  *              単一の pak バイナリへまとめ、`assets.pak` と RCDATA 参照用の
@@ -247,10 +277,26 @@ const embedXboxAssets = (): Promise<void> =>
             }
 
             // (key, データ) を読み込み、pak バイナリへ直列化する。
+            // js/ のホストスクリプト(bootstrap.js/selftest.js)は minify + 難読化する。
+            // assets/app の JS は vite が本番ビルドで minify 済みのためそのまま。
+            let minified = 0;
             const pakEntries: [string, Buffer][] = entries.map(
-                ([key, abs]): [string, Buffer] => [key, fs.readFileSync(abs)]
+                ([key, abs]): [string, Buffer] => {
+                    if (key.startsWith("js/") && key.endsWith(".js")) {
+                        const src = fs.readFileSync(abs, { "encoding": "utf8" });
+                        const out = minifyJs(key, src);
+                        if (out.length < src.length) {
+                            ++minified;
+                        }
+                        return [key, Buffer.from(out, "utf8")];
+                    }
+                    return [key, fs.readFileSync(abs)];
+                }
             );
             const pak: Buffer = buildPak(pakEntries);
+            if (minified > 0) {
+                console.log(pc.green(`Minified ${minified} host script(s) before embedding.`));
+            }
             fs.writeFileSync(pakPath, pak);
 
             // rc.exe が assets.pak を RCDATA "N2DASSETS" として取り込む .rc を生成する。

templates/xbox/src/bindings/Console.cpp

@@ -30,7 +30,12 @@ void Log(const v8::FunctionCallbackInfo<v8::Value>& args)
 
 void Error(const v8::FunctionCallbackInfo<v8::Value>& args)
 {
-    std::cerr << FormatArgs(args.GetIsolate(), args) << std::endl;
+    // console.error / console.warn は GUI 実行では stderr が見えないため、
+    // next2d-error.log にもミラーする (player 側のエラー・警告や WebGPU の
+    // error scope 結果などを掴む)。
+    const std::string msg = FormatArgs(args.GetIsolate(), args);
+    std::cerr << msg << std::endl;
+    v8util::AppendErrorLog("[console] " + msg);
 }
 
 } // namespace

templates/xbox/src/gpu/DawnContext.cpp

@@ -1,22 +1,44 @@
 #include "DawnContext.h"
 
 #include <Windows.h>
+#include <fstream>
 #include <iostream>
+#include <string>
 
 namespace next2d {
 
 namespace {
 
+// GUI 実行では stderr が見えないため、GPU の致命的エラー/検証エラーを
+// next2d-error.log にも残す。描画が黒くなる/表示されない等の原因(検証エラー・
+// device lost・リソース枯渇)を掴む唯一の手段。氾濫防止に総数を制限する。
+void AppendGpuLog(const std::string& line)
+{
+    static int count = 0;
+    if (count >= 60) {
+        return;
+    }
+    ++count;
+    std::ofstream ofs("next2d-error.log", std::ios::app);
+    if (ofs) {
+        ofs << line << std::endl;
+    }
+}
+
 void HandleDeviceLost(const wgpu::Device&, wgpu::DeviceLostReason reason, wgpu::StringView message)
 {
-    std::cerr << "[Dawn] Device lost (" << static_cast<int>(reason) << "): "
-              << std::string(message.data, message.length) << std::endl;
+    const std::string line = "[Dawn] Device lost (" + std::to_string(static_cast<int>(reason))
+        + "): " + std::string(message.data, message.length);
+    std::cerr << line << std::endl;
+    AppendGpuLog(line);
 }
 
 void HandleUncapturedError(const wgpu::Device&, wgpu::ErrorType type, wgpu::StringView message)
 {
-    std::cerr << "[Dawn] Uncaptured error (" << static_cast<int>(type) << "): "
-              << std::string(message.data, message.length) << std::endl;
+    const std::string line = "[Dawn] Uncaptured error (" + std::to_string(static_cast<int>(type))
+        + "): " + std::string(message.data, message.length);
+    std::cerr << line << std::endl;
+    AppendGpuLog(line);
 }
 
 } // namespace

tests/xbox.test.ts

@@ -7,7 +7,8 @@ import path from "node:path";
 import {
     buildPak,
     renderAssetsRc,
-    walkFiles
+    walkFiles,
+    minifyJs
 } from "../dist/xbox.js";
 
 // host 側 EmbeddedAssets.cpp の ParseEmbeddedPak と同じ手順で pak を復号する検証用デコーダ。
@@ -80,3 +81,23 @@ test("walkFiles: 入れ子を posix 相対キーで収集", () => {
 test("walkFiles: 存在しないディレクトリは空配列", () => {
     assert.deepEqual(walkFiles(path.join(os.tmpdir(), "n2d-does-not-exist-xyz")), []);
 });
+
+test("minifyJs: 圧縮しコメント除去、グローバル名は保持", () => {
+    const src = `
+// このコメントは除去される
+globalThis.__next2d_boot = function () {
+    var longLocalVariableName = 1 + 2;
+    return longLocalVariableName;
+};
+`;
+    const out = minifyJs("js/bootstrap.js", src);
+    assert.ok(out.length < src.length, "縮む");
+    assert.ok(!out.includes("除去される"), "コメント除去");
+    assert.ok(out.includes("__next2d_boot"), "グローバル名は保持 (ゲームが参照)");
+    assert.ok(!/\blongLocalVariableName\b/.test(out), "ローカル名はマングル");
+});
+
+test("minifyJs: 不正な JS は元コードのまま返す (埋め込み継続)", () => {
+    const broken = "globalThis.x = (((;";
+    assert.equal(minifyJs("js/bootstrap.js", broken), broken);
+});