HP-2870: Improve ChargeStealingException error message

[VadymHrechukha]

Summary by CodeRabbit

• Chores
  • Removed unused internal error handling code.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The ChargeStealingException class is removed from the codebase. This exception previously handled cases where a charge attempted to steal an existing charge from another bill, providing factory methods and context attachment functionality.

Changes

Cohort / File(s)	Summary
Exception deletion src/Exception/ChargeStealingException.php	Removed entire exception class, including static factory method fromPdoException() and private helper trimExceptionMessage()

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• HP-2824: Fix Failing Psalm Static Analysis in php-billing Composer Package Build #111 — Previously modified the same ChargeStealingException class to add HasContext handling; this PR removes the class entirely, making that prior work obsolete.

Suggested reviewers

• SilverFire

Poem

🐰 A charge that once stole and caused such a fright,
Has vanished away like the moon in the night,
No context, no factory, no exception's call—
Just code cleanup magic, and that's all! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title states 'Improve ChargeStealingException error message' but the PR actually removes the entire ChargeStealingException class, not improves it.	Update the PR title to accurately reflect the actual change, such as 'Remove ChargeStealingException class' or 'HP-2870: Remove ChargeStealingException'

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8f60fb6 and eee911f.

📒 Files selected for processing (1)

• src/Exception/ChargeStealingException.php

💤 Files with no reviewable changes (1)

• src/Exception/ChargeStealingException.php

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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