Skip to content

feat (backend): implementação do backend da tarefa S2-12 - #55

Merged
LoadCG merged 4 commits into
mainfrom
s2-12
Oct 25, 2026
Merged

LoadCG merged 4 commits into
mainfrom
s2-12

Conversation

@heitors1337

@heitors1337 heitors1337 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Basicamente consiste em uma busca aberta a todos os perfis (devs, PO e etc.) e devolve no máximo 5 itens, facilitando caso você queira usar uma funcionalidade de projeto x em y, mas ainda sem o frontend....

removido alguns bugs e adicionando outros

Basicamente consiste em uma busca aberta a todos os perfis (devs, PO e etc.) e devolve no máximo 5 itens, facilitando caso você queira usar uma funcionalidade de projeto x em y. Além disso foi removido bugs e adicionado outros.
@LoadCG

LoadCG commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

@heitors1337 Correções necessárias:

  1. A consulta limita os chunks antes de deduplicar as entidades — Em backend/src/modules/similar-items/similar-items.repository.ts, o limite de 100 é aplicado antes da deduplicação por entidade. Se uma entidade tiver vários chunks, eles podem ocupar a amostra e impedir que outras entidades relevantes apareçam no resultado.

    Sugestão: deduplicar ou agregar os resultados por entidade antes de aplicar o limite. Adicionar um teste com vários chunks da mesma entidade e outras entidades candidatas, verificando que as sugestões retornadas são distintas e incluem as candidatas esperadas.

  2. As novas rotas não estão documentadas no OpenAPI — As rotas de busca e reaproveitamento de itens similares foram adicionadas em backend/src/app.ts, mas não estão descritas em docs/api/openapi.yaml. Isso deixa sem contrato documentado os caminhos, parâmetros, payloads e respostas desses endpoints.

    Sugestão: documentar todas as novas rotas, incluindo autenticação, autorização, formatos de sucesso e erros. Adicionar ou atualizar testes de contrato para confirmar que a especificação acompanha as rotas implementadas.

as rotas estão documentadas também
…a branch

basicamente um painel discreto, com avisos, botões e sinalizando o projeto de origem

vscode 🤙
@LoadCG

LoadCG commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Pendências para concluir a S2-12

Revisei a PR no head 3869c6c. Os apontamentos anteriores sobre deduplicação dos resultados antes do limite e documentação OpenAPI parecem ter sido atendidos: a consulta seleciona um resultado por entidade antes do limite, há teste de regressão para muitos chunks da mesma entidade, e o contrato OpenAPI e seu teste foram adicionados.

Ainda não considero a PR pronta para aprovação:

1. Build do frontend está bloqueado

O CI falha ao compilar frontend/src/views/backlog/SimilarItemsPanel.tsx porque o componente importa ../../assets/styles/similar-items.css, mas esse arquivo não está presente na branch s2-12. Erro reportado:

Could not resolve "../../assets/styles/similar-items.css" from "src/views/backlog/SimilarItemsPanel.tsx"

Adicione o stylesheet no caminho esperado ou ajuste o import para o arquivo correto/remova-o caso a estilização já esteja coberta. Depois, confirme que o build de produção do frontend conclui.

2. A validação de QA não chegou às etapas finais

A etapa de QA também falhou durante a compilação da interface. Por isso, as validações posteriores de banco e E2E foram ignoradas; há testes ignorados no resultado disponível, então ainda não existe evidência de que os fluxos integrados dessa funcionalidade passaram.

Após corrigir o build, rode novamente o workflow completo e confirme que as validações aplicáveis à busca entre projetos e à interface foram executadas. Se algum teste depender de serviço/ambiente e continuar ignorado, documente qual cobertura foi executada e qual permanece pendente.

3. A descrição da PR está desatualizada

A descrição diz que a entrega é apenas backend e que não há frontend, embora o diff atual inclua componentes e integração de frontend. Atualize o resumo e a seção de validação para refletir o conteúdo e os resultados reais do CI.

Critérios para retomar a aprovação

  • O build do frontend passa no head atualizado.
  • O CI de QA completa as etapas aplicáveis de integração/E2E; eventuais etapas não executadas ficam identificadas e justificadas.
  • Os testes cobrem a deduplicação por entidade e a preservação de candidatos válidos quando há muitos chunks de uma mesma entidade.
  • A resposta mantém a origem do item sugerido e a relação/motivo de similaridade, preservando o escopo cross-project previsto para esta tarefa.
  • A descrição da PR representa corretamente o frontend, o backend e as validações entregues.

A S2-21 é um card separado e não encontrei PR aberta para ela; esta revisão trata somente da S2-12.

@LoadCG
LoadCG merged commit 8e92928 into main Oct 25, 2026
6 checks passed
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.

3 participants