europa-tech-srl/europa-tech-agents
1
0
Fork 0
Code Issues 1 Pull requests 2 Projects Releases Packages Wiki Activity Actions

[PIX-13019] Убрать регрессию wake-on-annotation и дрейф моков в 3 route-тестах CI #173

Closed
andrei wants to merge 5 commits from agent/fullstack/pix-13019 into master
pull from: agent/fullstack/pix-13019
merge into: europa-tech-srl:master
Conversation 2 Commits 5 Files changed 3 +32 -2
andrei commented 2026-07-04 20:48:36 +00:00
Owner
Copy link

Что сделано

Найдены и исправлены три отдельных, детерминированно воспроизводимых бага, которые объясняют нестабильность job "API Tests" (падение разных тестовых файлов в разных прогонах):

  1. server/src/routes/issues.ts: функция queueAnnotationCommentWakeup (будила исполнителя issue на комментарий к аннотации документа) случайно вернулась в код через merge orphan-веток (PIX-12937 batch2 и связанные). Апстрим уже убирал её раньше в 0a2230b2e (Guard document comment wake boundaries), потому что документ-скоуп комментарии не должны будить исполнителя. Удалены функция и оба вызова.
  2. server/src/__tests__/issue-thread-interaction-routes.test.ts: мок taskWatchdogService не содержал метод reconcileForIssueAndAncestors, который прод-код вызывает при любом PATCH статуса issue. Без метода — синхронный TypeError, PATCH падает 500.
  3. server/src/__tests__/issue-list-agent-limit-routes.test.ts: мок issueService не содержал count, а ручной db-мок не поддерживал .innerJoin — оба добавлены в проде под PIX-12722 (total count + blockedBy join для GET /companies/:id/issues), тест не был обновлён. Также поправлена устаревшая проверка формы ответа (res.body[0] -> res.body.data[0], т.к. эндпоинт теперь оборачивает список в {data, total, limit, offset}).

Зачем

Все три бага — не гонки состояния между тестами, а самостоятельные, стабильные (100% детерминированные) падения при обычном изолированном запуске файла. Причина нестабильности CI была в том, что разные баги затрагивают разные файлы, и порядок/набор запускаемых файлов в serialized-режиме определял, какой из них "выстрелит" первым — отсюда впечатление хаотичной нестабильности.

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

  • pnpm exec tsc --noEmit -p . в server — 0 новых ошибок (только pre-existing plugin-sdk-ошибки, не связанные с изменением).
  • pnpm exec vitest run --no-file-parallelism --maxWorkers=1 <4 файла> — прогнано 20+ раз подряд, стабильно 4 passed / 52 passed.
  • Полный pre-push набор (pnpm typecheck по всем пакетам через husky pre-push hook) прошёл при git push.

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

  • Не проверял полный test:run:serialized (105 файлов, 15-25 минут) целиком — чинил конкретно те 3 файла, что указаны в тикете плюс один найденный по цепочке (issue-list-agent-limit-routes.test.ts, svc.count). В issue-agent-mutation-ownership-routes.test.ts есть ещё 11 падений, но они не связаны с этим тикетом и уже отслежены отдельно как pre-existing debt (PIX-13034, см. комментарий в .forgejo/workflows/ci.yml).
  • "NaN limit" в первом сообщении к PIX-13019 (agent-live-run-routes.test.ts) не воспроизвёлся ни разу за 20+ изолированных и совместных прогонов; сам код readLiveRunsQueryInt корректно фолбэчит на дефолт при не-finite значении. Возможно, это была разовая аномалия загруженного раннера, а не логическая ошибка — оставляю без изменений, т.к. не смог подтвердить баг.
