@tamagui/native-registry 3.0.14 → 3.0.16

package/cpp/HybridTamaguiRegistry.cpp

@@ -536,9 +536,7 @@ class MountThread {
 
 }  // namespace
 
-// merge base + the named state's props for one view into `out`. a runtime
-// view also syncs its family's native props so React re-commits of unchanged
-// children merge our values instead of resurrecting a stale absorbed node.
+// merge base + the named state's props for one view into `out`.
 // returns false on a state miss.
 bool HybridTamaguiRegistry::buildViewUpdate(LinkedView& view,
                                             const std::string& stateName,
@@ -572,35 +570,10 @@ bool HybridTamaguiRegistry::buildViewUpdate(LinkedView& view,
       return false;
     }
   }
-  const auto& props = update.props();
-
-  const auto& family = *view.family;
-  // the retained node keeps its family alive; unmounted views resolve to
-  // no ancestors during the transaction and are skipped safely
-  if (view.table) {
-    // React renders a table view with its scope's current state and the
-    // engine's clones take over React's handles, so nothing goes stale. a
-    // sticky copy would make RN re-parse the view's props on every clone
-    out[&family] = std::move(update);
-    return true;
-  }
-  std::lock_guard<std::mutex> lock(family.nativePropsMutex);
-  if (family.nativeProps_DEPRECATED) {
-    family.nativeProps_DEPRECATED->update(props);
-  } else {
-    family.nativeProps_DEPRECATED = std::make_unique<folly::dynamic>(props);
-  }
-
-  // a null value means reset-to-default (a style key dropped by a media
-  // change): it must COMMIT as null so RawProps clears the prop, but must not
-  // stick in nativeProps_DEPRECATED or every future React render would keep
-  // re-clearing a key the render may now legitimately set
-  auto& sticky = *family.nativeProps_DEPRECATED;
-  for (const auto& item : props.items()) {
-    if (item.second.isNull()) sticky.erase(item.first.asString());
-  }
-
-  out[&family] = std::move(update);
+  // commitUpdates transfers the updated nodes to React's handles for both
+  // runtime and table links. sticky native props would make RN re-parse
+  // unchanged props on every state/children-only clone, including scrolls.
+  out[view.family.get()] = std::move(update);
   return true;
 }
 

package/cpp/HybridTamaguiRegistry.hpp

@@ -144,8 +144,8 @@ class HybridTamaguiRegistry : public HybridTamaguiRegistrySpec {
   // resolve the active state name for a scope: scope entry, else root
   const std::string* activeStateName(const std::string& scopeId) const;
 
-  // merge base + named state props for one view into `out` (with native-props
-  // sync); returns false on a state miss
+  // merge base + named state props for one view into `out`;
+  // returns false on a state miss
   bool buildViewUpdate(LinkedView& view, const std::string& stateName,
                        LeafUpdates& out);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tamagui/native-registry",
-  "version": "3.0.14",
+  "version": "3.0.16",
   "description": "Generic native style registry: zero-re-render style updates via Fabric ShadowTree commits",
   "type": "module",
   "sideEffects": false,
@@ -46,7 +46,7 @@
     }
   },
   "devDependencies": {
-    "@tamagui/build": "3.0.14",
+    "@tamagui/build": "3.0.16",
     "react": "19.2.3",
     "react-native": "0.87.1",
     "react-native-nitro-modules": "0.37.1",
@@ -77,5 +77,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "releaseSourceCommit": "61a8bd09550319d14c141581a8e4bb286a9795a6"
+  "releaseSourceCommit": "615595788ab0077217b078ff402c4620f4f17db7"
 }

package/src/types.ts

@@ -53,6 +53,6 @@ export interface ViewStateSnapshot {
   activeState: string
   base: Record<string, unknown> | null
   states: Record<string, Record<string, unknown>> | null
-  /** the family's sticky nativeProps_DEPRECATED merge, null when unset */
+  /** the family's nativeProps_DEPRECATED, which only RN's setNativeProps writes; null when unset */
   nativeProps: Record<string, unknown> | null
 }

package/types/types.d.ts

@@ -48,7 +48,7 @@ export interface ViewStateSnapshot {
 	activeState: string;
 	base: Record<string, unknown> | null;
 	states: Record<string, Record<string, unknown>> | null;
-	/** the family's sticky nativeProps_DEPRECATED merge, null when unset */
+	/** the family's nativeProps_DEPRECATED, which only RN's setNativeProps writes; null when unset */
 	nativeProps: Record<string, unknown> | null;
 }
 

