mediawiki: Suchfehler lesbar statt als Stacktrace (0.1.1)

Gleichstand mit inmedias.it/dokumentierer d1eeb94. Eine von CirrusSearch abgelehnte Suchanfrage liess mwclient.errors.APIError bis nach oben durchschlagen - der Benutzer sah einen Stacktrace statt einer Meldung. cmd_search faengt den Fehler jetzt ab und beendet mit Code 1. Fuer cirrussearch-parse-error gibt es einen eigenen Hinweis: "insource:wort" funktioniert, die Regex-Form "insource:/muster/" nicht. Letztere braucht das Elasticsearch-Plugin wikimedia-extra, das auf unserer Instanz fehlt. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-10-06 11:51:07 +02:00
5 changed files with 21 additions and 156 deletions
@@ -1,6 +1,6 @@
 {
  "name": "humhub",
-  "version": "0.4.1",
+  "version": "0.3.0",
  "description": "HumHub via REST-API: Wiki-Seiten, Posts, Umfragen und Kommentare auflisten, anzeigen, anlegen und aktualisieren (humhub.py).",
  "author": {
    "name": "inmedias.it"
@@ -94,50 +94,18 @@ und Wiki-Seiten.
 ### Direktnachrichten

 ```bash
-./humhub.py mail-send  <empfaenger> "<betreff>" [datei|-]          # Probelauf
-./humhub.py mail-send  <empfaenger> "<betreff>" [datei|-] --send   # senden
+./humhub.py mail-send  <empfaenger> "<betreff>" [datei|-]
 ./humhub.py mail-send  https://humhub.inmedias.it/u/serge/ "Betreff" entwurf.md
-./humhub.py mail-send  jan.stobbe,26 "Betreff" - --send
+./humhub.py mail-send  jan.stobbe,26 "Betreff" -
 ./humhub.py mail-reply <konversations-id> [datei|-]
-./humhub.py mail-user-remove <konversations-id> <empfaenger>
 ```

 Empfänger sind Benutzername, Profil-URL oder numerische ID; mehrere mit Komma.
-**Ohne `--send` wird nichts gesendet**, sondern nur aufgelöst und angezeigt,
-wer die Nachricht bekäme — der Probelauf ist der Default.
-
-**`recipient` will GUIDs, keine User-IDs.** `mail-send` löst jede Angabe
-deshalb bis zur GUID auf (`/user/<id>` liefert sie, die Benutzerliste je nach
-Fassung nicht). Mit numerischen IDs antwortet die API `400 Validation failed`
-und „Du kannst keine Nachricht ohne Empfänger senden!" — die Meldung klingt
-nach einem leeren Feld, obwohl das Feld gefüllt ist, und weist damit in die
-falsche Richtung.

 `mail-reply` hängt einen Beitrag an eine bestehende Konversation
 (`POST /mail/<id>/entry`). Die ID steht in der URL der Konversation:
 `…/mail/mail/index?id=35`.

-**Eine Fehlzustellung lässt sich nicht löschen.** `DELETE /mail/<id>` gibt es
-in dieser API-Version nicht (404, Route unbekannt), und über die Oberfläche
-kommt man auch nicht dran. Das einzige Gegenmittel ist
-`mail-user-remove`, das einen Teilnehmer wieder aus der Konversation nimmt
-(`DELETE /mail/<id>/user/<userid>`). Ob der Empfänger vorher schon eine
-Benachrichtigung bekommen hat, bleibt dabei offen.
-
-**Deshalb prüft `mail-send` nach dem Versand die Teilnehmerliste.** Eine
-falsche Kennung in `recipient` führt nicht verlässlich zu einer
-Fehlermeldung: am 25.09.2026 hat die API einen Versand mit numerischen IDs
-mit `HTTP 200` quittiert und die Nachricht an einen **anderen** Benutzer
-zugestellt. Der Befehl gibt deshalb aus, wer die Konversation tatsächlich
-bekommen hat, und nennt bei einer Abweichung das `mail-user-remove`-Kommando.
-Die Teilnehmerliste kann direkt nach dem Anlegen unvollständig sein — im
-Zweifel noch einmal nachsehen, bevor man daraus einen Schluss zieht.
-
-Zwei Routen, zwei Kennungen: `POST /mail` will die **GUID**,
-`DELETE /mail/<id>/user/<userid>` die **numerische ID**. Beides ist im
-Skript gekapselt, `mail-user-remove` nimmt wie alle Befehle Benutzername,
-Profil-URL oder ID.
-
 **Slug ≠ Benutzername.** Zur Profil-URL `/u/serge/` gehört der Benutzername
 `Serge`, und `get-by-username` unterscheidet Groß-/Kleinschreibung. Wer den
 Slug einsetzt, bekommt sonst `User not found!`. `mail-send` fängt das ab: erst
@@ -155,14 +123,6 @@ Container-ID, einen **Space-Slug** oder eine **Space-URL** (`resolve_container`
 löst Slug/URL selbst auf). Passende Container per `space-list` ermitteln, Seiten
 per `wiki-list` / `post-list`.

-**HumHub macht aus Zeichenfolgen wie `:)` Emojis — auch mitten im Fachtext.**
-Am 08.10.2026 wurde aus „(0.0.0.0 und ::)“ ein „(0.0.0.0 und :😃“: der zweite
-Doppelpunkt der IPv6-Adresse `::` bildete mit der schließenden Klammer ein
-Smiley. Betroffen sind typisch IPv6-Adressen (`::`, `::1`, `[::]:3306`),
-Portangaben und Pfade direkt vor `)` oder `(`. **Solche Werte immer in
-Backticks setzen** — in Inline-Code ersetzt HumHub nichts. Vor dem Absenden
-den Text außerhalb von Backticks auf `:)`, `:(`, `;)`, `:D`, `:P` prüfen.
-
 ## Zugangsdaten

 `humhub.py` bringt **keinen** Token mit. Jede/r hinterlegt ein eigenes
@@ -350,20 +350,8 @@ def cmd_comment_create(args: argparse.Namespace) -> None:

 # ── Mail (Direktnachrichten) ─────────────────────────────────────────────────

-def user_guid(session: requests.Session, user: dict) -> str:
-    """GUID eines Benutzers, notfalls über /user/<id> nachgeladen.
-
-    Die Benutzerliste liefert die GUID je nach Fassung nicht mit.
-    """
-    guid = user.get("guid") or (user.get("account") or {}).get("guid")
-    if guid:
-        return str(guid)
-    uid = user.get("id") or (user.get("account") or {}).get("id")
-    return str(api_get(session, f"/user/{uid}")["guid"])
-
-
-def resolve_user_info(session: requests.Session, value: str) -> dict:
-    """Benutzerdatensatz zu numerischer ID, Benutzername oder Profil-URL.
+def resolve_user(session: requests.Session, value: str) -> int:
+    """Resolve a user ID, username, or profile URL (…/u/<slug>/) to a user ID.

    Slug und Benutzername sind nicht dasselbe: zu https://…/u/serge/ gehört der
    Benutzername "Serge". `get-by-username` kennt nur den Benutzernamen und
@@ -372,7 +360,7 @@ def resolve_user_info(session: requests.Session, value: str) -> dict:
    """
    value = value.strip()
    if value.isdigit():
-        return api_get(session, f"/user/{int(value)}")
+        return int(value)
    name = value
    if "/" in value:
        teile = [t for t in urlparse(value).path.split("/") if t]
@@ -382,7 +370,7 @@ def resolve_user_info(session: requests.Session, value: str) -> dict:
    r = session.get(f"{BASE_URL}/user/get-by-username",
                    params={"username": name})
    if r.ok:
-        return r.json()
+        return int(r.json()["id"])

    page = 1
    while True:
@@ -393,107 +381,26 @@ def resolve_user_info(session: requests.Session, value: str) -> dict:
            slug_teile = [t for t in urlparse(u.get("url", "")).path.split("/") if t]
            slug = slug_teile[-1] if slug_teile else ""
            if name.lower() in (username.lower(), slug.lower()):
-                return u
+                return int(u["id"])
        if len(results) < 50:
            break
        page += 1
    sys.exit(f"Error: Benutzer {value!r} nicht gefunden")


-def resolve_user(session: requests.Session, value: str) -> str:
-    """Resolve a user ID, username, or profile URL (…/u/<slug>/) to a user GUID.
-
-    **GUID, nicht ID.** `POST /mail` nimmt als `recipient` ausschließlich
-    GUIDs an; mit numerischen IDs antwortet die API `400 Validation failed`
-    und „Du kannst keine Nachricht ohne Empfänger senden!" — eine Meldung,
-    die nach einem leeren Feld klingt, obwohl das Feld gefüllt ist.
-    """
-    return user_guid(session, resolve_user_info(session, value))
-
-
-def mail_teilnehmer(session: requests.Session, conv_id: int) -> list[dict]:
-    """Teilnehmer einer Konversation. Leere Liste, wenn nicht lesbar."""
-    r = session.get(f"{BASE_URL}/mail/{conv_id}/users")
-    if not r.ok:
-        return []
-    daten = r.json()
-    return daten if isinstance(daten, list) else daten.get("results", [])
-
-
-def eigene_guid(session: requests.Session) -> str:
-    """GUID des eigenen Kontos, leer wenn nicht ermittelbar."""
-    r = session.get(f"{BASE_URL}/user/me")
-    return str(r.json().get("guid", "")) if r.ok else ""
-
-
 def cmd_mail_send(args: argparse.Namespace) -> None:
    session, _ = get_session()
-    leute = [resolve_user_info(session, teil)
-             for teil in args.to.split(",") if teil.strip()]
-    if not leute:
+    ids = [resolve_user(session, teil) for teil in args.to.split(",") if teil.strip()]
+    if not ids:
        sys.exit("Error: kein Empfänger angegeben")
-    guids = [user_guid(session, u) for u in leute]
-    namen = {g: u.get("display_name", "?") for g, u in zip(guids, leute)}
-    for g in guids:
-        print(f"Empfänger: {namen[g]} ({g})")
-    if not args.send:
-        print("PROBELAUF — nichts gesendet. Mit --send abschicken.")
-        return
-
    message = read_file(args.file)
    data = api_post(session, "/mail", {
        "title": args.title,
        "message": message,
-        "recipient": guids,
+        "recipient": ids,
    })
-    conv_id = data.get("id")
-    print(f"Nachricht gesendet (Konversation {conv_id})")
-
-    # Gegenprobe. Am 25.09.2026 hat die API eine Nachricht mit numerischen IDs
-    # in `recipient` mit HTTP 200 quittiert und sie an einen *anderen*
-    # Benutzer zugestellt — sie meldet einen falschen Empfaenger also nicht
-    # zwingend als Fehler. Deshalb nach jedem Versand nachsehen, wer die
-    # Konversation tatsaechlich bekommen hat. Loeschen laesst sie sich nicht
-    # (`DELETE /mail/<id>` gibt es nicht), nur Teilnehmer entfernen.
-    teilnehmer = mail_teilnehmer(session, conv_id)
-    if not teilnehmer:
-        print("ACHTUNG: Teilnehmerliste nicht lesbar — Zustellung nicht geprüft.")
-        return
-    print("Teilnehmer laut Server:")
-    for u in teilnehmer:
-        print(f"  {u.get('display_name')} ({u.get('guid')})")
-    drin = {str(u.get("guid")) for u in teilnehmer}
-    ich = eigene_guid(session)
-    fehlend = [g for g in guids if g not in drin]
-    fremd = [u for u in teilnehmer
-             if str(u.get("guid")) not in guids and str(u.get("guid")) != ich]
-    if fehlend:
-        print("ACHTUNG: nicht zugestellt an " +
-              ", ".join(f"{namen[g]} ({g})" for g in fehlend))
-    if fremd:
-        print("ACHTUNG: unerwartete Teilnehmer — entfernen mit:")
-        for u in fremd:
-            print(f"  ./humhub.py mail-user-remove {conv_id} {u.get('id')}")
-
-
-def cmd_mail_user_remove(args: argparse.Namespace) -> None:
-    """Nimmt einen Teilnehmer aus einer Konversation.
-
-    Der einzige Weg, eine Fehlzustellung zurueckzunehmen: die Konversation
-    selbst laesst sich weder ueber die API noch ueber die Oberflaeche
-    loeschen. Anders als beim Versand will diese Route die **numerische ID**,
-    nicht die GUID.
-    """
-    session, _ = get_session()
-    user = resolve_user_info(session, args.user)
-    uid = user.get("id")
-    r = session.delete(f"{BASE_URL}/mail/{args.id}/user/{uid}")
-    if not r.ok:
-        sys.exit(f"Error {r.status_code}: {r.text[:200]}")
-    print(f"{user.get('display_name')} (ID {uid}) aus Konversation "
-          f"{args.id} entfernt. Verbleibend:")
-    for u in mail_teilnehmer(session, args.id):
-        print(f"  {u.get('display_name')} ({u.get('guid')})")
+    empfaenger = ", ".join(str(i) for i in ids)
+    print(f"Nachricht gesendet (Konversation {data.get('id')}) an Benutzer {empfaenger}")


 def cmd_mail_reply(args: argparse.Namespace) -> None:
@@ -681,17 +588,8 @@ def main() -> None:
    p.add_argument("to", help="Benutzername, Profil-URL oder ID; mehrere mit Komma")
    p.add_argument("title", help="Betreff")
    p.add_argument("file", help="Markdown-Datei oder '-' für stdin")
-    p.add_argument("--send", action="store_true",
-                   help="wirklich senden (ohne: Probelauf mit Empfängernamen)")
    p.set_defaults(func=cmd_mail_send)

-    # mail-user-remove
-    p = sub.add_parser("mail-user-remove",
-                       help="Teilnehmer aus einer Konversation entfernen (Fehlzustellung)")
-    p.add_argument("id", type=int, help="Konversations-ID")
-    p.add_argument("user", help="Benutzername, Profil-URL oder ID")
-    p.set_defaults(func=cmd_mail_user_remove)
-
    # mail-reply
    p = sub.add_parser("mail-reply", help="In einer bestehenden Konversation antworten")
    p.add_argument("id", type=int, help="Konversations-ID (aus /mail/mail/index?id=NN)")
@@ -1,6 +1,6 @@
 {
  "name": "mediawiki",
-  "version": "0.1.0",
+  "version": "0.1.1",
  "description": "MediaWiki-Seiten/Templates lesen, rendern und schreiben via wiki.py (fetch/update/render/search, Instanzen wiki & devwiki).",
  "author": {
    "name": "inmedias.it"
@@ -88,6 +88,7 @@ def cmd_search(args: argparse.Namespace) -> None:
            sys.exit(f"Error: unknown namespace '{ns_key}'. "
                     f"Known: all, {', '.join(NAMESPACES)}")

+    try:
        result = site.api(
            "query", list="search",
            srsearch=args.query,
@@ -95,6 +96,12 @@ def cmd_search(args: argparse.Namespace) -> None:
            srlimit=50,
            srprop="title|snippet",
        )
+    except mwclient.errors.APIError as exc:
+        if exc.code == "cirrussearch-parse-error":
+            sys.exit(f"Error: Suchanfrage nicht verstanden: {args.query!r}\n"
+                     f"Hinweis: 'insource:wort' funktioniert, die Regex-Form "
+                     f"'insource:/muster/' ist auf dieser Instanz nicht freigeschaltet.")
+        sys.exit(f"Error: Suche fehlgeschlagen ({exc.code}): {exc.info}")
    hits = result["query"]["search"]
    if not hits:
        print("(keine Ergebnisse)", file=sys.stderr)