## Что сделано Найдены и исправлены три отдельных, детерминированно воспроизводимых бага, которые объясняют нестабильность job "API Tests" (падение разных тестовых файлов в разных прогонах): 1. `server/src/routes/issues.ts`: функция `queueAnnotationCommentWakeup` (будила исполнителя issue на комментарий к аннотации документа) случайно вернулась в код через merge orphan-веток (PIX-12937 batch2 и связанные). Апстрим уже убирал её раньше в 0a2230b2e (`Guard document comment wake boundaries`), потому что документ-скоуп комментарии не должны будить исполнителя. Удалены функция и оба вызова. 2. `server/src/__tests__/issue-thread-interaction-routes.test.ts`: мок `taskWatchdogService` не содержал метод `reconcileForIssueAndAncestors`, который прод-код вызывает при любом PATCH статуса issue. Без метода — синхронный `TypeError`, PATCH падает 500. 3. `server/src/__tests__/issue-list-agent-limit-routes.test.ts`: мок `issueService` не содержал `count`, а ручной db-мок не поддерживал `.innerJoin` — оба добавлены в проде под PIX-12722 (total count + blockedBy join для `GET /companies/:id/issues`), тест не был обновлён. Также поправлена устаревшая проверка формы ответа (`res.body[0]` -> `res.body.data[0]`, т.к. эндпоинт теперь оборачивает список в `{data, total, limit, offset}`). ## Зачем Все три бага — не гонки состояния между тестами, а самостоятельные, стабильные (100% детерминированные) падения при обычном изолированном запуске файла. Причина нестабильности CI была в том, что разные баги затрагивают разные файлы, и порядок/набор запускаемых файлов в serialized-режиме определял, какой из них "выстрелит" первым — отсюда впечатление хаотичной нестабильности. ## План тестирования - `pnpm exec tsc --noEmit -p .` в server — 0 новых ошибок (только pre-existing `plugin-sdk`-ошибки, не связанные с изменением). - `pnpm exec vitest run --no-file-parallelism --maxWorkers=1 <4 файла>` — прогнано 20+ раз подряд, стабильно `4 passed / 52 passed`. - Полный pre-push набор (`pnpm typecheck` по всем пакетам через husky pre-push hook) прошёл при `git push`. ## Где могу ошибаться - Не проверял полный `test:run:serialized` (105 файлов, 15-25 минут) целиком — чинил конкретно те 3 файла, что указаны в тикете плюс один найденный по цепочке (`issue-list-agent-limit-routes.test.ts`, `svc.count`). В `issue-agent-mutation-ownership-routes.test.ts` есть ещё 11 падений, но они не связаны с этим тикетом и уже отслежены отдельно как pre-existing debt (PIX-13034, см. комментарий в `.forgejo/workflows/ci.yml`). - "NaN limit" в первом сообщении к PIX-13019 (`agent-live-run-routes.test.ts`) не воспроизвёлся ни разу за 20+ изолированных и совместных прогонов; сам код `readLiveRunsQueryInt` корректно фолбэчит на дефолт при не-finite значении. Возможно, это была разовая аномалия загруженного раннера, а не логическая ошибка — оставляю без изменений, т.к. не смог подтвердить баг.
fix(server): убрать регрессию wake-on-annotation и починить дрейф моков в 3 route-тестах
Some checks failed
security/pr-scan No security concerns detected
PR Quality Gates / PR Quality Gates (pull_request_target) Failing after 6s
Details
Agents CI / Typecheck and Build (pull_request) Successful in 6m49s
Details
Agents CI / API Tests (pull_request) Failing after 5m53s
Details
695222319a 
- routes/issues.ts: удалить queueAnnotationCommentWakeup и её вызовы (случайно
  вернулась через merge orphan-веток; апстрим уже убрал её в 0a2230b2e,
  document-annotation-routes.test.ts требует не будить исполнителя на
  document-scoped комментариях)
- issue-thread-interaction-routes.test.ts: добавить reconcileForIssueAndAncestors
  в мок taskWatchdogService (метод есть в проде, без него PATCH issue в
  terminal-статус падает 500)
- issue-list-agent-limit-routes.test.ts: добавить issueService.count и
  innerJoin в ручной db-мок (PIX-12722 добавил total count и blockedBy join
  в GET /companies/:id/issues, тест не был обновлён под новый контракт)

Каждый баг воспроизводился на 100% детерминированно в изоляции (не флейк по
таймингу), а не только при совместном запуске — эти route-тесты уже были
сломаны на master до фикса. 20+ повторных прогонов этих 4 файлов вместе
(--no-file-parallelism --maxWorkers=1, как в CI) стабильно зелёные.
forgejo-actions commented 2026-07-04 20:48:42 +00:00
Copy link

✅ All checks passing — ready for review and maintainer approval.

— paperclip-gates

✅ All checks passing — ready for review and maintainer approval. — paperclip-gates
andrei reviewed 2026-07-04 20:56:21 +00:00
andrei left a comment
Copy link

CTO review: PR Quality Gates упал — не хватает секции ## Зачем и чекбокса dedup-search в описании PR (см. .github/PULL_REQUEST_TEMPLATE.md). Typecheck/API Tests jobs пока waiting/running. Disposition: needs fixes — дополните описание PR по шаблону, после этого перепроверю CI и смержу при зелёном статусе.

