@symbiote-native/navigation 5.0.2 → 5.0.3

package/README.md

@@ -14,13 +14,28 @@ logic/view/lifecycle split every other SymbioteNative component follows.
 
 ## Install
 
+**New app:**
+
+```bash
+npx @symbiote-native/cli new my-app --navigation
+```
+
+**Existing SymbioteNative app:**
+
+```bash
+npx @symbiote-native/cli add --navigation
+```
+
+Either way: installs and wires this for you — see [`@symbiote-native/cli`](../cli).
+
+<details>
+<summary>Manual install</summary>
+
 ```bash
 npm install @symbiote-native/navigation
 ```
 
-Scaffolding or extending a SymbioteNative app? `npx @symbiote-native/cli new --navigation` (or
-`add --navigation` in an existing app) installs and wires this for you — see
-[`@symbiote-native/cli`](../cli).
+</details>
 
 Only this package — never `react-native-screens` directly. `@symbiote-native/navigation` depends
 on it and ships as the sole autolinked native proxy (`react-native.config.cjs` +

package/build/react/drawer/index.js

@@ -149,7 +149,12 @@ const DrawerImpl = forwardRef((props, forwardedRef) => {
         },
     })).current;
     const animated = isDrawerAnimated(options);
-    const geometry = useMemo(() => resolveDrawerGeometry(options), [options.drawerType, options.drawerPosition, options.drawerWidth]);
+    // Narrow on purpose: resolveDrawerGeometry only ever reads these 3 fields off options (see
+    // drawer-options/index.ts's resolveDrawerWidth/Type/Position) — depending on the whole
+    // `options` object would recompute on every unrelated options field, defeating the memo.
+    const geometry = useMemo(() => resolveDrawerGeometry(options), 
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [options.drawerType, options.drawerPosition, options.drawerWidth]);
     const panelSlot = animated
         ? resolveDrawerSlotInterpolation(geometry, 'panel')
         : undefined;
