@gg-web-engine/rapier3d 0.0.64 → 0.0.66

package/dist/components/rapier-3d-character-controller.component.d.ts

@@ -1,6 +1,7 @@
 import { CharacterController3dOptions, CollisionGroup, DebugBody3DSettings, Entity3d, ICharacterController3dComponent, Point3, Point4 } from '@gg-web-engine/core';
 import { Collider, InteractionGroups, KinematicCharacterController, RigidBody, RigidBodyDesc } from '@dimforge/rapier3d-compat';
 import { Rapier3dWorldComponent } from './rapier-3d-world.component';
+import { Rapier3dRigidBodyComponent } from './rapier-3d-rigid-body.component';
 import { Rapier3dGgWorld, Rapier3dPhysicsTypeDocRepo } from '../types';
 /**
  * A capsule-shaped kinematic character controller backed by Rapier's own `KinematicCharacterController`
@@ -43,6 +44,12 @@ export declare class Rapier3dCharacterControllerComponent implements ICharacterC
     get isGrounded(): boolean;
     private _groundNormal;
     get groundNormal(): Point3 | null;
+    /** See `ICharacterController3dComponent.ignoredBodies`'s doc. Consulted fresh every `move()` call
+     * via `computeColliderMovement`'s own `filterPredicate` - unlike `AmmoCharacterControllerComponent`
+     * (which has to fake this by temporarily pulling ignored bodies out of the collision world),
+     * Rapier's character controller supports excluding specific colliders from a single query
+     * natively, so no such trick is needed here. */
+    readonly ignoredBodies: Set<Rapier3dRigidBodyComponent>;
     private static warnedMissingDtForPush;
     protected _nativeBody: RigidBody | null;
     protected _nativeCollider: Collider | null;
@@ -71,6 +78,14 @@ export declare class Rapier3dCharacterControllerComponent implements ICharacterC
      * needed here.
      */
     private syncColliderTransform;
+    /**
+     * Builds `computeColliderMovement`'s `filterPredicate` from `ignoredBodies` - `undefined` when
+     * empty (the common case) rather than an always-true closure, so an empty `ignoredBodies` set costs
+     * nothing extra per query. Compares by `RigidBody.handle` (a plain numeric id), not object
+     * identity - `Collider.parent()` isn't guaranteed to return the same wrapper instance across calls
+     * for the pinned `@dimforge/rapier3d-compat` build, only the same underlying native body.
+     */
+    private ignoredBodiesFilterPredicate;
     move(desiredTranslation: Point3, dt?: number): void;
     /**
      * Shoves any dynamic body this tick's sweep bumped into - see `addToWorld`'s doc for why this is

package/dist/components/rapier-3d-character-controller.component.js

@@ -112,6 +112,12 @@ export class Rapier3dCharacterControllerComponent {
         this.name = '';
         this._isGrounded = false;
         this._groundNormal = null;
+        /** See `ICharacterController3dComponent.ignoredBodies`'s doc. Consulted fresh every `move()` call
+         * via `computeColliderMovement`'s own `filterPredicate` - unlike `AmmoCharacterControllerComponent`
+         * (which has to fake this by temporarily pulling ignored bodies out of the collision world),
+         * Rapier's character controller supports excluding specific colliders from a single query
+         * natively, so no such trick is needed here. */
+        this.ignoredBodies = new Set();
         this._nativeBody = null;
         this._nativeCollider = null;
         this._nativeController = null;
