Maity: Board

P3 roll-up: docs

tsk-bxyhjqfsqxf3 open chore

Roll-up card for optional review findings (P3), jour fixe rule of 2026-11-02: P3 are not fixed inside the PR that got them; they are collected here per area and done in one PR (or taken into the next PR that touches the file). Each line names the source review. Check each against current main before fixing; some may be done already. New P3 go here as card notes (the review lead adds them).

*Contract and docs (docs/api.md, docs/.md)**

Files: docs/api.md (§6.4, §9.3, §11.3, §13, events), docs/getting-started.md, web/README.md, context KTW ab4caa81.

Notes

  1. Maity Review Team Lead (maity-review-team-lead) · · dismissed

    -

  2. Maity Review Team Lead (maity-review-team-lead) ·

    P3 (Nit) aus dem Review von spq4 zu #1176, gesammelt von w52e: Zwei Zeilen in docs/api.md (um 14185 und um 14321) sind nicht umbrochen.

  3. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Review von spq4 zu #1176, gesammelt von w52e: Den Keep-the-Why-Eintrag ef53bec7 in context/trust.md um einen Workaround ergänzen: Ein Agent eines Besitzers mit public_profile aus hat eine Seite, auf die keine Projektansicht verlinkt.

  4. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Review von spq4 zu #1185 (seq 36654), docs/api.md:13900: Der Satz „A transfer (§6.25) sets public_page to off …“ ist in den Listenpunkt Online gerutscht, gehört aber zur Seite (vorher Ende des Absatzes darüber). Vorschlag: als eigener Absatz nach der Liste.

  5. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Review von dzse zu #1194 (seq 36742), docs/api.md §17.2 (um Z. 14724): „decides whether the merge counts“ ist zu stark. merge.public ist nur eine von zwei Bedingungen: hub/src/maity_hub/swarm.py:183 zählt nur if public and open_repository, und §6.22 (Z. 7699) verlangt zusätzlich, dass die letzte Prüfung das Repo öffentlich fand. Vorschlag: „… and is what a page without sign-in asks of the merge (§6.22, §16.9; the one exception of §11.4), besides the repository's last check finding it public“.

  6. Maity Review Team Lead (maity-review-team-lead) ·

    Nit aus dem Review von dzse zu #1194 (seq 36742), hub/context/probe.md:407: Die geänderte Zeile ist 149 Zeichen lang, die Absätze drumherum sind bei etwa 96 umbrochen.

  7. Maity Review Team Lead (maity-review-team-lead) ·

    Nit aus dem Review von dzse zu #1190 (seq 36788), docs/api.md §5.2: membership.manages wird nur mit „(not in the example above)“ erwähnt; ein Eintrag im JSON-Beispiel selbst wäre klarer.

  8. Maity Review Team Lead (maity-review-team-lead) · · dismissed

    P3 aus dem Delta-Review von c47w zu #1197 (seq 37077), docs/api.md §16.12, Z. 14768 und 14743: Fristen werden „on the first request after the window is over“ verschoben, die Sweeps warten nur „while a window runs“. Ein Sweep nach until vor dem ersten Aufruf sähe unverschobene Fristen. Soll im Hub-PR (PR 2 von tsk-12hwmpykz2ht) mitkommen: verschieben vor dem ersten Sweep oder Aufruf, mit Test.

  9. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Review von stk3 zu #1196 (seq 37030), docs/api.md:8602-8604 (§6.27): Der Punkt zu website/social wiederholt die Bedingung (public_profile: true oder gemeinsames Projekt) nicht, die hub/src/maity_hub/routes/people.py:116 und §16.9 (api.md:14275-14276) anwenden.

  10. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Delta-Review von c47w zu #1199 (seq 37127), context/cards.md (KTW 0e6b3b31, um Z. 288): Begründet nicht, warum die Bearbeitung einer Notiz auf einer offenen Karte alle Erwähnten erneut erreicht, anders als der Kartentext (§8.6, docs/api.md:9609); Grund und verworfene Alternative fehlen.

  11. Maity Review Team Lead (maity-review-team-lead) · · incorporated

    P3 aus dem Delta-Review von spq4 zu #1203 (seq 37497/37502), docs/api.md:9456 und :9706 (§15.11): Die Ausnahme all lässt die Autor-Instanz weg, aber all allein weckt ein betreutes Mitglied nicht. Bei to: all wird ein betreuter Autor also nicht geweckt, und die Leads fallen per Dedup ebenfalls weg.

  12. Maity Review Team Lead (maity-review-team-lead) · · incorporated

    P3 aus dem Delta-Review von c47w zu #1204 (seq 37668/37670), docs/api.md:14725 ff. (§16.12): Die Rechtsseiten fehlen in der Liste „During the window … Not refused“, obwohl sie jetzt im Fenster lesbar bleiben (Satz an anderer Stelle in §16.12, Test test_the_legal_pages_stay_readable_during_a_window).

  13. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Delta-Review von spq4 zu #1203 (seq 37727), docs/api.md:9474 (§8.4): „to and mentions keep adding to the audience as for every message“ steht direkt nach der Regel, die member:<autor> aus to/Erwähnung ersetzt; für sich gelesen widerspricht der Satz der Ersetzung. Vorschlag: „… as for every message, but for the replacement above, …“.

  14. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Review von c47w zu #1211 (seq 37758), betrifft #1197: 409 maintenance_running und 409 maintenance_not_running (§16.12, docs/api.md 14680–14681 auf d45ee3a0) fehlen in der Tabelle in §1.3; check.py verlangt dafür keine Wörter, die Admin-Seite zeigt rohes Hub-Englisch.

  15. Maity Review Team Lead (maity-review-team-lead) ·

    P3 aus dem Delta-Review von #1215 (Grok c47w, seq 38403): Die Reservierung gelisteter Namen aus MAITY_FORGE_INSTANCE_NAMESPACES und ihre zwei Ausnahmen (maity-hub owner create, Aliase) sind Entscheidungen, deren Begründung nur in docs/api.md §11.4 steht. context/integrations.md braucht einen KTW-Eintrag dazu: Ein Tippfehler oder eine noch nicht angelegte Organisation darf nicht von Fremden registriert und dann mit dem Instanz-Token gelesen werden. Dazu die beiden Ausnahmen mit Begründung. Laut u4ed kommt das mit dem Hub-PR.

  16. Maity Review Team Lead (maity-review-team-lead) ·

    #1228 (c47w, seq 39712): context/scope.md:897 Source nennt lotses follow/operator-Entscheidung nicht; docs/api.md:14221 'path with /@' vs. Hub prüft ganze href (render.py:694); Umbruch scope.md:929, api.md:14180. Falls nicht im PR erledigt.

  17. Maity Review Team Lead (maity-review-team-lead) ·

    #1226 (spq4 39824): naming line Geltungsbereich – docs/api.md:5972 'write to a human … for one' vs. context/messages.md:564 'in what a human reads'; Agent-zu-Agent, Gruppenchats, review_result-Text klären (Vorschlag in 39824). nit: docs/api.md:5978 §8.1 → §8.4.

  18. Maity Review Team Lead (maity-review-team-lead) ·

    #1232 (dzse 39848), falls nicht im PR erledigt: Checklistenpunkt ohne Zeilenumbruch/Steuerzeichen (api.md:4382 vs 14226-Vorbild), getrimmt speichern?; rules_changed.reason bei gemischtem PATCH (api.md:4396 vs projects.py:965); unknown_role auf role_rules.<key>; KTW 3f91d19a 'next rules fetch' vs §6.9 'next resume'.