chore(http): retirer deux appels sûrs sur un corps jamais nul

[InstaZDLL]

Deux avertissements de compilation traînaient dans ServerHttp.kt :

w: ServerHttp.kt:90:49 Unnecessary safe call on a non-null receiver of type 'ResponseBody'.
w: ServerHttp.kt:162:23 Unnecessary safe call on a non-null receiver of type 'ResponseBody'.

Response.body n'est pas nullable en OkHttp 5. Le ?. et le orEmpty() qui le suivait ne décrivaient plus rien. Un build qui crie pour rien est un build qu'on cesse de lire ; c'est ainsi qu'un vrai avertissement finit par passer inaperçu.

Ce que ça ne change pas

Aucun comportement. Un corps vide rendait la chaîne vide par orEmpty(), il la rend vide par string(). Aucun chemin ne passait par null — c'est précisément ce que le compilateur affirmait.

Pas de test de régression, donc : il n'y a pas de défaut à faire réapparaître. Les deux chemins touchés sont déjà couverts — le test du 204 sans corps pour la lecture réussie, celui de l'erreur en texte brut pour toException().

Vérifié

• ktlintCheck, detekt, testDebugUnitTest : 298 tests verts.
• compileDebugKotlin --rerun-tasks : plus aucun avertissement.

Summary by CodeRabbit

• Corrections
  • Amélioration de la lecture du contenu des réponses HTTP réussies et des erreurs réseau.
  • Les messages d’erreur affichent désormais correctement le corps de la réponse lorsqu’il est disponible.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 68c320e1-476b-4931-99ea-78c84d201d1d

📥 Commits

Reviewing files that changed from the base of the PR and between 3b5b3ad and e68ac4e.

📒 Files selected for processing (1)

• app/src/main/java/app/waveflow/data/remote/ServerHttp.kt

Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

---

📝 Walkthrough

Walkthrough

La lecture des corps HTTP utilise désormais directement body.string() pour les réponses réussies et les réponses d’erreur converties en exceptions.

Changes

Lecture des corps HTTP

Layer / File(s)	Summary
Lecture directe des réponses HTTP app/src/main/java/app/waveflow/data/remote/ServerHttp.kt	Les réponses réussies et les réponses d’erreur lisent directement leur corps avec body.string(), sans valeur vide de remplacement.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to e68ac

This is a localized cleanup that removes redundant safe calls without changing response handling; the reported checks pass, and no actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	Le titre décrit précisément la suppression des appels sûrs inutiles et respecte le format Conventional Commits.
Description check	✅ Passed	La description explique le changement, l’absence de régression et les validations exécutées, malgré l’absence du format complet du modèle.
Docstring Coverage	✅ Passed	Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/avertissements-corps-de-reponse

Warning

Your free Security trial is over. An organization admin can activate billing to continue.

---

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