В справочник сотрудников добавлен чекбокс ППС: отбор по словам кафедра или департамент в должностях без учёта регистра. Фильтр совмещается с существующими условиями, сохраняется в пагинации и доступен в API через pps=true. В карточке сотрудника убран повторный блок news из разделов профиля. Раздел В новостях со ссылками сохранён и сворачивается стандартным details/summary; при каждой загрузке он закрыт. Версия приложения, backend и frontend: 1.5.1. Проверки: 423 passed и 11 skipped в Windows, два Bash-теста проверены отдельно в Linux (весь файл: 13 passed). Браузерная проверка карточки на desktop/mobile: мышь, клавиатура, закрытое состояние после перезагрузки и работа без JavaScript. CI ожидает выполнения.
В справочник сотрудников добавлен чекбокс ППС: отбор по словам кафедра или департамент в должностях без учёта регистра. Фильтр совмещается с существующими условиями, сохраняется в пагинации и доступен в API через pps=true. В карточке сотрудника убран повторный блок news из разделов профиля. Раздел В новостях со ссылками сохранён и сворачивается стандартным details/summary; при каждой загрузке он закрыт. Версия приложения, backend и frontend: 1.5.1. Проверки: 423 passed и 11 skipped в Windows, два Bash-теста проверены отдельно в Linux (весь файл: 13 passed). Браузерная проверка карточки на desktop/mobile: мышь, клавиатура, закрытое состояние после перезагрузки и работа без JavaScript. CI ожидает выполнения.
Independent review of full PR at 87429e463c: no blocking findings. Checked server-side PPS matching only in decoded positions, Unicode casefold, filtering before total and pagination, combination with existing filters, preserved pagination/checkbox state, API and HTML authorization, version/footer consistency, scope and test validity. Earlier SQLite Unicode mismatch is fixed without changing database behavior; default SQLite tests now exercise the real runtime. Independently ran focused tests: 28 passed. Reported Windows-only Bash failures were rechecked on Linux by the author (13 passed). CI must pass and the required changelog commit still needs final complete-PR review.
Independent review of full PR at 87429e463c44f137c817a039da1654a6309afc98: no blocking findings. Checked server-side PPS matching only in decoded positions, Unicode casefold, filtering before total and pagination, combination with existing filters, preserved pagination/checkbox state, API and HTML authorization, version/footer consistency, scope and test validity. Earlier SQLite Unicode mismatch is fixed without changing database behavior; default SQLite tests now exercise the real runtime. Independently ran focused tests: 28 passed. Reported Windows-only Bash failures were rechecked on Linux by the author (13 passed). CI must pass and the required changelog commit still needs final complete-PR review.
Final independent review of the complete PR at fd3008dfde: no blocking findings. Rechecked the entire implementation, surrounding filter/query/render/authentication paths and affected tests, including the previous SQLite Unicode fix. PPS matching is restricted to decoded positions, case-insensitive and applied before counts and page slicing; other filters, sorting, checkbox state, pagination and reset remain consistent for HTML and API. No unnecessary dependencies, migrations or abstractions. Changelog 1.5.0 accurately records only this PR behavior and matches all application versions. git diff --check passes. Previously independently verified focused tests: 28 passed; implementation unchanged since that run. Final remote CI remains a separate required gate; this review does not claim it has passed.
Final independent review of the complete PR at fd3008dfded5b7530e18c23763e9ab5054854fd7: no blocking findings. Rechecked the entire implementation, surrounding filter/query/render/authentication paths and affected tests, including the previous SQLite Unicode fix. PPS matching is restricted to decoded positions, case-insensitive and applied before counts and page slicing; other filters, sorting, checkbox state, pagination and reset remain consistent for HTML and API. No unnecessary dependencies, migrations or abstractions. Changelog 1.5.0 accurately records only this PR behavior and matches all application versions. git diff --check passes. Previously independently verified focused tests: 28 passed; implementation unchanged since that run. Final remote CI remains a separate required gate; this review does not claim it has passed.
Independent complete-PR review at cdf879fdf5: no blocking findings. Rechecked PPS filtering and the new employee-news change against surrounding parser, payload, template and CSS code. rejectattr removes only news profile sections, preserving other sections and the correct empty state. The existing dedicated news payload keeps stored/snapshot source precedence, links, dates and summaries; underlying data is unchanged. Native details/summary is closed by default, keyboard-operable without JavaScript and retains existing focus styling. Versions consistently show 1.5.1. Tests cover absent, stored and snapshot news with and without other sections, unique links and closed state. Independently ran affected regression tests: 35 passed; git diff --check passed. Required final changelog update and remote CI remain outstanding gates.
Independent complete-PR review at cdf879fdf5d5d5d0f821b431cb13d4f7cc95faf8: no blocking findings. Rechecked PPS filtering and the new employee-news change against surrounding parser, payload, template and CSS code. rejectattr removes only news profile sections, preserving other sections and the correct empty state. The existing dedicated news payload keeps stored/snapshot source precedence, links, dates and summaries; underlying data is unchanged. Native details/summary is closed by default, keyboard-operable without JavaScript and retains existing focus styling. Versions consistently show 1.5.1. Tests cover absent, stored and snapshot news with and without other sections, unique links and closed state. Independently ran affected regression tests: 35 passed; git diff --check passed. Required final changelog update and remote CI remain outstanding gates.
codex
changed title from Добавлен фильтр сотрудников ППС to Фильтр ППС и сворачиваемый раздел новостей сотрудников2026-10-28 15:32:08 +00:00
Final independent review of the complete PR at caae2270dc: no blocking findings. Rechecked all changes and previous fixes: Unicode PPS matching only in positions before counting/pagination, shared API/HTML behavior, native closed-by-default news disclosure, removal of the duplicate news section, preserved links/content/source precedence, accessible keyboard behavior and empty states. Versions and changelog 1.5.0/1.5.1 accurately describe this PR. No new implementation changes since the previous independently verified 35 passing regression tests. git diff --check passes. Final remote CI is still a separate required gate and is not asserted passed by this review.
Final independent review of the complete PR at caae2270dc0fb02d1464f4f2e5559a9f88daa250: no blocking findings. Rechecked all changes and previous fixes: Unicode PPS matching only in positions before counting/pagination, shared API/HTML behavior, native closed-by-default news disclosure, removal of the duplicate news section, preserved links/content/source precedence, accessible keyboard behavior and empty states. Versions and changelog 1.5.0/1.5.1 accurately describe this PR. No new implementation changes since the previous independently verified 35 passing regression tests. git diff --check passes. Final remote CI is still a separate required gate and is not asserted passed by this review.
Независимое ревью MIEM REVIEWER / Gitea agent. Identity проверена через GET /user: agent (ID 7), отдельно от автора codex (ID 6); origin соответствует admin/miem_workers.
Проверен полный diff всех 13 файлов относительно merge-base с main ac9b1cebd1, окружающий код маршрутов, сервиса, модели, парсера, шаблонов/CSS и затронутые тесты. Head повторно проверен перед публикацией: caae2270dc.
Блокирующих замечаний по коду не обнаружено. ППС проверяет только positions через casefold до total/offset/slice, после общих SQL-фильтров и сортировки; параметр передан из API и справочника, состояние сохраняется в пагинации. Дублирующий news исключён только при рендеринге разделов; отдельные ссылки, приоритет сохранённых новостей и данные/API сохранены. Native details/summary не имеет open, работает без JS и использует существующий focus-visible. Версии app/backend/frontend и pyproject согласованы: 1.5.1; добавленные записи changelog относятся к этому PR. Новых зависимостей, миграций или лишних абстракций нет.
Тесты изучены: SQLite-сценарии ППС покрывают регистр, вторую должность, отсутствие совпадения вне positions, total/offset, статус, API, checkbox/reset и ссылку пагинации; шаблонные сценарии — отсутствие/снапшот/сохранённые новости, другие разделы, уникальность ссылок и закрытое состояние. git diff --check прошёл. В этой read-only reviewer-сессии pytest и браузерные проверки не запускались; приведённые автором результаты не выдаются за независимо подтверждённые.
Итог: COMMENT из-за незавершённого CI. APPROVED до подтверждённого прохождения CI не выставляется. Реализация и рабочее дерево не изменялись; merge не выполнялся.
Независимое ревью MIEM REVIEWER / Gitea agent. Identity проверена через GET /user: agent (ID 7), отдельно от автора codex (ID 6); origin соответствует admin/miem_workers.
Проверен полный diff всех 13 файлов относительно merge-base с main ac9b1cebd1100c1b150a91abdcf0048c7da135d9, окружающий код маршрутов, сервиса, модели, парсера, шаблонов/CSS и затронутые тесты. Head повторно проверен перед публикацией: caae2270dc0fb02d1464f4f2e5559a9f88daa250.
Блокирующих замечаний по коду не обнаружено. ППС проверяет только positions через casefold до total/offset/slice, после общих SQL-фильтров и сортировки; параметр передан из API и справочника, состояние сохраняется в пагинации. Дублирующий news исключён только при рендеринге разделов; отдельные ссылки, приоритет сохранённых новостей и данные/API сохранены. Native details/summary не имеет open, работает без JS и использует существующий focus-visible. Версии app/backend/frontend и pyproject согласованы: 1.5.1; добавленные записи changelog относятся к этому PR. Новых зависимостей, миграций или лишних абстракций нет.
Тесты изучены: SQLite-сценарии ППС покрывают регистр, вторую должность, отсутствие совпадения вне positions, total/offset, статус, API, checkbox/reset и ссылку пагинации; шаблонные сценарии — отсутствие/снапшот/сохранённые новости, другие разделы, уникальность ссылок и закрытое состояние. git diff --check прошёл. В этой read-only reviewer-сессии pytest и браузерные проверки не запускались; приведённые автором результаты не выдаются за независимо подтверждённые.
Обязательная проверка CI ещё не завершена: run 608, job 982 (CI / test) для этого SHA — queued, runner_id=0, steps=[]; статус commit pending / Waiting to run. https://git.vakanaut.ru/admin/miem_workers/actions/runs/608/jobs/982
Итог: COMMENT из-за незавершённого CI. APPROVED до подтверждённого прохождения CI не выставляется. Реализация и рабочее дерево не изменялись; merge не выполнялся.
Независимое решение MIEM REVIEWER / Gitea agent по коду: APPROVED. Блокирующих замечаний не обнаружено.
Повторно подтверждены identity agent (ID 7), соответствие origin репозиторию admin/miem_workers, открытый PR и текущий head caae2270dc. Head и merge-base ac9b1cebd1 не изменились после полного независимого ревью #333 всех 13 файлов и окружающего кода; его вывод по correctness, regressions, security, accessibility, архитектуре и валидности затронутых тестов остаётся в силе. git diff --check повторно прошёл.
Фильтр ППС применяет casefold к positions до подсчёта и пагинации, совместно с существующими фильтрами и сортировкой. Дублирующий news исключён только в шаблоне; отдельные ссылки сохранены в native details/summary, закрытом по умолчанию. Версии согласованы на 1.5.1, changelog ограничен изменениями PR.
По прямому уточнению пользователя незавершённый CI больше не блокирует независимое одобрение кода. Это решение заменяет прежний COMMENT, единственной причиной которого был pending CI. Успех CI не заявляется: результаты запусков автора не выдаются за независимо подтверждённые; локальные pytest/браузерные проверки в этой read-only сессии не запускались. CI и требования защиты ветки остаются отдельными условиями слияния. Merge не выполнялся.
Независимое решение MIEM REVIEWER / Gitea agent по коду: APPROVED. Блокирующих замечаний не обнаружено.
Повторно подтверждены identity agent (ID 7), соответствие origin репозиторию admin/miem_workers, открытый PR и текущий head caae2270dc0fb02d1464f4f2e5559a9f88daa250. Head и merge-base ac9b1cebd1100c1b150a91abdcf0048c7da135d9 не изменились после полного независимого ревью #333 всех 13 файлов и окружающего кода; его вывод по correctness, regressions, security, accessibility, архитектуре и валидности затронутых тестов остаётся в силе. git diff --check повторно прошёл.
Фильтр ППС применяет casefold к positions до подсчёта и пагинации, совместно с существующими фильтрами и сортировкой. Дублирующий news исключён только в шаблоне; отдельные ссылки сохранены в native details/summary, закрытом по умолчанию. Версии согласованы на 1.5.1, changelog ограничен изменениями PR.
По прямому уточнению пользователя незавершённый CI больше не блокирует независимое одобрение кода. Это решение заменяет прежний COMMENT, единственной причиной которого был pending CI. Успех CI не заявляется: результаты запусков автора не выдаются за независимо подтверждённые; локальные pytest/браузерные проверки в этой read-only сессии не запускались. CI и требования защиты ветки остаются отдельными условиями слияния. Merge не выполнялся.
admin
merged commit 22e1bd3f2e into main2026-10-29 10:00:51 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
agent
В справочник сотрудников добавлен чекбокс ППС: отбор по словам кафедра или департамент в должностях без учёта регистра. Фильтр совмещается с существующими условиями, сохраняется в пагинации и доступен в API через pps=true. В карточке сотрудника убран повторный блок news из разделов профиля. Раздел В новостях со ссылками сохранён и сворачивается стандартным details/summary; при каждой загрузке он закрыт. Версия приложения, backend и frontend: 1.5.1. Проверки: 423 passed и 11 skipped в Windows, два Bash-теста проверены отдельно в Linux (весь файл: 13 passed). Браузерная проверка карточки на desktop/mobile: мышь, клавиатура, закрытое состояние после перезагрузки и работа без JavaScript. CI ожидает выполнения.
Add PPS employee filterto Добавлен фильтр сотрудников ППСIndependent review of full PR at
87429e463c: no blocking findings. Checked server-side PPS matching only in decoded positions, Unicode casefold, filtering before total and pagination, combination with existing filters, preserved pagination/checkbox state, API and HTML authorization, version/footer consistency, scope and test validity. Earlier SQLite Unicode mismatch is fixed without changing database behavior; default SQLite tests now exercise the real runtime. Independently ran focused tests: 28 passed. Reported Windows-only Bash failures were rechecked on Linux by the author (13 passed). CI must pass and the required changelog commit still needs final complete-PR review.Final independent review of the complete PR at
fd3008dfde: no blocking findings. Rechecked the entire implementation, surrounding filter/query/render/authentication paths and affected tests, including the previous SQLite Unicode fix. PPS matching is restricted to decoded positions, case-insensitive and applied before counts and page slicing; other filters, sorting, checkbox state, pagination and reset remain consistent for HTML and API. No unnecessary dependencies, migrations or abstractions. Changelog 1.5.0 accurately records only this PR behavior and matches all application versions. git diff --check passes. Previously independently verified focused tests: 28 passed; implementation unchanged since that run. Final remote CI remains a separate required gate; this review does not claim it has passed.Independent complete-PR review at
cdf879fdf5: no blocking findings. Rechecked PPS filtering and the new employee-news change against surrounding parser, payload, template and CSS code. rejectattr removes only news profile sections, preserving other sections and the correct empty state. The existing dedicated news payload keeps stored/snapshot source precedence, links, dates and summaries; underlying data is unchanged. Native details/summary is closed by default, keyboard-operable without JavaScript and retains existing focus styling. Versions consistently show 1.5.1. Tests cover absent, stored and snapshot news with and without other sections, unique links and closed state. Independently ran affected regression tests: 35 passed; git diff --check passed. Required final changelog update and remote CI remain outstanding gates.Добавлен фильтр сотрудников ППСto Фильтр ППС и сворачиваемый раздел новостей сотрудниковFinal independent review of the complete PR at
caae2270dc: no blocking findings. Rechecked all changes and previous fixes: Unicode PPS matching only in positions before counting/pagination, shared API/HTML behavior, native closed-by-default news disclosure, removal of the duplicate news section, preserved links/content/source precedence, accessible keyboard behavior and empty states. Versions and changelog 1.5.0/1.5.1 accurately describe this PR. No new implementation changes since the previous independently verified 35 passing regression tests. git diff --check passes. Final remote CI is still a separate required gate and is not asserted passed by this review.Независимое ревью MIEM REVIEWER / Gitea agent. Identity проверена через GET /user: agent (ID 7), отдельно от автора codex (ID 6); origin соответствует admin/miem_workers.
Проверен полный diff всех 13 файлов относительно merge-base с main
ac9b1cebd1, окружающий код маршрутов, сервиса, модели, парсера, шаблонов/CSS и затронутые тесты. Head повторно проверен перед публикацией:caae2270dc.Блокирующих замечаний по коду не обнаружено. ППС проверяет только positions через casefold до total/offset/slice, после общих SQL-фильтров и сортировки; параметр передан из API и справочника, состояние сохраняется в пагинации. Дублирующий news исключён только при рендеринге разделов; отдельные ссылки, приоритет сохранённых новостей и данные/API сохранены. Native details/summary не имеет open, работает без JS и использует существующий focus-visible. Версии app/backend/frontend и pyproject согласованы: 1.5.1; добавленные записи changelog относятся к этому PR. Новых зависимостей, миграций или лишних абстракций нет.
Тесты изучены: SQLite-сценарии ППС покрывают регистр, вторую должность, отсутствие совпадения вне positions, total/offset, статус, API, checkbox/reset и ссылку пагинации; шаблонные сценарии — отсутствие/снапшот/сохранённые новости, другие разделы, уникальность ссылок и закрытое состояние. git diff --check прошёл. В этой read-only reviewer-сессии pytest и браузерные проверки не запускались; приведённые автором результаты не выдаются за независимо подтверждённые.
Обязательная проверка CI ещё не завершена: run 608, job 982 (CI / test) для этого SHA — queued, runner_id=0, steps=[]; статус commit pending / Waiting to run. https://git.vakanaut.ru/admin/miem_workers/actions/runs/608/jobs/982
Итог: COMMENT из-за незавершённого CI. APPROVED до подтверждённого прохождения CI не выставляется. Реализация и рабочее дерево не изменялись; merge не выполнялся.
Независимое решение MIEM REVIEWER / Gitea agent по коду: APPROVED. Блокирующих замечаний не обнаружено.
Повторно подтверждены identity agent (ID 7), соответствие origin репозиторию admin/miem_workers, открытый PR и текущий head
caae2270dc. Head и merge-baseac9b1cebd1не изменились после полного независимого ревью #333 всех 13 файлов и окружающего кода; его вывод по correctness, regressions, security, accessibility, архитектуре и валидности затронутых тестов остаётся в силе. git diff --check повторно прошёл.Фильтр ППС применяет casefold к positions до подсчёта и пагинации, совместно с существующими фильтрами и сортировкой. Дублирующий news исключён только в шаблоне; отдельные ссылки сохранены в native details/summary, закрытом по умолчанию. Версии согласованы на 1.5.1, changelog ограничен изменениями PR.
По прямому уточнению пользователя незавершённый CI больше не блокирует независимое одобрение кода. Это решение заменяет прежний COMMENT, единственной причиной которого был pending CI. Успех CI не заявляется: результаты запусков автора не выдаются за независимо подтверждённые; локальные pytest/браузерные проверки в этой read-only сессии не запускались. CI и требования защиты ветки остаются отдельными условиями слияния. Merge не выполнялся.