We can't make non static method call here, also RandomGeneratorTrait uses the same method of Random class.

That should be more of a maintenance call.

I think its required, please check the failing test here

That should be more of a maintenance call.

We are setting request time with initialise value which can be a valid timestamp.

This can be replaced with Drupal\Component\Utility\Random::string()

minimum support ^9.5

Remove this commented line 49-50

Changes not include any multi-line comma separator in the constructor or any function then I guess this variable also needs to be removed.

No need of this variable.

This should be "drupal/moderation_dashboard": "^2 || ^3",

This would work "drupal/inline_entity_form": "^3", (As it supports D8-11

Can we add minimum support ^9.5 if planning to keep support of D9

Typehint array missing