Skip to content

Add support for nested objects in plain arrays#4

Merged
brianlmoon merged 3 commits intomainfrom
next
Nov 15, 2025
Merged

Add support for nested objects in plain arrays#4
brianlmoon merged 3 commits intomainfrom
next

Conversation

@brianlmoon
Copy link
Owner

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for serializing nested objects within plain arrays in both ValueObject and ArrayObject classes. The implementation recursively processes arrays to convert any nested Export or JsonSerializable objects to their array representations.

Key changes:

  • Recursive array handling in toArray() methods to support nested object serialization
  • Test coverage for nested objects in arrays
  • Addition of missing ext-yaml dependency declaration

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/ValueObject.php Adds recursive array processing to handle nested objects in array properties
src/ArrayObject.php Adds recursive array processing to handle nested objects in array elements
tests/ValueObjectTest.php Adds test case verifying ValueObject can serialize nested objects in arrays
tests/ArrayObjectTest.php Adds test case for nested arrays with objects and modernizes @depends annotation to PHP 8+ attribute syntax
composer.json Declares previously missing ext-yaml dependency required by existing YAML functionality

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@brianlmoon brianlmoon merged commit 7ba1894 into main Nov 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments