@wasm-gaming/engine-specs 0.2.5 → 0.2.6

package/demo/demo.js

@@ -307,7 +307,11 @@ const demo = {
         assetKey: "rom",
       });
 
-      const handleSaveRom = (file) => {
+      // Handlers bound to a jq79 mount are called as (event, payload) — the
+      // DOM CustomEvent first, the emitted value second. `demo.on()` below is
+      // a different emitter that passes the payload alone, so the two must not
+      // be read the same way.
+      const handleSaveRom = (_event, file) => {
         loadedFiles.rom = file;
         saveStoredAsset("rom", file);
         const payload = { file, key: "rom", type: "rom", action: "save" };
@@ -327,7 +331,7 @@ const demo = {
         demo.emit("file", payload);
       };
 
-      const handleSaveBios = (file) => {
+      const handleSaveBios = (_event, file) => {
         loadedFiles.bios = file;
         saveStoredAsset("bios", file);
         const payload = { file, key: "bios", type: "bios", action: "save" };
@@ -377,7 +381,7 @@ const demo = {
           .on("remove:rom", handleRemoveRom)
           .on("save:bios", handleSaveBios)
           .on("remove:bios", handleRemoveBios)
-          .on("launch", (payload) => bootWithFiles(payload))
+          .on("launch", (_event, payload) => bootWithFiles(payload))
           .mount(mainEl);
       };
 
@@ -468,6 +472,17 @@ const demo = {
                 customItems: escMenuConfig.items || [],
                 messages,
               })
+              .mount(escContainer);
+          };
+
+          // Bound once, not per render. The $emit channel lives on the
+          // Component79 instance and deliberately survives destroy() — that is
+          // what lets it work while detached — so re-registering these inside
+          // updateEscMenu() stacked a fresh set of listeners on every open,
+          // close and restore-defaults. After N re-renders a single click
+          // reached the handler N times, growing without bound.
+          const bindEscMenuEvents = () => {
+            CEscMenu
               .on("close", () => closeMenu())
               .on("reset", () => {
                 closeMenu();
@@ -499,7 +514,7 @@ const demo = {
                   escMenuConfig.onRestoreDefaults(currentInstance);
                 }
               })
-              .on("option-change", ({ key, value, option }) => {
+              .on("option-change", (_event, { key, value, option }) => {
                 if (currentInstance && typeof currentInstance.setOption === "function") {
                   currentInstance.setOption(key, value);
                 }
@@ -533,14 +548,13 @@ const demo = {
                 }
                 exitToLauncher();
               })
-              .on("custom-action", (item) => {
+              .on("custom-action", (_event, item) => {
                 closeMenu();
                 demo.emit("customAction", { item, instance: currentInstance });
                 if (typeof item?.action === "function") {
                   item.action(currentInstance);
                 }
-              })
-              .mount(escContainer);
+              });
           };
 
           const openMenu = () => {
@@ -581,6 +595,7 @@ const demo = {
           if (typeof window !== "undefined") {
             window.addEventListener("keydown", handleKeyDown);
           }
+          bindEscMenuEvents();
           updateEscMenu();
         }
       };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wasm-gaming/engine-specs",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "The wasm-gaming engine contract: the single shared interface every WASM game/emulator engine and the host app conform to. TypeScript types + JSON Schema + a Zod runtime validator.",
   "type": "module",
   "types": "./dist/index.d.ts",