Autoincluded properties doesnt copied when use BulkRead

[s11th]

since EF Core 6.0. you can setup you property like that builder.Entity<Entity>.Navigation(i => i.Items).AutoInclude();
and usage context.BulkReadAsync(entities) transforms into query with additional join on Items table and return entities with filled Items navigation property

but

EFCore.BulkExtensions/EFCore.BulkExtensions/TableInfo.cs

Line 800 in 8622039

internal void UpdateReadEntities<T>(IList<T> entities, IList<T> existingEntities, DbContext context)

will set values only for simple properties and owned navigation properties as i understand

i think copy not empty navigation properties will be good decision instead of doing smth like this #733 (comment)

[borisdj]

BulkReading subEntites is a problem of itself, and one solution is given in the issue you already referenced.
So only when that would be implemented as a special method could it be connected with AutoInclude feature in which case that method would be automatically called.
But that might be too complex, so maybe best to leave that part for manual connection.
Still the main feature needs to be implemented first as method in the library instead of writing specific RawSql.

[s11th]

i mean than when property is autoincluded your

EFCore.BulkExtensions/EFCore.BulkExtensions/TableInfo.cs

Line 937 in 8622039

public List<T> LoadOutputEntities<T>(DbContext context, Type type, string sqlSelect) where T : class

will transformed as query
select entity.*, navEntity.* from (your rawSql query) as entity join navigationEntity navEntity on entity.id = navEntity.entityId
and when your call

EFCore.BulkExtensions/EFCore.BulkExtensions/TableInfo.cs

Line 800 in 8622039

internal void UpdateReadEntities<T>(IList<T> entities, IList<T> existingEntities, DbContext context)

every item of IList<T> existingEntities contains data in navigation property but it will be lost, that data dont copied to IList<T> entities.
you dont need to have implementation to read subEntites, it already implemented by efcore and subentities included when navProperty is autoincluded
but maybe i misunderstanding something

can write a test showing this behaviour

[borisdj]

This can now be done using config ReplaceReadEntities:

when set to True result of BulkRead operation will be provided using replace instead of update. Entities list parameter of BulkRead method will be repopulated with obtained data. Enables functionality of Contains/IN which will return all entities matching the criteria and only return the first (does not have to be by unique columns).