Auditoria de Código — 24 Problemas Encontrados

A TaskManager.defineTask chamada BACKGROUND_LOCATION_TASK apenas imprime a latitude/longitude no console. Ela não envia nada para o backend nem atualiza o Firebase.

// ❌ Como está:
TaskManager.defineTask(BACKGROUND_LOCATION_TASK, ({ data, error }) => {
  if (data) {
    const { latitude, longitude } = data.locations[0].coords;
    console.log('[BG Location]', latitude, longitude); // só loga!
  }
});

Deveria chamar enviarLocalizacao() e updatePosition() assim como faz o efeito no foreground.

Não existe nenhuma chamada a Location.startLocationUpdatesAsync(BACKGROUND_LOCATION_TASK, …) em nenhum lugar deste arquivo — nem no toggleOnline, nem em um useEffect.

// ❌ Falta isto em toggleOnline (quando next === true):
await Location.startLocationUpdatesAsync(BACKGROUND_LOCATION_TASK, {
  accuracy: Location.Accuracy.High,
  timeInterval: 10000,
  distanceInterval: 20,
  foregroundService: { notificationTitle: 'FederalPro', notificationBody: 'Você está online' },
});

// E isto ao ficar offline:
await Location.stopLocationUpdatesAsync(BACKGROUND_LOCATION_TASK);

Definir a task não a inicia automaticamente. Sem isso, ela nunca roda.

O estado começa com gpsServices: false e canGoOnline exige perms.gpsServices === true. Como refreshPermissions() só roda quando menuVisible === true, o usuário vê o banner "Você não pode ficar online" mesmo tendo concedido tudo — até abrir o menu.

// ❌ Hoje:
useEffect(() => {
  if (menuVisible) refreshPermissions();
}, [menuVisible, refreshPermissions]);

// ✅ Deveria rodar também no mount:
useEffect(() => {
  refreshPermissions();
}, [refreshPermissions]);

O efeito chama askLocation() / askBackgroundLocation(), mas nunca atualiza o estado perms depois. Além disso, faltam dependências (askLocation, askBackgroundLocation) — warning clássico do react-hooks/exhaustive-deps.

useEffect(() => {
  (async () => {
    const fg = await Location.getForegroundPermissionsAsync();
    if (fg.status !== 'granted') {
      await askLocation(); // ❌ não faz setPerms depois
      return;
    }
    ...
  })();
}, [refreshKey]); // ❌ deps incompletas

O usuário pode marcar pro + confort + black, mas o listener só recebe a categoria da primeira posição do array. As outras são ignoradas no filtro de corridas.

const categoriaAtualMotorista = useMemo(() => {
  if (selectedCategorias.length === 0) return null;
  const primeira = categoriasDB.find((c) => Number(c.id) === selectedCategorias[0]);
  return normalizarSlug(primeira || CATEGORIAS_ESTATICAS[0]); // ❌ string única
}, [selectedCategorias, categoriasDB]);

Solução: fazer categoriaMotorista ser um array e ajustar useOuvirCorridas para filtrar por qualquer uma delas.

Se a API de categorias falhar ou retornar vazio, o efeito desiste cedo e nenhuma categoria fica pré-selecionada — mesmo existindo as 3 estáticas (Federal Pet, Home Care, Mulher/Kids).

useEffect(() => {
  if (selectedCategorias.length > 0) return;
  if (categoriasDB.length === 0) return; // ❌ desiste antes de considerar as estáticas
  ...
}, [categoriasDB, selectedCategorias, user?.perfil, user?.categoria]);

Se o app fechar estando online, o Firebase mantém motoristas_online/{uid} ativo, mas a UI volta com isOnline = false. Estado divergente clássico.

Solução: ler o estado inicial via onValue(ref(db, 'motoristas_online/'+uid)) no mount, e/ou forçar goOffline() no boot quando o app inicia.

Cada toque em uma categoria, GPS ou biometria dispara um POST imediato. Trocar de categoria rapidamente gera N requisições desnecessárias.

useEffect(() => {
  if (!prefsLoaded) return;
  if (selectedCategorias.length === 0) return;
  persistirPreferencias({ gps: selectedGPS, biometria: biometriaEnabled, categorias_ids: ... });
}, [selectedGPS, biometriaEnabled, selectedCategorias, prefsLoaded, persistirPreferencias]);

Adicione um setTimeout de ~500ms no cleanup do efeito para agrupar mudanças.

Se corridaParaModal e mensagemParaModal aparecerem ao mesmo tempo, tocarSomNotificacao() toca 2x sobreposto.

useEffect(() => { if (corridaParaModal) tocarSomNotificacao(); }, [corridaParaModal]);
useEffect(() => { if (mensagemParaModal) tocarSomNotificacao(); }, [mensagemParaModal]);

Use um guard de tempo ("último som < 1s atrás, ignora") ou um único efeito combinado.

O backend recebe a localização em foreground mesmo quando o motorista está offline. Já o Firebase (updatePosition) só é atualizado se isOnline.

enviarLocalizacao(token, coords).catch(...); // ❌ sem if (isOnline)
if (isOnline) updatePosition(...);            // ✅ condicional

Se for heartbeat, documente. Se não, envolva no if (isOnline) para economizar banda.

Se goOffline falhar (rede ruim), o motorista fica "fantasma" online no Firebase permanentemente — e ainda por cima sem log.

try {
  if (user?.user_id) await goOffline(user.user_id);
} catch {} // ❌ engole erro silenciosamente
signOut();

Solução: logar, fazer retry, ou usar onDisconnect() do Firebase para limpar automaticamente.

Se o request expirar (8s de timeout), o usuário vê o botão voltar ao estado original sem nenhuma explicação. Nenhum Alert, nenhum toast.

catch (e: any) {
  console.log('[Home] toggleOnline erro:', e?.message ?? e); // só loga
}

Depois de goOnline, o código sobrescreve status: 'disponivel' via update direto. Se o backend já tiver setado outro status (ex.: aguardando_corrida), você o apaga.

await goOnline(...);
if (next) {
  await update(ref(db, `motoristas_online/${user.user_id}`), {
    status: 'disponivel', // ❌ sobrescreve qualquer status
    atualizadoEm: Date.now(),
  });
}

Prefira fazer esse reset dentro de goOnline ou via transação.

Se o Firebase não inicializou, ref(db, ...) estoura um erro obscuro.

import { db } from '@/services/firebase';
...
update(ref(db, `motoristas_online/${user.user_id}`), {...}) // ❌ sem guard

Adicione validação antes: if (!db) { Alert.alert('Firebase indisponível'); return; }

import('@/services/firebase') dentro de um useEffect é redundante — db já é importado estaticamente no topo do arquivo. Remova o efeito ou importe testarFirebase estaticamente.

O Pressable externo já tem accessibilityRole="switch" e o Switch nativo dentro também anuncia como switch. VoiceOver/TalkBack lê duas vezes.

Escolha um: ou mantenha só o Pressable com role e use um View visual, ou remova o role do wrapper.

Funciona, mas vale documentar: hitSlop e props de acessibilidade ficam no Pressable externo, enquanto style vai para o Animated.View. Pode confundir quem for dar manutenção.

getIconeCategoria retorna { lib: string, name, color } e o componente IconeCategoria faz uma cadeia de if (lib === ...). Mais simples: um mapa slug → componente direto.

Switch do React Native não documenta focusable como prop. Algumas versões emitem warning. Remova ou use accessible={false}.

Quando o usuário tenta desmarcar a última categoria, o código faz if (bloqueado) return; — nada visualmente acontece. Um toast do tipo "Mínimo de 1 categoria ativa" resolveria.

CATEGORIAS_ESTATICAS usa IDs -1, -2, -3, mas o POST filtra id > 0. Se o usuário só tiver estáticas selecionadas, após reload ele fica sem categoria nenhuma.

categorias_ids: selectedCategorias.filter((id) => id > 0) // ❌ descarta estáticas

Solução: persistir também os slugs das estáticas num campo separado, ou usar IDs positivos fictícios reservados.

Ele aparece depois de vários useEffect. Não quebra regras dos hooks, mas por convenção e para evitar surpresas futuras, agrupe todos os hooks de dados no topo do componente.

O fallback || 'info' cobre undefined, null e "", mas não valida se é um tipo reconhecido pelo modal (ex.: "erro" vs "error"). Considere um enum.

configurarHandlerNotificacoes() é chamado uma vez sem retornar um unsubscribe. Em app-level geralmente é ok, mas se o handler registrar listeners globais, ideal retornar um cleanup para o caso de hot reload em dev.

useEffect(() => {
  try { configurarHandlerNotificacoes(); } catch (e) { ... }
}, []); // ❌ sem return de cleanup