Fix/#32861

[micharoesler]

#32861 — Fix Confluence-Attachment-Download (REST-Endpoint) + Tests & CI

Problem

Attachments (z. B. die „Schnellhilfe"-Bilder) wurden nicht mehr heruntergeladen und
erschienen in der synchronisierten Dokumentation als 404. Ursache: Das Paket hat Attachments
über das Legacy-Servlet /wiki/download geladen, das API-Token-Authentifizierung nicht mehr
akzeptiert und HTTP 401 zurückgibt. Da der Fehler verschluckt wurde (ein bloßes echo +
return), wirkte der Sync erfolgreich, während jedes Attachment still übersprungen wurde.

Fix

Attachments werden stattdessen über den unterstützten REST-Endpoint geladen:

/wiki/rest/api/content/{pageId}/child/attachment/{attachmentId}/download

• Die Authentifizierung bleibt unverändert — es werden dieselben Basic-Auth-Credentials
  (E-Mail + API-Token) gesendet wie bei jedem anderen Aufruf; nur der Endpoint wurde auf die
  REST-API umgestellt.
• downloadAttachment() erhält jetzt die pageId, und ConfluenceAttachment::getId() wurde
  ergänzt, um die REST-URL zu bauen.

Fehlerbehandlung (keine stillen Fehlschläge mehr)

• ensureDownloadFolder() wirft jetzt eine RuntimeException, wenn der Download-Ordner nicht
  angelegt werden kann, statt auf stdout auszugeben und weiterzulaufen.
• ConfluencePageContentDownloader fängt Exceptions und meldet sie über einen injizierten
  PSR-3-LoggerInterface (Default: NullLogger, das Verhalten bleibt also unverändert, wenn
  kein Logger übergeben wird). Fügt eine psr/log-Abhängigkeit hinzu.

Cleanup / Aufräumen

• Das nun tote ConfluenceAttachment::findDownloadPath() entfernt (einziger Nutzer des kaputten
  Legacy-Pfads), damit der 401-anfällige Pfad nicht versehentlich wiederverwendet werden kann.
• Die Attachment-Download-URL auf einen relativen Pfad normalisiert (kein führender Slash),
  konsistent mit den übrigen Content-API-Aufrufen.
• Die eine verbliebene deutsche Exception-Message ins Englische übersetzt (konsistent mit den
  anderen).
• Doc-Blocks und @throws-Annotationen an den Content- und Download-Methoden ergänzt/erweitert
  und eine lokale Variable $filemtime → $fileModificationTime umbenannt.

Tests & CI

• 21 Unit-Tests hinzugefügt, die den Fix abdecken: die REST-Attachment-URL, den Basic-Auth-Header,
  die Skip-/Re-Download-Entscheidung (Datei-mtime vs. Attachment-lastUpdated), das Anlegen des
  Download-Ordners und die neue RuntimeException, das JSON→DTO-Parsing, die Non-200-/5xx-Behandlung
  sowie den neuen Error-Logging-Pfad.
• Sämtliches HTTP wird über Guzzles MockHandler bedient — kein Test ruft jemals die Live-
  Confluence-API auf, Builds bleiben damit netzwerkfrei.
• Die Dev-Abhängigkeiten (phpunit/phpunit, phpstan/phpstan-phpunit), eine phpunit.xml und ein
  composer test-Script wieder hinzugefügt.
• Einen eigenständigen PHPUnit-GitHub-Actions-Workflow ergänzt (PHP 8.2 + 8.3, gespiegelt an der
  PHPStan-Matrix), da der geteilte Workflows-Repo keinen wiederverwendbaren phpunit-Workflow hat.
  Die Tests laufen jetzt in jedem Build.

Warum PHPUnit 11 (und nicht 12)

Wir sind bewusst auf phpunit/phpunit ^11.5 geblieben statt auf dem zuvor entfernten ^12.5.6:
phpunit 12 benötigt PHP ≥ 8.3, der Floor dieser Library ist aber php: >=8.2 und die CI
fährt einen 8.2-Matrix-Leg. Mit phpunit 12 würde composer install auf PHP 8.2 fehlschlagen und den
Build brechen. PHPUnit 11.5 unterstützt PHP 8.2–8.4 und behält damit den 8.2-Support, ohne die
Runtime-Constraint require.php anzufassen (Consumer sind nicht betroffen — Dev-Abhängigkeiten
werden nicht transitiv installiert).

Verifikation

End-to-End in einer echten AGP-Installation über einen erzwungenen Voll-Sync verifiziert: Alle
Attachments wurden korrekt heruntergeladen, inklusive des Bildes, das zuvor 404 lieferte; Text und
Bilder werden in der UI gerendert. PHPUnit (21 Tests) und PHPStan Level 8 (über src + tests) sind
grün.