Code Review

Code Review

Architecture, commands, TypeScript and ESLint rules live in CLAUDE.md. This file holds the traps and repo conventions this review gates on. Confirm each finding against the code on disk before reporting it.

Action sheets

• Resolve an object, gate on a named field. A repo convention, not something the library enforces today. Two different payloads are involved, so keep them apart when reviewing:

  • the SHOW payload, SheetManager.show(name, { payload }), which the provider hands to the registered component as a prop. Every sheet uses this.
  • <ActionSheet payload={...}>, ActionSheet's OWN prop. Nothing in src/ sets it, and forwarding the show payload into it is not the same thing.

  Only the second one feeds payloadRef, and the library resolves a close with data || payloadRef.current || data. Since no sheet sets that prop, payloadRef.current is undefined and a falsy result reaches the caller intact today. Keep the convention anyway: adding payload to one <ActionSheet> would silently turn every falsy cancel in that sheet into a confirm. A dismissal also resolves undefined, which truthiness cannot tell apart from a deliberate false. Sheets resolve { cancelled: true } or { field: value } instead. Sheets document their own contract, e.g. src/components/searchFiltersSheet/searchFiltersSheet.tsx. Callers test the field, abridged from src/screens/searchResult/screen/searchResultScreen.tsx:65-77:

  const result = await SheetManager.show(SheetNames.SEARCH_FILTERS, {
    payload: { filters, searchValue: clipSearchValue(searchInputValue) },
  });
  if (result && typeof result === 'object' && result.filters) { ... }

• Sheets unmount on hide, so mount-time resets are enough and cleanups run on close (CLAUDE.md, Sheets). Reject "state persists between invocations".

• A throw in a sheet render or cleanup is fatal: SheetProvider wraps <Application/> (src/index.tsx:39-43), outside ErrorBoundary (src/screens/application/index.tsx:17). Watch native objects in cleanups.

• Payloads freeze at show time. A handler in a payload keeps the closure it had when the sheet opened, so route it through a ref (src/components/quickPostModal/quickPostModalContent.tsx:565-575).

• Missing SheetDefinition? everySheetHasDefinition (src/navigation/sheets.tsx:358) fails typecheck and names it. Keys are plain string literals.

Caret on programmatic writes

• setNativeProps({ text }) on its own moves the caret. Android keeps the caret's DISTANCE FROM THE END, so it lands inside the text just written and the next keystroke splits it. Pass selection whenever the caret position after the write matters: inserts and appends into existing text, plus full replacements of a focused field. A reset to text: '' does not need it. Call sites that pass it: src/components/quickPostModal/quickPostModalContent.tsx:559 and :604, plus src/components/markdownEditor/view/markdownEditorView.tsx:371.

Editor teardown order

• Pending work drains before the save, not in a child cleanup. componentWillUnmount runs in the commit phase ahead of descendant effect cleanups, so the screen calls flushPendingEditorWork() then _saveDraftToDB() (src/screens/editor/screen/editorScreen.tsx:116-127). Register new deferred editor work via registerPendingFlush (src/components/uploadsGalleryModal/mediaInsertQueue.ts:33), not a local cleanup.

Safe area

• edges REPLACES the defaults, it does not add to them. edges={['bottom']} removes the top inset. The top inset is generally the screen's job: a screen rendering BasicHeader wraps it in its own SafeAreaView (src/components/basicHeader/view/basicHeaderStyles.ts:13).

Notification routing

Notification type strings are matched in more than one place, whose spellings do NOT agree. A new type may need adding in several of them.

• Tap routing: the switch at src/screens/application/hook/useInitApplication.tsx:222-302; its default does nothing.
• Websocket to FCM bridge: the allowlist at src/screens/application/container/applicationContainer.tsx:891-901 admits mention, reply, transfer, delegations, scheduled_published, payouts, account_update and weekly_earnings. Each one also needs a case in the title/body switch at :914, whose default announces a bare @source.
• Foreground banner: the allowlist at src/components/foregroundNotification/foregroundNotification.tsx:51-58 admits reply, mention, transfer, delegations and scheduled_published; anything else shows nothing. Its own _onPress (:127) routes transfer and delegations to the wallet, everything else to a post.

Mind the singular/plural split: tap routing matches favorite and has no payout case at all, the list rendering (src/utils/notificationImage.ts:12, src/components/notificationLine/view/notificationLineView.tsx:138) matches favorites and payouts, while the websocket allowlist has payouts but no favorites.