vvzvlad/gitmost

feat(mcp): внятная диагностика ошибок тулов + fail-fast валидация comment-id (#437, #436) #441

Open

agent_coder wants to merge 4 commits from feat/437-error-diagnostics into develop

Author	SHA1	Message	Date
agent_coder	4809348457	test(mcp): ассертить collab-hint (pageId + transient/retry) в reject-текстах (ревью #441 ) Enrichment collab-ошибок из Phase C (#437) дописывает '(pageId …; transient — retry once; …)' к connect-timeout/connection-closed, но reject-регекспы матчили только базовый текст → проходили и С hint, и БЕЗ (vacuous). Ужесточил два ассерта (connection-closed, connect-timeout) до '<база> (pageId page-1; transient' — теперь рефактор, убравший hint(), их роняет. Мутационно: hint()->'' → ровно эти 2 теста краснеют (18->16). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-07-10 07:14:39 +03:00
agent_coder	d3d32d637b	fix(mcp): no-response диагностика — не отдавать сырой error.message (утечка host в модель) (внутр. ревью #437 ) no-response ветка формата использовала error.code ?? error.message: при отсутствии code axios-сообщения сетевых ошибок содержат host:port ('connect ECONNREFUSED 127.0.0.1:3000', 'getaddrinfo ENOTFOUND host'), что нарушает инвариант #437 «host никогда не попадает в видимое модели сообщение». Теперь только error.code (?? 'network error'); полный нативный текст уходит в stderr под DEBUG. code проставлен фактически для всех реальных no-response ошибок. Тест обновлён: сырое host-содержащее сообщение -> нейтральный reason, плюс ассерт что host в сообщении отсутствует. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-07-10 07:12:37 +03:00
agent_coder	9a435201b8	feat(mcp): enrich collab connect/persist/closed error texts with pageId + retry hint (#437 ) Append `(pageId <id>; transient — retry once; persistent failures mean the collab server is unreachable/overloaded)` to the connect-timeout, persist-timeout and connection-closed error texts in CollabSession, so the agent can self-correct instead of blind-looping. The Yjs-encode error is left untouched (it already names the offending attribute). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-07-10 07:12:37 +03:00
agent_coder	d6827b9210	feat(mcp): actionable tool errors — central axios diagnostics + fail-fast comment-id guard (#437 ) Phase A: add ONE response interceptor on DocmostClient's axios instance, registered AFTER the re-login interceptor, that reformats a failed request's error.message IN PLACE (never a custom Error subclass, so the live axios.isAxiosError / error.response?.status / config._retry checks keep working) into `<METHOD> <path> failed (<status> <statusText>): <serverMessage>`, or the no-response variant. serverMessage is built ONLY from the whitelisted message/error fields or statusText — raw string/HTML bodies, headers and config never appear; an arraybuffer body is size-capped JSON.parsed; the full body goes to stderr only under DEBUG (parity with downloadImage). A _docmostFormatted flag guards against double-processing. Phase B (#436): assertFullUuid throws an actionable error BEFORE any network call at all five commentId sites (resolve/update/delete/get_comment and create_comment's parentCommentId when provided), so a truncated id can no longer loop as an opaque 400/404. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-07-10 07:12:37 +03:00