[PIX-13097] fix(tests): откат несуществующего queueAnnotationCommentWakeup assertion #191

Merged

andrei merged 1 commit from agent/cto/pix-13097 into master

2026-07-05 13:45:17 +00:00

andrei commented

2026-07-05 12:55:29 +00:00

Owner

Что сделано

Откатил assertion в document-annotation-routes.test.ts, изменённый в PR #187 (коммит 19ae4c789), обратно на expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled().

Зачем

PR #187 сослался на функцию queueAnnotationCommentWakeup как источник wake-диспатча при добавлении комментария к аннотации документа. Эта функция нигде не существует в кодовой базе (grep не находит ничего, кроме самого комментария в тесте). Поведение "комментарий к аннотации будит агента" было намеренно удалено дважды как регрессия: upstream 0a2230b2e ("Guard document comment wake boundaries", #7766) и в этом форке 695222319 ("fix(server): убрать регрессию wake-on-annotation"). Маршрут POST /issues/:id/documents/:key/annotations/:threadId/comments в server/src/routes/issues.ts не вызывает heartbeat.wakeup.

То есть PR #187 по ошибке развернул уже дважды исправленную регрессию вместо того, чтобы починить реальную гонку в тестах.

План тестирования

vitest run server/src/__tests__/document-annotation-routes.test.ts прогнан 5/5 раз подряд в изоляции — все 6 тестов зелёные каждый раз.
grep -rn queueAnnotationCommentWakeup по server/app/ui — не находит ссылок за пределами исправленного теста.
pnpm --filter server typecheck (через pre-push hook) — прошёл без ошибок.

Где могу ошибаться

Если wake-on-annotation-comment на самом деле желаемое новое поведение (а не регрессия), нужно вместо отката тестов реализовать queueAnnotationCommentWakeup по-настоящему — это решение уровня CTO/CEO, см. issue PIX-13097.

Я поискал похожие PR и подтверждаю, что дубликатов нет.

## Что сделано Откатил assertion в `document-annotation-routes.test.ts`, изменённый в PR #187 (коммит 19ae4c789), обратно на `expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled()`. ## Зачем PR #187 сослался на функцию `queueAnnotationCommentWakeup` как источник wake-диспатча при добавлении комментария к аннотации документа. Эта функция нигде не существует в кодовой базе (grep не находит ничего, кроме самого комментария в тесте). Поведение "комментарий к аннотации будит агента" было намеренно удалено дважды как регрессия: upstream `0a2230b2e` ("Guard document comment wake boundaries", #7766) и в этом форке `695222319` ("fix(server): убрать регрессию wake-on-annotation"). Маршрут `POST /issues/:id/documents/:key/annotations/:threadId/comments` в `server/src/routes/issues.ts` не вызывает `heartbeat.wakeup`. То есть PR #187 по ошибке развернул уже дважды исправленную регрессию вместо того, чтобы починить реальную гонку в тестах. ## План тестирования - `vitest run server/src/__tests__/document-annotation-routes.test.ts` прогнан 5/5 раз подряд в изоляции — все 6 тестов зелёные каждый раз. - `grep -rn queueAnnotationCommentWakeup` по server/app/ui — не находит ссылок за пределами исправленного теста. - `pnpm --filter server typecheck` (через pre-push hook) — прошёл без ошибок. ## Где могу ошибаться Если wake-on-annotation-comment на самом деле желаемое новое поведение (а не регрессия), нужно вместо отката тестов реализовать `queueAnnotationCommentWakeup` по-настоящему — это решение уровня CTO/CEO, см. issue PIX-13097. - [x] Я поискал похожие PR и подтверждаю, что дубликатов нет.

andrei added 1 commit

2026-07-05 12:55:30 +00:00

fix(tests): revert non-existent queueAnnotationCommentWakeup assertion (PIX-13097)

security/pr-scan No security concerns detected

PR Quality Gates / PR Quality Gates (pull_request_target) Successful in 7s

Details

Agents CI / API Tests (pull_request) Successful in 18m29s

Details

Agents CI / Typecheck and Build (pull_request) Has been cancelled

Details

6eff966cf9

PR #187 changed document-annotation-routes.test.ts to expect
mockHeartbeatService.wakeup to be called on annotation comments,
citing queueAnnotationCommentWakeup as the wake source. That function
does not exist anywhere in the codebase; wake-on-annotation-comment
was intentionally removed twice as a regression (upstream 0a2230b2e
"Guard document comment wake boundaries" #7766, fork 695222319).
Restore the not.toHaveBeenCalled() assertions matching actual route
behavior in server/src/routes/issues.ts.