Skip to content

[PAV-133] Corrigir tipagem em autenticação e conexões - #246

Merged
Benevanio merged 1 commit into
Cla-Code-Community:developfrom
Jovinull:fix/pav-133-tipagem-auth
Sep 7, 2026
Merged

Benevanio merged 1 commit into
Cla-Code-Community:developfrom
Jovinull:fix/pav-133-tipagem-auth

Conversation

@Jovinull

Copy link
Copy Markdown
Contributor

Linear

Issue: PAV-133

Branch flow

  • Este PR é um fix destinado a develop.
  • Este PR não contém alteração direta ou fluxo indevido para master.
  • A branch foi criada a partir da base prevista pelo fluxo do projeto.

Objetivo

Corrigir erros de TypeScript nos fluxos de boas-vindas e desconexão de provedores, preservando o comportamento das rotas e evitando coerções de tipo inseguras.

Escopo

O que faz parte desta entrega:

  • Garantia de valores string no envio de e-mail de boas-vindas.
  • Fallback de nome para perfil OAuth sem nome ou username.
  • Uso dos valores validados no cadastro por credenciais.
  • Validação de req.params.provider como valor escalar antes da lista de providers e da desconexão.
  • Testes de regressão para os cenários corrigidos.

O que está fora do escopo:

  • Alterar templates ou provider de e-mail.
  • Alterar banco, migrations ou contratos HTTP.

Resumo das alterações

  • O OAuth usa Usuário como fallback de nome quando os dois campos opcionais não existem.
  • O cadastro usa o e-mail normalizado e o nome validado — ou o prefixo do e-mail — para o e-mail de boas-vindas.
  • A rota de conexões recusa provider em formato não escalar antes de chamar o serviço.
  • Adicionados testes de fallback de nome e provider inválido em array.

Como testar

npm exec --workspace=backend tsc -- --noEmit -p tsconfig.json
npm test --workspace=backend

Validation

  • Checagem de tipos do backend executada sem erros.
  • Testes relevantes executados.
  • Suíte completa do backend executada: 60 arquivos e 572 testes aprovados.
  • Validação manual não aplicável: a mudança é interna ao backend e está coberta por testes unitários e de rota.

Impacto de banco / migration

  • Não altera schema nem migrations.

Impacto em contratos/API

  • Não altera endpoint, payload, schema ou comportamento público de API.

Impacto de configuração / infraestrutura

  • Não altera configuração, env, Docker, filas, cache, serviços ou infraestrutura.

Impacto de segurança

  • Sem impacto relevante.

Impacto de dados pessoais / privacidade

  • Sem tratamento novo ou alteração de dados pessoais.

Compatibilidade / dependências

  • Não depende de outra task/PR.

Riscos

Baixo. O fallback impede que dados nulos cheguem ao template de e-mail, e a nova validação apenas rejeita formatos que a rota não suporta.

Rollback

Reverter este commit restaura o comportamento anterior sem migração nem alteração de dados.

Checklist final

  • Escopo limitado ao card.
  • Não inclui segredos, .env, certificados ou tokens.
  • Não inclui arquivos temporários, builds, caches ou artefatos desnecessários.
  • Não executa deploy como efeito desta PR.
  • Critérios de aceite do card foram conferidos individualmente.

