 agent_coder
|
14d7b21df0
|
refactor(mcp): распил client.ts (5206 строк) на доменные модули за тонким фасадом (#450)
client.ts был god-object'ом на 5206 строк / ~65 методов / 5 ответственностей —
любая правка рисковала всем write-path. Разнесли на доменные модули;
DocmostClient остаётся ТОНКИМ ФАСАДОМ с прежним внешним контрактом. Чистый
рефакторинг, поведение не меняется. closes #450
- Фасад client.ts (93 строки) композирует 10 миксинов над общим абстрактным
базовым классом. Паттерн МИКСИНЫ (не context-object): тесты субклассируют
DocmostClient и переопределяют seam'ы; единая цепочка прототипов сохраняет
виртуальную диспетчеризацию this.<method> сквозь модули.
- client/context.ts (база): общее состояние (axios, apiUrl, токены, кэши),
конструктор + оба интерсептора, login/ensureAuthenticated/paginateAll/
resolvePageId/mutateLiveContentUnlocked + write-seam'ы mutatePage/replacePage.
private→protected (внутреннее, не в публичном контракте).
- Модули (каждый ≤730): read, pages, nodes-write, media (images/attachments/
drawio), comments, transforms, tables, stash, doc-validate. errors.ts —
единый REST error-mapping (довершение #437; тексты сообщений без изменений).
- Внешний контракт СОХРАНЁН (доказано компиляцией с обеих сторон): 53 публичных
метода через `implements IXMixin` (без ручного зеркала, #446); оба Pick
(in-app loader + tool-specs) резолвятся; множество async-методов 59==59.
Нагруженные seam'ы (replaceImage один-лок #425, self-resolve #449, единый
error-путь) байт-идентичны; ни одного дубля публичного метода.
- zod v3→v4: investigate-only, отложено — SDK 1.29 поддерживает v4, но мажор-
бамп трогает всю схема-поверхность; отдельным follow-up.
Стоит на #449 (#475). Тесты: mcp node --test 800/800 (== база), tsc чисто в
client/. Ни одной строки кода не потеряно; убраны 52 дублированных JSDoc-блока.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-07-11 01:04:42 +03:00 |
|