To answer my own question (again!): Yes, that's the correct resource type. I hadn't done a
pulumi import
in a while, but that told me what I needed to know. Adding the DiagnosticSetting to the Azure Communications Services resource worked flawlessly.
Thanks again.