@sceneview-sdk/react-native 4.17.0 → 4.19.0

package/android/src/main/java/io/github/sceneview/reactnative/SceneViewManager.kt

@@ -3,6 +3,7 @@ package io.github.sceneview.reactnative
 import android.widget.FrameLayout
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.runtime.DisposableEffect
+import androidx.compose.runtime.key
 import androidx.compose.runtime.mutableStateListOf
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
@@ -109,10 +110,25 @@ class SceneViewManager : SimpleViewManager<FrameLayout>() {
                     position = Position(y = 0f, z = 3.0f)
                 }
 
-                val environment = state.environmentPath.value?.let { path ->
+                // Single, stable environment call site (issue #2365): compute exactly one
+                // non-null Environment rather than branching between a keyed HDR call and a
+                // separate `environment ?: rememberEnvironment(...)` fallback at the SceneView
+                // argument. The `path` is the remember key, so:
+                //   - null path           → the default neutral environment;
+                //   - path A → path B      → the Environment is rebuilt (skybox/IBL actually
+                //                            swaps; the stale-factory bug of #2361).
+                // Uses Compose key {} rather than rememberEnvironment's own key= param because
+                // this bridge compiles against the published Maven artifact (sceneview:4.7.0)
+                // that predates key=. key {} replaces the group on a new path, disposing the old
+                // Environment via its DisposableEffect.
+                // TODO(#2361): migrate to rememberEnvironment(..., key = path) once this bridge
+                // consumes a SceneView release that includes the public key= param.
+                val environmentPath = state.environmentPath.value
+                val environment = key(environmentPath) {
                     rememberEnvironment(environmentLoader) {
-                        environmentLoader.createHDREnvironment(path)
-                            ?: io.github.sceneview.createEnvironment(environmentLoader)
+                        environmentPath?.let { path ->
+                            environmentLoader.createHDREnvironment(path)
+                        } ?: io.github.sceneview.createEnvironment(environmentLoader)
                     }
                 }
 
@@ -123,7 +139,7 @@ class SceneViewManager : SimpleViewManager<FrameLayout>() {
                     modelLoader = modelLoader,
                     materialLoader = materialLoader,
                     cameraNode = cameraNode,
-                    environment = environment ?: rememberEnvironment(environmentLoader),
+                    environment = environment,
                     onGestureListener = gestureListener,
                 ) {
                     state.modelPaths.forEach { model ->

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sceneview-sdk/react-native",
-  "version": "4.17.0",
+  "version": "4.19.0",
   "description": "React Native bindings for SceneView — 3D and AR scenes powered by Filament (Android) and RealityKit (iOS)",
   "main": "lib/commonjs/index",
   "module": "lib/module/index",
@@ -70,7 +70,10 @@
     ]
   },
   "overrides": {
-    "fast-xml-parser": "^5.7.0"
+    "fast-xml-parser": "^5.7.0",
+    "@babel/core": "^7.29.6",
+    "shell-quote": "^1.8.4",
+    "js-yaml": "^3.15.0"
   },
   "react-native-builder-bob": {
     "source": "src",