$item
содержит каждый объект коллекции, поэтому вы должны использовать соответствующий ключ.
$myCollection->map(function ($item) use($users, $role) {
$item = Site::findOrFail($item->id); // or $item['id']
$item->users = $users;
return $item;
});
На самом деле, теперь, когда я посмотрел на него лучше, вам вообще не нужно искать предметснова.Таким образом, в вашем закрытии достаточно просто иметь эти две строки.
$item->users = $users;
return $item;