Files
mabibli/MaBibli.Api/Services/Isbn/IsbnLookupService.cs
T
mathieuandClaude Opus 5 5cf566bc33 Traiter les retours d'usage du 2026-08-18 (2ᵉ série)
Six lots, issus d'IDEES.md. Les décisions actées sont reportées dans
CLAUDE.md, et les entrées traitées retirées d'IDEES.md.

1. Douchette USB, ISSN et enchaînement du scan
   - Champ ISBN focalisé à l'ouverture : c'est tout ce qu'exige une
     douchette USB, qui se présente au système comme un clavier. Remède
     au scan caméra, qui rate sur la webcam d'un PC (optique, pas
     décodage).
   - Un EAN en 977 est un périodique : il porte un ISSN, donc un titre de
     revue. Intercepté AVANT la validation ISBN (c'est un EAN-13
     parfaitement valide), l'ISSN est déduit du code puis nommé via
     bib.issn. Auparavant la cascade s'exécutait en entier pour finir sur
     « aucun résultat ».
   - L'enchaînement après scan existait déjà mais était invisible : une
     étape « Recherche » affiche désormais le code interrogé.

2. ISBN affichés avec des tirets
   Tables extraites du RangeMessage.xml officiel — plusieurs tranches ne
   sont pas celles qu'on suppose. Le francophone est découpé en entier,
   ailleurs on s'arrête au groupe et à la clé : aucune coupure fausse.
   Corrige au passage l'export CSV, où un ISBN nu était lu comme un
   nombre par Excel.

3. Hors-ligne : la liste d'envies
   ListerSouhaitsAsync était le seul point de lecture hors du dispositif
   hors-ligne, d'où le « 404 Not Found » brut à l'écran. Cinquième
   instantané, écritures refusées, plus aucun message HTTP. Même défaut
   corrigé sur la bibliographie.

4. Navigation par onglets
   Catalogue / Auteurs / Prêts / Envies dans MainLayout ; les barres
   d'actions ne portent plus que des actions. Filtres repliés derrière un
   bouton compteur, ligne « format » masquée quand le fonds n'a qu'un
   format. Une seule entrée d'ajout, désactivée hors-ligne — pas masquée.

5. Liste d'envies : ordre, recherche, couvertures
   Migration RangDesEnvies. Le remplissage reconduit l'ordre affiché
   jusqu'ici : sans lui, les listes existantes se seraient réordonnées
   toutes seules. Réordonnancement par flèches et glisser-déposer (le
   drag & drop HTML5 ne marche pas au doigt). Ajout dans son propre
   écran, avec recherche par titre (bib.title) et couvertures enfin
   alimentées.

6. Bibliographie : une source muette n'est pas une liste vide
   L'écran affichait « BnF injoignable » PUIS « la BnF ne connaît aucun
   livre de cet auteur » — la seconde phrase étant fausse. Les deux cas
   s'excluent désormais, et un bouton Réessayer est offert.

   Vérifié en exécution : le diagnostic d'IDEES.md était faux sur un
   point. Robert A. Harper a bien 7 œuvres à la BnF (85 notices
   annoncées) ; c'était le même délai dépassé observé deux fois, pris
   pour deux causes distinctes.

