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

@@ -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]

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]

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>