Back and forth with opus 4.6 to review inconsistencies and DX issues: See MD file: [ai-context-naming-dx-review.md](/uploads/d1c45962b22f8275f9ede2be09561044/ai-context-naming-dx-review.md) ## Crossed-out ones were handled part 1 issue: #3586147 For the rest, we need to wait until a ton of MRs are merged first and the code base has settled down. ## Full list (need to address the ones that aren't crossed out) | Priority | Issue | Section | |---|---|---| | **High** | ~~Orphaned `AiContextScopeSiteSectionForm` writes to wrong config name~~ | 1c | | **High** | Dead `AiContextItemController::revisionOverview()` | 1b | | **Medium** | `Hook/` vs `Hooks/` dual directories | 1a | | **Medium** | Permission name with underscores | 2a | | **Medium** | Route names break `settings.*` pattern | 2b | | **Medium** | `new static()` in `final` classes (~20+ files) | 2c | | **Medium** | Mixed snake_case / camelCase locals within same files (internal inconsistency) | 2f | | **Low** | Codebase-wide variable convention split (Service/Model = camelCase, Form/Plugin = snake_case) | 2f | | **Low** | ~~camelCase `$childrenService` in procedural `ai_context.module` (PHPCS violation)~~ | 2f | | **Low** | ~~Missing `declare(strict_types=1)` on 2–3 files~~ | 2d | | **Low** | `ai_context.children_service` redundant suffix | 2e | | **Low** | Inconsistent `final` policy | 3c | | **Low** | ~~`canvas_page` in install config~~ | 3e | | **Low** | No service interfaces | 3d | | **Low** | Constructor style inconsistency (promotion vs traditional) | 3f | | **Info** | Constraint PascalCase plugin ID (follows core convention) | 2g | | **Info** | Lowercase `diff` plugin directory (required by diff module) | 2h | | **Info** | `getContextForEntity` naming ambiguity | 3a | | **Info** | ~~Menu link ID / route name collision~~ | 3b | | **Info** | FQCN service IDs for hook classes (Drupal 11 autowire pattern) | 2f (YAML) | **AI usage disclosures:** - [x] AI Assisted Issue - [x] AI Generated Code