Skip to content

feat: séparer snippets, documents et preuves#24

Merged
ForgedEmir merged 1 commit into
developfrom
feat/4-separate-discovery-documents-evidence
Jul 23, 2026
Merged

feat: séparer snippets, documents et preuves#24
ForgedEmir merged 1 commit into
developfrom
feat/4-separate-discovery-documents-evidence

Conversation

@ForgedEmir

Copy link
Copy Markdown
Owner

Résumé

  • sépare les snippets de découverte, les documents récupérés et les preuves ;
  • conserve la provenance de chaque snippet : fournisseur, requête, rang et URL découverte ;
  • stocke l'URL canonique et le contenu sur le document ;
  • rattache chaque preuve à une tranche exacte du document ;
  • refuse les liens directs snippet-claim et les identifiants de preuve inexistants ;
  • migre la fixture historique vers le nouveau modèle.

Pourquoi

Un extrait de moteur de recherche est une piste de découverte, pas une preuve. Le claim ne devient supported qu'après création d'une preuve correspondant exactement à un passage du document récupéré.

La nouvelle fixture le démontre : le snippet affirme que l'API est illimitée, tandis que le document indique que l'usage API est facturé séparément. Seul le passage du document est lié au claim.

Vérifications

  • test rouge observé sur l'ancien modèle qui attendait source_title dans la preuve ;
  • test rouge observé pour une citation ne correspondant pas au passage du document ;
  • test rouge observé pour un lien direct vers un snippet ;
  • test rouge observé pour un identifiant de preuve inexistant ;
  • suite locale : 18 passed in 0.05s ;
  • installation et suite dans un worktree propre : 18 passed in 0.08s ;
  • git diff --check réussi.

Ticket

Implémente #4. Le ticket sera fermé manuellement après fusion dans develop.

@ForgedEmir
ForgedEmir merged commit b511e7a into develop Jul 23, 2026
1 check passed
@ForgedEmir
ForgedEmir deleted the feat/4-separate-discovery-documents-evidence branch July 23, 2026 19:03
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.

1 participant