package/types/types.d.ts.map

@@ -6,6 +6,6 @@
   ],
   "version": 3,
   "sourcesContent": [
-    "/**\n * Slot props for one linked view.\n *\n * v1 implements two slot kinds: `base` (state-independent props, applied\n * under every state) and `state` (props keyed by state name, e.g. theme\n * name; the scope's active name selects which entry applies). Future kinds\n * (media, press, group, container) extend this shape; the engine merge\n * order is fixed: base, then state, then future kinds.\n *\n * Keys must be exhaustive for every state name the scope can take. The\n * engine does exact lookups only: no fallback chains, no name parsing. A\n * miss increments the engine's miss counter and is a bug in the emitter.\n */\nexport interface ViewSlots {\n  base?: Record<string, unknown>\n  state?: Record<string, Record<string, unknown>>\n}\n\nexport interface RegistryStats {\n  viewCount: number\n  commitCount: number\n  missCount: number\n}\n\n/** Returned by link(); call to unlink. Safe to call more than once. */\nexport type Unlink = () => void\n\n/**\n * Handle for a linked view. `id` addresses the view in batched per-view\n * operations (applyViewStates); `unlink` removes it. null when the ref had\n * no shadow node (never mounted) or the engine is unavailable (web).\n */\nexport interface LinkHandle {\n  id: number\n  unlink: Unlink\n}\n\n/**\n * One entry of a batched per-view state update. When `props` is present it\n * is merged into the view's state table under `state` first (lazy warm-up\n * for runtime integrations); the view's active state then becomes `state`\n * and all entries commit natively in one transaction.\n */\nexport interface ViewStateUpdate {\n  id: number\n  state: string\n  props?: Record<string, unknown>\n}\n\n/** Debug/test snapshot of a linked view's engine tables (getViewState). */\nexport interface ViewStateSnapshot {\n  scopeId: string\n  activeState: string\n  base: Record<string, unknown> | null\n  states: Record<string, Record<string, unknown>> | null\n  /** the family's sticky nativeProps_DEPRECATED merge, null when unset */\n  nativeProps: Record<string, unknown> | null\n}\n"
+    "/**\n * Slot props for one linked view.\n *\n * v1 implements two slot kinds: `base` (state-independent props, applied\n * under every state) and `state` (props keyed by state name, e.g. theme\n * name; the scope's active name selects which entry applies). Future kinds\n * (media, press, group, container) extend this shape; the engine merge\n * order is fixed: base, then state, then future kinds.\n *\n * Keys must be exhaustive for every state name the scope can take. The\n * engine does exact lookups only: no fallback chains, no name parsing. A\n * miss increments the engine's miss counter and is a bug in the emitter.\n */\nexport interface ViewSlots {\n  base?: Record<string, unknown>\n  state?: Record<string, Record<string, unknown>>\n}\n\nexport interface RegistryStats {\n  viewCount: number\n  commitCount: number\n  missCount: number\n}\n\n/** Returned by link(); call to unlink. Safe to call more than once. */\nexport type Unlink = () => void\n\n/**\n * Handle for a linked view. `id` addresses the view in batched per-view\n * operations (applyViewStates); `unlink` removes it. null when the ref had\n * no shadow node (never mounted) or the engine is unavailable (web).\n */\nexport interface LinkHandle {\n  id: number\n  unlink: Unlink\n}\n\n/**\n * One entry of a batched per-view state update. When `props` is present it\n * is merged into the view's state table under `state` first (lazy warm-up\n * for runtime integrations); the view's active state then becomes `state`\n * and all entries commit natively in one transaction.\n */\nexport interface ViewStateUpdate {\n  id: number\n  state: string\n  props?: Record<string, unknown>\n}\n\n/** Debug/test snapshot of a linked view's engine tables (getViewState). */\nexport interface ViewStateSnapshot {\n  scopeId: string\n  activeState: string\n  base: Record<string, unknown> | null\n  states: Record<string, Record<string, unknown>> | null\n  /** the family's nativeProps_DEPRECATED, which only RN's setNativeProps writes; null when unset */\n  nativeProps: Record<string, unknown> | null\n}\n"
   ]
 }