From e29775b23855feaa89ddb96a44fc522b1420c8de Mon Sep 17 00:00:00 2001 From: codex Date: Mon, 12 Oct 2026 15:56:32 +0300 Subject: [PATCH] fix: normalize guide CSS line endings --- .codex-session.md | 14 ++++++++------ app/static/admin.css | 6 +++--- app/version.py | 6 +++--- pyproject.toml | 2 +- tests/test_api.py | 2 +- tests/test_council.py | 2 +- 6 files changed, 17 insertions(+), 15 deletions(-) diff --git a/.codex-session.md b/.codex-session.md index a3f2c63..945ab62 100644 --- a/.codex-session.md +++ b/.codex-session.md @@ -1,18 +1,20 @@ ## Сделано -- Ветка `feature/first-login-guide`, версия приложения/frontend/backend `1.3.0`. +- Ветка `feature/first-login-guide`, версия приложения/frontend/backend `1.3.1`. - Шесть шагов подсказок после первой смены пароля, повторное открытие из меню, серверное сохранение завершения в истории действий без миграции БД. - Полная инструкция и завершение без JavaScript; при ошибке сохранения можно продолжить работу без отметки о завершении. -- Проверки: `pytest -q` — 412 passed, 11 skipped; браузерный сценарий `tests/browser_users.py` — passed (1440/320 px, клавиатура, ошибка сохранения, все шаги, без JavaScript); синтаксис всех четырёх JS-файлов и двух shell-скриптов CI — passed; `git diff --check` — passed. +- Исправлено неблокирующее замечание независимого ревью PR #71: только три новых правила CSS переведены с CRLF на LF, содержимое правил сохранено. +- После исправления: `pytest -q` — 412 passed, 11 skipped; `git diff --check main` — passed. Предыдущая запись о passed для полного diff коммита `5128fac` была неточной: ревью подтвердило три trailing whitespace, теперь они устранены. +- До этой правки: браузерный сценарий `tests/browser_users.py` — passed (1440/320 px, клавиатура, ошибка сохранения, все шаги, без JavaScript); синтаксис четырёх JS-файлов и двух shell-скриптов CI — passed. Браузерный сценарий повторно не запускался: поведение CSS не менялось. ## Остановились на -- Реализация сохранена коммитом `5128fac` (`feat: add first-login usage guide`). Открытого PR на момент проверки нет. -- Push и повторное обновление `main` заблокированы: DNS разрешается, но TCP 443 не отвечает; Git завершился после 128 секунд с `Failed to connect ... Couldn't connect to server`. Проверка вне sandbox также завершилась таймаутом. Первая команда `git fetch origin` прошла; основа ветки и локальный/удалённый `main` на момент проверки — `378d7ee`. +- Открыт PR #71: https://git.vakanaut.ru/admin/miem_workers/pulls/71. Ревью коммита `5128fac` одобрено с одним неблокирующим замечанием; пользователь поручил исправить его. +- Правка готова к коммиту и push в ту же ветку/PR. Доступ к Gitea восстановился; локальный `main` обновлён, основа — `378d7ee`. - PostgreSQL-проверки пропущены без отдельной тестовой БД. Проверки выполнялись на синтетических SQLite-данных. - Независимое ревью должно выполняться в отдельной сессии после публикации PR, согласно инструкциям пользователя. ## Следующий шаг -- После восстановления доступа выполнить `git push -u origin feature/first-login-guide`, повторно найти открытый PR через `gitea-api GET /repos/admin/miem_workers/pulls?state=open&base=main` и создать единственный PR в `main`, если отсутствует. -- После публикации остановиться и ждать независимого ревью и дальнейших указаний пользователя. Не выполнять ревью из авторской сессии и не сливать PR. +- Сохранить правку коммитом, отправить `git push -u origin feature/first-login-guide` в существующий PR #71. +- После push остановиться и ждать независимого ревью и дальнейших указаний пользователя. Не выполнять ревью из авторской сессии и не сливать PR. diff --git a/app/static/admin.css b/app/static/admin.css index 140d67c..2a69e02 100644 --- a/app/static/admin.css +++ b/app/static/admin.css @@ -967,9 +967,9 @@ .users__history-table .table__cell:nth-child(2) { min-width: 100px; white-space: nowrap; } -.guide__panel { box-sizing: border-box; display: flex; flex-direction: column; overflow: hidden; } -.guide__panel > :not([data-guide-step]) { flex-shrink: 0; } -.guide__panel .guide__step { min-height: 0; overflow-y: auto; overscroll-behavior: contain; } +.guide__panel { box-sizing: border-box; display: flex; flex-direction: column; overflow: hidden; } +.guide__panel > :not([data-guide-step]) { flex-shrink: 0; } +.guide__panel .guide__step { min-height: 0; overflow-y: auto; overscroll-behavior: contain; } .guide__progress { margin-block: 16px; color: #374151; font-variant-numeric: tabular-nums; } .guide__heading { margin-block: 20px 12px; font-size: 18px; } .guide__fields { margin-block: 16px; } diff --git a/app/version.py b/app/version.py index 58c2ce9..10e3afd 100644 --- a/app/version.py +++ b/app/version.py @@ -1,3 +1,3 @@ -APP_VERSION = "1.3.0" -FRONTEND_VERSION = "1.3.0" -BACKEND_VERSION = "1.3.0" +APP_VERSION = "1.3.1" +FRONTEND_VERSION = "1.3.1" +BACKEND_VERSION = "1.3.1" diff --git a/pyproject.toml b/pyproject.toml index bb99ce5..371303f 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -1,6 +1,6 @@ [project] name = "miem-workers" -version = "1.3.0" +version = "1.3.1" description = "MIEM employees parser, admin API, and web admin" requires-python = ">=3.11" dependencies = [ diff --git a/tests/test_api.py b/tests/test_api.py index 32df934..751f8f3 100644 --- a/tests/test_api.py +++ b/tests/test_api.py @@ -18,7 +18,7 @@ def test_health_returns_versions(): response = TestClient(app).get("/api/health") assert response.status_code == 200 - assert response.json()["backend_version"] == "1.3.0" + assert response.json()["backend_version"] == "1.3.1" def test_api_employees_and_stats_require_admin_session(): diff --git a/tests/test_council.py b/tests/test_council.py index 0c44d84..7d08317 100644 --- a/tests/test_council.py +++ b/tests/test_council.py @@ -223,7 +223,7 @@ def test_public_page_data_minimization_and_safe_links(db_session, export): assert "javascript:" not in fragment.text and "private@example.org" not in fragment.text assert "javascript:" not in response.text and "private@example.org" not in response.text assert "/admin/employees/" not in response.text and "/admin/logout" not in response.text - assert "Backend 1.3.0 · Frontend 1.3.0" in response.text + assert "Backend 1.3.1 · Frontend 1.3.1" in response.text assert "Вход в систему" in response.text assert client.get("/admin/directory", follow_redirects=False).status_code in (302, 303, 401, 403) assert "Сотрудники не найдены" in client.get("/admin/council?q=missing").text