spoint 0.1.673 → 0.1.675

package/client/BrowserServer.js

@@ -227,8 +227,12 @@ export class BrowserServer extends BaseClient {
   }
 }
 
+// Matches relative ('./x', '../x') AND root-absolute ('/x') import specifiers: the gh-pages deploy's
+// path-patch step (gh-pages.yml "Patch paths for gh-pages") absolutizes deep relative src imports
+// (apps/**/*.js climbing out via '../../src/...') into '/spoint/src/...' at build time, so a shipped
+// app's own source can carry either form depending on how many directories deep it lives.
 async function _resolveRelativeDeps(source, baseUrl, seen = new Map()) {
-  const re = /(?:from|import)\s*['"](\.[^'"]+)['"]/g
+  const re = /(?:from|import)\s*['"](\.[^'"]+|\/[^'"]+)['"]/g
   const out = {}
   const tasks = []
   let m

package/client/app.js

@@ -2753,6 +2753,10 @@ function buildFrameSectionNodes() {
     {
       id: 'frame-clock',
       reads: [], writes: ['frameDt', 'isEditorFrame', 'lerpFactor', 'localId'],
+      // lerpFactor is a debug mirror (inspectable via window.__renderGraph) with no downstream graph
+      // reader by design; frameDt/isEditorFrame/localId ARE read elsewhere, so only lerpFactor is
+      // exempt here, not the whole node.
+      debugMirrors: ['lerpFactor'],
       run(ctx) {
         const now = ctx.now
         ctx.res.frameDt = Math.min(Math.max((now - lastFrameTime) / 1000, 0.001), 0.1)
@@ -2797,6 +2801,9 @@ function buildFrameSectionNodes() {
     {
       id: 'scene-graph-tick',
       reads: ['frameDt', 'isEditorFrame', 'localId'], writes: ['sceneGraphMoved'],
+      // Intentionally-terminal completion marker: no downstream node reads sceneGraphMoved, it just
+      // records that this frame's scene-graph tick happened (inspectable via window.__renderGraph).
+      terminal: true,
       run(ctx) {
         const lid = ctx.res.localId
         if (_hierarchyDirty && latestState && latestState.entities.length > 0) { el.rebuildEntityHierarchy(latestState.entities); _hierarchyDirty = false }
@@ -2830,6 +2837,9 @@ function buildFrameSectionNodes() {
     {
       id: 'app-dispatch-frame',
       reads: ['frameDt'], writes: ['appFrameDispatched'],
+      // Intentionally-terminal completion marker: no downstream node reads appFrameDispatched, it
+      // just records that this frame's app dispatch happened (inspectable via window.__renderGraph).
+      terminal: true,
       run(ctx) {
         ams.dispatchFrame(ctx.res.frameDt, engineCtx)
         if (engineCtx.facial) engineCtx.facial.update(ctx.res.frameDt)
@@ -2853,6 +2863,10 @@ function buildFrameSectionNodes() {
     {
       id: 'camera-input-update',
       reads: ['frameDt', 'isEditorFrame', 'localId'], writes: ['localState', 'vegFocus'],
+      // localState is a debug mirror (inspectable via window.__renderGraph) with no downstream
+      // graph reader by design; vegFocus IS read (floating-origin-rebase), so only localState is
+      // exempt here, not the whole node.
+      debugMirrors: ['localState'],
       run(ctx) {
         const lid = ctx.res.localId
         const local = client.getLocalState() || pm.playerStates.get(lid)
@@ -2901,6 +2915,9 @@ function buildFrameSectionNodes() {
       // *shape* changes from {position:[...]} to {x,y,z}, both read the same way by consumers below).
       id: 'floating-origin-rebase',
       reads: ['vegFocus'], writes: ['originRebased'],
+      // Intentionally-terminal completion marker: no downstream node reads originRebased, it just
+      // records whether this frame rebased (inspectable via window.__renderGraph).
+      terminal: true,
       run(ctx) {
         const p = camera.position
         const rebased = floatingOrigin.update(p.x, p.y, p.z)
@@ -2962,6 +2979,9 @@ function buildFrameSectionNodes() {
     {
       id: 'shadow-move-gate',
       reads: ['localId'], writes: ['shadowMoved'],
+      // Intentionally-terminal completion marker: no downstream node reads shadowMoved, it just
+      // records whether this frame's shadow camera moved (inspectable via window.__renderGraph).
+      terminal: true,
       run(ctx) {
         const _shadowTgt = pm.playerMeshes.get(ctx.res.localId)?.position || camera.position
         // ShadowPipeline (single owner) follows the player with a TEXEL-SNAPPED shadow camera so the

package/client/core/RenderGraph.js

@@ -77,16 +77,6 @@ export function createRenderGraph(nodes, opts = {}) {
     if (!readersOf.has(key)) readersOf.set(key, [])
     readersOf.get(key).push(n.id)
   }
-  // written-never-read: a construction-time warning, not an error (a terminal debug mirror is legal,
-  // but an unread resource is usually a consumer someone forgot to wire -- the silently-dropped-
-  // bridge class this repo keeps re-finding).
-  const unreadKeys = []
-  for (const [key, writer] of writerOf) {
-    if (!readersOf.has(key)) {
-      unreadKeys.push(key)
-      console.warn(`[render-graph] resource '${key}' (written by '${writer}') has no reader -- debug mirror or missing consumer?`)
-    }
-  }
   // Render-target aliasing (the other tractable half of the row): group resource KEYS by their
   // declared physical target (node.targets[key], defaulting to the key itself -- i.e. "no sharing
   // assumed" unless a node author opts in). Two distinct keys sharing one target is legitimate
@@ -112,6 +102,25 @@ export function createRenderGraph(nodes, opts = {}) {
     if (!keysByTarget.has(target)) keysByTarget.set(target, [])
     keysByTarget.get(target).push({ key, writer: writerOf.get(key) })
   }
+  // written-never-read: a construction-time warning, not an error (a terminal debug mirror is legal,
+  // but an unread resource is usually a consumer someone forgot to wire -- the silently-dropped-
+  // bridge class this repo keeps re-finding). Exempt: (1) a declared non-self target -- the
+  // canvas/physical-target IS the real consumer, same as dead-pass detection below; (2) a writer
+  // node marked required/terminal -- the WHOLE node is an intentionally-terminal completion marker;
+  // (3) a key listed in the writer's own `debugMirrors` -- a PER-KEY escape for a node that writes
+  // some resources real consumers read and others that are deliberately self-contained debug
+  // mirrors (e.g. terrain-depth-color's 'camera-context', read inline via ctx.res, not a declared
+  // `reads` edge), so marking the whole node terminal would wrongly also silence a genuinely-
+  // missing consumer on one of its OTHER writes.
+  const unreadKeys = []
+  for (const [key, writer] of writerOf) {
+    if (readersOf.has(key) || hasDeclaredTarget.has(key)) continue
+    const writerNode = byId.get(writer)
+    if (writerNode && (writerNode.required || writerNode.terminal)) continue
+    if (writerNode && writerNode.debugMirrors && writerNode.debugMirrors.includes(key)) continue
+    unreadKeys.push(key)
+    console.warn(`[render-graph] resource '${key}' (written by '${writer}') has no reader -- debug mirror or missing consumer?`)
+  }
   // Dead-pass detection (construction-time, static -- the tractable half of "auto-cull dead
   // passes"): a node whose EVERY declared write is BOTH unread by another node's `reads` AND not
   // aliased onto a shared physical target (targets[key]) is a real candidate for a forgotten/

package/client/core/RenderGraph.nodes.js

@@ -83,6 +83,10 @@ export function buildRenderSectionNodes() {
       reads: ['hostNearFar'],
       writes: ['camera-context', 'terrainDepth', 'terrainColor'],
       targets: { terrainDepth: 'canvas', terrainColor: 'canvas' },
+      // 'camera-context' is read inline via ctx.res['camera-context'] below (a same-node self-
+      // consume), never through a declared `reads` edge -- a real, intentional debug mirror, not a
+      // forgotten consumer.
+      debugMirrors: ['camera-context'],
       shouldRun: ctx => !!ctx.terrainBackdrop,
       run(ctx) {
         ctx.terrainBackdrop.renderPlanet(ctx.camera, ctx.now / 1000, ctx.sun, ctx.floatingOrigin ? ctx.floatingOrigin.toAuthoritative : undefined)
@@ -256,6 +260,9 @@ export function buildRenderSectionNodes() {
       id: 'visibility-commit',
       reads: ['sceneDepth'],
       writes: ['occlusionCommitted'],
+      // Intentionally-terminal completion marker (see comment below): no downstream node reads
+      // occlusionCommitted, it just stamps that this frame's queries were issued.
+      terminal: true,
       // occlusionCommitted is a pure "queries issued this frame" stamp -- no other node reads it;
       // this node's real effect is entirely external (GPU occlusion-query submission whose results
       // land next frame via modelPool/terrainBackdrop/sceneOcclusion's own internal state, not a

package/package.json

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

package/src/apps/AppLoader.js

@@ -250,7 +250,7 @@ export class AppLoader {
       revokes.push(url)
       urlMap[spec] = url
     }
-    return source.replace(/((?:from|import)\s*)(['"])(\.[^'"]+)\2/g, (m, pre, q, spec) =>
+    return source.replace(/((?:from|import)\s*)(['"])(\.[^'"]+|\/[^'"]+)\2/g, (m, pre, q, spec) =>
       urlMap[spec] ? `${pre}${q}${urlMap[spec]}${q}` : m
     )
   }