Merge: words_pictures natives Directus M2M

- GET: Deep-Query über pictures.linked_words statt manuelle Junction-Abfrage
- POST: PATCH /items/pictures/{id} mit linked_words.create statt _ensure_link
- _ensure_junction/_ensure_link für words_pictures entfernt
- Setup-Logik in _setup_words_pictures() ausgelagert (idempotent)
- Batch-Insert aller neuen Links in einem einzigen PATCH-Call

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

app.py

@@ -113,60 +113,117 @@ def directus_object(obj_id):
     return jsonify(data), status
 
 
+def _setup_words_pictures(token: str):
+    """Richtet M2M-Relation words ↔ pictures idempotent ein (läuft einmalig)."""
+    _directus("PATCH", "/fields/pictures/linked_words", token, {
+        "type": "alias",
+        "meta": {"special": ["m2m"], "interface": "list-m2m",
+                 "options": {"template": "{{words_id.title_de}}"},
+                 "note": "Verknüpfte Safe Words"},
+    })
+    _directus("PATCH", "/fields/words/linked_pictures", token, {
+        "type": "alias",
+        "meta": {"special": ["m2m"], "interface": "list-m2m",
+                 "options": {"template": "{{pictures_id.media}}"},
+                 "note": "Verknüpfte Bilder"},
+    })
+    _directus("POST", "/relations", token, {
+        "collection": "words_pictures", "field": "words_id",
+        "related_collection": "words",
+        "schema": {"on_delete": "CASCADE"},
+        "meta": {"junction_field": "pictures_id", "one_field": "linked_pictures",
+                 "one_deselect_action": "nullify"},
+    })
+    _directus("POST", "/relations", token, {
+        "collection": "words_pictures", "field": "pictures_id",
+        "related_collection": "pictures",
+        "schema": {"on_delete": "CASCADE"},
+        "meta": {"junction_field": "words_id", "one_field": "linked_words",
+                 "one_deselect_action": "nullify"},
+    })
+
+
 @app.route("/api/directus/pictures/<pic_id>/words", methods=["GET", "POST"])
 def directus_picture_words(pic_id):
-    """Proxy: Safe-Words eines Bildes laden (GET) oder speichern (POST)."""
+    """Proxy: Safe-Words eines Bildes laden (GET) oder speichern (POST).
+    Nutzt natives Directus M2M über pictures.linked_words.
+    """
     token = request.headers.get("Authorization", "")
 
     if request.method == "GET":
-        junc, _ = _directus(
+        # Natives Directus Deep-Query über M2M-Relation
+        data, status = _directus(
             "GET",
-            f"/items/words_pictures?filter[pictures_id][_eq]={pic_id}&fields=id,words_id&limit=500",
+            f"/items/pictures/{pic_id}"
+            f"?fields[]=linked_words.id"
+            f"&fields[]=linked_words.words_id.id"
+            f"&fields[]=linked_words.words_id.title_de"
+            f"&fields[]=linked_words.words_id.level"
+            f"&fields[]=linked_words.words_id.status",
             token,
         )
-        w_ids = [e["words_id"] for e in (junc.get("data") or []) if e.get("words_id")]
-        if not w_ids:
+        if status != 200:
             return jsonify({"data": []})
-        ids_param = urllib.parse.quote(",".join(w_ids), safe="")
-        w_data, _ = _directus(
-            "GET",
-            f"/items/words?filter[id][_in]={ids_param}&filter[status][_neq]=archived&fields=id,title_de,level,status&limit=500",
-            token,
-        )
-        junc_by_word = {e["words_id"]: e["id"] for e in (junc.get("data") or [])}
-        items = [
-            {
-                "id": junc_by_word.get(w["id"], ""),
-                "word_id": w["id"],
-                "title_de": w["title_de"],
-                "level": w.get("level") or 50,
-                "status": w.get("status", ""),
-            }
-            for w in (w_data.get("data") or [])
-        ]
+        items = []
+        for entry in ((data.get("data") or {}).get("linked_words") or []):
+            word = entry.get("words_id") or {}
+            if not isinstance(word, dict) or not word.get("id"):
+                continue
+            if word.get("status") == "archived":
+                continue
+            items.append({
+                "id": entry.get("id", ""),
+                "word_id": word["id"],
+                "title_de": word.get("title_de", ""),
+                "level": word.get("level") or 50,
+                "status": word.get("status", ""),
+            })
         return jsonify({"data": items})
 
     else:  # POST
         body = request.get_json(force=True, silent=True) or {}
-        words = body.get("words", [])
-        _ensure_junction("words_pictures", "words_id", "pictures_id", token)
+        words_to_save = body.get("words", [])
+
+        # Relationen einmalig sicherstellen
+        _setup_words_pictures(token)
+
+        # Bereits verknüpfte Word-IDs laden (Duplikat-Schutz)
+        existing_data, _ = _directus(
+            "GET",
+            f"/items/pictures/{pic_id}?fields[]=linked_words.words_id",
+            token,
+        )
+        existing_ids = set()
+        for e in ((existing_data.get("data") or {}).get("linked_words") or []):
+            wid = e.get("words_id")
+            if wid:
+                existing_ids.add(wid if isinstance(wid, str) else wid.get("id", ""))
+
+        # Wörter anlegen/finden, Level updaten, neue Links sammeln
+        new_links = []
         saved = 0
-        for entry in words:
+        for entry in words_to_save:
             title_de = (entry.get("title_de") or "").strip()
             level = int(entry.get("level") or 50)
             if not title_de:
                 continue
             try:
-                wid, _ = _find_or_create_word(title_de, level, token)
-                _ensure_link(
-                    "words_pictures",
-                    {"words_id": wid, "pictures_id": pic_id},
-                    {"words_id": wid, "pictures_id": pic_id},
-                    token,
-                )
+                wid, is_new = _find_or_create_word(title_de, level, token)
+                if not is_new:
+                    _directus("PATCH", f"/items/words/{wid}", token, {"level": level})
+                if wid not in existing_ids:
+                    new_links.append({"words_id": wid})
+                    existing_ids.add(wid)
                     saved += 1
             except Exception as e:
                 print(f"[picture_words] error for '{title_de}': {e}")
+
+        # Alle neuen Links in einem einzigen Directus-PATCH
+        if new_links:
+            _directus("PATCH", f"/items/pictures/{pic_id}", token, {
+                "linked_words": {"create": new_links}
+            })
+
         return jsonify({"ok": True, "saved": saved})
 
 
@@ -1572,6 +1629,64 @@ def setup_directus_schema():
                     "failed": len(failed), "results": results})
 
 
