spoint 0.1.685 → 0.1.687

package/apps/tps-game/client-app.js

@@ -136,7 +136,7 @@ export const tpsGameClient = {
         if (lp && lp.velocity && dir) { lp.velocity[0] -= dir[0] * 2; lp.velocity[2] -= dir[2] * 2 }
         const animator = engine.players.getAnimator(engine.playerId)
         if (animator) animator.shoot()
-        tps.flash.color.setHex(0xffaa00); tps.flash.position.set(pos[0], pos[1] + 0.5, pos[2]); tps.flash.intensity = 5; tps.flash.distance = 12; tps.flashOff = Date.now() + 60
+        tps.flash.color.setHex(0xffaa00); tps.flash.position.set(pos[0], pos[1] + 0.5, pos[2]); tps.flash.intensity = 0; tps.flash.distance = 0; tps.flashOff = Date.now() + 60
         tps.ammo = Math.max(0, tps.ammo - 1)
         if (tps.juice) tps.juice.tone(160, 0.05, 0.22, 90)
         // tracer: origin -> full weapon range along dir; a real 'hit' event (below) shortens it to the actual impact point once the server responds, but the muzzle-to-somewhere streak reads correctly even before that RTT.
@@ -170,7 +170,7 @@ export const tpsGameClient = {
       tps._predHitAt = 0
       if (payload.headshot) { tps.headshotMarkerTime = now; if (window.__funJuice) window.__funJuice.headshot++; if (!justPredicted) tps.juice?.tone(1100, 0.08, 0.2); mobileVibrate(engine, [15, 30, 15]) }
       else { if (window.__funJuice) window.__funJuice.hit++; if (!justPredicted) tps.juice?.tone(820, 0.06, 0.18); mobileVibrate(engine, 20) }
-      if (payload.pos && tps.flash) { tps.flash.position.set(payload.pos[0], payload.pos[1], payload.pos[2]); tps.flash.color.setHex(0xffffff); tps.flash.intensity = 4; tps.flashOff = now + 80 }
+      if (payload.pos && tps.flash) { tps.flash.position.set(payload.pos[0], payload.pos[1], payload.pos[2]); tps.flash.color.setHex(0xffffff); tps.flash.intensity = 0; tps.flashOff = now + 80 }
       // A player hit doesn't get a scorch decal (blood-optional per roadmap #48 -- this engine has no
       // gore toggle yet, so player hits stay decal-free; only a miss against world geometry decals below).
       // celebrate the kill now on the lethal 'hit' (RTT sooner); the 'death' path below de-dupes against this

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/client/core/Grass.js

@@ -57,7 +57,7 @@ export async function createGrass(opts = {}) {
   // terrain-shader shell/noise band) would duplicate work the streaming ring + fade already do, for a
   // regime (grass blades near the world's own draw-distance edge) that's rarely on-screen densely enough
   // to matter. This is the documented "simpler, safely achievable" branch named in the task.
-  const LOD_NEAR_DIST = Number.isFinite(cfg.grassLodNearDistance) ? cfg.grassLodNearDistance : 15
+  const LOD_NEAR_DIST = Number.isFinite(cfg.grassLodNearDistance) ? cfg.grassLodNearDistance : 8
   const geoNear = makeBladeGeo(5)     // full 5-segment curved blade, 20 tris/blade (10 quads x 2 tris)
   const geoMid = makeBladeGeo(1)      // flat single-segment crossed quad, 4 tris/blade
   const mat = makeGrassMaterial(wind)

package/client/core/SceneSetup.js

@@ -290,7 +290,7 @@ export async function probeAndCreateWebGPURenderer(isMobile) {
 }
 
 export function setupLights(scene) {
-  const ambient = new THREE.AmbientLight(0xfff4d6, 0.3)
+  const ambient = new THREE.AmbientLight(0xfff4d6, 0.5)
   scene.add(ambient)
   ambient.updateMatrix(); ambient.matrixAutoUpdate = false
   const studio = new THREE.DirectionalLight(0x4488ff, 0.4)

package/client/core/TimeOfDay.js

@@ -39,11 +39,11 @@ const _DEG = Math.PI / 180
 const KEYFRAMES = [
   { deg: -90, sunColor: 0x0a1030, sunIntensity: 0.0, ambientColor: 0x0d1428, ambientIntensity: 0.10 }, // deep night
   { deg: -6, sunColor: 0x1a2550, sunIntensity: 0.0, ambientColor: 0x1a2440, ambientIntensity: 0.14 }, // astronomical twilight, sun still below horizon (no direct light)
-  { deg: 0, sunColor: 0xff7a3c, sunIntensity: 0.55, ambientColor: 0x4a4060, ambientIntensity: 0.22 }, // horizon: warm orange sunrise/sunset
-  { deg: 8, sunColor: 0xffa552, sunIntensity: 1.8, ambientColor: 0xffcf9e, ambientIntensity: 0.55 }, // low sun: golden hour (matches tps-game.js's intended fill-light-visible palette)
-  { deg: 30, sunColor: 0xfff0dc, sunIntensity: 1.45, ambientColor: 0xc9d4e8, ambientIntensity: 0.32 }, // mid-morning/afternoon
-  { deg: 70, sunColor: 0xffffff, sunIntensity: 1.6, ambientColor: 0xfff4d6, ambientIntensity: 0.3 }, // near-noon, matches SceneSetup's original static values
-  { deg: 90, sunColor: 0xffffff, sunIntensity: 1.55, ambientColor: 0xfff4d6, ambientIntensity: 0.3 }, // straight overhead
+  { deg: 0, sunColor: 0xff7a3c, sunIntensity: 0.55, ambientColor: 0x4a4060, ambientIntensity: 0.34 }, // horizon: warm orange sunrise/sunset
+  { deg: 8, sunColor: 0xffa552, sunIntensity: 1.8, ambientColor: 0xffcf9e, ambientIntensity: 0.65 }, // low sun: golden hour (matches tps-game.js's intended fill-light-visible palette)
+  { deg: 30, sunColor: 0xfff0dc, sunIntensity: 1.45, ambientColor: 0xc9d4e8, ambientIntensity: 0.48 }, // mid-morning/afternoon
+  { deg: 70, sunColor: 0xffffff, sunIntensity: 1.6, ambientColor: 0xfff4d6, ambientIntensity: 0.5 }, // near-noon, Performance-Mode flat-ambient interior floor (was 0.3, too dark indoors with no bounce/GI)
+  { deg: 90, sunColor: 0xffffff, sunIntensity: 1.55, ambientColor: 0xfff4d6, ambientIntensity: 0.5 }, // straight overhead
 ]
 
 const _c1 = new THREE.Color(), _c2 = new THREE.Color(), _cOut = new THREE.Color()

package/package.json

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