forked from zovos/vk_hackathon
Compare commits
29 commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
19af82ba11 | ||
|
|
dc3a5b9dfd | ||
|
|
6f122eabd0 | ||
|
|
3561a064d5 | ||
|
|
f9f1b332bd | ||
|
|
7247248b27 | ||
|
|
a4475abfb2 | ||
|
|
91aae61b65 | ||
|
|
24505baa1c | ||
|
|
deb422d03e | ||
|
|
d2e531cc07 | ||
|
|
965fc906f3 | ||
|
|
9750868ea8 | ||
|
|
0fbf69e359 | ||
|
|
f4b14edabf | ||
|
|
d5ed8c7764 | ||
|
|
bd4c7c24a3 | ||
|
|
143efd6531 | ||
|
|
1e276512fb | ||
|
|
e244b8bf30 | ||
|
|
d28964aa7e | ||
|
|
b15c59a579 | ||
|
|
3423200625 | ||
|
|
e95ca82a4a | ||
|
|
9db7d3544c | ||
|
|
6c75cc8376 | ||
|
|
aad36f0ac5 | ||
|
|
e49e3417eb | ||
|
|
46a40fc65e |
37 changed files with 1938 additions and 1105 deletions
|
|
@ -1,27 +0,0 @@
|
||||||
# AI Update Log
|
|
||||||
|
|
||||||
## Scope
|
|
||||||
- File: `search/main.py`
|
|
||||||
- Purpose: fixed and extended retrieval/rerank pipeline according to TODO items.
|
|
||||||
|
|
||||||
## Done Changes
|
|
||||||
- Switched base query selection to `question.search_text` with fallback to `question.text`.
|
|
||||||
- Added support for `question.variants` as additional query variants in retrieval.
|
|
||||||
- Added support for `question.hyde` as additional dense-only queries.
|
|
||||||
- Added support for `question.keywords` as the primary source for sparse query text with fallback to current query variant.
|
|
||||||
- Stopped losing retrieval candidates after rerank: rerank is applied to head (`RERANK_LIMIT`), tail candidates are preserved.
|
|
||||||
- Added deduplication of retrieval points by Qdrant point id before rerank.
|
|
||||||
- Implemented score aggregation by `message_id` (sum of chunk scores mapped to same message).
|
|
||||||
- Limited final response to `top-50` message ids via `FINAL_TOP_K = 50`.
|
|
||||||
- Final output message ids are now selected from aggregated scores (sorted by score desc, tie-break by message_id).
|
|
||||||
|
|
||||||
## Notes
|
|
||||||
- Earlier step introduced direct `message_ids` deduplication before response.
|
|
||||||
- Current logic supersedes this by ranking and selecting unique `message_id` values from aggregated scores.
|
|
||||||
|
|
||||||
## Verification
|
|
||||||
- Syntax check passed after each main change: `python -m py_compile search/main.py`.
|
|
||||||
|
|
||||||
## How To Use This Log
|
|
||||||
- Treat this file as the source of truth for already completed `search/main.py` tasks.
|
|
||||||
- On next tasks, read this file first to avoid duplicate edits.
|
|
||||||
13
.env.example
Normal file
13
.env.example
Normal file
|
|
@ -0,0 +1,13 @@
|
||||||
|
# Local docker compose configuration
|
||||||
|
QDRANT_URL=http://qdrant:6333
|
||||||
|
QDRANT_COLLECTION_NAME=evaluation
|
||||||
|
QDRANT_DENSE_VECTOR_NAME=dense
|
||||||
|
QDRANT_SPARSE_VECTOR_NAME=sparse
|
||||||
|
|
||||||
|
EMBEDDINGS_DENSE_URL=http://83.166.249.64:18001/embeddings
|
||||||
|
RERANKER_URL=http://83.166.249.64:18001/score
|
||||||
|
|
||||||
|
# Fill either API_KEY or both OPEN_API_LOGIN and OPEN_API_PASSWORD.
|
||||||
|
API_KEY=
|
||||||
|
OPEN_API_LOGIN=
|
||||||
|
OPEN_API_PASSWORD=
|
||||||
4
.gitignore
vendored
4
.gitignore
vendored
|
|
@ -149,6 +149,10 @@ activemq-data/
|
||||||
|
|
||||||
# Environments
|
# Environments
|
||||||
.env
|
.env
|
||||||
|
.env.local
|
||||||
|
.env.*.local
|
||||||
|
!.env.example
|
||||||
|
.ai_update/
|
||||||
.envrc
|
.envrc
|
||||||
.venv
|
.venv
|
||||||
env/
|
env/
|
||||||
|
|
|
||||||
44
AGENTS.md
Normal file
44
AGENTS.md
Normal file
|
|
@ -0,0 +1,44 @@
|
||||||
|
# Repository Instructions
|
||||||
|
|
||||||
|
This repository uses a shared Codex collaboration workflow.
|
||||||
|
|
||||||
|
## Mandatory behavior for every Codex session
|
||||||
|
|
||||||
|
- Before doing any work, restore context from:
|
||||||
|
1. `README.md` and project docs if present
|
||||||
|
2. `.ai_update/current_status.md`
|
||||||
|
3. `.ai_update/handoff.md`
|
||||||
|
4. latest files in `.ai_update/sessions/`
|
||||||
|
5. `.ai_update/changelog.md`
|
||||||
|
6. git state (`git status`, recent `git log`)
|
||||||
|
|
||||||
|
- For any coding task in this repo, use the skill:
|
||||||
|
`team-sync-hackathon`
|
||||||
|
|
||||||
|
- After any meaningful change, update `.ai_update/` so another human or Codex session can continue with zero guesswork.
|
||||||
|
|
||||||
|
- Prefer continuing existing architecture and conventions over rewriting working code.
|
||||||
|
|
||||||
|
- Do not delete or reset teammates' work unless explicitly requested.
|
||||||
|
|
||||||
|
## Shared memory rules
|
||||||
|
|
||||||
|
Codex must treat `.ai_update/` as the canonical shared handoff area between:
|
||||||
|
- humans
|
||||||
|
- current Codex session
|
||||||
|
- future Codex sessions
|
||||||
|
|
||||||
|
If `.ai_update/` is missing or incomplete, create/fill it before large changes.
|
||||||
|
|
||||||
|
## Local startup rule
|
||||||
|
|
||||||
|
If local validation is needed and the project is not running:
|
||||||
|
- first try `./bin/codex-start`
|
||||||
|
- otherwise follow the startup instructions from the skill
|
||||||
|
|
||||||
|
## Safety rules for repo work
|
||||||
|
|
||||||
|
- Never change public API contracts unless explicitly requested.
|
||||||
|
- Never run destructive cleanup commands without explicit instruction.
|
||||||
|
- Never force-push without explicit instruction.
|
||||||
|
- Prefer small verifiable steps.
|
||||||
|
|
@ -374,14 +374,13 @@ score = recall_avg * 0.8 + ndcg_avg * 0.2
|
||||||
|
|
||||||
Для локального запуска используйте `docker compose`.
|
Для локального запуска используйте `docker compose`.
|
||||||
|
|
||||||
Перед запуском укажите учетные данные для внешнего dense/rerank API:
|
Сначала подготовьте локальный env:
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
export OPEN_API_LOGIN=...
|
cp .env.example .env
|
||||||
export OPEN_API_PASSWORD=...
|
|
||||||
```
|
```
|
||||||
|
|
||||||
Если эти переменные не заданы, `docker compose up` завершится с ошибкой.
|
После этого заполните в `.env` либо `API_KEY`, либо пару `OPEN_API_LOGIN` / `OPEN_API_PASSWORD`.
|
||||||
|
|
||||||
Запуск:
|
Запуск:
|
||||||
|
|
||||||
|
|
|
||||||
290
doc/ai_update.md
290
doc/ai_update.md
|
|
@ -1,290 +0,0 @@
|
||||||
# AI Update по ТЗ
|
|
||||||
|
|
||||||
Дата: 2026-04-18
|
|
||||||
|
|
||||||
## Что просмотрено
|
|
||||||
|
|
||||||
- `doc/ТЗ_на_хакатон_Индексация_и_поиск_по_сообщениям.pdf`
|
|
||||||
- `README.md`
|
|
||||||
- `docker-compose.yml`
|
|
||||||
- `index/main.py`, `search/main.py`
|
|
||||||
- `index/Dockerfile`, `search/Dockerfile`
|
|
||||||
- `index/Makefile`, `search/Makefile`
|
|
||||||
- `data/Go Nova.json`
|
|
||||||
|
|
||||||
По примеру данных:
|
|
||||||
|
|
||||||
- всего сообщений: `25`
|
|
||||||
- с `parts`: `14`
|
|
||||||
- с цитатами: `5`
|
|
||||||
- с пересланными сообщениями: `2`
|
|
||||||
- с `mentions`: `4`
|
|
||||||
- с `file_snippets`: `1`
|
|
||||||
- системных сообщений: `1`
|
|
||||||
|
|
||||||
Это важно, потому что в текущем коде часть этих сигналов либо не используется вообще, либо теряет смысл при индексации.
|
|
||||||
|
|
||||||
## Что уже соответствует ТЗ
|
|
||||||
|
|
||||||
1. В репозитории есть оба требуемых сервиса: `index` и `search`.
|
|
||||||
2. Обязательные endpoints реализованы:
|
|
||||||
- `GET /health`, `POST /index`, `POST /sparse_embedding` в `index/main.py`
|
|
||||||
- `GET /health`, `POST /search` в `search/main.py`
|
|
||||||
3. Контракты request/response по основным endpoint'ам не менялись и в целом совпадают с шаблоном и ТЗ.
|
|
||||||
4. `search` использует `Qdrant`, dense endpoint и reranker через HTTP.
|
|
||||||
5. В обоих Dockerfile sparse-модель предзагружается внутрь образа, что соответствует оффлайн-ограничению контейнеров.
|
|
||||||
6. `HOST` и `PORT` читаются из env, как требует ТЗ.
|
|
||||||
|
|
||||||
## Что отсутствует или реализовано частично
|
|
||||||
|
|
||||||
### 1. Обогащенный вопрос из ТЗ почти не используется
|
|
||||||
|
|
||||||
В `search/main.py:90-100` описаны поля:
|
|
||||||
|
|
||||||
- `search_text`
|
|
||||||
- `variants`
|
|
||||||
- `hyde`
|
|
||||||
- `keywords`
|
|
||||||
- `entities`
|
|
||||||
- `date_mentions`
|
|
||||||
- `date_range`
|
|
||||||
- `asker`
|
|
||||||
|
|
||||||
Но в реальном поиске используется только `question.text`:
|
|
||||||
|
|
||||||
- `search/main.py:307-316`
|
|
||||||
|
|
||||||
Это главный недобор относительно ТЗ. Само ТЗ явно дает эти поля как сигналы для retrieval, а код их сейчас просто игнорирует.
|
|
||||||
|
|
||||||
### 2. Метаданные чанков объявлены, но не участвуют в поиске
|
|
||||||
|
|
||||||
В `README.md:52-58` отдельно сказано, что в metadata чанка сохраняются:
|
|
||||||
|
|
||||||
- `participants`
|
|
||||||
- `mentions`
|
|
||||||
- `contains_forward`
|
|
||||||
- `contains_quote`
|
|
||||||
|
|
||||||
В `search/main.py:132-145` есть модель `ChunkMetadata`, но дальше она никак не используется в `query_points`. Поиск не делает:
|
|
||||||
|
|
||||||
- фильтрацию по `mentions`
|
|
||||||
- фильтрацию по `participants`
|
|
||||||
- учет `thread_sn`
|
|
||||||
- учет временного диапазона через `start`/`end`
|
|
||||||
- отдельную обработку quote/forward чанков
|
|
||||||
|
|
||||||
То есть сильный канал улучшения качества уже предусмотрен схемой, но сейчас не задействован.
|
|
||||||
|
|
||||||
### 3. Реранк отбрасывает часть кандидатов
|
|
||||||
|
|
||||||
Сейчас:
|
|
||||||
|
|
||||||
- retrieval берет до `20` чанков: `search/main.py:174-177`
|
|
||||||
- rerank берет только первые `10`: `search/main.py:278-296`
|
|
||||||
- после rerank возвращаются только эти `10`, а хвост `11-20` теряется: `search/main.py:321-328`
|
|
||||||
|
|
||||||
Это не нарушение контракта, но это реальная потеря recall.
|
|
||||||
|
|
||||||
### 4. Выдача не дедуплицируется и не ограничивается по полезному top-K
|
|
||||||
|
|
||||||
Сейчас `message_ids` просто конкатенируются:
|
|
||||||
|
|
||||||
- `search/main.py:323-328`
|
|
||||||
|
|
||||||
Проблемы:
|
|
||||||
|
|
||||||
- дубликаты message id не удаляются
|
|
||||||
- результаты не агрегируются по лучшему score сообщения
|
|
||||||
- нет явного ограничения на топ полезных `50`, хотя именно `K=50` участвует в метрике из ТЗ
|
|
||||||
|
|
||||||
Если один и тот же `message_id` попал в несколько чанков, он тратит место в выдаче.
|
|
||||||
|
|
||||||
### 5. Индексация пока очень базовая: фиксированные символьные чанки
|
|
||||||
|
|
||||||
В `index/main.py:118-190` чанки строятся просто по длине строки:
|
|
||||||
|
|
||||||
- `CHUNK_SIZE = 512`
|
|
||||||
- `OVERLAP_SIZE = 256`
|
|
||||||
- разбиение идет по символам, а не по сообщениям, тайм-гепам, тредам или смысловым блокам
|
|
||||||
|
|
||||||
Из-за этого:
|
|
||||||
|
|
||||||
- длинные пересланные сообщения и цитаты могут резаться в неудобных местах
|
|
||||||
- один и тот же смысловой блок может быть разнесен по чанкам неестественно
|
|
||||||
- overlap строится по хвосту текста, а не по границе сообщений
|
|
||||||
|
|
||||||
### 6. `page_content`, `dense_content` и `sparse_content` сейчас одинаковые
|
|
||||||
|
|
||||||
См. `index/main.py:180-186`.
|
|
||||||
|
|
||||||
ТЗ прямо оставляет это место как точку оптимизации качества, но пока этот резерв не используется.
|
|
||||||
|
|
||||||
### 7. Индексация берет только `text` и `parts[*].text`, остальное почти теряется
|
|
||||||
|
|
||||||
См. `index/main.py:99-115`.
|
|
||||||
|
|
||||||
Сейчас не используются как поисковые сигналы:
|
|
||||||
|
|
||||||
- `sender_id`
|
|
||||||
- `mentions`
|
|
||||||
- `file_snippets`
|
|
||||||
- `member_event`
|
|
||||||
- `thread_sn`
|
|
||||||
- `is_hidden`
|
|
||||||
- `is_system`
|
|
||||||
- явное различение `quote` и `forward`
|
|
||||||
|
|
||||||
Особенно важные пробелы:
|
|
||||||
|
|
||||||
- `member_event` у системных сообщений сейчас фактически пропадает, если обычного текста нет
|
|
||||||
- `file_snippets` не разбирается, хотя там могут быть имена файлов, URL и служебные поля
|
|
||||||
- запросы вида "кто писал", "кого упоминали", "какой файл/документ кидали" сейчас поддержаны слабо
|
|
||||||
|
|
||||||
### 8. Смысл `quote` и `forward` не маркируется
|
|
||||||
|
|
||||||
В `index/main.py:105-113` текст из `parts` просто подшивается в общий текст без явных маркеров вида:
|
|
||||||
|
|
||||||
- "цитата:"
|
|
||||||
- "пересланное сообщение:"
|
|
||||||
- "автор цитаты:"
|
|
||||||
|
|
||||||
В итоге dense/sparse видят просто общий текстовый комок. Для поиска по обсуждениям это ощутимая потеря контекста.
|
|
||||||
|
|
||||||
### 9. Есть расхождение между локальной инфраструктурой и ТЗ
|
|
||||||
|
|
||||||
По ТЗ для `search` ожидается `API_KEY`.
|
|
||||||
|
|
||||||
В коде это поддержано:
|
|
||||||
|
|
||||||
- `search/main.py:21-30`
|
|
||||||
- `search/main.py:42-47`
|
|
||||||
- `search/Makefile:10-15`
|
|
||||||
|
|
||||||
Но локальный `docker-compose.yml:57-60` требует `OPEN_API_LOGIN` и `OPEN_API_PASSWORD`.
|
|
||||||
|
|
||||||
Итог:
|
|
||||||
|
|
||||||
- сам сервис гибче ТЗ
|
|
||||||
- локальный compose не повторяет боевую схему из ТЗ один в один
|
|
||||||
|
|
||||||
Это не ломает контракт, но может запутать при локальной отладке.
|
|
||||||
|
|
||||||
### 10. Есть еще одна инфраструктурная несостыковка со сдачей
|
|
||||||
|
|
||||||
В `doc/upload_to_docker.md:45-48` явно сказано собирать образы с `--platform linux/amd64`.
|
|
||||||
|
|
||||||
Но `index/Makefile:16-18` и `search/Makefile:26-28` собирают без `--platform linux/amd64`.
|
|
||||||
|
|
||||||
На x86 это может пройти незаметно, а на ARM-машине дать неправильный образ для отправки.
|
|
||||||
|
|
||||||
### 11. Есть неоднозначность между README и PDF по sparse в `search`
|
|
||||||
|
|
||||||
- `README.md:211` говорит, что sparse-модель для `search` должна быть локально внутри образа
|
|
||||||
- PDF в формулировке требований к `Search Service` делает акцент, что обращения к dense/sparse/rerank идут через проверяющую систему
|
|
||||||
|
|
||||||
Текущий код следует логике README/example: sparse считается локально в `search/main.py:148-151` и `search/main.py:198-207`.
|
|
||||||
|
|
||||||
Я бы это не считал блокером, но как минимум это место стоит держать в голове как неоднозначное требование.
|
|
||||||
|
|
||||||
## Что улучшать в первую очередь
|
|
||||||
|
|
||||||
### Приоритет 1. Начать использовать все поля `question`
|
|
||||||
|
|
||||||
Минимально стоит задействовать:
|
|
||||||
|
|
||||||
- `search_text` как основной нормализованный запрос
|
|
||||||
- `variants` как дополнительные формулировки
|
|
||||||
- `hyde` как дополнительные dense-запросы
|
|
||||||
- `keywords` как основу для sparse
|
|
||||||
- `entities` для фильтров и lexical boost
|
|
||||||
- `date_range` и `date_mentions` для ограничения по времени
|
|
||||||
- `asker` как сигнал по людям и email
|
|
||||||
|
|
||||||
Самый логичный путь без смены стэка: несколько dense/sparse запросов + fusion в `Qdrant`.
|
|
||||||
|
|
||||||
### Приоритет 2. Перестроить chunking под структуру чата, а не под символы
|
|
||||||
|
|
||||||
Нужны чанки по:
|
|
||||||
|
|
||||||
- окнам сообщений
|
|
||||||
- временным разрывам
|
|
||||||
- границам thread/forward/quote
|
|
||||||
- ограничению на размер по сообщениям, а не только по символам
|
|
||||||
|
|
||||||
Для чатов это обычно дает больше пользы, чем любые косметические тюнинги rerank.
|
|
||||||
|
|
||||||
### Приоритет 3. Развести `page_content`, `dense_content`, `sparse_content`
|
|
||||||
|
|
||||||
Хорошая схема:
|
|
||||||
|
|
||||||
- `page_content`: читабельный исходный текст чанка
|
|
||||||
- `dense_content`: нормализованный текст с ролями, автором, маркерами quote/forward
|
|
||||||
- `sparse_content`: keyword-heavy версия с email, mentions, именами файлов, ссылками, документами, леммами
|
|
||||||
|
|
||||||
Сейчас эта возможность не используется вообще.
|
|
||||||
|
|
||||||
### Приоритет 4. Нормально собирать финальную выдачу
|
|
||||||
|
|
||||||
Нужно:
|
|
||||||
|
|
||||||
- не терять кандидатов после rerank
|
|
||||||
- удалять дубликаты `message_id`
|
|
||||||
- агрегировать по лучшему score сообщения или чанка
|
|
||||||
- отдавать осмысленный top-50
|
|
||||||
|
|
||||||
Это прямой выигрыш по Recall@50 и nDCG@50.
|
|
||||||
|
|
||||||
### Приоритет 5. Начать использовать metadata в `Qdrant`
|
|
||||||
|
|
||||||
Особенно полезно для:
|
|
||||||
|
|
||||||
- `mentions`
|
|
||||||
- `participants`
|
|
||||||
- `contains_quote`
|
|
||||||
- `contains_forward`
|
|
||||||
- `thread_sn`
|
|
||||||
- `start` / `end`
|
|
||||||
|
|
||||||
Для многих вопросов это позволит не просто "лучше ранжировать", а сразу отрезать нерелевантный шум.
|
|
||||||
|
|
||||||
### Приоритет 6. Превратить скрытые сигналы в индексируемый текст
|
|
||||||
|
|
||||||
Стоит отдельно материализовать:
|
|
||||||
|
|
||||||
- `member_event` в текст вида "пользователь X добавил Y"
|
|
||||||
- `file_snippets` в текст вида "файл: NAME, url: ..."
|
|
||||||
- автора сообщения
|
|
||||||
- список упомянутых пользователей
|
|
||||||
|
|
||||||
Сейчас эти сигналы либо не попадают в индекс, либо попадают слишком слабо.
|
|
||||||
|
|
||||||
## Что можно добавить по Python-библиотекам, не меняя стек
|
|
||||||
|
|
||||||
Стек `Qdrant` менять не нужно. Самые полезные добавки я бы смотрел такие:
|
|
||||||
|
|
||||||
- `pymorphy3` для лемматизации русских слов при подготовке `sparse_content`
|
|
||||||
- `razdel` для аккуратной токенизации русского текста
|
|
||||||
- `rapidfuzz` для точного lexical match по именам, email, документам, ссылкам и названиям
|
|
||||||
- `python-dateutil` или `dateparser` для нормализации дат, если захотите усиливать работу с `date_mentions`
|
|
||||||
- `tenacity` для аккуратных retry/timeout-оберток вокруг dense/rerank HTTP вызовов
|
|
||||||
|
|
||||||
Что не нужно делать:
|
|
||||||
|
|
||||||
- менять `Qdrant`
|
|
||||||
- тащить внешние LLM/API
|
|
||||||
- усложнять архитектуру ради "модности", пока не использованы базовые сигналы из самого ТЗ
|
|
||||||
|
|
||||||
## Короткий вывод
|
|
||||||
|
|
||||||
Сейчас репозиторий соответствует ТЗ как рабочий базовый шаблон, но почти не использует те сигналы, ради которых это ТЗ вообще интересно:
|
|
||||||
|
|
||||||
- обогащение вопроса
|
|
||||||
- metadata чанков
|
|
||||||
- структуру chat messages
|
|
||||||
- сигналы автора, упоминаний, файлов, системных событий, цитат и пересылок
|
|
||||||
|
|
||||||
Самый большой потенциал улучшения здесь не в замене базы или модели, а в трех вещах:
|
|
||||||
|
|
||||||
1. умный chunking
|
|
||||||
2. multi-query hybrid retrieval
|
|
||||||
3. использование metadata и нормальной сборки финального top-50
|
|
||||||
49
doc/todo.md
49
doc/todo.md
|
|
@ -1,49 +0,0 @@
|
||||||
# TODO
|
|
||||||
|
|
||||||
## `search/main.py`
|
|
||||||
|
|
||||||
- [ ] P0: Переключить основной query на `question.search_text` с fallback на `question.text`
|
|
||||||
- [ ] P0: Подключить `question.variants` как дополнительные query-варианты
|
|
||||||
- [ ] P0: Подключить `question.hyde` как дополнительные dense-запросы
|
|
||||||
- [ ] P0: Подключить `question.keywords` как основу для sparse-запросов
|
|
||||||
- [ ] P0: Перестать терять retrieval-кандидатов после rerank
|
|
||||||
- [ ] P0: Дедуплицировать `message_ids` перед ответом
|
|
||||||
- [ ] P0: Ограничить финальную выдачу до `top-50`
|
|
||||||
- [ ] P0: Агрегировать score по `message_id`
|
|
||||||
- [ ] P2: Использовать `entities.people` и `entities.emails` для boost или фильтрации
|
|
||||||
- [ ] P2: Использовать `entities.documents`, `entities.names`, `entities.links` для lexical boost
|
|
||||||
- [ ] P2: Использовать `date_range` для фильтрации по `metadata.start` и `metadata.end`
|
|
||||||
- [ ] P2: Использовать `contains_quote` и `contains_forward` как сигналы ранжирования
|
|
||||||
- [ ] P2: Добавить multi-query fusion в `Qdrant`
|
|
||||||
- [ ] P2: Подобрать `prefetch`, `retrieve_k`, `rerank_limit`
|
|
||||||
- [ ] P3: Добавить retry и timeout политику для dense/rerank HTTP вызовов
|
|
||||||
|
|
||||||
## `index/main.py`
|
|
||||||
|
|
||||||
- [ ] P1: Перейти с символьного chunking на chunking по сообщениям
|
|
||||||
- [ ] P1: Учитывать time gap при сборке чанков
|
|
||||||
- [ ] P1: Маркировать в тексте `quote`, `forward`, автора сообщения и автора цитаты
|
|
||||||
- [ ] P1: Развести `page_content`, `dense_content`, `sparse_content`
|
|
||||||
- [ ] P1: Материализовать `sender_id` и `mentions` в индексируемый текст
|
|
||||||
- [ ] P1: Разбирать `file_snippets` и вытаскивать имя файла, mime и url
|
|
||||||
- [ ] P1: Разбирать `member_event` и превращать его в индексируемый текст
|
|
||||||
|
|
||||||
## `docker-compose.yml`
|
|
||||||
|
|
||||||
- [ ] P3: Привести локальный `docker-compose.yml` к схеме с `API_KEY`
|
|
||||||
- [ ] P3: Добавить `--platform linux/amd64` в сборку образов
|
|
||||||
|
|
||||||
## `doc/` (новый файл с регрессионными вопросами)
|
|
||||||
|
|
||||||
- [ ] P3: Зафиксировать набор локальных тестовых вопросов для проверки регрессий
|
|
||||||
|
|
||||||
## `search/requirements.txt`
|
|
||||||
|
|
||||||
- [ ] P4: Добавить `python-dateutil` или `dateparser`
|
|
||||||
- [ ] P4: Добавить `tenacity`
|
|
||||||
|
|
||||||
## `index/requirements.txt` и/или `search/requirements.txt`
|
|
||||||
|
|
||||||
- [ ] P4: Добавить `razdel`
|
|
||||||
- [ ] P4: Добавить `pymorphy3`
|
|
||||||
- [ ] P4: Добавить `rapidfuzz`
|
|
||||||
|
|
@ -1,223 +0,0 @@
|
||||||
# To-Do и целевой pipeline
|
|
||||||
|
|
||||||
Дата: 2026-04-18
|
|
||||||
|
|
||||||
## Цель
|
|
||||||
|
|
||||||
Поднять `Recall@50` и `nDCG@50` без смены стэка:
|
|
||||||
|
|
||||||
- оставить `Qdrant`
|
|
||||||
- оставить внешний dense endpoint
|
|
||||||
- оставить внешний reranker
|
|
||||||
- усиливать только индекс, retrieval, rerank и post-processing
|
|
||||||
|
|
||||||
## Приоритетный to-do
|
|
||||||
|
|
||||||
### P0. Быстрые и самые окупаемые правки
|
|
||||||
|
|
||||||
- [ ] Переключить основной запрос в `search` на `question.search_text` с fallback на `question.text`
|
|
||||||
- [ ] Подключить `question.variants` как дополнительные query-формулировки
|
|
||||||
- [ ] Подключить `question.hyde` как дополнительные dense-запросы
|
|
||||||
- [ ] Подключить `question.keywords` как основу для sparse-запроса
|
|
||||||
- [ ] Перестать терять кандидатов после rerank: возвращать не только top-10 rerank, но и хвост retrieval
|
|
||||||
- [ ] Дедуплицировать `message_ids` перед ответом
|
|
||||||
- [ ] Ограничить финальную выдачу осмысленным `top-50`
|
|
||||||
- [ ] Агрегировать score по `message_id`, а не просто конкатенировать ids из чанков
|
|
||||||
|
|
||||||
### P1. Улучшение индексации
|
|
||||||
|
|
||||||
- [ ] Перейти с символьного chunking на chunking по сообщениям
|
|
||||||
- [ ] Учитывать временные разрывы между сообщениями при сборке чанка
|
|
||||||
- [ ] Не смешивать в одном чанке слишком далекие по смыслу блоки
|
|
||||||
- [ ] Отдельно маркировать `quote`, `forward`, автора сообщения и автора цитаты
|
|
||||||
- [ ] Развести `page_content`, `dense_content`, `sparse_content`
|
|
||||||
- [ ] Материализовать `mentions` в текст и metadata
|
|
||||||
- [ ] Материализовать `sender_id` в индексируемый текст
|
|
||||||
- [ ] Разбирать `file_snippets` и вытаскивать имя файла, mime, url
|
|
||||||
- [ ] Разбирать `member_event` и превращать его в индексируемый текст
|
|
||||||
|
|
||||||
### P2. Улучшение retrieval и фильтрации
|
|
||||||
|
|
||||||
- [ ] Использовать `entities.people` и `entities.emails` для boost или фильтрации по `participants` и `mentions`
|
|
||||||
- [ ] Использовать `entities.documents`, `entities.names`, `entities.links` для lexical boost
|
|
||||||
- [ ] Использовать `date_range` для фильтрации по `metadata.start` и `metadata.end`
|
|
||||||
- [ ] Использовать `contains_quote` и `contains_forward` как дополнительные сигналы ранжирования
|
|
||||||
- [ ] Добавить multi-query fusion в `Qdrant` для dense и sparse запросов
|
|
||||||
- [ ] Подобрать новые значения `prefetch`, `retrieve_k`, `rerank_limit`
|
|
||||||
|
|
||||||
### P3. Инфраструктура и надежность
|
|
||||||
|
|
||||||
- [ ] Привести локальный `docker-compose.yml` к схеме с `API_KEY`, чтобы локальный запуск был ближе к ТЗ
|
|
||||||
- [ ] Добавить `--platform linux/amd64` в сборку образов
|
|
||||||
- [ ] Добавить retry и timeout политику для dense/rerank HTTP вызовов
|
|
||||||
- [ ] Зафиксировать набор локальных тестовых запросов для регрессии качества
|
|
||||||
|
|
||||||
### P4. Библиотеки, которые можно добавить без смены стэка
|
|
||||||
|
|
||||||
- [ ] `razdel` для токенизации русского текста
|
|
||||||
- [ ] `pymorphy3` для лемматизации при подготовке `sparse_content`
|
|
||||||
- [ ] `rapidfuzz` для точного match по именам, email, документам и ссылкам
|
|
||||||
- [ ] `python-dateutil` или `dateparser` для нормализации дат
|
|
||||||
- [ ] `tenacity` для retry вокруг внешних HTTP запросов
|
|
||||||
|
|
||||||
## Целевой pipeline индексации
|
|
||||||
|
|
||||||
### 1. Подготовка сообщения
|
|
||||||
|
|
||||||
На входе каждое сообщение должно раскладываться на сигналы:
|
|
||||||
|
|
||||||
- основной текст сообщения
|
|
||||||
- `parts[*].text`
|
|
||||||
- тип части: `text`, `quote`, `forward`
|
|
||||||
- `sender_id`
|
|
||||||
- `mentions`
|
|
||||||
- `file_snippets`
|
|
||||||
- `member_event`
|
|
||||||
- `thread_sn`
|
|
||||||
- флаги `is_system`, `is_quote`, `is_forward`
|
|
||||||
|
|
||||||
### 2. Нормализация и разметка
|
|
||||||
|
|
||||||
Перед chunking сообщение стоит приводить к структурированному виду, например:
|
|
||||||
|
|
||||||
- `author: ...`
|
|
||||||
- `mentions: ...`
|
|
||||||
- `quote: ...`
|
|
||||||
- `forwarded: ...`
|
|
||||||
- `file: ...`
|
|
||||||
- `system_event: ...`
|
|
||||||
|
|
||||||
Смысл не в красивом выводе, а в том, чтобы dense и sparse видели роль каждого куска текста.
|
|
||||||
|
|
||||||
### 3. Chunking
|
|
||||||
|
|
||||||
Целевой принцип:
|
|
||||||
|
|
||||||
- базовая единица не символ, а сообщение
|
|
||||||
- чанк собирается как окно из нескольких соседних сообщений
|
|
||||||
- окно режется по лимиту размера
|
|
||||||
- окно закрывается на большом time gap
|
|
||||||
- `forward` и длинные `quote` не должны ломать соседний контекст
|
|
||||||
- overlap должен работать по границам сообщений, а не по хвосту строки
|
|
||||||
|
|
||||||
### 4. Формирование трех видов текста
|
|
||||||
|
|
||||||
`page_content`:
|
|
||||||
|
|
||||||
- человекочитаемый текст чанка для payload
|
|
||||||
|
|
||||||
`dense_content`:
|
|
||||||
|
|
||||||
- нормализованный текст с автором, role-маркерами, quote/forward маркерами
|
|
||||||
|
|
||||||
`sparse_content`:
|
|
||||||
|
|
||||||
- keyword-heavy текст
|
|
||||||
- леммы
|
|
||||||
- email
|
|
||||||
- mentions
|
|
||||||
- имена файлов
|
|
||||||
- ссылки
|
|
||||||
- названия документов и сервисов
|
|
||||||
|
|
||||||
### 5. Metadata для Qdrant
|
|
||||||
|
|
||||||
В metadata стоит стабильно сохранять:
|
|
||||||
|
|
||||||
- `message_ids`
|
|
||||||
- `participants`
|
|
||||||
- `mentions`
|
|
||||||
- `thread_sn`
|
|
||||||
- `start`
|
|
||||||
- `end`
|
|
||||||
- `contains_quote`
|
|
||||||
- `contains_forward`
|
|
||||||
- `chat_id`
|
|
||||||
- `chat_type`
|
|
||||||
|
|
||||||
## Целевой pipeline поиска
|
|
||||||
|
|
||||||
### 1. Подготовка query
|
|
||||||
|
|
||||||
Собирать query не из одного поля, а из набора:
|
|
||||||
|
|
||||||
- основной запрос: `search_text` или `text`
|
|
||||||
- дополнительные dense-query: `variants` и `hyde`
|
|
||||||
- дополнительные sparse-query: `keywords`
|
|
||||||
- entity-сигналы: `people`, `emails`, `documents`, `names`, `links`
|
|
||||||
- time constraints: `date_range`, `date_mentions`
|
|
||||||
|
|
||||||
### 2. Query builder
|
|
||||||
|
|
||||||
Нужно строить несколько представлений запроса:
|
|
||||||
|
|
||||||
- dense-query для смысла
|
|
||||||
- sparse-query для точных слов и терминов
|
|
||||||
- filter/boost по metadata
|
|
||||||
|
|
||||||
### 3. Retrieval в Qdrant
|
|
||||||
|
|
||||||
Практическая схема:
|
|
||||||
|
|
||||||
1. Выполнить несколько dense prefetch по разным вариантам запроса
|
|
||||||
2. Выполнить несколько sparse prefetch по keyword-heavy запросам
|
|
||||||
3. Добавить filters по датам, mentions, participants, если это явно следует из вопроса
|
|
||||||
4. Объединить результаты через fusion
|
|
||||||
5. Забрать расширенный пул кандидатов для rerank
|
|
||||||
|
|
||||||
### 4. Rerank
|
|
||||||
|
|
||||||
Rerank должен работать не на слишком маленьком пуле. Целевой принцип:
|
|
||||||
|
|
||||||
- retrieval дает расширенный пул
|
|
||||||
- rerank сортирует top-N кандидатов
|
|
||||||
- хвост retrieval не теряется полностью
|
|
||||||
|
|
||||||
### 5. Агрегация к `message_id`
|
|
||||||
|
|
||||||
После rerank:
|
|
||||||
|
|
||||||
- собрать `message_ids` из чанков
|
|
||||||
- удалить дубликаты
|
|
||||||
- агрегировать лучший score на сообщение
|
|
||||||
- собрать финальный `top-50`
|
|
||||||
|
|
||||||
Это особенно важно, потому что метрики в ТЗ считаются именно по `message_id`, а не по chunk id.
|
|
||||||
|
|
||||||
## Порядок внедрения
|
|
||||||
|
|
||||||
### Этап 1. Quick wins
|
|
||||||
|
|
||||||
- использовать `search_text`, `variants`, `hyde`, `keywords`
|
|
||||||
- перестать терять кандидатов после rerank
|
|
||||||
- добавить dedup и top-50
|
|
||||||
|
|
||||||
### Этап 2. Пересборка индекса
|
|
||||||
|
|
||||||
- новый renderer сообщения
|
|
||||||
- новый chunking по сообщениям
|
|
||||||
- разные `page_content`, `dense_content`, `sparse_content`
|
|
||||||
|
|
||||||
### Этап 3. Metadata-aware retrieval
|
|
||||||
|
|
||||||
- filters по дате
|
|
||||||
- boost по mentions/participants
|
|
||||||
- учет `contains_quote` и `contains_forward`
|
|
||||||
|
|
||||||
### Этап 4. Тюнинг
|
|
||||||
|
|
||||||
- подобрать размеры чанков
|
|
||||||
- подобрать `retrieve_k`
|
|
||||||
- подобрать `rerank_limit`
|
|
||||||
- прогнать локальный набор контрольных вопросов
|
|
||||||
|
|
||||||
## Минимальный критерий готовности
|
|
||||||
|
|
||||||
Можно считать, что pipeline собран в рабочем виде, если:
|
|
||||||
|
|
||||||
- `search` использует не только `question.text`
|
|
||||||
- индексация не режет чанки посреди сообщения как основной механизм
|
|
||||||
- `message_ids` дедуплицируются
|
|
||||||
- финальная выдача ограничивается top-50
|
|
||||||
- retrieval умеет использовать хотя бы часть metadata
|
|
||||||
- локальная сборка и запуск не расходятся с ТЗ по критичным env и platform
|
|
||||||
|
|
@ -1,64 +0,0 @@
|
||||||
Для того чтобы раскидать задачи между тремя людьми, я распределю их по сложности и приоритету.
|
|
||||||
|
|
||||||
### Человек 1 (Основной фокус на поиске):
|
|
||||||
|
|
||||||
#### `search/main.py`
|
|
||||||
|
|
||||||
* **P0**: Переключить основной query на `question.search_text` с fallback на `question.text`
|
|
||||||
* **P0**: Подключить `question.variants` как дополнительные query-варианты
|
|
||||||
* **P0**: Подключить `question.hyde` как дополнительные dense-запросы
|
|
||||||
* **P0**: Подключить `question.keywords` как основу для sparse-запросов
|
|
||||||
* **P0**: Перестать терять retrieval-кандидатов после rerank
|
|
||||||
* **P0**: Дедуплицировать `message_ids` перед ответом
|
|
||||||
* **P0**: Ограничить финальную выдачу до `top-50`
|
|
||||||
* **P0**: Агрегировать score по `message_id`
|
|
||||||
* **P2**: Использовать `entities.people` и `entities.emails` для boost или фильтрации
|
|
||||||
* **P2**: Использовать `entities.documents`, `entities.names`, `entities.links` для lexical boost
|
|
||||||
* **P2**: Использовать `date_range` для фильтрации по `metadata.start` и `metadata.end`
|
|
||||||
* **P2**: Использовать `contains_quote` и `contains_forward` как сигналы ранжирования
|
|
||||||
* **P2**: Добавить multi-query fusion в `Qdrant`
|
|
||||||
* **P2**: Подобрать `prefetch`, `retrieve_k`, `rerank_limit`
|
|
||||||
* **P3**: Добавить retry и timeout политику для dense/rerank HTTP вызовов
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
### Человек 2 (Основной фокус на индексации и разметке):
|
|
||||||
|
|
||||||
#### `index/main.py`
|
|
||||||
|
|
||||||
* **P1**: Перейти с символьного chunking на chunking по сообщениям
|
|
||||||
* **P1**: Учитывать time gap при сборке чанков
|
|
||||||
* **P1**: Маркировать в тексте `quote`, `forward`, автора сообщения и автора цитаты
|
|
||||||
* **P1**: Развести `page_content`, `dense_content`, `sparse_content`
|
|
||||||
* **P1**: Материализовать `sender_id` и `mentions` в индексируемый текст
|
|
||||||
* **P1**: Разбирать `file_snippets` и вытаскивать имя файла, mime и url
|
|
||||||
* **P1**: Разбирать `member_event` и превращать его в индексируемый текст
|
|
||||||
|
|
||||||
#### `index/requirements.txt` и/или `search/requirements.txt`
|
|
||||||
|
|
||||||
* **P4**: Добавить `razdel`
|
|
||||||
* **P4**: Добавить `pymorphy3`
|
|
||||||
* **P4**: Добавить `rapidfuzz`
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
### Человек 3 (Основной фокус на Docker и зависимостях):
|
|
||||||
|
|
||||||
#### `docker-compose.yml`
|
|
||||||
|
|
||||||
* **P3**: Привести локальный `docker-compose.yml` к схеме с `API_KEY`
|
|
||||||
* **P3**: Добавить `--platform linux/amd64` в сборку образов
|
|
||||||
|
|
||||||
#### `doc/` (новый файл с регрессионными вопросами)
|
|
||||||
|
|
||||||
* **P3**: Зафиксировать набор локальных тестовых вопросов для проверки регрессий
|
|
||||||
|
|
||||||
#### `search/requirements.txt`
|
|
||||||
|
|
||||||
* **P4**: Добавить `python-dateutil` или `dateparser`
|
|
||||||
* **P4**: Добавить `tenacity`
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
Таким образом, задачи равномерно распределены по 3 участникам с учётом сложности и области фокуса.
|
|
||||||
|
|
||||||
|
|
@ -1,56 +0,0 @@
|
||||||
Шаг 2. Настройка Docker
|
|
||||||
|
|
||||||
Registry для хранения образов будет доступен по адресу 83.166.249.64:5000. Поскольку он работает без TLS, необходимо добавить его в список insecure registries в настройках Docker.
|
|
||||||
Docker Desktop (macOS / Windows)
|
|
||||||
|
|
||||||
Откройте Docker Desktop -> Settings -> Docker Engine.
|
|
||||||
Добавьте в JSON-конфиг поле insecure-registries:
|
|
||||||
|
|
||||||
{
|
|
||||||
"insecure-registries": ["83.166.249.64:5000"]
|
|
||||||
}
|
|
||||||
|
|
||||||
Нажмите Apply & Restart.
|
|
||||||
|
|
||||||
CLI — Linux
|
|
||||||
|
|
||||||
Откройте файл с конфигурацией докер демона в режиме редактирования
|
|
||||||
|
|
||||||
sudo nano /etc/docker/daemon.json
|
|
||||||
|
|
||||||
Добавьте в JSON-конфиг поле insecure-registries:
|
|
||||||
|
|
||||||
{
|
|
||||||
"insecure-registries": ["83.166.249.64:5000"]
|
|
||||||
}
|
|
||||||
|
|
||||||
Перезапустите docker
|
|
||||||
|
|
||||||
sudo systemctl restart docker
|
|
||||||
|
|
||||||
Шаг 3. Логин в registry
|
|
||||||
|
|
||||||
Необходимо пройти аутентификацию в docker registry, используя логин и пароль, полученные на шаге 1.
|
|
||||||
|
|
||||||
docker login 83.166.249.64:5000 -u <login> -p <password>
|
|
||||||
|
|
||||||
Ожидаемый вывод: Login Succeeded.
|
|
||||||
Шаг 4. Сборка образов
|
|
||||||
|
|
||||||
Соберите образы своих Index Service и Search Service. Образы должны иметь тег, который имеет строгий формат:
|
|
||||||
|
|
||||||
Для Index Service - 83.166.249.64:5000/35230/index-service:latest
|
|
||||||
Для Search Service - 83.166.249.64:5000/35230/search-service:latest
|
|
||||||
|
|
||||||
Образы должны быть собраны под платформу linux/amd64
|
|
||||||
|
|
||||||
docker build --platform linux/amd64 -t 83.166.249.64:5000/35230/index-service:latest {path_to_index_service_dir}
|
|
||||||
docker build --platform linux/amd64 -t 83.166.249.64:5000/35230/search-service:latest {path_to_search_service_dir}
|
|
||||||
|
|
||||||
Шаг 5. Публикация образов в registry
|
|
||||||
|
|
||||||
docker push 83.166.249.64:5000/35230/index-service:latest
|
|
||||||
docker push 83.166.249.64:5000/35230/search-service:latest
|
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
|
@ -2,37 +2,39 @@ services:
|
||||||
qdrant:
|
qdrant:
|
||||||
image: qdrant/qdrant:v1.14.1
|
image: qdrant/qdrant:v1.14.1
|
||||||
ports:
|
ports:
|
||||||
- "6333:6333"
|
- "6334:6333"
|
||||||
|
|
||||||
qdrant-init:
|
qdrant-init:
|
||||||
image: curlimages/curl:8.12.1
|
image: curlimages/curl:8.12.1
|
||||||
|
env_file:
|
||||||
|
- .env
|
||||||
depends_on:
|
depends_on:
|
||||||
- qdrant
|
- qdrant
|
||||||
command:
|
command:
|
||||||
- sh
|
- sh
|
||||||
- -c
|
- -c
|
||||||
- |
|
- |
|
||||||
until curl -sf http://qdrant:6333/collections; do
|
until curl -sf "$$QDRANT_URL/collections"; do
|
||||||
sleep 1
|
sleep 1
|
||||||
done
|
done
|
||||||
if curl -sf http://qdrant:6333/collections/evaluation >/dev/null; then
|
if curl -sf "$$QDRANT_URL/collections/$$QDRANT_COLLECTION_NAME" >/dev/null; then
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
curl -sf -X PUT http://qdrant:6333/collections/evaluation \
|
curl -sf -X PUT "$$QDRANT_URL/collections/$$QDRANT_COLLECTION_NAME" \
|
||||||
-H 'Content-Type: application/json' \
|
-H 'Content-Type: application/json' \
|
||||||
-d '{
|
-d "{
|
||||||
"vectors": {
|
\"vectors\": {
|
||||||
"dense": {
|
\"$$QDRANT_DENSE_VECTOR_NAME\": {
|
||||||
"size": 1024,
|
\"size\": 1024,
|
||||||
"distance": "Cosine"
|
\"distance\": \"Cosine\"
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"sparse_vectors": {
|
\"sparse_vectors\": {
|
||||||
"sparse": {
|
\"$$QDRANT_SPARSE_VECTOR_NAME\": {
|
||||||
"modifier": "idf"
|
\"modifier\": \"idf\"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}'
|
}"
|
||||||
restart: "no"
|
restart: "no"
|
||||||
|
|
||||||
index:
|
index:
|
||||||
|
|
@ -46,18 +48,10 @@ services:
|
||||||
search:
|
search:
|
||||||
build:
|
build:
|
||||||
context: ./search
|
context: ./search
|
||||||
|
env_file:
|
||||||
|
- .env
|
||||||
depends_on:
|
depends_on:
|
||||||
qdrant-init:
|
qdrant-init:
|
||||||
condition: service_completed_successfully
|
condition: service_completed_successfully
|
||||||
environment:
|
|
||||||
QDRANT_URL: http://qdrant:6333
|
|
||||||
QDRANT_COLLECTION_NAME: evaluation
|
|
||||||
QDRANT_DENSE_VECTOR_NAME: dense
|
|
||||||
QDRANT_SPARSE_VECTOR_NAME: sparse
|
|
||||||
EMBEDDINGS_DENSE_URL: ${EMBEDDINGS_DENSE_URL:-http://83.166.249.64:18001/embeddings}
|
|
||||||
RERANKER_URL: ${RERANKER_URL:-http://83.166.249.64:18001/score}
|
|
||||||
OPEN_API_LOGIN: ${OPEN_API_LOGIN:?set OPEN_API_LOGIN before docker compose up}
|
|
||||||
OPEN_API_PASSWORD: ${OPEN_API_PASSWORD:?set OPEN_API_PASSWORD before docker compose up}
|
|
||||||
ports:
|
ports:
|
||||||
- "8002:8000"
|
- "8002:8000"
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -9,7 +9,6 @@ COPY main.py .
|
||||||
|
|
||||||
ENV HOST=0.0.0.0
|
ENV HOST=0.0.0.0
|
||||||
ENV PORT=8000
|
ENV PORT=8000
|
||||||
ENV CHUNK_SIZE=10
|
|
||||||
ENV FASTEMBED_CACHE_PATH=/models/fastembed
|
ENV FASTEMBED_CACHE_PATH=/models/fastembed
|
||||||
ENV HF_HOME=/models/huggingface
|
ENV HF_HOME=/models/huggingface
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -1,24 +1,26 @@
|
||||||
LOGIN ?=
|
LOGIN ?=
|
||||||
PASSWORD ?=
|
PASSWORD ?=
|
||||||
TEAM_ID ?=
|
TEAM_ID ?= 35230
|
||||||
DOCKER_REGISTRY_URL ?= 83.166.249.64:5000
|
DOCKER_REGISTRY_URL ?= 83.166.249.64:5000
|
||||||
PORT ?= 8000
|
PORT ?= 8000
|
||||||
|
|
||||||
IMAGE = $(DOCKER_REGISTRY_URL)/$(TEAM_ID)/index-service:latest
|
IMAGE = $(DOCKER_REGISTRY_URL)/$(TEAM_ID)/index-service:latest
|
||||||
|
|
||||||
.PHONY: login build run push
|
.PHONY: login build run push release
|
||||||
|
|
||||||
login:
|
login:
|
||||||
@: $(if $(LOGIN),,$(error LOGIN is required for make login))
|
@: $(if $(LOGIN),,$(error LOGIN is required))
|
||||||
@: $(if $(PASSWORD),,$(error PASSWORD is required for make login))
|
@: $(if $(PASSWORD),,$(error PASSWORD is required))
|
||||||
docker login $(DOCKER_REGISTRY_URL) -u $(LOGIN) -p $(PASSWORD)
|
docker login $(DOCKER_REGISTRY_URL) -u $(LOGIN) -p $(PASSWORD)
|
||||||
|
|
||||||
build:
|
build:
|
||||||
@: $(if $(TEAM_ID),,$(error TEAM_ID is required for make build))
|
docker build --platform linux/amd64 -t $(IMAGE) ./
|
||||||
docker build -t $(IMAGE) ./
|
|
||||||
|
|
||||||
run: build
|
run: build
|
||||||
docker run --rm -p $(PORT):8000 $(IMAGE)
|
docker run --rm -p $(PORT):8000 $(IMAGE)
|
||||||
|
|
||||||
push: login build
|
push:
|
||||||
docker push $(IMAGE)
|
docker push $(IMAGE)
|
||||||
|
|
||||||
|
release: build push
|
||||||
|
@echo "index-service pushed → $(IMAGE)"
|
||||||
|
|
|
||||||
107
index/chunking.py
Normal file
107
index/chunking.py
Normal file
|
|
@ -0,0 +1,107 @@
|
||||||
|
"""Message-based chunking with window by count, length, and time gap."""
|
||||||
|
|
||||||
|
from cleaning import CleanedMessage, clean_message
|
||||||
|
from rendering import render_dense_content, render_page_content, render_sparse_content
|
||||||
|
from index_schemas import IndexAPIItem, Message
|
||||||
|
|
||||||
|
WINDOW_MAX_MESSAGES = 5
|
||||||
|
WINDOW_MAX_CHARS = 512
|
||||||
|
TIME_GAP_SECONDS = 3600
|
||||||
|
OVERLAP_MESSAGES = 2
|
||||||
|
|
||||||
|
|
||||||
|
def _clean_all(messages: list[Message]) -> list[CleanedMessage]:
|
||||||
|
cleaned = [clean_message(m) for m in messages if not m.is_system and not m.is_hidden]
|
||||||
|
return [c for c in cleaned if not c.is_empty]
|
||||||
|
|
||||||
|
|
||||||
|
def _render_chunk(
|
||||||
|
overlap: list[CleanedMessage],
|
||||||
|
window: list[CleanedMessage],
|
||||||
|
) -> IndexAPIItem:
|
||||||
|
page_lines: list[str] = []
|
||||||
|
dense_lines: list[str] = []
|
||||||
|
sparse_tokens: list[str] = []
|
||||||
|
|
||||||
|
for msg in overlap + window:
|
||||||
|
page = render_page_content(msg)
|
||||||
|
dense = render_dense_content(msg)
|
||||||
|
sparse = render_sparse_content(msg)
|
||||||
|
if page:
|
||||||
|
page_lines.append(page)
|
||||||
|
if dense:
|
||||||
|
dense_lines.append(dense)
|
||||||
|
if sparse:
|
||||||
|
sparse_tokens.append(sparse)
|
||||||
|
|
||||||
|
return IndexAPIItem(
|
||||||
|
page_content="\n".join(page_lines),
|
||||||
|
dense_content="\n".join(dense_lines),
|
||||||
|
sparse_content=" ".join(sparse_tokens),
|
||||||
|
message_ids=[msg.id for msg in window],
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def _split_windows(messages: list[CleanedMessage]) -> list[list[CleanedMessage]]:
|
||||||
|
"""Split cleaned messages into windows respecting count, length, and time gap."""
|
||||||
|
if not messages:
|
||||||
|
return []
|
||||||
|
|
||||||
|
windows: list[list[CleanedMessage]] = []
|
||||||
|
current: list[CleanedMessage] = []
|
||||||
|
current_chars = 0
|
||||||
|
|
||||||
|
for msg in messages:
|
||||||
|
msg_text = render_page_content(msg)
|
||||||
|
msg_chars = len(msg_text)
|
||||||
|
|
||||||
|
time_break = (
|
||||||
|
current
|
||||||
|
and (msg.time - current[-1].time) > TIME_GAP_SECONDS
|
||||||
|
)
|
||||||
|
size_break = (
|
||||||
|
current
|
||||||
|
and (
|
||||||
|
len(current) >= WINDOW_MAX_MESSAGES
|
||||||
|
or current_chars + msg_chars > WINDOW_MAX_CHARS
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
if time_break or size_break:
|
||||||
|
if current:
|
||||||
|
windows.append(current)
|
||||||
|
current = [msg]
|
||||||
|
current_chars = msg_chars
|
||||||
|
else:
|
||||||
|
current.append(msg)
|
||||||
|
current_chars += msg_chars
|
||||||
|
|
||||||
|
if current:
|
||||||
|
windows.append(current)
|
||||||
|
|
||||||
|
return windows
|
||||||
|
|
||||||
|
|
||||||
|
def build_chunks(
|
||||||
|
overlap_messages: list[Message],
|
||||||
|
new_messages: list[Message],
|
||||||
|
) -> list[IndexAPIItem]:
|
||||||
|
clean_overlap = _clean_all(overlap_messages)
|
||||||
|
clean_new = _clean_all(new_messages)
|
||||||
|
|
||||||
|
if not clean_new:
|
||||||
|
return []
|
||||||
|
|
||||||
|
overlap_tail = clean_overlap[-OVERLAP_MESSAGES:] if clean_overlap else []
|
||||||
|
windows = _split_windows(clean_new)
|
||||||
|
|
||||||
|
result: list[IndexAPIItem] = []
|
||||||
|
prev_window_tail: list[CleanedMessage] = overlap_tail
|
||||||
|
|
||||||
|
for window in windows:
|
||||||
|
chunk = _render_chunk(prev_window_tail, window)
|
||||||
|
if chunk.message_ids:
|
||||||
|
result.append(chunk)
|
||||||
|
prev_window_tail = window[-OVERLAP_MESSAGES:]
|
||||||
|
|
||||||
|
return result
|
||||||
157
index/cleaning.py
Normal file
157
index/cleaning.py
Normal file
|
|
@ -0,0 +1,157 @@
|
||||||
|
"""Local message cleaning and normalization. No external API calls."""
|
||||||
|
|
||||||
|
import json
|
||||||
|
import re
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
_ZERO_WIDTH = re.compile(r"[\u200b\u200c\u200d\ufeff]")
|
||||||
|
_MULTI_NEWLINE = re.compile(r"\n{3,}")
|
||||||
|
_MULTI_SPACE = re.compile(r"[ \t]{2,}")
|
||||||
|
|
||||||
|
|
||||||
|
def normalize_unicode(text: str) -> str:
|
||||||
|
text = _ZERO_WIDTH.sub("", text)
|
||||||
|
text = text.replace("\r\n", "\n").replace("\r", "\n")
|
||||||
|
text = _MULTI_SPACE.sub(" ", text)
|
||||||
|
text = _MULTI_NEWLINE.sub("\n\n", text)
|
||||||
|
return text.strip()
|
||||||
|
|
||||||
|
|
||||||
|
def _safe_normalize(text: str | None) -> str:
|
||||||
|
if not text:
|
||||||
|
return ""
|
||||||
|
return normalize_unicode(str(text))
|
||||||
|
|
||||||
|
|
||||||
|
def parse_file_snippets(raw: str) -> list[dict[str, Any]]:
|
||||||
|
if not raw or not raw.strip():
|
||||||
|
return []
|
||||||
|
try:
|
||||||
|
parsed = json.loads(raw)
|
||||||
|
if isinstance(parsed, list):
|
||||||
|
return parsed
|
||||||
|
if isinstance(parsed, dict):
|
||||||
|
return [parsed]
|
||||||
|
return []
|
||||||
|
except (json.JSONDecodeError, ValueError):
|
||||||
|
return []
|
||||||
|
|
||||||
|
|
||||||
|
def extract_file_info(snippet: dict[str, Any]) -> dict[str, str]:
|
||||||
|
return {
|
||||||
|
"name": str(snippet.get("name") or ""),
|
||||||
|
"mime": str(snippet.get("mime") or ""),
|
||||||
|
"url": str(snippet.get("original_url") or ""),
|
||||||
|
"date": str(snippet.get("date_create") or ""),
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
def normalize_member_event(event: dict[str, Any] | None) -> str:
|
||||||
|
if not event:
|
||||||
|
return ""
|
||||||
|
event_type = str(event.get("type") or "unknown_event")
|
||||||
|
members = event.get("members") or []
|
||||||
|
if event_type == "addMembers" and members:
|
||||||
|
joined = ", ".join(str(m) for m in members)
|
||||||
|
return f"[system: {joined} added to chat]"
|
||||||
|
payload_str = json.dumps(event, ensure_ascii=False)
|
||||||
|
return f"[system: {event_type} {payload_str}]"
|
||||||
|
|
||||||
|
|
||||||
|
def normalize_part(part: dict[str, Any]) -> dict[str, str]:
|
||||||
|
"""Normalize a single message part by its mediaType."""
|
||||||
|
media_type = str(part.get("mediaType") or "text")
|
||||||
|
text = _safe_normalize(part.get("text"))
|
||||||
|
|
||||||
|
if media_type == "text":
|
||||||
|
return {"type": "text", "text": text}
|
||||||
|
|
||||||
|
if media_type == "quote":
|
||||||
|
sender = _safe_normalize(part.get("sn") or part.get("sender_id") or "")
|
||||||
|
label = f"[quote from {sender}]" if sender else "[quote]"
|
||||||
|
return {"type": "quote", "text": f"{label}: {text}" if text else label}
|
||||||
|
|
||||||
|
if media_type == "forward":
|
||||||
|
origin = _safe_normalize(part.get("sn") or "")
|
||||||
|
label = f"[forwarded from {origin}]" if origin else "[forwarded]"
|
||||||
|
return {"type": "forward", "text": f"{label}: {text}" if text else label}
|
||||||
|
|
||||||
|
label = f"[{media_type}]"
|
||||||
|
return {"type": media_type, "text": f"{label}: {text}" if text else label}
|
||||||
|
|
||||||
|
|
||||||
|
class CleanedMessage:
|
||||||
|
__slots__ = (
|
||||||
|
"id",
|
||||||
|
"sender_id",
|
||||||
|
"time",
|
||||||
|
"thread_sn",
|
||||||
|
"text",
|
||||||
|
"parts",
|
||||||
|
"mentions",
|
||||||
|
"member_event_text",
|
||||||
|
"file_info",
|
||||||
|
"is_system",
|
||||||
|
"is_forward",
|
||||||
|
"is_quote",
|
||||||
|
"is_empty",
|
||||||
|
)
|
||||||
|
|
||||||
|
def __init__(
|
||||||
|
self,
|
||||||
|
id: str,
|
||||||
|
sender_id: str,
|
||||||
|
time: int,
|
||||||
|
thread_sn: str | None,
|
||||||
|
text: str,
|
||||||
|
parts: list[dict[str, str]],
|
||||||
|
mentions: list[str],
|
||||||
|
member_event_text: str,
|
||||||
|
file_info: list[dict[str, str]],
|
||||||
|
is_system: bool,
|
||||||
|
is_forward: bool,
|
||||||
|
is_quote: bool,
|
||||||
|
):
|
||||||
|
self.id = id
|
||||||
|
self.sender_id = sender_id
|
||||||
|
self.time = time
|
||||||
|
self.thread_sn = thread_sn
|
||||||
|
self.text = text
|
||||||
|
self.parts = parts
|
||||||
|
self.mentions = mentions
|
||||||
|
self.member_event_text = member_event_text
|
||||||
|
self.file_info = file_info
|
||||||
|
self.is_system = is_system
|
||||||
|
self.is_forward = is_forward
|
||||||
|
self.is_quote = is_quote
|
||||||
|
self.is_empty = not (text or parts or member_event_text or file_info)
|
||||||
|
|
||||||
|
|
||||||
|
def clean_message(msg: Any) -> CleanedMessage:
|
||||||
|
"""Extract and normalize all signals from a raw Message object."""
|
||||||
|
text = _safe_normalize(msg.text)
|
||||||
|
parts = [normalize_part(p) for p in (msg.parts or []) if isinstance(p, dict)]
|
||||||
|
parts = [p for p in parts if p.get("text")]
|
||||||
|
mentions = [_safe_normalize(m) for m in (msg.mentions or []) if m]
|
||||||
|
member_event_text = normalize_member_event(msg.member_event)
|
||||||
|
|
||||||
|
file_info: list[dict[str, str]] = []
|
||||||
|
for snippet in parse_file_snippets(msg.file_snippets):
|
||||||
|
info = extract_file_info(snippet)
|
||||||
|
if any(info.values()):
|
||||||
|
file_info.append(info)
|
||||||
|
|
||||||
|
return CleanedMessage(
|
||||||
|
id=msg.id,
|
||||||
|
sender_id=_safe_normalize(msg.sender_id),
|
||||||
|
time=msg.time,
|
||||||
|
thread_sn=msg.thread_sn,
|
||||||
|
text=text,
|
||||||
|
parts=parts,
|
||||||
|
mentions=mentions,
|
||||||
|
member_event_text=member_event_text,
|
||||||
|
file_info=file_info,
|
||||||
|
is_system=bool(msg.is_system),
|
||||||
|
is_forward=bool(msg.is_forward),
|
||||||
|
is_quote=bool(msg.is_quote),
|
||||||
|
)
|
||||||
59
index/index_schemas.py
Normal file
59
index/index_schemas.py
Normal file
|
|
@ -0,0 +1,59 @@
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
from pydantic import BaseModel
|
||||||
|
|
||||||
|
|
||||||
|
class Chat(BaseModel):
|
||||||
|
id: str
|
||||||
|
name: str
|
||||||
|
sn: str
|
||||||
|
type: str
|
||||||
|
is_public: bool | None = None
|
||||||
|
members_count: int | None = None
|
||||||
|
members: list[dict[str, Any]] | None = None
|
||||||
|
|
||||||
|
|
||||||
|
class Message(BaseModel):
|
||||||
|
id: str
|
||||||
|
thread_sn: str | None = None
|
||||||
|
time: int
|
||||||
|
text: str
|
||||||
|
sender_id: str
|
||||||
|
file_snippets: str
|
||||||
|
parts: list[dict[str, Any]] | None = None
|
||||||
|
mentions: list[str] | None = None
|
||||||
|
member_event: dict[str, Any] | None = None
|
||||||
|
is_system: bool
|
||||||
|
is_hidden: bool
|
||||||
|
is_forward: bool
|
||||||
|
is_quote: bool
|
||||||
|
|
||||||
|
|
||||||
|
class ChatData(BaseModel):
|
||||||
|
chat: Chat
|
||||||
|
overlap_messages: list[Message]
|
||||||
|
new_messages: list[Message]
|
||||||
|
|
||||||
|
|
||||||
|
class IndexAPIRequest(BaseModel):
|
||||||
|
data: ChatData
|
||||||
|
|
||||||
|
|
||||||
|
class IndexAPIItem(BaseModel):
|
||||||
|
page_content: str
|
||||||
|
dense_content: str
|
||||||
|
sparse_content: str
|
||||||
|
message_ids: list[str]
|
||||||
|
|
||||||
|
|
||||||
|
class IndexAPIResponse(BaseModel):
|
||||||
|
results: list[IndexAPIItem]
|
||||||
|
|
||||||
|
|
||||||
|
class SparseEmbeddingRequest(BaseModel):
|
||||||
|
texts: list[str]
|
||||||
|
|
||||||
|
|
||||||
|
class SparseVector(BaseModel):
|
||||||
|
indices: list[int]
|
||||||
|
values: list[float]
|
||||||
107
index/main.py
107
index/main.py
|
|
@ -1,29 +1,27 @@
|
||||||
|
import asyncio
|
||||||
import logging
|
import logging
|
||||||
import os
|
import os
|
||||||
from functools import lru_cache
|
from functools import lru_cache
|
||||||
from typing import Any
|
from typing import Any
|
||||||
import asyncio
|
|
||||||
import hashlib
|
|
||||||
|
|
||||||
from fastapi import FastAPI, Request
|
from fastapi import FastAPI, Request
|
||||||
from fastapi.exceptions import RequestValidationError
|
from fastapi.exceptions import RequestValidationError
|
||||||
from fastapi.responses import JSONResponse
|
from fastapi.responses import JSONResponse
|
||||||
from pydantic import BaseModel
|
from pydantic import BaseModel
|
||||||
|
|
||||||
# Ваш сервис должен считывать эти переменные из окружения (env), так как проверяющая система управляет ими
|
|
||||||
HOST = os.getenv("HOST", "0.0.0.0")
|
HOST = os.getenv("HOST", "0.0.0.0")
|
||||||
PORT = int(os.getenv("PORT", "8004"))
|
PORT = int(os.getenv("PORT", "8000"))
|
||||||
|
UVICORN_WORKERS = 8
|
||||||
|
|
||||||
logging.basicConfig(level=os.getenv("LOG_LEVEL", "INFO"))
|
logging.basicConfig(level=os.getenv("LOG_LEVEL", "INFO"))
|
||||||
logger = logging.getLogger("index-service")
|
logger = logging.getLogger("index-service")
|
||||||
|
|
||||||
|
|
||||||
# Модель данных, которую мы предоставляем и рассчитываем получать от вас
|
|
||||||
class Chat(BaseModel):
|
class Chat(BaseModel):
|
||||||
id: str
|
id: str
|
||||||
name: str
|
name: str
|
||||||
sn: str
|
sn: str
|
||||||
type: str # group, channel, private
|
type: str
|
||||||
is_public: bool | None = None
|
is_public: bool | None = None
|
||||||
members_count: int | None = None
|
members_count: int | None = None
|
||||||
members: list[dict[str, Any]] | None = None
|
members: list[dict[str, Any]] | None = None
|
||||||
|
|
@ -55,10 +53,6 @@ class IndexAPIRequest(BaseModel):
|
||||||
data: ChatData
|
data: ChatData
|
||||||
|
|
||||||
|
|
||||||
# dense_content будет передан в dense embedding модель для построения семантического вектора.
|
|
||||||
# sparse_content будет передан в sparse модель для построения разреженного индекса "по словам".
|
|
||||||
# Можно оставить dense_content и sparse_content равными page_content,
|
|
||||||
# а можно формировать для них разные версии текста.
|
|
||||||
class IndexAPIItem(BaseModel):
|
class IndexAPIItem(BaseModel):
|
||||||
page_content: str
|
page_content: str
|
||||||
dense_content: str
|
dense_content: str
|
||||||
|
|
@ -79,46 +73,48 @@ class SparseVector(BaseModel):
|
||||||
values: list[float]
|
values: list[float]
|
||||||
|
|
||||||
|
|
||||||
class SparseEmbeddingResponse(BaseModel):
|
CHUNK_SIZE = 256
|
||||||
vectors: list[SparseVector]
|
OVERLAP_SIZE = 128
|
||||||
|
|
||||||
|
|
||||||
app = FastAPI(title="Index Service", version="0.1.0")
|
|
||||||
|
|
||||||
# Ваша внутренняя логика построения чанков. Можете делать всё, что посчитаете нужным.
|
|
||||||
# Текущий код – минимальный пример
|
|
||||||
|
|
||||||
CHUNK_SIZE = 512
|
|
||||||
OVERLAP_SIZE = 256
|
|
||||||
SPARSE_MODEL_NAME = "Qdrant/bm25"
|
SPARSE_MODEL_NAME = "Qdrant/bm25"
|
||||||
FASTEMBED_CACHE_PATH = "/models/fastembed"
|
FASTEMBED_CACHE_PATH = "/models/fastembed"
|
||||||
|
|
||||||
# Важная переманная, которая позволяет вычислять sparse вектор в несколько ядер. Не рекомендуется изменять.
|
|
||||||
UVICORN_WORKERS=8
|
|
||||||
|
|
||||||
def render_message(message: Message) -> str:
|
def render_message(message: Message) -> str:
|
||||||
text = ""
|
parts_list: list[str] = []
|
||||||
|
|
||||||
|
if message.sender_id:
|
||||||
|
sender_name = message.sender_id.split("@")[0].replace(".", " ")
|
||||||
|
parts_list.append(f"[{sender_name}]:")
|
||||||
|
|
||||||
if message.text:
|
if message.text:
|
||||||
text += message.text
|
parts_list.append(message.text)
|
||||||
|
|
||||||
if message.parts:
|
if message.parts:
|
||||||
parts_text: list[str] = []
|
|
||||||
for part in message.parts:
|
for part in message.parts:
|
||||||
# parts различаются по своему типу, см. README.md
|
media_type = part.get("mediaType", "text")
|
||||||
part_text = part.get("text")
|
part_text = part.get("text")
|
||||||
if isinstance(part_text, str) and part_text:
|
if isinstance(part_text, str) and part_text:
|
||||||
parts_text.append(part_text)
|
if media_type == "forward":
|
||||||
if parts_text:
|
parts_list.append(f"[Пересланное]: {part_text}")
|
||||||
text += "\n".join(parts_text)
|
elif media_type == "quote":
|
||||||
|
parts_list.append(f"[Цитата]: {part_text}")
|
||||||
|
else:
|
||||||
|
parts_list.append(part_text)
|
||||||
|
|
||||||
return text
|
if message.file_snippets:
|
||||||
|
parts_list.append(f"[Файл]: {message.file_snippets}")
|
||||||
|
|
||||||
|
return " ".join(parts_list).strip()
|
||||||
|
|
||||||
|
|
||||||
def build_chunks(
|
def build_chunks(
|
||||||
|
chat: Chat,
|
||||||
overlap_messages: list[Message],
|
overlap_messages: list[Message],
|
||||||
new_messages: list[Message],
|
new_messages: list[Message],
|
||||||
) -> list[IndexAPIItem]:
|
) -> list[IndexAPIItem]:
|
||||||
|
new_messages = [m for m in new_messages if not m.is_system and not m.is_hidden]
|
||||||
|
overlap_messages = [m for m in overlap_messages if not m.is_system and not m.is_hidden]
|
||||||
|
|
||||||
result: list[IndexAPIItem] = []
|
result: list[IndexAPIItem] = []
|
||||||
|
|
||||||
def build_text_and_ranges(messages: list[Message]) -> tuple[str, list[tuple[int, int, str]]]:
|
def build_text_and_ranges(messages: list[Message]) -> tuple[str, list[tuple[int, int, str]]]:
|
||||||
|
|
@ -142,17 +138,13 @@ def build_chunks(
|
||||||
|
|
||||||
return "".join(text_parts), message_ranges
|
return "".join(text_parts), message_ranges
|
||||||
|
|
||||||
def slice_tail(
|
def slice_tail(text: str, tail_size: int) -> str:
|
||||||
text: str,
|
|
||||||
tail_size: int,
|
|
||||||
) -> str:
|
|
||||||
if tail_size <= 0:
|
if tail_size <= 0:
|
||||||
return ""
|
return ""
|
||||||
|
|
||||||
tail_start = max(0, len(text) - tail_size)
|
tail_start = max(0, len(text) - tail_size)
|
||||||
return text[tail_start:]
|
return text[tail_start:]
|
||||||
|
|
||||||
overlap_text, overlap_message_ranges = build_text_and_ranges(overlap_messages)
|
overlap_text, _ = build_text_and_ranges(overlap_messages)
|
||||||
previous_chunk_text = slice_tail(overlap_text, OVERLAP_SIZE)
|
previous_chunk_text = slice_tail(overlap_text, OVERLAP_SIZE)
|
||||||
|
|
||||||
new_text, new_message_ranges = build_text_and_ranges(new_messages)
|
new_text, new_message_ranges = build_text_and_ranges(new_messages)
|
||||||
|
|
@ -171,17 +163,21 @@ def build_chunks(
|
||||||
for message_start, message_end, message_id in new_message_ranges
|
for message_start, message_end, message_id in new_message_ranges
|
||||||
if message_end > start and message_start < start + len(chunk_body)
|
if message_end > start and message_start < start + len(chunk_body)
|
||||||
]
|
]
|
||||||
|
|
||||||
chunk_overlap = previous_chunk_text
|
chunk_overlap = previous_chunk_text
|
||||||
chunk_text = chunk_overlap
|
chunk_text = chunk_overlap
|
||||||
if chunk_text and chunk_body:
|
if chunk_text and chunk_body:
|
||||||
chunk_text += "\n"
|
chunk_text += "\n"
|
||||||
chunk_text += chunk_body
|
chunk_text += chunk_body
|
||||||
|
|
||||||
|
dense_text = f"[{chat.name}] {chunk_text}"
|
||||||
|
sparse_text = chunk_body
|
||||||
|
|
||||||
result.append(
|
result.append(
|
||||||
IndexAPIItem(
|
IndexAPIItem(
|
||||||
page_content=chunk_text,
|
page_content=chunk_text,
|
||||||
dense_content=chunk_text,
|
dense_content=dense_text,
|
||||||
sparse_content=chunk_text,
|
sparse_content=sparse_text,
|
||||||
message_ids=[message_id for _, _, message_id in chunk_body_ranges],
|
message_ids=[message_id for _, _, message_id in chunk_body_ranges],
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
@ -189,7 +185,10 @@ def build_chunks(
|
||||||
|
|
||||||
return result
|
return result
|
||||||
|
|
||||||
# Ваш сервис должен имплементировать оба этих метода
|
|
||||||
|
app = FastAPI(title="Index Service", version="0.1.0")
|
||||||
|
|
||||||
|
|
||||||
@app.get("/health")
|
@app.get("/health")
|
||||||
async def health() -> dict[str, str]:
|
async def health() -> dict[str, str]:
|
||||||
return {"status": "ok"}
|
return {"status": "ok"}
|
||||||
|
|
@ -199,6 +198,7 @@ async def health() -> dict[str, str]:
|
||||||
async def index(payload: IndexAPIRequest) -> IndexAPIResponse:
|
async def index(payload: IndexAPIRequest) -> IndexAPIResponse:
|
||||||
return IndexAPIResponse(
|
return IndexAPIResponse(
|
||||||
results=build_chunks(
|
results=build_chunks(
|
||||||
|
payload.data.chat,
|
||||||
payload.data.overlap_messages,
|
payload.data.overlap_messages,
|
||||||
payload.data.new_messages,
|
payload.data.new_messages,
|
||||||
)
|
)
|
||||||
|
|
@ -209,20 +209,13 @@ async def index(payload: IndexAPIRequest) -> IndexAPIResponse:
|
||||||
def get_sparse_model():
|
def get_sparse_model():
|
||||||
from fastembed import SparseTextEmbedding
|
from fastembed import SparseTextEmbedding
|
||||||
|
|
||||||
# можете делать любой вектор, который будет совместим с вашим поиском в Qdrant
|
logger.info("Loading sparse model %s from cache %s", SPARSE_MODEL_NAME, FASTEMBED_CACHE_PATH)
|
||||||
# помните об ограничении времени выполнения вашей работы в тестирующей системе
|
|
||||||
logger.info(
|
|
||||||
"Loading sparse model %s from cache %s",
|
|
||||||
SPARSE_MODEL_NAME,
|
|
||||||
FASTEMBED_CACHE_PATH,
|
|
||||||
)
|
|
||||||
return SparseTextEmbedding(model_name=SPARSE_MODEL_NAME)
|
return SparseTextEmbedding(model_name=SPARSE_MODEL_NAME)
|
||||||
|
|
||||||
|
|
||||||
def embed_sparse_texts(texts: list[str]) -> list[SparseVector]:
|
def embed_sparse_texts(texts: list[str]) -> list[dict]:
|
||||||
model = get_sparse_model()
|
model = get_sparse_model()
|
||||||
vectors: list[dict[str, list[int] | list[float]]] = []
|
vectors = []
|
||||||
|
|
||||||
for item in model.embed(texts):
|
for item in model.embed(texts):
|
||||||
vectors.append(
|
vectors.append(
|
||||||
{
|
{
|
||||||
|
|
@ -230,37 +223,27 @@ def embed_sparse_texts(texts: list[str]) -> list[SparseVector]:
|
||||||
"values": item.values.tolist(),
|
"values": item.values.tolist(),
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
|
||||||
return vectors
|
return vectors
|
||||||
|
|
||||||
|
|
||||||
@app.post("/sparse_embedding")
|
@app.post("/sparse_embedding")
|
||||||
async def sparse_embedding(payload: SparseEmbeddingRequest) -> dict[str, Any]:
|
async def sparse_embedding(payload: SparseEmbeddingRequest) -> dict[str, Any]:
|
||||||
# Проверяющая система вызывает этот endpoint при создании коллекции
|
|
||||||
vectors = await asyncio.to_thread(embed_sparse_texts, payload.texts)
|
vectors = await asyncio.to_thread(embed_sparse_texts, payload.texts)
|
||||||
return {"vectors": vectors}
|
return {"vectors": vectors}
|
||||||
|
|
||||||
# красивая обработка ошибок
|
|
||||||
@app.exception_handler(Exception)
|
@app.exception_handler(Exception)
|
||||||
async def exception_handler(request: Request, exc: Exception) -> JSONResponse:
|
async def exception_handler(request: Request, exc: Exception) -> JSONResponse:
|
||||||
logger.exception(exc)
|
logger.exception(exc)
|
||||||
|
|
||||||
if isinstance(exc, RequestValidationError):
|
if isinstance(exc, RequestValidationError):
|
||||||
return JSONResponse(status_code=422, content={"detail": exc.errors()})
|
return JSONResponse(status_code=422, content={"detail": exc.errors()})
|
||||||
|
|
||||||
return JSONResponse(status_code=500, content={"detail": str(exc)})
|
return JSONResponse(status_code=500, content={"detail": str(exc)})
|
||||||
|
|
||||||
|
|
||||||
def main() -> None:
|
def main() -> None:
|
||||||
import uvicorn
|
import uvicorn
|
||||||
|
|
||||||
uvicorn.run(
|
uvicorn.run("main:app", host=HOST, port=PORT, reload=False, workers=UVICORN_WORKERS)
|
||||||
"main:app",
|
|
||||||
host=HOST,
|
|
||||||
port=PORT,
|
|
||||||
reload=False,
|
|
||||||
workers=UVICORN_WORKERS,
|
|
||||||
)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
|
|
|
||||||
102
index/rendering.py
Normal file
102
index/rendering.py
Normal file
|
|
@ -0,0 +1,102 @@
|
||||||
|
"""Render cleaned messages into page_content, dense_content, sparse_content."""
|
||||||
|
|
||||||
|
import datetime
|
||||||
|
|
||||||
|
from cleaning import CleanedMessage
|
||||||
|
|
||||||
|
|
||||||
|
def _format_time(ts: int) -> str:
|
||||||
|
try:
|
||||||
|
return datetime.datetime.fromtimestamp(ts, tz=datetime.timezone.utc).strftime("%Y-%m-%d %H:%M")
|
||||||
|
except (OSError, OverflowError, ValueError):
|
||||||
|
return str(ts)
|
||||||
|
|
||||||
|
|
||||||
|
def render_page_content(msg: CleanedMessage) -> str:
|
||||||
|
"""Human-readable text for the chunk payload."""
|
||||||
|
lines: list[str] = []
|
||||||
|
|
||||||
|
prefix = f"{msg.sender_id}:"
|
||||||
|
if msg.text:
|
||||||
|
lines.append(f"{prefix} {msg.text}")
|
||||||
|
elif not msg.parts and not msg.member_event_text:
|
||||||
|
lines.append(prefix)
|
||||||
|
|
||||||
|
for part in msg.parts:
|
||||||
|
lines.append(part["text"])
|
||||||
|
|
||||||
|
if msg.member_event_text:
|
||||||
|
lines.append(msg.member_event_text)
|
||||||
|
|
||||||
|
for fi in msg.file_info:
|
||||||
|
name = fi.get("name") or fi.get("url") or "file"
|
||||||
|
lines.append(f"[attachment: {name}]")
|
||||||
|
|
||||||
|
return "\n".join(lines)
|
||||||
|
|
||||||
|
|
||||||
|
def render_dense_content(msg: CleanedMessage) -> str:
|
||||||
|
"""Text optimized for semantic dense embedding: role markers + full context."""
|
||||||
|
lines: list[str] = []
|
||||||
|
ts = _format_time(msg.time)
|
||||||
|
|
||||||
|
header_parts = [f"[{ts}]", f"sender:{msg.sender_id}"]
|
||||||
|
if msg.is_forward:
|
||||||
|
header_parts.append("type:forward")
|
||||||
|
if msg.is_quote:
|
||||||
|
header_parts.append("type:quote")
|
||||||
|
if msg.is_system:
|
||||||
|
header_parts.append("type:system")
|
||||||
|
if msg.thread_sn:
|
||||||
|
header_parts.append(f"thread:{msg.thread_sn}")
|
||||||
|
lines.append(" ".join(header_parts))
|
||||||
|
|
||||||
|
if msg.text:
|
||||||
|
lines.append(msg.text)
|
||||||
|
|
||||||
|
for part in msg.parts:
|
||||||
|
lines.append(part["text"])
|
||||||
|
|
||||||
|
if msg.member_event_text:
|
||||||
|
lines.append(msg.member_event_text)
|
||||||
|
|
||||||
|
for fi in msg.file_info:
|
||||||
|
fi_parts = []
|
||||||
|
if fi.get("name"):
|
||||||
|
fi_parts.append(fi["name"])
|
||||||
|
if fi.get("mime"):
|
||||||
|
fi_parts.append(fi["mime"])
|
||||||
|
if fi.get("url"):
|
||||||
|
fi_parts.append(fi["url"])
|
||||||
|
if fi_parts:
|
||||||
|
lines.append(f"[file: {' '.join(fi_parts)}]")
|
||||||
|
|
||||||
|
if msg.mentions:
|
||||||
|
lines.append("mentions: " + ", ".join(msg.mentions))
|
||||||
|
|
||||||
|
return "\n".join(lines)
|
||||||
|
|
||||||
|
|
||||||
|
def render_sparse_content(msg: CleanedMessage) -> str:
|
||||||
|
"""Keyword-heavy text for sparse/BM25 embedding."""
|
||||||
|
tokens: list[str] = []
|
||||||
|
|
||||||
|
tokens.append(msg.sender_id)
|
||||||
|
tokens.extend(msg.mentions)
|
||||||
|
|
||||||
|
if msg.text:
|
||||||
|
tokens.append(msg.text)
|
||||||
|
|
||||||
|
for part in msg.parts:
|
||||||
|
tokens.append(part["text"])
|
||||||
|
|
||||||
|
if msg.member_event_text:
|
||||||
|
tokens.append(msg.member_event_text)
|
||||||
|
|
||||||
|
for fi in msg.file_info:
|
||||||
|
for key in ("name", "mime", "url"):
|
||||||
|
val = fi.get(key)
|
||||||
|
if val:
|
||||||
|
tokens.append(val)
|
||||||
|
|
||||||
|
return " ".join(tokens)
|
||||||
31
index/sparse.py
Normal file
31
index/sparse.py
Normal file
|
|
@ -0,0 +1,31 @@
|
||||||
|
import logging
|
||||||
|
import os
|
||||||
|
from functools import lru_cache
|
||||||
|
|
||||||
|
from index_schemas import SparseVector
|
||||||
|
|
||||||
|
SPARSE_MODEL_NAME = "Qdrant/bm25"
|
||||||
|
FASTEMBED_CACHE_PATH = "/models/fastembed"
|
||||||
|
|
||||||
|
logger = logging.getLogger("index-service")
|
||||||
|
|
||||||
|
|
||||||
|
@lru_cache(maxsize=1)
|
||||||
|
def get_sparse_model():
|
||||||
|
from fastembed import SparseTextEmbedding
|
||||||
|
|
||||||
|
logger.info("Loading sparse model %s from cache %s", SPARSE_MODEL_NAME, FASTEMBED_CACHE_PATH)
|
||||||
|
return SparseTextEmbedding(model_name=SPARSE_MODEL_NAME)
|
||||||
|
|
||||||
|
|
||||||
|
def embed_sparse_texts(texts: list[str]) -> list[SparseVector]:
|
||||||
|
model = get_sparse_model()
|
||||||
|
result: list[SparseVector] = []
|
||||||
|
for item in model.embed(texts):
|
||||||
|
result.append(
|
||||||
|
SparseVector(
|
||||||
|
indices=[int(i) for i in item.indices.tolist()],
|
||||||
|
values=[float(v) for v in item.values.tolist()],
|
||||||
|
)
|
||||||
|
)
|
||||||
|
return result
|
||||||
41
run_codex.sh
Executable file
41
run_codex.sh
Executable file
|
|
@ -0,0 +1,41 @@
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## 3) `bin/codex-start` чтобы всё поднималось само
|
||||||
|
|
||||||
|
`SKILL.md` сам по себе **не умеет магически запускать Codex при открытии терминала**. Для этого нужен обычный wrapper-скрипт. Вот рабочий вариант:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
#!/usr/bin/env bash
|
||||||
|
set -euo pipefail
|
||||||
|
|
||||||
|
ROOT="$(git rev-parse --show-toplevel 2>/dev/null || pwd)"
|
||||||
|
cd "$ROOT"
|
||||||
|
|
||||||
|
mkdir -p .ai_update/sessions
|
||||||
|
touch .ai_update/current_status.md
|
||||||
|
touch .ai_update/handoff.md
|
||||||
|
touch .ai_update/changelog.md
|
||||||
|
touch .ai_update/touched_files.md
|
||||||
|
|
||||||
|
COMPOSE_FILE=""
|
||||||
|
if [ -f docker-compose.yml ]; then
|
||||||
|
COMPOSE_FILE="docker-compose.yml"
|
||||||
|
elif [ -f compose.yaml ]; then
|
||||||
|
COMPOSE_FILE="compose.yaml"
|
||||||
|
elif [ -f compose.yml ]; then
|
||||||
|
COMPOSE_FILE="compose.yml"
|
||||||
|
fi
|
||||||
|
|
||||||
|
if [ -n "$COMPOSE_FILE" ] && command -v docker >/dev/null 2>&1; then
|
||||||
|
if ! docker compose ps --status running >/dev/null 2>&1; then
|
||||||
|
docker compose up -d --build || true
|
||||||
|
else
|
||||||
|
RUNNING_COUNT="$(docker compose ps --status running --services 2>/dev/null | wc -l | tr -d ' ')"
|
||||||
|
if [ "${RUNNING_COUNT:-0}" = "0" ]; then
|
||||||
|
docker compose up -d --build || true
|
||||||
|
fi
|
||||||
|
fi
|
||||||
|
fi
|
||||||
|
|
||||||
|
exec codex "$@"
|
||||||
|
|
@ -1,6 +1,6 @@
|
||||||
LOGIN ?=
|
LOGIN ?=
|
||||||
PASSWORD ?=
|
PASSWORD ?=
|
||||||
TEAM_ID ?=
|
TEAM_ID ?= 35230
|
||||||
DOCKER_REGISTRY_URL ?= 83.166.249.64:5000
|
DOCKER_REGISTRY_URL ?= 83.166.249.64:5000
|
||||||
PORT ?= 8000
|
PORT ?= 8000
|
||||||
QDRANT_URL ?=
|
QDRANT_URL ?=
|
||||||
|
|
@ -16,16 +16,15 @@ REQUIRED_RUN_VARS := QDRANT_URL EMBEDDINGS_DENSE_URL API_KEY RERANKER_URL
|
||||||
|
|
||||||
IMAGE = $(DOCKER_REGISTRY_URL)/$(TEAM_ID)/search-service:latest
|
IMAGE = $(DOCKER_REGISTRY_URL)/$(TEAM_ID)/search-service:latest
|
||||||
|
|
||||||
.PHONY: login build run push check-run-env
|
.PHONY: login build run push release
|
||||||
|
|
||||||
login:
|
login:
|
||||||
@: $(if $(LOGIN),,$(error LOGIN is required for make login))
|
@: $(if $(LOGIN),,$(error LOGIN is required))
|
||||||
@: $(if $(PASSWORD),,$(error PASSWORD is required for make login))
|
@: $(if $(PASSWORD),,$(error PASSWORD is required))
|
||||||
docker login $(DOCKER_REGISTRY_URL) -u $(LOGIN) -p $(PASSWORD)
|
docker login $(DOCKER_REGISTRY_URL) -u $(LOGIN) -p $(PASSWORD)
|
||||||
|
|
||||||
build:
|
build:
|
||||||
@: $(if $(TEAM_ID),,$(error TEAM_ID is required for make build))
|
docker build --platform linux/amd64 -t $(IMAGE) ./
|
||||||
docker build -t $(IMAGE) ./
|
|
||||||
|
|
||||||
run: build
|
run: build
|
||||||
@: $(foreach var,$(REQUIRED_RUN_VARS),$(if $($(var)),,$(error $(var) is required for make run)))
|
@: $(foreach var,$(REQUIRED_RUN_VARS),$(if $($(var)),,$(error $(var) is required for make run)))
|
||||||
|
|
@ -41,5 +40,8 @@ run: build
|
||||||
-e QDRANT_SPARSE_VECTOR_NAME=$(QDRANT_SPARSE_VECTOR_NAME) \
|
-e QDRANT_SPARSE_VECTOR_NAME=$(QDRANT_SPARSE_VECTOR_NAME) \
|
||||||
$(IMAGE)
|
$(IMAGE)
|
||||||
|
|
||||||
push: login build
|
push:
|
||||||
docker push $(IMAGE)
|
docker push $(IMAGE)
|
||||||
|
|
||||||
|
release: build push
|
||||||
|
@echo "search-service pushed → $(IMAGE)"
|
||||||
|
|
|
||||||
0
search/__init__.py
Normal file
0
search/__init__.py
Normal file
23
search/aggregation.py
Normal file
23
search/aggregation.py
Normal file
|
|
@ -0,0 +1,23 @@
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
from config import TOP_K
|
||||||
|
from retrieval import extract_message_ids
|
||||||
|
|
||||||
|
|
||||||
|
def aggregate_message_ids(
|
||||||
|
reranked_head: list[Any],
|
||||||
|
retrieval_tail: list[Any],
|
||||||
|
) -> list[str]:
|
||||||
|
"""Deduplicate and collect top-K message_ids, reranked head first."""
|
||||||
|
seen: set[str] = set()
|
||||||
|
result: list[str] = []
|
||||||
|
|
||||||
|
for point in reranked_head + retrieval_tail:
|
||||||
|
for mid in extract_message_ids(point):
|
||||||
|
if mid not in seen:
|
||||||
|
seen.add(mid)
|
||||||
|
result.append(mid)
|
||||||
|
if len(result) >= TOP_K:
|
||||||
|
return result
|
||||||
|
|
||||||
|
return result
|
||||||
56
search/config.py
Normal file
56
search/config.py
Normal file
|
|
@ -0,0 +1,56 @@
|
||||||
|
import logging
|
||||||
|
import os
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
EMBEDDINGS_DENSE_MODEL = "Qwen/Qwen3-Embedding-0.6B"
|
||||||
|
SPARSE_MODEL_NAME = "Qdrant/bm25"
|
||||||
|
RERANKER_MODEL = "nvidia/llama-nemotron-rerank-1b-v2"
|
||||||
|
|
||||||
|
HOST = os.getenv("HOST", "0.0.0.0")
|
||||||
|
PORT = int(os.getenv("PORT", "8003"))
|
||||||
|
|
||||||
|
API_KEY = os.getenv("API_KEY")
|
||||||
|
EMBEDDINGS_DENSE_URL = os.getenv("EMBEDDINGS_DENSE_URL")
|
||||||
|
RERANKER_URL = os.getenv("RERANKER_URL")
|
||||||
|
QDRANT_URL = os.getenv("QDRANT_URL")
|
||||||
|
QDRANT_COLLECTION_NAME = os.getenv("QDRANT_COLLECTION_NAME", "evaluation")
|
||||||
|
QDRANT_DENSE_VECTOR_NAME = os.getenv("QDRANT_DENSE_VECTOR_NAME", "dense")
|
||||||
|
QDRANT_SPARSE_VECTOR_NAME = os.getenv("QDRANT_SPARSE_VECTOR_NAME", "sparse")
|
||||||
|
OPEN_API_LOGIN = os.getenv("OPEN_API_LOGIN")
|
||||||
|
OPEN_API_PASSWORD = os.getenv("OPEN_API_PASSWORD")
|
||||||
|
|
||||||
|
DENSE_PREFETCH_K = 80
|
||||||
|
SPARSE_PREFETCH_K = 200
|
||||||
|
RETRIEVE_K = 150
|
||||||
|
RERANK_LIMIT = 15
|
||||||
|
TOP_K = 50
|
||||||
|
|
||||||
|
HTTP_TIMEOUT = 30.0
|
||||||
|
HTTP_MAX_RETRIES = 2
|
||||||
|
|
||||||
|
REQUIRED_ENV_VARS = ["EMBEDDINGS_DENSE_URL", "RERANKER_URL", "QDRANT_URL"]
|
||||||
|
|
||||||
|
logging.basicConfig(level=os.getenv("LOG_LEVEL", "INFO"))
|
||||||
|
logger = logging.getLogger("search-service")
|
||||||
|
|
||||||
|
|
||||||
|
def validate_required_env() -> None:
|
||||||
|
if bool(OPEN_API_LOGIN) != bool(OPEN_API_PASSWORD):
|
||||||
|
raise RuntimeError("OPEN_API_LOGIN and OPEN_API_PASSWORD must be set together")
|
||||||
|
if not API_KEY and not (OPEN_API_LOGIN and OPEN_API_PASSWORD):
|
||||||
|
raise RuntimeError("Either API_KEY or OPEN_API_LOGIN and OPEN_API_PASSWORD must be set")
|
||||||
|
missing = [name for name in REQUIRED_ENV_VARS if not os.getenv(name)]
|
||||||
|
if missing:
|
||||||
|
logger.error("Empty required env vars: %s", ", ".join(missing))
|
||||||
|
raise RuntimeError(f"Empty required env vars: {', '.join(missing)}")
|
||||||
|
|
||||||
|
|
||||||
|
def get_upstream_kwargs() -> dict[str, Any]:
|
||||||
|
headers = {"Content-Type": "application/json"}
|
||||||
|
kwargs: dict[str, Any] = {"headers": headers}
|
||||||
|
if OPEN_API_LOGIN and OPEN_API_PASSWORD:
|
||||||
|
kwargs["auth"] = (OPEN_API_LOGIN, OPEN_API_PASSWORD)
|
||||||
|
return kwargs
|
||||||
|
if API_KEY:
|
||||||
|
headers["Authorization"] = f"Bearer {API_KEY}"
|
||||||
|
return kwargs
|
||||||
435
search/main.py
435
search/main.py
|
|
@ -1,3 +1,4 @@
|
||||||
|
import asyncio
|
||||||
import logging
|
import logging
|
||||||
import os
|
import os
|
||||||
from contextlib import asynccontextmanager
|
from contextlib import asynccontextmanager
|
||||||
|
|
@ -14,9 +15,8 @@ from qdrant_client import AsyncQdrantClient, models
|
||||||
|
|
||||||
EMBEDDINGS_DENSE_MODEL = "Qwen/Qwen3-Embedding-0.6B"
|
EMBEDDINGS_DENSE_MODEL = "Qwen/Qwen3-Embedding-0.6B"
|
||||||
|
|
||||||
# Ваш сервис должен считывать эти переменные из окружения (env), так как проверяющая система управляет ими
|
|
||||||
HOST = os.getenv("HOST", "0.0.0.0")
|
HOST = os.getenv("HOST", "0.0.0.0")
|
||||||
PORT = int(os.getenv("PORT", "8003"))
|
PORT = int(os.getenv("PORT", "8000"))
|
||||||
|
|
||||||
API_KEY = os.getenv("API_KEY")
|
API_KEY = os.getenv("API_KEY")
|
||||||
EMBEDDINGS_DENSE_URL = os.getenv("EMBEDDINGS_DENSE_URL")
|
EMBEDDINGS_DENSE_URL = os.getenv("EMBEDDINGS_DENSE_URL")
|
||||||
|
|
@ -73,7 +73,6 @@ def get_upstream_request_kwargs() -> dict[str, Any]:
|
||||||
return kwargs
|
return kwargs
|
||||||
|
|
||||||
|
|
||||||
# Модель данных, которую мы предоставляем и рассчитываем получать от вас
|
|
||||||
class DateRange(BaseModel):
|
class DateRange(BaseModel):
|
||||||
from_: str = Field(alias="from")
|
from_: str = Field(alias="from")
|
||||||
to: str
|
to: str
|
||||||
|
|
@ -126,13 +125,9 @@ class SparseVector(BaseModel):
|
||||||
values: list[float] = Field(default_factory=list)
|
values: list[float] = Field(default_factory=list)
|
||||||
|
|
||||||
|
|
||||||
class SparseEmbeddingResponse(BaseModel):
|
|
||||||
vectors: list[SparseVector]
|
|
||||||
|
|
||||||
# Метадата чанков в Qdrant'e, по которой вы можете фильтровать
|
|
||||||
class ChunkMetadata(BaseModel):
|
class ChunkMetadata(BaseModel):
|
||||||
chat_name: str
|
chat_name: str
|
||||||
chat_type: str # channel, group, private, thread
|
chat_type: str
|
||||||
chat_id: str
|
chat_id: str
|
||||||
chat_sn: str
|
chat_sn: str
|
||||||
thread_sn: str | None = None
|
thread_sn: str | None = None
|
||||||
|
|
@ -167,18 +162,14 @@ async def lifespan(app: FastAPI):
|
||||||
|
|
||||||
app = FastAPI(title="Search Service", version="0.1.0", lifespan=lifespan)
|
app = FastAPI(title="Search Service", version="0.1.0", lifespan=lifespan)
|
||||||
|
|
||||||
|
DENSE_PREFETCH_K = 120
|
||||||
|
SPARSE_PREFETCH_K = 200
|
||||||
|
RETRIEVE_K = 150
|
||||||
|
RERANK_LIMIT = 35
|
||||||
|
KEYWORD_BOOST_EXTRA = 10
|
||||||
|
|
||||||
# Внутри шаблона dense и rerank берутся из внешних HTTP endpoint'ов,
|
|
||||||
# которые предоставляет проверяющая система.
|
|
||||||
# Текущий код ниже — минимальный пример search pipeline.
|
|
||||||
DENSE_PREFETCH_K = 10
|
|
||||||
SPRASE_PREFETCH_K = 30
|
|
||||||
RETRIEVE_K = 20
|
|
||||||
RERANK_LIMIT = 10
|
|
||||||
FINAL_TOP_K = 50
|
|
||||||
|
|
||||||
async def embed_dense(client: httpx.AsyncClient, text: str) -> list[float]:
|
async def embed_dense(client: httpx.AsyncClient, text: str) -> list[float]:
|
||||||
# Dense endpoint ожидает OpenAI-compatible body с input как списком строк.
|
|
||||||
response = await client.post(
|
response = await client.post(
|
||||||
EMBEDDINGS_DENSE_URL,
|
EMBEDDINGS_DENSE_URL,
|
||||||
**get_upstream_request_kwargs(),
|
**get_upstream_request_kwargs(),
|
||||||
|
|
@ -188,127 +179,125 @@ async def embed_dense(client: httpx.AsyncClient, text: str) -> list[float]:
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
response.raise_for_status()
|
response.raise_for_status()
|
||||||
|
|
||||||
payload = DenseEmbeddingResponse.model_validate(response.json())
|
payload = DenseEmbeddingResponse.model_validate(response.json())
|
||||||
if not payload.data:
|
if not payload.data:
|
||||||
raise ValueError("Dense embedding response is empty")
|
raise ValueError("Dense embedding response is empty")
|
||||||
|
|
||||||
return payload.data[0].embedding
|
return payload.data[0].embedding
|
||||||
|
|
||||||
|
|
||||||
async def embed_sparse(text: str) -> SparseVector:
|
async def embed_dense_batch(client: httpx.AsyncClient, texts: list[str]) -> list[list[float]]:
|
||||||
|
response = await client.post(
|
||||||
|
EMBEDDINGS_DENSE_URL,
|
||||||
|
**get_upstream_request_kwargs(),
|
||||||
|
json={
|
||||||
|
"model": os.getenv("EMBEDDINGS_DENSE_MODEL", EMBEDDINGS_DENSE_MODEL),
|
||||||
|
"input": texts,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
response.raise_for_status()
|
||||||
|
payload = DenseEmbeddingResponse.model_validate(response.json())
|
||||||
|
payload.data.sort(key=lambda x: x.index)
|
||||||
|
return [item.embedding for item in payload.data]
|
||||||
|
|
||||||
|
|
||||||
|
def embed_sparse_sync(text: str) -> SparseVector:
|
||||||
vectors = list(get_sparse_model().embed([text]))
|
vectors = list(get_sparse_model().embed([text]))
|
||||||
if not vectors:
|
if not vectors:
|
||||||
raise ValueError("Sparse embedding response is empty")
|
raise ValueError("Sparse embedding response is empty")
|
||||||
|
|
||||||
item = vectors[0]
|
item = vectors[0]
|
||||||
return SparseVector(
|
return SparseVector(
|
||||||
indices=[int(index) for index in item.indices.tolist()],
|
indices=[int(index) for index in item.indices.tolist()],
|
||||||
values=[float(value) for value in item.values.tolist()],
|
values=[float(value) for value in item.values.tolist()],
|
||||||
)
|
)
|
||||||
|
|
||||||
# ПЕРЕПИСАТЬ
|
|
||||||
|
def build_dense_query(question: Question) -> str:
|
||||||
|
q = question.search_text.strip() if question.search_text else question.text.strip()
|
||||||
|
return q
|
||||||
|
|
||||||
|
|
||||||
|
def build_sparse_query(question: Question) -> str:
|
||||||
|
base = question.search_text.strip() if question.search_text else question.text.strip()
|
||||||
|
parts = [base]
|
||||||
|
if question.keywords:
|
||||||
|
parts.extend(question.keywords)
|
||||||
|
return " ".join(parts)
|
||||||
|
|
||||||
|
|
||||||
|
def _build_keyword_set(question: Question) -> list[str]:
|
||||||
|
tokens: list[str] = []
|
||||||
|
if question.keywords:
|
||||||
|
tokens.extend(kw.lower() for kw in question.keywords if kw)
|
||||||
|
if question.entities:
|
||||||
|
for field in (
|
||||||
|
question.entities.people,
|
||||||
|
question.entities.emails,
|
||||||
|
question.entities.documents,
|
||||||
|
question.entities.names,
|
||||||
|
question.entities.links,
|
||||||
|
):
|
||||||
|
tokens.extend(e.lower() for e in (field or []) if e)
|
||||||
|
return tokens
|
||||||
|
|
||||||
|
|
||||||
|
def prefilter_for_rerank(
|
||||||
|
points: list[Any],
|
||||||
|
question: Question,
|
||||||
|
) -> tuple[list[Any], list[Any]]:
|
||||||
|
"""Select candidates for reranking: top by RRF + keyword-boosted stragglers."""
|
||||||
|
if not points:
|
||||||
|
return [], []
|
||||||
|
|
||||||
|
head = points[:RERANK_LIMIT]
|
||||||
|
tail = points[RERANK_LIMIT:]
|
||||||
|
|
||||||
|
keywords = _build_keyword_set(question)
|
||||||
|
if not keywords or not tail:
|
||||||
|
return head, tail
|
||||||
|
|
||||||
|
extra: list[Any] = []
|
||||||
|
remaining_tail: list[Any] = []
|
||||||
|
for p in tail:
|
||||||
|
if len(extra) >= KEYWORD_BOOST_EXTRA:
|
||||||
|
remaining_tail.append(p)
|
||||||
|
continue
|
||||||
|
content = ((p.payload or {}).get("page_content") or "").lower()
|
||||||
|
if any(kw in content for kw in keywords):
|
||||||
|
extra.append(p)
|
||||||
|
else:
|
||||||
|
remaining_tail.append(p)
|
||||||
|
|
||||||
|
return head + extra, remaining_tail
|
||||||
|
|
||||||
|
|
||||||
async def qdrant_search(
|
async def qdrant_search(
|
||||||
client: AsyncQdrantClient,
|
client: AsyncQdrantClient,
|
||||||
dense_vector: list[float],
|
dense_vectors: list[list[float]],
|
||||||
sparse_vector: SparseVector,
|
sparse_vector: SparseVector,
|
||||||
question_data: Question
|
) -> list[Any] | None:
|
||||||
) -> Any | None:
|
prefetch_list = []
|
||||||
must_conditions: []
|
for dv in dense_vectors:
|
||||||
|
prefetch_list.append(
|
||||||
# Фильтр по диапазону дат (поле metadata.start в Qdrant) [cite: 147, 148, 175]
|
|
||||||
if question_data.date_range:
|
|
||||||
must_conditions.append(
|
|
||||||
models.FieldCondition(
|
|
||||||
key="metadata.start",
|
|
||||||
range=models.Range(
|
|
||||||
gte=question_data.date_range.from_,
|
|
||||||
lte=question_data.date_range.to_
|
|
||||||
)
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
# Фильтр по автору вопроса (поле metadata.participants) [cite: 161, 163]
|
|
||||||
if question_data.asker:
|
|
||||||
must_conditions.append(
|
|
||||||
models.FieldCondition(
|
|
||||||
key="metadata.participants",
|
|
||||||
match=models.MatchValue(value=question_data.asker)
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
# Создаем итоговый объект фильтра, если есть условия
|
|
||||||
search_filter = models.Filter(must=must_conditions) if must_conditions else None
|
|
||||||
|
|
||||||
response = await client.query_points(
|
|
||||||
collection_name=QDRANT_COLLECTION_NAME,
|
|
||||||
prefetch=[
|
|
||||||
models.Prefetch(
|
models.Prefetch(
|
||||||
query=dense_vector,
|
query=dv,
|
||||||
using=QDRANT_DENSE_VECTOR_NAME,
|
using=QDRANT_DENSE_VECTOR_NAME,
|
||||||
limit=DENSE_PREFETCH_K,
|
limit=DENSE_PREFETCH_K,
|
||||||
filter=search_filter,
|
)
|
||||||
),
|
)
|
||||||
|
prefetch_list.append(
|
||||||
models.Prefetch(
|
models.Prefetch(
|
||||||
query=models.SparseVector(
|
query=models.SparseVector(
|
||||||
indices=sparse_vector.indices,
|
indices=sparse_vector.indices,
|
||||||
values=sparse_vector.values,
|
values=sparse_vector.values,
|
||||||
),
|
),
|
||||||
using=QDRANT_SPARSE_VECTOR_NAME,
|
using=QDRANT_SPARSE_VECTOR_NAME,
|
||||||
limit=SPRASE_PREFETCH_K,
|
limit=SPARSE_PREFETCH_K,
|
||||||
filter=search_filter,
|
|
||||||
),
|
|
||||||
],
|
|
||||||
query=models.FusionQuery(fusion=models.Fusion.RRF),
|
|
||||||
limit=RETRIEVE_K,
|
|
||||||
with_payload=True,
|
|
||||||
)
|
|
||||||
|
|
||||||
if not response.points:
|
|
||||||
return None
|
|
||||||
|
|
||||||
return response.points
|
|
||||||
|
|
||||||
|
|
||||||
async def qdrant_search_dense_only(
|
|
||||||
client: AsyncQdrantClient,
|
|
||||||
dense_vector: list[float],
|
|
||||||
question_data: Question,
|
|
||||||
) -> Any | None:
|
|
||||||
must_conditions: []
|
|
||||||
|
|
||||||
if question_data.date_range:
|
|
||||||
must_conditions.append(
|
|
||||||
models.FieldCondition(
|
|
||||||
key="metadata.start",
|
|
||||||
range=models.Range(
|
|
||||||
gte=question_data.date_range.from_,
|
|
||||||
lte=question_data.date_range.to_,
|
|
||||||
),
|
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
if question_data.asker:
|
|
||||||
must_conditions.append(
|
|
||||||
models.FieldCondition(
|
|
||||||
key="metadata.participants",
|
|
||||||
match=models.MatchValue(value=question_data.asker),
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
search_filter = models.Filter(must=must_conditions) if must_conditions else None
|
|
||||||
|
|
||||||
response = await client.query_points(
|
response = await client.query_points(
|
||||||
collection_name=QDRANT_COLLECTION_NAME,
|
collection_name=QDRANT_COLLECTION_NAME,
|
||||||
prefetch=[
|
prefetch=prefetch_list,
|
||||||
models.Prefetch(
|
|
||||||
query=dense_vector,
|
|
||||||
using=QDRANT_DENSE_VECTOR_NAME,
|
|
||||||
limit=DENSE_PREFETCH_K,
|
|
||||||
filter=search_filter,
|
|
||||||
),
|
|
||||||
],
|
|
||||||
query=models.FusionQuery(fusion=models.Fusion.RRF),
|
query=models.FusionQuery(fusion=models.Fusion.RRF),
|
||||||
limit=RETRIEVE_K,
|
limit=RETRIEVE_K,
|
||||||
with_payload=True,
|
with_payload=True,
|
||||||
|
|
@ -320,93 +309,10 @@ async def qdrant_search_dense_only(
|
||||||
return response.points
|
return response.points
|
||||||
|
|
||||||
|
|
||||||
def collect_query_variants(question: Question) -> list[str]:
|
|
||||||
variants: list[str] = []
|
|
||||||
seen: set[str] = set()
|
|
||||||
|
|
||||||
def add_query(text: str | None) -> None:
|
|
||||||
if text is None:
|
|
||||||
return
|
|
||||||
normalized = text.strip()
|
|
||||||
if not normalized:
|
|
||||||
return
|
|
||||||
if normalized in seen:
|
|
||||||
return
|
|
||||||
seen.add(normalized)
|
|
||||||
variants.append(normalized)
|
|
||||||
|
|
||||||
add_query(question.search_text)
|
|
||||||
add_query(question.text)
|
|
||||||
|
|
||||||
for variant in question.variants or []:
|
|
||||||
add_query(variant)
|
|
||||||
|
|
||||||
return variants
|
|
||||||
|
|
||||||
|
|
||||||
def collect_hyde_queries(question: Question, base_queries: list[str]) -> list[str]:
|
|
||||||
hyde_queries: list[str] = []
|
|
||||||
seen: set[str] = set(base_queries)
|
|
||||||
|
|
||||||
for hyde_query in question.hyde or []:
|
|
||||||
normalized = hyde_query.strip()
|
|
||||||
if not normalized:
|
|
||||||
continue
|
|
||||||
if normalized in seen:
|
|
||||||
continue
|
|
||||||
seen.add(normalized)
|
|
||||||
hyde_queries.append(normalized)
|
|
||||||
|
|
||||||
return hyde_queries
|
|
||||||
|
|
||||||
|
|
||||||
def build_sparse_query_text(question: Question, fallback_query: str) -> str:
|
|
||||||
keywords: list[str] = []
|
|
||||||
seen: set[str] = set()
|
|
||||||
|
|
||||||
for keyword in question.keywords or []:
|
|
||||||
normalized = keyword.strip()
|
|
||||||
if not normalized:
|
|
||||||
continue
|
|
||||||
if normalized in seen:
|
|
||||||
continue
|
|
||||||
seen.add(normalized)
|
|
||||||
keywords.append(normalized)
|
|
||||||
|
|
||||||
if keywords:
|
|
||||||
return " ".join(keywords)
|
|
||||||
|
|
||||||
return fallback_query
|
|
||||||
|
|
||||||
|
|
||||||
def deduplicate_points(points: list[Any]) -> list[Any]:
|
|
||||||
unique_points: list[Any] = []
|
|
||||||
seen_ids: set[str] = set()
|
|
||||||
|
|
||||||
for point in points:
|
|
||||||
point_id = str(getattr(point, "id", ""))
|
|
||||||
if not point_id:
|
|
||||||
continue
|
|
||||||
if point_id in seen_ids:
|
|
||||||
continue
|
|
||||||
seen_ids.add(point_id)
|
|
||||||
unique_points.append(point)
|
|
||||||
|
|
||||||
return unique_points
|
|
||||||
|
|
||||||
|
|
||||||
def extract_point_score(point: Any) -> float:
|
|
||||||
score = getattr(point, "score", 0.0)
|
|
||||||
if score is None:
|
|
||||||
return 0.0
|
|
||||||
return float(score)
|
|
||||||
|
|
||||||
|
|
||||||
def extract_message_ids(point: Any) -> list[str]:
|
def extract_message_ids(point: Any) -> list[str]:
|
||||||
payload = point.payload or {}
|
payload = point.payload or {}
|
||||||
metadata = payload.get("metadata") or {}
|
metadata = payload.get("metadata") or {}
|
||||||
message_ids = metadata.get("message_ids") or []
|
message_ids = metadata.get("message_ids") or []
|
||||||
|
|
||||||
return [str(message_id) for message_id in message_ids]
|
return [str(message_id) for message_id in message_ids]
|
||||||
|
|
||||||
|
|
||||||
|
|
@ -418,7 +324,8 @@ async def get_rerank_scores(
|
||||||
if not targets:
|
if not targets:
|
||||||
return []
|
return []
|
||||||
|
|
||||||
# Rerank endpoint возвращает score для пары query -> candidate text.
|
for attempt in range(5):
|
||||||
|
try:
|
||||||
response = await client.post(
|
response = await client.post(
|
||||||
RERANKER_URL,
|
RERANKER_URL,
|
||||||
**get_upstream_request_kwargs(),
|
**get_upstream_request_kwargs(),
|
||||||
|
|
@ -429,64 +336,51 @@ async def get_rerank_scores(
|
||||||
"text_2": targets,
|
"text_2": targets,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
if response.status_code == 429:
|
||||||
|
wait = 2 ** attempt
|
||||||
|
logger.warning(f"Rerank 429, retry {attempt+1}/5 in {wait}s")
|
||||||
|
await asyncio.sleep(wait)
|
||||||
|
continue
|
||||||
response.raise_for_status()
|
response.raise_for_status()
|
||||||
|
|
||||||
payload = response.json()
|
payload = response.json()
|
||||||
data = payload.get("data") or []
|
data = payload.get("data") or []
|
||||||
|
|
||||||
return [float(sample["score"]) for sample in data]
|
return [float(sample["score"]) for sample in data]
|
||||||
|
except Exception as e:
|
||||||
|
logger.warning(f"Rerank error attempt {attempt+1}/5: {e}")
|
||||||
|
if attempt < 4:
|
||||||
|
await asyncio.sleep(2 ** attempt)
|
||||||
|
continue
|
||||||
|
logger.error("Rerank failed after 5 attempts, using fallback")
|
||||||
|
return []
|
||||||
|
|
||||||
|
logger.error("Rerank 429 after 5 retries, using fallback")
|
||||||
|
return []
|
||||||
|
|
||||||
|
|
||||||
async def rerank_points(
|
async def rerank_points(
|
||||||
client: httpx.AsyncClient,
|
client: httpx.AsyncClient,
|
||||||
query: str,
|
query: str,
|
||||||
points: list[Any],
|
points: list[Any],
|
||||||
) -> list[tuple[Any, float]]:
|
) -> list[Any]:
|
||||||
rerank_candidates = points[:RERANK_LIMIT]
|
if not points:
|
||||||
tail_candidates = points[RERANK_LIMIT:]
|
return []
|
||||||
rerank_targets = [point.payload.get("page_content") for point in rerank_candidates]
|
targets = [point.payload.get("page_content") for point in points]
|
||||||
scores = await get_rerank_scores(client, query, rerank_targets)
|
scores = await get_rerank_scores(client, query, targets)
|
||||||
|
|
||||||
reranked_candidates = [
|
if not scores or len(scores) != len(points):
|
||||||
(point, float(score))
|
logger.warning("Reranker unavailable or score mismatch, returning RRF order")
|
||||||
for score, point in sorted(
|
return points
|
||||||
zip(scores, rerank_candidates, strict=True),
|
|
||||||
key=lambda item: item[0],
|
return [
|
||||||
reverse=True,
|
point
|
||||||
)
|
|
||||||
]
|
|
||||||
tail_with_scores = [
|
|
||||||
(point, extract_point_score(point))
|
|
||||||
for _, point in sorted(
|
for _, point in sorted(
|
||||||
[(extract_point_score(point), point) for point in tail_candidates],
|
zip(scores, points),
|
||||||
key=lambda item: item[0],
|
key=lambda item: item[0],
|
||||||
reverse=True,
|
reverse=True,
|
||||||
)
|
)
|
||||||
]
|
]
|
||||||
|
|
||||||
return reranked_candidates + tail_with_scores
|
|
||||||
|
|
||||||
|
|
||||||
def aggregate_message_scores(scored_points: list[tuple[Any, float]]) -> dict[str, float]:
|
|
||||||
aggregated_scores: dict[str, float] = {}
|
|
||||||
|
|
||||||
for point, point_score in scored_points:
|
|
||||||
point_message_ids = set(extract_message_ids(point))
|
|
||||||
for message_id in point_message_ids:
|
|
||||||
aggregated_scores[message_id] = aggregated_scores.get(message_id, 0.0) + point_score
|
|
||||||
|
|
||||||
return aggregated_scores
|
|
||||||
|
|
||||||
|
|
||||||
def select_top_message_ids(aggregated_scores: dict[str, float], limit: int) -> list[str]:
|
|
||||||
sorted_items = sorted(
|
|
||||||
aggregated_scores.items(),
|
|
||||||
key=lambda item: (-item[1], item[0]),
|
|
||||||
)
|
|
||||||
return [message_id for message_id, _ in sorted_items[:limit]]
|
|
||||||
|
|
||||||
|
|
||||||
# Ваш сервис должен имплементировать оба этих метода
|
|
||||||
@app.get("/health")
|
@app.get("/health")
|
||||||
async def health() -> dict[str, str]:
|
async def health() -> dict[str, str]:
|
||||||
return {"status": "ok"}
|
return {"status": "ok"}
|
||||||
|
|
@ -494,41 +388,65 @@ async def health() -> dict[str, str]:
|
||||||
|
|
||||||
@app.post("/search", response_model=SearchAPIResponse)
|
@app.post("/search", response_model=SearchAPIResponse)
|
||||||
async def search(payload: SearchAPIRequest) -> SearchAPIResponse:
|
async def search(payload: SearchAPIRequest) -> SearchAPIResponse:
|
||||||
queries = collect_query_variants(payload.question)
|
question = payload.question
|
||||||
if not queries:
|
query = question.text.strip()
|
||||||
raise HTTPException(status_code=400, detail="question.search_text or question.text is required")
|
if not query:
|
||||||
|
raise HTTPException(status_code=400, detail="question.text is required")
|
||||||
|
|
||||||
hyde_queries = collect_hyde_queries(payload.question, queries)
|
|
||||||
query = queries[0]
|
|
||||||
client: httpx.AsyncClient = app.state.http
|
client: httpx.AsyncClient = app.state.http
|
||||||
qdrant: AsyncQdrantClient = app.state.qdrant
|
qdrant: AsyncQdrantClient = app.state.qdrant
|
||||||
|
|
||||||
all_points: list[Any] = []
|
dense_query = build_dense_query(question)
|
||||||
for query_variant in queries:
|
sparse_query = build_sparse_query(question)
|
||||||
dense_vector = await embed_dense(client, query_variant)
|
|
||||||
sparse_query_text = build_sparse_query_text(payload.question, query_variant)
|
|
||||||
sparse_vector = await embed_sparse(sparse_query_text)
|
|
||||||
points = await qdrant_search(qdrant, dense_vector, sparse_vector, payload.question)
|
|
||||||
if points:
|
|
||||||
all_points.extend(list(points))
|
|
||||||
|
|
||||||
for hyde_query in hyde_queries:
|
dense_task = embed_dense(client, dense_query)
|
||||||
hyde_dense_vector = await embed_dense(client, hyde_query)
|
sparse_task = asyncio.to_thread(lambda: embed_sparse_sync(sparse_query))
|
||||||
hyde_points = await qdrant_search_dense_only(qdrant, hyde_dense_vector, payload.question)
|
dense_vector, sparse_vector = await asyncio.gather(dense_task, sparse_task)
|
||||||
if hyde_points:
|
|
||||||
all_points.extend(list(hyde_points))
|
|
||||||
|
|
||||||
best_points = deduplicate_points(all_points)
|
dense_vectors = [dense_vector]
|
||||||
if not best_points:
|
extra_texts: list[str] = []
|
||||||
|
raw_text = question.text.strip()
|
||||||
|
if raw_text and raw_text != dense_query:
|
||||||
|
extra_texts.append(raw_text)
|
||||||
|
for v in (question.variants or []):
|
||||||
|
q_v = v.strip()
|
||||||
|
if q_v and q_v != dense_query and q_v not in extra_texts:
|
||||||
|
extra_texts.append(q_v)
|
||||||
|
for h in (question.hyde or []):
|
||||||
|
q_h = h.strip()
|
||||||
|
if q_h and q_h != dense_query and q_h not in extra_texts:
|
||||||
|
extra_texts.append(q_h)
|
||||||
|
extra_texts = extra_texts[:3]
|
||||||
|
if extra_texts:
|
||||||
|
try:
|
||||||
|
extra_vecs = await embed_dense_batch(client, extra_texts)
|
||||||
|
dense_vectors.extend(extra_vecs)
|
||||||
|
except Exception as e:
|
||||||
|
logger.warning(f"Extra dense embedding failed: {e}")
|
||||||
|
|
||||||
|
all_points = await qdrant_search(qdrant, dense_vectors, sparse_vector)
|
||||||
|
|
||||||
|
if all_points is None:
|
||||||
return SearchAPIResponse(results=[])
|
return SearchAPIResponse(results=[])
|
||||||
|
|
||||||
scored_points = await rerank_points(client, query, list(best_points))
|
all_points = list(all_points)
|
||||||
aggregated_scores = aggregate_message_scores(scored_points)
|
|
||||||
message_ids = select_top_message_ids(aggregated_scores, FINAL_TOP_K)
|
|
||||||
|
|
||||||
return SearchAPIResponse(
|
rerank_pool, rerank_tail = prefilter_for_rerank(all_points, question)
|
||||||
results=[SearchAPIItem(message_ids=message_ids)]
|
reranked = await rerank_points(client, query, rerank_pool)
|
||||||
)
|
final_points = reranked + rerank_tail
|
||||||
|
|
||||||
|
msg_score: dict[str, float] = {}
|
||||||
|
for rank, point in enumerate(reranked):
|
||||||
|
score = 1.0 / (rank + 1)
|
||||||
|
for mid in extract_message_ids(point):
|
||||||
|
msg_score[mid] = msg_score.get(mid, 0.0) + score
|
||||||
|
for rank, point in enumerate(rerank_tail):
|
||||||
|
score = 1.0 / (60 + rank + 1)
|
||||||
|
for mid in extract_message_ids(point):
|
||||||
|
msg_score[mid] = msg_score.get(mid, 0.0) + score
|
||||||
|
message_ids = sorted(msg_score, key=lambda m: msg_score[m], reverse=True)[:50]
|
||||||
|
|
||||||
|
return SearchAPIResponse(results=[SearchAPIItem(message_ids=message_ids)])
|
||||||
|
|
||||||
|
|
||||||
@app.exception_handler(Exception)
|
@app.exception_handler(Exception)
|
||||||
|
|
@ -548,12 +466,7 @@ async def exception_handler(request: Request, exc: Exception) -> JSONResponse:
|
||||||
def main() -> None:
|
def main() -> None:
|
||||||
import uvicorn
|
import uvicorn
|
||||||
|
|
||||||
uvicorn.run(
|
uvicorn.run("main:app", host=HOST, port=PORT, reload=False)
|
||||||
"main:app",
|
|
||||||
host=HOST,
|
|
||||||
port=PORT,
|
|
||||||
reload=False,
|
|
||||||
)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
|
|
|
||||||
109
search/query_builder.py
Normal file
109
search/query_builder.py
Normal file
|
|
@ -0,0 +1,109 @@
|
||||||
|
import os
|
||||||
|
import re
|
||||||
|
from functools import lru_cache
|
||||||
|
|
||||||
|
import httpx
|
||||||
|
from fastembed import SparseTextEmbedding
|
||||||
|
|
||||||
|
from config import (
|
||||||
|
EMBEDDINGS_DENSE_MODEL,
|
||||||
|
EMBEDDINGS_DENSE_URL,
|
||||||
|
SPARSE_MODEL_NAME,
|
||||||
|
get_upstream_kwargs,
|
||||||
|
logger,
|
||||||
|
)
|
||||||
|
from schemas import DenseEmbeddingResponse, Question, SparseVector
|
||||||
|
|
||||||
|
|
||||||
|
@lru_cache(maxsize=1)
|
||||||
|
def get_sparse_model() -> SparseTextEmbedding:
|
||||||
|
logger.info("Loading local sparse model %s", SPARSE_MODEL_NAME)
|
||||||
|
return SparseTextEmbedding(model_name=SPARSE_MODEL_NAME)
|
||||||
|
|
||||||
|
|
||||||
|
async def embed_dense(client: httpx.AsyncClient, text: str) -> list[float]:
|
||||||
|
response = await client.post(
|
||||||
|
str(EMBEDDINGS_DENSE_URL),
|
||||||
|
**get_upstream_kwargs(),
|
||||||
|
json={
|
||||||
|
"model": os.getenv("EMBEDDINGS_DENSE_MODEL", EMBEDDINGS_DENSE_MODEL),
|
||||||
|
"input": [text],
|
||||||
|
},
|
||||||
|
)
|
||||||
|
response.raise_for_status()
|
||||||
|
payload = DenseEmbeddingResponse.model_validate(response.json())
|
||||||
|
if not payload.data:
|
||||||
|
raise ValueError("Dense embedding response is empty")
|
||||||
|
return payload.data[0].embedding
|
||||||
|
|
||||||
|
|
||||||
|
async def embed_dense_batch(client: httpx.AsyncClient, texts: list[str]) -> list[list[float]]:
|
||||||
|
"""Single request for multiple texts — avoids N parallel calls and rate limiting."""
|
||||||
|
response = await client.post(
|
||||||
|
str(EMBEDDINGS_DENSE_URL),
|
||||||
|
**get_upstream_kwargs(),
|
||||||
|
json={
|
||||||
|
"model": os.getenv("EMBEDDINGS_DENSE_MODEL", EMBEDDINGS_DENSE_MODEL),
|
||||||
|
"input": texts,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
response.raise_for_status()
|
||||||
|
payload = DenseEmbeddingResponse.model_validate(response.json())
|
||||||
|
payload.data.sort(key=lambda x: x.index)
|
||||||
|
return [item.embedding for item in payload.data]
|
||||||
|
|
||||||
|
|
||||||
|
def embed_sparse(text: str) -> SparseVector:
|
||||||
|
vectors = list(get_sparse_model().embed([text]))
|
||||||
|
if not vectors:
|
||||||
|
raise ValueError("Sparse embedding response is empty")
|
||||||
|
item = vectors[0]
|
||||||
|
return SparseVector(
|
||||||
|
indices=[int(i) for i in item.indices.tolist()],
|
||||||
|
values=[float(v) for v in item.values.tolist()],
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def _normalize_query(text: str) -> str:
|
||||||
|
return re.sub(r"\s+", " ", text).strip()
|
||||||
|
|
||||||
|
|
||||||
|
def build_primary_query(question: Question) -> str:
|
||||||
|
q = question.search_text.strip() if question.search_text else ""
|
||||||
|
if not q:
|
||||||
|
q = question.text.strip()
|
||||||
|
return _normalize_query(q)
|
||||||
|
|
||||||
|
|
||||||
|
def build_extra_dense_queries(question: Question) -> list[str]:
|
||||||
|
extras: list[str] = []
|
||||||
|
for v in question.variants or []:
|
||||||
|
q = _normalize_query(v)
|
||||||
|
if q:
|
||||||
|
extras.append(q)
|
||||||
|
for h in question.hyde or []:
|
||||||
|
q = _normalize_query(h)
|
||||||
|
if q:
|
||||||
|
extras.append(q)
|
||||||
|
return extras
|
||||||
|
|
||||||
|
|
||||||
|
def build_sparse_query(question: Question) -> str:
|
||||||
|
kws = question.keywords or []
|
||||||
|
if kws:
|
||||||
|
return " ".join(kws)
|
||||||
|
return build_primary_query(question)
|
||||||
|
|
||||||
|
|
||||||
|
def build_entity_tokens(question: Question) -> list[str]:
|
||||||
|
tokens: list[str] = []
|
||||||
|
if question.entities:
|
||||||
|
for field in (
|
||||||
|
question.entities.people,
|
||||||
|
question.entities.emails,
|
||||||
|
question.entities.documents,
|
||||||
|
question.entities.names,
|
||||||
|
question.entities.links,
|
||||||
|
):
|
||||||
|
tokens.extend(field or [])
|
||||||
|
return [t.strip() for t in tokens if t.strip()]
|
||||||
73
search/rerank.py
Normal file
73
search/rerank.py
Normal file
|
|
@ -0,0 +1,73 @@
|
||||||
|
import asyncio
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
import httpx
|
||||||
|
|
||||||
|
from config import RERANK_LIMIT, RERANKER_MODEL, RERANKER_URL, get_upstream_kwargs, logger
|
||||||
|
from retrieval import extract_page_content
|
||||||
|
|
||||||
|
|
||||||
|
async def _get_rerank_scores(
|
||||||
|
client: httpx.AsyncClient,
|
||||||
|
query: str,
|
||||||
|
targets: list[str],
|
||||||
|
) -> list[float]:
|
||||||
|
if not targets:
|
||||||
|
return []
|
||||||
|
|
||||||
|
for attempt in range(5):
|
||||||
|
try:
|
||||||
|
response = await client.post(
|
||||||
|
str(RERANKER_URL),
|
||||||
|
**get_upstream_kwargs(),
|
||||||
|
json={
|
||||||
|
"model": RERANKER_MODEL,
|
||||||
|
"encoding_format": "float",
|
||||||
|
"text_1": query,
|
||||||
|
"text_2": targets,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
except Exception as exc:
|
||||||
|
if attempt < 4:
|
||||||
|
await asyncio.sleep(2 ** attempt)
|
||||||
|
continue
|
||||||
|
raise exc
|
||||||
|
|
||||||
|
if response.status_code == 429:
|
||||||
|
wait = 2 ** attempt
|
||||||
|
logger.warning("Rerank 429, retry %d/5 in %ds", attempt + 1, wait)
|
||||||
|
await asyncio.sleep(wait)
|
||||||
|
continue
|
||||||
|
|
||||||
|
response.raise_for_status()
|
||||||
|
data = response.json().get("data") or []
|
||||||
|
return [float(sample["score"]) for sample in data]
|
||||||
|
|
||||||
|
logger.error("Rerank 429 after all retries, falling back")
|
||||||
|
return []
|
||||||
|
|
||||||
|
|
||||||
|
async def rerank_points(
|
||||||
|
client: httpx.AsyncClient,
|
||||||
|
query: str,
|
||||||
|
points: list[Any],
|
||||||
|
) -> tuple[list[Any], list[Any]]:
|
||||||
|
if not points:
|
||||||
|
return [], []
|
||||||
|
|
||||||
|
head = points[:RERANK_LIMIT]
|
||||||
|
tail = points[RERANK_LIMIT:]
|
||||||
|
targets = [extract_page_content(p) for p in head]
|
||||||
|
|
||||||
|
try:
|
||||||
|
scores = await _get_rerank_scores(client, query, targets)
|
||||||
|
except Exception as exc:
|
||||||
|
logger.warning("Rerank failed, using retrieval order: %s", exc)
|
||||||
|
return head, tail
|
||||||
|
|
||||||
|
if len(scores) != len(head):
|
||||||
|
logger.warning("Rerank score count mismatch, using retrieval order")
|
||||||
|
return head, tail
|
||||||
|
|
||||||
|
reranked = [p for _, p in sorted(zip(scores, head), key=lambda x: x[0], reverse=True)]
|
||||||
|
return reranked, tail
|
||||||
119
search/retrieval.py
Normal file
119
search/retrieval.py
Normal file
|
|
@ -0,0 +1,119 @@
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
from qdrant_client import AsyncQdrantClient, models
|
||||||
|
|
||||||
|
from config import (
|
||||||
|
DENSE_PREFETCH_K,
|
||||||
|
QDRANT_COLLECTION_NAME,
|
||||||
|
QDRANT_DENSE_VECTOR_NAME,
|
||||||
|
QDRANT_SPARSE_VECTOR_NAME,
|
||||||
|
RETRIEVE_K,
|
||||||
|
SPARSE_PREFETCH_K,
|
||||||
|
logger,
|
||||||
|
)
|
||||||
|
from schemas import Question, SparseVector
|
||||||
|
|
||||||
|
|
||||||
|
def _build_filter(question: Question) -> models.Filter | None:
|
||||||
|
must_conditions: list[models.Condition] = []
|
||||||
|
|
||||||
|
if question.date_range:
|
||||||
|
try:
|
||||||
|
must_conditions.append(
|
||||||
|
models.FieldCondition(
|
||||||
|
key="metadata.end",
|
||||||
|
range=models.Range(gte=question.date_range.from_),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
must_conditions.append(
|
||||||
|
models.FieldCondition(
|
||||||
|
key="metadata.start",
|
||||||
|
range=models.Range(lte=question.date_range.to),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
except Exception as e:
|
||||||
|
logger.warning("Date filter failed: %s", e)
|
||||||
|
|
||||||
|
if question.asker:
|
||||||
|
must_conditions.append(
|
||||||
|
models.FieldCondition(
|
||||||
|
key="metadata.participants",
|
||||||
|
match=models.MatchValue(value=question.asker),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
return models.Filter(must=must_conditions) if must_conditions else None
|
||||||
|
|
||||||
|
|
||||||
|
async def qdrant_search(
|
||||||
|
client: AsyncQdrantClient,
|
||||||
|
primary_dense: list[float],
|
||||||
|
extra_dense: list[list[float]],
|
||||||
|
sparse_vector: SparseVector,
|
||||||
|
question: Question,
|
||||||
|
) -> list[Any]:
|
||||||
|
search_filter = _build_filter(question)
|
||||||
|
|
||||||
|
prefetch: list[models.Prefetch] = []
|
||||||
|
|
||||||
|
# Primary dense
|
||||||
|
prefetch.append(
|
||||||
|
models.Prefetch(
|
||||||
|
query=primary_dense,
|
||||||
|
using=QDRANT_DENSE_VECTOR_NAME,
|
||||||
|
limit=DENSE_PREFETCH_K,
|
||||||
|
filter=search_filter,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
# Extra dense (variants / hyde) - smaller budget per query
|
||||||
|
extra_k = max(10, DENSE_PREFETCH_K // max(1, len(extra_dense)))
|
||||||
|
for vec in extra_dense:
|
||||||
|
prefetch.append(
|
||||||
|
models.Prefetch(
|
||||||
|
query=vec,
|
||||||
|
using=QDRANT_DENSE_VECTOR_NAME,
|
||||||
|
limit=extra_k,
|
||||||
|
filter=search_filter,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
# Sparse
|
||||||
|
prefetch.append(
|
||||||
|
models.Prefetch(
|
||||||
|
query=models.SparseVector(
|
||||||
|
indices=sparse_vector.indices,
|
||||||
|
values=sparse_vector.values,
|
||||||
|
),
|
||||||
|
using=QDRANT_SPARSE_VECTOR_NAME,
|
||||||
|
limit=SPARSE_PREFETCH_K,
|
||||||
|
filter=search_filter,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
response = await client.query_points(
|
||||||
|
collection_name=QDRANT_COLLECTION_NAME,
|
||||||
|
prefetch=prefetch,
|
||||||
|
query=models.FusionQuery(fusion=models.Fusion.RRF),
|
||||||
|
limit=RETRIEVE_K,
|
||||||
|
with_payload=True,
|
||||||
|
)
|
||||||
|
|
||||||
|
if not response.points:
|
||||||
|
logger.debug("Qdrant returned 0 points")
|
||||||
|
return []
|
||||||
|
|
||||||
|
logger.debug("Qdrant returned %d points", len(response.points))
|
||||||
|
return list(response.points)
|
||||||
|
|
||||||
|
|
||||||
|
def extract_message_ids(point: Any) -> list[str]:
|
||||||
|
payload = point.payload or {}
|
||||||
|
metadata = payload.get("metadata") or {}
|
||||||
|
message_ids = metadata.get("message_ids") or []
|
||||||
|
return [str(mid) for mid in message_ids]
|
||||||
|
|
||||||
|
|
||||||
|
def extract_page_content(point: Any) -> str:
|
||||||
|
payload = point.payload or {}
|
||||||
|
return payload.get("page_content") or ""
|
||||||
72
search/schemas.py
Normal file
72
search/schemas.py
Normal file
|
|
@ -0,0 +1,72 @@
|
||||||
|
from pydantic import BaseModel, Field
|
||||||
|
|
||||||
|
|
||||||
|
class DateRange(BaseModel):
|
||||||
|
from_: str = Field(alias="from")
|
||||||
|
to: str
|
||||||
|
|
||||||
|
|
||||||
|
class Entities(BaseModel):
|
||||||
|
people: list[str] | None = None
|
||||||
|
emails: list[str] | None = None
|
||||||
|
documents: list[str] | None = None
|
||||||
|
names: list[str] | None = None
|
||||||
|
links: list[str] | None = None
|
||||||
|
|
||||||
|
|
||||||
|
class Question(BaseModel):
|
||||||
|
text: str
|
||||||
|
asker: str = ""
|
||||||
|
asked_on: str = ""
|
||||||
|
variants: list[str] | None = None
|
||||||
|
hyde: list[str] | None = None
|
||||||
|
keywords: list[str] | None = None
|
||||||
|
entities: Entities | None = None
|
||||||
|
date_mentions: list[str] | None = None
|
||||||
|
date_range: DateRange | None = None
|
||||||
|
search_text: str = ""
|
||||||
|
|
||||||
|
|
||||||
|
class SearchAPIRequest(BaseModel):
|
||||||
|
question: Question
|
||||||
|
|
||||||
|
|
||||||
|
class SearchAPIItem(BaseModel):
|
||||||
|
message_ids: list[str]
|
||||||
|
|
||||||
|
|
||||||
|
class SearchAPIResponse(BaseModel):
|
||||||
|
results: list[SearchAPIItem]
|
||||||
|
|
||||||
|
|
||||||
|
class DenseEmbeddingItem(BaseModel):
|
||||||
|
index: int
|
||||||
|
embedding: list[float]
|
||||||
|
|
||||||
|
|
||||||
|
class DenseEmbeddingResponse(BaseModel):
|
||||||
|
data: list[DenseEmbeddingItem]
|
||||||
|
|
||||||
|
|
||||||
|
class SparseVector(BaseModel):
|
||||||
|
indices: list[int] = Field(default_factory=list)
|
||||||
|
values: list[float] = Field(default_factory=list)
|
||||||
|
|
||||||
|
|
||||||
|
class SparseEmbeddingResponse(BaseModel):
|
||||||
|
vectors: list[SparseVector]
|
||||||
|
|
||||||
|
|
||||||
|
class ChunkMetadata(BaseModel):
|
||||||
|
chat_name: str
|
||||||
|
chat_type: str
|
||||||
|
chat_id: str
|
||||||
|
chat_sn: str
|
||||||
|
thread_sn: str | None = None
|
||||||
|
message_ids: list[str]
|
||||||
|
start: str
|
||||||
|
end: str
|
||||||
|
participants: list[str] = Field(default_factory=list)
|
||||||
|
mentions: list[str] = Field(default_factory=list)
|
||||||
|
contains_forward: bool = False
|
||||||
|
contains_quote: bool = False
|
||||||
59
skill.md
Normal file
59
skill.md
Normal file
|
|
@ -0,0 +1,59 @@
|
||||||
|
# Skill: Automating Task Tracking and Git Push with Codex
|
||||||
|
|
||||||
|
## Overview
|
||||||
|
This skill involves using **OpenAI Codex** to automate the tracking of task changes, logging updates, and automatically pushing changes to a Git repository. It combines **Codex's ability to generate code** with the power of **Git automation** to keep track of development tasks, log updates, and push them to version control.
|
||||||
|
|
||||||
|
## Objectives
|
||||||
|
1. Track the status of development tasks (e.g., status, assignee, comments).
|
||||||
|
2. Log all changes to tasks in a local directory (`.ai_update/`).
|
||||||
|
3. Automatically commit and push changes to a Git repository (GitHub, GitLab, etc.).
|
||||||
|
4. Provide clear and structured task progress reports.
|
||||||
|
|
||||||
|
## Components
|
||||||
|
- **Codex System Prompt**: Codex will act as the task manager, processing task updates and tracking changes.
|
||||||
|
- **Log Files**: Changes will be saved as text files in `.ai_update/`.
|
||||||
|
- **Git Integration**: Each task update will be automatically committed and pushed to a Git repository.
|
||||||
|
|
||||||
|
### 1. Codex System Prompt
|
||||||
|
Codex needs a **system prompt** that instructs it to track tasks, store updates in `.ai_update/`, and commit them to Git. Here is the **system prompt** for Codex:
|
||||||
|
|
||||||
|
```python
|
||||||
|
"""
|
||||||
|
You are a highly capable task manager for a development team working on a software project. Your job is to:
|
||||||
|
1. Track the tasks and changes in the project, including assigning tasks to team members and tracking progress.
|
||||||
|
2. Log every change made to the tasks, including updates to the status, comments, and assigned team members.
|
||||||
|
3. Ensure that no task is missed and that progress is clearly reported.
|
||||||
|
4. Store all updates, status changes, and task comments in a folder named `.ai_update/` on the local disk.
|
||||||
|
5. Automatically commit and push changes to the Git repository (on GitHub, GitLab, or other Git platforms) whenever updates are made.
|
||||||
|
6. Provide clear and detailed reports on what has been done so far and what tasks remain.
|
||||||
|
|
||||||
|
The folder `.ai_update/` will contain a log file where every change to a task is recorded, along with:
|
||||||
|
- Task name
|
||||||
|
- Updated status (if changed)
|
||||||
|
- Assignee (if changed)
|
||||||
|
- Any new comments (with timestamps)
|
||||||
|
- A summary of task progress
|
||||||
|
|
||||||
|
For each change:
|
||||||
|
1. Save the task update to the `.ai_update/` folder as a new log file.
|
||||||
|
2. Commit the new log file with a descriptive commit message, such as "Updated task [task_name] status to [status]".
|
||||||
|
3. Push the changes to the remote Git repository, ensuring that the changes are tracked properly.
|
||||||
|
4. If no change occurs in a task, do not log or push.
|
||||||
|
|
||||||
|
Ensure that no steps are skipped in the task completion process, and when a task is marked as "completed", ensure that all relevant information has been logged and pushed.
|
||||||
|
|
||||||
|
Provide the following feedback format for every task:
|
||||||
|
1. Task name
|
||||||
|
2. Current status
|
||||||
|
3. Comments and updates
|
||||||
|
4. Task progress summary
|
||||||
|
|
||||||
|
The `.ai_update/` folder will serve as a log of your progress. You should always commit the updates to Git in a way that shows a clear history of the changes.
|
||||||
|
|
||||||
|
Example of the log entry in `.ai_update/` folder:
|
||||||
|
Task: [task_name]
|
||||||
|
- Status: [status]
|
||||||
|
- Assignee: [assignee_name]
|
||||||
|
- Comment: [comment]
|
||||||
|
- Timestamp: [timestamp]
|
||||||
|
"""
|
||||||
0
tests/__init__.py
Normal file
0
tests/__init__.py
Normal file
58
tests/test_aggregation.py
Normal file
58
tests/test_aggregation.py
Normal file
|
|
@ -0,0 +1,58 @@
|
||||||
|
"""Unit tests for search/aggregation.py"""
|
||||||
|
import sys
|
||||||
|
import os
|
||||||
|
|
||||||
|
_SEARCH_DIR = os.path.join(os.path.dirname(__file__), "..", "search")
|
||||||
|
sys.path.insert(0, _SEARCH_DIR)
|
||||||
|
|
||||||
|
os.environ.setdefault("EMBEDDINGS_DENSE_URL", "http://localhost/embed")
|
||||||
|
os.environ.setdefault("RERANKER_URL", "http://localhost/rerank")
|
||||||
|
os.environ.setdefault("QDRANT_URL", "http://localhost:6333")
|
||||||
|
os.environ.setdefault("API_KEY", "test-key")
|
||||||
|
|
||||||
|
from aggregation import aggregate_message_ids
|
||||||
|
from config import TOP_K
|
||||||
|
|
||||||
|
|
||||||
|
def _point(message_ids: list[str]):
|
||||||
|
"""Fake qdrant point with payload."""
|
||||||
|
class FakePoint:
|
||||||
|
payload = {"metadata": {"message_ids": message_ids}}
|
||||||
|
return FakePoint()
|
||||||
|
|
||||||
|
|
||||||
|
class TestAggregateMessageIds:
|
||||||
|
def test_empty_inputs(self):
|
||||||
|
result = aggregate_message_ids([], [])
|
||||||
|
assert result == []
|
||||||
|
|
||||||
|
def test_basic_dedup(self):
|
||||||
|
head = [_point(["m1", "m2"]), _point(["m2", "m3"])]
|
||||||
|
result = aggregate_message_ids(head, [])
|
||||||
|
assert result.count("m2") == 1
|
||||||
|
|
||||||
|
def test_head_before_tail(self):
|
||||||
|
head = [_point(["head_msg"])]
|
||||||
|
tail = [_point(["tail_msg"])]
|
||||||
|
result = aggregate_message_ids(head, tail)
|
||||||
|
assert result.index("head_msg") < result.index("tail_msg")
|
||||||
|
|
||||||
|
def test_top_k_limit(self):
|
||||||
|
# Create enough points to exceed TOP_K
|
||||||
|
points = [_point([f"m{i}"]) for i in range(TOP_K + 20)]
|
||||||
|
result = aggregate_message_ids(points, [])
|
||||||
|
assert len(result) <= TOP_K
|
||||||
|
|
||||||
|
def test_cross_point_dedup(self):
|
||||||
|
head = [_point(["shared"]), _point(["shared", "unique"])]
|
||||||
|
result = aggregate_message_ids(head, [])
|
||||||
|
assert result.count("shared") == 1
|
||||||
|
assert "unique" in result
|
||||||
|
|
||||||
|
def test_tail_fills_after_head(self):
|
||||||
|
head = [_point(["h1"])]
|
||||||
|
tail = [_point(["t1"]), _point(["t2"])]
|
||||||
|
result = aggregate_message_ids(head, tail)
|
||||||
|
assert "h1" in result
|
||||||
|
assert "t1" in result
|
||||||
|
assert "t2" in result
|
||||||
125
tests/test_chunking.py
Normal file
125
tests/test_chunking.py
Normal file
|
|
@ -0,0 +1,125 @@
|
||||||
|
"""Unit tests for index/chunking.py"""
|
||||||
|
import sys
|
||||||
|
import os
|
||||||
|
|
||||||
|
_INDEX_DIR = os.path.join(os.path.dirname(__file__), "..", "index")
|
||||||
|
sys.path.insert(0, _INDEX_DIR)
|
||||||
|
|
||||||
|
from chunking import build_chunks, _split_windows, WINDOW_MAX_MESSAGES, TIME_GAP_SECONDS
|
||||||
|
from cleaning import CleanedMessage
|
||||||
|
from index_schemas import Message
|
||||||
|
|
||||||
|
|
||||||
|
def _make_message(id: str, time: int, text: str = "hello", **kwargs) -> Message:
|
||||||
|
defaults = dict(
|
||||||
|
thread_sn=None,
|
||||||
|
sender_id="user@example.com",
|
||||||
|
file_snippets="",
|
||||||
|
parts=None,
|
||||||
|
mentions=None,
|
||||||
|
member_event=None,
|
||||||
|
is_system=False,
|
||||||
|
is_hidden=False,
|
||||||
|
is_forward=False,
|
||||||
|
is_quote=False,
|
||||||
|
)
|
||||||
|
defaults.update(kwargs)
|
||||||
|
return Message(id=id, time=time, text=text, **defaults)
|
||||||
|
|
||||||
|
|
||||||
|
def _make_cleaned(id: str, time: int, text: str = "hello") -> CleanedMessage:
|
||||||
|
return CleanedMessage(
|
||||||
|
id=id,
|
||||||
|
sender_id="user@x.com",
|
||||||
|
time=time,
|
||||||
|
thread_sn=None,
|
||||||
|
text=text,
|
||||||
|
parts=[],
|
||||||
|
mentions=[],
|
||||||
|
member_event_text="",
|
||||||
|
file_info=[],
|
||||||
|
is_system=False,
|
||||||
|
is_forward=False,
|
||||||
|
is_quote=False,
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
class TestBuildChunks:
|
||||||
|
def test_empty_new_messages(self):
|
||||||
|
result = build_chunks([], [])
|
||||||
|
assert result == []
|
||||||
|
|
||||||
|
def test_single_message(self):
|
||||||
|
msgs = [_make_message("m1", 1000000, text="A simple message")]
|
||||||
|
result = build_chunks([], msgs)
|
||||||
|
assert len(result) == 1
|
||||||
|
assert "m1" in result[0].message_ids
|
||||||
|
|
||||||
|
def test_message_ids_preserved(self):
|
||||||
|
msgs = [
|
||||||
|
_make_message("m1", 1000000, text="First"),
|
||||||
|
_make_message("m2", 1000100, text="Second"),
|
||||||
|
]
|
||||||
|
result = build_chunks([], msgs)
|
||||||
|
all_ids = [mid for chunk in result for mid in chunk.message_ids]
|
||||||
|
assert "m1" in all_ids
|
||||||
|
assert "m2" in all_ids
|
||||||
|
|
||||||
|
def test_different_content_fields(self):
|
||||||
|
msgs = [_make_message("m1", 1000000, text="test")]
|
||||||
|
result = build_chunks([], msgs)
|
||||||
|
chunk = result[0]
|
||||||
|
assert chunk.page_content
|
||||||
|
assert chunk.dense_content
|
||||||
|
assert chunk.sparse_content
|
||||||
|
|
||||||
|
def test_overlap_appears_in_chunk(self):
|
||||||
|
overlap = [_make_message("o1", 999000, text="overlap message")]
|
||||||
|
new_msgs = [_make_message("m1", 1000000, text="new message")]
|
||||||
|
result = build_chunks(overlap, new_msgs)
|
||||||
|
assert len(result) >= 1
|
||||||
|
# overlap ids should NOT be in message_ids (they're context only)
|
||||||
|
assert "o1" not in result[0].message_ids
|
||||||
|
assert "m1" in result[0].message_ids
|
||||||
|
|
||||||
|
def test_time_gap_splits_window(self):
|
||||||
|
msgs = [
|
||||||
|
_make_message("m1", 1000000, text="morning message"),
|
||||||
|
_make_message("m2", 1000000 + TIME_GAP_SECONDS + 1, text="evening message"),
|
||||||
|
]
|
||||||
|
result = build_chunks([], msgs)
|
||||||
|
# large time gap should create 2 chunks
|
||||||
|
assert len(result) == 2
|
||||||
|
|
||||||
|
def test_empty_messages_skipped(self):
|
||||||
|
msgs = [
|
||||||
|
_make_message("m1", 1000000, text=""),
|
||||||
|
_make_message("m2", 1000100, text="real content"),
|
||||||
|
]
|
||||||
|
result = build_chunks([], msgs)
|
||||||
|
all_ids = [mid for chunk in result for mid in chunk.message_ids]
|
||||||
|
assert "m1" not in all_ids
|
||||||
|
assert "m2" in all_ids
|
||||||
|
|
||||||
|
|
||||||
|
class TestSplitWindows:
|
||||||
|
def test_empty(self):
|
||||||
|
assert _split_windows([]) == []
|
||||||
|
|
||||||
|
def test_single(self):
|
||||||
|
msgs = [_make_cleaned("m1", 1000000)]
|
||||||
|
windows = _split_windows(msgs)
|
||||||
|
assert len(windows) == 1
|
||||||
|
|
||||||
|
def test_time_gap_splits(self):
|
||||||
|
msgs = [
|
||||||
|
_make_cleaned("m1", 1000000),
|
||||||
|
_make_cleaned("m2", 1000000 + TIME_GAP_SECONDS + 1),
|
||||||
|
]
|
||||||
|
windows = _split_windows(msgs)
|
||||||
|
assert len(windows) == 2
|
||||||
|
|
||||||
|
def test_max_messages_splits(self):
|
||||||
|
msgs = [_make_cleaned(f"m{i}", 1000000 + i * 10) for i in range(WINDOW_MAX_MESSAGES + 2)]
|
||||||
|
windows = _split_windows(msgs)
|
||||||
|
assert len(windows) >= 2
|
||||||
170
tests/test_cleaning.py
Normal file
170
tests/test_cleaning.py
Normal file
|
|
@ -0,0 +1,170 @@
|
||||||
|
"""Unit tests for index/cleaning.py"""
|
||||||
|
import sys
|
||||||
|
import os
|
||||||
|
|
||||||
|
_INDEX_DIR = os.path.join(os.path.dirname(__file__), "..", "index")
|
||||||
|
sys.path.insert(0, _INDEX_DIR)
|
||||||
|
|
||||||
|
import pytest
|
||||||
|
from cleaning import (
|
||||||
|
normalize_unicode,
|
||||||
|
parse_file_snippets,
|
||||||
|
normalize_member_event,
|
||||||
|
normalize_part,
|
||||||
|
clean_message,
|
||||||
|
)
|
||||||
|
from index_schemas import Message
|
||||||
|
|
||||||
|
|
||||||
|
def _make_message(**kwargs) -> Message:
|
||||||
|
defaults = dict(
|
||||||
|
id="msg1",
|
||||||
|
thread_sn=None,
|
||||||
|
time=1000000,
|
||||||
|
text="",
|
||||||
|
sender_id="user@example.com",
|
||||||
|
file_snippets="",
|
||||||
|
parts=None,
|
||||||
|
mentions=None,
|
||||||
|
member_event=None,
|
||||||
|
is_system=False,
|
||||||
|
is_hidden=False,
|
||||||
|
is_forward=False,
|
||||||
|
is_quote=False,
|
||||||
|
)
|
||||||
|
defaults.update(kwargs)
|
||||||
|
return Message(**defaults)
|
||||||
|
|
||||||
|
|
||||||
|
class TestNormalizeUnicode:
|
||||||
|
def test_removes_zero_width(self):
|
||||||
|
assert "\u200b" not in normalize_unicode("hello\u200bworld")
|
||||||
|
assert "\u200c" not in normalize_unicode("a\u200cb")
|
||||||
|
assert "\ufeff" not in normalize_unicode("\ufefftext")
|
||||||
|
|
||||||
|
def test_collapses_whitespace(self):
|
||||||
|
result = normalize_unicode(" too many spaces ")
|
||||||
|
assert " " not in result
|
||||||
|
|
||||||
|
def test_normalizes_newlines(self):
|
||||||
|
result = normalize_unicode("line1\r\nline2\rline3")
|
||||||
|
assert "\r" not in result
|
||||||
|
|
||||||
|
def test_collapses_multiple_newlines(self):
|
||||||
|
result = normalize_unicode("a\n\n\n\nb")
|
||||||
|
assert "\n\n\n" not in result
|
||||||
|
|
||||||
|
def test_preserves_url(self):
|
||||||
|
url = "https://example.com/path?q=1&page=2"
|
||||||
|
assert url in normalize_unicode(url)
|
||||||
|
|
||||||
|
def test_preserves_email(self):
|
||||||
|
email = "user@corp.example"
|
||||||
|
assert email in normalize_unicode(email)
|
||||||
|
|
||||||
|
|
||||||
|
class TestParseFileSnippets:
|
||||||
|
def test_empty_string(self):
|
||||||
|
assert parse_file_snippets("") == []
|
||||||
|
|
||||||
|
def test_valid_list(self):
|
||||||
|
raw = '[{"name": "doc.pdf", "mime": "application/pdf"}]'
|
||||||
|
result = parse_file_snippets(raw)
|
||||||
|
assert len(result) == 1
|
||||||
|
assert result[0]["name"] == "doc.pdf"
|
||||||
|
|
||||||
|
def test_valid_dict(self):
|
||||||
|
raw = '{"name": "file.txt"}'
|
||||||
|
result = parse_file_snippets(raw)
|
||||||
|
assert len(result) == 1
|
||||||
|
|
||||||
|
def test_invalid_json(self):
|
||||||
|
assert parse_file_snippets("{broken json}") == []
|
||||||
|
|
||||||
|
def test_whitespace_only(self):
|
||||||
|
assert parse_file_snippets(" ") == []
|
||||||
|
|
||||||
|
|
||||||
|
class TestNormalizeMemberEvent:
|
||||||
|
def test_none_event(self):
|
||||||
|
assert normalize_member_event(None) == ""
|
||||||
|
|
||||||
|
def test_add_members(self):
|
||||||
|
event = {"type": "addMembers", "members": ["alice@example.com", "bob@example.com"]}
|
||||||
|
result = normalize_member_event(event)
|
||||||
|
assert "alice@example.com" in result
|
||||||
|
assert "added to chat" in result
|
||||||
|
|
||||||
|
def test_unknown_event(self):
|
||||||
|
event = {"type": "banUser", "member": "x@example.com"}
|
||||||
|
result = normalize_member_event(event)
|
||||||
|
assert "banUser" in result
|
||||||
|
|
||||||
|
|
||||||
|
class TestNormalizePart:
|
||||||
|
def test_text_part(self):
|
||||||
|
part = {"mediaType": "text", "text": "hello"}
|
||||||
|
result = normalize_part(part)
|
||||||
|
assert result["type"] == "text"
|
||||||
|
assert result["text"] == "hello"
|
||||||
|
|
||||||
|
def test_quote_part(self):
|
||||||
|
part = {"mediaType": "quote", "sn": "alice@x.com", "text": "original text"}
|
||||||
|
result = normalize_part(part)
|
||||||
|
assert result["type"] == "quote"
|
||||||
|
assert "alice@x.com" in result["text"]
|
||||||
|
assert "original text" in result["text"]
|
||||||
|
|
||||||
|
def test_forward_part(self):
|
||||||
|
part = {"mediaType": "forward", "sn": "channel@x.com", "text": "forwarded"}
|
||||||
|
result = normalize_part(part)
|
||||||
|
assert result["type"] == "forward"
|
||||||
|
assert "forwarded" in result["text"]
|
||||||
|
|
||||||
|
def test_unknown_media_type(self):
|
||||||
|
part = {"mediaType": "sticker", "text": ""}
|
||||||
|
result = normalize_part(part)
|
||||||
|
assert "sticker" in result["type"]
|
||||||
|
|
||||||
|
|
||||||
|
class TestCleanMessage:
|
||||||
|
def test_empty_message_is_empty(self):
|
||||||
|
msg = _make_message()
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert cleaned.is_empty
|
||||||
|
|
||||||
|
def test_text_message(self):
|
||||||
|
msg = _make_message(text="Hello world")
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert not cleaned.is_empty
|
||||||
|
assert cleaned.text == "Hello world"
|
||||||
|
|
||||||
|
def test_parts_extracted(self):
|
||||||
|
msg = _make_message(
|
||||||
|
parts=[{"mediaType": "text", "text": "from parts"}]
|
||||||
|
)
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert not cleaned.is_empty
|
||||||
|
assert any("from parts" in p["text"] for p in cleaned.parts)
|
||||||
|
|
||||||
|
def test_member_event_extracted(self):
|
||||||
|
msg = _make_message(
|
||||||
|
is_system=True,
|
||||||
|
member_event={"type": "addMembers", "members": ["u@x.com"]},
|
||||||
|
)
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert not cleaned.is_empty
|
||||||
|
assert "u@x.com" in cleaned.member_event_text
|
||||||
|
|
||||||
|
def test_file_snippets_parsed(self):
|
||||||
|
msg = _make_message(
|
||||||
|
file_snippets='[{"name": "report.pdf", "mime": "application/pdf"}]'
|
||||||
|
)
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert not cleaned.is_empty
|
||||||
|
assert cleaned.file_info[0]["name"] == "report.pdf"
|
||||||
|
|
||||||
|
def test_zero_width_stripped_from_text(self):
|
||||||
|
msg = _make_message(text="hello\u200bworld")
|
||||||
|
cleaned = clean_message(msg)
|
||||||
|
assert "\u200b" not in cleaned.text
|
||||||
118
tests/test_query_builder.py
Normal file
118
tests/test_query_builder.py
Normal file
|
|
@ -0,0 +1,118 @@
|
||||||
|
"""Unit tests for search/query_builder.py (pure logic only, no HTTP)"""
|
||||||
|
import sys
|
||||||
|
import os
|
||||||
|
|
||||||
|
_SEARCH_DIR = os.path.join(os.path.dirname(__file__), "..", "search")
|
||||||
|
sys.path.insert(0, _SEARCH_DIR)
|
||||||
|
|
||||||
|
# Stub env vars before importing search modules
|
||||||
|
os.environ.setdefault("EMBEDDINGS_DENSE_URL", "http://localhost/embed")
|
||||||
|
os.environ.setdefault("RERANKER_URL", "http://localhost/rerank")
|
||||||
|
os.environ.setdefault("QDRANT_URL", "http://localhost:6333")
|
||||||
|
os.environ.setdefault("API_KEY", "test-key")
|
||||||
|
|
||||||
|
from schemas import Entities, Question
|
||||||
|
from query_builder import (
|
||||||
|
build_primary_query,
|
||||||
|
build_extra_dense_queries,
|
||||||
|
build_sparse_query,
|
||||||
|
build_entity_tokens,
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def _q(**kwargs) -> Question:
|
||||||
|
defaults = dict(text="default question")
|
||||||
|
defaults.update(kwargs)
|
||||||
|
return Question(**defaults)
|
||||||
|
|
||||||
|
|
||||||
|
class TestBuildPrimaryQuery:
|
||||||
|
def test_uses_search_text_over_text(self):
|
||||||
|
q = _q(text="original", search_text="refined query")
|
||||||
|
assert build_primary_query(q) == "refined query"
|
||||||
|
|
||||||
|
def test_fallback_to_text(self):
|
||||||
|
q = _q(text="fallback text", search_text="")
|
||||||
|
assert build_primary_query(q) == "fallback text"
|
||||||
|
|
||||||
|
def test_strips_whitespace(self):
|
||||||
|
q = _q(text=" trimmed ")
|
||||||
|
assert build_primary_query(q) == "trimmed"
|
||||||
|
|
||||||
|
def test_collapses_internal_spaces(self):
|
||||||
|
q = _q(text="too many spaces")
|
||||||
|
result = build_primary_query(q)
|
||||||
|
assert " " not in result
|
||||||
|
|
||||||
|
|
||||||
|
class TestBuildExtraDenseQueries:
|
||||||
|
def test_no_extras_when_none(self):
|
||||||
|
q = _q(text="q")
|
||||||
|
assert build_extra_dense_queries(q) == []
|
||||||
|
|
||||||
|
def test_includes_variants(self):
|
||||||
|
q = _q(text="q", variants=["var1", "var2"])
|
||||||
|
extras = build_extra_dense_queries(q)
|
||||||
|
assert "var1" in extras
|
||||||
|
assert "var2" in extras
|
||||||
|
|
||||||
|
def test_includes_hyde(self):
|
||||||
|
q = _q(text="q", hyde=["hypothetical answer"])
|
||||||
|
extras = build_extra_dense_queries(q)
|
||||||
|
assert "hypothetical answer" in extras
|
||||||
|
|
||||||
|
def test_skips_empty_strings(self):
|
||||||
|
q = _q(text="q", variants=["", " ", "valid"])
|
||||||
|
extras = build_extra_dense_queries(q)
|
||||||
|
assert "" not in extras
|
||||||
|
assert " " not in extras
|
||||||
|
assert "valid" in extras
|
||||||
|
|
||||||
|
|
||||||
|
class TestBuildSparseQuery:
|
||||||
|
def test_uses_keywords_when_present(self):
|
||||||
|
q = _q(text="question", keywords=["go", "golang", "performance"])
|
||||||
|
result = build_sparse_query(q)
|
||||||
|
assert "go" in result
|
||||||
|
assert "golang" in result
|
||||||
|
|
||||||
|
def test_fallback_to_primary_when_no_keywords(self):
|
||||||
|
q = _q(text="fallback question", search_text="refined")
|
||||||
|
result = build_sparse_query(q)
|
||||||
|
assert result == "refined"
|
||||||
|
|
||||||
|
def test_empty_keywords_fallback(self):
|
||||||
|
q = _q(text="my question", keywords=[])
|
||||||
|
result = build_sparse_query(q)
|
||||||
|
assert result == "my question"
|
||||||
|
|
||||||
|
|
||||||
|
class TestBuildEntityTokens:
|
||||||
|
def test_no_entities(self):
|
||||||
|
q = _q(text="q")
|
||||||
|
assert build_entity_tokens(q) == []
|
||||||
|
|
||||||
|
def test_people_extracted(self):
|
||||||
|
q = _q(text="q", entities=Entities(people=["Alice", "Bob"]))
|
||||||
|
tokens = build_entity_tokens(q)
|
||||||
|
assert "Alice" in tokens
|
||||||
|
assert "Bob" in tokens
|
||||||
|
|
||||||
|
def test_all_entity_fields(self):
|
||||||
|
q = _q(
|
||||||
|
text="q",
|
||||||
|
entities=Entities(
|
||||||
|
people=["Alice"],
|
||||||
|
emails=["alice@corp.com"],
|
||||||
|
documents=["report.pdf"],
|
||||||
|
names=["Project X"],
|
||||||
|
links=["https://example.com"],
|
||||||
|
),
|
||||||
|
)
|
||||||
|
tokens = build_entity_tokens(q)
|
||||||
|
assert len(tokens) == 5
|
||||||
|
|
||||||
|
def test_strips_whitespace(self):
|
||||||
|
q = _q(text="q", entities=Entities(people=[" Alice "]))
|
||||||
|
tokens = build_entity_tokens(q)
|
||||||
|
assert "Alice" in tokens
|
||||||
110
tests/test_rendering.py
Normal file
110
tests/test_rendering.py
Normal file
|
|
@ -0,0 +1,110 @@
|
||||||
|
"""Unit tests for index/rendering.py"""
|
||||||
|
import sys
|
||||||
|
import os
|
||||||
|
|
||||||
|
_INDEX_DIR = os.path.join(os.path.dirname(__file__), "..", "index")
|
||||||
|
sys.path.insert(0, _INDEX_DIR)
|
||||||
|
|
||||||
|
from cleaning import CleanedMessage
|
||||||
|
from rendering import render_page_content, render_dense_content, render_sparse_content
|
||||||
|
|
||||||
|
|
||||||
|
def _make_cleaned(**kwargs) -> CleanedMessage:
|
||||||
|
defaults = dict(
|
||||||
|
id="msg1",
|
||||||
|
sender_id="alice@example.com",
|
||||||
|
time=1700000000,
|
||||||
|
thread_sn=None,
|
||||||
|
text="",
|
||||||
|
parts=[],
|
||||||
|
mentions=[],
|
||||||
|
member_event_text="",
|
||||||
|
file_info=[],
|
||||||
|
is_system=False,
|
||||||
|
is_forward=False,
|
||||||
|
is_quote=False,
|
||||||
|
)
|
||||||
|
defaults.update(kwargs)
|
||||||
|
return CleanedMessage(**defaults)
|
||||||
|
|
||||||
|
|
||||||
|
class TestRenderPageContent:
|
||||||
|
def test_text_message(self):
|
||||||
|
msg = _make_cleaned(text="Hello world")
|
||||||
|
result = render_page_content(msg)
|
||||||
|
assert "alice@example.com" in result
|
||||||
|
assert "Hello world" in result
|
||||||
|
|
||||||
|
def test_quote_part(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
parts=[{"type": "quote", "text": "[quote from bob]: original"}]
|
||||||
|
)
|
||||||
|
result = render_page_content(msg)
|
||||||
|
assert "[quote from bob]" in result
|
||||||
|
|
||||||
|
def test_forward_part(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
parts=[{"type": "forward", "text": "[forwarded from channel]: content"}],
|
||||||
|
is_forward=True,
|
||||||
|
)
|
||||||
|
result = render_page_content(msg)
|
||||||
|
assert "forwarded" in result
|
||||||
|
|
||||||
|
def test_member_event(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
member_event_text="[system: alice@x.com added to chat]",
|
||||||
|
is_system=True,
|
||||||
|
)
|
||||||
|
result = render_page_content(msg)
|
||||||
|
assert "added to chat" in result
|
||||||
|
|
||||||
|
def test_file_attachment(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
file_info=[{"name": "report.pdf", "mime": "application/pdf", "url": "", "date": ""}]
|
||||||
|
)
|
||||||
|
result = render_page_content(msg)
|
||||||
|
assert "report.pdf" in result
|
||||||
|
|
||||||
|
|
||||||
|
class TestRenderDenseContent:
|
||||||
|
def test_includes_timestamp(self):
|
||||||
|
msg = _make_cleaned(text="test", time=1700000000)
|
||||||
|
result = render_dense_content(msg)
|
||||||
|
assert "2023-" in result # UTC date
|
||||||
|
|
||||||
|
def test_includes_sender(self):
|
||||||
|
msg = _make_cleaned(text="hi", sender_id="bob@corp.com")
|
||||||
|
result = render_dense_content(msg)
|
||||||
|
assert "sender:bob@corp.com" in result
|
||||||
|
|
||||||
|
def test_forward_marker(self):
|
||||||
|
msg = _make_cleaned(is_forward=True, parts=[{"type": "forward", "text": "[forwarded]: x"}])
|
||||||
|
result = render_dense_content(msg)
|
||||||
|
assert "type:forward" in result
|
||||||
|
|
||||||
|
def test_mentions_included(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
text="hey",
|
||||||
|
mentions=["charlie@corp.com"],
|
||||||
|
)
|
||||||
|
result = render_dense_content(msg)
|
||||||
|
assert "charlie@corp.com" in result
|
||||||
|
|
||||||
|
|
||||||
|
class TestRenderSparseContent:
|
||||||
|
def test_includes_sender(self):
|
||||||
|
msg = _make_cleaned(text="hello")
|
||||||
|
result = render_sparse_content(msg)
|
||||||
|
assert "alice@example.com" in result
|
||||||
|
|
||||||
|
def test_includes_mentions(self):
|
||||||
|
msg = _make_cleaned(text="ping", mentions=["dave@corp.com"])
|
||||||
|
result = render_sparse_content(msg)
|
||||||
|
assert "dave@corp.com" in result
|
||||||
|
|
||||||
|
def test_includes_filename(self):
|
||||||
|
msg = _make_cleaned(
|
||||||
|
file_info=[{"name": "budget.xlsx", "mime": "", "url": "", "date": ""}]
|
||||||
|
)
|
||||||
|
result = render_sparse_content(msg)
|
||||||
|
assert "budget.xlsx" in result
|
||||||
Loading…
Reference in a new issue