Skip to content

[S2-03/S2-08/S2-09/S2-10] Integra indexação, busca no acervo, harness e copiloto sem conflitos - #49

Merged
RafaMatesco merged 9 commits into
mainfrom
integracao/s2-03-s2-08-s2-10
Oct 20, 2026
Merged

RafaMatesco merged 9 commits into
mainfrom
integracao/s2-03-s2-08-s2-10

Conversation

@Garakis

@Garakis Garakis commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

O que este PR faz

Integra na main, de uma vez e já sem conflitos, três entregas da Sprint 2 que estavam em branches separadas:

Cada branch entrava limpa na main sozinha, mas conflitavam entre si. Os conflitos foram todos de soma (cada branch acrescentou algo no mesmo ponto), e a resolução manteve os dois lados em todos eles:

Arquivo Resolução
backend/src/app.ts mantidos os imports e app.use de indexingRouter, documentTextRouter e dos routers do harness
backend/package.json mantidos os scripts test:integration:indexing, search, knowledge, harness, copilot e test:smoke:harness:local
docs/api/openapi.yaml mantidas as rotas index-events, excerpts, admin/ai-harness e ai/requests, e os schemas IndexEvent, HarnessConfig e HarnessRevision
.github/workflows/ci.yml mantidos os dois passos novos: testes de indexação (PostgreSQL) e validação do harness

Nenhuma linha de lógica foi alterada na resolução.

Ponto de atenção

Ficam duas migrations com número 019: 019_ai_harness.sql (S2-09) e 019_entity_index_events.sql (S2-03). Elas não dependem uma da outra, o runner registra o nome completo e a main já tem outros números repetidos (004, 005, 007...), então mantive os nomes originais. Se o grupo preferir, dá para renomear uma delas para 020 antes do merge.

Validação local

  • Backend: npm run build ok; npm test com 347 testes passando, 0 falhas (inclui o teste que confere o OpenAPI contra as rotas reais).
  • Frontend: npm run build ok; vitest com 278 testes passando.
  • ai-service: 14 testes novos do harness passando. Há 5 falhas em test_analyzer e test_safety_limits, que acontecem igual na main quando rodadas no macOS (no CI, em Linux, elas passam).

Depois do merge

🤖 Generated with Claude Code

@RafaMatesco
RafaMatesco merged commit fa86487 into main Oct 20, 2026
5 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