@Benevanio
Benevanio merged commit 6dc09d9 into Cla-Code-Community:develop Sep 7, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in JobAtlas – Kanban Sep 7, 2026
Benevanio added a commit that referenced this pull request Sep 28, 2026
…rd (#253)

## Release

Promoção da branch `develop` para `master`, consolidando 26 commits e 49
arquivos alterados.

## Objetivo

Disponibilizar em produção as entregas recentes de segurança,
versionamento da API, notas privadas por candidatura, melhorias no
dashboard e ajustes de confiabilidade/documentação.

## Escopo

Inclui as entregas das PRs `#234`, `#239`, `#242`, `#245`, `#246`,
`#250` e `#252`.

### API e documentação

- Adicionado o prefixo versionado `/api/v1` para os endpoints da API.
- Mantidas temporariamente as rotas sem prefixo para compatibilidade com
clientes existentes.
- Expandida a documentação OpenAPI/Swagger com schemas, respostas,
exemplos e endpoints principais.
- Atualizado o README com a estratégia de versionamento da API.

### Segurança

- Aplicado rate limit ao `POST /auth/register`, por IP e por e-mail.
- Endurecido o fallback de CORS em produção, sem permitir `localhost`
quando `CORS_ALLOWED_ORIGINS` não estiver configurada.
- Adicionado HSTS para tráfego HTTPS/produção.
- Aplicada CSP restritiva na API, frontend, painel administrativo, Nginx
e Vercel.
- Removido script inline de bootstrap do tema para compatibilidade com
CSP.
- Reforçada a renderização segura de descrições de vagas, bloqueando
atributos ativos, `iframe` e protocolos não permitidos.

### Candidaturas e dashboard

- Adicionado suporte a múltiplas notas privadas por candidatura.
- Criada a tabela `application_notes`, com migration, índices e remoção
em cascata.
- Disponibilizados endpoints autenticados para listar, criar, editar e
remover notas de vagas salvas.
- Garantido isolamento por usuário e ownership da vaga em todas as
operações de notas.
- Reorganizado o detalhe da candidatura com tiles para local,
modalidade, nível, fonte, salário e match.
- Removida a duplicação entre os dados principais e o bloco de detalhes
adicionais.
- Mantida compatibilidade com o campo legado de nota única para vagas
não rastreadas.

### Autenticação e confiabilidade

- Corrigidos tipos e fallbacks no envio de e-mail de boas-vindas.
- Validado `provider` como valor escalar antes da desconexão de contas
OAuth.
- Ajustado o timeout de teste de inicialização do servidor sob execução
completa da suíte.

### Documentação e fluxo de contribuição

- Adicionado template padrão de Pull Request.
- Atualizados `README.md`, `BACKEND.md` e `SECURITY.md`.
- Documentadas variáveis de ambiente de CORS e rate limiting.
- Registradas especificação, validação e handoff da PAV-92.

## Impacto de banco / migration

- [x] Altera schema e migrations.

Inclui a migration `0014_hard_korath`, que cria `application_notes` e
seus índices/foreign keys. A migration deve ser aplicada antes da
disponibilização do backend em produção.

## Impacto em contratos/API

- [x] Altera endpoint e comportamento de API.

A API passa a expor os endpoints em `/api/v1`. As rotas legadas sem
prefixo permanecem disponíveis temporariamente para evitar quebra
imediata de clientes.

Também foram adicionados endpoints de notas privadas em:

- `GET /saved-jobs/:id/notes`
- `POST /saved-jobs/:id/notes`
- `PATCH /saved-jobs/:id/notes/:noteId`
- `DELETE /saved-jobs/:id/notes/:noteId`

## Impacto de configuração / infraestrutura

- [x] Altera configuração de segurança.

A produção deve revisar:

- `CORS_ALLOWED_ORIGINS`
- `AUTH_RATE_LIMIT_IP_MAX`
- `AUTH_RATE_LIMIT_ACCOUNT_MAX`
- `AUTH_RATE_LIMIT_WINDOW_SECONDS`
- Headers CSP configurados em Nginx e Vercel

## Validação

As PRs de origem possuem cobertura automatizada para:

- Versionamento `/api/v1` e contrato Swagger.
- Rate limiting, CORS e HSTS.
- CSP e sanitização de conteúdo externo.
- CRUD e isolamento de notas privadas.
- Detalhe de candidatura e regressões de interface.
- Tipagem de autenticação e desconexão de provedores.

Também foram executados testes, lint e builds de backend/frontend nos
fluxos de origem.

## Riscos

- A aplicação de `0014_hard_korath` é obrigatória antes do deploy do
backend.
- Clientes devem migrar gradualmente para `/api/v1`; as rotas legadas
continuam disponíveis nesta etapa.
- CSP e CORS restritivos podem bloquear integrações externas não
declaradas.
- A PR está divergente de `master`; sincronizar a base com os dois
commits pendentes e revalidar os checks antes do merge.

## Rollback

O rollback pode ser realizado revertendo o merge da release.

Caso a migration já tenha sido aplicada, a reversão do código deve
preservar a tabela `application_notes` até que exista uma migration
reversa planejada, evitando perda de notas já criadas.

## Checklist final

- [x] Consolida entregas de `develop` para `master`.
- [x] Inclui alterações de API, banco, segurança, frontend e
documentação.
- [x] Mantém compatibilidade temporária com rotas legadas.
- [x] Possui migrations a serem aplicadas antes do deploy.
- [x] Sincronizar `develop` com os commits pendentes de `master`.
- [ ] Confirmar todos os checks da PR antes do merge.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants