feat(observability): integra Sentry (error tracking) + health check#64
Merged
Conversation
Adiciona observabilidade de producao -- a lacuna que deixou o 500 do clock-out passar despercebido ate a usuaria reclamar. Erros server-side (route handlers, server actions, RSC) e client-side passam a ser capturados com alerta. - @sentry/nextjs no App Router: init server/edge via instrumentation.ts (sem quebrar a validacao do keyring no boot), client via instrumentation-client.ts, onRequestError e global-error.tsx. sendDefaultPii:false (nao envia dados da usuaria). Ligado so em producao. - hour-bank: os recalculos fail-safe engoliam o erro por design (mutacao ja commitada); agora reportam ao Sentry sem deixar de engolir -- era exatamente a classe de falha silenciosa que motivou a integracao. - /api/health: liveness/DB check (SELECT 1) para o health check do App Service e o uptime monitor do Sentry. await connection() em vez de force-dynamic (cacheComponents). Publico -- proxy.ts libera api/health e o tunnel /monitoring. - next.config: withSentryConfig por cima do withSerwist; tunnelRoute /monitoring (same-origin, nao mexe na CSP); source maps so sobem com auth token; release pelo SHA do commit (SENTRY_RELEASE), ja que o .git nao entra no build Docker. - Dockerfile/build-image: NEXT_PUBLIC_SENTRY_DSN como build-arg; SENTRY_AUTH_TOKEN como build secret (nao persiste na imagem). Validado: gates locais verdes (tsc, 295 testes, biome, react-doctor) e build standalone via container -- /api/health responde 200 no container.
johnlaff
added a commit
that referenced
this pull request
Jul 10, 2026
Cobre a nova observabilidade (PR #64) em toda a documentacao viva: - docs/adr/0006-observabilidade-sentry.md: ADR novo com contexto (o 500 do clock-out invisivel), decisao (Sentry server+client, tunnel /monitoring, source maps por SHA, sendDefaultPii false, hour-bank reportando, /api/health + healthCheckPath), consequencias e alternativas (Datadog/App Insights/self-host). - AGENTS.md: secao Observabilidade com as regras operacionais (init, tunnel na allowlist do proxy, catches silenciosos reportando, connection() no health). - README.md: feature de observabilidade + Sentry na linha de infra. - supabase-security-checklist.md: DSN publico vs SENTRY_AUTH_TOKEN segredo.
johnlaff
added a commit
that referenced
this pull request
Jul 10, 2026
Cobre a nova observabilidade (PR #64) em toda a documentacao viva: - docs/adr/0006-observabilidade-sentry.md: ADR novo com contexto (o 500 do clock-out invisivel), decisao (Sentry server+client, tunnel /monitoring, source maps por SHA, sendDefaultPii false, hour-bank reportando, /api/health + healthCheckPath), consequencias e alternativas (Datadog/App Insights/self-host). - AGENTS.md: secao Observabilidade com as regras operacionais (init, tunnel na allowlist do proxy, catches silenciosos reportando, connection() no health). - README.md: feature de observabilidade + Sentry na linha de infra. - docs/assets/architecture.svg: caixa Sentry (erros via tunnel + /api/health) abaixo do app, e o passo de source maps no CI/CD. - supabase-security-checklist.md: DSN publico vs SENTRY_AUTH_TOKEN segredo.
johnlaff
added a commit
that referenced
this pull request
Jul 10, 2026
Cobre a nova observabilidade (PR #64) em toda a documentacao viva: - docs/adr/0006-observabilidade-sentry.md: ADR novo com contexto (o 500 do clock-out invisivel), decisao (Sentry server+client, tunnel /monitoring, source maps por SHA, sendDefaultPii false, hour-bank reportando, /api/health + healthCheckPath), consequencias e alternativas (Datadog/App Insights/self-host). - AGENTS.md: secao Observabilidade com as regras operacionais (init, tunnel na allowlist do proxy, catches silenciosos reportando, connection() no health). - README.md: feature de observabilidade + Sentry na linha de infra; alt do diagrama enriquecido (a11y: via <img> o aria-label interno nao e lido). - docs/assets/architecture.svg: caixa Observabilidade (Sentry via tunnel + /api/health) abaixo do app, seta de source maps CI->Sentry, legenda das setas, contraste AA, geometria alinhada e fallback de presentation attributes. - supabase-security-checklist.md: DSN publico vs SENTRY_AUTH_TOKEN segredo.
johnlaff
added a commit
that referenced
this pull request
Jul 10, 2026
Cobre a observabilidade (PR #64) em toda a documentacao viva: - docs/adr/0006: ADR da decisao (Sentry server+client, tunnel /monitoring, source maps por SHA, sendDefaultPii false, hour-bank reportando, /api/health + healthCheckPath), consequencias e alternativas. - AGENTS.md: secao Observabilidade com as regras operacionais. - README.md: feature + Sentry na infra; alt do diagrama enriquecido (a11y). - docs/assets/architecture.svg: caixa Observabilidade + seta de source maps CI->Sentry + legenda das setas; contraste AA, geometria alinhada e fallback de presentation attributes (revisado para craft nos dois temas). - supabase-security-checklist.md: DSN publico vs SENTRY_AUTH_TOKEN segredo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O quê e por quê
Fecha a lacuna de observabilidade identificada na auditoria: o 500 do clock-out só foi descoberto porque a usuária reclamou. Agora erros server-side (route handlers, server actions, RSC) e client-side são capturados com alerta — via Sentry no plano patrocinado (Student Pack).
Mudança
src/instrumentation.tsonRequestError, preservando a validação do keyring no bootsrc/instrumentation-client.tsonRouterTransitionStartsrc/sentry.{server,edge}.config.tsSentry.init—enabledsó em produção,sendDefaultPii: false, tracing levesrc/app/global-error.tsx<html>/<body>próprios) →captureExceptionsrc/lib/hour-bank.tssrc/app/api/health/route.tsSELECT 1) para o App Service + uptime monitor;await connection()(nãoforce-dynamic, incompatível comcacheComponents)src/proxy.tsapi/healthemonitoringno matcher (senão o health leva 307 e o tunnel quebra)next.config.tswithSentryConfigpor cima dowithSerwist;tunnelRoutesame-origin (não mexe na CSP); source maps só com auth token;release= SHADockerfile/build-image.ymlSENTRY_AUTH_TOKENcomo build secret (não persiste na imagem)Design validado por 3 lentes independentes antes do código
As decisões não-óbvias (matcher do proxy,
connection()vsforce-dynamic, composição dos plugins, wiring do source-map secret, captura dos catches do hour-bank) foram levantadas e verificadas em revisão adversarial pré-implementação.Como verificar
tsc, 295 testes, Biome (188 arquivos), react-doctor zerado.sw.jspresente, container boota eGET /api/health→ 200 (com DB real). O health voltou 200 (não 307) → matcher do proxy correto.verify) roda sem os secrets Sentry →Sentry.initvira no-op e o upload de source maps é pulado; build segue verde.Pós-merge (não automático)
healthCheckPath=/api/healthno App Service (faço viaazapós o deploy).https://archtime.app/api/health./api/health200, disparar um erro de teste e confirmar o evento chegando comrelease= SHA do commit.