Description
PR #3120 (issue #3115) fixed SessionsSheet and HomeScreen's session-open path rendering raw Error.message strings (function names + HTTP status codes) as user-facing UI. The review found the same defect class in four more places, out of scope for that PR:
apps/mobile/src/screens/systems/SystemsScreen.tsx (~line 210)apps/mobile/src/screens/systems/useSystemsData.ts (~line 105)apps/mobile/src/screens/devices/DeviceDetailScreen.tsx (~line 174)apps/mobile/src/screens/chat/HomeScreen.tsx (~line 194, create-session path)
(Paths/lines as reported by the #3120 review against ToddHebebrand/ios-app-testing; re-verify at fix time.)
Proposed Fix
Sweep each site with the pattern established in #3120: static user-facing copy on screen, Sentry.captureException(err, { tags: { area: ... } }) for the internal detail. Reference implementations: SessionsSheet.tsx and SettingsSheet.tsx.
Reported By
Review findings on #3120; follow-up to #3115
Description
PR #3120 (issue #3115) fixed SessionsSheet and HomeScreen's session-open path rendering raw
Error.messagestrings (function names + HTTP status codes) as user-facing UI. The review found the same defect class in four more places, out of scope for that PR:apps/mobile/src/screens/systems/SystemsScreen.tsx(~line 210)apps/mobile/src/screens/systems/useSystemsData.ts(~line 105)apps/mobile/src/screens/devices/DeviceDetailScreen.tsx(~line 174)apps/mobile/src/screens/chat/HomeScreen.tsx(~line 194, create-session path)(Paths/lines as reported by the #3120 review against
ToddHebebrand/ios-app-testing; re-verify at fix time.)Proposed Fix
Sweep each site with the pattern established in #3120: static user-facing copy on screen,
Sentry.captureException(err, { tags: { area: ... } })for the internal detail. Reference implementations:SessionsSheet.tsxandSettingsSheet.tsx.Reported By
Review findings on #3120; follow-up to #3115