agent/akyldash
1
1
Fork 0
Code Issues 4 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: hide search highlight markup in snippets #34

Merged
admin merged 1 commits from fix/plain-search-snippets into main 2026-09-16 06:22:35 +00:00
Conversation 1 Commits 1 Files Changed 9 +15 -13
agent commented 2026-09-16 06:17:05 +00:00
Owner
Copy Link

Изменения

  • Скрыты технические теги <em> в поисковых фрагментах; сами совпавшие слова и текст сохранены.
  • Обновлена версия backend до 0.9.4.

Проверки

  • python3 -m unittest discover -s backend -v — 28 passed, 1 skipped (OpenSearch integration без настроенного URL).
  • git diff --check — clean.
## Изменения - Скрыты технические теги `<em>` в поисковых фрагментах; сами совпавшие слова и текст сохранены. - Обновлена версия backend до 0.9.4. ## Проверки - `python3 -m unittest discover -s backend -v` — 28 passed, 1 skipped (OpenSearch integration без настроенного URL). - `git diff --check` — clean.
agent added 1 commit 2026-09-16 06:17:05 +00:00
fix: hide search highlight markup in snippets 9fb266e8ac
agent commented 2026-09-16 06:18:19 +00:00
Author
Owner
Copy Link

APPROVE — независимо проверен полный diff PR #34 (9fb266e) относительно main (f045233) и релевантный flow. Backend удаляет OpenSearch <em>/</em> из snippet до передачи клиенту; UI выводит snippet через textContent, HTML-инъекция не появляется. Тест проверяет очищенный контракт API; локально python3 -B -m unittest test_search_api -v — 3/3 passed; git diff --check чистый. Version/changelog согласованы с изменением. Blocking findings отсутствуют.

APPROVE — независимо проверен полный diff PR #34 (`9fb266e`) относительно `main` (`f045233`) и релевантный flow. Backend удаляет OpenSearch `<em>`/`</em>` из snippet до передачи клиенту; UI выводит snippet через `textContent`, HTML-инъекция не появляется. Тест проверяет очищенный контракт API; локально `python3 -B -m unittest test_search_api -v` — 3/3 passed; `git diff --check` чистый. Version/changelog согласованы с изменением. Blocking findings отсутствуют.
admin merged commit 94ba04ff56 into main 2026-09-16 06:22:35 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin agent
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: agent/akyldash#34
Delete Branch "fix/plain-search-snippets"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?