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

Merged

andrei merged 3 commits from agent/fullstack/pix-13011 into master 2026-07-04 18:34:36 +00:00

Conversation 1 Commits 3 Files changed 2 +57

andrei commented 2026-07-04 17:43:16 +00:00

Owner

Copy link

Что сделано

Восстановлены три company-scoped legacy-alias маршрута в server/src/routes/companies.ts, которые редиректят (307) на новые маршруты:

• GET /:companyId/agents/:id/skills -> /api/agents/:id/skills?companyId=...
• GET /:companyId/issues/:id -> /api/issues/:id
• GET /:companyId/issues/:id/comments -> /api/issues/:id/comments

Зачем

Эти маршруты существовали в прод-снапшоте, но были потеряны при merge upstream v2026.626.0 (commit 9cf9ab60). companies-route-path-guard.test.ts честно ловил регресс: 3 из 4 тестов ожидали 307, получали 404. Правка идентична уже проверенному фиксу из commit e922a90cc (ветка feature/claude-ci-port-collision, PR #169), cherry-pick принесён без конфликтов.

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

• pnpm exec vitest run server/src/tests/companies-route-path-guard.test.ts -> 4 passed (4)
• tsc --noEmit по server/: ошибок в companies.ts нет (оставшиеся ошибки — предсуществующие, в plugin-host-services.ts/plugin-sdk, не связаны с этим изменением)

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

Если PR #169 смержится первым, эта ветка станет no-op diff при мердже (изменения идентичны) — конфликтов не будет, т.к. содержимое одинаковое.

• Я поискал похожие PR и подтверждаю, что дубликатов нет (PR #169 содержит идентичный фикс, но он ещё не смержен; эта ветка независима и станет no-op при мердже #169 первым).

## Что сделано Восстановлены три company-scoped legacy-alias маршрута в server/src/routes/companies.ts, которые редиректят (307) на новые маршруты: - GET /:companyId/agents/:id/skills -> /api/agents/:id/skills?companyId=... - GET /:companyId/issues/:id -> /api/issues/:id - GET /:companyId/issues/:id/comments -> /api/issues/:id/comments ## Зачем Эти маршруты существовали в прод-снапшоте, но были потеряны при merge upstream v2026.626.0 (commit 9cf9ab60). companies-route-path-guard.test.ts честно ловил регресс: 3 из 4 тестов ожидали 307, получали 404. Правка идентична уже проверенному фиксу из commit e922a90cc (ветка feature/claude-ci-port-collision, PR #169), cherry-pick принесён без конфликтов. ## План тестирования - pnpm exec vitest run server/src/__tests__/companies-route-path-guard.test.ts -> 4 passed (4) - tsc --noEmit по server/: ошибок в companies.ts нет (оставшиеся ошибки — предсуществующие, в plugin-host-services.ts/plugin-sdk, не связаны с этим изменением) ## Где могу ошибаться Если PR #169 смержится первым, эта ветка станет no-op diff при мердже (изменения идентичны) — конфликтов не будет, т.к. содержимое одинаковое. - [x] Я поискал похожие PR и подтверждаю, что дубликатов нет (PR #169 содержит идентичный фикс, но он ещё не смержен; эта ветка независима и станет no-op при мердже #169 первым).

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

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

Три форк-роута с 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-тест.

forgejo-actions commented 2026-07-04 17:43:22 +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 added 2 commits 2026-07-04 18:07:35 +00:00

test(server): add PIX-13011 regression test for legacy company-alias routes ... d03d88c0e4

Explicit regression assertion that all three restored company-scoped
legacy aliases (agents/:id/skills, issues/:id, issues/:id/comments) return
307 and never 404, matching the exact upstream-merge failure mode this
fix addresses (PIX-13012).

merge: sync with master to bring package.json/pre-push scripts up to date (PIX-13012) 5f130ade33

andrei merged commit 3a87b3386f into master 2026-07-04 18:34:36 +00:00

andrei referenced this pull request from a commit 2026-07-04 18:34:36 +00:00

Merge pull request '[PIX-13011] Восстановить legacy company-scoped алиасы редиректов (307 вместо 404)' (#171) from agent/fullstack/pix-13011 into master

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

No description provided.