@@ -191,7 +196,11 @@ const DrawerImpl = forwardRef((props, forwardedRef) => {
     // route object, so a jumpTo-with-no-params re-focus of the ALREADY-focused route (a no-op in
     // drawerRouterReducer) and any future params merge don't spuriously re-fire focus/blur.
     const focusedRouteKey = focusedRoute?.key;
-    const routeEmitter = useMemo(() => createNavigationEmitter(), [focusedRouteKey]);
+    // focusedRouteKey is a pure invalidation signal — createNavigationEmitter() takes no
+    // arguments, the key only forces a fresh emitter per focus change (see comment above).
+    const routeEmitter = useMemo(() => createNavigationEmitter(), 
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [focusedRouteKey]);
     // Drawer paints its own panel in pure JS - there is no native onAppear/onDisappear to hook
     // (unlike Stack's RNSScreen), so focus/blur is synthesized here: mount = focus, cleanup = blur,
     // exactly what an effect keyed on focusedRouteKey already encodes - no diffFocusedRoute

package/build/react/tabs/index.js

@@ -106,7 +106,11 @@ const TabImpl = forwardRef((props, forwardedRef) => {
     // object so a setParams-only change (new route object, same key) doesn't spuriously re-fire
     // focus/blur.
     const focusedRouteKey = focusedRoute?.key;
-    const routeEmitter = useMemo(() => createNavigationEmitter(), [focusedRouteKey]);
+    // focusedRouteKey is a pure invalidation signal — createNavigationEmitter() takes no
+    // arguments, the key only forces a fresh emitter per focus change (see comment above).
+    const routeEmitter = useMemo(() => createNavigationEmitter(), 
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [focusedRouteKey]);
     // Tab paints its own bar in pure JS - there is no native onAppear/onDisappear to hook (unlike
     // Stack's RNSScreen), so focus/blur is synthesized here: mount = focus, cleanup = blur, exactly
     // what an effect keyed on focusedRouteKey already encodes - no diffFocusedRoute indirection

package/build/svelte/drawer/index.svelte

@@ -279,6 +279,9 @@
   // Only the focused route's screen is ever mounted (like Tab, unlike Stack which keeps every
   // pushed route alive). One emitter per route.key, created lazily and cached for the navigator's
   // whole lifetime.
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);

package/build/svelte/stack/index.svelte

@@ -132,6 +132,9 @@
 
   // One emitter per route.key, created lazily the first time a route is rendered and pruned once
   // it is popped off the stack (the broadcast effect below).
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);

package/build/svelte/tabs/index.svelte

@@ -123,6 +123,9 @@
   // mirrors Stack's own `emitters` map. This decouples emitter IDENTITY (stable, looked up by
   // key, read below when a route's NavigationScope is built) from emit TIMING (must wait until
   // the focused screen has actually mounted and subscribed).
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@symbiote-native/navigation",
-  "version": "5.0.2",
+  "version": "5.0.3",
   "description": "Native stack navigator for SymbioteNative, built on react-native-screens' native view primitives (RNSScreen/RNSScreenStack) — one framework-agnostic core, reachable from the React, Vue, Svelte, Solid, and Angular adapters.",
   "keywords": [
     "react-native",
@@ -160,7 +160,7 @@
   },
   "dependencies": {
     "react-native-screens": "4.26.0",
-    "@symbiote-native/components": "3.1.0"
+    "@symbiote-native/components": "3.1.1"
   },
   "peerDependencies": {
     "@angular/common": ">=20",
@@ -171,12 +171,12 @@
     "solid-js": ">=1.9.0",
     "svelte": ">=5.56.0",
     "vue": ">=3.5.0",
-    "@symbiote-native/react": "^3.0.2",
+    "@symbiote-native/angular": "^3.1.1",
     "@symbiote-native/engine": "^1.3.0",
-    "@symbiote-native/svelte": "^3.0.2",
-    "@symbiote-native/angular": "^3.1.0",
-    "@symbiote-native/vue": "^3.0.2",
-    "@symbiote-native/solid": "^3.0.2"
+    "@symbiote-native/react": "^3.0.3",
+    "@symbiote-native/solid": "^3.0.3",
+    "@symbiote-native/svelte": "^3.0.3",
+    "@symbiote-native/vue": "^3.0.3"
   },
   "peerDependenciesMeta": {
     "@angular/common": {
@@ -228,13 +228,13 @@
     "solid-js": "^1.9.14",
     "svelte": "^5.56.0",
     "typescript": "~6.0.0",
-    "@symbiote-native/angular": "3.1.0",
+    "@symbiote-native/angular": "3.1.1",
     "@symbiote-native/engine": "1.3.0",
-    "@symbiote-native/react": "3.0.2",
-    "@symbiote-native/solid": "3.0.2",
-    "@symbiote-native/svelte": "3.0.2",
-    "@symbiote-native/test-utils": "0.4.2",
-    "@symbiote-native/vue": "3.0.2"
+    "@symbiote-native/react": "3.0.3",
+    "@symbiote-native/solid": "3.0.3",
+    "@symbiote-native/svelte": "3.0.3",
+    "@symbiote-native/test-utils": "0.4.3",
+    "@symbiote-native/vue": "3.0.3"
   },
   "scripts": {
     "clean": "rm -rf build-ngc",

package/src/react/drawer/index.ts

@@ -320,8 +320,12 @@ const DrawerImpl = forwardRef<IDrawerNavigatorHandle, IDrawerProps>(
     ).current;
 
     const animated = isDrawerAnimated(options);
+    // Narrow on purpose: resolveDrawerGeometry only ever reads these 3 fields off options (see
+    // drawer-options/index.ts's resolveDrawerWidth/Type/Position) — depending on the whole
+    // `options` object would recompute on every unrelated options field, defeating the memo.
     const geometry = useMemo(
       () => resolveDrawerGeometry(options),
+      // eslint-disable-next-line react-hooks/exhaustive-deps
       [options.drawerType, options.drawerPosition, options.drawerWidth],
     );
 
@@ -371,8 +375,11 @@ const DrawerImpl = forwardRef<IDrawerNavigatorHandle, IDrawerProps>(
     // route object, so a jumpTo-with-no-params re-focus of the ALREADY-focused route (a no-op in
     // drawerRouterReducer) and any future params merge don't spuriously re-fire focus/blur.
     const focusedRouteKey = focusedRoute?.key;
+    // focusedRouteKey is a pure invalidation signal — createNavigationEmitter() takes no
+    // arguments, the key only forces a fresh emitter per focus change (see comment above).
     const routeEmitter = useMemo(
       () => createNavigationEmitter(),
+      // eslint-disable-next-line react-hooks/exhaustive-deps
       [focusedRouteKey],
     );
 

package/src/react/tabs/index.ts

@@ -216,8 +216,11 @@ const TabImpl = forwardRef<ITabNavigatorHandle, ITabProps>(
     // object so a setParams-only change (new route object, same key) doesn't spuriously re-fire
     // focus/blur.
     const focusedRouteKey = focusedRoute?.key;
+    // focusedRouteKey is a pure invalidation signal — createNavigationEmitter() takes no
+    // arguments, the key only forces a fresh emitter per focus change (see comment above).
     const routeEmitter = useMemo(
       () => createNavigationEmitter(),
+      // eslint-disable-next-line react-hooks/exhaustive-deps
       [focusedRouteKey],
     );
 

package/src/svelte/drawer/index.svelte

@@ -279,6 +279,9 @@
   // Only the focused route's screen is ever mounted (like Tab, unlike Stack which keeps every
   // pushed route alive). One emitter per route.key, created lazily and cached for the navigator's
   // whole lifetime.
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);

package/src/svelte/stack/index.svelte

@@ -132,6 +132,9 @@
 
   // One emitter per route.key, created lazily the first time a route is rendered and pruned once
   // it is popped off the stack (the broadcast effect below).
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);

package/src/svelte/tabs/index.svelte

@@ -123,6 +123,9 @@
   // mirrors Stack's own `emitters` map. This decouples emitter IDENTITY (stable, looked up by
   // key, read below when a route's NavigationScope is built) from emit TIMING (must wait until
   // the focused screen has actually mounted and subscribed).
+  // Plain Map: a get/set/delete identity cache never read by the reactive graph, not rendered
+  // state — SvelteMap's tracking would be pure overhead here.
+  // eslint-disable-next-line svelte/prefer-svelte-reactivity
   const emitters = new Map<string, INavigationEmitter>();
   function emitterFor(routeKey: string): INavigationEmitter {
     let emitter = emitters.get(routeKey);