🔍

Code Reviews

11 code reviews depuis votre vault Obsidian

Reviews🔍
11
Sections📄
173
Points +
0
Améliorations⚠️
0
11 reviews

Repos analysés

/workspace/extra/obsidian → 4 commits (actif)
/workspace/project → 0 commits
/workspace/project/dashboard → 0 commits
📁

obsidian — Auto-sync appartements

Auteur: Clawdo (bot auto-sync)

Commits: 4 syncs automatiques

Fichiers: 32 nouvelles annonces + 4 mises à jour index.md

Sources: Bienici (12 annonces), SeLoger (20 annonces)

Volume: ~2 031 insertions au total

Détail des commits

sous-section
10:11 09/03 — 8 annonces (Paris 12e, Puteaux, Châtillon, Villejuif...)
20:12 09/03 — 10 annonces (Fontenay-le-Fleury, Vitry, Fontenay-s/Bois...)
22:14 09/03 — 10 annonces (Courbevoie ×3, Ivry ×2, Le Perreux, Nanterre...)
04:08 10/03 — 4 annonces (Asnières 40pts, Choisy 35pts, Paris 20e 40pts, Thiais 65pts)

Structure & cohérence

sous-section
Template YAML uniforme : tous les fichiers ont les mêmes champs frontmatter
Naming convention cohérente : YYYY-MM-DD-source-ville-type-score.md
Callouts Obsidian bien utilisés : [!summary], [!map], [!check], [!info], [!tip]
index.md mis à jour automatiquement à chaque sync
Système de scoring transparent : détail des points affiché dans chaque fiche

Qualité des données

sous-section
Deux destinations calculées (Montparnasse + Chelles) avec détail du trajet en transports
Liens Google Maps générés pour chaque trajet
DPE/GES inclus avec les valeurs numériques (kWh/m²/an, kgCO₂/m²/an)
Section Analyse avec points forts/faibles générés automatiquement

1. Données SeLoger moins riches que Bienici

sous-section

Les fichiers SeLoger font ~53-57 lignes contre ~70-73 pour Bienici. Les fiches Bienici incluent plus de données : ascenseur, type de chauffage, année de construction, cave. Les fiches SeLoger manquent souvent ces champs dans le bloc [!info].

Suggestion : Enrichir le scraper SeLoger pour extraire les diagnostics manquants, ou indiquer explicitement "données non disponibles" pour garder la cohérence du template.

2. Incohérence dans le nommage des fichiers

sous-section
Certains fichiers : ...courbevoie-b712-1p-55.md (code bâtiment inclus)
Autres : ...seloger-courbevoie-studio-60.md (type à la place de pièces)

Le format varie entre Np-score et type-score selon la source. Cela rend le tri par filename moins fiable.

Suggestion : Standardiser sur un seul format, ex. type-score ou Np-score pour toutes les sources.

3. Trajets très longs sans filtrage

sous-section

Plusieurs annonces ont trajet_chelles > 100 min (Asnières 100min, Thiais 102min). Ces propriétés sont quand même indexées et génèrent du bruit dans la recherche.

Suggestion : Envisager un filtre minimum ou un tag trajet_chelles: trop_loin pour les isoler visuellement dans Obsidian.

4. Score 35 indexé (Choisy-le-Roi)

sous-section

L'appartement Choisy avec DPE F et trajet >60min des deux côtés est quand même sauvegardé avec un score 35/100. Le pipeline ne filtre pas les scores bas.

Suggestion : Définir un seuil minimum (ex. 45/100) en-dessous duquel les annonces sont ignorées ou archivées séparément.

🔍

Résumé

Le pipeline d'auto-sync fonctionne bien et produit des fiches structurées et lisibles. Les principaux points d'attention sont la disparité de richesse des données entre Bienici et SeLoger, et l'absence de filtrage sur les scores bas ou trajets trop longs. Aucun bug critique détecté — le système remplit sa fonction correctement.