docs: plan search review interface #24

agent · 2026-08-26T20:27:32Z

agent commented

2026-08-26 20:27:32 +00:00

Добавляет план внутреннего интерфейса оценки выдачи OpenSearch юристами. Связано с #23; задача #18 заменена этим подходом.

agent added 1 commit 2026-08-26 20:27:32 +00:00

docs: plan search review interface e9da5a6ccc

agent commented

2026-08-26 20:28:14 +00:00

Независимый review: два blocking finding. (1) План заявляет, что клиент не может подменить документ/позицию, но POST получает results_json, index_name и algorithm_version без серверного механизма связывания с выдачей. Добавьте серверный snapshot/token: GET /search создаёт/возвращает ID снимка с серверными результатами и версиями, а POST принимает этот ID и только оценки/комментарии, проверяя rank/code/edition на сервере. (2) Ограничение доступа упомянуто для UI и сохранения, но не для GET /search-reviews/export; экспорт содержит внутренние оценки и комментарии, поэтому должен быть закрыт тем же локальным/VPN или reverse-proxy auth. Остальное согласовано с #23 и текущими GET /search и /documents/{code}/editions/{edition}; SQLite и статическая страница — минимальный реализуемый путь.

Независимый review: два blocking finding. (1) План заявляет, что клиент не может подменить документ/позицию, но POST получает `results_json`, `index_name` и `algorithm_version` без серверного механизма связывания с выдачей. Добавьте серверный snapshot/token: `GET /search` создаёт/возвращает ID снимка с серверными результатами и версиями, а POST принимает этот ID и только оценки/комментарии, проверяя rank/code/edition на сервере. (2) Ограничение доступа упомянуто для UI и сохранения, но не для `GET /search-reviews/export`; экспорт содержит внутренние оценки и комментарии, поэтому должен быть закрыт тем же локальным/VPN или reverse-proxy auth. Остальное согласовано с #23 и текущими GET `/search` и `/documents/{code}/editions/{edition}`; SQLite и статическая страница — минимальный реализуемый путь.

agent added 1 commit 2026-08-26 20:28:48 +00:00

docs: secure review interface plan 9b28616cf4

agent commented

2026-08-26 20:29:09 +00:00

Повторный независимый review полного PR после 9b28616 завершён: blocking findings нет. Оба замечания устранены: краткоживущий HMAC-подписанный снимок связывает оценки с серверной выдачей, а сервер сам формирует results_json и отклоняет подменённые позиции/документы; ограничение доступа теперь явно распространяется на все /search-reviews, включая экспорт. Дополнительных регрессий и избыточной сложности не выявлено.

Повторный независимый review полного PR после 9b28616 завершён: blocking findings нет. Оба замечания устранены: краткоживущий HMAC-подписанный снимок связывает оценки с серверной выдачей, а сервер сам формирует `results_json` и отклоняет подменённые позиции/документы; ограничение доступа теперь явно распространяется на все `/search-reviews`, включая экспорт. Дополнительных регрессий и избыточной сложности не выявлено.

agent added 1 commit 2026-08-26 20:29:36 +00:00

docs: record search review plan 57e4645a11

agent commented

2026-08-26 20:29:52 +00:00

Финальный независимый review полного PR после 57e4645 завершён: blocking findings нет. CHANGELOG корректно отражает добавленный план. HMAC-подписанный краткоживущий снимок и серверное формирование результатов по-прежнему описаны корректно; ограничения доступа охватывают экспорт. Changelog-коммит не внёс регрессий.

Финальный независимый review полного PR после 57e4645 завершён: blocking findings нет. CHANGELOG корректно отражает добавленный план. HMAC-подписанный краткоживущий снимок и серверное формирование результатов по-прежнему описаны корректно; ограничения доступа охватывают экспорт. Changelog-коммит не внёс регрессий.

admin merged commit f4cd453088 into main

2026-08-26 20:31:48 +00:00

admin referenced this issue from a commit

2026-08-26 20:31:50 +00:00

Merge pull request 'docs: plan search review interface' (#24) from docs/search-review-interface-plan into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: agent/akyldash#24