fix: create pg_textsearch in public schema before age setup in Docker…

[Kaiohz]

…file.db

[Kaiohz]

Code Review - PR #24

📊 Score: 8/10

✅ Ce qui est bien

1. Fix correct et ciblé
Le problème identifié est réel : créer pg_textsearch après SET search_path = ag_catalog... aurait créé l'extension dans le schema ag_catalog au lieu de public. En la déplaçant avant le changement de search_path, l'extension est maintenant créée dans le schema par défaut (public), ce qui est le comportement attendu.

2. Changement minimal
Un seul fichier, une seule ligne déplacée. Pas de changements superflus, c'est clean.

3. Conventional commit
Le message fix: create pg_textsearch in public schema before age setup... suit bien le format conventional commits.

---

🔧 Suggestions d'amélioration

1. Ajouter un commentaire explicatif
Un petit commentaire SQL éviterait qu'une future modification ne recrée le bug :

-- Must be created BEFORE changing search_path, otherwise ends up in ag_catalog
CREATE EXTENSION IF NOT EXISTS pg_textsearch;

2. Ajouter un newline final
Le fichier termine sans newline, ce qui génère un warning lint courant.

3. (Optionnel) Test de validation
Pourrait être utile d'avoir un test qui vérifie que les extensions sont dans les bons schemas.

---

🟢 Verdict

Le fix est correct et résout le problème d'ordonnancement des extensions PostgreSQL. C'est un changement de qualité, simple et efficace.

Recommendation: APPROVE avec les suggestions mineures ci-dessus (commentaire + newline).