Bling ERP: minors não bloqueantes das reviews da #802

[vitorrgg]

Follow-up da review de 19/08 na #802 — minors 🟢 não bloqueantes que sobraram (dois já caíram na própria PR: a guarda morta stockBalances && e o contador de testes do corpo da PR; o do EVENT_SKIP_FLAG ficou obsoleto com a remoção do skip-event-headers.ts).

Em packages/apps/bling-erp:

• src/integration/export-product-to-bling.ts — dois GET /produtos/{blingProductId} idênticos na mesma execução (blocos de variações com preço divergente e de releitura pós-POST não são exclusivos), num app que economiza cota diária.
• src/integration/export-product-to-bling.ts — isDetailLoaded é derivável de blingProducts, que segue em escopo e inalterado.
• src/bling-auth/client.ts:12-15 — comentário afirma que "um campo de instância nunca espaçaria nada", o que é falso (o checkTime antigo espaçava chamadas sequenciais da mesma instância; o que não cobria era concorrência). O trade-off real do escopo de módulo — todo tráfego Bling do processo serializado a 1 req/s entre handlers independentes — fica escondido.
• src/bling-auth/check-enable-api.ts — RATE_LIMIT_WINDOW_MS mora no consumidor e é importado por quem implementa a política (create-access.ts), enquanto EXPIRES_IN_GAP_SEC foi para o módulo neutro (tokens-doc.ts): duas convenções para o mesmo problema. E o bloco de doc do checkEnableApi ficou colado na constante inserida entre ele e a função.
• src/integration/parsers/order-to-bling.ts:16 — hasWarnedHolidays é estado mutável de módulo num arquivo documentado como parser puro (scripts/tests.sh); a ordem dos testes passa a importar.
• src/integration/import-product-from-bling.ts — isStockOnly virou parâmetro morto (implica !update_product, primeiro disjunto do guard nunca muda o resultado) e o isQuantityOnly é a terceira cópia do mesmo predicado.
• src/integration/integration-handler.ts — export default de um tipo, sem consumidor; precedente do repo é export type { ... } no módulo dono (pubsub.ts).
• turbo.json — a aresta test → build cobre o bling-erp, mas test:apps segue sem dependsOn: as duas definições divergem (encosta na Add apps tests to release script #237).

Da review de 13/08 (Optional, seguem válidos): preço não exportado em loja multiloja, estoque de variação sempre relançado sem comparação, /estoques/saldos sem paginação (>100 variações), feriados hardcoded até 2027, datas em UTC (off-by-one de fuso; tiny-erp subtrai 3h), numeroLoja numérico na busca do import, fallback de forma de pagamento pegando data[0] de qualquer tipo, recursão de categoria sem guarda de ciclo, pictureId podendo apontar imagem de outra variação.

[leomp12]

Dois itens da 4ª rodada da #802, opcionais: os dois defeitos existem no app-bling-erp-v2 em produção (portados, não regressões da migração), e cada um tem uma mudança de comportamento observável — por isso ficaram de fora do merge em vez de entrarem com os outros dois fixes (881404f24, ead0d3fb3).

• export-product-to-bling.ts — canCreateNew: false não segura a criação quando a busca no Bling volta vazia. A guarda de :86-89 vive dentro do if (Array.isArray(blingProducts) && blingProducts.length), então listagem vazia (ou o 404 convertido em null em :73) passa direto e :110 deixa entrar por !blingStore ou por export_quantity ligado → POST /produtos cria o produto mesmo com new_products desligado. Idêntico no v1 (lib/integration/export-product.js:72-94); o 82b95d0d6 já fechou o caso de products-quantitySet. Fix: generalizar a guarda de :103-106 para if (!canCreateNew && !originalBlingProduct) e apagar a de :86-89 — o lugar tem que ser depois do if/else de :81-94, antes disso originalBlingProduct ainda não foi atribuído. Trade-off: produto com bling:id apagado no Bling deixa de ser recriado por evento de preço/estoque (hoje, e no v1, é recriado).

• parsers/product-to-bling.ts — variação sem codigo no Bling não tem o estoque exportado. A variação criada na interface do Bling sem SKU tem codigo vazio e é importada com o id do Bling como SKU da loja (product-from-bling.ts:262); na volta o find de :147-149 casa só por codigo e não acha, então :270 sai sem lançar /estoques. Nunca funcionou no v1 (parsers/product-to-bling.js:122-138, que ainda mandava o PUT sem id e tomava 400); o que mudou com o 82b95d0d6 é que agora é no-op silencioso — vale um logger.warn no skip de qualquer forma. Fix: espelhar import-product-from-bling.ts:58 em duas passadas (primeiro codigo, depois String(id)), não num OR de uma passada. Colateral: em export completo a variação passa a ir com id — conserta também o 400 do PUT — e :156 grava o SKU-id como codigo da variação no Bling, onde hoje está vazio.