## Summary Test this on beta4: On Drupal 10, `ai_context` cannot be installed or uninstalled, and returns HTTP 500 on every request once enabled. `info.yml` declares `^10.5 || ^11.2`, but `DiffHooks` and `DynamicEntityReferenceHooks` are only registered as services by Drupal 11's `HookCollectorPass`, while `ai_context.module` requests them unconditionally from `hook_entity_type_alter()` — which runs on every container rebuild. ### Testing Setup The project's standard setup uses Drupal 11 and will **not** reproduce this. A Drupal 10 site is required: `mkdir ai-context-d10 && cd ai-context-d10 ddev config --project-type=drupal10 --docroot=web ddev start ddev composer create-project drupal/recommended-project:^10.6 ddev composer require drush/drush ddev drush site:install standard -y ddev composer require drupal/ai_context:^1.0@beta ddev drush en ai_context -y` ## Steps to reproduce 1. Install a Drupal 10.6 site as above 2. `ddev composer require drupal/ai_context:^1.0@beta` (resolves to 1.0.0-beta3) 3. `ddev drush en ai_context -y` 4. Request any page 5. Attempt to back out with `ddev drush pmu ai_context -y` ## Expected result The module installs on a core version that it itself `info.yml` declares support for. Failing that, it refuses to install with a clear message, leaves the site working, and remains uninstallable in the ordinary way. ## Actual result Step 3 fails with: `You have requested a non-existent service "Drupal\ai_context\Hook\DiffHooks". ` The module is nevertheless left enabled in `core.extension`, so step 4 returns **HTTP 500 on every route**, including `/user/login` and `/admin`. Step 5 fails with the _same_ exception, because uninstalling triggers `hook_modules_installed()` / `hook_entity_type_alter()`. The site cannot be recovered through the UI or Drush — only by editing `core.extension` directly or by patching the module. The unregistered classes are requested at: * `ai_context.module:678` — `ai_context_entity_base_field_info()` → `DynamicEntityReferenceHooks` * `ai_context.module:686` — `ai_context_entity_type_alter()` → `DiffHooks` * `ai_context.module:694` — `ai_context_modules_installed()` → `DiffHooks` * `ai_context.install:265` and `:606` — `DiffHooks::configureDiffFields()` `ai_context.services.yml` registers only `AiContextThemeHooks`. The comment beneath it states the assumption directly: _"DiffHooks and DynamicEntityReferenceHooks are auto-registered as services by Drupal 11's HookCollectorPass via their #\[Hook\] attributes. Explicit YAML registration is unnecessary."_ Drupal 10 has neither `HookCollectorPass` nor `Drupal\Core\Hook\Attribute\Hook`. **Suggested fix** — register both alongside the existing entry: ` Drupal\ai_context\Hook\DiffHooks: autowire: true Drupal\ai_context\Hook\DynamicEntityReferenceHooks: autowire: true ` Both constructors take a single autowirable argument (`ConfigFactoryInterface` and `ModuleHandlerInterface`), so `autowire: true` suffices and matches the existing `AiContextThemeHooks` style. Harmless on Drupal 11, where an explicit definition takes precedence over the collected one. Verified on 10.6.15: install, uninstall, cache rebuild and normal requests all work with the two definitions added. `SchedulerHooks` is in the same unregistered position and may want the same treatment, though I found no procedural caller for it in beta3. ## Environment * **Drupal version:** 10.6.15 * **Module version:** 1.0.0-beta3 * **PHP version:** 8.3 * **Provider:** Not provider-specific — fails before any AI call is made * **Last known working version:** None; first install attempt on this site ### Error messages or logs _(optional)_ `You have requested a non-existent service "Drupal\ai_context\Hook\DiffHooks". In Container.php line 159: You have requested a non-existent service "Drupal\ai_context\Hook\DiffHooks".` > [!note] > > Check the one that best describes your usage, or leave all unchecked if AI was not significantly used. > > * [x] **AI-Assisted Issue** \ > _Mainly written by a human; AI used for partial generation under full human supervision._ > * [ ] **AI-Generated Issue** \ > _Mainly generated by AI, reviewed and approved by a human before this issue was created._