Normalize domains at read time in getStoredDomains

[Copilot]

Domains injected directly into browser.storage.sync (e.g. via devtools or an external process) could contain leading/trailing whitespace or mixed case, causing incorrect display in the popup and broken remove operations.

Changes

• popup.js — getStoredDomains: normalize every entry on retrieval with a filter + trim + toLowerCase chain before returning the array.

// Before
return Array.isArray(stored[STORAGE_KEY]) ? stored[STORAGE_KEY] : [];

// After
return Array.isArray(stored[STORAGE_KEY])
  ? stored[STORAGE_KEY]
      .filter((domain) => typeof domain === "string")
      .map((domain) => domain.trim().toLowerCase())
  : [];

The typeof guard defensively drops non-string entries that would otherwise throw at .trim().

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Tolérance aux espaces accidentels dans le stockage des exceptions</issue_title>
<issue_description>## Contexte
L'extension synchronise les domaines en liste blanche via browser.storage.sync. Le background script lit ces domaines pour peupler un Set utilisé pour valider les requêtes.

Problème

Bien que l'ajout via le popup gère correctement la normalisation (en minuscules), si un utilisateur ou un autre processus modifie manuellement le stockage synchronisé en y injectant des espaces (ex: " example.com "), le popup affichera ces espaces de manière erronée et la comparaison dans background.js pourrait échouer malgré le .trim() présent dans updateExceptionDomains, en cas de comparaison directe depuis le stockage sans rechargement.

Scénario de Reproduction

1. Simuler l'ajout d'un domaine mal formaté dans browser.storage.sync via la console (ex: browser.storage.sync.set({ exceptionDomains: [" test.com "] })).
2. Ouvrir le popup. Le domaine s'affiche avec des espaces.
3. L'appui sur le bouton de suppression de ce domaine spécifique pourrait mal lier la chaîne s'il n'est pas proprement assaini.

Comportement Attendu

Les données issues de l'espace de stockage externe doivent être systématiquement normalisées (minuscules et sans espaces superflus) dès leur récupération par la fonction getStoredDomains du popup.

Pistes Techniques

• Fichier concerné : popup.js.
• Dans la fonction getStoredDomains, ajouter un .map(domain => domain.trim().toLowerCase()) avant de retourner le tableau des domaines.

Environnement

Toutes les plateformes.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Tolérance aux espaces accidentels dans le stockage des exceptions #62

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

Normalizes exception domain entries when reading them from browser.storage.sync in the popup, improving robustness against manually-injected/externally-modified storage values (e.g., whitespace and mixed case) that could otherwise display incorrectly or break removal.

Changes:

• Normalize stored domain entries at read time by filtering to strings and applying trim().toLowerCase() in getStoredDomains.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

getStoredDomains now trims/lowers strings but it can still return empty strings when a stored entry is whitespace-only (e.g. " "). That will render a blank list item and may lead to confusing remove behavior. Consider filtering out entries where domain.trim() is falsy (similar to background.js’s updateExceptionDomains).

Suggested change

	.filter((domain) => typeof domain === "string")
	.filter((domain) => typeof domain === "string" && domain.trim())