feat: persist search review drafts #27

Merged
admin merged 2 commits from feature/search-review-drafts into main 2026-08-28 05:13:09 +00:00
Owner

Реализует недостающий критерий Issue #23: черновик разметки сохраняется в localStorage и восстанавливается после перезагрузки. После успешного сохранения черновик удаляется. Обновлена версия до 0.8.1 и changelog. Проверки: unittest discover, Node syntax check, git diff --check.

Реализует недостающий критерий Issue #23: черновик разметки сохраняется в localStorage и восстанавливается после перезагрузки. После успешного сохранения черновик удаляется. Обновлена версия до 0.8.1 и changelog. Проверки: unittest discover, Node syntax check, git diff --check.
agent added 1 commit 2026-08-28 04:30:09 +00:00
Author
Owner

Независимый review: есть два blocking finding. (1) input/change слушатели добавлены на #review, но поля «Проверяющий» и «Общий комментарий» находятся в .review__actions за пределами main #review. Их изменения не сохраняют черновик; после перезагрузки легко потерять имя и общий комментарий. Повесьте обработчик на document/body либо на форму, включающую action panel, и добавьте поведенческую проверку. (2) В backend/search/review.html footer всё ещё показывает Backend v0.8.0, тогда как приложение и документация обновлены до 0.8.1. Это нарушает обязательное отображение текущей версии. CHANGELOG.md и остальные version bump корректны.

Независимый review: есть два blocking finding. (1) input/change слушатели добавлены на #review, но поля «Проверяющий» и «Общий комментарий» находятся в .review__actions за пределами main #review. Их изменения не сохраняют черновик; после перезагрузки легко потерять имя и общий комментарий. Повесьте обработчик на document/body либо на форму, включающую action panel, и добавьте поведенческую проверку. (2) В backend/search/review.html footer всё ещё показывает Backend v0.8.0, тогда как приложение и документация обновлены до 0.8.1. Это нарушает обязательное отображение текущей версии. CHANGELOG.md и остальные version bump корректны.
agent added 1 commit 2026-08-28 04:31:31 +00:00
Author
Owner

Повторный независимый review полного PR после 4210c24 завершён: blocking findings нет. Обработчики на document теперь охватывают поля из action panel, поэтому reviewer и общий комментарий входят в localStorage-черновик; footer показывает Backend v0.8.1. Очистка черновика остаётся только после успешного POST, новых регрессий не выявлено.

Повторный независимый review полного PR после 4210c24 завершён: blocking findings нет. Обработчики на document теперь охватывают поля из action panel, поэтому reviewer и общий комментарий входят в localStorage-черновик; footer показывает Backend v0.8.1. Очистка черновика остаётся только после успешного POST, новых регрессий не выявлено.
admin merged commit b91bc98611 into main 2026-08-28 05:13:09 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: agent/akyldash#27