address copilot review

Author: JakubKorytko

src/libs/Navigation/AppNavigator/routerExtensions/addRootHistoryRouterExtensionUtils.ts

@@ -121,7 +121,7 @@ function getRevealDismissState(
         // so it does not inflate the computed offset.
         const lengthDelta = (state.history?.length ?? 0) - rehydratedHistory.length;
         if (lengthDelta > 0) {
-            Log.info(`[addRootHistoryRouterExtension] reveal committed; freezing history with offset ${lengthDelta}`);
+            Log.hmmm(`[addRootHistoryRouterExtension] reveal committed; freezing history with offset ${lengthDelta}`);
             return {pendingReveal: null, state: {...rehydrated, history: buildPaddedHistory(rehydratedHistory, lengthDelta)}};
         }
         Log.hmmm('[addRootHistoryRouterExtension] reveal committed with non-positive lengthDelta; no freeze', {lengthDelta});

tests/unit/Navigation/routerExtensions/addRootHistoryRouterExtension.test.ts

@@ -695,12 +695,11 @@ describe('addRootHistoryRouterExtension', () => {
             expect(rehydrated.history?.length).toBe(1);
         });
 
-        it('preserves leading reveal-padding sentinels when a partial state carries them through getRehydratedState', () => {
-            // Symmetric to the previous test: if a resetRoot DOES install state whose history
-            // includes leading reveal-padding sentinels (e.g. a captured snapshot), the
-            // sentinels currently survive and are honoured by the default branch of
-            // getStateForAction. This validates the documented trade-off in the extension
-            // ("offset bleeds back if the installed state carries it").
+        it('drops leading reveal-padding sentinels when a partial state carries them through getRehydratedState', () => {
+            // Symmetric to the previous test: even if a resetRoot installs state whose history
+            // includes leading reveal-padding sentinels (e.g. a captured snapshot),
+            // enhanceStateWithHistory rebuilds history from routes and drops that offset at
+            // the rehydration boundary.
             const factory = createMockRouterFactory();
             const enhancedRouter = addRootHistoryRouterExtension(factory)({} as PlatformStackRouterOptions);