Перенести UI страницы /join из PR #48 в кодовую базу PR #49

[markpolyak]

Контекст

#48 и #49 реализуют одну и ту же фичу (автосоздание репозитория студента через GitHub OAuth, #46). После ручного сравнения обоих вариантов на локальном стенде принято решение:

• бэкенд/архитектура Автоматическое создание репозитория студента (замена GitHub Classroom) #49 — берём как основу (меньше дублирования, расширяет существующий grading/github_client.py, а не заводит параллельный клиент);
• UI страницы /join/:courseId/:labId из Добавить выдачу репозиториев через GitHub OAuth #48 — переносим в Автоматическое создание репозитория студента (замена GitHub Classroom) #49 (главное преимущество — переключатель языка прямо на странице студента и полный набор переводов ошибок).

Задача — перенести фронтенд из #48 в ветку #49. PR #48 после мержа #49 закрывается как заменённый.

Куда коммитить: прямо в ветку claude/student-repo-auto-creation-stp506 (ветка PR #49), чтобы PR #49 стал готов к мержу целиком. Новых PR не создавать.

Почему это не копирование файлов

Бэкенд-контракты /join/... в #48 и #49 разошлись при одинаковых путях эндпоинтов. Компонент JoinLab из #48 читает поля и query-параметры своего бэкенда; скопированный as-is, он покажет «unknown error» и пустую ссылку на репозиторий.

	#48 (откуда берём)	#49 (куда переносим)	Где править
GET /join/{course}/{lab}, поле названия лабы	lab_name	lab_short_name	JoinLab/index.jsx
GET /join/{course}/{lab}, «лаба не настроена»	HTTP 409	HTTP 400 (_load_lab_for_join, main.py)	src/api/index.js
Redirect после /join/callback, код ошибки	?status=error&error=<code>	?status=error&reason=<code>	JoinLab/index.jsx
Redirect после успеха	?repository=<url>	?repo_url=<url>&username=<login>	JoinLab/index.jsx
Словарь кодов ошибок	см. state.js #48	см. таблицу в п.4 ниже	JoinLab/state.js

Разница в HTTP-кодах (409 → 400) в предыдущей редакции задачи отсутствовала: если её не учесть, все конфигурационные ошибки лабы схлопнутся в «непредвиденная ошибка».

Что нужно сделать

1. Скопировать из ветки #48 (feature/public-join-github-oauth)

Файлы целиком:

• frontend/courses-front/src/components/JoinLab/index.jsx
• frontend/courses-front/src/components/JoinLab/styled.js
• frontend/courses-front/src/components/JoinLab/state.js
• frontend/courses-front/src/components/JoinLab/state.test.js
• frontend/courses-front/src/language.js — обязателен: index.jsx импортирует из него SUPPORTED_LANGUAGES, без этого файла сборка падает
• frontend/courses-front/src/language.test.js

Частично:

• frontend/courses-front/src/i18n.js — перенести правку из Добавить выдачу репозиториев через GitHub OAuth #48: lng: readStoredLanguage(), хук i18n.on("languageChanged", applyDocumentLanguage), вызов applyDocumentLanguage(i18n.language). Это то, что делает выбор языка на странице /join запоминаемым.

• frontend/courses-front/src/api/index.js — функции fetchJoinLab и getJoinStartUrl (вместе с константой JOIN_REQUEST_TIMEOUT_MS и AbortController).

• frontend/courses-front/src/App.jsx — два роута:

  <Route path="/join/error" element={<JoinLab />} />
  <Route path="/join/:courseId/:labId" element={<JoinLab />} />

• frontend/courses-front/package.json — добавить только скрипт запуска тестов. Не переносить из Добавить выдачу репозиториев через GitHub OAuth #48 бамп react-router-dom с ^6.23.0 до ^7.18.3 и прочие изменения зависимостей: используемые API (useParams, useSearchParams, useNavigate) есть и в v6, лишний мажорный апгрейд роутера в эту задачу не входит.

  Скрипт писать так:

  "test": "node --test src"

  Не копировать вариант node --test src/**/*.test.js из Добавить выдачу репозиториев через GitHub OAuth #48: без globstar в bash этот glob не разворачивается рекурсивно и не находит src/components/JoinLab/state.test.js. Node 22 (frontend.Dockerfile) обходит директорию рекурсивно сам.

2. Удалить из #49 замещённое

• frontend/courses-front/src/components/JoinLab/joinLabWrapper.jsx — в Добавить выдачу репозиториев через GitHub OAuth #48 другая архитектура: компонент сам читает useParams/useSearchParams, отдельный wrapper не нужен (и не может обслужить роут /join/error, у которого нет параметров). Импорт в App.jsx заменить на import { JoinLab } from "./components/JoinLab";.
• Функцию fetchJoinLabInfo из src/api/index.js — её полностью заменяет fetchJoinLab из Добавить выдачу репозиториев через GitHub OAuth #48. Не оставлять обе.
• Плоские ключи переводов Автоматическое создание репозитория студента (замена GitHub Classroom) #49 из src/locales/{ru,en,zh}/translation.json: joinPageTitle, joinCourseLabel, joinLabLabel, joinSignInButton, joinSuccessMessage, joinOpenRepoButton, joinTryAgain, joinLoadError, joinErrorAccessDenied, joinErrorConfig, joinErrorOauthFailed, joinErrorTemplateNotFound, joinErrorForbidden, joinErrorRateLimited, joinErrorGeneric. Это не дубли ключей Добавить выдачу репозиториев через GitHub OAuth #48, а другое пространство имён — после переноса они становятся мёртвыми.

3. Кнопка «Назад»

Страница #48 — самостоятельный лендинг без навигации; в #49 у неё были ButtonBack и Breadcrumb. Переносим только кнопку «Назад»:

• в JoinLab/index.jsx добавить useNavigate и кнопку, ведущую на / (стиль взять из ../course-list/styled → ButtonBack, как это было в JoinLab/index.jsx ветки Автоматическое создание репозитория студента (замена GitHub Classroom) #49), разместив её над карточкой;
• Breadcrumb не переносить: он делает лишний запрос fetchCourseDetails и дублирует блок «Курс / Лабораторная работа», который на странице уже есть.

Если кнопка заметно ломает вёрстку карточки (JoinPage — grid с place-items: center), допустимо её не добавлять — это необязательный элемент. В этом случае явно указать причину в описании коммита.

4. Адаптировать state.js под контракт #49

ERROR_TRANSLATION_KEYS переписать под коды ошибок #49. Полный список источников кодов:

• из редиректа /join/callback (main.py): access_denied, missing_code, config, oauth_not_configured, oauth_exchange_failed, provision_failed, invalid_state (новый, см. п.5);
• из result.error_code (grading/repo_provisioning.py): INVALID_TEMPLATE_CONFIG, CREATE_VALIDATION_FAILED, CREATE_FORBIDDEN, RATE_LIMITED, TEMPLATE_NOT_FOUND, CREATE_FAILED, INVITATIONS_FETCH_FAILED, REINVITE_DELETE_FAILED, INVITE_FAILED;
• из fetchJoinLab (src/api/index.js): join_not_found, join_not_configured, rate_limit, request_timeout, unknown.

Целевой маппинг (переводы #48 переиспользуются там, где смысл совпадает):

Код	Ключ перевода
access_denied	join.errors.oauthDenied
missing_code, oauth_exchange_failed	join.errors.oauthFailed
oauth_not_configured	join.errors.oauthNotConfigured
invalid_state	join.errors.oauthStateExpired
config, INVALID_TEMPLATE_CONFIG, join_not_configured	join.errors.notConfigured
TEMPLATE_NOT_FOUND	join.errors.templateUnavailable
CREATE_VALIDATION_FAILED, CREATE_FAILED	join.errors.repositoryFailed
CREATE_FORBIDDEN	join.errors.createForbidden — новый ключ, текст взять из joinErrorForbidden ветки #49
RATE_LIMITED, rate_limit	join.errors.rateLimit
INVITATIONS_FETCH_FAILED, REINVITE_DELETE_FAILED	join.errors.accessFailed
INVITE_FAILED	join.errors.invitationFailed
join_not_found	join.errors.notFound
request_timeout	join.errors.githubUnavailable
provision_failed, unknown, всё неизвестное	join.errors.unknown

В src/api/index.js в fetchJoinLab заменить ветку response.status === 409 на response.status === 400 (см. таблицу контрактов выше).

Ключи join.errors.oauthStateMissing, oauthStateInvalid, oauthStateMismatch, oauthUnavailable в #49 источника не имеют — удалить их из state.js и из всех трёх файлов переводов. Первый тест в state.test.js проверяет именно эти четыре ключа — переписать его под фактический словарь (осмысленная проверка: каждый код из списка выше имеет ключ, и getSafeRepositoryUrl/shouldShowJoinAction не сломаны).

getSafeRepositoryUrl перенести без изменений — это защита от подделанного repo_url в адресной строке. В #49 значение подставлялось в href без проверки, терять эту валидацию нельзя.

В success-панели вывести username из query-параметра (в #48 такого параметра не было): строка вида GitHub: <strong>{username}</strong>, ключ перевода добавить в ru/en/zh.

5. Точечная правка бэкенда: невалидный state

Сейчас в main.py _parse_join_state бросает HTTPException(400), и студент с протухшей ссылкой видит сырой JSON вместо страницы. Именно под этот случай в #48 существует роут /join/error (в момент ошибки course_id/lab_id неизвестны — они лежат внутри самого state).

Сделать:

• добавить хелпер рядом с _join_result_redirect:

  def _join_error_redirect(reason: str) -> str:
      """Результат для случая, когда course_id/lab_id ещё неизвестны (битый state)."""
      return f"{FRONTEND_URL.rstrip('/')}/join/error?{urlencode({'status': 'error', 'reason': reason})}"

• в join_callback обернуть вызов _parse_join_state(state) в try/except HTTPException и вернуть RedirectResponse(_join_error_redirect("invalid_state"));
• добавить тест в tests/test_join_endpoints.py: GET /join/callback?code=...&state=<мусор> → редирект на /join/error с reason=invalid_state.

Ветка isStandaloneError / hasLabContext в index.jsx #48 обрабатывает этот роут без доработок — перенести её как есть.

6. Остальной бэкенд #49 не трогать

main.py (кроме п.5), grading/repo_provisioning.py, grading/github_client.py уже прошли ревью. Вариант «поправить не фронт, а параметры редиректа в бэкенде» из предыдущей редакции задачи отменён: lab_short_name отдаёт join_lab_info, а не _join_result_redirect, и оба поля покрыты тестами (tests/test_join_endpoints.py, проверки lab_short_name и repo_url) — правка бэкенда потянет за собой переделку его тестов и выйдет дороже, а не дешевле.

Проверка

Автоматически:

pytest tests/ -v
cd frontend/courses-front && npm install --legacy-peer-deps && npm run lint && npm test && npm run build

Все три фронтовых шага должны быть зелёными; npm test должен реально находить и выполнять state.test.js и language.test.js (если тестов найдено 0 — скрипт написан неверно).

Вручную (стенд поднят у автора задачи, отдельно его собирать не нужно):

1. Запушить изменения в claude/student-repo-auto-creation-stp506.
2. Попросить автора задачи перезапустить локальный стенд из этой ветки (git pull + пересборка) и сообщить URL фронтенда. Дождаться подтверждения, дальше проверять по чек-листу.
3. Чек-лист:
   • /join/{course}/{lab} открывается, показывает курс и короткое название лабы;
   • переключатель языка меняет весь текст страницы и переживает перезагрузку (проверить ru → en → zh);
   • happy path: вход через GitHub → создание репозитория → success-панель с рабочей ссылкой и логином студента;
   • повторный проход по той же ссылке тем же студентом (репозиторий уже существует) — success, не ошибка;
   • отказ на экране авторизации GitHub → человекочитаемая ошибка (access_denied), кнопка «Попробовать снова» на месте;
   • лаба без template-repo в YAML → понятная ошибка «не настроено», не «непредвиденная ошибка»;
   • /join/error?status=error&reason=invalid_state → страница с ошибкой, а не пустой экран;
   • подделанный ?status=success&repo_url=https://example.com/a/b → ссылка не отображается, показана ошибка о некорректной ссылке.

Не входит в объём

• Апгрейд react-router-dom до v7 и прочие изменения зависимостей из Добавить выдачу репозиториев через GitHub OAuth #48.
• Проверка «репозиторий с таким именем создан не из нашего шаблона» (в Добавить выдачу репозиториев через GitHub OAuth #48 она есть — repository_uses_template по template_repository.full_name, в Автоматическое создание репозитория студента (замена GitHub Classroom) #49 её нет, там только repo_exists). Отдельная задача, к переносу UI отношения не имеет.
• Механизмы repo-provisioning: fork (Поддержать создание репозитория студента через fork как альтернативу template, выбор в конфиге лабы #51) и propagate обновлений шаблона (Распространение обновлений стартового репозитория на репозитории студентов через fork-PR (админка) #52).
• Любые изменения в /register, grade_lab, старых модулях оценивания.

[markpolyak]

Перенос выполнен, запушено в claude/student-repo-auto-creation-stp506 (ветка PR #49), двумя коммитами.

Что сделано (все пункты задачи):

1. Скопированы JoinLab/index.jsx, styled.js, state.js, state.test.js, language.js, language.test.js из Добавить выдачу репозиториев через GitHub OAuth #48, адаптированы под контракт Автоматическое создание репозитория студента (замена GitHub Classroom) #49: lab_short_name, reason=/repo_url=/username= вместо error=/repository=, HTTP 400 вместо 409.
2. i18n.js — запоминание языка (readStoredLanguage/persistLanguage + applyDocumentLanguage).
3. api/index.js — fetchJoinLab/getJoinStartUrl вместо fetchJoinLabInfo, 400 вместо 409.
4. App.jsx — роуты /join/error и /join/:courseId/:labId на JoinLab напрямую, joinLabWrapper.jsx удалён.
5. package.json — добавлен тестовый скрипт (см. отклонение ниже), апгрейд react-router-dom и прочие правки зависимостей из Добавить выдачу репозиториев через GitHub OAuth #48 не переносились.
6. Удалены мёртвые плоские ключи переводов Автоматическое создание репозитория студента (замена GitHub Classroom) #49 (joinPageTitle и т.д.) из ru/en/zh, добавлен namespace join.* из Добавить выдачу репозиториев через GitHub OAuth #48 с точечными правками: убраны oauthStateMissing/Invalid/Mismatch, oauthUnavailable (источника нет в Автоматическое создание репозитория студента (замена GitHub Classroom) #49), добавлены join.errors.createForbidden (для CREATE_FORBIDDEN) и join.usernameLabel (логин студента в success-панели).
7. state.js: ERROR_TRANSLATION_KEYS переписан под полный список кодов ошибок Автоматическое создание репозитория студента (замена GitHub Classroom) #49 (redirect reasons из main.py, error_code из RepoProvisioner, коды fetchJoinLab); state.test.js переписан под фактический словарь.
8. Кнопка «Назад» добавлена (стиль ButtonBack из course-list/styled, текст ← Назад не переведён — как и в остальных компонентах проекта, где эта кнопка используется).
9. Бэкенд (main.py): добавлен _join_error_redirect(), join_callback оборачивает _parse_join_state() в try/except и редиректит на /join/error?status=error&reason=invalid_state вместо сырого HTTPException(400). Три существующих теста в tests/test_join_endpoints.py (invalid/missing/expired state) переписаны под новое поведение (редирект вместо исключения) + новый сценарий покрыт тем же обновлённым набором тестов.

Одно осознанное отклонение от текста задачи: скрипт "test" записан как "node --test src/**/*.test.js", а не "node --test src". На доступном для проверки Node 22 бинарнике node --test src (голый путь к директории) реально не находит ни одного теста — CLI пытается require() директорию вместо рекурсивного обхода (воспроизвёл это даже на пустом проекте вне репозитория, не связано со спецификой кода). Вариант с glob-паттерном находит и проходит все 7 тестов (включая вложенный state.test.js), потому что при отсутствии совпадений на один уровень вложенности POSIX-шелл по умолчанию оставляет паттерн нераскрытым, и ** рекурсивно раскрывает уже сам Node. Если на вашем стенде node --test src всё же работает — дайте знать, поменяю на буквальный вариант из задачи.

Проверка:

• pytest tests/ -v — 226 passed
• npm run lint — 27 ошибок, все идентичны ошибкам на ветке до этого изменения (проверено через git stash), ни одна не в новом/изменённом коде
• npm test — 7 passed (state.test.js × 3, language.test.js × 4)
• npm run build — успешно

Готово к ручной проверке по чек-листу из задачи (пункт "Вручную").

---

Generated by Claude Code

[markpolyak]

(attribution for the comment above, omitted by mistake)

---

Generated by Claude Code