If you make it trigger at the end of enchanted player's turn, there are less memory issues in case there is an extended period of time between your turn and enchanted player's turn (e. g. multiplayer if the player goes right after you).

You could even change it to "At the beginning of enchanted player's end step, each player that has been dealt combat damage this turn creates a 1/1 white Soldier creature token." to get around some questions e. g. "Does this card have negative synergy with Dread?"

The cost implies this might net you usually three tokens in a game, which seems about right.

That's a good point. I'll update the text. I'll also allow it to count damage from noncreature sources as well, since that buff doesn't seem too unreasonable.