feat: #3636054 Add the AI Code Review recipe and onboarding documentation

AI Yasrequested to merge
issue/ai_code_review-3636054:3636054-add-recipe-onboarding into 1.x

Issue: https://www.drupal.org/project/ai_code_review/issues/3636054

Summary

Adds the AI Code Review Drupal Recipe to prepare a UI site for connection to a separately installed AI Code Review Server.

The Recipe enables AI Code Review UI and creates a dedicated reviewer role with Add, View, and List permissions. Existing role properties and unrelated permissions are retained. View permission applies across the UI site; cancellation requires an administrator to grant Edit permission separately.

The package contains no runtime PHP classes or interfaces and no credentials or deployment-specific connection settings. The README documents the separate-site architecture, alpha package installation, Recipe application, and links to the UI and Server setup guides.

Validation

Validated on Drupal 11.4.12 with PHP 8.4.24, SQLite, Drush 13.8, and AI Code Review UI 1.0.0-alpha1. Composer installation and Recipe unpacking succeeded with both alpha packages explicitly required. Clean-site and configured-site application and reapplication passed, including 2 Functional tests and 40 assertions covering reviewer access and configuration preservation. PHP syntax and Drupal/DrupalPractice coding standards checks passed. Eight dependency deprecations from AI, highlight.php, and the UI-provided View configuration were reported without suppression. No AI provider calls were made.

Public CI passed all seven automatic jobs. The PHPUnit job ran the actual Recipe Functional suite: 2 tests, 40 assertions, zero errors, failures or skipped tests on PHP 8.3.36 and Drupal 11.4.12. The optional test-only comparison job remains manual. Independent review of the final source and public content found no blocking issue.

Remaining steps

  • Record local final-source validation results.
  • Verify public CI on the source branch.
  • Complete independent source review and resolve actionable findings.
  • Obtain maintainer approval and accept the merge request.
  • Confirm post-merge branch validation before creating a release tag.
  • Verify the published alpha package and Composer installation after publication.

Acceptance

Merged into 1.x as 8912b462c3f51b752ad0a70b5723e17f76805296 after yas RTBC and actual approval. The accepted commit message, parent and tree match the reviewed candidate. Official branch CI passed all seven automatic jobs, including two Functional tests with 40 assertions. The Issue is Fixed and contribution credits are saved for yas and ai_yas. The alpha release and its package verification are recorded below.

Published package acceptance

Recipe 1.0.0-alpha1 is published from the accepted commit. Both advertised archives contain the seven expected files; only LICENSE heading whitespace differs due to packaging. The official Composer distribution matches all seven accepted files exactly. Normal Composer installation and clean/configured-site application and reapplication succeeded without lowering global minimum stability or substituting local source.

Separate-site verification then exposed missing UI policy and review-list configuration on fresh Recipe installations. Full functional release acceptance remains incomplete. The confirmed defect is tracked separately in #3636118, and the release notes disclose it. The published alpha tag is unchanged. No paid provider request was made.

  • Resolve and verify the configuration import defect through its own issue-fork merge request.
Edited by AI Yas

Merge request reports