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

feature/tax-profit-search-intent #37

Merged
admin merged 4 commits from feature/tax-profit-search-intent into main 2026-09-18 05:56:45 +00:00
Conversation 2 Commits 4 Files Changed 11 +96 -36
admin commented 2026-09-16 19:42:13 +00:00
Owner
Copy Link
No description provided.
admin added 3 commits 2026-09-16 19:42:15 +00:00
Improve tax profit search relevance 14e86ccdd8
Preserve cross-field search matching 2c43c7c9ad
Record search work handoff status 6d393d38bb
agent commented 2026-09-18 05:39:52 +00:00
Owner
Copy Link

Независимое ревью: найдено 1 замечание.

[P2] Обновите handoff после создания PR — docs/operations/project-status.md:28

В новом разделе по-прежнему сказано, что PR не создан, DNS является блокером, а следующим действием должно быть создание PR. Изменение уже находится в PR #37, поэтому документ направит следующего исполнителя по устаревшему сценарию. Укажите фактическое состояние PR и актуальный следующий шаг либо удалите временные детали handoff.

Других дефектов не выявлено. Проверки: 33 backend-теста прошли, 1 интеграционный тест с реальным OpenSearch пропущен; git diff --check чистый. Остаточный риск: тесты проверяют DSL, но не фактический порядок выдачи и влияние общего match_phrase на relevance set.

Независимое ревью: найдено 1 замечание. [P2] Обновите handoff после создания PR — `docs/operations/project-status.md:28` В новом разделе по-прежнему сказано, что PR не создан, DNS является блокером, а следующим действием должно быть создание PR. Изменение уже находится в PR #37, поэтому документ направит следующего исполнителя по устаревшему сценарию. Укажите фактическое состояние PR и актуальный следующий шаг либо удалите временные детали handoff. Других дефектов не выявлено. Проверки: 33 backend-теста прошли, 1 интеграционный тест с реальным OpenSearch пропущен; `git diff --check` чистый. Остаточный риск: тесты проверяют DSL, но не фактический порядок выдачи и влияние общего `match_phrase` на relevance set.
agent added 1 commit 2026-09-18 05:50:15 +00:00
docs: refresh search work handoff 5e029be5ee
agent commented 2026-09-18 05:53:17 +00:00
Owner
Copy Link

Повторное независимое ревью после 5e029be: No findings.

Прежнее замечание P2 устранено: docs/operations/project-status.md теперь корректно указывает PR #37 и не утверждает, что PR отсутствует. Полный merge-diff 47d51cd..5e029be проверен; git diff --check чистый, 33 backend-теста прошли, 1 интеграционный тест пропущен. Остаточный риск: фактическое ранжирование не проверено на реальном OpenSearch, поскольку локальный endpoint недоступен без AKYLDASH_OPENSEARCH_URL.

Повторное независимое ревью после `5e029be`: **No findings.** Прежнее замечание P2 устранено: `docs/operations/project-status.md` теперь корректно указывает PR #37 и не утверждает, что PR отсутствует. Полный merge-diff `47d51cd..5e029be` проверен; `git diff --check` чистый, 33 backend-теста прошли, 1 интеграционный тест пропущен. Остаточный риск: фактическое ранжирование не проверено на реальном OpenSearch, поскольку локальный endpoint недоступен без `AKYLDASH_OPENSEARCH_URL`.
admin merged commit 9832c0c795 into main 2026-09-18 05:56:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin agent
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: agent/akyldash#37
Delete Branch "feature/tax-profit-search-intent"

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