spoint 0.1.685 → 0.1.686

package/client/app.js

@@ -905,7 +905,7 @@ const engineCtx = {
   get worldConfig() { return worldConfig }, get inputConfig() { return inputConfig },
   playerVrms: pm.playerVrms, entityAppMap, kit: _designKit,
   network: { send: msg => client.send(0x33, msg) },
-  setInputConfig(cfg) { Object.assign(inputConfig,cfg); if (!inputConfig.pointerLock) { if (clickPrompt) clickPrompt.style.display='none'; if (document.pointerLockElement) document.exitPointerLock() } },
+  setInputConfig(cfg) { Object.assign(inputConfig,cfg); if (!inputConfig.pointerLock) { if (clickPrompt) clickPrompt.style.display='none'; if (document.pointerLockElement) _safeExitPointerLock() } },
   players: { getMesh: id=>pm.playerMeshes.get(id), getState: id=>pm.playerStates.get(id), getAnimator: id=>pm.playerAnimators.get(id), setExpression: (id,n,v)=>pm.setVRMExpression(id,n,v), setAiming: (id,v)=>{ const s=pm.playerStates.get(id); if (s) s._aiming=v } },
   // Pooled hit decals + hitscan tracers (roadmap #48/#93). point/normal/origin/target are [x,y,z] world-space.
   decals: { spawnDecal: (point, normal) => decalSystem.spawnDecal(point, normal), spawnTracer: (origin, target) => decalSystem.spawnTracer(origin, target) },
@@ -1941,7 +1941,7 @@ editor.onSelectionChange((id,data) => {
   // selectEntity's machine.send('DESELECT')) clears this client's presence for every other connected editor.
   editorPresence.sendPresence(id, false)
 })
-editor.onEditModeChange(on => { cam.setEditMode(on, pm.playerMeshes.get(client.playerId)); if (on) { if (document.pointerLockElement) document.exitPointerLock(); editPanel.show(); client.send(MSG.SCENE_GRAPH,{}); client.send(MSG.LIST_APPS,{}) } else { editPanel.hide(); editorPresence.hide(); editorPresence.sendPresence(null, false) } })
+editor.onEditModeChange(on => { cam.setEditMode(on, pm.playerMeshes.get(client.playerId)); if (on) { if (document.pointerLockElement) _safeExitPointerLock(); editPanel.show(); client.send(MSG.SCENE_GRAPH,{}); client.send(MSG.LIST_APPS,{}) } else { editPanel.hide(); editorPresence.hide(); editorPresence.sendPresence(null, false) } })
 // Reverse sync: editor.js's own Y/Alt+P keyboard shortcuts change _gizmoSpace/_pivotMode directly
 // (drag math needs zero round-trip latency), so mirror the toolbar's displayed value after the fact.
 editor.onGizmoSpaceChange(space => { try { editPanel.setGizmoSpace(space) } catch (_) {} })
@@ -2200,11 +2200,24 @@ document.addEventListener('keydown', e => { const _mod=e.ctrlKey||e.metaKey; con
 // Match/Invite Friends, shown on Esc when pointer-lock exits DURING ACTIVE GAMEPLAY (not the
 // initial click-to-play prompt -- _hasEverLocked below distinguishes the two, since pointerlockchange
 // fires on both the very first click-prompt dismissal and every later in-match Esc).
+// document.exitPointerLock() can throw the same class of error as requestPointerLock() in unusual
+// document/element states (observed live: "The root document of this element is not valid for
+// pointer lock" from an exitPointerLock() call, not just request). 5 call sites across this file
+// called it raw; centralize the guard here so every future call site gets it for free.
+function _safeExitPointerLock() {
+  try { document.exitPointerLock() } catch (e) { console.warn('[input] exitPointerLock failed:', e?.message || e) }
+}
+function _safeRequestPointerLock() {
+  try {
+    const p = renderer.domElement.requestPointerLock()
+    if (p && typeof p.catch === 'function') p.catch(e => console.warn('[input] requestPointerLock rejected:', e?.message || e))
+  } catch (e) { console.warn('[input] requestPointerLock failed:', e?.message || e) }
+}
 const settingsMenu = createSettingsMenu({ getCam: () => cam, getRenderer: () => renderer })
 if (window.__app) window.__app.settingsMenu = settingsMenu
 let _hasEverLocked = false
 const pauseMenu = createPauseMenu({
-  requestPointerLock: () => { try { renderer.domElement.requestPointerLock() } catch (e) { console.warn('[input] requestPointerLock failed:', e?.message || e) } },
+  requestPointerLock: _safeRequestPointerLock,
   settingsMenu,
   getRoomInfo: () => _wwRoom ? { code: _wwRoom, joinLink: `${location.origin}${location.pathname}?wwjoin&room=${_wwRoom}` } : null,
 })
@@ -2219,7 +2232,7 @@ document.addEventListener('keydown', e => {
   // Editor has its own Esc-adjacent flows (deselect etc.) via editor.onKeyDown above; don't also pop
   // the gameplay pause menu while editing, and don't pop it over the lobby overlay either.
   if (clientMachine.isEditor || clientMachine.isLobby) return
-  if (_hasEverLocked && document.pointerLockElement === renderer.domElement) { document.exitPointerLock() }
+  if (_hasEverLocked && document.pointerLockElement === renderer.domElement) { _safeExitPointerLock() }
 })
 // Spectator mode entry/exit + follow-target-cycle keybinds. O enters spectator (S/Space are already
 // gameplay movement keys); inside spectator: F toggles free<->follow, [ and ] cycle the follow target.
@@ -2247,7 +2260,7 @@ function _setViewportRelaxed(on) { if (_viewportMeta) _viewportMeta.setAttribute
 clientMachine.subscribe(() => {
   const wantLobby = clientMachine.isLobby
   if (wantLobby && !(_lobby && _lobby.isOpen)) {
-    if (document.pointerLockElement) document.exitPointerLock()
+    if (document.pointerLockElement) _safeExitPointerLock()
     _setViewportRelaxed(true)
     // Re-check isLobby after the async import gap in case the player toggled again before it resolved.
     _getLobby().then(lobby => { if (clientMachine.isLobby && !lobby.isOpen) lobby.open() })
@@ -2274,7 +2287,7 @@ let _specWasSpectator = false
 clientMachine.subscribe(() => {
   const isSpec = clientMachine.isSpectator
   if (isSpec && !_specWasSpectator) {
-    if (document.pointerLockElement) document.exitPointerLock()
+    if (document.pointerLockElement) _safeExitPointerLock()
     if (clientMachine.spectatorSubmode === 'free') cam.setEditMode(true)
   } else if (!isSpec && _specWasSpectator) {
     cam.setEditMode(false)
@@ -2426,15 +2439,9 @@ renderer.domElement.addEventListener('click', ()=>{
   if (!inputConfig.pointerLock || document.pointerLockElement) return
   // requestPointerLock() can both throw synchronously AND return a Promise that rejects
   // asynchronously (spec-dependent by browser/context) -- e.g. "The root document of this
-  // element is not valid for pointer lock". A try/catch alone only catches the synchronous
-  // case; the async rejection surfaces as an unhandledrejection that still trips the app's
-  // global error boundary, converting a benign lock-denial into a full page crash (reproduced
-  // live even after the sync try/catch was added). Guard both paths. The keydown fallback below
-  // still recovers gameplay either way.
-  try {
-    const p = renderer.domElement.requestPointerLock()
-    if (p && typeof p.catch === 'function') p.catch(e => console.warn('[input] requestPointerLock rejected:', e?.message || e))
-  } catch (e) { console.warn('[input] requestPointerLock failed:', e?.message || e) }
+  // element is not valid for pointer lock". _safeRequestPointerLock guards both. The keydown
+  // fallback below still recovers gameplay either way.
+  _safeRequestPointerLock()
 })
 // requestPointerLock() is a trusted-gesture-gated browser API that can silently reject (denied
 // engagement heuristic, automation-driven click, embedded/iframe context) with no visible error --

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spoint",
-  "version": "0.1.685",
+  "version": "0.1.686",
   "description": "Physics and netcode SDK for multiplayer game servers",
   "type": "module",
   "workspaces": [