[liliangiraudo5]

• Added or updated tests
• Documented user facing changes

Changes

RelationBatchLoader::load() keyed parents by ModelKey, so a second instance of the same row under one path replaced the first, and only the surviving instance got the relation.
RelationDirective returns the batch result by key and is unaffected.
@with and @withCount run the field resolver on their own $parent, which never got the relation.
With a belongsToMany holding two pivot rows to the same model, that means a lazy load per duplicate (LazyLoadingViolationException under Model::preventLazyLoading()), or a missing *_count attribute.

 RelationBatchLoader::load($model)
-  parents[ModelKey($model)] = $model
+  parents[spl_object_id($model)] = $model

 PaginatedModelsLoader::load($parents)
-  one UNION subquery per parent
+  one UNION subquery per unique ModelKey
+  initRelation on every parent before match()

The same instance reached twice (e.g. a shared belongsTo) is still loaded once.

Breaking changes

None. No @api surface changes; RelationBatchLoader::$parents is now keyed by object id.

Changelog

Fixed

• Load @with and @withCount relations onto every instance of a model that is hydrated more than once under the same path, e.g. through duplicate pivot rows

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[spawnia]

🤖 @liliangiraudo5 #2803 fixes the same bug differently: @with and @withCount store the loaded result on their own parent, and the batch loader stays as it is.
Would that solve your problem too?

I lean towards your approach, since it fixes the batch loader for every caller.
The branch conflicts with master, so it needs an update before it can merge.

[Copilot]

🟢 Approval recommended

The focused loader changes preserve result lookup behavior, and targeted regression tests cover the affected cases with no blocking issues identified.

No open issues 🎉

_{Review effort: Balanced}

[liliangiraudo5]

Yes, #2803 would fix our case too, but I agree fixing the batch loader is better. Thanks for merging master in, nothing else to change on my side.