build_hooks_entity_* call to the service container twice for the same service

+++ b/build_hooks.module
@@ -39,10 +39,11 @@ function build_hooks_cron() {
-      build_hooks_get_trigger_service()->deployFrontendEntityUpdateEnvironments();
...
+      $loggerService->deployFrontendEntityUpdateEnvironments();
+      $loggerService->invalidateToolbarCacheTag();

@@ -52,10 +53,11 @@ function build_hooks_entity_update(EntityInterface $entity) {
-      build_hooks_get_trigger_service()->deployFrontendEntityUpdateEnvironments();
-      build_hooks_get_trigger_service()->invalidateToolbarCacheTag();

@@ -65,10 +67,11 @@ function build_hooks_entity_insert(EntityInterface $entity) {
-      build_hooks_get_trigger_service()->deployFrontendEntityUpdateEnvironments();
-      build_hooks_get_trigger_service()->invalidateToolbarCacheTag();

the logger service isn't the one we need a variable for (its only called once) - instead its the trigger service we need a variable for.

Thanks for this

+++ b/build_hooks.module
@@ -158,8 +162,9 @@ function build_hooks_toolbar_alter(&$items) {
+  $triggerService = build_hooks_get_trigger_service();
...
-    'tags' => ['config:frontend_environment_list', build_hooks_get_trigger_service()->getToolbarCacheTag()],

let's reverse this one, we're only using it once in this function so there's no need to create a variable - unlike the other functions, where we use it twice

Thanks! 💪