Cursor · Grok 4.6 High

• [reference] Выбор уместный и амбициозный; хиро, арка-манифест, пилюля Menu, шрифты и палитра узнаются рядом с референсом на обеих ширинах, все 5 приёмов из чек-листа реально есть (reveal, hover cream↔прозрачность, reduced-motion, прелоадер). Но дальше хиро страница расходится: у референса ритм olive → beige (арочное фото) → dark (наклонные карточки treats, гигантский вордмарк) → dark-футер с pill-карточками, у лендинга все секции однотонно-оливковые, а футер бежевый с прямоугольными карточками; трекинг вордмарка 0.08em вместо «размазанного» по всей ширине. На 1440 и 768 арка наезжает на нижнюю линейку хиро и обрезает вордмарк (в референсе линейка целиком над аркой).
• [composition] Клавиатура: skip-link, Tab по хиро → форме, стрелки в listbox выбирают слот, Esc закрывает меню, фокус уходит на «Закрыть» — всё работает; alt осмысленные; reduced-motion отключает reveal и прелоадер. Минусы: перекрытие хиро аркой и пилюлей на всех трёх ширинах, виджет слотов — плоская лента кнопок «вс 12:00» без дат и группировки по дням (на текущей неделе один день, подпись «31.08 — 06.09»), фикс overflow-x: hidden на 360 как страховка вместо причины.
• [content] Отзывы живые, различимы по голосу, услуги конкретные (цена, длительность, «без скрытых допов»), оффер сформулирован. Но по русскому лендингу разбросаны английские заголовки, дословно взятые у референса (tagline хиро, Past wisdom / Modern experiences, Step into tranquillity, Treatments/Gallery/Masters, Reach us, Your self-care awaits) — читается как «шкурка» референса, а не как текст студии.
• [frontend-code] CSS осмысленный: токены, BEM-подобные имена, один файл без мёртвых правил, всё локально (шрифты TTF, картинки, нет внешних ресурсов — проверено по DOM). JS формы читается, состояния (ошибка/успех/сеть/disabled submit) обрабатываются явно, слоты перезапрашиваются после SLOT_TAKEN. Минусы: пустой styles.css, body{overflow-x:hidden} как заплатка, сломанный отступ в обоих boot(), шрифты только TTF без woff2.
• [solution-doc, visual] Секция Reference — рабочий чек-лист, и всё обещанное на странице есть. Расхождения: после удаления модуля компонент физически удаляется, так что «ошибку модуля» показывать некому; таблица секций обещает «Sanctuary + rounded photo» и «Exclusive treats cards», но бежевой и тёмной секций у лендинга нет.
• [architecture-d7] Слои реальные: тонкие контроллеры, сервисы с Result/Error, репозитории за интерфейсами через ServiceLocator, почтовое событие, CAdminList/CAdminFilter/menu.php/права модуля, компонент по канону. Что не сделал бы опытный битриксовик: услуги и мастера тянутся прямо в index.php через ServiceLocator вместо компонента; в админке слотов постраничная навигация отсутствует — жёсткий лимит 500 при 560 слотах (19.09 без фильтра не показать), а в заявках навигация имитирована пустым CAdminResult; каталог читается legacy CIBlockElement::GetList при выставленном API_CODE (обоснование в SOLUTION.md неубедительно).
• [data] Контракт по таблицам и колонкам соблюдён, инфоблоки services/masters с 8 и 4 элементами созданы миграцией, слоты 14 дней × 10 (в БД 560), шаблон сайта назначается миграцией. Гонка проверена: 6 параллельных create на один слот → ровно один bookingId, остальные SLOT_TAKEN. Мелочь: расписание считается от даты миграции (today), повторный up корректно пропускает мастеров со слотами.
• [errors-security] Коды и фильтры по контракту, вывод экранирован, согласие — чекбокс с отметкой времени, админка закрыта GetGroupRight (для группы «все» право D, меню не строится). Минусы: slotId=abc возвращает стек-трейс ядра с путями сервера (включён debug в .settings.php стенда — не код модуля, но ответ течёт); $_REQUEST['action_target'] читается без isset.
• [readability] Имена и размеры методов хорошие, сервисы читаются сразу. Шум: дублированный пролог двух админ-страниц, две почти одинаковые приватные выборки в каталоге, бессмысленный eventType() и method_exists в инсталляторе, неиспользуемые Reference-поля ENTRY/SLOT в таблетках.
• [code-style] Библиотека модуля по AGENTS.md: strict_types, final/readonly, типизированные сигнатуры, <?= в шаблонах, Loc в админке и инсталляторе, код только в /local (стабы в /bitrix/admin — штатный приём). Отклонения: $_REQUEST в админ-страницах вместо Request, strict_types нет в admin/*, install/index.php, миграциях и шаблоне, тексты компонента захардкожены без Loc.
• [solution-doc, backend] Инженерные решения названы и совпадают с кодом, отклонения (закрытый слот как taken, нормализация weekStart, EMAIL_TO) честно перечислены и подтверждаются на стенде. Слабые места: обоснование legacy-API сомнительное (у инфоблоков есть API_CODE, D7-ORM был доступен), фраза про поведение после удаления модуля не соответствует UnInstallFiles.

Итого ревьювера: впечатление решило хиро: рядом с antaraspawellness.com оно узнаётся сразу — фото, тонкий вордмарк Fraunces, линейка, пилюля Menu и арка, а форма записи работает с мыши и клавиатуры без перезагрузки. Главный плюс — бэкенд: честные слои Controller → Service → Repository на ServiceLocator, гонка за слот закрыта уникальным индексом и FOR UPDATE (6 параллельных запросов дали ровно одну запись), всё воспроизводится тремя миграциями sprint.migration, админка на штатных CAdminList/CAdminFilter с правами и меню. Главный минус — визуал после первого экрана: арка наезжает на нижнюю линейку хиро на всех ширинах, секции однотонно-оливковые без ритма beige/dark референса, по русскому тексту рассыпаны английские заголовки референса; в админке слотов нет постраничной навигации, и 60 слотов последнего дня не видны без фильтра.