fix: Laravel 12.45.x ResourceCollection compatibility

[kevalyq]

Summary

Fixes test failures in all Dependabot PRs (#478, #479, #480, #481) caused by Laravel 12.45.0 breaking change in ResourceCollection handling.

Problem

Laravel 12.45.0 introduced PR #58299 which changed how ResourceCollection::collection() handles plain arrays vs Model Collections. The test SecretAttachmentResourceTest > resource transforms collection correctly was passing a plain array, which now returns incorrect data structure.

Failing Test:

$collection = SecretAttachmentResource::collection([$attachment1, $attachment2]);
// Returns: [null, null] instead of proper attachment data

Error:

FAILED  Tests\Feature\Resources\SecretAttachmentResourceTest > resource transforms collection correctly
Failed asserting that null is identical to 'file1.pdf'.
at tests/Feature/Resources/SecretAttachmentResourceTest.php:108

Solution

Wrap the array in collect() to ensure proper Collection type:

$collection = SecretAttachmentResource::collection(collect([$attachment1, $attachment2]));

This ensures the ResourceCollection receives an Eloquent Collection instead of a plain array, maintaining compatibility with Laravel 12.45.x.

Testing

• ✅ Local: PHPStan Level 9 clean
• ✅ Local: Laravel Pint (PSR-12) compliant
• ✅ CI will verify: All 1817 tests pass

Impact

• Blocks: chore(deps-dev): bump laravel/sail from 1.51.0 to 1.52.0 #478, chore(deps): bump laravel/framework from 12.44.0 to 12.45.1 #479, chore(deps): bump laravel/sanctum from 4.2.1 to 4.2.2 #480, chore(deps): bump laravel/tinker from 2.10.2 to 2.11.0 #481 (all Dependabot PRs)
• Breaking Change Mitigation: Adapts to Laravel 12.45.0 ResourceCollection behavior
• No functional changes: Only test code modified

Related

• Laravel Framework PR: [12.x] Fix ResourceCollection usage when used with an array instead of Model collection laravel/framework#58299
• Laravel Release: v12.45.0 (2026-01-06)

[github-actions]

💡 Tip: Consider Using Draft PRs

Benefits of opening PRs as drafts initially:

• 💰 Saves CI runtime and Copilot review credits
• 🎯 Automatically sets linked issues to "🚧 In Progress" status
• 🚀 Mark "Ready for review" when done to trigger full CI pipeline

How to convert:

1. Click "Still in progress? Convert to draft" in the sidebar, OR
2. Use gh pr ready when ready for review

This is just a friendly reminder - feel free to continue as is! 😊

[Copilot]

Pull request overview

This PR fixes a test failure caused by Laravel 12.45.0's breaking change in ResourceCollection handling, which now requires proper Collection types instead of plain arrays for correct data transformation.

Key Changes

• Wraps test array in collect() to ensure SecretAttachmentResource::collection() receives an Eloquent Collection instead of a plain array
• Maintains compatibility with Laravel 12.45.x while preserving test functionality

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!