TASK-010: HTTP Client (Android+iOS)

[Gabriel-Pereira1788]

Status: ⬜ Não iniciado
Prioridade: P1 (núcleo do MVP)
Área: Android+iOS
Depende de: TASK-002 (#3)
Bloqueia: TASK-009 (#10) (chamada de refresh), TASK-012 (#13) (chamada de sync)
Paralelizável: Sim, total — não depende de nenhuma peça do core C++, só do spec da bridge (TASK-002 (#3)). Bom candidato pra pegar bem cedo.
Skills obrigatórias: kotlin, swift

Antes de começar

Invoque kotlin (OkHttp ou HttpURLConnection — decidir e documentar qual, atentar a threading/coroutines) e swift (URLSession, async/await). Cruza os dois domínios, mesmo sendo cada metade relativamente simples.

Descrição

Cliente HTTP nativo por plataforma — decisão desta sessão de planejamento: não é C++ compartilhado (ao contrário do resto do core), porque integra melhor com cert store do SO, proxy, e não traz dependência extra tipo libcurl pro bundle. Mesmo assim, ele expõe uma interface fina que o core C++ (TASK-012 (#13), TASK-009 (#10)) chama via callback/JSI — do ponto de vista do orchestrator, "fazer um request HTTP" é uma função só, a implementação por trás é que é nativa-por-plataforma.

Escopo do MVP, REST apenas (transport: "rest" é a única opção implementada — graphql/grpc existem no type mas não são suportados, mvp-scope.md):

1. Executar request (method, path relativo a baseUrl, headers, body) e devolver (status, body, headers).
2. baseUrl + path → URL final (docs/overview.md já documenta a concatenação simples: baseUrl + path).
3. Timeout configurável via DatabaseConfigDefinition.network.timeout.
4. Erro de rede (sem conexão, timeout) precisa ser distinguível de erro HTTP (4xx/5xx) por quem chama — o Sync Orchestrator (TASK-012 (TASK-012: Sync Orchestrator (C++) #13)) trata isso diferente (retry vs falha definitiva).

Docs de referência

• docs/architecture.md → seção "Endpoint" (EndpointDefinition: method, path, headers) — o shape do que este cliente recebe como entrada.
• docs/overview.md → seção "Base URL" — regra de concatenação baseUrl + path.
• docs/mvp-scope.md → linha "Transport" — confirma REST-only no MVP e por quê.
• docs/architecture.md → DatabaseConfigDefinition.network.timeout (dentro de "Database Config (global)") — de onde vem o timeout configurável.

Critérios de aceite

• GET/POST/PUT/PATCH/DELETE funcionando nas duas plataformas.
• baseUrl + path concatenados corretamente (incluindo casos de barra duplicada/faltando, normalizados).
• Timeout de network.timeout respeitado — teste contra um endpoint lento/simulado.
• Erro de rede (sem conexão) e erro HTTP (ex: 500) chegam como tipos de erro distintos pra quem chamou.
• Headers customizados (EndpointDefinition.headers) e o header de auth (injetado pela TASK-009 (TASK-009: Credential Provider (Android+iOS) #10)) coexistem sem um sobrescrever o outro incorretamente.
• Body de request/response tratado como JSON (serialização/deserialização), consistente com o formato que a TASK-008 (TASK-008: Interpretador de Expressões Declarativas (C++) #9) (interpretador de expressões) produz/consome.

Fora de escopo

Decidir o conteúdo do body ou como extrair da response (isso é TASK-008 (#9)). Retry (fixo, 3 tentativas/5s — implementado na TASK-012 (#13), este cliente só executa uma tentativa por chamada, quem decide repetir é o orchestrator). Suporte a graphql/grpc (fora do MVP).

---

📄 Fonte: docs/tasks/010-http-client.md

[Gabriel-Pereira1788]

Progresso parcial nesta sessão (branch feat/http-client-native, do zero sobre develop — a branch anterior feat/http-client foi abandonada como base, ver histórico de comentários/decisões acima).

Concluído e testado de verdade:

• Core C++: HttpTypes, HttpUrlBuilder, NetworkConfig (valor, sem singleton), platform::httpExecute (contrato síncrono), CredentialHttpCaller (adapter para CredentialProvider::HttpCaller — a ponte entre TASK-010 e TASK-009 estava faltando, endereçada aqui), configure() agora popula NetworkConfig via DatabaseManager. npm run test:native: 218 assertions, 82 casos, todos passando.
• Android: OkHttpAdapter (core Kotlin puro) + bridge JNI (platform_android_http.cpp/SalveDbHttpClient.kt) implementando platform::httpExecute de ponta a ponta. android/src/test criado do zero (não existia). npm run test:native:android novo, 10 testes passando. Build real do app de exemplo (:app:assembleDebug) validado.

Ainda faltando: iOS (Swift/URLSession + bridge ObjC++), verificação final e PR.

Continuo na próxima sessão.