page-templates: спеки transclusion не компилируются (TS2554, 11 vs 10 аргументов) — 0 тестов, проверки безопасности отключены #79

New Issue

Ghost · 2026-06-21T02:33:11+03:00

Ghost commented

2026-06-21 02:33:11 +03:00

Найдено в multi-aspect code review всех изменений с коммита 053a9c0d (ветка develop).

Грань: test-coverage / build · Severity: warning
Где: apps/server/src/core/page/transclusion/spec/page-template-access.spec.ts:60,209; page-template-lookup.spec.ts:26; transclusion-unsync-html-embed.spec.ts:55

Проблема
Хардненинг-коммит добавил 11-й параметр конструктора workspaceRepo (transclusion.service.ts:68), но спеки создают сервис с 10 позиционными аргументами. Подтверждено запуском из apps/server: «TS2554: Expected 11 arguments, but got 10», Test Suites: 1 failed, Tests: 0 total (контрольная ai-roles спека — 14 passed, т.е. харнесс рабочий). Это отключает все ключевые проверки безопасности фичи: отбрасывание кросс-workspace записей page_template_references, фильтр доступа, срез comment-марок, admin-гейт среза htmlEmbed.

Предлагаемый фикс
Добавить 11-й аргумент workspaceRepo во все 4 места. ВНИМАНИЕ: в transclusion-unsync-html-embed.spec.ts:55 аргументы позиционные и сдвинуты — выровнять весь порядок (db, pageTransclusionsRepo, pageTransclusionReferencesRepo, pageTemplateReferencesRepo, pageRepo, pagePermissionRepo, spaceMemberRepo, attachmentRepo, storageService, pageAccessService, workspaceRepo). Перезапустить: cd apps/server && npx jest src/core/page/transclusion/spec.

Связанные: #56

Найдено в multi-aspect code review всех изменений с коммита `053a9c0d` (ветка develop). **Грань:** test-coverage / build · **Severity:** warning **Где:** `apps/server/src/core/page/transclusion/spec/page-template-access.spec.ts:60,209; page-template-lookup.spec.ts:26; transclusion-unsync-html-embed.spec.ts:55` **Проблема** Хардненинг-коммит добавил 11-й параметр конструктора workspaceRepo (transclusion.service.ts:68), но спеки создают сервис с 10 позиционными аргументами. Подтверждено запуском из apps/server: «TS2554: Expected 11 arguments, but got 10», Test Suites: 1 failed, Tests: 0 total (контрольная ai-roles спека — 14 passed, т.е. харнесс рабочий). Это отключает все ключевые проверки безопасности фичи: отбрасывание кросс-workspace записей page_template_references, фильтр доступа, срез comment-марок, admin-гейт среза htmlEmbed. **Предлагаемый фикс** Добавить 11-й аргумент workspaceRepo во все 4 места. ВНИМАНИЕ: в transclusion-unsync-html-embed.spec.ts:55 аргументы позиционные и сдвинуты — выровнять весь порядок (db, pageTransclusionsRepo, pageTransclusionReferencesRepo, pageTemplateReferencesRepo, pageRepo, pagePermissionRepo, spaceMemberRepo, attachmentRepo, storageService, pageAccessService, workspaceRepo). Перезапустить: `cd apps/server && npx jest src/core/page/transclusion/spec`. **Связанные:** #56

Ghost referenced this issue

2026-06-21 02:33:45 +03:00

test (сквозное): анти-паттерн «спека тестирует копию продакшен-логики» + неработающие сьюты security-фич #96

Ghost closed this issue

2026-06-21 03:02:11 +03:00

~~Ghost referenced this issue 2026-06-21 04:15:11 +03:00~~

test (сквозное): анти-паттерн «спека тестирует копию продакшен-логики» + неработающие сьюты security-фич #96

Sign in to join this conversation.