Skip to content

fix(extract-document-markdown): exiger la description du contenu grap… - #492

Merged
Manolbh merged 1 commit into
mainfrom
fix/extract-document-markdown-visual-content
Aug 2, 2026
Merged

fix(extract-document-markdown): exiger la description du contenu grap…#492
Manolbh merged 1 commit into
mainfrom
fix/extract-document-markdown-visual-content

Conversation

@Manolbh

@Manolbh Manolbh commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Contexte

Constaté sur la transcription réelle d'un cours de mathématiques : en fin de page, trois intitulés
(« Fonctions continues », « Fonctions discontinues », « Fonctions discrètes ») n'ont produit aucun
contenu. Ce qui distinguait chaque cas était entièrement porté par les courbes, qui ont disparu de
la transcription — alors que PRESERVATION_INSTRUCTIONS protège déjà les tableaux avec une dizaine
de phrases explicites (interdiction d'omettre, obligation de transcrire même si complexe, etc.). Le
contenu graphique, lui, n'avait droit qu'à une clause facultative ("décris brièvement... si elle est
visible").

Changements

  • Étend PRESERVATION_INSTRUCTIONS (supabase/functions/extract-document-markdown/index.ts) d'un
    paragraphe qui met le contenu graphique (courbe, graphe, diagramme, carte, frise, montage, schéma
    annoté) au même niveau d'obligation que les tableaux, dans toutes les disciplines :
    • un intitulé suivi d'une figure ne doit jamais être transcrit seul ;
    • une série de figures comparatives doit décrire ce qui distingue chaque cas ;
    • la description reste bornée à ce que la figure montre — l'interdiction d'inventer prime ;
    • marqueur unique et reconnaissable **Schéma visible :**, placé juste après l'intitulé, pour
      distinguer sans ambiguïté une description produite par le modèle du texte du professeur ;
    • exclusion explicite des éléments purement décoratifs (logo, filet, photo d'illustration, charte
      graphique) pour ne pas faire gonfler le coût d'extraction.
  • Aucune consigne existante modifiée ou retirée (tableaux, notes de présentateur, numérotation des
    pages, interdiction d'inventer) — ajout pur en fin de bloc.

Scope

Touche uniquement supabase/functions/extract-document-markdown/index.ts et ses tests. Aucun
déploiement effectué — extract-document-markdown est une Edge Function en production, l'effet
réel ne sera visible qu'après déploiement.

Test plan

  • Nouveau fichier extract-document-markdown-visual-content-source.test.mjs (7 tests) qui
    reconstruit le texte réel de PRESERVATION_INSTRUCTIONS (les segments de chaîne concaténés
    dans le source) et vérifie : l'exigence non facultative, la règle sur l'intitulé isolé, la
    règle sur les séries comparatives, le marqueur unique, l'exclusion du décoratif, et la
    non-régression des consignes existantes.
  • npm test : 1904 → 1911 tests, 0 échec.
  • Après déploiement : relancer l'extraction sur le document source ayant révélé le défaut et
    vérifier la page des trois séries de graphiques (présence de **Schéma visible :** décrivant
    ce qui distingue chaque courbe, sans déduction de propriété absente du support).

🤖 Generated with Claude Code

https://claude.ai/code/session_018KTokszFL97BnVfrF2AR4q

…hique

Sur une transcription réelle de cours de maths, trois séries de graphiques
comparatifs (fonctions continues/discontinues/discrètes) ont disparu sans
laisser aucun texte citable, alors que les tableaux étaient déjà protégés
par une dizaine de consignes explicites. Étend PRESERVATION_INSTRUCTIONS
pour que le contenu porté par une figure (courbe, schéma, carte, frise...)
soit soumis à la même obligation que les tableaux, dans toutes les
disciplines, via un marqueur unique et reconnaissable (**Schéma visible :**).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018KTokszFL97BnVfrF2AR4q
@Manolbh
Manolbh merged commit 7988529 into main Aug 2, 2026
2 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.

1 participant