Fix sentry error on dev, local scripts and user card icon

Author: nabramow

app/mobile/app/_layout.tsx

@@ -31,6 +31,7 @@ import { SafeAreaProvider } from "react-native-safe-area-context";
 import DevSettingsButton from "@/components/DevSettingsButton";
 import { hydrateUrlOverrides } from "@/config/urls";
 import { AuthProvider, useAuthContext } from "@/features/auth/AuthContext";
+import { appVariant } from "@/service/buildInfo";
 import { useRegisterPushNotifications } from "@/features/notifications/useRegisterPushNotifications";
 import { reconfigureApiClient } from "@/service/client";
 import { getNotificationPath } from "@/utils/getNotificationPath";
@@ -91,7 +92,9 @@ function RootNavigator({ fontsLoaded }: { fontsLoaded: boolean }) {
   );
 }
 
-export default Sentry.wrap(function RootLayout() {
+const sentryEnabled = appVariant === "production" || appVariant === "staging";
+
+function RootLayout() {
   const colorScheme = useColorScheme();
   const [fontsLoaded] = useFonts({
     Ubuntu_300Light,
@@ -131,7 +134,9 @@ export default Sentry.wrap(function RootLayout() {
       </ThemeProvider>
     </SafeAreaProvider>
   );
-});
+}
+
+export default sentryEnabled ? Sentry.wrap(RootLayout) : RootLayout;
 
 /**
  * Generates a unique ID for a notification response to prevent duplicate handling.

app/scripts/dev-mobile-setup.py

@@ -11,10 +11,12 @@
   python3 scripts/dev-mobile-setup.py --restore     # undo all changes
 
 Files modified:
-  app/proxy/envoy.yaml      - adds IP to CORS allow list
-  app/backend.dev.env       - sets COOKIE_DOMAIN and media server URLs
-  app/web/.env.localdev     - sets API and media URLs
-  app/mobile/.env           - switches from staging to local dev mode
+  app/proxy/envoy.yaml        - adds IP to CORS allow list
+  app/backend.dev.env         - sets COOKIE_DOMAIN and media server URLs
+  app/media.dev.env           - sets media server base URL (for image uploads)
+  app/web/.env.localdev       - sets API and media URLs
+  app/web/.env.development    - sets API and media URLs
+  app/mobile/.env             - switches from staging to local dev mode
 
 Note: do NOT commit the envoy.yaml change.
 """
@@ -29,7 +31,9 @@
 
 ENVOY_YAML = APP / "proxy" / "envoy.yaml"
 BACKEND_ENV = APP / "backend.dev.env"
+MEDIA_ENV = APP / "media.dev.env"
 WEB_ENV = APP / "web" / ".env.localdev"
+WEB_DEV_ENV = APP / "web" / ".env.development"
 MOBILE_ENV = APP / "mobile" / ".env"
 
 MOBILE_ENV_DEFAULT = """\
@@ -80,13 +84,26 @@ def setup(ip: str) -> None:
     BACKEND_ENV.write_text(content)
     print("  backend.dev.env updated COOKIE_DOMAIN and media server URLs")
 
+    # media.dev.env — media server base URL (needed for upload response URLs)
+    content = MEDIA_ENV.read_text()
+    content = re.sub(r"^MEDIA_SERVER_BASE_URL=.*", f"MEDIA_SERVER_BASE_URL=http://{ip}:5001", content, flags=re.MULTILINE)
+    MEDIA_ENV.write_text(content)
+    print("  media.dev.env   updated MEDIA_SERVER_BASE_URL")
+
     # web/.env.localdev — API + media URLs
     content = WEB_ENV.read_text()
     content = re.sub(r'^NEXT_PUBLIC_API_BASE_URL=.*', f'NEXT_PUBLIC_API_BASE_URL="http://{ip}:8888"', content, flags=re.MULTILINE)
     content = re.sub(r'^NEXT_PUBLIC_MEDIA_BASE_URL=.*', f'NEXT_PUBLIC_MEDIA_BASE_URL="http://{ip}:5001"', content, flags=re.MULTILINE)
     WEB_ENV.write_text(content)
     print("  web/.env.localdev updated API and media URLs")
 
+    # web/.env.development — API + media URLs
+    content = WEB_DEV_ENV.read_text()
+    content = re.sub(r'^NEXT_PUBLIC_API_BASE_URL=.*', f'NEXT_PUBLIC_API_BASE_URL="http://{ip}:8888"', content, flags=re.MULTILINE)
+    content = re.sub(r'^NEXT_PUBLIC_MEDIA_BASE_URL=.*', f'NEXT_PUBLIC_MEDIA_BASE_URL="http://{ip}:5001"', content, flags=re.MULTILINE)
+    WEB_DEV_ENV.write_text(content)
+    print("  web/.env.development updated API and media URLs")
+
     # mobile/.env — comment out STAGE block, activate LOCAL block with real IP
     MOBILE_ENV.write_text(
         f"# STAGE:\n"
@@ -130,13 +147,26 @@ def restore() -> None:
     BACKEND_ENV.write_text(content)
     print("  backend.dev.env restored to localhost")
 
+    # media.dev.env — back to localhost
+    content = MEDIA_ENV.read_text()
+    content = re.sub(r"^MEDIA_SERVER_BASE_URL=.*", "MEDIA_SERVER_BASE_URL=http://localhost:5001", content, flags=re.MULTILINE)
+    MEDIA_ENV.write_text(content)
+    print("  media.dev.env   restored to localhost")
+
     # web/.env.localdev — back to localhost
     content = WEB_ENV.read_text()
     content = re.sub(r'^NEXT_PUBLIC_API_BASE_URL=.*', 'NEXT_PUBLIC_API_BASE_URL="http://localhost:8888"', content, flags=re.MULTILINE)
     content = re.sub(r'^NEXT_PUBLIC_MEDIA_BASE_URL=.*', 'NEXT_PUBLIC_MEDIA_BASE_URL="http://localhost:5001"', content, flags=re.MULTILINE)
     WEB_ENV.write_text(content)
     print("  web/.env.localdev restored to localhost")
 
+    # web/.env.development — back to staging
+    content = WEB_DEV_ENV.read_text()
+    content = re.sub(r'^NEXT_PUBLIC_API_BASE_URL=.*', 'NEXT_PUBLIC_API_BASE_URL="https://next.couchershq.org/api"', content, flags=re.MULTILINE)
+    content = re.sub(r'^NEXT_PUBLIC_MEDIA_BASE_URL=.*', 'NEXT_PUBLIC_MEDIA_BASE_URL="https://dev-user-media.couchershq.org"', content, flags=re.MULTILINE)
+    WEB_DEV_ENV.write_text(content)
+    print("  web/.env.development restored to staging")
+
     # mobile/.env — back to staging mode
     MOBILE_ENV.write_text(MOBILE_ENV_DEFAULT)
     print("  mobile/.env     restored to staging mode")

app/web/components/UserSummary.tsx

@@ -13,6 +13,7 @@ import ProfileLink from "components/ProfileLink/ProfileLink";
 import { LiteUser } from "proto/api_pb";
 import { BlockedUser } from "proto/blocking_pb";
 import React, { useState } from "react";
+import useIsScreenSizeOrSmaller from "utils/useIsScreenSizeOrSmaller";
 
 import StrongVerificationBadge from "./StrongVerificationBadge";
 
@@ -83,6 +84,7 @@ export default function UserSummary({
     },
   );
 
+  const isMobile = useIsScreenSizeOrSmaller("mobile");
   const [menuAnchorEl, setMenuAnchorEl] = useState<HTMLButtonElement | null>(
     null,
   );
@@ -158,11 +160,11 @@ export default function UserSummary({
             <ProfileLink
               userId={"userId" in user ? user.userId : undefined}
               username={user.username}
-              openInNewTab
+              openInNewTab={!isMobile}
               style={{ display: "flex", alignItems: "center" }}
             >
               {title}
-              <StyledOpenInNewIcon />
+              {!isMobile && <StyledOpenInNewIcon />}
             </ProfileLink>
           ) : (
             title

docs/run-local-app-on-mobile.md

@@ -40,6 +40,8 @@ python3 app/scripts/dev-mobile-setup.py --restore
 | File | Change |
 |------|--------|
 | `app/proxy/envoy.yaml` | Adds your IP to the CORS allow list |
-| `app/backend.dev.env` | Sets `COOKIE_DOMAIN` and media server URLs to your IP |
+| `app/backend.dev.env` | Sets `COOKIE_DOMAIN`, `MEDIA_SERVER_BASE_URL`, and `MEDIA_SERVER_UPLOAD_BASE_URL` |
+| `app/media.dev.env` | Sets `MEDIA_SERVER_BASE_URL` (needed for image upload response URLs) |
 | `app/web/.env.localdev` | Points API and media URLs at your IP |
+| `app/web/.env.development` | Points API and media URLs at your IP |
 | `app/mobile/.env` | Switches from staging to local dev mode |