Merge f912866fcb into 0874e426af

* Fix bulk_insert with moving archetypes

* Reword message

jecs.luau

@@ -2120,15 +2120,13 @@ local function ecs_bulk_insert(world: world, entity: i53, ids: { i53 }, values:
 		local dst_types = ids
 		local to = archetype_ensure(world, dst_types)
 		new_entity(entity, r, to)
-		local row = r.row
-		local columns_map = to.columns_map
 		for i, id in ids do
 			local value = values[i]
 			local cdr = component_index[id]
 
 			local on_add = cdr.on_add
 			if value then
-				columns_map[id][row] = value
+				r.archetype.columns_map[id][r.row] = value
 				if on_add then
 					on_add(entity, id, value :: any)
 				end
@@ -2158,12 +2156,10 @@ local function ecs_bulk_insert(world: world, entity: i53, ids: { i53 }, values:
 	end
 
 	local to = archetype_ensure(world, dst_types)
-	local columns_map = to.columns_map
 
 	if from ~= to then
 		entity_move(entity_index, entity, r, to)
 	end
-	local row = r.row
 
 	for i, set in emplaced do
 		local id = ids[i]
@@ -2174,7 +2170,7 @@ local function ecs_bulk_insert(world: world, entity: i53, ids: { i53 }, values:
 		local on_add = idr.on_add
 
 		if value ~= nil then
-			columns_map[id][row] = value
+			r.archetype.columns_map[id][r.row] = value
 			local on_change = idr.on_change
 			local hook = if set then on_change else on_add
 			if hook then

test/tests.luau

@@ -331,6 +331,37 @@ TEST("bulk", function()
 		CHECK(world:has(e, c1, c2, c3))
 		CHECK(count == 3)
 	end
+
+	do CASE "Should bulk add with hooks moving archetypes without previous"
+		local world = jecs.world()
+		local c1, c2, c3 = world:component(), world:component(), world:component()
+		
+		world:added(c1, function(e)
+			world:set(e, c3, "hello")
+		end)
+		
+		local e = world:entity()
+		jecs.bulk_insert(world, e, {c1,c2}, {true, 123})
+		CHECK(world:get(e, c1) == true)
+		CHECK(world:get(e, c2) == 123)
+		CHECK(world:get(e, c3) == "hello")
+	end
+
+	do CASE "Should bulk add with hooks moving archetypes with previous"
+		local world = jecs.world()
+		local c1, c2, c3 = world:component(), world:component(), world:component()
+		
+		world:added(c1, function(e)
+			world:set(e, c3, "hello")
+		end)
+		
+		local e = world:entity()
+		world:add(e, world:entity())
+		jecs.bulk_insert(world, e, {c1,c2}, {true, 123})
+		CHECK(world:get(e, c1) == true)
+		CHECK(world:get(e, c2) == 123)
+		CHECK(world:get(e, c3) == "hello")
+	end
 end)
 
 TEST("repro", function()