>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3539214. --> Reported by: [kim.pepper](https://www.drupal.org/user/370574) Related to !811 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>Currently, <code>\Drupal\ai\Base\AiVdbProviderClientBase::__construct()</code> requires the following</p> <ul> <li><code>EventDispatcherInterface</code></li> <li><code>MessengerInterface</code></li> <li><code>\Drupal\key\KeyRepositoryInterface</code></li> </ul> <p>However these are never used in the base class.</p> <p>Also, some providers don't rely on <a href="https://www.drupal.org/project/key">Key</a> module.</p> <p>We should only include dependencies that the base class itself requires.</p> <h4 id="summary-steps-reproduce">Steps to reproduce (required for bugs, but not feature requests)</h4> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Remove unused dependencies from the constructor.</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <h3>Optional: Other details as applicable (e.g., User interface changes, API changes, Data model changes)</h3>