Simplify code formatting by removing trailing commas, consolidating doc comments, and inline constructors. Add friendsofphp/php-cs-fixer for consistent style.

[shaunluedeke]

Summary by CodeRabbit

• Verbesserungen

  • Ungültige JSON-Zeilen in Streaming-Daten werden jetzt sicher übersprungen, statt die Verarbeitung zu unterbrechen.
  • Allgemeine Verbindungsfehler werden konsistenter erkannt und verständlich weitergegeben.
  • Datenobjekte sind konsistenter gegen unbeabsichtigte Änderungen geschützt.

• Qualitätssicherung

  • Automatische Prüfungen für Code-Stil und Tests wurden ergänzt.
  • Tests laufen über mehrere PHP-Versionen und decken Unit- sowie Docker-Integration ab.

• Dokumentation

  • README und technische Dokumentation wurden sprachlich und formal vereinheitlicht.

[coderabbitai]

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: 55ed98ff-28a5-4180-b23f-bc8078a8b408

📥 Commits

Reviewing files that changed from the base of the PR and between 6cb7a94 and 0720545.

📒 Files selected for processing (46)

• .github/workflows/code-style.yml
• .github/workflows/tests.yml
• .gitignore
• .php-cs-fixer.dist.php
• README.md
• composer.json
• src/DTO/ContainerInfo.php
• src/DTO/ContainerSummary.php
• src/DTO/ImageInfo.php
• src/DTO/ImageSummary.php
• src/DTO/NetworkInfo.php
• src/DTO/VolumeInfo.php
• src/DockerClient.php
• src/Exceptions/DockerApiException.php
• src/Exceptions/DockerConnectionException.php
• src/Exceptions/DockerException.php
• src/Exceptions/DockerNotFoundException.php
• src/Http/DockerResponse.php
• src/Http/DockerTransport.php
• src/Http/DockerTransportInterface.php
• src/Http/StreamingSink.php
• src/Resources/AbstractResource.php
• src/Resources/Containers.php
• src/Resources/Exec.php
• src/Resources/Images.php
• src/Resources/Networks.php
• src/Resources/System.php
• src/Resources/Volumes.php
• src/Support/NdjsonLineBuffer.php
• src/Support/StdioDemultiplexer.php
• tests/Integration/DockerIntegrationTest.php
• tests/Support/FakeDockerTransport.php
• tests/Support/TestableDockerTransport.php
• tests/Unit/DTO/ContainerInfoTest.php
• tests/Unit/DTO/ContainerSummaryTest.php
• tests/Unit/DTO/ImageInfoTest.php
• tests/Unit/DTO/ImageSummaryTest.php
• tests/Unit/DTO/NetworkInfoTest.php
• tests/Unit/DTO/VolumeInfoTest.php
• tests/Unit/DockerClientTest.php
• tests/Unit/Exceptions/DockerApiExceptionTest.php
• tests/Unit/Exceptions/DockerConnectionExceptionTest.php
• tests/Unit/Http/DockerResponseTest.php
• tests/Unit/Http/DockerTransportTcpTest.php
• tests/Unit/Http/DockerTransportTest.php
• tests/Unit/Resources/ContainersTest.php

💤 Files with no reviewable changes (1)

• tests/Unit/DockerClientTest.php

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

Die Änderung ergänzt CI-Prüfungen für Code-Style und Tests, vereinheitlicht DTO-Readonly-Deklarationen, erweitert die Guzzle-Fehlerbehandlung und formatiert PHP-Code, Dokumentation sowie Tests kompakter.

Changes

Codequalität und CI

Layer / File(s)	Summary
Code-Style- und CI-Prüfungen .github/workflows/*, .php-cs-fixer.dist.php, .gitignore, composer.json, README.md	GitHub Actions prüfen Code-Style und Tests. Composer führt PHPStan und PHP-CS-Fixer aus. Der Cache wird ignoriert. README-Trennstriche werden vereinheitlicht.
Readonly-DTO-Verträge src/DTO/*Info.php, src/DTO/*Summary.php	DTOs verwenden konsistente readonly-Klassendeklarationen. ContainerInfo verliert die Readonly-Semantik.
Transport- und Ausnahmefluss src/DockerClient.php, src/Exceptions/*, src/Http/*	DockerTransport behandelt allgemeine Guzzle-Fehler und löst API-Versionen direkt auf. DockerConnectionException erhält eine Guzzle-Factory. JSON-Typen werden präzisiert.
Ressourcen- und Streaming-Verarbeitung src/Resources/*, src/Support/*	Ressourcenaufrufe werden verdichtet. registryAuthHeader() kapselt JSON-Fehler. NDJSON überspringt ungültige JSON-Zeilen. PHPDoc-Typen werden präzisiert.
Tests und Testunterstützung tests/Integration/*, tests/Support/*, tests/Unit/*	Tests und Testhilfen verwenden kompaktere Ausdrücke. Geprüfte Ergebnisse und Testabläufe bleiben unverändert.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Merge Risk: ⚪ Minimal · up to 07205

The PR standardizes formatting and development checks without any supplied evidence of a concrete correctness, security, availability, or deployment risk; no actionable merge-blocking risk remains beyond normal checks.

Sequence Diagram(s)

sequenceDiagram
  participant DockerClient
  participant DockerTransport
  participant Guzzle
  participant DockerConnectionException
  DockerClient->>DockerTransport: request oder stream
  DockerTransport->>Guzzle: HTTP-Anfrage
  Guzzle-->>DockerTransport: Antwort oder GuzzleException
  DockerTransport->>DockerConnectionException: fromGuzzleError
  DockerConnectionException-->>DockerClient: DockerConnectionException

Loading

Poem

Ich hoppse durch den Code im Kreis
Der Formatter macht die Zeilen weiß
Guzzle stolpert, Fehler steh’n
DTOs bleiben fest und schön
CI prüft den Hasenbau
Alles läuft nun glatt und schlau

---

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/settings/billing.

_{Comment @coderabbitai help to get the list of available commands.}