Skip to content

feat(candidaturas): adicionar notas privadas múltiplas - #242

Merged
Benevanio merged 2 commits into
Cla-Code-Community:developfrom
Jovinull:feature/pav-20-notas-privadas
Sep 7, 2026
Merged

feat(candidaturas): adicionar notas privadas múltiplas#242
Benevanio merged 2 commits into
Cla-Code-Community:developfrom
Jovinull:feature/pav-20-notas-privadas

Conversation

@Jovinull

@Jovinull Jovinull commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Card

Objetivo e escopo

  • Implementar a alternativa de múltiplas notas privadas por candidatura descrita na atualização do card.
  • Preservar o isolamento das notas por usuário e vaga salva.

O que foi feito

  • Adiciona o modelo application_notes, migration e índices para notas vinculadas a uma candidatura.
  • Disponibiliza endpoints autenticados para listar, criar, editar e remover notas em /saved-jobs/:id/notes.
  • Garante ownership nas operações: uma nota só é acessível pelo dono da candidatura.
  • Atualiza o detalhe de candidatura com estados de carregamento, vazio e erro, além de ações para criar, editar e remover notas.
  • Mantém o campo legada de nota única apenas em vagas ainda não rastreadas, evitando quebra de compatibilidade.

Módulos afetados

  • Schema e migration do backend.
  • Serviço, controller e rotas de notas de candidatura.
  • Detalhe da vaga no dashboard e cliente HTTP do frontend.
  • Testes de serviço e interação na interface.

Validação

  • npm test --workspace=backend — 61 arquivos e 573 testes aprovados.
  • npm test --workspace=frontend — 52 arquivos e 344 testes aprovados.
  • npm run lint --workspace=frontend — concluído sem erros; permanece um warning preexistente de hook.
  • npm run build --workspace=frontend — concluído com sucesso.
  • Teste da interface cobre cliques para adicionar, editar e remover uma nota no detalhe da candidatura.
  • Testes do serviço cobrem listagem por owner, bloqueio de acesso a vaga de outro usuário e CRUD de notas.

Riscos e observações

  • A migration 0014_hard_korath.sql deve ser aplicada antes do deploy do backend.
  • A proposta implementa explicitamente o cenário de múltiplas notas. Caso a decisão do produto seja manter uma única nota por candidatura, esta PR pode ser dispensada e o card encerrado pelo escopo simplificado já existente.

@Jovinull
Jovinull requested a review from hltav as a code owner August 29, 2026 19:06
Benevanio
Benevanio previously approved these changes Sep 7, 2026
View reviewed changes
@Benevanio
Benevanio merged commit 4d108b9 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
@Jovinull Jovinull mentioned this pull request Sep 7, 2026
Closed
5 tasks done
Benevanio added a commit that referenced this pull request Sep 28, 2026
@Benevanio
release: consolida entregas de segurança, API, candidaturas e dashboa…
008fda0 
…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

Reviewers

@Benevanio Benevanio Benevanio approved these changes
@lima300 lima300 Awaiting requested review from lima300
@nayarakarinesilva nayarakarinesilva Awaiting requested review from nayarakarinesilva nayarakarinesilva is a code owner
@hltav hltav Awaiting requested review from hltav hltav is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Jovinull @Benevanio