fix: partage dirigé — ACL court-circuitée après résolution Partage/ + chemin canonique au token #196

- le partage EST l'autorisation: plus de 403 'Accès refusé à la vault' pour un
  destinataire sans accès au vault émetteur (api_file, raw, download)
- chemin virtuel canonique Partage/<token>/<nom> — désambiguïse les homonymes,
  fallback par nom conservé
- 'reçu' = partagé_avec contient l'utilisateur: un admin ne voit plus les
  partages des autres comme reçus (fix File not found: Partage/…)
- +3 tests (14 total)

CHANGELOG.md

@@ -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.67.0**.
+> [Unreleased](#unreleased). La dernière version livrée est **2.67.2**.
 
 ---
 
@@ -14,6 +14,14 @@ et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ---
 
+## [2.67.2] — 2026-10-10
+
+---
+
+## [2.67.1] — 2026-10-10
+
+---
+
 ## [2.67.0] — 2026-10-10
 
 ### Ajouté
@@ -29,6 +37,18 @@ et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
     couvre les partages dirigés reçus (résultats `home-<user>/Partage/…`,
     clic → page de partage, jamais l'éditeur du vault d'autrui).
   - Cache d'icônes partagé invalidé à la création/révocation.
+  - **Itération 3 :** correction « Directory not found: Partage » (le dossier
+    virtuel ne déclenche plus la navigation) ; un fichier reçu s'ouvre dans
+    l'application en onglet — arbre, recherche et dashboard — via résolution
+    serveur `home-<user>/Partage/<fichier>` vers le fichier source
+    (`/api/file`, raw, download) ; un `path` vide/null est rejeté (400) à la
+    création d'un partage.
+  - **Itération 4 :** correctifs d'ouverture — « Accès refusé à la vault … »
+    (le partage dirigé est l'autorisation, l'ACL du vault source est
+    court-circuitée après résolution `Partage/…`), « File not found:
+    Partage/… » (les partages non adressés à l'utilisateur ne s'affichent
+    plus comme reçus ; chemin canonique `Partage/<token>/<nom>` pour
+    désambiguïser les fichiers homonymes).
 
 ---
 

README.fr.md

@@ -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.
 
-[![Version](https://img.shields.io/badge/Version-2.67.0-blue.svg)]()
+[![Version](https://img.shields.io/badge/Version-2.67.2-blue.svg)]()
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Docker](https://img.shields.io/badge/Docker-Ready-blue.svg)](https://www.docker.com/)
 [![Python](https://img.shields.io/badge/Python-3.11+-green.svg)](https://www.python.org/)
@@ -977,8 +977,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.67.0).
+Consultez le [CHANGELOG.md](./CHANGELOG.md) pour l'historique complet de toutes les versions (v1.0.0 → v2.67.2).
 
 ---
 
-*Projet : ObsiGate | Version : 2.67.0 | Dernière mise à jour : Septembre 2026*
+*Projet : ObsiGate | Version : 2.67.2 | Dernière mise à jour : Septembre 2026*

README.md

@@ -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.
 
-[![Version](https://img.shields.io/badge/Version-2.67.0-blue.svg)]()
+[![Version](https://img.shields.io/badge/Version-2.67.2-blue.svg)]()
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Docker](https://img.shields.io/badge/Docker-Ready-blue.svg)](https://www.docker.com/)
 [![Python](https://img.shields.io/badge/Python-3.11+-green.svg)](https://www.python.org/)
@@ -1152,8 +1152,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.67.0).
+See [CHANGELOG.md](./CHANGELOG.md) for the complete version history (v1.0.0 → v2.67.2).
 
 ---
 
-*Project: ObsiGate | Version: 2.67.0 | Last updated: September 2026*
+*Project: ObsiGate | Version: 2.67.2 | Last updated: September 2026*

VERSION

@@ -1 +1 @@
-2.67.0
+2.67.2

backend/routers/files_read.py

@@ -45,9 +45,46 @@ from backend.services.files import read_raw_file
 from backend.services.mutations import file_revision
 from backend.services.paths import resolve_safe_path
 from backend.services.vaults import browse_directory, get_vault_root
+from backend.share import list_shares
 
 logger = logging.getLogger("obsigate")
 
+
+def _resolve_shared_file(vault_name: str, path: str, username: str | None):
+    """#196 — resolve ``home-<user>/Partage/<file>`` to the real source file.
+
+    Returns ``(real_vault, real_path)`` when *path* is a received share
+    mounted in the user's personal folder, else ``None``. Read-only: only a
+    recipient (or the share creator, whose own file is already accessible)
+    gets a mapping — a share directed to someone else never resolves here.
+
+    Chemin canonique : ``Partage/<token>/<nom>`` (token = lève l'ambiguïté
+    de deux partages au même nom). Fallback par nom seul pour les liens
+    ne transportant pas le token.
+    """
+    if not username or not vault_name.startswith("home-") or not path.startswith("Partage/"):
+        return None
+    owner = vault_name[len("home-"):]
+    if owner != username:
+        return None
+    # Chemin virtuel canonique : Partage/<token>/<nom> — le token lève
+    # l'ambiguïté (deux partages, même nom de base). Fallback : match par
+    # nom pour les liens ne portant pas le token.
+    rest = path.split("/", 1)[1]
+    parts = rest.split("/", 1)
+    if len(parts) == 2 and len(parts[0]) >= 20:  # token (64 hex) vs nom de fichier
+        token, _name = parts
+        for s in list_shares(user=username):
+            if s.get("token") == token and s.get("created_by") != username:
+                return s["vault"], s["path"]
+        return None
+    for s in list_shares(user=username):
+        if s.get("created_by") == username:
+            continue
+        if (s.get("path") or "").split("/")[-1] == rest:
+            return s["vault"], s["path"]
+    return None
+
 # Map file extensions to highlight.js language hints
 EXT_TO_LANG = {
     ".py": "python", ".js": "javascript", ".ts": "typescript",
@@ -104,7 +141,11 @@ async def api_file_raw(vault_name: str, path: str = Query(..., description="Rela
     Returns:
         ``FileRawResponse`` with vault, path, and raw text content.
     """
-    if not check_vault_access(vault_name, current_user):
+    shared = _resolve_shared_file(vault_name, path, current_user.get("username"))
+    if shared:
+        # Partage dirigé = autorisation (cf. api_file).
+        vault_name, path = shared
+    elif not check_vault_access(vault_name, current_user):
         raise HTTPException(status_code=403, detail=f"Accès refusé à la vault '{vault_name}'")
     return read_raw_file(vault_name, path)
 
@@ -120,7 +161,11 @@ async def api_file_download(vault_name: str, path: str = Query(..., description=
     Returns:
         ``FileResponse`` with ``application/octet-stream`` content-type.
     """
-    if not check_vault_access(vault_name, current_user):
+    shared = _resolve_shared_file(vault_name, path, current_user.get("username"))
+    if shared:
+        # Partage dirigé = autorisation (cf. api_file).
+        vault_name, path = shared
+    elif not check_vault_access(vault_name, current_user):
         raise HTTPException(status_code=403, detail=f"Accès refusé à la vault '{vault_name}'")
     vault_data = get_vault_data(vault_name)
     if not vault_data:
@@ -305,8 +350,18 @@ async def api_file(vault_name: str, path: str = Query(..., description="Relative
     Returns:
         ``FileContentResponse`` with HTML, metadata, and tags.
     """
-    if not check_vault_access(vault_name, current_user):
+    # #196 — fichier reçu par partage dirigé : home-<user>/Partage/<fichier>
+    # est résolu vers le fichier source (lecture seule, viewer standard).
+    # Résolu AVANT l'ACL vault : le dossier est virtuel, l'autorisation réelle
+    # est l'appartenance au partage (vérifiée dans le resolver).
+    shared = _resolve_shared_file(vault_name, path, current_user.get("username"))
+    if shared:
+        # Le partage dirigé EST l'autorisation : le destinataire n'a par
+        # définition pas accès au vault source — on ne passe PAS par l'ACL.
+        vault_name, path = shared
+    elif not check_vault_access(vault_name, current_user):
         raise HTTPException(status_code=403, detail=f"Accès refusé à la vault '{vault_name}'")
+
     vault_data = get_vault_data(vault_name)
     if not vault_data:
         raise HTTPException(status_code=404, detail=f"Vault '{vault_name}' not found")

backend/routers/sharing.py

@@ -88,7 +88,9 @@ async def api_share_create(
     """
     if not check_vault_access(vault_name, current_user):
         raise HTTPException(403, f"Accès refusé à la vault '{vault_name}'")
-    path = body.get("path", "")
+    path = body.get("path") or ""
+    if not path:
+        raise HTTPException(400, "Chemin de fichier requis")
     expires = body.get("expires_in_hours")
     # #196 — directed share: validate recipients before creating anything.
     recipients = body.get("shared_with") or []

desktop/Cargo.lock

@@ -2626,7 +2626,7 @@ dependencies = [
 
 [[package]]
 name = "obsigate-desktop"
-version = "2.67.0"
+version = "2.67.2"
 dependencies = [
  "chrono",
  "env_logger",

desktop/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "obsigate-desktop"
-version = "2.67.0"
+version = "2.67.2"
 description = "ObsiGate Desktop — Porte d'entrée native pour vos vaults Obsidian"
 authors = ["Bruno Charest"]
 edition = "2021"

desktop/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://raw.githubusercontent.com/nicedoc/obsigate/main/desktop/tauri.conf.schema.json",
   "productName": "ObsiGate",
-  "version": "2.67.0",
+  "version": "2.67.2",
   "identifier": "com.obsigate.desktop",
   "build": {
     "frontendDist": "../frontend",

docs/ROADMAP.md

@@ -1,6 +1,6 @@
 # ObsiGate — Roadmap
 
-> **Version :** 2.67.0 | **Dernière mise à jour :** 2026-10-10
+> **Version :** 2.67.2 | **Dernière mise à jour :** 2026-10-10
 > **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)**
@@ -57,6 +57,21 @@
         jamais l'éditeur du vault d'autrui) ; cache front invalidé à la
         création/révocation. +2 tests (recherche trouvée par le destinataire,
         aucune fuite vers un non-destinataire).
+  - [x] **Itération 3 (2026-10-11)** : le dossier Partage ne déclenche plus la
+        navigation (dossier virtuel — corrige « Directory not found: Partage ») ;
+        un fichier reçu s'ouvre **dans l'application en onglet** (arbre,
+        recherche, dashboard) via la résolution serveur
+        `home-<user>/Partage/<fichier>` → fichier source dans `/api/file`
+        (+ raw, download) ; validation `path` non-nulle à la création d'un
+        partage. +2 tests.
+  - [x] **Itération 4 (2026-10-11)** : le partage dirigé EST l'autorisation —
+        la résolution `Partage/…` court-circuite l'ACL du vault source (fix
+        « Accès refusé à la vault … » : le destinataire n'a par définition
+        pas accès au vault de l'émetteur) ; chemin virtuel canonique
+        `Partage/<token>/<nom>` (lève l'ambiguïté de fichiers homonymes,
+        fallback par nom conservé) ; « reçu » = adressé à moi (un admin ne
+        voit plus les partages d'autres comptes comme reçus — fix 404
+        « File not found: Partage/… »). +3 tests.
 
 ### 194. Dossier personnel par utilisateur (`<racine>/Home/<user>`)
 

docs/features/directed-shares-196.md

@@ -105,3 +105,39 @@ Décisions :
 
 +2 tests : `test_search_finds_received_share_for_recipient`,
 `test_search_does_not_leak_share_to_other_user`.
+
+
+## Itération 3 — ouverture en onglet (2026-10-11)
+
+- **Résolution serveur** : `_resolve_shared_file()` (`backend/routers/files_read.py`)
+  mappe `home-<user>/Partage/<fichier>` vers le fichier source pour les routes
+  `/api/file`, `/api/file/raw`, `/api/file/download`. Résolu AVANT l'ACL vault
+  (le home virtuel peut ne pas être dans `user.vaults`) — l'autorisation réelle
+  est l'appartenance au partage, vérifiée dans le resolver (destinataire seul).
+- **Front** : clic dossier Partage → simple dépliage (plus de `openNav`, fix
+  « Directory not found ») ; clic fichier reçu → `TabManager.openPreview/openPersistent`
+  (arbre, recherche, dashboard) — même parcours qu'un fichier ordinaire.
+- `POST /api/share` : `path` vide/null → 400 (un share `path: None` cassait
+  `/api/shares` en 500).
++2 tests : ouverture applicative du fichier reçu (file + raw), résolution
+scopée par utilisateur (home d'autrui → 403).
+
+
+## Itération 4 — correctifs d'ouverture (2026-10-11)
+
+- **Le partage EST l'autorisation** : après résolution `Partage/…`, l'ACL du
+  vault source n'est plus appliquée (`elif` dans `/api/file`, raw, download) —
+  un destinataire sans accès au vault émetteur ouvrait « Accès refusé à la
+  vault '…' », précisément le cas d'usage du partage. L'appartenance au
+  partage (vérifiée dans `_resolve_shared_file` via `list_shares(user=…)`)
+  reste la seule autorisation.
+- **Chemin canonique `Partage/<token>/<nom>`** : lève l'ambiguïté de deux
+  partages homonymes (le token identifie le partage exact). Fallback par nom
+  conservé pour les liens historiques. Transporté par l'arbre (`data-path`),
+  la recherche et le dashboard.
+- **« Reçu » = adressé à moi** : sidebar `_loadReceivedShares` et dashboard
+  filtrent désormais `shared_with.includes(me)` — un admin voyant TOUS les
+  partages (`/api/shares` non scopé pour lui) n'a plus les partages d'autres
+  comptes dans son dossier Partage (source des « File not found »).
++3 tests : destinataire sans accès au vault source (200), disambiguïsation
+par token, home d'autrui (403). 14 tests au total.

frontend/js/dashboard.js

@@ -62,14 +62,20 @@ const DashboardSharedWidget = {
       if (empty) empty.style.display = "none";
       const me = (AuthManager.getUser() || {}).username;
       grid.innerHTML = shares.map(s => {
-        const received = me && s.created_by && s.created_by !== me;
+        // Reçu = adressé à moi (et pas créé par moi). Un admin voit TOUS les
+        // partages via /api/shares — ceux des autres comptes ne sont pas
+        // « reçus » pour lui : ils ne se résolvent pas dans son Partage/.
+        const received = me && s.created_by && s.created_by !== me
+          && Array.isArray(s.shared_with) && s.shared_with.includes(me);
         const actions = received
           ? `<button class="shared-open-btn">${t('dashboard.open_btn')}</button>`
           : `<button class="shared-copy-btn" data-url="${window.location.origin}/s/${s.token}">${t('dashboard.copy_btn')}</button>
              <button class="shared-open-btn">${t('dashboard.open_btn')}</button>
              <button class="shared-revoke-btn" data-id="${s.id}">🗑</button>`;
+        const cardVault = received ? `home-${me}` : s.vault;
+        const cardPath = received ? `Partage/${(s.path || "").split("/").pop()}` : s.path;
         return `
-        <div class="shared-card" data-vault="${escapeHtml(s.vault)}" data-path="${escapeHtml(s.path)}" data-token="${s.token}" data-received="${received ? "1" : ""}">
+        <div class="shared-card" data-vault="${escapeHtml(cardVault)}" data-path="${escapeHtml(cardPath)}" data-token="${s.token}" data-received="${received ? "1" : ""}">
           <div class="shared-card-header">
             <i data-lucide="file-text" style="width:14px;height:14px"></i>
             <span class="shared-card-title">${escapeHtml(s.path.split("/").pop().replace(/\\.md$/i, ""))}</span>
@@ -93,10 +99,7 @@ const DashboardSharedWidget = {
       grid.querySelectorAll(".shared-open-btn").forEach(b => b.addEventListener("click", (e) => {
         e.stopPropagation();
         const card = b.closest(".shared-card");
-        if (card && card.dataset.received === "1" && card.dataset.token) {
-          window.open(`/s/${card.dataset.token}`, "_blank");
-          return;
-        }
+        // #196 : onglet applicatif — /api/file résout Partage/ vers la source.
         if (card) TabManager.openPreview(card.dataset.vault, card.dataset.path);
       }));
       grid.querySelectorAll(".shared-revoke-btn").forEach(b => b.addEventListener("click", async (e) => {
@@ -106,10 +109,6 @@ const DashboardSharedWidget = {
         this.load();
       }));
       grid.querySelectorAll(".shared-card").forEach(card => card.addEventListener("click", () => {
-        if (card.dataset.received === "1" && card.dataset.token) {
-          window.open(`/s/${card.dataset.token}`, "_blank");
-          return;
-        }
         TabManager.openPreview(card.dataset.vault, card.dataset.path);
       }));
     } catch (err) { if (empty) empty.style.display = ""; }

frontend/js/search.js

@@ -911,9 +911,10 @@ export function renderSearchResults(data, query, tagFilter) {
       });
       if (tagsDiv.children.length > 0) item.appendChild(tagsDiv);
     }
-    // #196 : document reçu par partage → ouvre la page de partage (lecture seule).
+    // #196 : document reçu par partage → onglet applicatif (/api/file résout
+    // Partage/ vers la source, lecture seule). Fallback /s/ si résolution KO.
     if (r.share_token) {
-      item.addEventListener("click", () => window.open(`/s/${r.share_token}`, "_blank"));
+      item.addEventListener("click", () => TabManager.openPreview(r.vault, r.path));
     } else {
       item.addEventListener("click", () => TabManager.openPreview(r.vault, r.path));
       item.addEventListener("dblclick", (e) => { e.preventDefault(); TabManager.openPersistent(r.vault, r.path); });

frontend/js/sidebar.js

@@ -32,7 +32,10 @@ async function _loadReceivedShares() {
   if (!me) return [];
   try {
     const shares = await api("/api/shares");
-    return shares.filter(s => s.created_by && s.created_by !== me);
+    // Adressés à moi uniquement : un admin voit tous les partages via
+    // /api/shares, ceux des autres ne sont pas dans SON Partage/.
+    return shares.filter(s => s.created_by && s.created_by !== me
+      && Array.isArray(s.shared_with) && s.shared_with.includes(me));
   } catch { return []; }
 }
 
@@ -366,16 +369,24 @@ async function incrementalLoadDirectory(vaultName, dirPath, container) {
       const shareSub = el("div", { class: "tree-children collapsed", id: `dir-${vaultName}-Partage` });
       item.shares.forEach((s) => {
         const name = (s.path || "").split("/").pop().replace(/\.md$/i, "");
-        const fItem = el("div", { class: "tree-item", "data-vault": vaultName, "data-path": `Partage/${s.path.split("/").pop()}`, "data-type": "file", "data-share-token": s.token }, [_sharedIcon(), el("span", { class: "tree-item-text" }, [document.createTextNode(name)])]);
+        const virtualPath = `Partage/${s.token}/${s.path.split("/").pop()}`;
+        const fItem = el("div", { class: "tree-item", "data-vault": vaultName, "data-path": virtualPath, "data-type": "file", "data-share-token": s.token }, [_sharedIcon(), el("span", { class: "tree-item-text" }, [document.createTextNode(name)])]);
         fItem.title = t("dashboard.shared_by", { user: s.created_by || "" });
         fItem.addEventListener("click", () => {
           scrollTreeItemIntoView(fItem, false);
-          window.open(`/s/${s.token}`, "_blank");
+          // #196 : ouverture en onglet applicatif — /api/file résout
+          // Partage/<token>/<nom> vers le fichier source (lecture seule).
+          if (window.innerWidth <= 768) {
+            TabManager.openPersistent(vaultName, virtualPath);
+          } else {
+            TabManager.openPreview(vaultName, virtualPath);
+          }
+          closeMobileSidebar();
         });
         shareSub.appendChild(fItem);
       });
       shareDir.addEventListener("click", () => {
-        TabManager.openNav(vaultName, "Partage");
+        // #196 : dossier virtuel — pas de navigation (pas de répertoire physique).
         const expanded = !shareSub.classList.contains("collapsed");
         shareSub.classList.toggle("collapsed", expanded);
         const chev = shareDir.querySelector("[data-lucide]");
@@ -729,16 +740,24 @@ async function loadDirectory(vaultName, dirPath, container) {
       const shareSub = el("div", { class: "tree-children collapsed", id: `dir-${vaultName}-Partage` });
       item.shares.forEach((s) => {
         const name = (s.path || "").split("/").pop().replace(/\.md$/i, "");
-        const fItem = el("div", { class: "tree-item", "data-vault": vaultName, "data-path": `Partage/${s.path.split("/").pop()}`, "data-type": "file", "data-share-token": s.token }, [_sharedIcon(), el("span", { class: "tree-item-text" }, [document.createTextNode(name)])]);
+        const virtualPath = `Partage/${s.token}/${s.path.split("/").pop()}`;
+        const fItem = el("div", { class: "tree-item", "data-vault": vaultName, "data-path": virtualPath, "data-type": "file", "data-share-token": s.token }, [_sharedIcon(), el("span", { class: "tree-item-text" }, [document.createTextNode(name)])]);
         fItem.title = t("dashboard.shared_by", { user: s.created_by || "" });
         fItem.addEventListener("click", () => {
           scrollTreeItemIntoView(fItem, false);
-          window.open(`/s/${s.token}`, "_blank");
+          // #196 : ouverture en onglet applicatif — /api/file résout
+          // Partage/<token>/<nom> vers le fichier source (lecture seule).
+          if (window.innerWidth <= 768) {
+            TabManager.openPersistent(vaultName, virtualPath);
+          } else {
+            TabManager.openPreview(vaultName, virtualPath);
+          }
+          closeMobileSidebar();
         });
         shareSub.appendChild(fItem);
       });
       shareDir.addEventListener("click", () => {
-        TabManager.openNav(vaultName, "Partage");
+        // #196 : dossier virtuel — pas de navigation (pas de répertoire physique).
         const expanded = !shareSub.classList.contains("collapsed");
         shareSub.classList.toggle("collapsed", expanded);
         const chev = shareDir.querySelector("[data-lucide]");

package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsigate",
-  "version": "2.67.0",
+  "version": "2.67.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": {

tests/test_directed_shares.py

@@ -167,3 +167,73 @@ class TestDirectedShareGate:
         resp = user_client.get("/api/search", params={"q": "dirigé", "vault": "all"})
         assert resp.status_code == 200
         assert not [r for r in resp.json()["results"] if r.get("share_token")]
+
+    def test_recipient_opens_shared_file_in_app(self, sessions):
+        # #196 : /api/file résout home-<user>/Partage/<token>/<f> vers la
+        # source — même quand le destinataire N'A PAS accès au vault source
+        # (le partage dirigé EST l'autorisation).
+        admin_client, user_client = sessions
+        path = _ensure_shared_file()
+        resp = admin_client.post("/api/share/TestVault", json={
+            "path": path, "shared_with": ["normaluser"],
+        })
+        assert resp.status_code == 200
+        token = resp.json()["token"]
+
+        # Retire l'accès direct au vault source : le partage doit suffire.
+        from backend.auth.user_store import update_user
+        update_user("normaluser", {"vaults": ["home-normaluser"]})
+
+        # Chemin canonique avec token
+        r = user_client.get("/api/file/home-normaluser", params={"path": f"Partage/{token}/share_directed.md"})
+        assert r.status_code == 200, r.text
+        body = r.json()
+        assert body["is_markdown"] is True
+        assert "dirigé" in body["html"]
+
+        # Fallback par nom (liens sans token)
+        r = user_client.get("/api/file/home-normaluser", params={"path": "Partage/share_directed.md"})
+        assert r.status_code == 200, r.text
+
+        # raw + download too
+        r = user_client.get("/api/file/home-normaluser/raw", params={"path": f"Partage/{token}/share_directed.md"})
+        assert r.status_code == 200
+        assert "dirigé" in r.json()["raw"]
+
+        # Restore fixture state for other tests
+        update_user("normaluser", {"vaults": ["TestVault"]})
+
+    def test_shared_file_resolution_is_user_scoped(self, sessions):
+        # home-admin/Partage/x.md demandé par normaluser → pas de mapping
+        # (le home d'un autre user lui est interdit, 403 avant tout).
+        admin_client, user_client = sessions
+        path = _ensure_shared_file()
+        admin_client.post("/api/share/TestVault", json={
+            "path": path, "shared_with": ["normaluser"],
+        })
+        r = user_client.get("/api/file/home-admin", params={"path": "Partage/share_directed.md"})
+        assert r.status_code == 403
+
+    def test_same_name_two_shares_token_disambiguates(self, sessions):
+        # Deux partages de fichiers homonymes reçus : le token sélectionne
+        # le bon, le fallback par nom prend le premier sans crash.
+        admin_client, user_client = sessions
+        import os as _os
+        vault = Path(_os.environ["VAULT_1_PATH"])
+        (vault / "a.md").write_text("contenu alpha unique", encoding="utf-8")
+        (vault / "b.md").write_text("contenu beta unique", encoding="utf-8")
+        # subdirectory with same basename to force ambiguity
+        (vault / "d1").mkdir(exist_ok=True)
+        (vault / "d2").mkdir(exist_ok=True)
+        (vault / "d1" / "same.md").write_text("contenu GAMMA unique", encoding="utf-8")
+        (vault / "d2" / "same.md").write_text("contenu DELTA unique", encoding="utf-8")
+        r1 = admin_client.post("/api/share/TestVault", json={"path": "d1/same.md", "shared_with": ["normaluser"]})
+        r2 = admin_client.post("/api/share/TestVault", json={"path": "d2/same.md", "shared_with": ["normaluser"]})
+        t1, t2 = r1.json()["token"], r2.json()["token"]
+
+        r = user_client.get("/api/file/home-normaluser", params={"path": f"Partage/{t2}/same.md"})
+        assert r.status_code == 200
+        assert "DELTA" in r.json()["html"]
+        r = user_client.get("/api/file/home-normaluser", params={"path": f"Partage/{t1}/same.md"})
+        assert r.status_code == 200
+        assert "GAMMA" in r.json()["html"]