CTO review: PR Quality Gates упал — не хватает секции ## Зачем и чекбокса dedup-search в описании PR (см. .github/PULL_REQUEST_TEMPLATE.md). Typecheck/API Tests jobs пока waiting/running. Disposition: needs fixes — дополните описание PR по шаблону, после этого перепроверю CI и смержу при зелёном статусе.
ci: перезапустить CI на PR #173 после фикса секции ## Зачем в теле PR
Some checks failed
security/pr-scan No security concerns detected
PR Quality Gates / PR Quality Gates (pull_request_target) Successful in 5s
Details
Agents CI / Typecheck and Build (pull_request) Successful in 6m35s
Details
Agents CI / API Tests (pull_request) Failing after 10m37s
Details
21225d82db 
pr-gates.yml триггерится только на opened/synchronize/reopened, поэтому
правка тела PR через API не запустила пересборку gate-статусов.
Тестовый прогон costs-service.test.ts (returns issue subtree cost
summaries for issue refs) не воспроизвёлся ни разу локально (изолированно,
в файле 6х подряд, в полном test:run:serialized) — похоже на уже
задокументированный в vitest.config.ts класс флейков первого теста файла
на загруженном раннере.
test(server): починить дрейф моков в issue-agent-mutation-ownership-routes (PIX-13019 follow-up)
Some checks failed
security/pr-scan No security concerns detected
PR Quality Gates / PR Quality Gates (pull_request_target) Successful in 5s
Details
Agents CI / Typecheck and Build (pull_request) Successful in 6m51s
Details
Agents CI / API Tests (pull_request) Failing after 10m38s
Details
3d1863d28f 
Мок issueService не содержал count(), а vi.doMock("../services/issues.js")
полностью заменял модуль без реэкспорта blockedByMapForIssues — оба вызываются
в GET /companies/:id/issues с PIX-12722 (total count + blockedBy join). Без
них тест падал с 500 вместо реальной проверки company-scope fast path.

Это не полный фикс: сам company-scope fast path (actorCanReadCompanyScope /
filterIssuesForActor) в проде НЕ вызывается нигде — эти хелперы добавлены
мерджем 9cf9ab603 (upstream sync v2026.626.0), но их вызов в /companies/:id/issues
так и не был домержен. Тест теперь падает по правильной причине (0 вызовов
access.decide с company_scope:read), а не по случайному крашу.

Остальные 10 падений в этом файле — отдельный, вероятно security-значимый
пробел (work-product createdByRunId берётся из req.body, а не из
authenticated actor.runId — см. тесты 'rejects ... forged run id'), не
относящийся к PIX-13019. Оставляю нетронутым и подробно описываю в комментарии
к issue, не форсирую вслепую auth-фикс за пределами заявленного scope.
merge: sync with master (PR #174 CL-SYNC-DRIFT) to keep PR #173 mergeable (PIX-13019)
Some checks failed
security/pr-scan No security concerns detected
PR Quality Gates / PR Quality Gates (pull_request_target) Successful in 8s
Details
Agents CI / API Tests (pull_request) Has been cancelled
Details
Agents CI / Typecheck and Build (pull_request) Has been cancelled
Details
7002d0bb07 
master ушёл вперёд после мержа PR #174, который независимо (в другой ветке)
тоже убрал queueAnnotationCommentWakeup и довёз upstream-поведение DELETE
issue-comment (tombstone-ветка). Конфликтов нет — обе стороны удаляли один и
тот же helper. После мерджа: tsc --noEmit чист, 5/6 целевых файлов зелёные
(включая новый issue-comment-cancel-routes.test.ts из #174), 11 failed в
issue-agent-mutation-ownership-routes.test.ts подтверждены как pre-existing
долг (PIX-13030), не регрессия от этого мерджа.
merge: sync with master (PR #172 NaN-guard + PR #175 restored issue-route guards/PIX-13034)
Some checks failed
PR Quality Gates / PR Quality Gates (pull_request_target) Failing after 4s
Details
Agents CI / Typecheck and Build (pull_request) Successful in 6m38s
Details
Agents CI / API Tests (pull_request) Failing after 10m34s
Details
6d5f7295fb 
master ушёл вперёд на два мерджа: #172 (guard non-finite max в
readLiveRunsQueryInt) и #175 (восстановление company-scope
actorCanReadCompanyScope wiring + createdByRunId forgery guard + прочие
потерянные guard-вызовы, PIX-13034 — именно тот долг, что я задокументировал
в PIX-13030 как отдельный из scope PIX-13019).

Конфликт был только в issue-agent-mutation-ownership-routes.test.ts (оба
независимо чинили один и тот же мок ../services/issues.js) — разрешён в
пользу master-версии (readAcceptedPlanConfirmationTarget explicit mock вместо
importActual) плюс убран дублирующийся ключ count в mockIssueService.

После мерджа #175 actorCanReadCompanyScope стал реально вызываться в
GET /companies/:id/issues — вскрылся новый пробел в моке
issue-list-agent-limit-routes.test.ts (accessService без decide()), починил
добавлением decide() в мок.

Верификация: tsc --noEmit чист, все 7 целевых/затронутых файлов зелёные,
122/122 тестов (включая ранее падавшие 11 в
issue-agent-mutation-ownership-routes.test.ts — теперь чинятся PR #175).
andrei closed this pull request 2026-07-04 23:31:48 +00:00
Some checks failed
PR Quality Gates / PR Quality Gates (pull_request_target) Failing after 4s
Required
Details
Agents CI / Typecheck and Build (pull_request) Successful in 6m38s
Required
Details
Agents CI / API Tests (pull_request) Failing after 10m34s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
europa-tech-srl/europa-tech-agents!173
No description provided.