Le picker lit /api/ai/status une seule fois, à sa construction, et le panneau de l'assistant est un singleton monté pour toute la session : ajouter ou supprimer une clé API dans la configuration du projet laissait la liste des fournisseurs figée jusqu'à un rechargement de page. - ai.js : nouveau refreshAIPickers() qui reconstruit chaque picker monté dans son emplacement .ai-picker-slot (constante PICKER_SLOT_CLASS) ; le slot est conservé même sans fournisseur configuré, donc le premier fournisseur ajouté s'y monte aussi ; une sélection persistée dont le fournisseur n'est plus configuré est purgée de obsigate_ai_picker (retour au défaut, plus de modèle fantôme dans le déclencheur). - bookslm.js : l'emplacement .bookslm-picker-host porte la classe ai-picker-slot et reste dans la barre (replaceChildren au lieu de replaceWith). - config.js : refreshAIPickers() après saveAIKeys() et deleteAIKey(). - tests : +4 tests JSDOM (ajout/retrait dans la barre, premier montage dans un slot vide, purge de la sélection orpheline, câblage save/delete). Vérifié : tests frontend 66/66 (IA) + 9 suites JSDOM, validate-imports 36 modules, pytest 963 passed / 6 skipped, ruff 0, et contrôle navigateur (Playwright) sur l'instance de test — ajout de nvidia visible sans rechargement, retrait effectif + sélection réinitialisée.
120 lines
8.4 KiB
Markdown
120 lines
8.4 KiB
Markdown
# #82 — Assistant IA — Section « Fournisseur & modèle » compacte
|
|
|
|
> **Statut :** ✅ Livré (en attente de validation utilisateur)
|
|
> **Effort :** 1 jour | **Impact :** 🟢
|
|
> **Références :** [Roadmap](../ROADMAP.md) · [Assistant IA #81](./ai-assistant-commands.md) · [Changelog](../../CHANGELOG.md)
|
|
|
|
- **Description :** rendre la barre de sélection du modèle de l'assistant IA **plus discrète** tout
|
|
en conservant les informations utiles :
|
|
1. **Capacités via bulle** — la liste complète des types d'endpoints (Chat, Embeddings, Rerank,
|
|
Images, Video, Audio Speech, Audio Transcriptions, Vision) n'est plus affichée en permanence.
|
|
Un bouton **ⓘ** révèle une bulle d'information au **survol**, au **clic** ou par **appui long**
|
|
(mobile).
|
|
2. **Recherche de modèle** — le `<select>` natif est remplacé par une liste déroulante compacte
|
|
dotée d'un **champ de recherche** qui filtre les modèles en tapant (insensible à la casse et
|
|
aux accents).
|
|
3. **Compatibilité** — un `<select class="ai-picker-model">` masqué reste présent (source de
|
|
vérité) pour les commandes admin `/model` et l'état persisté `obsigate_ai_picker`.
|
|
|
|
## A. Picker partagé — ✅ livré
|
|
- [x] `frontend/js/ai.js` : `buildAIPickerUI()` réécrit (provider compact + combobox modèle + bulle
|
|
capacités). Le picker reste partagé par l'assistant (`bookslm.js`) et la barre d'édition IA.
|
|
- [x] `renderCapabilityList()` conservé et affiché dans la bulle (le panneau de configuration
|
|
`#cfg-ai-default-model-caps` continue de l'utiliser directement).
|
|
- [x] Fermeture des popovers au clic extérieur via un unique listener délégué (pas de fuite).
|
|
- [x] i18n FR/EN (`ai.model_search`, `ai.model_info`).
|
|
|
|
## B. Correctifs associés — ✅ livré
|
|
- [x] **BUG-007** — détection `/` et `@` Unicode + filtrage insensible aux accents + navigation
|
|
clavier ↑/↓ fiabilisée (jeton de séquence, `scrollIntoView`).
|
|
- [x] **BUG-008** — contexte ad-hoc `@` pris en compte en mode Général côté backend
|
|
(`backend/bookslm_routes.py`) et recherche de mention sur `vault=all` sans vault courant.
|
|
|
|
## C. Tests — ✅ livré
|
|
- [x] `tests/frontend/ai.test.mjs` : filtrage accentué, mentions accentuées, navigation ↑/↓,
|
|
recherche de modèle + bulle de capacités, plafonnement des longues listes (OpenRouter).
|
|
- [x] `tests/test_bookslm.py` : contexte ad-hoc en mode Général (`/chat`).
|
|
|
|
## D. Durcissement anti-cache (BUG-009) — ✅ livré
|
|
- [x] Locales chargées avec `cache: 'no-store'` (`frontend/js/i18n.js`) : plus de clés brutes
|
|
affichées à cause d'un `fr.json` obsolète en cache HTTP.
|
|
- [x] `SW_VERSION` incrémenté (`frontend/sw.js`) pour purger les caches du service worker.
|
|
- [x] Styles critiques du picker appliqués **en ligne** (popover absolu, liste en colonne, options
|
|
`display:block`) pour rester corrects même si `style.css` est servi depuis un cache.
|
|
- [x] Rendu plafonné à 200 modèles (`MODEL_RENDER_LIMIT`) + indicateur `ai.model_more`.
|
|
|
|
## E. Lisibilité & contexte `@` (BUG-010, BUG-011) — ✅ livré
|
|
- [x] Popover modèle **aligné à droite** du déclencheur (`right: 0`), largeur 340 px bornée à
|
|
`100vw - 24px` : ne dépasse plus le bord de la sidebar ancrée à droite (BUG-011).
|
|
- [x] Noms de modèles sur **plusieurs lignes** (`overflow-wrap: anywhere`, police 0,78 rem) avec
|
|
attribut `title` complet.
|
|
- [x] La sélection `@` **capture le vault** renvoyé par `/api/tree-search` ; `_contextVault()`
|
|
propage ce vault aux requêtes `/context` et `/chat` et les recherches suivantes restent dans
|
|
ce vault (BUG-010, mode Général).
|
|
|
|
## F. Barre latérale épurée & suivi visuel (BUG-012, #82) — ✅ livré
|
|
- [x] Section « Fournisseur & modèle » sans intitulés (ni titre de section, ni « Fournisseur : »).
|
|
- [x] Description du contexte Général retirée de l'affichage (bandeau de statut masqué) et
|
|
exposée via `title` au survol de l'en-tête.
|
|
- [x] Placeholder « Posez une question sur ces documents… » retiré ; bouton « Envoyer » remplacé
|
|
par l'emoji compact ✈️.
|
|
- [x] **Indicateur d'activité** (`.bookslm-activity`) affichant le workflow : envoi, réception,
|
|
outil en cours, attente de confirmation, terminé, échec (`ai.activity_*`).
|
|
- [x] `@` : le menu est toujours rendu et `_mentionVault()` retombe sur le vault de contexte puis
|
|
le premier vault disponible (BUG-012).
|
|
|
|
## G. Robustesse menus & liens (BUG-013, BUG-014) — ✅ livré
|
|
- [x] Navigation clavier ↑/↓ des menus `/` et `@` gérée au niveau du **panneau en phase de
|
|
capture** : fonctionne quel que soit l'élément focalisé (BUG-014).
|
|
- [x] Les liens fichiers/répertoires des réponses utilisent `_activeVault()` (vault de contexte →
|
|
vault sélectionné → premier vault), ce qui supprime l'erreur « Aucun vault actif » en mode
|
|
Général (BUG-013).
|
|
- [x] Purge des caches élargie : `SW_VERSION` v6 + migration qui supprime tous les caches
|
|
`obsigate-*` (clé `obsigate-sw-migration` → `v3`).
|
|
- [x] Indice clavier sous la zone de saisie retiré pour un composeur plus épuré (#82).
|
|
- [x] Sélection clavier des menus `/` et `@` rendue visible : état actif en `--bg-hover` +
|
|
barre d'accent à gauche (`--surface2` étant identique à `--bg-primary` en thème sombre,
|
|
la sélection était invisible) — BUG-015.
|
|
|
|
## H. Performance `@`, sélecteurs agrandis & BooksLM racine (#82) — ✅ livré
|
|
- [x] Nouvel endpoint `GET /api/vault/{vault}/paths` (`backend/services/search.py: list_paths`,
|
|
`backend/main.py`) : index des chemins plat et plafonné (5000 par défaut).
|
|
- [x] Frontend : `_ensurePaths()` précharge cette liste **une fois par vault** (au `openContext`),
|
|
puis `_showMentionMenu` filtre **côté client** → affichage instantané des fichiers/
|
|
répertoires, sans requête par frappe (repli serveur si le cache est vide).
|
|
- [x] Sélecteurs Fournisseur/Modèle agrandis pour s'aligner sur les autres contrôles
|
|
(police 0,8 rem, hauteur 34 px, rayon 6 px).
|
|
- [x] Menu contextuel de la **racine d'une vault** : ajout de « 🧠 BooksLM » (`frontend/js/ui.js`).
|
|
|
|
## J. Alignement à droite & ordre des sélecteurs (#82) — ✅ livré
|
|
- [x] Le groupe de sélection Fournisseur/Modèle est **aligné sur le bord droit** de la barre
|
|
(`frontend/style.css` : `.bookslm-toolbar { justify-content: flex-end; }`).
|
|
- [x] Ordre visuel : **capacité du modèle (ⓘ) → menu fournisseurs → menu des modèles**
|
|
(`frontend/js/ai.js`, ordre d'ajout dans `_buildPickerUI()`).
|
|
- [x] La bulle de capacités s'ouvre vers la droite (`left: 0`, inline + CSS) pour rester dans la
|
|
barre latérale désormais que le bouton ⓘ est en tête de groupe.
|
|
- [x] Test JSDOM : l'ordre des trois contrôles visibles est vérifié (`tests/frontend/ai.test.mjs`).
|
|
|
|
## K. Synchronisation de la liste des fournisseurs (BUG-043) — ✅ livré
|
|
- [x] `frontend/js/ai.js` : nouveau `refreshAIPickers()` — reconstruit chaque picker
|
|
monté dans son emplacement `.ai-picker-slot` (constante `PICKER_SLOT_CLASS`).
|
|
Le slot est conservé même quand aucun picker ne peut être construit (aucun
|
|
fournisseur configuré), donc le **premier** fournisseur ajouté s'y monte aussi.
|
|
- [x] `frontend/js/bookslm.js` : l'emplacement du picker (`span.bookslm-picker-host`)
|
|
porte la classe `ai-picker-slot` et **reste dans la barre** — le picker est monté
|
|
à l'intérieur au lieu de remplacer le host (`replaceChildren`).
|
|
- [x] `frontend/js/config.js` : `refreshAIPickers()` appelé après l'enregistrement
|
|
(`saveAIKeys()`) et la suppression (`deleteAIKey()`) d'une clé API.
|
|
- [x] Une sélection persistée dont le fournisseur n'est plus configuré est purgée de
|
|
`obsigate_ai_picker` (fournisseur **et** modèle effacés → retour au défaut, plus
|
|
de nom de modèle fantôme dans le déclencheur).
|
|
- [x] Tests JSDOM : ajout/retrait d'un fournisseur dans la barre de l'assistant,
|
|
montage du premier fournisseur dans un slot vide, purge de la sélection orpheline,
|
|
et garde-fou sur le câblage `saveAIKeys`/`deleteAIKey` (`tests/frontend/ai.test.mjs`).
|
|
|
|
## I. Points d'attention
|
|
- La table de capacités reste **statique** (`backend/model_capabilities.py`) : un modèle inconnu
|
|
retombe sur le défaut du fournisseur.
|
|
- Le select masqué est conservé uniquement pour la rétro-compatibilité ; ne pas le supprimer sans
|
|
migrer `_cmdSwitchModel` / `_applyPickerSelection` dans `frontend/js/bookslm.js`.
|