From 427664dfa7cfd7b49602e9db82326f72896cd2fd Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Mon, 20 Apr 2026 10:45:02 +0700 Subject: [PATCH] fix(app): drop getId from workspace stack route Expo Router maps getId to React Navigation getId, which reorders an already-mounted workspace screen on Android native-stack/Fabric. Keep workspace identity/retention outside the route-level API and leave a comment warning against re-adding it. --- packages/app/src/app/_layout.tsx | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/packages/app/src/app/_layout.tsx b/packages/app/src/app/_layout.tsx index c0bd3a0e7..3b5279dbf 100644 --- a/packages/app/src/app/_layout.tsx +++ b/packages/app/src/app/_layout.tsx @@ -815,14 +815,14 @@ function RootStack() { - { - const serverId = getRouteParamValue(params?.serverId); - const workspaceId = getRouteParamValue(params?.workspaceId); - return serverId && workspaceId ? `${serverId}:${workspaceId}` : undefined; - }} - /> + {/* + Do not add getId or dangerouslySingular back to the workspace route. + Expo Router maps dangerouslySingular to React Navigation getId, and + getId repeatedly breaks Android native-stack/Fabric by reordering an + already-mounted workspace screen. Keep workspace identity/retention + outside this route-level native-stack API. + */} +