+@app.route("/api/setup-words-pictures", methods=["POST"])
+def setup_words_pictures():
+    """
+    Einmalig ausführen: Konfiguriert die M2M-Relation words ↔ pictures
+    via words_pictures Junction in Directus.
+    Idempotent – bereits vorhandene Felder/Relationen werden übersprungen.
+    """
+    token = request.headers.get("Authorization", "")
+    results = []
+
+    def do(label, method, path, body=None):
+        data, status = _directus(method, path, token, body)
+        ok = status in (200, 201, 204)
+        results.append({"label": label, "status": status, "ok": ok,
+                         "err": None if ok else (data.get("errors") or data)})
+        return ok
+
+    # Sicherstellen dass Junction-Collection existiert
+    _ensure_junction("words_pictures", "words_id", "pictures_id", token)
+
+    # special=m2m auf Alias-Felder setzen (idempotent)
+    do("patch pictures.linked_words special", "PATCH", "/fields/pictures/linked_words", {
+        "type": "alias",
+        "meta": {"special": ["m2m"], "interface": "list-m2m",
+                 "options": {"template": "{{words_id.title_de}}"},
+                 "note": "Verknüpfte Safe Words"},
+    })
+    do("patch words.linked_pictures special", "PATCH", "/fields/words/linked_pictures", {
+        "type": "alias",
+        "meta": {"special": ["m2m"], "interface": "list-m2m",
+                 "options": {"template": "{{pictures_id.media}}"},
+                 "note": "Verknüpfte Bilder"},
+    })
+
+    # Relation words_pictures.words_id → words
+    do("relation words_pictures.words_id→words", "POST", "/relations", {
+        "collection": "words_pictures", "field": "words_id",
+        "related_collection": "words",
+        "schema": {"on_delete": "CASCADE"},
+        "meta": {"junction_field": "pictures_id",
+                 "one_field": "linked_pictures",
+                 "one_deselect_action": "nullify"},
+    })
+    # Relation words_pictures.pictures_id → pictures
+    do("relation words_pictures.pictures_id→pictures", "POST", "/relations", {
+        "collection": "words_pictures", "field": "pictures_id",
+        "related_collection": "pictures",
+        "schema": {"on_delete": "CASCADE"},
+        "meta": {"junction_field": "words_id",
+                 "one_field": "linked_words",
+                 "one_deselect_action": "nullify"},
+    })
+
+    failed = [r for r in results if not r["ok"]]
+    return jsonify({"ok": len(failed) == 0, "total": len(results),
+                    "failed": len(failed), "results": results})
+
+
 if __name__ == "__main__":
     app.run(host="0.0.0.0", port=8000, debug=True)
 

frontend/src/pages/DrawIt.tsx

@@ -30,6 +30,7 @@ export default function DrawIt() {
 
   const [pictureList, setPictureList] = useState<DirectusPicture[]>([])
   const [currentIndex, setCurrentIndex] = useState(-1)
+  const [debouncedIndex, setDebouncedIndex] = useState(-1)
   const [objects, setObjects] = useState<DirectusObject[]>([])
   const [selectedObjectId, setSelectedObjectId] = useState<string | null>(null)
   const [currentSelections, setCurrentSelections] = useState<Selection[]>([])
@@ -52,6 +53,12 @@ export default function DrawIt() {
 
   const canvasRef = useRef<DrawCanvasHandle>(null)
 
+  // Debounce: Bild erst laden wenn 350ms keine weitere Navigation
+  useEffect(() => {
+    const t = setTimeout(() => setDebouncedIndex(currentIndex), 350)
+    return () => clearTimeout(t)
+  }, [currentIndex])
+
   useEffect(() => {
     if (safeWordInputVisible) safeWordInputRef.current?.focus()
   }, [safeWordInputVisible])
@@ -83,8 +90,9 @@ export default function DrawIt() {
     }
   }
 
+  // currentPicture folgt dem debouncedIndex → lädt erst wenn Navigation pausiert
   const currentPicture: DirectusPicture | null =
-    currentIndex >= 0 && currentIndex < pictureList.length ? pictureList[currentIndex] : null
+    debouncedIndex >= 0 && debouncedIndex < pictureList.length ? pictureList[debouncedIndex] : null
 
   // Map DirectusObject → CanvasObject for rendering
   const canvasObjects: CanvasObject[] = objects.map((obj, i) => ({