>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3586112. --> Reported by: [kristen pol](https://www.drupal.org/user/8389) >>> <p>[Tracker]<br> <strong>Update Summary: </strong>[One-line status update for stakeholders]<br> <strong>Check-in Date: </strong>MM/DD/YYYY<br> <strong>Blocked by: </strong>[#XXXXXX] (New issues on new lines)<br> <strong>Additional Collaborators: </strong> @username1, @username2<br> <em>Metadata is used by the <a href="https://www.drupalstarforge.ai/" title="AI Tracker">AI Tracker.</a> Docs and additional fields <a href="https://www.drupalstarforge.ai/ai-dashboard/docs" title="AI Issue Tracker Documentation">here</a>.</em><br> [/Tracker]</p> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>Follow-up from:</p> <p><span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-2"><a href="https://www.drupal.org/project/ai_context/issues/3586091" title="Status: Fixed">#3586091: Review CCC todo comments for action items</a></span></p> <p>## P2 Defer (UI Polish)</p> <p>These are admin UI rendering or link-visibility issues. Important for polish but not functional blockers for RC1.</p> <p>| File | Method | TODO | Line |<br> |------|--------|------|------|<br> | `tests/src/Functional/AiContextScopeSettingsTest.php` | `testScopeOverviewPage` | Fix Configure link visibility | 54 |<br> | `tests/src/Functional/AiContextScopeSettingsTest.php` | `testTargetEntitySettings` | Fix canvas_page entity type availability in test environment | 154 |<br> | `tests/src/Functional/AiContextScopeSettingsTest.php` | `testLocalTasks` | Fix Settings link visibility | 198 |</p> <p>---</p> <p>## Defer External Dependencies / Future Architecture</p> <p>These TODOs are gated on external changes (Drupal 10 EOL, core patches, architecture decisions). They represent working code with documented cleanup opportunities.</p> <p>### Drupal 10 compatibility workarounds (5 items)</p> <p>All tagged `@todo Remove when dropping Drupal 10 support`. The module declares `core_version_requirement: ^10.5 || ^11.2`, so these are intentional. Track as a follow-up for when D10 support is dropped.</p> <p>| File | Line | Description |<br> |------|------|-------------|<br> | `ai_context.module` | 215 | `hook_module_implements_alter` scheduler workaround |<br> | `ai_context.module` | 234 | `hook_scheduler_hide_publish_date` workaround |<br> | `ai_context.module` | 249 | `hook_scheduler_hide_unpublish_date` workaround |<br> | `src/Hooks/AiSchedulerContentModerationSync.php` | 26 | Entire class is a D10 SCMI field sync workaround |<br> | `tests/src/Functional/AiContextSchedulerRaceConditionTest.php` | 168 | Test skipped: SCMI .module not loaded in D10 test runner |</p> <p>### Upstream Drupal core issue (1 item)</p> <p>| File | Line | Description |<br> |------|------|-------------|<br> | `src/Controller/AiContextItemController.php` | 193 | Simplify once <a href="https://www.drupal.org/node/2334319">https://www.drupal.org/node/2334319</a> lands |</p> <p>Current code works correctly. The TODO is a simplification opportunity gated on a pending core patch.</p> <p>### Performance / architecture note (1 item)</p> <p>| File | Line | Description |<br> |------|------|-------------|<br> | `src/Service/AiContextSelector.php` | 171 | Consider denormalized scope index table if item count grows |</p> <p>Aligns with the existing postponed issue "CCC: Refactor to denormalize scope values and replace SQL LIKE on serialized scope." The current chunked-streaming approach is adequate for RC1 volumes.</p> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Fix todos</p> <h3 id="summary-remaining-tasks">Target date or deadline</h3> <p>TBD</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <ul> <li>Fix todos</li> <li>Review and run tests</li> </ul> <h3 id="summary-ai-usage">AI usage (if applicable)</h3> <p>[x] AI Assisted Issue<br> This issue was generated with AI assistance, but was reviewed and refined by the creator.</p> <p>[ ] AI Assisted Code<br> This code was mainly generated by a human, with AI autocompleting or parts AI generated, but under full human supervision.</p> <p>[ ] AI Generated Code<br> This code was mainly generated by an AI with human guidance, and reviewed, tested, and refined by a human.</p> <p>[ ] Vibe Coded<br> This code was generated by an AI and has only been functionally tested.</p> > Related issue: [Issue #3586111](https://www.drupal.org/node/3586111)