Is dynamic action name building a bad practice?

Usually yes, sometimes no, but in your example, definitely bad practice

You see, instead of putting the name or the post ID in the action name, you should have passed it as a parameter instead. Otherwise there is no way to grab all elementor css file parsing, or no way to work after any arbitrary after imported post. You need to know in advance every possible post ID or name that could turn up, making the action very limited in use.

Examples where it might make sense, are where it’s shorthand for an entire class of objects. This is a rarer situation, but an example would be publish_posts versus publish_book where book is a custom post type.

Note that in those situations, these are additional actions/hooks that are done in addition to not instead of generic hooks.