Remove KeyRepositoryInterface from \Drupal\ai\Base\AiVdbProviderClientBase

Created on 1 August 2025, 5 days ago

Problem/Motivation

Currently, \Drupal\ai\Base\AiVdbProviderClientBase requires a \Drupal\key\KeyRepositoryInterface constructor param. However, this is never used in this base class.

Some providers don't rely on Key → module, so why do we need for force this as a requirement for all providers?

Steps to reproduce (required for bugs, but not feature requests)

Proposed resolution

Remove KeyRepositoryInterface param from \Drupal\ai\Base\AiVdbProviderClientBase::__construct()

Remaining tasks

Optional: Other details as applicable (e.g., User interface changes, API changes, Data model changes)

📌 Task

Status

Active

Version

1.2

Component

AI Search

Created by

🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia

Live updates comments and jobs are added and updated live.

Merge Requests

!811Remove KeyRepositoryInterface from \Drupal\ai\Base\AiVdbProviderClientBase
Open
Unnamed author
updated 4 days ago

Comments & Activities

Issue created by @kim.pepper
Comment 5 days ago →
🇦🇺Australia kim.pepper 🏄‍♂️🇦🇺Sydney, Australia
Updating to include the following params which are also unused:

protected EventDispatcherInterface $eventDispatcher

protected MessengerInterface $messenger
Comment 4 days ago →
🇮🇳India anjaliprasannan
Merge request !811Issue #3539214: Remove unused dependencies from \Drupal\ai\Base\AiVdbProviderClientBase → (Open) created by Unnamed author
Pipeline finished with Failed
4 days ago
#562325
Pipeline finished with Failed
4 days ago
Total: 207s
#562340
Comment 4 days ago →
🇮🇳India anjaliprasannan
Comment 4 days ago →
🇮🇳India anjaliprasannan

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024