Update our override of Laravel's attributesToArray()

[mjauvin]

• Use base Model methods instead of re-implementing them ourselves.
• Add our before/after events.
• Handle model's $jsonable array since Laravel doesn't seem to handle this.
• Using base model addCastAttributesToArray() also adds casting types not supported before:
  • date:<format>
  • datetime:<format>

Note: in order for created_at/updated_at columns to be used in $casts, the model's $timestamps property needs to be false.

Summary by CodeRabbit

• Refactor

  • Consolidated internal attribute transformation logic into dedicated helper routines to improve maintainability and reliability.

• Tests

  • Added coverage verifying date and cast handling to ensure attribute serialization behaves as expected.

Note: No user-facing changes or public API updates in this release.

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

[coderabbitai]

Walkthrough

Refactor: attribute transformation logic in src/Database/Model.php was moved out of attributesToArray into three helper methods; a new test testAddDatesCasts was added to tests/Database/ModelTest.php (duplicated in the file).

Changes

Cohort / File(s)	Change Summary
Model attribute transformation refactor src/Database/Model.php	Extracted inlined processing from attributesToArray into addDateAttributesToArray(), addMutatedAttributesToArray(), and addCastAttributesToArray(); mutated attributes retrieved earlier and passed into helpers; removed original inline loops.
Tests — added date/cast assertions (duplicate) tests/Database/ModelTest.php	Added public function testAddDatesCasts() which configures casts and date strings, asserts Argon instances and attributesToArray output — the method was inserted twice (duplicate definition).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Verify helpers preserve exact original behavior and ordering of date/mutate/cast transformations.
• Confirm mutatedAttributes retrieval and parameter passing is correct and side-effect free.
• Inspect getArrayableAppends() interaction remains identical.
• Remove or reconcile the duplicated testAddDatesCasts() method in tests/Database/ModelTest.php.

Poem

🐇 I nudged the loops into neat little burrows,
Dates and Casts now hop in rows and rows.
Tests arrived twice — oh my, what a fuss,
I'll tidy the warren and jitterbug thus! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: refactoring the attributesToArray() method implementation to delegate processing to helper methods.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch attributes-to-array

---

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.}

[github-actions]

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.