@operato/scene-storage 10.0.0-beta.43 → 10.0.0-beta.46

package/src/crane-3d.ts

@@ -42,6 +42,26 @@ const RAIL_COLOR = 0x1a1f24          // rail — dark steel
 const LAMP_OFF = 0x222222
 
 export class Crane3D extends RealObjectGroup {
+  /**
+   * carriageHeight 의 실제 최댓값 — *mast 의 carriage 이동 가능 범위* 기반.
+   * mastH = D - railH * 2 - baseH - topFrameH - topGuideH (mast 의 실제 길이).
+   * carriage 의 *top edge 가 mast top 안에 머물도록* carriageH/2 추가 차감.
+   *
+   * 이전 D * 0.85 magic 대체 — base/top frame 의 실제 차지 비율과 무관한 추정값
+   * 이라 crane.depth = rack.depth 동일 모델에서도 carriage 가 rack 상위 cell
+   * 도달 못 함. 정확한 mastH 사용으로 *동일 size 모델에서 정상 도달* 보장.
+   */
+  static _maxCarriageHeight(D: number, width: number, height: number): number {
+    const S = Math.min(width, height)
+    const railH = S * 0.04
+    const baseH = S * 0.18
+    const topFrameH = S * 0.1
+    const topGuideH = S * 0.1
+    const carriageH = S * 0.12
+    const mastH = Math.max(D - railH * 2 - baseH - topFrameH - topGuideH, S * 0.5)
+    return Math.max(0, mastH - carriageH / 2)
+  }
+
   private _forkGroup?: THREE.Group
   private _carrierBaseY: number = 0
   private _bladeMidZ: number = 0
@@ -79,7 +99,12 @@ export class Crane3D extends RealObjectGroup {
     // Actuators
     const D = numOr(depth, Math.max(width, height) * 4)
     const carriageRaw = numOr((this.component.state as any).carriageHeight, (this.component as any)._canonicalDefault?.('carriageHeight') ?? D * 0.4)
-    const carriageHeight = Math.max(0, Math.min(carriageRaw, D * 0.85))
+    // carriageHeight clamp — 실제 mast 의 carriage 이동 가능 범위 기반. helper 사용
+    // (render + onchange partial update 둘 다 동일 식). 이전엔 D * 0.85 magic 사용 —
+    // base/top frame 의 실제 차지 비율과 무관한 추정. crane.depth = rack.depth 동일
+    // 모델에서도 carriage 가 rack 상위 cell 도달 불가 → 14 levels rack 의 *마지막
+    // shelf 만 한 층 아래에서 포킹* 증상의 root cause 였음.
+    const carriageHeight = Math.max(0, Math.min(carriageRaw, Crane3D._maxCarriageHeight(D, width, height)))
 
     const forkLength = numOr((this.component.state as any).forkLength, height * 0.6)
     const forkExtensionRaw = numOr((this.component.state as any).forkExtension, 0)
@@ -362,9 +387,11 @@ export class Crane3D extends RealObjectGroup {
       let meshUpdated = false
       if (('carriageHeight' in after || 'forkLiftRT' in after) && this._carriageLiftGroup) {
         const state = this.component.state as any
-        const D = numOr(state.depth, Math.max(numOr(state.width, 100), numOr(state.height, 100)) * 4)
+        const W = numOr(state.width, 100)
+        const H = numOr(state.height, 100)
+        const D = numOr(state.depth, Math.max(W, H) * 4)
         const carriageRaw = numOr(state.carriageHeight, D * 0.4)
-        const carriageHeight = Math.max(0, Math.min(carriageRaw, D * 0.85))
+        const carriageHeight = Math.max(0, Math.min(carriageRaw, Crane3D._maxCarriageHeight(D, W, H)))
         const forkLift = numOr(state.forkLiftRT, 0)
         this._carriageLiftGroup.position.y = this._computeLiftGroupY(carriageHeight, forkLift)
         meshUpdated = true
@@ -458,7 +485,10 @@ export class Crane3D extends RealObjectGroup {
     if (this._forkGroup) {
       for (const child of this._forkGroup.children) {
         const ctx = (child as any).userData?.context
-        if (ctx && ctx !== this && ctx instanceof RealObject) {
+        // `ctx instanceof RealObject` 는 *cross-module identity 실패* 가능
+        // (Parcel3D 가 다른 things-scene 인스턴스의 RealObject 를 extend 한 경우
+        // instanceof 가 false 반환). duck-type 으로 완화 — *crane 자신만 제외*.
+        if (ctx && ctx !== this) {
           child.position.z = this._bladeMidZ
         }
       }