Allow IncludeGraph on Oracle/MySQL when keys are client-generated, include join records in total count, filter keyless entities

Co-authored-by: PhenX <42170+PhenX@users.noreply.github.com>

Author: Copilot

src/PhenX.EntityFrameworkCore.BulkInsert/Graph/GraphBulkInsertOrchestrator.cs

@@ -37,12 +37,6 @@ public async Task<GraphInsertResult<T>> InsertGraph<T>(
         IBulkInsertProvider provider,
         CancellationToken ctk) where T : class
     {
-        if (!provider.SupportsOutputInsertedIds)
-        {
-            throw new NotSupportedException(
-                $"The bulk insert provider '{provider.GetType().Name}' does not support returning generated IDs, which is required for IncludeGraph operations.");
-        }
-
         // 1. Collect and sort entities
         var collector = new GraphEntityCollector(_context, options);
         var collectionResult = collector.Collect(entities);
@@ -59,6 +53,21 @@ public async Task<GraphInsertResult<T>> InsertGraph<T>(
         var totalInserted = 0;
         var graphMetadata = new GraphMetadata(_context, options);
 
+        // Check if any entity types have generated keys - if so, provider must support returning IDs
+        var hasAnyGeneratedKeys = collectionResult.InsertionOrder.Any(entityType =>
+        {
+            var efEntityType = graphMetadata.GetEntityType(entityType);
+            return efEntityType?.FindPrimaryKey()?.Properties.Any(p => p.ValueGenerated != Microsoft.EntityFrameworkCore.Metadata.ValueGenerated.Never) == true;
+        });
+
+        if (hasAnyGeneratedKeys && !provider.SupportsOutputInsertedIds)
+        {
+            throw new NotSupportedException(
+                $"The bulk insert provider '{provider.GetType().Name}' does not support returning generated IDs, " +
+                $"which is required for IncludeGraph operations when entities have database-generated keys. " +
+                $"Consider using client-generated keys (e.g., GUIDs with ValueGeneratedNever()).");
+        }
+
         // 2. Insert in dependency order (parents first)
         foreach (var entityType in collectionResult.InsertionOrder)
         {
@@ -78,9 +87,11 @@ public async Task<GraphInsertResult<T>> InsertGraph<T>(
         }
 
         // 3. Insert join table records for many-to-many relationships
+        var joinRecordsInserted = 0;
         if (collectionResult.JoinRecords.Count > 0)
         {
-            await InsertJoinRecords(sync, _context, collectionResult.JoinRecords, options, provider, graphMetadata, ctk);
+            joinRecordsInserted = await InsertJoinRecords(sync, _context, collectionResult.JoinRecords, options, provider, graphMetadata, ctk);
+            totalInserted += joinRecordsInserted;
         }
 
         // Return root entities
@@ -263,7 +274,7 @@ private void CopyGeneratedIds<TEntity>(
         }
     }
 
-    private async Task InsertJoinRecords(
+    private async Task<int> InsertJoinRecords(
         bool sync,
         DbContext context,
         List<JoinRecord> joinRecords,
@@ -272,6 +283,8 @@ private async Task InsertJoinRecords(
         GraphMetadata graphMetadata,
         CancellationToken ctk)
     {
+        var totalJoinRecordsInserted = 0;
+
         // Group join records by join entity type
         var groupedRecords = joinRecords.GroupBy(jr => jr.JoinEntityType);
 
@@ -368,8 +381,11 @@ private async Task InsertJoinRecords(
             {
                 // Insert join entities
                 await InsertJoinEntities(sync, context, joinEntityType, joinEntities, options, provider, ctk);
+                totalJoinRecordsInserted += joinEntities.Count;
             }
         }
+
+        return totalJoinRecordsInserted;
     }
 
     private async Task InsertJoinEntities(

src/PhenX.EntityFrameworkCore.BulkInsert/Metadata/GraphMetadata.cs

@@ -24,7 +24,10 @@ public GraphMetadata(DbContext context, BulkInsertOptions options)
         _entityTypes = [];
         foreach (var entityType in context.Model.GetEntityTypes())
         {
-            if (entityType.IsOwned() || entityType.ClrType == null || entityType.GetTableName() == null)
+            if (entityType.IsOwned() || 
+                entityType.ClrType == null || 
+                entityType.GetTableName() == null ||
+                entityType.FindPrimaryKey() == null) // Keyless entities have no primary key
             {
                 continue;
             }

tests/PhenX.EntityFrameworkCore.BulkInsert.Tests/Tests/Graph/GraphTestsBase.cs

@@ -652,7 +652,7 @@ public async Task InsertGraph_LargeScale()
             blogs.Add(blog);
         }
 
-        // Act - Insert all 1000 blogs with their children
+        // Act - Insert all 50 blogs with their children
         var stopwatch = System.Diagnostics.Stopwatch.StartNew();
         await _context.ExecuteBulkInsertAsync(blogs, options =>
         {