vvzvlad/gitmost
1
0
Fork 0
Code Issues 14 Pull Requests 12 Packages Projects Releases Activity

[test] Тест-стратегия #204: оставшиеся фазы (хвост Ф2 + Ф3 безопасность/DB-харнесс + Ф4 контракты) #245

New Issue
Closed
opened 2026-06-28 03:42:58 +03:00 by Ghost · 0 comments
Ghost commented 2026-06-28 03:42:58 +03:00
Copy Link

Выделено из #204 (закрыт: приоритетный поднабор плана реализован в PR #230 — Фаза 1 + часть Фазы 2). Здесь — остаток тест-стратегии, чтобы план не потерялся.

Уже сделано (PR #230)

  • Фаза 1: mcp-clients.lease.spec.ts (жизненный цикл lease/refcount/eviction MCP-клиента).
  • Фаза 2 (частично): editor-ext table-utils (transpose/moveRow/round-trip) + math-tokenizer false-positive; client emoji-menu (+it.fails на незащищённый JSON.parse), sort-cells, normalizeTableColumnWidths; mcp htmlEmbed/pageBreak data-loss + footnote-diff; server export getInternalLinkPageName (баг исправлен + тест).

Остаток

Фаза 2 (хвост, data-integrity чистых функций)

  • ai-chat: экспорт+тест SSRF guardedFetch/lookup, decideClose; decryptHeaders fail-open.
  • editor-ext: recreateTransform-инвариант (apply(diff)=target), moveRow/moveColumn, getSelectionRangeInColumn, addUniqueIdsToDoc.
  • client: ordered-emitter диктовки, paste-хелперы, decideTurnEnd/sendNow (после R-рефакторов).
  • mcp: round-trip медиа (video/youtube/embed/excalidraw/audio/pdf), cookie-парсинг performLogin, applyAnchorInDoc.

Фаза 3 — серверная безопасность/доступ (нужен DB-харнесс Postgres + R-core)

  • getShareForPage наследование share, isSharingAllowed toggle, getShareTree restricted-ancestor, lookupTransclusionForShare access-граф, acceptInvitation атомарность.
  • AiSettingsService (секреты write-only, неутечка ключей), environment.validation (после снятия process.exit).
  • common: yjs.util (setYjsMark/removeYjsMarkByAttribute/updateYjsMarkAttribute) — 0 тестов.

Фаза 4 — добивка и контракты

  • client tree-reducers/utils (applyUpdateOne, tree-utils, formatRelativeTime), transclusion-lookup-context.
  • contract-тесты: паритет SHARED_TOOL_SPECS ↔ in-app/MCP; дрейф recreate-transform (vendored editor-ext vs npm в mcp/diff.ts).
  • замена тавтологичных спеков (storage.service.spec.ts, environment.service.spec.ts) на проверки делегирования.
  • завести issue/фикс на findBreadcrumbPath-мутацию входа.

Инфраструктура

  • Поставить @vitest/coverage-v8 и/или починить istanbul на ESM-импорте @docmost/editor-ext, ввести порог покрытия (coverage-gate сейчас отсутствует).
  • DB-харнесс (Postgres) — блокирует 5 share/invitation integration-тестов.

Полный детальный план по модулям — в #204.

> Выделено из #204 (закрыт: приоритетный поднабор плана реализован в PR #230 — Фаза 1 + часть Фазы 2). Здесь — **остаток** тест-стратегии, чтобы план не потерялся. ## Уже сделано (PR #230) - **Фаза 1:** `mcp-clients.lease.spec.ts` (жизненный цикл lease/refcount/eviction MCP-клиента). - **Фаза 2 (частично):** editor-ext table-utils (transpose/moveRow/round-trip) + math-tokenizer false-positive; client emoji-menu (+`it.fails` на незащищённый `JSON.parse`), sort-cells, `normalizeTableColumnWidths`; mcp htmlEmbed/pageBreak data-loss + footnote-diff; server export `getInternalLinkPageName` (баг исправлен + тест). ## Остаток ### Фаза 2 (хвост, data-integrity чистых функций) - ai-chat: экспорт+тест SSRF `guardedFetch`/`lookup`, `decideClose`; `decryptHeaders` fail-open. - editor-ext: `recreateTransform`-инвариант (apply(diff)=target), `moveRow`/`moveColumn`, `getSelectionRangeInColumn`, `addUniqueIdsToDoc`. - client: ordered-emitter диктовки, paste-хелперы, `decideTurnEnd`/`sendNow` (после R-рефакторов). - mcp: round-trip медиа (`video/youtube/embed/excalidraw/audio/pdf`), cookie-парсинг `performLogin`, `applyAnchorInDoc`. ### Фаза 3 — серверная безопасность/доступ (нужен DB-харнесс Postgres + R-core) - `getShareForPage` наследование share, `isSharingAllowed` toggle, `getShareTree` restricted-ancestor, `lookupTransclusionForShare` access-граф, `acceptInvitation` атомарность. - `AiSettingsService` (секреты write-only, неутечка ключей), `environment.validation` (после снятия `process.exit`). - common: `yjs.util` (`setYjsMark`/`removeYjsMarkByAttribute`/`updateYjsMarkAttribute`) — 0 тестов. ### Фаза 4 — добивка и контракты - client tree-reducers/utils (`applyUpdateOne`, tree-utils, `formatRelativeTime`), `transclusion-lookup-context`. - contract-тесты: паритет `SHARED_TOOL_SPECS` ↔ in-app/MCP; дрейф `recreate-transform` (vendored editor-ext vs npm в `mcp/diff.ts`). - замена тавтологичных спеков (`storage.service.spec.ts`, `environment.service.spec.ts`) на проверки делегирования. - завести issue/фикс на `findBreadcrumbPath`-мутацию входа. ### Инфраструктура - Поставить `@vitest/coverage-v8` и/или починить istanbul на ESM-импорте `@docmost/editor-ext`, ввести порог покрытия (coverage-gate сейчас отсутствует). - DB-харнесс (Postgres) — блокирует 5 share/invitation integration-тестов. Полный детальный план по модулям — в #204.
Ghost added the test label 2026-06-28 03:42:58 +03:00
Ghost closed this issue 2026-06-28 03:46:13 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 epic
Large multi-phase effort spanning many changes
 feature
New functionality request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 idea
Idea / proposal for discussion
 invalid
This doesn't seem right
 needs-human
эскалация: нужно решение человека
 question
Further information is requested
 refactor
Code cleanup / refactoring
 review/approved
в последнем ревью нет открытых blocking-находок
 review/changes-requested
последнее ревью оставило открытые blocking-находки
 review/needs
head не ревьюился (head != reviewed_head)
 security
Security / hardening issue
 status/blocked
ждёт зависимость blocked_by
 status/done
закрыто и проверено
 status/in-progress
в активной работе (мягкая заявка)
 status/ready
специфицировано, не заблокировано, ждёт исполнителя
 test
Test coverage / test infrastructure
 wontfix
This will not be worked on
No Label test
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent_coder agent_reviewer vvzvlad
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vvzvlad/gitmost#245
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?