@@ -135,6 +141,31 @@ export class Rapier3dCharacterControllerComponent {
     syncColliderTransform() {
         this.world.nativeWorld.propagateModifiedBodyPositionsToColliders();
     }
+    /**
+     * Builds `computeColliderMovement`'s `filterPredicate` from `ignoredBodies` - `undefined` when
+     * empty (the common case) rather than an always-true closure, so an empty `ignoredBodies` set costs
+     * nothing extra per query. Compares by `RigidBody.handle` (a plain numeric id), not object
+     * identity - `Collider.parent()` isn't guaranteed to return the same wrapper instance across calls
+     * for the pinned `@dimforge/rapier3d-compat` build, only the same underlying native body.
+     */
+    ignoredBodiesFilterPredicate() {
+        if (this.ignoredBodies.size === 0) {
+            return undefined;
+        }
+        const ignoredHandles = new Set();
+        for (const body of this.ignoredBodies) {
+            if (body.nativeBody) {
+                ignoredHandles.add(body.nativeBody.handle);
+            }
+        }
+        if (ignoredHandles.size === 0) {
+            return undefined;
+        }
+        return (collider) => {
+            const parent = collider.parent();
+            return !parent || !ignoredHandles.has(parent.handle);
+        };
+    }
     move(desiredTranslation, dt) {
         if (!this._nativeBody || !this._nativeCollider || !this._nativeController) {
             // not yet added to the world - nothing to sweep against (matches
@@ -180,7 +211,7 @@ export class Rapier3dCharacterControllerComponent {
             }
         }
         const desired = new Vector3(desiredTranslation.x, desiredTranslation.y, desiredTranslation.z);
-        this._nativeController.computeColliderMovement(this._nativeCollider, desired);
+        this._nativeController.computeColliderMovement(this._nativeCollider, desired, undefined, undefined, this.ignoredBodiesFilterPredicate());
         const computed = this._nativeController.computedMovement();
         const current = this._nativeBody.translation();
         const next = new Vector3(current.x + computed.x, current.y + computed.y, current.z + computed.z);

package/dist/components/rapier-3d-rigid-body.component.js

@@ -6,7 +6,7 @@ export class Rapier3dRigidBodyComponent {
     }
     set position(value) {
         if (this.nativeBody) {
-            this.nativeBody.setTranslation(new Vector3(value.x, value.y, value.z), false);
+            this.nativeBody.setTranslation(new Vector3(value.x, value.y, value.z), true);
         }
         else {
             this._bodyDescr.setTranslation(value.x, value.y, value.z);
@@ -17,7 +17,7 @@ export class Rapier3dRigidBodyComponent {
     }
     set rotation(value) {
         if (this.nativeBody) {
-            this.nativeBody.setRotation(new Quaternion(value.x, value.y, value.z, value.w), false);
+            this.nativeBody.setRotation(new Quaternion(value.x, value.y, value.z, value.w), true);
         }
         else {
             this._bodyDescr.setRotation(new Quaternion(value.x, value.y, value.z, value.w));
@@ -29,7 +29,8 @@ export class Rapier3dRigidBodyComponent {
     }
     set linearVelocity(value) {
         if (this.nativeBody) {
-            this.nativeBody.setLinvel(new Vector3(value.x, value.y, value.z), false);
+            // see `position`'s setter above for why `true` (wake up) is required
+            this.nativeBody.setLinvel(new Vector3(value.x, value.y, value.z), true);
         }
     }
     get angularVelocity() {
@@ -38,7 +39,8 @@ export class Rapier3dRigidBodyComponent {
     }
     set angularVelocity(value) {
         if (this.nativeBody) {
-            this.nativeBody.setAngvel(new Vector3(value.x, value.y, value.z), false);
+            // see `position`'s setter above for why `true` (wake up) is required
+            this.nativeBody.setAngvel(new Vector3(value.x, value.y, value.z), true);
         }
     }
     get nativeBody() {

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@gg-web-engine/rapier3d",
-  "version": "0.0.64",
+  "version": "0.0.66",
   "description": "An attempt to create open source game engine for browser",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "scripts": {
     "test": "jest",
-    "prettier-format": "prettier --config ../core/.prettierrc 'src/**/*.ts' --write",
+    "prettier-format": "prettier --config ../../.prettierrc 'src/**/*.ts' --write",
     "prepublish": "rm -rf ./dist/ && tsc",
     "build": "tsc"
   },
@@ -28,7 +28,7 @@
   "homepage": "https://github.com/AndyGura/gg-web-engine#readme",
   "devDependencies": {
     "@dimforge/rapier3d-compat": "0.20.0",
-    "@gg-web-engine/core": "0.0.64",
+    "@gg-web-engine/core": "0.0.66",
     "@types/jest": "^30.0.0",
     "@types/node": "^26.4.0",
     "jest": "^30.5.0",
@@ -40,7 +40,7 @@
   },
   "peerDependencies": {
     "@dimforge/rapier3d-compat": "0.20.0",
-    "@gg-web-engine/core": "0.0.64",
+    "@gg-web-engine/core": "0.0.66",
     "rxjs": "7.8.2"
   },
   "jest": {

package/test/components/rapier-3d-character-controller.component.spec.ts

@@ -144,6 +144,46 @@ describe('Rapier3dCharacterControllerComponent', () => {
     expect(character.isGrounded).toBe(true);
   });
 
+  it(
+    'walks straight through a body added to ignoredBodies instead of sliding to a stop against it ' +
+      "(regression: collision groups alone can't express excluding just one specific body while both " +
+      'it and the character still need to collide with the rest of the world - see ' +
+      '`ICharacterController3dComponent.ignoredBodies`\'s doc)',
+    () => {
+      addFloor(0, 20, 0);
+      const wall = addWall(2);
+      const character = factory.createCharacterController(CHAR_OPTIONS, {
+        position: { x: 0, y: 0, z: HALF_HEIGHT + 0.5 },
+      });
+      character.addToWorld({ physicsWorld: world } as any);
+      character.ignoredBodies.add(wall);
+      settleWorld();
+      character.move({ x: 0, y: 0, z: -1 });
+
+      character.move({ x: 5, y: 0, z: 0 });
+
+      // unlike the baseline test above, nothing stopped it short of the wall's x=2 near face
+      expect(character.position.x).toBeCloseTo(5);
+    },
+  );
+
+  it('removing a body from ignoredBodies makes it block the character again', () => {
+    addFloor(0, 20, 0);
+    const wall = addWall(2);
+    const character = factory.createCharacterController(CHAR_OPTIONS, {
+      position: { x: 0, y: 0, z: HALF_HEIGHT + 0.5 },
+    });
+    character.addToWorld({ physicsWorld: world } as any);
+    character.ignoredBodies.add(wall);
+    character.ignoredBodies.delete(wall);
+    settleWorld();
+    character.move({ x: 0, y: 0, z: -1 });
+
+    character.move({ x: 5, y: 0, z: 0 });
+
+    expect(character.position.x).toBeLessThan(1.6);
+  });
+
   it('should step up a ledge shorter than maxStepHeight without getting stuck', () => {
     const maxStepHeight = 0.3;
     const stepZ = 0.2; // below maxStepHeight

package/test/components/rapier-3d-world.component.spec.ts

@@ -87,6 +87,34 @@ describe('Rapier3dWorldComponent', () => {
     });
   });
 
+  describe('Sleeping bodies', () => {
+    it(
+      'wakes a sleeping body when position/rotation/velocity is set programmatically ' +
+        '(regression: Rapier skips a sleeping body\'s island entirely during step(), so a ' +
+        'transform/velocity write passing wakeUp=false was silently never simulated)',
+      () => {
+        const ball = world.factory.createRigidBody(
+          { shape: { shape: 'SPHERE', radius: 1 }, body: { dynamic: true, mass: 5 } },
+          { position: { x: 0, y: 0, z: 0 } },
+        );
+        ball.addToWorld({ physicsWorld: world } as any);
+
+        // let it fall asleep: zero gravity (see beforeEach) and zero velocity means nothing ever
+        // disturbs it, so Rapier's own sleep threshold/timer runs out on its own.
+        for (let i = 0; i < 200; i++) {
+          world.simulate(60);
+        }
+        expect((ball as any).nativeBody.isSleeping()).toBe(true);
+
+        ball.linearVelocity = { x: 1, y: 0, z: 0 };
+        for (let i = 0; i < 60; i++) {
+          world.simulate(60);
+        }
+        expect(ball.position.x).toBeGreaterThan(0.5);
+      },
+    );
+  });
+
   describe('Raycast', () => {
 
     it('should return no hit when ray does not intersect any object', () => {