fix: [CL-CI-SERVICE-PORT-COLLISION] убрать host-port publish у postgres и починить нестабильные тесты API #169

Merged

andrei merged 10 commits from feature/claude-ci-port-collision into master 2026-07-04 17:50:57 +00:00

Conversation 1 Commits 10 Files changed 7 +75 -13

andrei commented 2026-07-04 13:31:44 +00:00

Owner

Copy link

Что сделано

Убран ports: 5432:5432 у postgres service-контейнера джобы API Tests в .forgejo/workflows/ci.yml; DATABASE_URL переведён с localhost:5432 на service hostname postgres:5432.

Зачем

Run 768 (merge-коммит 0d8ed19, PR #160) упал не из-за кода: service-контейнер не стартовал — Bind for 127.0.0.1:5432 failed: port is already allocated. Фиксированный host-порт конфликтует с параллельными CI-ранами на едином раннере (в тот момент шёл CI europa-tech, который раньше страдал тем же — исправлено там 2026-06-01, CL-CI-SERVICE-PORT-COLLISION). Джоба контейнеризована и ходит в БД по docker network alias — publish на host не нужен вовсе.

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

• yaml.safe_load на изменённом workflow — ok.
• CI этого PR сам прогонит джобу API Tests с новой конфигурацией сервиса (postgres поднимается без host-порта, тесты ходят на postgres:5432).
• После merge проверить зелёный статус push-рана на master.

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

• Если runner когда-то переключат в host network mode, alias postgres перестанет резолвиться — но текущий config.yml (container.network: "", авто-сеть) это исключает.
• Джоба стоит continue-on-error: true (PIX-13034), при этом Forgejo всё равно пометил run как failure — это отдельная известная особенность forgejo-runner, этим PR не решается.

## Что сделано Убран `ports: 5432:5432` у postgres service-контейнера джобы **API Tests** в `.forgejo/workflows/ci.yml`; `DATABASE_URL` переведён с `localhost:5432` на service hostname `postgres:5432`. ## Зачем Run 768 (merge-коммит `0d8ed19`, PR #160) упал не из-за кода: service-контейнер не стартовал — `Bind for 127.0.0.1:5432 failed: port is already allocated`. Фиксированный host-порт конфликтует с параллельными CI-ранами на едином раннере (в тот момент шёл CI europa-tech, который раньше страдал тем же — исправлено там 2026-06-01, CL-CI-SERVICE-PORT-COLLISION). Джоба контейнеризована и ходит в БД по docker network alias — publish на host не нужен вовсе. ## План тестирования - `yaml.safe_load` на изменённом workflow — ok. - CI этого PR сам прогонит джобу API Tests с новой конфигурацией сервиса (postgres поднимается без host-порта, тесты ходят на `postgres:5432`). - После merge проверить зелёный статус push-рана на master. ## Где могу ошибаться - Если runner когда-то переключат в host network mode, alias `postgres` перестанет резолвиться — но текущий config.yml (`container.network: ""`, авто-сеть) это исключает. - Джоба стоит `continue-on-error: true` (PIX-13034), при этом Forgejo всё равно пометил run как failure — это отдельная известная особенность forgejo-runner, этим PR не решается.

andrei added 1 commit 2026-07-04 13:31:44 +00:00

[CL-CI-SERVICE-PORT-COLLISION] fix(ci): убрать host-port publish у postgres-сервиса API Tests ... f4446a908f

Фиксированный host-порт 5432 у service-контейнера конфликтовал с параллельными
CI-ранами на едином раннере: контейнер падал на старте с
"Bind for 127.0.0.1:5432 failed: port is already allocated" (run 768,
коммит 0d8ed19). Джоба ходит в БД по docker network alias postgres:5432 —
publish на host не нужен. Тот же класс и то же решение, что
CL-CI-SERVICE-PORT-COLLISION 2026-06-01 в europa-tech ci.yml.

forgejo-actions commented 2026-07-04 13:42: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 changed title from [CL-CI-SERVICE-PORT-COLLISION] fix(ci): убрать host-port publish у postgres-сервиса API Tests to ci: убрать host-port publish у postgres-сервиса API Tests [CL-CI-SERVICE-PORT-COLLISION] 2026-07-04 13:48:24 +00:00

andrei changed title from ci: убрать host-port publish у postgres-сервиса API Tests [CL-CI-SERVICE-PORT-COLLISION] to ci: [CL-CI-SERVICE-PORT-COLLISION] убрать host-port publish у postgres-сервиса API Tests 2026-07-04 13:50:38 +00:00

andrei force-pushed feature/claude-ci-port-collision from f4446a908f to f46f5d2328 2026-07-04 13:52:42 +00:00 Compare

andrei added 1 commit 2026-07-04 14:28:02 +00:00

[CL-CI-SERVICE-PORT-COLLISION] test(server): поднять timeout первого теста agent-instructions-routes ... 2fea87b88d

'returns bundle metadata' — первый тест файла, платит cold-start приложения на
нагруженном CI-раннере: 5005ms при дефолтном бюджете 5000ms (run 35384, flaky).
Vitest 4 signature: options вторым аргументом. Локально 5/5 passed.

andrei added 1 commit 2026-07-04 15:07:49 +00:00

[PIX-12997] test(server): прогреть createApp() в beforeAll, поднять timeout до 30s ... ed6414bb0c

Тест agent-instructions-routes "returns bundle metadata" всё ещё падал по
таймауту 15000ms на run 16759 из-за cold-start модулей routes/agents.js и
middleware/index.js под нагрузкой раннера. Выносим прогрев vi.importActual
в beforeAll (один раз на файл, до per-test бюджета) и поднимаем таймаут
первого теста до 30000ms как дополнительный запас.

andrei force-pushed feature/claude-ci-port-collision from ed6414bb0c to 8c0e73b16b 2026-07-04 15:18:37 +00:00 Compare

andrei added 1 commit 2026-07-04 15:57:42 +00:00

[CL-CI-SERVICE-PORT-COLLISION] test(server): обновить stale assertions в agent-live-run-routes (PIX-13034) ... f6570fccf7

Suite впервые дошёл до этого файла после порт-фикса (run 35451) и вскрыл два
устаревших ожидания: decorateActiveRunStatus получает companyId как UUID из
issue (agents.ts:4324), а legacy company runs список передаёт 4-й аргумент
{ summary: false } (agents.ts:3775). Поведение роутов намеренное — обновлены
тестовые ожидания. Локально 23/23 passed.

andrei added 1 commit 2026-07-04 16:33:54 +00:00

[CL-CI-SERVICE-PORT-COLLISION] fix(scripts): win32 spawn для run-vitest-stable ... ead40dcc08

spawnSync('pnpm', ...) на Windows падает ENOENT: pnpm резолвится через pnpm.cmd,
который spawnSync находит только с shell. Опция включается лишь при
process.platform === 'win32' — поведение Linux-раннера байт-в-байт прежнее.

andrei added 1 commit 2026-07-04 16:38:13 +00:00

[CL-CI-SERVICE-PORT-COLLISION] fix(server): testTimeout 30s против transform cold-start первых тестов (PIX-13034) ... a570c4c877

Первый тест каждого route-файла платит vite transform routes/agents.ts (~4000
строк) внутри своего 5s бюджета; на нагруженном раннере это валило по одному
файлу за ран (35384/35411/35484 — три разных файла, всегда первый тест).
Глобальный testTimeout 30s закрывает класс; настоящие зависания по-прежнему
режутся, шаговый лимит джобы 5 минут не тронут.

andrei added 1 commit 2026-07-04 17:03:25 +00:00

[CL-CI-SERVICE-PORT-COLLISION] test(server): UUID-фикстуры для status-only approval тестов (PIX-13034) ... 11b035fccf

Роут approvals валидирует id через isUuidLike до authz (approvals.ts:33-40),
slug-фикстуры approval-7/approval-8 получали 400 Invalid approval id вместо
ожидаемого 403 (run 35505). Переведены на APPROVAL_7_ID/APPROVAL_8_ID по
конвенции остальных тестов файла. Локально 18/18 passed.

andrei changed title from ci: [CL-CI-SERVICE-PORT-COLLISION] убрать host-port publish у postgres-сервиса API Tests to fix: [CL-CI-SERVICE-PORT-COLLISION] убрать host-port publish у postgres и починить нестабильные тесты API 2026-07-04 17:06:25 +00:00

andrei added 1 commit 2026-07-04 17:12:14 +00:00

ci: retrigger PR Quality Gates after retitle (PIX-13007) d864df349c

andrei added 1 commit 2026-07-04 17:17:21 +00:00

[CL-CI-SERVICE-PORT-COLLISION] fix(ci): поднять лимиты джобы API Tests до 30/25 минут ... 0b7280a621

Serialized suite — 105 vitest-процессов по файлу, каждый со своим vite
transform (~4-6s на нагруженном раннере): suite нужно 15+ минут. Run 35511
оборван старым 5-минутным шаговым лимитом на середине с полностью зелёными
выполненными тестами (context canceled).

andrei added 1 commit 2026-07-04 17:36:44 +00:00

[CL-CI-SERVICE-PORT-COLLISION] fix(server): восстановить legacy company-scoped aliases, потерянные при upstream-merge ... e922a90cce

Три форк-роута с 307 redirect (/:companyId/agents/:id/skills,
/:companyId/issues/:id, /:companyId/issues/:id/comments) существовали в
прод-снапшоте 247c7749 и потерялись при merge upstream v2026.626.0 (9cf9ab60) —
companies-route-path-guard.test.ts честно ловил регресс (run 35520: 404 вместо
307). Восстановлены дословно из снапшота, helper requestQuerySuffix уже был.
Локально 4/4, typecheck чистый. Настоящий fork-drift, не stale-тест.

andrei referenced this pull request 2026-07-04 17:43:16 +00:00

[PIX-13011] Восстановить legacy company-scoped алиасы редиректов (307 вместо 404) #171

andrei merged commit 32b6f8b663 into master 2026-07-04 17:50:57 +00:00

andrei referenced this pull request from a commit 2026-07-04 17:50:59 +00:00

Merge pull request 'fix: [CL-CI-SERVICE-PORT-COLLISION] убрать host-port publish у postgres и починить нестабильные тесты API' (#169) from feature/claude-ci-port-collision into master

andrei referenced this pull request 2026-07-04 17:54:57 +00:00

[CI] API Tests красный 3 рана подряд: agent-instructions-routes «returns bundle metadata» — timeout 5s #170

andrei referenced this pull request 2026-07-04 21:23:26 +00:00

fix: довезти upstream-поведение issue-comment роутов после merge v2026.626.0 [CL-SYNC-DRIFT] #174

andrei referenced this pull request 2026-07-04 22:53:17 +00:00

fix: восстановить потерянные guard-вызовы issue-роутов и обновить ownership-тесты [CL-SYNC-DRIFT] #175

andrei referenced this pull request 2026-07-05 03:26:13 +00:00

[CI] API Tests красный 3 рана подряд: agent-instructions-routes «returns bundle metadata» — timeout 5s #170

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

Subscribe

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!169

No description provided.