From 97da6f9f710a2cd537762bf0b1dd5d1e6972232c Mon Sep 17 00:00:00 2001
From: escouade-bot <escouade-bot@lacompagniemaximus.com>
Date: Wed, 18 Mar 2026 04:01:43 -0400
Subject: [PATCH] fix: address reviewer feedback (#41)

- Correct CHANGELOG to reflect default type is expense, not all types
- Validate select onChange value against allowed CategoryTypeFilter values

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 CHANGELOG.fr.md                              | 2 +-
 CHANGELOG.md                                 | 2 +-
 src/components/reports/ReportFilterPanel.tsx | 6 +++++-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/CHANGELOG.fr.md b/CHANGELOG.fr.md
index 5daf118..df06301 100644
--- a/CHANGELOG.fr.md
+++ b/CHANGELOG.fr.md
@@ -3,7 +3,7 @@
 ## [Non publié]
 
 ### Modifié
-- Rapport Catégorie dans le temps : suppression du filtre codé en dur sur les dépenses, affiche maintenant tous les types de transactions par défaut (#41)
+- Rapport Catégorie dans le temps : suppression du filtre codé en dur sur les dépenses, ajout d'un sélecteur de type avec dépense par défaut (#41)
 - Rapport Catégorie dans le temps : ajout d'un filtre par type (dépense/revenu/transfert) dans le panneau de filtre à droite (#41)
 
 ## [0.6.6]
diff --git a/CHANGELOG.md b/CHANGELOG.md
index f764f12..e6ba290 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,7 +3,7 @@
 ## [Unreleased]
 
 ### Changed
-- Category Over Time report: removed hard-coded expense-only filter, now shows all transaction types by default (#41)
+- Category Over Time report: removed hard-coded expense-only filter, added type selector defaulting to expense (#41)
 - Category Over Time report: added type filter (expense/income/transfer) in the right filter panel (#41)
 
 ## [0.6.6]
diff --git a/src/components/reports/ReportFilterPanel.tsx b/src/components/reports/ReportFilterPanel.tsx
index 7f21a2a..3421185 100644
--- a/src/components/reports/ReportFilterPanel.tsx
+++ b/src/components/reports/ReportFilterPanel.tsx
@@ -72,7 +72,11 @@ export default function ReportFilterPanel({
           <div className="border-t border-[var(--border)] px-2 py-2">
             <select
               value={categoryType ?? ""}
-              onChange={(e) => onCategoryTypeChange((e.target.value || null) as CategoryTypeFilter)}
+              onChange={(e) => {
+                const v = e.target.value;
+                const valid: CategoryTypeFilter[] = ["expense", "income", "transfer"];
+                onCategoryTypeChange(valid.includes(v as CategoryTypeFilter) ? (v as CategoryTypeFilter) : null);
+              }}
               className="w-full px-2 py-1.5 text-xs rounded-lg border border-[var(--border)] bg-[var(--background)] text-[var(--foreground)] focus:outline-none focus:ring-1 focus:ring-[var(--primary)]"
             >
               <option value="">{t("reports.filters.allTypes")}</option>