Pier-Jean Malandrino f6030bb2f1 fix(docs): resolve mkdocs strict mode build failure (#145 )

- Add all process docs to mkdocs.yml nav (15 missing pages)
- Replace relative links to files outside docs/ with absolute GitHub URLs
- Add docs/audit/ directory (templates + reports)
- Add validation config to allow unlisted audit reports as info-level

2026-04-10 15:11:29 +02:00

3 KiB

Raw Permalink Blame History

Rapport d'audit : Clean Architecture

Release : 0.3.1 Date : 2026-04-10 Auditeur : claude-code Derniere mise a jour : 2026-04-10 (re-audit post PR #133)

Score de compliance

Metrique	Valeur
Items conformes	13 / 13
Score	100 / 100
Ecarts CRITICAL	0
Ecarts MAJOR	0
Ecarts MINOR	0
Ecarts INFO	0

Ecarts resolus (PR #133 — fix/clean-architecture-audit)

[CRIT — RESOLU] Services importent directement les modules persistence (concretions)

Localisation : services/analysis_service.py, services/document_service.py
Resolution : AnalysisService et DocumentService recoivent analysis_repo et document_repo par injection dans leur constructeur. Plus aucun import direct de persistence.* au top-level des services.

[CRIT — RESOLU] Services importent directement `infra.settings`

Localisation : services/analysis_service.py, services/document_service.py
Resolution : Les valeurs de configuration sont encapsulees dans des dataclasses AnalysisConfig et DocumentConfig injectees dans les constructeurs. Plus aucun import direct de infra.settings dans les services.

[CRIT — RESOLU] Pas de protocol pour les repositories dans `domain/ports.py`

Localisation : domain/ports.py
Resolution : DocumentRepository et AnalysisRepository ajoutes comme Protocol dans domain/ports.py. Les services dependent maintenant de ces abstractions.

[MAJ — RESOLU] `document_service` est un module procedural, non une classe injectable

Resolution : document_service transforme en classe DocumentService avec injection du repository et de la config.

[MAJ — RESOLU] Logique metier dans le service (`_classify_error`, `_merge_results`, validation)

Resolution : _merge_results et _classify_error deplaces dans domain/. Validation de fichier encapsulee.

[MIN — RESOLU] `api/documents.py` importe `infra.settings`

Resolution : La valeur max_file_size_mb obtenue via le service, plus d'import direct dans la couche API.

Points positifs

Domain pur : domain/models.py, domain/value_objects.py et domain/ports.py n'importent aucune librairie externe. Les modeles sont des dataclasses pures avec des methodes de transition d'etat.
Ports complets : DocumentConverter, DocumentChunker, DocumentRepository et AnalysisRepository couvrent toutes les interactions externes.
Injection de dependances complete : services, repositories et config injectes via constructeur + Depends FastAPI.
Pydantic confine a la couche API : Tous les schemas Pydantic sont dans api/schemas.py. Le domaine n'utilise que des dataclasses.
Routes delegent aux services : Les endpoints se contentent de mapper les requetes/reponses et de deleguer.
Configuration centralisee dans infra/settings.py et propagee par injection.

Verdict : GO

Score 100/100 — 0 ecart critique. Toutes les non-conformites precedentes resolues via PR #133.

3 KiB Raw Permalink Blame History