fix(query, gizmos): correct projTop/projBottom and viewFrustum Y sign

deps/tree/src/query.js
- projTop/projBottom formulas were swapped — returned geometric bottom/top
  respectively. Fix: projTop = (1−p[13])/p[5] ortho, (1+p[9])·n/p[5] persp;
  projBottom = (−1−p[13])/p[5] ortho, (p[9]−1)·n/p[5] persp.
  Sign contract now explicit in JSDoc: top > 0, bottom < 0, right > 0, left < 0.

src/visibility.js
- bounds(): rename eMatrix → mat4Eye for consistency with the rest of the bridge.
- Sign contract added to module header (covers all of frustumPlanes, viewFrustum,
  projTop/Bottom/Left/Right, mat4Frustum/Ortho, and p5 v2 frustum()/ortho()).

src/gizmos.js
- viewFrustum: negate t/b when drawing into p5 main canvas / createGraphics
  (Y-down). projTop/projBottom are Y-up (OpenGL eye space); sign flip aligns
  the gizmo with screen geometry. l/r/n/f unaffected.
- Remove the stale isOrtho conditional negation that was papering over the
  swapped formulas.

Author: nakednous

README.md

@@ -590,7 +590,7 @@ Both accept the same options object:
 # Utilities
 
 ```js
-p5.Tree.VERSION   // '0.0.32'
+p5.Tree.VERSION   // '0.0.33'
 ```
 
 ## Shader helpers
@@ -665,9 +665,9 @@ Latest:
 
 Tagged:
 
-* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.js)
-* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.min.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.min.js)
-* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.esm.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.32/dist/p5.tree.esm.js)
+* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.js)
+* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.min.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.min.js)
+* [https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.esm.js](https://cdn.jsdelivr.net/npm/p5.tree@0.0.33/dist/p5.tree.esm.js)
 
 ---
 

deps/tree/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nakednous/tree",
-  "version": "0.0.13",
+  "version": "0.0.14",
   "description": "tree — pure numeric core. Zero dependencies.",
   "type": "module",
   "main": "dist/index.js",

deps/tree/src/query.js

@@ -177,8 +177,22 @@ export function projFar(p) {
 
 export function projLeft  (p, ndcZMin) { return p[15]===1 ? -(1+p[12])/p[0]  : projNear(p,ndcZMin)*(p[8]-1)/p[0];  }
 export function projRight (p, ndcZMin) { return p[15]===1 ?  (1-p[12])/p[0]  : projNear(p,ndcZMin)*(1+p[8])/p[0];  }
-export function projTop   (p, ndcZMin) { return p[15]===1 ?  (p[13]-1)/p[5]  : projNear(p,ndcZMin)*(p[9]-1)/p[5];  }
-export function projBottom(p, ndcZMin) { return p[15]===1 ?  (1+p[13])/p[5]  : projNear(p,ndcZMin)*(1+p[9])/p[5];  }
+
+/**
+ * Top extent of the near plane in camera space (y_max).
+ * Positive for a standard y-up camera.
+ * @param {ArrayLike<number>} p       Projection mat4.
+ * @param {number}            ndcZMin WEBGL (−1) or WEBGPU (0).
+ */
+export function projTop   (p, ndcZMin) { return p[15]===1 ?  (1+p[13])/p[5]  : projNear(p,ndcZMin)*(1+p[9])/p[5];  }
+
+/**
+ * Bottom extent of the near plane in camera space (y_min).
+ * Negative for a standard y-up camera.
+ * @param {ArrayLike<number>} p       Projection mat4.
+ * @param {number}            ndcZMin WEBGL (−1) or WEBGPU (0).
+ */
+export function projBottom(p, ndcZMin) { return p[15]===1 ?  (p[13]-1)/p[5]  : projNear(p,ndcZMin)*(p[9]-1)/p[5];  }
 
 /** Vertical field of view in radians (perspective only). */
 export function projFov (p) { return Math.abs(2*Math.atan(1/p[5])); }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "p5.tree",
-  "version": "0.0.32",
+  "version": "0.0.33",
   "description": "Render pipeline for p5.js v2 — pose and camera interpolation, space transforms, frustum visibility, HUD, post-processing pipe, picking, and declarative control panels.",
   "type": "module",
   "main": "dist/p5.tree.esm.js",
@@ -49,7 +49,7 @@
     "p5": "^2.2.3"
   },
   "dependencies": {
-    "@nakednous/tree": "^0.0.13",
+    "@nakednous/tree": "^0.0.14",
     "@nakednous/ui": "^0.0.8"
   },
   "devDependencies": {

src/constants.js

@@ -12,7 +12,7 @@ export function installConstants(p5) {
   const CONST = value => ({ value, writable: false, enumerable: true, configurable: false });
 
   Object.defineProperties(p5.Tree, {
-    VERSION: CONST('0.0.32'),
+    VERSION: CONST('0.0.33'),
     NONE: CONST(0),
 
     // Core constants (spaces, visibility, NDC, basis vectors)

src/gizmos.js

@@ -254,8 +254,11 @@ export function installGizmos(p5, fn) {
     const apex    = !isOrtho && ((bits & p5.Tree.APEX) !== 0);
     const n = -projNear(pRaw, ndcZ), f = -projFar(pRaw);
     const l =  projLeft(pRaw, ndcZ),  r = projRight(pRaw, ndcZ);
-    const t = isOrtho ? -projTop(pRaw, ndcZ)    : projTop(pRaw, ndcZ);
-    const b = isOrtho ? -projBottom(pRaw, ndcZ) : projBottom(pRaw, ndcZ);
+    // projTop/projBottom are Y-up (OpenGL eye space). Negate y to draw in
+    // p5.js main-canvas / createGraphics space, which is Y-down: y > 0 is lower
+    // on screen than y < 0.
+    const t = -projTop(pRaw, ndcZ);
+    const b = -projBottom(pRaw, ndcZ);
     const ratio = isOrtho ? 1 : f/n;
     const _l=ratio*l, _r=ratio*r, _b=ratio*b, _t=ratio*t;
 

src/visibility.js

@@ -15,6 +15,19 @@
  *   m._c1.set([px - hw, py - hh, pz - hd])
  *   m._c2.set([px + hw, py + hh, pz + hd])
  *   m.visibility = p.visibility({ corner1: m._c1, corner2: m._c2 })
+ *
+ * ── Sign contract ─────────────────────────────────────────────────────────
+ *
+ * Frustum extents are near-plane coordinates in camera space (y-up, z into
+ * screen):
+ *
+ *   top    > 0   bottom < 0   (y axis)
+ *   right  > 0   left   < 0   (x axis)
+ *   near, far > 0              (positive distances along −z)
+ *
+ * All of frustumPlanes, viewFrustum, projTop/projBottom, projLeft/projRight,
+ * mat4Frustum, mat4Ortho, and p5 v2's frustum()/ortho() share this contract.
+ * p5 v2 call order: frustum(left, right, bottom, top, near, far).
  */
 
 'use strict';
@@ -234,11 +247,11 @@ export function installVisibility(p5, fn) {
    *
    * @method bounds
    * @for p5
-   * @param {{ eMatrix?: Float32Array | ArrayLike | p5.Matrix }} [opts]
+   * @param {{ mat4Eye?: Float32Array | ArrayLike | p5.Matrix }} [opts]
    * @returns {object}
    */
-  p5.Renderer3D.prototype.bounds = function ({ eMatrix } = {}) {
-    const eRaw = _rawMat4(eMatrix) ?? (mat4Invert(_eye, _viewMat4(this)), _eye);
+  p5.Renderer3D.prototype.bounds = function ({ mat4Eye } = {}) {
+    const eRaw = _rawMat4(mat4Eye) ?? (mat4Invert(_eye, _viewMat4(this)), _eye);
     computePlanes(this, eRaw);
     const keys   = [p5.Tree.LEFT, p5.Tree.RIGHT, p5.Tree.NEAR, p5.Tree.FAR, p5.Tree.TOP, p5.Tree.BOTTOM];
     const result = {};