Feature/pre 3360 add refund tool

[hdelaforce-payplug]

Description

Motivation:

Related issue(s): Closes #

---

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue) [ ]
• ✨ New feature (non-breaking change that adds functionality) [ ]
• 💥 Breaking change (fix or feature that causes existing functionality to change and that could impact other libs) [ ]
• 🔧 Refactor (no functional changes, code improvement only) [ ]
• 📦 Dependency update [ ]
• 🔒 Security fix [ ]
• 📝 Documentation update [ ]

---

Checklist

Code Quality

• Code is linted and formatted
• No unnecessary commented-out code or debug logs
• No hardcoded values (use env variables or config)

Testing

• Unit tests added / updated

Security & Ops

• No sensitive data or secrets introduced
• Logging and error handling are appropriate

[wiz-14d684d7a6]

Wiz Scan Summary

Scanner	Findings
Vulnerabilities	-
Sensitive Data	-
Secrets	-
IaC Misconfigurations	1  4  1  1
SAST Findings	-
Software Management Findings	-
Total	1  4  1  1

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

[jhoaraupp]

getCustomerId() et getReason() retournent ?int / ?string. Quand ils ne sont pas renseignés, la requête envoie {"metadata": {"customer_id": null, "reason": null}} à l'API PayPlug, qui pourrait rejeter les valeurs null ou écraser des métadonnées existantes. Il faudrait n'inclure ces clés que si elles sont non-null (ex. array_filter ou ajout conditionnel).

[jhoaraupp]

Import use Payplug\Refund inutilisé.
Payplug\Refund est utilisé dans Api::refundPaymentResource(), pas dans PaymentAction

[jhoaraupp]

RefundInputDTO::hydrate() déclare resource comme champ required et lève une exception si null avant même que le DTO ne soit créé. Cette garde est donc inatteignable — un DTO valide ne peut jamais avoir resource === null à ce stade.

[jhoaraupp]

N'importe quel développeur qui tape make commit-push va force-push silencieusement la branche courante, effaçant potentiellement le travail d'un collaborateur. Au minimum, utiliser --force-with-lease qui échoue si quelqu'un a poussé entre-temps.