Skip to content

fix(bibtex): ne plus rejeter silencieusement les entrées sans author (#32)#50

Merged
cliodeck merged 1 commit into
mainfrom
fix/32-bibtex-editor-fallback
Jul 24, 2026
Merged

fix(bibtex): ne plus rejeter silencieusement les entrées sans author (#32)#50
cliodeck merged 1 commit into
mainfrom
fix/32-bibtex-editor-fallback

Conversation

@inactinique

Copy link
Copy Markdown
Collaborator

Résumé

BibTeXParser.createCitation exigeait fields.author : les volumes dirigés (editor sans author) et les œuvres anonymes étaient rejetés avec un simple console.warn, sans aucun retour utilisateur ni compteur — perte silencieuse à l'import, reconnue par un test skippé (BibTeXParser.test.ts:158-161).

Correctifs

  • author = fields.author || fields.editor || '' — repli standard BibTeX pour les ouvrages dirigés ; seule l'absence de titre rejette encore une entrée.
  • displayString retombe sur le titre quand author est vide (le test skippé décrivait exactement ce comportement — réactivé).
  • Nouveau parser.lastRejected (réinitialisé à chaque parse()) : les appelants peuvent journaliser ou afficher les entrées écartées au lieu d'un warn console perdu. Le branchement d'une alerte UI sur ce compteur peut suivre dans une PR dédiée.

Vérification

  • npx vitest run backend/__tests__/BibTeXParser.test.ts : 13/13 verts (dont le test réactivé + 2 nouveaux).
  • npm run build vert.

Closes #32

🤖 Generated with Claude Code

…32)

Le parseur exigeait fields.author : un volume dirigé (editor sans
author) ou une œuvre anonyme (title seul) était rejeté avec un simple
console.warn, invisible pour l'utilisateur — perte silencieuse à
l'import, admise par un test skippé.

- author retombe sur editor, puis sur vide ; seule l'absence de title
  rejette encore ;
- displayString retombe sur le titre quand author est vide ;
- les rejets sont suivis (parser.lastRejected, réinitialisé à chaque
  parse) pour que les appelants puissent les journaliser/afficher ;
- test skippé réactivé + couverture editor et lastRejected.

Closes #32

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cliodeck
cliodeck merged commit 652d621 into main Jul 24, 2026
2 checks passed
@cliodeck
cliodeck deleted the fix/32-bibtex-editor-fallback branch July 24, 2026 16:02
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.

BibTeX import silently drops author-less entries (no editor fallback, no user feedback)

2 participants