I agree with you OP. I don’t really think much is wrong with the initial conversation, but there should be another opportunity in the future. From a roleplaying standpoint, offering to let Astarion feed on my Tav indefinitely didn’t make a lot of sense because they weren’t close at that point, so I didn’t pick it. I really wish there was a way to bring the topic up again after you start romancing him, since it seems obvious to me that Tav (and the player) might feel differently after getting attached to him.