380 tests, dont un qui applique réellement la migration (EnsureCreated
n'en joue aucune) et un qui verrouille les messages atteignant
l'utilisateur.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-19 00:14:01 +02:00

129 lines
5.0 KiB
C#

using MaBibli.Shared.Dtos;
using MaBibli.Shared.Isbn;
namespace MaBibli.Api.Services.Isbn;
public interface IIsbnLookupService
{
/// <summary>
/// Cherche toutes les notices correspondant à un ISBN.
/// </summary>
/// <returns><c>null</c> si l'ISBN fourni n'est pas un ISBN valide.</returns>
Task<ResultatLookupIsbn?> RechercherAsync(string isbnBrut, CancellationToken ct = default);
}
/// <summary>
/// Cascade de lookup ISBN : <b>BnF d'abord, OpenLibrary ensuite</b>.
/// </summary>
/// <remarks>
/// L'ordre est dicté par CLAUDE.md : la collection est majoritairement francophone et le dépôt
/// légal français donne à la BnF la meilleure couverture possible sur ce fonds, là où OpenLibrary
/// est lacunaire. OpenLibrary n'est interrogée que si la BnF ne renvoie rien sous
/// <b>aucune</b> des deux formes d'ISBN.
/// </remarks>
public sealed class IsbnLookupService(
IBnfClient bnf,
IOpenLibraryClient openLibrary,
ILogger<IsbnLookupService> logger) : IIsbnLookupService
{
public async Task<ResultatLookupIsbn?> RechercherAsync(string isbnBrut, CancellationToken ct = default)
{
var isbn = IsbnUtils.Normaliser(isbnBrut);
// Un code en 977 est un PÉRIODIQUE : il porte un ISSN, donc un titre de revue, et
// aucune recherche bib.isbn ne peut aboutir. Sans cette interception la cascade
// s'exécutait en entier pour répondre « aucun résultat » — ce qui envoie l'utilisateur
// chercher un défaut là où il n'y en a pas. On sort AVANT la validation ISBN parce
// qu'un code de périodique est un EAN-13 parfaitement valide : seul son préfixe le trahit.
if (CodePeriodique.EstCodePeriodique(isbn))
{
return await DecrirePeriodiqueAsync(isbn!, ct);
}
if (!IsbnUtils.EstValide(isbn))
{
return null;
}
// La couverture ne vient jamais de la BnF : son Dublin Core n'en fournit aucune.
// On construit l'URL OpenLibrary quelle que soit la source des métadonnées.
var couverture = IsbnUtils.UrlCouverture(isbn!);
IsbnUtils.TryConvertirEnIsbn10(isbn, out var isbn10);
var avertissements = new List<string>();
// 1. BnF avec l'ISBN tel que scanné.
var (candidats, avertissement) = await bnf.RechercherAsync(isbn!, couverture, ct);
Ajouter(avertissements, avertissement);
// 2. BnF avec l'ISBN-10 converti. Indispensable : la BnF indexe l'ISBN tel qu'imprimé,
// et les ouvrages d'avant 2007 ne portent qu'un ISBN-10.
if (candidats.Count == 0 && isbn10 is not null)
{
var (candidats10, avertissement10) = await bnf.RechercherAsync(isbn10, couverture, ct);
Ajouter(avertissements, avertissement10);
candidats = candidats10;
}
// 3. OpenLibrary en second rideau, seulement si la BnF n'a rien donné du tout.
if (candidats.Count == 0)
{
var (candidatsOl, avertissementOl) = await openLibrary.RechercherAsync(isbn!, couverture, ct);
Ajouter(avertissements, avertissementOl);
candidats = candidatsOl;
}
logger.LogInformation(
"Lookup ISBN {Isbn} : {Nombre} candidat(s), {Avertissements} avertissement(s)",
isbn, candidats.Count, avertissements.Count);
return new ResultatLookupIsbn
{
IsbnDemande = isbn!,
Isbn10 = isbn10,
Candidats = candidats,
Avertissements = avertissements,
};
}
/// <summary>
/// Nomme le périodique scanné, sans rien chercher du côté des livres.
/// </summary>
/// <remarks>
/// Le résultat ne porte <b>jamais</b> de candidat : le modèle de données ne gère pas les
/// périodiques (ni numéro, ni date de parution), et cataloguer douze numéros d'une même
/// revue créerait douze fiches identiques. L'objectif est uniquement de dire à l'utilisateur
/// ce qu'il tient en main. Une BnF muette ou injoignable n'est pas un échec : on connaît
/// l'ISSN de toute façon, puisqu'il se déduit du code-barres.
/// </remarks>
private async Task<ResultatLookupIsbn> DecrirePeriodiqueAsync(string ean, CancellationToken ct)
{
var issn = CodePeriodique.IssnDepuis(ean)!;
var (periodique, avertissement) = await bnf.RechercherPeriodiqueAsync(issn, ct);
var avertissements = new List<string>();
Ajouter(avertissements, avertissement);
logger.LogInformation(
"Code {Ean} : périodique ISSN {Issn} ({Titre})",
ean, issn, periodique?.Titre ?? "titre inconnu");
return new ResultatLookupIsbn
{
IsbnDemande = ean,
Candidats = [],
Avertissements = avertissements,
Periodique = periodique ?? new PeriodiqueDetecte { Issn = issn },
};
}
private static void Ajouter(List<string> avertissements, string? message)
{
if (!string.IsNullOrEmpty(message))
{
avertissements.Add(message);
}
}
}