Recherche partout (#68) : page de résultats unifiée + draco et 401 sur /api/search #30

Closed

bruno wants to merge 11 commits from recherche-partout-68 into main

pull from: recherche-partout-68

merge into: :main

:main

:fix-cors-search-doublons

:federation-recherche-console

:console-federation-recherche

:flowdeck-recherche-onglets

:flowdeck-texte-parasite

:align-recherche-console

:alertes-onglets-5-rebase-main

:alertes-onglets-5

:recherche-onglets-ux-68

:cors-recherche-alertes-68-69

:recherche-partout-68-rebase

:recherche-partout-68

:alertes-unifiees-69

:fusion-70-barre-console-spark

:barre-apps-v0.1.6-flowdeck

:feat/coffre-reference-obsigate

:feat/v6.5.0-synced-db

:feat/v6.0

:feat/v5.6.0-import

:feat/v5.5.0-embeds-media

:docs/roadmap-v5.4.0-completed

:feat/v5.4.0-v5.5.0

:feat/v4.5.0-sprints-my-tasks

:feat/v4.4.0-tasks-dependencies

:feat/v4.3.0-database-views

:feat/v4.2.0-templates-dashboards

:feat/v4.1.0-database-sources

:develop

:fix/no-workspace-sidebar

:feat/quality

:feat/onboarding

Conversation 1 Commits 11 Files Changed 10

+992 -3

bruno commented 2026-10-28 12:12:11 -04:00

Owner

Copy Link

Copy Source

Rechercher partout — page de résultats unifiée + draco sur /api/search (fusion #68, chantier 1)

Branche recherche-partout-68 → main. Ne pas merger sans le go du coordinateur ; aucune version bumpée, aucune issue fermée par cette PR.

Audit des endpoints de recherche réels (vérifié sur main, 2026-10-28)

App	Endpoint JSON	Auth acceptée	Lien profond vérifié
ObsiGate	GET /api/search?q=&vault=all&limit= (SearchResponse.results[] : vault, path, title, snippet)	Bearer access_token, cookie access_token, ou repli cookie SSO draco_session	{og}/#file={vault}:{path}
Flowdeck	GET /api/search?q= (app/routers/search.py)	cookie flowdeck_session seulement (voir correctif ci-dessous)	{fd}/pages/<id>, {fd}/db/<id> (émis par le service lui-même)
TikTak	GET /api/search?q= (et /api/v1/search)	session native, cookie SSO draco_session, ou Bearer tck_…	{tk}/search?q=<q>&task=<id>
Kanbante	aucun endpoint JSON — page SSR GET /search uniquement (sa PR fournit GET /api/search)	—	{kb}/board/<owner>/<repo> (la carte n'a pas d'URL propre adressable)
Console agent-commun	aucun endpoint de recherche des discussions ; l'agent n'est pas un hôte de la barre	—	—

Conséquence honnête : pas de section Console (rien à agréger sans inventer une API).

Architecture choisie : page par hôte + agrégation côté client

Identique aux autres PR du chantier (voir PR TikTak #220 pour l'argumentaire complet) : pas de hub central (SPOF, dépendance runtime entre apps), pas de proxy fan-out serveur (l'hôte ne possède pas de credential sœur sans créer un relais d'identité générique). La page appelle les 4 API en parallèle depuis le navigateur (credentials: "include" : cookies de chaque origine, dont le SSO partagé draco_session, Domain=.dracodev.net), avec debounce 250 ms, timeout 8 s par section, et états chargement/vide/erreur par application. Résolution des bases : data-base-* rendus par l'hôte > DRACO_BAR.urls (barre v0.1.6) > dérivation -dev > href des onglets > défauts prod. Aucun fichier ui/barre-apps.* touché. Clavier : / ou Ctrl+K focalise la recherche, ↑/↓ navigue, Entrée ouvre, Échap efface.

Ce qui change (Flowdeck)

• app/templates/recherche.html (nouveau) + static/js/recherche.js (nouveau, logique partagée du chantier) + static/css/recherche.css (nouveau) : page GET /recherche avec 4 sections (ObsiGate, Flowdeck, TikTak, Kanbante), titre/extrait/contexte par résultat, liens profonds.
• app/routers/search.py :
  • GET /recherche (nouveau) : coquille rendue via ENV (même patron que les pages existantes), bases sœurs passées explicitement depuis draco_urls.toutes(). La palette Ctrl+K native est inchangée.
  • GET /api/search : repli SSO draco_session via le validateur local existant app.services.draco.user_for_draco (même sémantique que TikTak/Kanbante), et authentification désormais requise (401 sinon). Correctif de sécurité préexistant : sans session, la route donnait auparavant une recherche sans périmètre (user_id=None → 1=1 dans _scope_where, tous les espaces) — y compris pour la palette d'un visiteur non connecté. Les résultats d'un utilisateur authentifié restent périmétrés comme avant.
• app/middleware/recherche_cors.py (nouveau, ASGI pur) + app/main.py : CORS bornée pour GET /api/search uniquement — origines sœurs résolues par draco_urls.toutes(), plus les origines prod ou -dev officielles de l'environnement de la requête (nécessaire car ce middleware s'exécute avant le middleware CSP qui pose le contexte d'hôte pour draco_urls ; sans cela, une page servie par flowdeck-dev verrait ses sœurs -dev refusées quand la configuration seule ne les déclare pas). Jamais *, jamais d'autre route, jamais d'écriture. Le middleware répond lui-même aux preflights OPTIONS (qui ne portent jamais de cookies). La politique CORS globale est inchangée.
• app/middleware/security.py (ContentSecurityPolicyMiddleware) : connect-src de la page /recherche uniquement, étendue aux origines sœurs (ObsiGate, TikTak, Kanbante) — sinon le navigateur bloque les appels avant même CORS. Les autres pages gardent la CSP inchangée.
• app/templates/_draco_bar.html : 1 ligne — l'action palette de la barre va désormais sur /recherche au lieu du toast « arrive avec la fédération ». Le reste du fragment est inchangé (attention au chantier parallèle sur la barre : conflit possible sur ce fichier, à résoudre au merge).
• Tests : tests/test_recherche.py (pytest, 6 tests) + tests/recherche.test.js (Node, 25 assertions : résolution d'URL, liens profonds, normalisation des 4 formats).

Limites connues (à valider après déploiement)

• Les sections ObsiGate, TikTak et Kanbante restent en erreur tant que leurs PR respectives (CORS bornée, CORS bornée, endpoint JSON) ne sont pas déployées — comportement prévu, pas une régression.
• La correction 401 de /api/search change le comportement pour un visiteur non connecté (la palette ne renvoie plus de résultats sans périmètre) — c'est le correctif voulu, à confirmer au merge.
• Validation en direct (clic réel sur chaque lien profond, depuis flowdeck-dev et flowdeck) à faire par Bruno après redéploiement — non vérifiable depuis la sandbox (pas d'accès aux instances).

Tests (chiffres réels, exécutés le 2026-10-28)

• node tests/recherche.test.js : 25 assertions OK.
• Vérification Python de app/middleware/recherche_cors.py (chargé par chemin, pytest absent de la sandbox) : assertions OK — origines sœurs dev, ajout de l'environnement de la requête, refus tiers, preflight, middleware ASGI (preflight sœur → 204 + Access-Control-Allow-Origin exact, autre route → aucun en-tête ajouté).
• python3 -m py_compile sur app/main.py, app/middleware/security.py, app/routers/search.py, app/middleware/recherche_cors.py, tests/test_recherche.py : OK.
• Non exécutés ici (dépendances applicatives absentes de la sandbox) : pytest tests/test_recherche.py et le serveur — à valider en CI / après déploiement. Zéro régression revendiquée au-delà de ce qui est mesuré ci-dessus.

## Rechercher partout — page de résultats unifiée + draco sur /api/search (fusion #68, chantier 1) Branche `recherche-partout-68` → `main`. Ne pas merger sans le go du coordinateur ; aucune version bumpée, aucune issue fermée par cette PR. ### Audit des endpoints de recherche réels (vérifié sur `main`, 2026-10-28) | App | Endpoint JSON | Auth acceptée | Lien profond vérifié | |---|---|---|---| | ObsiGate | `GET /api/search?q=&vault=all&limit=` (`SearchResponse.results[] : vault, path, title, snippet`) | Bearer `access_token`, cookie `access_token`, ou repli cookie SSO `draco_session` | `{og}/#file={vault}:{path}` | | Flowdeck | `GET /api/search?q=` (`app/routers/search.py`) | cookie `flowdeck_session` **seulement** (voir correctif ci-dessous) | `{fd}/pages/<id>`, `{fd}/db/<id>` (émis par le service lui-même) | | TikTak | `GET /api/search?q=` (et `/api/v1/search`) | session native, cookie SSO `draco_session`, ou Bearer `tck_…` | `{tk}/search?q=<q>&task=<id>` | | Kanbante | **aucun endpoint JSON** — page SSR `GET /search` uniquement (sa PR fournit `GET /api/search`) | — | `{kb}/board/<owner>/<repo>` (la carte n'a pas d'URL propre adressable) | | Console agent-commun | **aucun endpoint de recherche** des discussions ; l'agent n'est pas un hôte de la barre | — | — | Conséquence honnête : **pas de section Console** (rien à agréger sans inventer une API). ### Architecture choisie : page par hôte + agrégation côté client Identique aux autres PR du chantier (voir PR TikTak #220 pour l'argumentaire complet) : pas de hub central (SPOF, dépendance runtime entre apps), pas de proxy fan-out serveur (l'hôte ne possède pas de credential sœur sans créer un relais d'identité générique). La page appelle les 4 API en parallèle depuis le navigateur (`credentials: "include"` : cookies de chaque origine, dont le SSO partagé `draco_session`, `Domain=.dracodev.net`), avec debounce 250 ms, timeout 8 s par section, et états chargement/vide/erreur **par application**. Résolution des bases : `data-base-*` rendus par l'hôte > `DRACO_BAR.urls` (barre v0.1.6) > dérivation `-dev` > href des onglets > défauts prod. Aucun fichier `ui/barre-apps.*` touché. Clavier : `/` ou Ctrl+K focalise la recherche, ↑/↓ navigue, Entrée ouvre, Échap efface. ### Ce qui change (Flowdeck) - `app/templates/recherche.html` (nouveau) + `static/js/recherche.js` (nouveau, logique partagée du chantier) + `static/css/recherche.css` (nouveau) : page `GET /recherche` avec 4 sections (ObsiGate, Flowdeck, TikTak, Kanbante), titre/extrait/contexte par résultat, liens profonds. - `app/routers/search.py` : - `GET /recherche` (nouveau) : coquille rendue via `ENV` (même patron que les pages existantes), bases sœurs passées explicitement depuis `draco_urls.toutes()`. La palette Ctrl+K native est inchangée. - `GET /api/search` : **repli SSO `draco_session`** via le validateur local existant `app.services.draco.user_for_draco` (même sémantique que TikTak/Kanbante), **et authentification désormais requise (401 sinon)**. Correctif de sécurité préexistant : sans session, la route donnait auparavant une recherche **sans périmètre** (`user_id=None` → `1=1` dans `_scope_where`, tous les espaces) — y compris pour la palette d'un visiteur non connecté. Les résultats d'un utilisateur authentifié restent périmétrés comme avant. - `app/middleware/recherche_cors.py` (nouveau, ASGI pur) + `app/main.py` : CORS bornée pour `GET /api/search` uniquement — origines sœurs résolues par `draco_urls.toutes()`, **plus** les origines prod ou `-dev` officielles de l'environnement de la requête (nécessaire car ce middleware s'exécute avant le middleware CSP qui pose le contexte d'hôte pour `draco_urls` ; sans cela, une page servie par `flowdeck-dev` verrait ses sœurs `-dev` refusées quand la configuration seule ne les déclare pas). Jamais `*`, jamais d'autre route, jamais d'écriture. Le middleware répond lui-même aux preflights OPTIONS (qui ne portent jamais de cookies). La politique CORS globale est inchangée. - `app/middleware/security.py` (`ContentSecurityPolicyMiddleware`) : `connect-src` de la page `/recherche` uniquement, étendue aux origines sœurs (ObsiGate, TikTak, Kanbante) — sinon le navigateur bloque les appels avant même CORS. Les autres pages gardent la CSP inchangée. - `app/templates/_draco_bar.html` : **1 ligne** — l'action `palette` de la barre va désormais sur `/recherche` au lieu du toast « arrive avec la fédération ». Le reste du fragment est inchangé (attention au chantier parallèle sur la barre : conflit possible sur ce fichier, à résoudre au merge). - Tests : `tests/test_recherche.py` (pytest, 6 tests) + `tests/recherche.test.js` (Node, 25 assertions : résolution d'URL, liens profonds, normalisation des 4 formats). ### Limites connues (à valider après déploiement) - Les sections ObsiGate, TikTak et Kanbante restent en erreur tant que leurs PR respectives (CORS bornée, CORS bornée, endpoint JSON) ne sont pas déployées — comportement prévu, pas une régression. - La correction 401 de `/api/search` change le comportement pour un visiteur non connecté (la palette ne renvoie plus de résultats sans périmètre) — c'est le correctif voulu, à confirmer au merge. - Validation en direct (clic réel sur chaque lien profond, depuis `flowdeck-dev` et `flowdeck`) à faire par Bruno après redéploiement — non vérifiable depuis la sandbox (pas d'accès aux instances). ### Tests (chiffres réels, exécutés le 2026-10-28) - `node tests/recherche.test.js` : **25 assertions OK**. - Vérification Python de `app/middleware/recherche_cors.py` (chargé par chemin, pytest absent de la sandbox) : **assertions OK** — origines sœurs dev, ajout de l'environnement de la requête, refus tiers, preflight, middleware ASGI (preflight sœur → 204 + `Access-Control-Allow-Origin` exact, autre route → aucun en-tête ajouté). - `python3 -m py_compile` sur `app/main.py`, `app/middleware/security.py`, `app/routers/search.py`, `app/middleware/recherche_cors.py`, `tests/test_recherche.py` : **OK**. - Non exécutés ici (dépendances applicatives absentes de la sandbox) : `pytest tests/test_recherche.py` et le serveur — à valider en CI / après déploiement. Zéro régression revendiquée au-delà de ce qui est mesuré ci-dessus.

bruno added 10 commits 2026-10-28 12:12:12 -04:00

Recherche partout (#68) : Recherche-partout-68-JS 7365d00c4c

Recherche partout (#68) : Recherche-partout-68-CSS 5c570b485c

Recherche partout (#68) : Recherche-partout-68-gabarit 46d26ca0c7

Recherche partout (#68) : Recherche-partout-68-CORS 79ea841a81

Recherche partout (#68) : Recherche-partout-68-tests cd4fcf78a9

Recherche partout (#68) : Recherche-partout-68-tests-Node 1abb9f97de

Recherche partout (#68) : Recherche-partout-68-route c8be099aed

Recherche partout (#68) : Recherche-partout-68-middleware 842204dadb

Recherche partout (#68) : Recherche-partout-68-CSP fc3e61f266

Recherche partout (#68) : Recherche-partout-68-palette edd2c24014

bruno added 1 commit 2026-10-28 12:18:26 -04:00

Recherche partout (#68) : JS compatible ESM (test ObsiGate) cf837edfbb

bruno referenced this pull request 2026-10-28 12:55:14 -04:00

Recherche partout (#68) : page de résultats unifiée + draco et 401 sur /api/search #31

bruno commented 2026-10-28 12:55:19 -04:00

Author

Owner

Copy Link

Copy Source

Remplacée par la PR #31 (rebase sur main post-#70 et post-#69 : conflits _draco_bar.html et app/main.py résolus en conservant la suppression du bouton •••, le contenu alertes #69 et la palette /recherche ; security.py non modifié car la CSP sœurs #69 couvre déjà /recherche. Contenu par ailleurs identique). Cette PR est fermée sans merge.

Remplacée par la PR #31 (rebase sur main post-#70 et post-#69 : conflits _draco_bar.html et app/main.py résolus en conservant la suppression du bouton •••, le contenu alertes #69 et la palette /recherche ; security.py non modifié car la CSP sœurs #69 couvre déjà /recherche. Contenu par ailleurs identique). Cette PR est fermée sans merge.

bruno closed this pull request 2026-10-28 12:55:29 -04:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Bug à corriger

design

UI/UX

documentation

Documentation

duplicate

Doublon

enhancement

Amélioration

feature

Nouvelle fonctionnalité

kanban:agent-task

kanban:in_progress

kanban:qa

kanban:review

kanban:todo

kanban_Backlog

Colonne Backlog

kanban_En_cours

Colonne En cours

kanban_Révision

Colonne Révision

kanban_Terminé

Colonne Terminé

kanban_a_faire

Colonne À faire

performance

Optimisation

question

Question

refactor

Refactoring code

roadmap

Feuille de route — programme Fusion

security

Sécurité

wontfix

Ne sera pas corrigé

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

bruno

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: bruno/flowdeck#30