Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
634d10cdd4 | ||
|
|
0d4f43a8bf | ||
|
|
4231f2e929 |
@@ -47,6 +47,7 @@ jobs:
|
||||
node plugins.test.mjs
|
||||
node ai.test.mjs
|
||||
node ai-sidebar.test.mjs
|
||||
node sidebar-filters.test.mjs
|
||||
node sw.test.mjs
|
||||
node collab.test.mjs
|
||||
node mobile-editor.test.mjs
|
||||
@@ -62,6 +63,7 @@ jobs:
|
||||
node plugins.test.mjs
|
||||
node ai.test.mjs
|
||||
node ai-sidebar.test.mjs
|
||||
node sidebar-filters.test.mjs
|
||||
node sw.test.mjs
|
||||
node collab.test.mjs
|
||||
node mobile-editor.test.mjs
|
||||
|
||||
+50
-1
@@ -6,7 +6,7 @@ Format basé sur [Keep a Changelog](https://keepachangelog.com/fr/1.1.0/),
|
||||
et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
|
||||
|
||||
> **En cours de développement** : les changements à venir sont listés dans la section
|
||||
> [Unreleased](#unreleased). La dernière version livrée est **2.5.2**.
|
||||
> [Unreleased](#unreleased). La dernière version livrée est **2.6.2**.
|
||||
|
||||
---
|
||||
|
||||
@@ -14,6 +14,55 @@ et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
|
||||
|
||||
---
|
||||
|
||||
## [2.6.2] — 2026-09-16
|
||||
|
||||
### Corrigé
|
||||
|
||||
- **BUG-050 — Assistant IA : échec de la création d'un sous-dossier contenant un fichier** :
|
||||
lors d'une pause de confirmation, la boucle d'agent ne renvoyait le résultat que du seul
|
||||
appel confirmé alors que le message assistant annonçait tous les appels d'outils du tour —
|
||||
la conversation devenait invalide (identifiant `tool_call_id` sans réponse) et la reprise
|
||||
échouait. Les appels non atteints reçoivent désormais un résultat `deferred` explicite
|
||||
(`backend/agent/loop.py`). En complément, `create_directory` est idempotent côté outil IA
|
||||
(succès si le dossier existe déjà, `backend/services/mutations.py`) et les consignes
|
||||
(`create_file` crée les dossiers parents) invitent le modèle à un seul appel avec un chemin
|
||||
imbriqué (`backend/bookslm.py`, `backend/tools/service.py`). Tests : `tests/test_agent_loop.py`
|
||||
(+1), `tests/test_tools_mutations.py` (+1), `tests/test_api_main.py` (+1).
|
||||
|
||||
---
|
||||
|
||||
## [2.6.1] — 2026-09-16
|
||||
|
||||
---
|
||||
|
||||
## [2.6.0] — 2026-09-16
|
||||
|
||||
### Ajouté
|
||||
|
||||
- **Barre de filtrage de la sidebar sur « Récents » et « Sauvegardes » (#99)** : la barre
|
||||
de recherche de la sidebar agit désormais sur les onglets **Récents**
|
||||
(`filterRecentFiles`, filtrage titre/chemin/vault/aperçu/tags) et **Sauvegardes**
|
||||
(`filterSavedSearches`, cumulable avec les pills Tous/Recherches/Répertoires) —
|
||||
insensible à la casse et aux accents, avec message d'absence de résultat et
|
||||
placeholders dédiés (`sidebar.filter_recent`, `sidebar.filter_saved`). Le routage de
|
||||
`initSidebarFilter` est unifié (`routeFilter`/`routeClear`) pour couvrir les cinq
|
||||
onglets. Fiche : [docs/features/sidebar-filters.md](./docs/features/sidebar-filters.md).
|
||||
- **Assistant IA — Deep Research en pastille (#100)** : « Deep Research » ajoute
|
||||
désormais une **pastille** (comme les skills) au lieu d'écrire la directive dans la
|
||||
zone de saisie ; le mode Agent est activé et la directive est injectée au moment de
|
||||
l'envoi, sans polluer le message affiché.
|
||||
|
||||
### Corrigé
|
||||
|
||||
- **BUG-049 — icône du bouton « + » de l'assistant invisible** : la règle générique
|
||||
`.bookslm-input-area button` écrasait `.bookslm-btn-plus` (`padding: 8px 16px` sur une
|
||||
largeur de 32 px ⇒ largeur de contenu nulle ⇒ SVG à 0 px). Sélecteur porté à
|
||||
`.bookslm-input-area button.bookslm-btn-plus`, l'icône `plus` est de nouveau visible
|
||||
(vérifié en navigateur : SVG 0 px → 18 px). Tests : `tests/frontend/ai.test.mjs` (+1),
|
||||
`tests/frontend/sidebar-filters.test.mjs` (nouveau, 8 tests).
|
||||
|
||||
---
|
||||
|
||||
## [2.5.2] — 2026-09-16
|
||||
|
||||
---
|
||||
|
||||
+3
-3
@@ -4,7 +4,7 @@
|
||||
|
||||
**Porte d'entrée web ultra-léger pour vos vaults Obsidian** — Accédez, naviguez et recherchez dans toutes vos notes Obsidian depuis n'importe quel appareil via une interface web moderne et responsive.
|
||||
|
||||
[]()
|
||||
[]()
|
||||
[](https://opensource.org/licenses/MIT)
|
||||
[](https://www.docker.com/)
|
||||
[](https://www.python.org/)
|
||||
@@ -916,8 +916,8 @@ Ce projet est sous licence **MIT** — voir le fichier [LICENSE](LICENSE) pour l
|
||||
|
||||
## 📝 Changelog
|
||||
|
||||
Consultez le [CHANGELOG.md](./CHANGELOG.md) pour l'historique complet de toutes les versions (v1.0.0 → v2.5.2).
|
||||
Consultez le [CHANGELOG.md](./CHANGELOG.md) pour l'historique complet de toutes les versions (v1.0.0 → v2.6.2).
|
||||
|
||||
---
|
||||
|
||||
*Projet : ObsiGate | Version : 2.5.2 | Dernière mise à jour : Juin 2026*
|
||||
*Projet : ObsiGate | Version : 2.6.2 | Dernière mise à jour : Juin 2026*
|
||||
|
||||
@@ -2,7 +2,7 @@
|
||||
|
||||
**Ultra-light web gateway for your Obsidian vaults** — Access, browse, and search all your Obsidian notes from any device via a modern, responsive web interface.
|
||||
|
||||
[]()
|
||||
[]()
|
||||
[](https://opensource.org/licenses/MIT)
|
||||
[](https://www.docker.com/)
|
||||
[](https://www.python.org/)
|
||||
@@ -1085,8 +1085,8 @@ This project is licensed under the **MIT License** - see the [LICENSE](LICENSE)
|
||||
|
||||
## 📝 Changelog
|
||||
|
||||
See [CHANGELOG.md](./CHANGELOG.md) for the complete version history (v1.0.0 → v2.5.2).
|
||||
See [CHANGELOG.md](./CHANGELOG.md) for the complete version history (v1.0.0 → v2.6.2).
|
||||
|
||||
---
|
||||
|
||||
*Project: ObsiGate | Version: 2.5.2 | Last updated: May 2026*
|
||||
*Project: ObsiGate | Version: 2.6.2 | Last updated: May 2026*
|
||||
|
||||
+30
-1
@@ -109,6 +109,30 @@ def _assistant_tool_message(content: str | None, tool_calls: list[Any]) -> dict[
|
||||
}
|
||||
|
||||
|
||||
def _deferred_tool_message(call: Any) -> dict[str, Any]:
|
||||
"""Answer a tool call that was not reached because the run paused.
|
||||
|
||||
A single LLM response may carry several tool calls. When one of them is
|
||||
mutating and pauses the run for confirmation, the assistant message already
|
||||
lists *all* of them, so every ``tool_call_id`` must get a tool result before
|
||||
the next LLM call (the OpenAI tool protocol rejects dangling ids). The calls
|
||||
that were not reached get a synthetic ``deferred`` result; the model
|
||||
re-issues them once the confirmed call has been applied (BUG-050).
|
||||
"""
|
||||
return {
|
||||
"role": "tool",
|
||||
"tool_call_id": call.id,
|
||||
"name": call.name,
|
||||
"content": json.dumps({
|
||||
"status": "deferred",
|
||||
"reason": (
|
||||
"Not executed: the run paused to confirm an earlier tool call. "
|
||||
"Re-issue this call if it is still needed."
|
||||
),
|
||||
}, ensure_ascii=False),
|
||||
}
|
||||
|
||||
|
||||
def _execute_confirmed(
|
||||
ctx: ToolContext,
|
||||
confirm_pending: dict[str, Any],
|
||||
@@ -248,7 +272,7 @@ async def run_agent(
|
||||
_emit_note(response.content or "")
|
||||
convo.append(_assistant_tool_message(response.content, response.tool_calls))
|
||||
|
||||
for call in response.tool_calls:
|
||||
for index, call in enumerate(response.tool_calls):
|
||||
if quota is not None and len(executed) >= quota:
|
||||
logger.warning(f"Agent reached the tool-call quota ({quota})")
|
||||
return AgentResult(
|
||||
@@ -268,6 +292,11 @@ async def run_agent(
|
||||
pending = e.to_dict()
|
||||
# Include the tool-call id so the client can echo it back.
|
||||
pending["error"]["id"] = call.id
|
||||
# BUG-050: the assistant message lists every tool call of this
|
||||
# batch, so answer the ones we did not reach to keep the
|
||||
# conversation valid for the resumed turn.
|
||||
for skipped in response.tool_calls[index + 1:]:
|
||||
convo.append(_deferred_tool_message(skipped))
|
||||
return AgentResult(
|
||||
content=response.content or "",
|
||||
messages=convo,
|
||||
|
||||
+5
-2
@@ -478,7 +478,9 @@ def build_system_prompt(context: dict[str, Any], scope: str = "directory", vault
|
||||
f"\n\nCes documents appartiennent au vault « {vault_name} ». Quand tu utilises un outil "
|
||||
"d'écriture (`append_to_file`, `edit_file`, `create_file`), passe TOUJOURS "
|
||||
f"exactement `\"vault\": \"{vault_name}\"` (jamais un nom inventé) et un `path` "
|
||||
"relatif au vault, identique à celui affiché ci-dessus."
|
||||
"relatif au vault, identique à celui affiché ci-dessus. Pour créer un fichier "
|
||||
"dans un nouveau dossier, un seul `create_file` avec le chemin complet suffit "
|
||||
"(les dossiers parents sont créés automatiquement)."
|
||||
)
|
||||
|
||||
return prompt
|
||||
@@ -490,7 +492,7 @@ Tes deux rôles :
|
||||
1. **Aider sur l'application** : expliquer la navigation, la recherche (full-text, filtres `tag:`, `created:`, `path:`), l'éditeur (CodeMirror, autosave, raccourcis), les onglets et le split view, les sauvegardes et la restauration, le partage public, l'export (HTML/Markdown/ePub/PDF), Mermaid, Excalidraw, les plugins, les thèmes, le mode hors-ligne, le MFA, etc.
|
||||
2. **Proposer des actions concrètes** : créer un fichier ou un dossier dans un vault.
|
||||
|
||||
Quand l'utilisateur demande explicitement de créer un fichier, inclus EXACTEMENT un bloc de ce type dans ta réponse (et rien d'autre à l'intérieur du bloc) :
|
||||
Quand l'utilisateur demande explicitement de créer un fichier, inclus un bloc de ce type dans ta réponse (un bloc par fichier, et rien d'autre à l'intérieur du bloc) :
|
||||
|
||||
```obsigate-action
|
||||
{"action": "create_file", "vault": "<nom du vault>", "path": "<chemin/relatif.md>", "content": "<contenu markdown>"}
|
||||
@@ -506,6 +508,7 @@ Règles :
|
||||
- Ne propose une action que si l'utilisateur la demande explicitement.
|
||||
- Explique en une phrase ce que fait l'action avant le bloc.
|
||||
- Utilise un chemin relatif se terminant par `.md` pour un fichier.
|
||||
- Pour créer un fichier dans un nouveau dossier, utilise **un seul** bloc `create_file` avec le chemin complet (ex. `"path": "Dossier/fichier.md"`) : les dossiers parents sont créés automatiquement, inutile d'émettre un `create_directory` séparé.
|
||||
- N'invente jamais un nom de vault : utilise l'un des vaults disponibles listés ci-dessous.
|
||||
- Réponds dans la langue de l'utilisateur, de façon concise et structurée (Markdown).
|
||||
"""
|
||||
|
||||
@@ -131,18 +131,33 @@ def create_file(
|
||||
return {"success": True, "vault": vault_name, "path": rel_path, "size": len(content)}
|
||||
|
||||
|
||||
def create_directory(vault_name: str, path: str) -> dict[str, Any]:
|
||||
def create_directory(vault_name: str, path: str, *, exist_ok: bool = False) -> dict[str, Any]:
|
||||
"""Create a directory (and its parents) in a vault.
|
||||
|
||||
Args:
|
||||
vault_name: Name of the vault.
|
||||
path: Vault-relative path of the new directory.
|
||||
exist_ok: When True, an existing directory is a success (idempotent)
|
||||
instead of raising ``already_exists``. Used by the AI tool layer so
|
||||
a "create folder then create file" plan does not fail when the
|
||||
folder is already there (``create_file`` creates parents anyway).
|
||||
|
||||
Raises:
|
||||
ServiceError: ``not_found`` (404), ``read_only`` (403) or
|
||||
``already_exists`` (409).
|
||||
``already_exists`` (409) when *exist_ok* is False.
|
||||
"""
|
||||
root = get_vault_root(vault_name)
|
||||
_ensure_writable(root)
|
||||
dir_path = resolve_safe_path(root, path)
|
||||
|
||||
if dir_path.exists():
|
||||
if exist_ok and dir_path.is_dir():
|
||||
return {
|
||||
"success": True,
|
||||
"vault": vault_name,
|
||||
"path": _rel(root, dir_path),
|
||||
"existed": True,
|
||||
}
|
||||
raise ServiceError(
|
||||
f"Directory already exists: {path}",
|
||||
code="already_exists",
|
||||
|
||||
@@ -355,7 +355,12 @@ def list_recent(ctx: ToolContext, params: ListRecentInput) -> dict[str, Any]:
|
||||
|
||||
@tool(
|
||||
name="create_file",
|
||||
description="Create a new text file in a vault with optional initial content.",
|
||||
description=(
|
||||
"Create a new text file in a vault with optional initial content. "
|
||||
"Parent directories are created automatically, so a single call with a "
|
||||
"nested path (e.g. 'Folder/note.md') is enough to create a file inside "
|
||||
"a new folder."
|
||||
),
|
||||
input_model=CreateFileInput,
|
||||
risk=ToolRisk.WRITE,
|
||||
requires_vault=True,
|
||||
@@ -367,14 +372,18 @@ def create_file(ctx: ToolContext, params: CreateFileInput) -> dict[str, Any]:
|
||||
|
||||
@tool(
|
||||
name="create_directory",
|
||||
description="Create a new directory (and parents) in a vault.",
|
||||
description=(
|
||||
"Create a new directory (and parents) in a vault. Succeeds if it "
|
||||
"already exists. Optional when creating a file: create_file already "
|
||||
"creates parent directories."
|
||||
),
|
||||
input_model=CreateDirectoryInput,
|
||||
risk=ToolRisk.WRITE,
|
||||
requires_vault=True,
|
||||
)
|
||||
def create_directory(ctx: ToolContext, params: CreateDirectoryInput) -> dict[str, Any]:
|
||||
"""Create a vault directory."""
|
||||
return _create_directory(params.vault, params.path)
|
||||
"""Create a vault directory (idempotent)."""
|
||||
return _create_directory(params.vault, params.path, exist_ok=True)
|
||||
|
||||
|
||||
@tool(
|
||||
|
||||
Generated
+1
-1
@@ -2626,7 +2626,7 @@ dependencies = [
|
||||
|
||||
[[package]]
|
||||
name = "obsigate-desktop"
|
||||
version = "2.5.2"
|
||||
version = "2.6.2"
|
||||
dependencies = [
|
||||
"chrono",
|
||||
"env_logger",
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
[package]
|
||||
name = "obsigate-desktop"
|
||||
version = "2.5.2"
|
||||
version = "2.6.2"
|
||||
description = "ObsiGate Desktop — Porte d'entrée native pour vos vaults Obsidian"
|
||||
authors = ["Bruno Charest"]
|
||||
edition = "2021"
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
{
|
||||
"$schema": "https://raw.githubusercontent.com/nicedoc/obsigate/main/desktop/tauri.conf.schema.json",
|
||||
"productName": "ObsiGate",
|
||||
"version": "2.5.2",
|
||||
"version": "2.6.2",
|
||||
"identifier": "com.obsigate.desktop",
|
||||
"build": {
|
||||
"frontendDist": "../frontend",
|
||||
|
||||
@@ -157,6 +157,9 @@ Avant de corriger quoi que ce soit, un agent IA doit :
|
||||
| *BUG-045* | [🟡 IMPORTANT] Éditeur « Editer » : deux barres de défilement superposées sur les documents longs | 🟢 corrigé | P1 | 📱 frontend | Éditeur | `frontend/style.css` | Ouvrir un fichier long (ex. IT/Docker Guide.md), cliquer Editer, mesurer `#editor-body` et `.cm-scroller` | `.editor-body-cm` gardait `overflow:auto` et un `.cm-editor{height:100%}` sous la rangée barre d'outils IA → le corps (toolbar+éditeur) ET le scroller CodeMirror débordaient simultanément. L'override global legacy `.cm-scroller{min-height:100%;overflow-y:auto!important}` aggravait. Passé en flex column : corps `overflow:hidden`, toolbar `flex:0 0 auto`, `.cm-editor` `flex:1 1 auto; height:auto`, seul le scroller défile ; override legacy retiré. Test : `tests/frontend/editor-inline.test.mjs` (+1) ; vérifié Playwright sur l'instance de test (un seul conteneur scrollable). |
|
||||
| *BUG-046* | [🔴 BLOQUANT] Assistant IA : « Échec de l'action : [object Object] » à l'application d'un ajout de texte au document courant | 🟢 corrigé | P0 | 📱 frontend + ⚙️ backend | IA | `frontend/js/bookslm.js`, `backend/bookslm.py`, `backend/bookslm_routes.py` | Mode agent : demander d'ajouter du texte au document ouvert puis cliquer « Appliquer » | Trois causes : (1) continuation de confirmation avec `payload:null` → second « Appliquer » sans `message` → 422 ; (2) `new Error(detail)` sur un `detail` tableau d'objets FastAPI → « [object Object] » ; (3) prompt documents/directory sans nom de vault → le modèle inventait `"vault":"test"` → échec silencieux de l'outil. Nouveau `_responseError()` (aplatit tableau/objet), payload porté à la continuation, bloc « Ces documents appartiennent au vault « X » » + consigne outils d'écriture (`build_system_prompt(vault_name=...)`). `SW_VERSION` v20. Tests : `tests/test_bookslm.py::test_vault_name_guidance`, `tests/frontend/ai.test.mjs` (+2). |
|
||||
| *BUG-047* | [🔴 BLOQUANT] La version affichée par l'application ne suit pas les livraisons : 66 commits livrés depuis v2.2.1 et l'UI/API restent bloquées sur `2.2.1` (et les numéros codés en dur divergent : `package.json` 1.0.0, desktop Tauri 2.0.0, `Dockerfile` 2.2.1, README 1.7.0) | 🟢 corrigé | P0 | ⚙️ build + 📄 docs | IA | `VERSION` (nouveau), `scripts/bump_version.py` (nouveau), `.githooks/prepare-commit-msg` + `.githooks/post-commit` (nouveaux), `scripts/install-hooks.sh` (nouveau), `backend/version.py`, `Dockerfile`, `docker-compose.yml`, `build.sh`, `.gitea/workflows/ci.yml`, `desktop/build.rs`, `tests/test_version.py` (nouveau) | `git tag -l \| tail -1` puis `python scripts/bump_version.py --print-version` ; `curl -s http://localhost:2020/api/health \| jq .version` | Le numéro provenait du **dernier tag git** et aucun tag n'était créé aux livraisons (`bump_version.sh` jamais appelé) → version figée, plus quatre numéros codés en dur ailleurs. Corrigé : **`VERSION` (racine) = source unique de vérité**, incrémentée automatiquement à chaque commit par le hook versionné `prepare-commit-msg` (SemVer : `!:`/`BREAKING CHANGE` → MAJEUR, `feat` → MINEUR, sinon CORRECTIF), tag `vX.Y.Z` créé par `post-commit` et publié au push (`push.followTags`) ; `bump_version.py` resynchronise `package.json`, desktop Tauri, ROADMAP, READMEs et fait la rotation du CHANGELOG dans le même commit ; backend, image Docker (`COPY VERSION`) et desktop lisent ce fichier. Contournement ponctuel : `SKIP_VERSION_BUMP=1`. | Garde-fou : `tests/test_version.py::TestRepoVersionAlignment` échoue dès qu'un dérivé diverge de `VERSION`. Vérifié : pytest complet vert, ruff/mypy 0, `/api/health` → `2.3.0` sur l'instance de test. |
|
||||
| *BUG-048* | [🟡 IMPORTANT] Assistant IA : les entrées « Contextes » et « Skills » du menu « + » n'ouvraient pas leur menu (`@` / `/`) | 🟢 corrigé | P2 | 📱 frontend | IA | `frontend/js/bookslm.js` | Menu « + » de l'assistant → cliquer « Contextes » ou « Skills » | `e.stopPropagation()` sur les entrées du panneau `.bookslm-ext-menu` (le clic remontait au gestionnaire du panneau qui annulait le rendu asynchrone) | Journal 2026-09-16. Tests : `tests/frontend/ai.test.mjs` (+3) |
|
||||
| *BUG-049* | [🔵 MINEUR] Assistant IA : icône du bouton « + » invisible (largeur SVG nulle) | 🟢 corrigé | P3 | 📱 frontend | IA | `frontend/style.css` | Ouvrir l'assistant et observer le bouton « + » | Sélecteur porté à `.bookslm-input-area button.bookslm-btn-plus` (la règle générique `padding: 8px 16px` sur un bouton 32 px annulait la largeur de contenu) | Vérifié navigateur : SVG 0 px → 18 px. Journal 2026-09-16 |
|
||||
| *BUG-050* | [🟡 IMPORTANT] Assistant IA : échec de la création d'un sous-dossier contenant un fichier (appels d'outils parallèles + confirmation) | 🟢 corrigé | P1 | ⚙️ backend + 🤖 ia | IA | `backend/agent/loop.py`, `backend/services/mutations.py`, `backend/tools/service.py`, `backend/bookslm.py` | Mode Agent : « crée le dossier X et un fichier Y dedans » puis Appliquer | `backend/agent/loop.py` : résultats « deferred » (`_deferred_tool_message`) pour les `tool_calls` non atteints lors d'une pause de confirmation ; `backend/services/mutations.py` : `create_directory(..., exist_ok=True)` ; `backend/tools/service.py` + `backend/bookslm.py` : consignes `create_file` (parents auto-créés, chemin imbriqué unique) | Cause : le message assistant listait plusieurs `tool_calls` mais la pause n'ajoutait le résultat que du seul appel confirmé → conversation invalide (tool_call_id sans réponse) au resume. Tests : `tests/test_agent_loop.py` (+1), `tests/test_tools_mutations.py` (+1), `tests/test_api_main.py` (+1) |
|
||||
| | | | | | | | | | | |
|
||||
|
||||
### TODOs techniques (améliorations / nouvelles tâches)
|
||||
@@ -205,6 +208,8 @@ Avant de corriger quoi que ce soit, un agent IA doit :
|
||||
|
||||
| 2026-09-16 | #94, #95, #96, #97 | Feature | `backend/ai_history.py` (nouveau), `backend/bookslm_routes.py`, `frontend/js/bookslm.js`, `frontend/js/config.js`, `frontend/index.html`, `frontend/style.css`, `frontend/locales/{fr,en}.json`, `tests/test_bookslm.py`, `tests/frontend/ai.test.mjs`, `docs/features/ai-assistant-history.md` (nouvelle), `docs/ROADMAP.md`, `CHANGELOG.md` | **Assistant IA** : #95 historique **permanent** (backend `data/ai_history/{user}.json`, cap 200, endpoints CRUD `/api/ai/bookslm/history[…]` résumés/full, sync frontend debounced 600 ms + repli localStorage + migration des clés legacy `bookslm-sessions-*`/`bookslm-history-*`, événement `bookslm:history-updated`) ; #96 onglet sidebar `#sidebar-tab-ai` (`messages-square`) + panneau `#sidebar-panel-ai` (liste chronologique, ouverture via `openWithSession`) ; #97 bouton **« + »** remplaçant « Attach an image » + panneau modulaire `.bookslm-ext-menu` (registre `_extensions` : Fichiers, Image, Contextes, Skills, Deep Research = mode agent + prompt, Recherche web & Canva en « Bientôt ») ; #94 bouton d'envoi circulaire + icône Lucide `arrow-up`. Vérifié : pytest 1088 passed / 6 skipped, ruff 0, mypy 0 (71 fichiers), tests frontend IA 84/84, unit 9/9, validate-imports 38 modules. | 🟢 corrigé (en attente vérif utilisateur) |
|
||||
| 2026-09-16 | BUG-048, #98 | Correction + feature | `frontend/js/bookslm.js`, `frontend/js/config.js`, `frontend/js/sidebar.js`, `frontend/style.css`, `frontend/locales/{fr,en}.json`, `.gitea/workflows/ci.yml`, `tests/frontend/ai.test.mjs`, `tests/frontend/ai-sidebar.test.mjs` (nouveau), `docs/features/ai-assistant-history.md`, `docs/ROADMAP.md`, `CHANGELOG.md` | **BUG-048** : les entrées « Contextes » et « Skills » du menu « + » ouvraient bien leur menu (`@` / `/`), mais le clic remontait au gestionnaire du panneau qui annulait le rendu asynchrone → menu jamais affiché ; correction par `e.stopPropagation()` sur les entrées du panneau `.bookslm-ext-menu`. **#98** : la barre de filtrage de la sidebar agit désormais sur l'onglet « Historique IA » — `filterAIHistory()` (config.js) filtre par titre, aperçu, répertoire, contexte ou libellé de mode, insensible casse/accents (`_aiNorm`), cache sessions `_aiSessionsCache`, message « aucune correspondance » (`bookslm.history_no_match`) dans la liste et placeholder dédié (`sidebar.filter_ai`) ; `initSidebarFilter` (sidebar.js) route saisie/touche casse/bouton « × » vers `filterAIHistory` quand l'onglet IA est actif ; chaque entrée du panneau « + » porte l'icône Lucide `plus`. Vérifié : tests frontend IA 87/87 (+3), nouvelle suite `ai-sidebar` 6/6, unit 9/9, 9 suites JSDOM vertes, validate-imports 38 modules, pytest / ruff / mypy inchangés (aucune modification backend). | 🟢 corrigé (en attente vérif utilisateur) — CI Gitea verte (lint, test, security, build, e2e) pour v2.5.0 (run #1511) |
|
||||
| 2026-09-16 | BUG-049, #99, #100 | Correction + feature | `frontend/style.css`, `frontend/js/config.js`, `frontend/js/viewer.js`, `frontend/js/sidebar.js`, `frontend/js/bookslm.js`, `frontend/locales/{fr,en}.json`, `.gitea/workflows/ci.yml`, `tests/frontend/ai.test.mjs`, `tests/frontend/sidebar-filters.test.mjs` (nouveau), `docs/features/sidebar-filters.md` (nouvelle), `docs/features/ai-assistant-history.md`, `docs/ROADMAP.md`, `CHANGELOG.md` | **BUG-049** : icône du bouton « + » de l'assistant invisible — la règle générique `.bookslm-input-area button` (spécificité supérieure) imposait `padding: 8px 16px` sur un bouton `width: 32px` ⇒ largeur de contenu nulle ⇒ SVG `width: 0px` ; sélecteur porté à `.bookslm-input-area button.bookslm-btn-plus` (+ `:hover`), vérifié en navigateur (Playwright : SVG 0 px → 18 px). **#99** : la barre de filtrage de la sidebar agit désormais sur les vues **Récents** (`filterRecentFiles`, titre/chemin/vault/aperçu/tags) et **Sauvegardes** (`filterSavedSearches`, cumulable avec les pills type), insensible casse/accents (`_sidebarNorm`/`_savedNorm`), message d'absence de résultat (`sidebar.no_results`) et placeholders dédiés (`sidebar.filter_recent`, `sidebar.filter_saved`) ; `initSidebarFilter` refactoré en `routeFilter`/`routeClear` couvrant les 5 onglets. **#100** : « Deep Research » ajoute une **pastille** `.bookslm-chip-deep-research` (au lieu d'injecter la directive dans le composeur), active le mode Agent et injecte la directive au moment de l'envoi. Vérifié : tests frontend IA 88/88 (+1), `sidebar-filters` 8/8 (nouveau), `ai-sidebar` 6/6, unit 9/9, 9 suites JSDOM vertes, validate-imports 38 modules, vérification navigateur du bouton « + » ; backend inchangé (pytest / ruff / mypy valides). | 🟢 corrigé (en attente vérif utilisateur) |
|
||||
| 2026-09-16 | BUG-050 | Correction | `backend/agent/loop.py`, `backend/services/mutations.py`, `backend/tools/service.py`, `backend/bookslm.py`, `tests/test_agent_loop.py`, `tests/test_tools_mutations.py`, `tests/test_api_main.py`, `docs/features/ai-tools-mcp.md`, `CHANGELOG.md` | **BUG-050** : création d'un sous-dossier contenant un fichier en mode Agent. (1) La boucle d'agent renvoyait la conversation sans réponse pour les `tool_calls` non atteints lorsqu'un appel mutateur déclenchait une confirmation → le provider rejetait le tour de reprise (« tool_call_id » orphelin) ; les appels restants reçoivent désormais un résultat `deferred` explicite (`_deferred_tool_message`) que le modèle réémet après confirmation. (2) `create_directory` est idempotent côté outil IA (`exist_ok=True`, succès si le dossier existe), le REST restant strict (409). (3) Consignes renforcées : `create_file` crée les dossiers parents, un seul appel avec chemin imbriqué suffit (`backend/bookslm.py`, descriptions d'outils). Vérifié : pytest 1091 passed / 6 skipped, ruff 0 (backend), mypy 0 (71 fichiers), tests frontend validate-imports 38 modules + unit 9/9. | 🟢 corrigé (en attente vérif utilisateur) |
|
||||
|
||||
---
|
||||
|
||||
|
||||
+4
-2
@@ -1,6 +1,6 @@
|
||||
# ObsiGate — Roadmap
|
||||
|
||||
> **Version :** 2.5.2 | **Dernière mise à jour :** 2026-09-16
|
||||
> **Version :** 2.6.2 | **Dernière mise à jour :** 2026-09-16
|
||||
> **Ce fichier ne contient que le travail à venir** (🔵 En cours + ⚪ Backlog) et un index compact
|
||||
> vers les fonctionnalités livrées.
|
||||
> - **Méthode de livraison à appliquer pour toute tâche : [DELIVERY_WORKFLOW.md](./DELIVERY_WORKFLOW.md)**
|
||||
@@ -195,6 +195,8 @@
|
||||
| 96 | Assistant IA — Accès rapide à l'historique depuis la sidebar de navigation | 2.4.0 | [features/ai-assistant-history.md](./features/ai-assistant-history.md) |
|
||||
| 97 | Assistant IA — Panneau « + » extensible (fichiers, Deep Research, contextes, skills…) | 2.4.0 | [features/ai-assistant-history.md](./features/ai-assistant-history.md) |
|
||||
| 98 | Assistant IA — Filtre de recherche dans la sidebar « Historique IA » | 2.5.0 | [features/ai-assistant-history.md](./features/ai-assistant-history.md) |
|
||||
| 99 | Sidebar — Filtrage des vues « Récents » et « Sauvegardes » | 2.6.0 | [features/sidebar-filters.md](./features/sidebar-filters.md) |
|
||||
| 100 | Assistant IA — Deep Research en pastille (au lieu du texte injecté) | 2.6.0 | [features/ai-assistant-history.md](./features/ai-assistant-history.md) |
|
||||
|
||||
---
|
||||
|
||||
@@ -202,7 +204,7 @@
|
||||
|
||||
| Priorité | Items | Effort total estimé |
|
||||
|---|---|---|
|
||||
| ✅ Complété | #1 → #59, #61–72, #74–76, #78–84, #88–93, #94–98 | ~113 jours réalisés |
|
||||
| ✅ Complété | #1 → #59, #61–72, #74–76, #78–84, #88–93, #94–100 | ~114 jours réalisés |
|
||||
| 🔵 P2 restant | #77 Desktop : signature de code (non retenue), 6 tests E2E **manuels** ([protocole](./DESKTOP_E2E_CHECKLIST.md)) | ~0,5-1 jour |
|
||||
| ⚪ P4 restant | #73 Sync (6-8j) | 6-8 jours |
|
||||
| ⚪ P2 restant | #92 Assistant IA — écosystème d'outils phase 2 (web étendu, sources connectées, documents) | 3-5 jours |
|
||||
|
||||
@@ -141,4 +141,37 @@ bien l'icône Lucide `plus` (vérifié par test JSDOM).
|
||||
- `MAX_SESSIONS = 200` : le comportement de purge est testé (`TestAIGatewayHistoryStore`)
|
||||
; la rétention configurable (item #95 du backlog) pourra s'appuyer sur ce plafond.
|
||||
- Les modules web/Canva du panneau « + » sont volontairement désactivés tant que
|
||||
l'écosystème d'outils phase 2 (#92) n'est pas livré.
|
||||
l'écosystème d'outils phase 2 (#92) n'est pas livré.
|
||||
|
||||
## I. Complément — icône « + » et pastille Deep Research (BUG-049, #100) — ✅ livré
|
||||
|
||||
*(1) **BUG-049 — l'icône du bouton « + » n'était pas visible.*** Le SVG Lucide était
|
||||
bien rendu, mais la règle générique `.bookslm-input-area button { padding: 8px 16px;
|
||||
background: var(--accent); color:#fff }` l'emportait en **spécificité** sur
|
||||
`.bookslm-btn-plus` (une classe seule). Le bouton conservait `width:32px` avec
|
||||
`padding: 8px 16px` → **largeur de contenu = 0 px**, donc SVG à `width: 0px`
|
||||
(invisible). Correctif CSS : sélecteur porté à
|
||||
`.bookslm-input-area button.bookslm-btn-plus` (et `:hover`), qui reprend la main
|
||||
(`padding:0`, fond transparent, couleur `--text-secondary`). Vérifié en navigateur
|
||||
(Playwright) : `svgWidth` passe de `0px` à `18px`.
|
||||
|
||||
*(2) **#100 — Deep Research devient une pastille (comme les skills).*** Auparavant,
|
||||
cliquer sur « Deep Research » injectait la directive de recherche dans la zone de
|
||||
saisie. Désormais :
|
||||
|
||||
- `_startDeepResearch()` active le **mode Agent** (si nécessaire), positionne le drapeau
|
||||
`_activeDeepResearch` et rend une **pastille** `.bookslm-chip-deep-research`
|
||||
(`_renderAttachments()`), sans rien écrire dans le composeur.
|
||||
- La pastille se retire via son « × » (comme les chips skills/fichiers) et remet le
|
||||
drapeau à `false`.
|
||||
- La directive (`bookslm.deep_research_prompt`) est injectée **au moment de l'envoi**
|
||||
dans le `message` du payload (`_sendMessage()`), sans polluer le message affiché à
|
||||
l'utilisateur.
|
||||
- Message d'information mis à jour (`bookslm.deep_research_started`) : « Deep Research
|
||||
activé — ajoutez votre question puis envoyez. »
|
||||
|
||||
### Tests du complément
|
||||
- `tests/frontend/ai.test.mjs` (+1) : le clic sur « Deep Research » ajoute une pastille,
|
||||
laisse le composeur vide, positionne le drapeau, et le retrait de la pastille remet
|
||||
le drapeau à `false`.
|
||||
- Vérification navigateur du bouton « + » (Playwright, instance de test).
|
||||
@@ -83,4 +83,5 @@
|
||||
- ✅ Transport MCP : **Streamable HTTP** (2026-09-11)
|
||||
- ✅ Confirmation MCP : **two-step `propose`/`apply`** (2026-09-11)
|
||||
- ✅ Périmètre des mutations externes : **toutes autorisées** (create/edit/rename/move/delete) — encadrées par confirmation + backup auto + audit + toggle par vault (2026-09-11)
|
||||
- ✅ Confirmation d'un lot d'appels (BUG-050) : quand un tour contient plusieurs appels d'outils et qu'un seul est mutateur, les appels non atteints reçoivent un résultat `deferred` pour préserver la validité du protocole tool-calling ; ils sont réémis après confirmation (2026-09-16)
|
||||
- Détail et justification dans le [guide §8](../AI_ARCHITECTURE_GUIDE.md).
|
||||
|
||||
@@ -0,0 +1,72 @@
|
||||
# #99 — Barre de filtrage de la sidebar sur « Récents » et « Sauvegardes »
|
||||
|
||||
> **Statut :** ✅ livré · **Version :** 2.6.0 · **ID :** #99
|
||||
> **Lié :** BUG-049 (icône du bouton « + » de l'assistant), #100 (pastille Deep Research).
|
||||
|
||||
## Contexte
|
||||
|
||||
La barre de recherche/filtrage de la sidebar (`#sidebar-filter-input`) agissait
|
||||
historiquement sur les onglets **Fichiers** (arborescence) et **Tags**. L'onglet
|
||||
**Historique IA** a reçu son propre filtrage en #98. Les onglets **Récents** et
|
||||
**Sauvegardes** (« saved searches ») n'étaient pas couverts : taper dans la barre
|
||||
n'avait aucun effet dans ces deux vues.
|
||||
|
||||
## A. Onglet « Récents » — ✅ livré
|
||||
|
||||
- Nouveau cache module `_recentQuery` et fonction exportée `filterRecentFiles(query)`
|
||||
(`frontend/js/config.js`).
|
||||
- `_applyRecentFilter(files)` filtre le cache `_recentFilesCache` sur **titre, chemin,
|
||||
vault, aperçu et tags**, via `_sidebarNorm()` (normalisation `NFD` + suppression des
|
||||
diacritiques + minuscules) — donc insensible à la casse **et** aux accents.
|
||||
- `loadRecentFiles()` applique désormais le filtre courant après chaque chargement.
|
||||
- Aucun résultat → un message `sidebar.no_results` est inséré dans `#recent-list`
|
||||
(classe `.sidebar-filter-empty`, style existant).
|
||||
- `switchSidebarTab("recent")` applique la requête courante et pose le placeholder
|
||||
`sidebar.filter_recent`.
|
||||
|
||||
## B. Onglet « Sauvegardes » — ✅ livré
|
||||
|
||||
- Nouveau `_savedQuery` et fonction exportée `filterSavedSearches(query)`
|
||||
(`frontend/js/viewer.js`).
|
||||
- `_applySavedFilter()` combine désormais **le filtre de type** (pills Tous / Recherches
|
||||
/ Répertoires) **et la requête texte** : un élément est visible si son `data-type`
|
||||
correspond au pill **et** si son texte (requête, vault, chemins inclus/exclus) contient
|
||||
la requête normalisée (`_savedNorm`).
|
||||
- Aucun résultat avec une requête active → message `sidebar.no_results` ajouté à
|
||||
`#saved-searches-list` (nettoyé à chaque ré-application pour ne pas s'accumuler).
|
||||
- `switchSidebarTab("saved")` ré-applique la requête et pose le placeholder
|
||||
`sidebar.filter_saved`.
|
||||
|
||||
## C. Routage unifié de la barre de filtrage — ✅ livré
|
||||
|
||||
`initSidebarFilter()` (`frontend/js/sidebar.js`) est refactorisé autour de deux
|
||||
fonctions `routeFilter(q)` / `routeClear()` qui dirigent la saisie, la bascule
|
||||
casse (`Aa`) et le bouton « × » vers le filtre de l'onglet actif :
|
||||
|
||||
| Onglet | Filtre |
|
||||
|---|---|
|
||||
| `vaults` | `performTreeSearch` / `restoreSidebarTree` |
|
||||
| `recent` | `filterRecentFiles` |
|
||||
| `saved` | `filterSavedSearches` |
|
||||
| `ai` | `filterAIHistory` |
|
||||
| `tags` | `filterTagCloud` |
|
||||
|
||||
## D. i18n — ✅ livré
|
||||
|
||||
- `sidebar.filter_recent` : « Filtrer les fichiers récents... » / "Filter recent files..."
|
||||
- `sidebar.filter_saved` : « Filtrer les recherches sauvegardées... » / "Filter saved searches..."
|
||||
- Le message d'absence de résultat réutilise `sidebar.no_results`.
|
||||
|
||||
## E. Tests — ✅ livré
|
||||
|
||||
- `tests/frontend/sidebar-filters.test.mjs` (nouveau, 8 tests) : rendu des listes,
|
||||
filtrage par titre/casse, accents/tags, vault/chemin, restauration, message
|
||||
d'absence de résultat, et combinaison pill + requête sur les sauvegardes.
|
||||
- Ajouté à la liste explicite du job `lint` de `.gitea/workflows/ci.yml`.
|
||||
|
||||
## F. Points d'attention
|
||||
|
||||
- Le filtrage est **client-side** (les listes sont déjà chargées) : aucune requête
|
||||
réseau supplémentaire n'est déclenchée par la frappe.
|
||||
- Sur « Sauvegardes », le filtre de type et la requête sont **cumulatifs** ; vider la
|
||||
barre (ou « × ») ne réinitialise pas le pill actif.
|
||||
+17
-14
@@ -114,6 +114,9 @@ class BooksLM {
|
||||
this._adhocDirs = [];
|
||||
this._images = [];
|
||||
this._activeSkill = null;
|
||||
// Deep Research: a skill-like chip that enables agent tools and injects a
|
||||
// research directive into the request (without polluting the composer).
|
||||
this._activeDeepResearch = false;
|
||||
this._skills = [];
|
||||
this._menuItems = [];
|
||||
this._menuIndex = 0;
|
||||
@@ -214,6 +217,7 @@ class BooksLM {
|
||||
this._adhocDirs = [];
|
||||
this._images = [];
|
||||
this._activeSkill = null;
|
||||
this._activeDeepResearch = false;
|
||||
this._isLoading = true;
|
||||
await this._loadHistory(preferredSessionId);
|
||||
|
||||
@@ -1216,7 +1220,7 @@ class BooksLM {
|
||||
const host = this._panel.querySelector('.bookslm-attachments');
|
||||
if (!host) return;
|
||||
host.innerHTML = '';
|
||||
const hasAny = this._adhocFiles.length || this._adhocDirs.length || this._images.length || this._activeSkill;
|
||||
const hasAny = this._adhocFiles.length || this._adhocDirs.length || this._images.length || this._activeSkill || this._activeDeepResearch;
|
||||
host.classList.toggle('empty', !hasAny);
|
||||
if (!hasAny) return;
|
||||
|
||||
@@ -1228,6 +1232,13 @@ class BooksLM {
|
||||
() => { this._activeSkill = null; this._renderAttachments(); },
|
||||
));
|
||||
}
|
||||
if (this._activeDeepResearch) {
|
||||
host.appendChild(this._chip(
|
||||
`🔎 ${t('bookslm.ext_deep_research')}`,
|
||||
'deep-research',
|
||||
() => { this._activeDeepResearch = false; this._renderAttachments(); },
|
||||
));
|
||||
}
|
||||
for (const file of this._adhocFiles) {
|
||||
host.appendChild(this._chip(`📄 ${file.path}`, 'file', () => this._removeAdhocFile(file.path)));
|
||||
}
|
||||
@@ -1449,21 +1460,13 @@ class BooksLM {
|
||||
if (typeof safeCreateIcons === 'function') safeCreateIcons();
|
||||
}
|
||||
|
||||
/** #97 — Deep Research: enabled agent tools + a pre-filled directive. */
|
||||
/** #97/#99 — Deep Research: agent tools + a skill-like chip (no composer text). */
|
||||
_startDeepResearch() {
|
||||
this._closeExtMenu();
|
||||
if (!this._agentMode) this._toggleAgentMode();
|
||||
const textarea = this._panel && this._panel.querySelector('textarea');
|
||||
if (textarea) {
|
||||
const directive = textarea.value.trim()
|
||||
? textarea.value.replace(/\s*$/, '')
|
||||
: '';
|
||||
textarea.value = (directive ? directive + '\n\n' : '') + t('bookslm.deep_research_prompt');
|
||||
textarea.dispatchEvent(new Event('input', { bubbles: true }));
|
||||
textarea.focus();
|
||||
} else {
|
||||
showToast(t('bookslm.deep_research_started'), 'info');
|
||||
}
|
||||
this._activeDeepResearch = true;
|
||||
this._renderAttachments();
|
||||
showToast(t('bookslm.deep_research_started'), 'info');
|
||||
}
|
||||
|
||||
_onPaste(e) {
|
||||
@@ -2728,7 +2731,7 @@ class BooksLM {
|
||||
img.path ? { path: img.path } : { data: img.data, mime_type: img.mime_type }
|
||||
)),
|
||||
skill: this._activeSkill,
|
||||
message: text,
|
||||
message: this._activeDeepResearch ? `${t('bookslm.deep_research_prompt')}\n\n${text}` : text,
|
||||
conversation_history: history,
|
||||
provider,
|
||||
model,
|
||||
|
||||
+48
-11
@@ -1,7 +1,7 @@
|
||||
// config.js — extracted from app.js (3872-4865)
|
||||
import { api, AuthManager, initMfaSettings } from './auth.js';
|
||||
import { state } from './state.js';
|
||||
import { el, icon, openFile } from './viewer.js';
|
||||
import { el, icon, openFile, filterSavedSearches } from './viewer.js';
|
||||
import { syncVaultSelectors, setSelectedVaultContext, refreshSidebarForContext, loadVaults, loadVaultSettings, loadTags, TagFilterService, refreshSidebarTreePreservingState } from './sidebar.js';
|
||||
import { escapeHtml, safeCreateIcons } from './utils.js';
|
||||
import { showToast, closeHeaderMenu, closeMobileSidebar } from './ui.js';
|
||||
@@ -11,6 +11,7 @@ import { getModelCapabilities, renderCapabilityList, refreshAIPickers } from './
|
||||
let _recentTimestampTimer = null;
|
||||
let _recentFilesCache = [];
|
||||
let _recentRefreshTimer = null;
|
||||
let _recentQuery = "";
|
||||
let _aiSessionsCache = [];
|
||||
let _aiQuery = "";
|
||||
|
||||
@@ -24,7 +25,7 @@ export async function loadRecentFiles(vaultFilter) {
|
||||
try {
|
||||
const data = await api(url);
|
||||
_recentFilesCache = data.files || [];
|
||||
renderRecentList(_recentFilesCache);
|
||||
renderRecentList(_applyRecentFilter(_recentFilesCache));
|
||||
} catch (err) {
|
||||
console.error("Failed to load recent files:", err);
|
||||
listEl.innerHTML = "";
|
||||
@@ -41,6 +42,9 @@ function renderRecentList(files) {
|
||||
listEl.innerHTML = "";
|
||||
|
||||
if (!files || files.length === 0) {
|
||||
if (_recentFilesCache.length && _recentQuery) {
|
||||
listEl.appendChild(el("div", { class: "sidebar-filter-empty" }, [document.createTextNode(t("sidebar.no_results"))]));
|
||||
}
|
||||
if (emptyEl) {
|
||||
emptyEl.classList.remove("hidden");
|
||||
safeCreateIcons();
|
||||
@@ -162,25 +166,45 @@ function _formatAIDate(ts) {
|
||||
}
|
||||
}
|
||||
|
||||
/** Accent- and case-insensitive normalization for the AI history filter (#98). */
|
||||
function _aiNorm(value) {
|
||||
/** Accent- and case-insensitive normalization for the sidebar filters (#98/#99). */
|
||||
function _sidebarNorm(value) {
|
||||
return String(value || "")
|
||||
.normalize("NFD")
|
||||
.replace(/[\u0300-\u036f]/g, "")
|
||||
.toLowerCase();
|
||||
}
|
||||
|
||||
/** Apply the current sidebar query to the recent-files list (#99). */
|
||||
function _applyRecentFilter(files) {
|
||||
const q = _sidebarNorm(_recentQuery);
|
||||
if (!q) return files;
|
||||
return (files || []).filter((f) => {
|
||||
const tags = Array.isArray(f.tags) ? f.tags.join(" ") : "";
|
||||
return _sidebarNorm(f.title).includes(q)
|
||||
|| _sidebarNorm(f.path).includes(q)
|
||||
|| _sidebarNorm(f.vault).includes(q)
|
||||
|| _sidebarNorm(f.preview).includes(q)
|
||||
|| _sidebarNorm(tags).includes(q);
|
||||
});
|
||||
}
|
||||
|
||||
/** Filter the sidebar recent-files list by the sidebar filter input (#99). */
|
||||
export function filterRecentFiles(query) {
|
||||
_recentQuery = (query || "").trim();
|
||||
renderRecentList(_applyRecentFilter(_recentFilesCache));
|
||||
}
|
||||
|
||||
/** Apply the current sidebar query to a session list, resolving the mode label. */
|
||||
function _applyAIFilter(sessions) {
|
||||
const q = _aiNorm(_aiQuery);
|
||||
const q = _sidebarNorm(_aiQuery);
|
||||
if (!q) return sessions;
|
||||
return (sessions || []).filter((s) => {
|
||||
const modeLabel = t("bookslm.mode_" + (s.mode || "general"));
|
||||
return _aiNorm(s.title).includes(q)
|
||||
|| _aiNorm(s.preview).includes(q)
|
||||
|| _aiNorm(s.directory).includes(q)
|
||||
|| _aiNorm(s.context).includes(q)
|
||||
|| _aiNorm(modeLabel).includes(q);
|
||||
return _sidebarNorm(s.title).includes(q)
|
||||
|| _sidebarNorm(s.preview).includes(q)
|
||||
|| _sidebarNorm(s.directory).includes(q)
|
||||
|| _sidebarNorm(s.context).includes(q)
|
||||
|| _sidebarNorm(modeLabel).includes(q);
|
||||
});
|
||||
}
|
||||
|
||||
@@ -274,21 +298,34 @@ function switchSidebarTab(tab) {
|
||||
});
|
||||
const filterInput = document.getElementById("sidebar-filter-input");
|
||||
if (filterInput) {
|
||||
const placeholders = { vaults: "Filtrer fichiers...", tags: "Filtrer tags...", recent: "", ai: t("sidebar.filter_ai") };
|
||||
const placeholders = {
|
||||
vaults: "Filtrer fichiers...",
|
||||
tags: "Filtrer tags...",
|
||||
recent: t("sidebar.filter_recent"),
|
||||
saved: t("sidebar.filter_saved"),
|
||||
ai: t("sidebar.filter_ai"),
|
||||
};
|
||||
filterInput.placeholder = placeholders[tab] || "";
|
||||
}
|
||||
const query = filterInput ? (state.sidebarFilterCaseSensitive ? filterInput.value.trim() : filterInput.value.trim().toLowerCase()) : "";
|
||||
if (query) {
|
||||
if (tab === "vaults") performTreeSearch(query);
|
||||
else if (tab === "tags") filterTagCloud(query);
|
||||
else if (tab === "recent") filterRecentFiles(query);
|
||||
else if (tab === "saved") filterSavedSearches(query);
|
||||
else if (tab === "ai") filterAIHistory(query);
|
||||
}
|
||||
// Auto-load recent files when switching to the recent tab
|
||||
if (tab === "recent") {
|
||||
_populateRecentVaultFilter();
|
||||
if (filterInput) filterRecentFiles(filterInput.value.trim());
|
||||
const vaultFilter = document.getElementById("recent-vault-filter");
|
||||
loadRecentFiles(vaultFilter ? vaultFilter.value || null : null);
|
||||
}
|
||||
// #99 — Re-apply the current sidebar query on the saved-searches tab.
|
||||
if (tab === "saved" && filterInput) {
|
||||
filterSavedSearches(filterInput.value.trim());
|
||||
}
|
||||
// #96/#98 — Auto-load the AI conversation history when entering its tab,
|
||||
// re-applying the current sidebar filter query.
|
||||
if (tab === "ai") {
|
||||
|
||||
+24
-35
@@ -2,8 +2,8 @@ import { state } from './state.js';
|
||||
import { safeCreateIcons, getFileIcon, flushIcons } from './utils.js';
|
||||
import { api } from './auth.js';
|
||||
import { populateCustomDropdown, TabManager, closeMobileSidebar, ContextMenuManager } from './ui.js';
|
||||
import { _populateRecentVaultFilter, switchSidebarTab, filterAIHistory } from './config.js';
|
||||
import { el, icon, getVaultIcon, smallBadge, attachTreeItemActionButton, attachTreeItemLongPress, showWelcome, appendHighlightedText } from './viewer.js';
|
||||
import { _populateRecentVaultFilter, switchSidebarTab, filterAIHistory, filterRecentFiles } from './config.js';
|
||||
import { el, icon, getVaultIcon, smallBadge, attachTreeItemActionButton, attachTreeItemLongPress, showWelcome, appendHighlightedText, filterSavedSearches } from './viewer.js';
|
||||
import { performAdvancedSearch } from './search.js';
|
||||
import { t } from './i18n.js';
|
||||
|
||||
@@ -700,29 +700,32 @@ function initSidebarFilter() {
|
||||
const caseBtn = document.getElementById("sidebar-filter-case-btn");
|
||||
const clearBtn = document.getElementById("sidebar-filter-clear-btn");
|
||||
|
||||
// Route the query to the active tab's own filter (#98/#99).
|
||||
const routeFilter = async (q) => {
|
||||
const tab = state.activeSidebarTab;
|
||||
if (tab === "vaults") await performTreeSearch(q);
|
||||
else if (tab === "recent") filterRecentFiles(q);
|
||||
else if (tab === "saved") filterSavedSearches(q);
|
||||
else if (tab === "ai") filterAIHistory(q);
|
||||
else filterTagCloud(q);
|
||||
};
|
||||
const routeClear = async () => {
|
||||
const tab = state.activeSidebarTab;
|
||||
if (tab === "vaults") await restoreSidebarTree();
|
||||
else if (tab === "recent") filterRecentFiles("");
|
||||
else if (tab === "saved") filterSavedSearches("");
|
||||
else if (tab === "ai") filterAIHistory("");
|
||||
else filterTagCloud("");
|
||||
};
|
||||
|
||||
input.addEventListener("input", () => {
|
||||
const hasText = input.value.length > 0;
|
||||
clearBtn.style.display = hasText ? "flex" : "none";
|
||||
clearTimeout(state.filterDebounce);
|
||||
state.filterDebounce = setTimeout(async () => {
|
||||
const q = state.sidebarFilterCaseSensitive ? input.value.trim() : input.value.trim().toLowerCase();
|
||||
if (hasText) {
|
||||
if (state.activeSidebarTab === "vaults") {
|
||||
await performTreeSearch(q);
|
||||
} else if (state.activeSidebarTab === "ai") {
|
||||
filterAIHistory(q);
|
||||
} else {
|
||||
filterTagCloud(q);
|
||||
}
|
||||
} else {
|
||||
if (state.activeSidebarTab === "vaults") {
|
||||
await restoreSidebarTree();
|
||||
} else if (state.activeSidebarTab === "ai") {
|
||||
filterAIHistory("");
|
||||
} else {
|
||||
filterTagCloud("");
|
||||
}
|
||||
}
|
||||
if (hasText) await routeFilter(q);
|
||||
else await routeClear();
|
||||
}, 220);
|
||||
});
|
||||
|
||||
@@ -730,15 +733,7 @@ function initSidebarFilter() {
|
||||
state.sidebarFilterCaseSensitive = !state.sidebarFilterCaseSensitive;
|
||||
caseBtn.classList.toggle("active");
|
||||
const q = state.sidebarFilterCaseSensitive ? input.value.trim() : input.value.trim().toLowerCase();
|
||||
if (input.value.trim()) {
|
||||
if (state.activeSidebarTab === "vaults") {
|
||||
await performTreeSearch(q);
|
||||
} else if (state.activeSidebarTab === "ai") {
|
||||
filterAIHistory(q);
|
||||
} else {
|
||||
filterTagCloud(q);
|
||||
}
|
||||
}
|
||||
if (input.value.trim()) await routeFilter(q);
|
||||
});
|
||||
|
||||
clearBtn.addEventListener("click", async () => {
|
||||
@@ -747,13 +742,7 @@ function initSidebarFilter() {
|
||||
state.sidebarFilterCaseSensitive = false;
|
||||
caseBtn.classList.remove("active");
|
||||
clearTimeout(state.filterDebounce);
|
||||
if (state.activeSidebarTab === "vaults") {
|
||||
await restoreSidebarTree();
|
||||
} else if (state.activeSidebarTab === "ai") {
|
||||
filterAIHistory("");
|
||||
} else {
|
||||
filterTagCloud("");
|
||||
}
|
||||
await routeClear();
|
||||
});
|
||||
|
||||
clearBtn.style.display = "none";
|
||||
|
||||
+32
-1
@@ -1811,6 +1811,14 @@ function _renderSyncPanel(panel) {
|
||||
// ── Saved searches filter ──
|
||||
var _savedFilterType = 'all';
|
||||
var _savedFilterInitDone = false;
|
||||
var _savedQuery = '';
|
||||
|
||||
function _savedNorm(value) {
|
||||
return String(value || '')
|
||||
.normalize('NFD')
|
||||
.replace(/[\u0300-\u036f]/g, '')
|
||||
.toLowerCase();
|
||||
}
|
||||
|
||||
function _initSavedFilter() {
|
||||
if (_savedFilterInitDone) return;
|
||||
@@ -1827,18 +1835,41 @@ function _initSavedFilter() {
|
||||
_savedFilterInitDone = true;
|
||||
}
|
||||
|
||||
/** #99 — Filter the saved-searches list by the global sidebar filter input. */
|
||||
export function filterSavedSearches(query) {
|
||||
_savedQuery = (query || '').trim();
|
||||
_applySavedFilter();
|
||||
}
|
||||
|
||||
function _applySavedFilter() {
|
||||
var items = document.querySelectorAll('.saved-search-item');
|
||||
var list = document.getElementById('saved-searches-list');
|
||||
var q = _savedNorm(_savedQuery);
|
||||
var hasVisible = false;
|
||||
items.forEach(function(item) {
|
||||
var type = item.dataset.type;
|
||||
var show = _savedFilterType === 'all' || type === _savedFilterType;
|
||||
var typeOk = _savedFilterType === 'all' || type === _savedFilterType;
|
||||
var textOk = !q || _savedNorm(item.textContent).includes(q);
|
||||
var show = typeOk && textOk;
|
||||
item.style.display = show ? '' : 'none';
|
||||
if (show) hasVisible = true;
|
||||
});
|
||||
// Remove a previous "no match" hint before deciding whether to add one.
|
||||
if (list) {
|
||||
var prev = list.querySelector('.sidebar-filter-empty');
|
||||
if (prev) prev.remove();
|
||||
}
|
||||
// Show/hide empty state
|
||||
var empty = document.getElementById('saved-searches-empty');
|
||||
if (empty) empty.style.display = hasVisible ? 'none' : '';
|
||||
// Explain an empty result caused by the global search rather than by an
|
||||
// actually empty saved-searches list.
|
||||
if (!hasVisible && q && list && items.length) {
|
||||
var hint = document.createElement('div');
|
||||
hint.className = 'sidebar-filter-empty';
|
||||
hint.textContent = t('sidebar.no_results');
|
||||
list.appendChild(hint);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
||||
@@ -1545,6 +1545,8 @@
|
||||
"sidebar.file_icons": "Icons",
|
||||
"sidebar.filter_instant": "Instant search",
|
||||
"sidebar.filter_ai": "Filter AI history...",
|
||||
"sidebar.filter_recent": "Filter recent files...",
|
||||
"sidebar.filter_saved": "Filter saved searches...",
|
||||
"sidebar.filter_path": "Path filters",
|
||||
"sidebar.filter_placeholder": "Filter files...",
|
||||
"sidebar.filter_results_grouped": "Grouped results",
|
||||
@@ -1849,7 +1851,7 @@
|
||||
"bookslm.soon": "Soon",
|
||||
"bookslm.ext_more_coming": "More options coming…",
|
||||
"bookslm.deep_research_prompt": "Perform an in-depth analysis of the request by breaking it into steps, searching for relevant information in the vault, cross-referencing them, and producing a structured summary with sources.",
|
||||
"bookslm.deep_research_started": "Deep research started — enable Agent mode to use the tools.",
|
||||
"bookslm.deep_research_started": "Deep Research enabled — add your question and send.",
|
||||
"bookslm.mode_general": "General",
|
||||
"bookslm.mode_directory": "Directory",
|
||||
"bookslm.mode_documents": "Documents",
|
||||
|
||||
@@ -1545,6 +1545,8 @@
|
||||
"sidebar.file_icons": "Icônes",
|
||||
"sidebar.filter_instant": "Recherche instantanée",
|
||||
"sidebar.filter_ai": "Filtrer l'historique IA...",
|
||||
"sidebar.filter_recent": "Filtrer les fichiers récents...",
|
||||
"sidebar.filter_saved": "Filtrer les recherches sauvegardées...",
|
||||
"sidebar.filter_path": "Filtres de chemin",
|
||||
"sidebar.filter_placeholder": "Filtrer fichiers...",
|
||||
"sidebar.filter_results_grouped": "Résultats groupés",
|
||||
@@ -1849,7 +1851,7 @@
|
||||
"bookslm.soon": "Bientôt",
|
||||
"bookslm.ext_more_coming": "D'autres options à venir…",
|
||||
"bookslm.deep_research_prompt": "Réalise une analyse approfondie de la demande en décomposant les étapes, en cherchant les informations pertinentes dans le vault, en les croisant, et en produisant une synthèse structurée avec sources.",
|
||||
"bookslm.deep_research_started": "Deep Research lancé — activez le mode Agent pour utiliser les outils.",
|
||||
"bookslm.deep_research_started": "Deep Research activé — ajoutez votre question puis envoyez.",
|
||||
"bookslm.mode_general": "Général",
|
||||
"bookslm.mode_directory": "Répertoire",
|
||||
"bookslm.mode_documents": "Documents",
|
||||
|
||||
+3
-2
@@ -9676,6 +9676,7 @@ body.bookslm-resizing { cursor: ew-resize; user-select: none; }
|
||||
.bookslm-chip-remove:hover { color: #f87171; }
|
||||
.bookslm-chip-thumb { width: 18px; height: 18px; object-fit: cover; border-radius: 4px; }
|
||||
.bookslm-chip-skill { border-color: var(--accent); }
|
||||
.bookslm-chip-deep-research { border-color: var(--accent); }
|
||||
/* Composer menus (`/` commands and `@` mentions). */
|
||||
.bookslm-menu-layer { position: relative; }
|
||||
.bookslm-command-menu, .bookslm-mention-menu { position: absolute; left: 12px; right: 12px; bottom: 4px;
|
||||
@@ -9696,10 +9697,10 @@ body.bookslm-resizing { cursor: ew-resize; user-select: none; }
|
||||
white-space: nowrap; }
|
||||
.bookslm-menu-empty { padding: 10px; font-size: 12px; color: var(--text-secondary); text-align: center; }
|
||||
/* Image attach button. */
|
||||
.bookslm-btn-plus { display: flex; align-items: center; justify-content: center; flex-shrink: 0;
|
||||
.bookslm-input-area button.bookslm-btn-plus { display: flex; align-items: center; justify-content: center; flex-shrink: 0;
|
||||
background: none; border: 1px solid var(--border); color: var(--text-secondary); cursor: pointer;
|
||||
border-radius: 50%; width: 32px; height: 32px; padding: 0; }
|
||||
.bookslm-btn-plus:hover { color: var(--accent); border-color: var(--accent); }
|
||||
.bookslm-input-area button.bookslm-btn-plus:hover { color: var(--accent); border-color: var(--accent); }
|
||||
.bookslm-file-input { display: none; }
|
||||
.bookslm-files-input { display: none; }
|
||||
/* Extensible "+" panel (#97). */
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "obsigate",
|
||||
"version": "2.5.2",
|
||||
"version": "2.6.2",
|
||||
"description": "**Porte d'entrée web ultra-léger pour vos vaults Obsidian** — Accédez, naviguez et recherchez dans toutes vos notes Obsidian depuis n'importe quel appareil via une interface web moderne et responsive.",
|
||||
"main": "patch.js",
|
||||
"directories": {
|
||||
|
||||
+41
-10
@@ -1371,10 +1371,15 @@ async function main() {
|
||||
stateMod.state.allVaults = [];
|
||||
stateMod.state.selectedContextVault = null;
|
||||
stateMod.state.currentVault = null;
|
||||
let lastUrl = "";
|
||||
// Collect every request: `new BooksLM()` also hydrates the AI history
|
||||
// asynchronously, so the *last* URL is not necessarily the tree-search one.
|
||||
const urls = [];
|
||||
const sawTreeSearchAll = () => urls.some(
|
||||
(u) => u.includes("/api/tree-search") && u.includes("vault=all"),
|
||||
);
|
||||
globalThis.fetch = async (url) => {
|
||||
lastUrl = String(url);
|
||||
if (lastUrl.includes("/api/tree-search")) {
|
||||
urls.push(String(url));
|
||||
if (String(url).includes("/api/tree-search")) {
|
||||
return {
|
||||
ok: true, status: 200,
|
||||
json: async () => ({ results: [{ path: "Café/note.md", type: "file" }] }),
|
||||
@@ -1392,13 +1397,13 @@ async function main() {
|
||||
b._onComposerInput();
|
||||
const menu = b._panel.querySelector(".bookslm-mention-menu");
|
||||
let waited = 0;
|
||||
while (!lastUrl.includes("tree-search") && waited < 500) {
|
||||
while (!sawTreeSearchAll() && waited < 500) {
|
||||
await sleep(20);
|
||||
waited += 20;
|
||||
}
|
||||
assert.ok(!menu.classList.contains("hidden"), "accented mention keeps the menu open");
|
||||
assert.ok(lastUrl.includes("vault=all"),
|
||||
`no vault -> searches all vaults (got last fetch "${lastUrl}")`);
|
||||
assert.ok(sawTreeSearchAll(),
|
||||
`no vault -> searches all vaults (got ${JSON.stringify(urls)})`);
|
||||
assert.equal(menu.querySelectorAll(".bookslm-menu-item").length, 1);
|
||||
b._panel.remove();
|
||||
localStorage.clear();
|
||||
@@ -1687,10 +1692,10 @@ async function main() {
|
||||
const stateMod = await import(pathToFileURL(path.join(JS_DIR, "state.js")).href);
|
||||
const prev = stateMod.state.allVaults;
|
||||
stateMod.state.allVaults = [{ name: "FallbackVault" }];
|
||||
let lastUrl = "";
|
||||
const urls = [];
|
||||
globalThis.fetch = async (url) => {
|
||||
lastUrl = String(url);
|
||||
if (lastUrl.includes("/files")) {
|
||||
urls.push(String(url));
|
||||
if (String(url).includes("/files")) {
|
||||
return { ok: true, status: 200, json: async () => ({ files: [{ path: "a.md" }] }) };
|
||||
}
|
||||
return { ok: true, status: 200, json: async () => ({}) };
|
||||
@@ -1703,7 +1708,8 @@ async function main() {
|
||||
await b._showMentionMenu("");
|
||||
const menu = b._panel.querySelector(".bookslm-mention-menu");
|
||||
assert.ok(!menu.classList.contains("hidden"), "empty-query menu is shown");
|
||||
assert.ok(lastUrl.includes("FallbackVault"), "uses the fallback vault");
|
||||
assert.ok(urls.some((u) => u.includes("/files") && u.includes("FallbackVault")),
|
||||
`uses the fallback vault (got ${JSON.stringify(urls)})`);
|
||||
stateMod.state.allVaults = prev;
|
||||
b._panel.remove();
|
||||
localStorage.clear();
|
||||
@@ -1908,6 +1914,31 @@ async function main() {
|
||||
b._panel.remove();
|
||||
});
|
||||
|
||||
await test("ext menu Deep Research adds a chip instead of composer text", async () => {
|
||||
localStorage.clear();
|
||||
document.body.replaceChildren();
|
||||
const b = new BooksLM();
|
||||
b._panel = b._render();
|
||||
document.body.appendChild(b._panel);
|
||||
const ta = b._panel.querySelector("textarea");
|
||||
ta.value = "";
|
||||
b._toggleExtMenu();
|
||||
const btn = b._panel.querySelector('.bookslm-ext-item[data-ext-id="deep_research"]');
|
||||
assert.ok(btn, "Deep Research entry present in the + panel");
|
||||
btn.click();
|
||||
await sleep(20);
|
||||
const chip = b._panel.querySelector(".bookslm-chip-deep-research");
|
||||
assert.ok(chip, "a Deep Research chip is added");
|
||||
assert.equal(ta.value, "", "the composer stays empty (no injected directive)");
|
||||
assert.equal(b._activeDeepResearch, true, "the deep-research flag is set");
|
||||
// Removing the chip clears the flag.
|
||||
chip.querySelector(".bookslm-chip-remove").click();
|
||||
assert.ok(!b._panel.querySelector(".bookslm-chip-deep-research"), "the chip is removed");
|
||||
assert.equal(b._activeDeepResearch, false, "the flag is cleared with the chip");
|
||||
b._panel.remove();
|
||||
localStorage.clear();
|
||||
});
|
||||
|
||||
// ── Summary ──
|
||||
console.log(`\n${passCount}/${testCount} tests passed`);
|
||||
if (passCount !== testCount) {
|
||||
|
||||
@@ -0,0 +1,183 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* ObsiGate - JSDOM tests for the sidebar search/filter bar on the
|
||||
* "Recent" and "Saved searches" tabs (#99).
|
||||
*
|
||||
* Covers:
|
||||
* - `filterRecentFiles()` (config.js): narrows the recent-files list by
|
||||
* title / path / vault / preview / tags, case- and accent-insensitively
|
||||
* - `filterSavedSearches()` (viewer.js): narrows the saved-searches list and
|
||||
* combines with the type pills, with a "no match" hint when empty
|
||||
*
|
||||
* Usage: node tests/frontend/sidebar-filters.test.mjs
|
||||
*/
|
||||
|
||||
import { strict as assert } from "node:assert";
|
||||
import { JSDOM } from "jsdom";
|
||||
import { fileURLToPath, pathToFileURL } from "node:url";
|
||||
import path from "node:path";
|
||||
|
||||
const __filename = fileURLToPath(import.meta.url);
|
||||
const __dirname = path.dirname(__filename);
|
||||
const JS_DIR = path.resolve(__dirname, "..", "..", "frontend", "js");
|
||||
|
||||
const dom = new JSDOM(
|
||||
`<!DOCTYPE html>
|
||||
<html><body>
|
||||
<aside class="sidebar">
|
||||
<input id="sidebar-filter-input" value="" />
|
||||
<button id="sidebar-filter-case-btn"></button>
|
||||
<button id="sidebar-filter-clear-btn"></button>
|
||||
|
||||
<div id="recent-list" class="recent-list"></div>
|
||||
<div id="recent-empty" class="recent-empty hidden"></div>
|
||||
<select id="recent-vault-filter"></select>
|
||||
|
||||
<div class="saved-filter-bar" id="saved-filter-bar">
|
||||
<button class="saved-filter-pill active" data-filter="all">Tous</button>
|
||||
<button class="saved-filter-pill" data-filter="search">Recherches</button>
|
||||
<button class="saved-filter-pill" data-filter="directory">Repertoires</button>
|
||||
</div>
|
||||
<div id="saved-searches-list" class="recent-list"></div>
|
||||
<div id="saved-searches-empty" class="recent-empty"></div>
|
||||
</aside>
|
||||
</body></html>`,
|
||||
{ url: "http://localhost/", pretendToBeVisual: true },
|
||||
);
|
||||
|
||||
const w = dom.window;
|
||||
globalThis.window = w;
|
||||
globalThis.document = w.document;
|
||||
globalThis.HTMLElement = w.HTMLElement;
|
||||
globalThis.Element = w.Element;
|
||||
globalThis.Node = w.Node;
|
||||
globalThis.Event = w.Event;
|
||||
globalThis.CustomEvent = w.CustomEvent;
|
||||
globalThis.KeyboardEvent = w.KeyboardEvent;
|
||||
globalThis.localStorage = w.localStorage;
|
||||
globalThis.sessionStorage = w.sessionStorage;
|
||||
Object.defineProperty(globalThis, "navigator", {
|
||||
value: w.navigator,
|
||||
configurable: true,
|
||||
writable: true,
|
||||
});
|
||||
globalThis.MutationObserver = w.MutationObserver;
|
||||
globalThis.getComputedStyle = w.getComputedStyle.bind(w);
|
||||
globalThis.requestAnimationFrame = (cb) => setTimeout(cb, 0);
|
||||
globalThis.cancelAnimationFrame = (id) => clearTimeout(id);
|
||||
|
||||
let testCount = 0;
|
||||
let passCount = 0;
|
||||
|
||||
async function test(name, fn) {
|
||||
testCount++;
|
||||
try {
|
||||
await fn();
|
||||
passCount++;
|
||||
console.log(` ✓ ${name}`);
|
||||
} catch (err) {
|
||||
console.error(` ✗ ${name}\n ${err.message}`);
|
||||
process.exitCode = 1;
|
||||
}
|
||||
}
|
||||
|
||||
const RECENT = [
|
||||
{ vault: "V", path: "Recettes/Pizza.md", title: "Pizza maison", preview: "pâte", tags: ["cuisine"], mtime_human: "il y a 1 h" },
|
||||
{ vault: "V", path: "Notes/café.md", title: "Café du matin", preview: "arôme", tags: ["journal"], mtime_human: "il y a 2 h" },
|
||||
{ vault: "W", path: "Projets/plan.md", title: "Plan de projet", preview: "étapes", tags: [], mtime_human: "hier" },
|
||||
];
|
||||
|
||||
const SAVED = [
|
||||
{ id: "s1", query: "pizza", vault: "V", include_paths: "Recettes", case_sensitive: false },
|
||||
{ id: "s2", query: "", include_paths: "Recettes", vault: "V" },
|
||||
{ id: "s3", query: "roadmap", vault: "all" },
|
||||
];
|
||||
|
||||
globalThis.fetch = async (url) => {
|
||||
const u = String(url);
|
||||
if (u.includes("/api/recent")) {
|
||||
return { ok: true, status: 200, json: async () => ({ files: RECENT }) };
|
||||
}
|
||||
if (u.includes("/api/saved-searches")) {
|
||||
return { ok: true, status: 200, json: async () => SAVED };
|
||||
}
|
||||
return { ok: true, status: 200, json: async () => ({}) };
|
||||
};
|
||||
|
||||
const configMod = await import(pathToFileURL(path.join(JS_DIR, "config.js")).href);
|
||||
const viewerMod = await import(pathToFileURL(path.join(JS_DIR, "viewer.js")).href);
|
||||
const { loadRecentFiles, filterRecentFiles } = configMod;
|
||||
const { loadSavedSearches, filterSavedSearches } = viewerMod;
|
||||
|
||||
const recentItems = () => Array.from(document.querySelectorAll("#recent-list .recent-item"));
|
||||
const visibleRecent = () => recentItems().filter((el) => el.style.display !== "none").length;
|
||||
const savedItems = () => Array.from(document.querySelectorAll(".saved-search-item"));
|
||||
const visibleSaved = () => savedItems().filter((el) => el.style.display !== "none").length;
|
||||
|
||||
console.log("Sidebar filters (#99) — Recent & Saved searches");
|
||||
|
||||
await test("loadRecentFiles renders every recent file", async () => {
|
||||
await loadRecentFiles(null);
|
||||
assert.equal(recentItems().length, 3, "three recent items rendered");
|
||||
});
|
||||
|
||||
await test("filterRecentFiles narrows by title, case-insensitively", async () => {
|
||||
filterRecentFiles("pizza");
|
||||
assert.equal(visibleRecent(), 1, "one recent item matches 'pizza'");
|
||||
filterRecentFiles("PLAN");
|
||||
assert.equal(visibleRecent(), 1, "case-insensitive match on 'PLAN'");
|
||||
});
|
||||
|
||||
await test("filterRecentFiles matches accents and tags", async () => {
|
||||
filterRecentFiles("cafe");
|
||||
assert.equal(visibleRecent(), 1, "'cafe' matches 'Café du matin'");
|
||||
filterRecentFiles("journal");
|
||||
assert.equal(visibleRecent(), 1, "tag match");
|
||||
});
|
||||
|
||||
await test("filterRecentFiles matches the vault and clears", async () => {
|
||||
filterRecentFiles("Projets");
|
||||
assert.equal(visibleRecent(), 1, "path match");
|
||||
filterRecentFiles("");
|
||||
assert.equal(visibleRecent(), 3, "no query restores everything");
|
||||
});
|
||||
|
||||
await test("filterRecentFiles shows a no-match hint", async () => {
|
||||
filterRecentFiles("zzz-inexistant");
|
||||
assert.equal(visibleRecent(), 0, "no item left");
|
||||
assert.ok(document.querySelector("#recent-list .sidebar-filter-empty"),
|
||||
"a no-match hint is rendered in the recent list");
|
||||
filterRecentFiles("");
|
||||
});
|
||||
|
||||
await test("loadSavedSearches renders every saved search", async () => {
|
||||
await loadSavedSearches();
|
||||
assert.equal(savedItems().length, 3, "three saved searches rendered");
|
||||
});
|
||||
|
||||
await test("filterSavedSearches narrows by query text", async () => {
|
||||
filterSavedSearches("pizza");
|
||||
assert.equal(visibleSaved(), 1, "one saved search matches 'pizza'");
|
||||
filterSavedSearches("roadmap");
|
||||
assert.equal(visibleSaved(), 1, "match on the query field");
|
||||
});
|
||||
|
||||
await test("filterSavedSearches combines with the type pills", async () => {
|
||||
filterSavedSearches("");
|
||||
assert.equal(visibleSaved(), 3, "all visible before the pill filter");
|
||||
document.querySelector('.saved-filter-pill[data-filter="directory"]').click();
|
||||
assert.equal(visibleSaved(), 1, "only the directory search stays visible");
|
||||
filterSavedSearches("roadmap");
|
||||
assert.equal(visibleSaved(), 0, "the query excludes the directory search");
|
||||
assert.ok(document.querySelector("#saved-searches-list .sidebar-filter-empty"),
|
||||
"a no-match hint is rendered in the saved list");
|
||||
document.querySelector('.saved-filter-pill[data-filter="all"]').click();
|
||||
filterSavedSearches("");
|
||||
assert.equal(visibleSaved(), 3, "clearing both filters restores everything");
|
||||
});
|
||||
|
||||
// ── Summary ──
|
||||
console.log(`\n${passCount}/${testCount} tests passed`);
|
||||
if (passCount !== testCount) {
|
||||
process.exit(1);
|
||||
}
|
||||
@@ -1,6 +1,8 @@
|
||||
# tests/test_agent_loop.py — Unit tests for the in-app agent loop (Phase B)
|
||||
"""Tests for backend.agent.loop.run_agent using a scripted (mocked) LLM."""
|
||||
|
||||
import json
|
||||
|
||||
import pytest
|
||||
|
||||
from backend.agent.loop import (
|
||||
@@ -244,6 +246,59 @@ class TestConfirmationResume:
|
||||
assert len(assistant_tool_msgs) == 1
|
||||
assert assistant_tool_msgs[0]["tool_calls"][0]["id"] == "call_9"
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_confirmation_with_parallel_calls_keeps_conversation_valid(self, monkeypatch):
|
||||
"""BUG-050: pausing on one tool call of a batch must answer the others.
|
||||
|
||||
The assistant message lists every tool call of the response, so the
|
||||
provider rejects the resumed turn when a ``tool_call_id`` has no tool
|
||||
result (the "create a folder and a file inside" scenario).
|
||||
"""
|
||||
_register(monkeypatch, "_write", lambda ctx, params: {"done": params}, risk=ToolRisk.WRITE)
|
||||
|
||||
llm1 = ScriptedLLM([LLMResponse(tool_calls=[
|
||||
ToolCall(id="1", name="_write", arguments={"x": 1}),
|
||||
ToolCall(id="2", name="_write", arguments={"x": 2}),
|
||||
])])
|
||||
paused = await run_agent([{"role": "user", "content": "write both"}], ctx=_ctx(), llm=llm1)
|
||||
assert paused.stopped == STOP_CONFIRMATION_REQUIRED
|
||||
assert paused.pending["error"]["id"] == "1"
|
||||
|
||||
# The call that was not reached is answered right away; the pending one
|
||||
# gets its result on resume, when the user applies it.
|
||||
answered = {m["tool_call_id"] for m in paused.messages if m.get("role") == "tool"}
|
||||
assert "2" in answered
|
||||
assert "1" not in answered
|
||||
|
||||
# Resume: the pending call is applied, the next turn stays valid.
|
||||
llm2 = ScriptedLLM([LLMResponse(content="ok")])
|
||||
resumed = await run_agent(
|
||||
[{"role": "user", "content": "write both"}],
|
||||
ctx=_ctx(),
|
||||
llm=llm2,
|
||||
resume_messages=paused.messages,
|
||||
confirm_pending=paused.pending,
|
||||
)
|
||||
assert resumed.stopped == STOP_DONE
|
||||
assert resumed.content == "ok"
|
||||
assert len(resumed.tool_calls) == 1
|
||||
assert resumed.tool_calls[0].ok is True
|
||||
# Before the resumed LLM call, every announced tool_call_id is answered.
|
||||
resumed_messages = llm2.calls[0]["messages"]
|
||||
assistant = next(
|
||||
m for m in resumed_messages
|
||||
if m.get("role") == "assistant" and m.get("tool_calls")
|
||||
)
|
||||
announced = {tc["id"] for tc in assistant["tool_calls"]}
|
||||
answered = {m["tool_call_id"] for m in resumed_messages if m.get("role") == "tool"}
|
||||
assert announced <= answered
|
||||
# The skipped call is flagged "deferred" so the model can re-issue it.
|
||||
deferred = [
|
||||
m for m in resumed_messages
|
||||
if m.get("role") == "tool" and json.loads(m["content"]).get("status") == "deferred"
|
||||
]
|
||||
assert [m["tool_call_id"] for m in deferred] == ["2"]
|
||||
|
||||
|
||||
class TestAgentPermissions:
|
||||
@pytest.mark.asyncio
|
||||
|
||||
@@ -271,6 +271,12 @@ class TestDirectoryCRUD:
|
||||
data = resp.json()
|
||||
assert data["success"] is True
|
||||
|
||||
def test_create_directory_existing_conflicts(self, client):
|
||||
"""The REST endpoint stays strict: an existing folder is a 409."""
|
||||
client.post("/api/directory/TestVault", json={"path": "Dup"})
|
||||
resp = client.post("/api/directory/TestVault", json={"path": "Dup"})
|
||||
assert resp.status_code == 409
|
||||
|
||||
def test_rename_directory(self, client):
|
||||
# Create first
|
||||
client.post("/api/directory/TestVault", json={"path": "OldName"})
|
||||
|
||||
@@ -143,6 +143,13 @@ class TestCreate:
|
||||
assert result.ok
|
||||
assert (_vault_path() / "A" / "B").is_dir()
|
||||
|
||||
def test_create_directory_idempotent(self, client):
|
||||
"""BUG-050: re-creating an existing folder is a success for the AI layer."""
|
||||
call_tool("create_directory", _ctx(), {"vault": "TestVault", "path": "Idem/Dir"}, confirm=True)
|
||||
second = call_tool("create_directory", _ctx(), {"vault": "TestVault", "path": "Idem/Dir"}, confirm=True)
|
||||
assert second.ok
|
||||
assert second.data.get("existed") is True
|
||||
|
||||
|
||||
# ═══════════════════════════════════════════════════════════════════
|
||||
# D2. Edit / append / rename / move
|
||||
|
||||
Reference in New Issue
Block a user