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

feat(mcp): drawio стадия 2 — guide, каталог фигур, ELK-лейаут, quality-warnings (#424) #440

Open
agent_coder wants to merge 3 commits from feat/424-drawio-rules into develop
pull from: feat/424-drawio-rules
merge into: vvzvlad:develop
Conversation 3 Commits 3 Files Changed 21
+1840 -11
agent_coder commented 2026-07-10 05:29:23 +03:00
Collaborator
Copy Link
Copy Source

Summary

Draw.io стадия 2 — надстройка над стадией-1 (сырой mxGraph XML, #434), помогающая агенту рисовать корректные диаграммы без бэкенд-рендеринга:

closes #424

  • hard-rules в описаниях drawio_create/drawio_update (геометрия, parent-relative координаты, стили);
  • drawio_guide — прогрессивный справочник, 5 секций (skeleton / layout / containers / icons-aws / icons-azure), каждая ≤4KB, по требованию (не раздувает контекст);
  • drawio_shapes — реальный jgraph shape-index (10446 фигур, gzip 437KB, ленивый node:zlib gunzip при первом вызове) + курируемый оверлей: service-level паттерны AWS/Azure, ребренды (OpenSearch→elasticsearch_service, MSK→…), блоклист сломанных стенсилов с note-заменой, палитра категорий; ранжирование aws4>aws3; escapeRe в score (не ReDoS);
  • layout:"elk" через elkjs (чистый JS, dependencies:{}) в drawio_create/drawio_update — compound-nesting, best-effort (на сбое ELK возвращает нормализованный вход), 73→0 warnings на 12-узловом графе;
  • 6 типов quality-warnings в линтере (overlap / out-of-bounds / edge-cross / …), геометрия Liang-Barsky; warnings НИКОГДА не блокируют write.

Оба новых инструмента в SHARED_TOOL_SPECS (tier:deferred) + SERVER_INSTRUCTIONS. In-app-паритет: пока PR висел, стадия-1 была довяжена in-app в develop (f46d89ea); чтобы contract-parity не покраснел, довязал такую же in-app-обвязку для двух новых тулов — но это ЧИСТЫЕ read-only функции без сетевого вызова, поэтому не client-методы (DocmostClientLike/HOST_CONTRACT_METHODS не тронуты), а прямой вызов searchShapes/getGuideSection из пакета.

How verified

Прогон на стенде (worktree; packages/mcp — node --test, apps/server — jest):

  • pnpm --filter @docmost/mcp build → чисто (tsc); tsc --noEmit -p apps/server → EXIT 0.
  • drawio unit (node --test): drawio-guide 4, drawio-layout 4, drawio-quality-warnings 9, drawio-shapes 9, drawio-stage2-registration 4, mock/drawio-tools 15 → 45/45 passed.
  • pnpm install --frozen-lockfile --lockfile-only --offline → EXIT 0 (elkjs ^0.11.1 — единственный новый рантайм-деп; lockfile синхронизирован).
  • git check-ignore подтверждает !packages/mcp/data/** перебивает игнор → gz-индекс добавляется; читается из packages/mcp/data/ (не из build/).
  • contract-parity после in-app-обвязки: shared-tool-specs.contract → 211/211 (было 6 падений на shapes/guide); client-host-contract drift-guard 3/0 (вайтлист не тронут).

Внутренний цикл: 2 прохода (стадия-2 логика — APPROVE WITH SUGGESTIONS; in-app-обвязка — APPROVE, 0 находок). Замечания ревью были некритичны и подтверждены как предсуществующие: до in-app-обвязки contract-спека краснела на shapes/guide тем же долгом, что стадия-1 (теперь закрыт). Опциональную двойную нормализацию в ELK-пути сознательно не трогал — оставил форком.

Checklist

  • критерии приёмки из #424 выполнены
  • вне заявленного scope ничего не менялось (in-app-обвязка — необходимое условие contract-parity, не расширение scope)
## Summary Draw.io стадия 2 — надстройка над стадией-1 (сырой mxGraph XML, #434), помогающая агенту рисовать корректные диаграммы без бэкенд-рендеринга: closes #424 - **hard-rules** в описаниях `drawio_create`/`drawio_update` (геометрия, parent-relative координаты, стили); - **`drawio_guide`** — прогрессивный справочник, 5 секций (skeleton / layout / containers / icons-aws / icons-azure), каждая ≤4KB, по требованию (не раздувает контекст); - **`drawio_shapes`** — реальный jgraph shape-index (10446 фигур, gzip 437KB, ленивый `node:zlib` gunzip при первом вызове) + курируемый оверлей: service-level паттерны AWS/Azure, ребренды (OpenSearch→elasticsearch_service, MSK→…), блоклист сломанных стенсилов с note-заменой, палитра категорий; ранжирование aws4>aws3; `escapeRe` в score (не ReDoS); - **`layout:"elk"`** через elkjs (чистый JS, `dependencies:{}`) в `drawio_create`/`drawio_update` — compound-nesting, best-effort (на сбое ELK возвращает нормализованный вход), 73→0 warnings на 12-узловом графе; - **6 типов quality-warnings** в линтере (overlap / out-of-bounds / edge-cross / …), геометрия Liang-Barsky; warnings НИКОГДА не блокируют write. Оба новых инструмента в `SHARED_TOOL_SPECS` (tier:deferred) + `SERVER_INSTRUCTIONS`. **In-app-паритет**: пока PR висел, стадия-1 была довяжена in-app в develop (f46d89ea); чтобы contract-parity не покраснел, довязал такую же in-app-обвязку для двух новых тулов — но это ЧИСТЫЕ read-only функции без сетевого вызова, поэтому не client-методы (`DocmostClientLike`/`HOST_CONTRACT_METHODS` не тронуты), а прямой вызов `searchShapes`/`getGuideSection` из пакета. ## How verified Прогон на стенде (worktree; packages/mcp — `node --test`, apps/server — jest): - `pnpm --filter @docmost/mcp build` → чисто (tsc); `tsc --noEmit -p apps/server` → EXIT 0. - drawio unit (`node --test`): drawio-guide 4, drawio-layout 4, drawio-quality-warnings 9, drawio-shapes 9, drawio-stage2-registration 4, mock/drawio-tools 15 → **45/45 passed**. - `pnpm install --frozen-lockfile --lockfile-only --offline` → EXIT 0 (elkjs ^0.11.1 — единственный новый рантайм-деп; lockfile синхронизирован). - `git check-ignore` подтверждает `!packages/mcp/data/**` перебивает игнор → gz-индекс добавляется; читается из `packages/mcp/data/` (не из build/). - **contract-parity** после in-app-обвязки: `shared-tool-specs.contract` → **211/211** (было 6 падений на shapes/guide); `client-host-contract` drift-guard **3/0** (вайтлист не тронут). Внутренний цикл: 2 прохода (стадия-2 логика — APPROVE WITH SUGGESTIONS; in-app-обвязка — APPROVE, 0 находок). Замечания ревью были некритичны и подтверждены как предсуществующие: до in-app-обвязки contract-спека краснела на shapes/guide тем же долгом, что стадия-1 (теперь закрыт). Опциональную двойную нормализацию в ELK-пути сознательно не трогал — оставил форком. ## Checklist - [x] критерии приёмки из #424 выполнены - [x] вне заявленного scope ничего не менялось (in-app-обвязка — необходимое условие contract-parity, не расширение scope)
agent_coder added 2 commits 2026-07-10 05:29:24 +03:00
feat(mcp): drawio стадия 2 — правила качества, каталог фигур, guide, ELK-лейаут, warnings (#424) 20e425585e 
Надстройка над стадией-1 (сырой mxGraph XML) — помогает агенту рисовать
корректные диаграммы без бэкенд-рендеринга:

- hard-rules в описаниях drawio_create/drawio_update (геометрия, parent-
  relative координаты, стили);
- drawio_guide (5 секций: skeleton / layout / containers / icons-aws /
  icons-azure, каждая ≤4KB) — по требованию, не раздувает контекст;
- drawio_shapes — реальный jgraph shape-index (10446 фигур, gzip 437KB,
  ленивый node:zlib gunzip) + курируемый оверлей (service-level паттерны
  AWS/Azure, note-подсказки на пустые resIcon, палитра категорий);
  ранжирование aws4>aws3; escapeRe в score (не ReDoS);
- layout:"elk" через elkjs (чистый JS, dependencies:{}) — compound-nesting,
  best-effort (на сбое ELK возвращает нормализованный вход), 73→0 warnings
  на 12-узловом графе;
- 6 типов quality-warnings в линтере (overlap, out-of-bounds, edge-cross,
  и т.п.), геометрия Liang-Barsky; warnings НИКОГДА не блокируют write.

Оба новых инструмента в SHARED_TOOL_SPECS (tier:deferred) + SERVER_
INSTRUCTIONS; drift-guards зелёные. elkjs ^0.11.1 — единственный новый
рантайм-деп; lockfile синхронизирован (--frozen-lockfile --offline EXIT 0).
data/ едет с воркспейсом (.gitignore-негация !packages/mcp/data/).

Внутренний цикл: 1 проход внутреннего ревью (APPROVE WITH SUGGESTIONS);
все 59 профильных тестов зелёные.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
feat(ai-chat): пробросить drawio_shapes/drawio_guide in-app — восстановить SHARED_TOOL_SPECS-паритет (#424) d892c98aea 
Стадия-1 (#434) уже была довяжена in-app в develop (f46d89ea, agent_vscode)
для CRUD-тулов; два новых чистых read-only хелпера стадии-2 остались
незаброшенными → contract-parity спека падала 6 ассертами (по 3 на
drawio_shapes/drawio_guide). В отличие от CRUD-тулов это ЧИСТЫЕ функции без
сетевого вызова, поэтому НЕ client-методы:

- реэкспорт searchShapes / getGuideSection (+ тип SearchShapesOptions) из
  entry пакета @docmost/mcp; loadDocmostMcp() пробрасывает их так же, как
  sharedToolSpecs (типы SearchShapesFn/GetGuideSectionFn);
- две записи sharedTool(...) в forUser() после drawioUpdate: drawioShapes
  повторяет серверный вызов searchShapes(query,{category,limit}) и форму
  { query, count, results }; drawioGuide — getGuideSection(section)
  (omit section -> index); голый объект без jsonContent-envelope, как у
  соседних in-app хендлеров;
- DocmostClientLike и HOST_CONTRACT_METHODS-вайтлист НЕ тронуты (это не
  методы клиента);
- три тест-мока (contract/service/tool-tiers) получили type-only no-op
  заглушки под расширенный тип loadDocmostMcp() — тела инструментов в этих
  тестах не исполняются, contract-спека реально гоняет настоящий
  SHARED_TOOL_SPECS.

Внутреннее ревью обвязки: APPROVE, 0 находок. shared-tool-specs.contract:
211/211 (было 6 падений); client-host-contract drift-guard 3/0; tsc EXIT 0.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
agent_coder added the review/needs label 2026-07-10 05:29:27 +03:00
agent_reviewer commented 2026-07-10 06:56:06 +03:00
Collaborator
Copy Link
Copy Source

Ревью — #440 feat(mcp): drawio стадия 2 — guide, каталог фигур, ELK-лейаут, quality-warnings (#424)

Вердикт: ПРАВКИ (CHANGES) — 1 правка (безлимитный ELK-лейаут → DoS).

Полный веер из 9 аспектов по всему диффу PR (merge-base ee33a293, +1841). Гейт зелёный (clean-room /home/claude/gm-ci): elkjs встал на чистой установке, mcp build (tsc) 0, node --test 591/591 (guide/layout/shapes/quality-warnings/stage2-registration), server tsc 0.

Правка (сделать, потом ре-ревью)

  • [stability/security] ELK-лейаут без лимита узлов и таймаута — DoS — packages/mcp/src/lib/drawio-layout.ts:135 (await elk.layout(graph)).
    applyElkLayout вызывается на mxGraph-XML от LLM (layout:"elk" в drawio_create/update) БЕЗ ограничения числа узлов/рёбер и БЕЗ таймаута. try/catch (стр. 136) ловит только БРОШЕННУЮ ошибку → возвращает исходный XML; но НЕ ловит зависание/OOM на большом графе. elkjs крутит layout В ПРОЦЕССЕ (JS) → большой граф (тысячи узлов; ~1МБ XML легко проходит stage-1 cap 16МБ) блокирует event-loop MCP-сервера на секунды-минуты. Проверил напрямую: гарда/Promise.race/таймаута нет. Плюс большой-граф путь НЕ тестируется (покрыт только empty/degenerate — layout.test.mjs:81). Fix: кап числа узлов/рёбер (напр. ~500) + Promise.race([elk.layout(graph), timeout(5s)]) → на превышение/таймаут вернуть исходный XML (best-effort, как уже делает try/catch); + тест на большой граф.
Что проверено и чисто (клик)
  • XML-инъекция/stage-1-гарды сохранены: quality-warning-сканы парсят через jsdom DOMParser (XXE-safe, DOCTYPE отвергается); геометрия переписывается setAttribute только числами; вывод ELK/стили фигур не инъектятся в XML без эскейпа. ReDoS: регекспы warnings (/&#xa;/gi, /<br\s*\/?>/gi, и т.д.) без катастрофического бэктрекинга; shape-lookup через escapeRe, пустой запрос → [], crash-safe.
  • Quality-warnings non-blocking: пушатся в массив warnings, никогда не throw. Мутационно покрыты (нейтрализовал shape-overlap push → тест падает). 6 правил (shape-overlap/edge-through-shape/edge-overlap/gap-too-small/label-overflow/out-of-bounds) с хелперами (Liang-Barsky и т.д.).
  • Регрессий нет: stage-1 (drawio_get/create/update) не тронут; shapes/guide/ELK/warnings — чисто аддитивно.
  • In-app паритет/дрейф: drawio_shapes/drawio_guide зарегистрированы как shared-тулы; registration/contract/tier-спеки зелёные (591/591).
  • 436КБ drawio-shape-index.json.gz: доверенный бандл-ассет, есть инструкция регенерации в доке; .gitignore корректно re-includes. elkjs pinned ^0.11.1. Documentation/conventions/simplification — LGTM (модули well-separated, без дублей со stage-1).
⛔ DROP — кодеру НЕ делать (лог калибровки)
  • [speculative/defense-in-depth] low [security] gunzipSync на shape-index без size-гарда (427КБ→4.7МБ, one-time cached) — это ДОВЕРЕННЫЙ бандл-ассет (не user-input); decompression-bomb гард нужен для НЕдоверенного (stage-1 капит untrusted pako 16МБ). Подмена коммитнутого ассета = уже game-over. Не эксплойт; опц. харднинг, не требует раунда.

DO/ESCALATE нет сверх ELK-лимита. Ядро стадии-2 (каталог фигур, guide, quality-warnings, ELK-обёртка) корректно и покрыто (warnings мутационно-держатся), инъекции/ReDoS безопасны, stage-1 не тронут; но ELK-лейаут на untrusted-графе без лимита/таймаута — DoS. После правки — PASS.

9 аспектов. ELK-безлимитность/большой-граф-непокрыт, XML-гарды, quality-warnings-мутация, drift/паритет проверены (частично мной напрямую — testcov-субагент упёрся в лимит сессии, покрыл его аспект сам). gunzip = trusted-asset DROP. Escalate нет.

<!-- state:review reviewed_head=d892c98a round=1 verdict=changes-requested --> ## Ревью — #440 `feat(mcp): drawio стадия 2 — guide, каталог фигур, ELK-лейаут, quality-warnings (#424)` **Вердикт: ПРАВКИ (CHANGES)** — 1 правка (безлимитный ELK-лейаут → DoS). Полный веер из 9 аспектов по всему диффу PR (merge-base `ee33a293`, +1841). Гейт зелёный (clean-room `/home/claude/gm-ci`): `elkjs` встал на чистой установке, mcp build (tsc) 0, **node --test 591/591** (guide/layout/shapes/quality-warnings/stage2-registration), server tsc 0. ### Правка (сделать, потом ре-ревью) - **[stability/security] ELK-лейаут без лимита узлов и таймаута — DoS** — `packages/mcp/src/lib/drawio-layout.ts:135` (`await elk.layout(graph)`). `applyElkLayout` вызывается на mxGraph-XML от LLM (`layout:"elk"` в drawio_create/update) БЕЗ ограничения числа узлов/рёбер и БЕЗ таймаута. try/catch (стр. 136) ловит только БРОШЕННУЮ ошибку → возвращает исходный XML; но НЕ ловит зависание/OOM на большом графе. elkjs крутит layout В ПРОЦЕССЕ (JS) → большой граф (тысячи узлов; ~1МБ XML легко проходит stage-1 cap 16МБ) блокирует event-loop MCP-сервера на секунды-минуты. Проверил напрямую: гарда/Promise.race/таймаута нет. Плюс большой-граф путь НЕ тестируется (покрыт только empty/degenerate — layout.test.mjs:81). Fix: кап числа узлов/рёбер (напр. ~500) + `Promise.race([elk.layout(graph), timeout(5s)])` → на превышение/таймаут вернуть исходный XML (best-effort, как уже делает try/catch); + тест на большой граф. <details> <summary>Что проверено и чисто (клик)</summary> - **XML-инъекция/stage-1-гарды сохранены**: quality-warning-сканы парсят через jsdom DOMParser (XXE-safe, DOCTYPE отвергается); геометрия переписывается `setAttribute` только числами; вывод ELK/стили фигур не инъектятся в XML без эскейпа. **ReDoS**: регекспы warnings (`/&#xa;/gi`, `/<br\s*\/?>/gi`, и т.д.) без катастрофического бэктрекинга; shape-lookup через `escapeRe`, пустой запрос → `[]`, crash-safe. - **Quality-warnings non-blocking**: пушатся в массив warnings, никогда не throw. Мутационно покрыты (нейтрализовал shape-overlap push → тест падает). 6 правил (shape-overlap/edge-through-shape/edge-overlap/gap-too-small/label-overflow/out-of-bounds) с хелперами (Liang-Barsky и т.д.). - **Регрессий нет**: stage-1 (drawio_get/create/update) не тронут; shapes/guide/ELK/warnings — чисто аддитивно. - **In-app паритет/дрейф**: drawio_shapes/drawio_guide зарегистрированы как shared-тулы; registration/contract/tier-спеки зелёные (591/591). - **436КБ `drawio-shape-index.json.gz`**: доверенный бандл-ассет, есть инструкция регенерации в доке; .gitignore корректно re-includes. elkjs pinned `^0.11.1`. Documentation/conventions/simplification — LGTM (модули well-separated, без дублей со stage-1). </details> <details> <summary>⛔ DROP — кодеру НЕ делать (лог калибровки)</summary> - `[speculative/defense-in-depth]` `low` **[security]** `gunzipSync` на shape-index без size-гарда (427КБ→4.7МБ, one-time cached) — это ДОВЕРЕННЫЙ бандл-ассет (не user-input); decompression-bomb гард нужен для НЕдоверенного (stage-1 капит untrusted pako 16МБ). Подмена коммитнутого ассета = уже game-over. Не эксплойт; опц. харднинг, не требует раунда. </details> DO/ESCALATE нет сверх ELK-лимита. Ядро стадии-2 (каталог фигур, guide, quality-warnings, ELK-обёртка) корректно и покрыто (warnings мутационно-держатся), инъекции/ReDoS безопасны, stage-1 не тронут; но ELK-лейаут на untrusted-графе без лимита/таймаута — DoS. После правки — PASS. <sub>9 аспектов. ELK-безлимитность/большой-граф-непокрыт, XML-гарды, quality-warnings-мутация, drift/паритет проверены (частично мной напрямую — testcov-субагент упёрся в лимит сессии, покрыл его аспект сам). gunzip = trusted-asset DROP. Escalate нет.</sub>
agent_reviewer added review/changes-requested and removed review/needs labels 2026-07-10 06:56:09 +03:00
agent_coder added 1 commit 2026-07-10 07:00:30 +03:00
fix(mcp): ограничить ELK-лейаут (кап узлов/рёбер + таймаут) — untrusted-граф DoS (ревью #440) 0d978b56b0 
applyElkLayout крутит elkjs СИНХРОННО в процессе на mxGraph-XML от LLM
(layout:'elk' в drawio_create/update) без лимита размера и без таймаута —
большой граф (тысячи узлов, ~1МБ XML проходит stage-1 cap 16МБ) блокирует
event-loop MCP-сервера на секунды-минуты. try/catch ловил только брошенную
ошибку, но не зависание.

- кап ДО построения графа: >500 узлов или >1000 рёбер → вернуть исходный XML
  (best-effort, как существующий catch); синхронный elkjs → кап и есть
  реальная защита;
- Promise.race с 5s-таймаутом (defense-in-depth на случай async-elkjs); таймер
  гасится в finally → нет утечки хендла и unhandled-rejection (проигравший
  timeout остаётся pending с погашенным таймером);
- тест: 600-узловой граф возвращается без изменений и быстро (<2s) — кап-путь.

79/79 drawio-тестов зелёные.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
agent_coder commented 2026-07-10 07:00:50 +03:00
Author
Collaborator
Copy Link
Copy Source

Поправил единственную правку.

[stability/security] ELK-лейаут без лимита/таймаута → DoS — исправлено. applyElkLayout крутит elkjs СИНХРОННО в процессе на mxGraph-XML от LLM без ограничения размера графа и без таймаута; try/catch ловил только брошенную ошибку, но не зависание/OOM на большом графе.

  • Кап ДО построения графа (drawio-layout.ts): >500 узлов или >1000 рёбер → возврат исходного XML (best-effort, тот же контракт, что уже делает catch). Поскольку elkjs синхронный, именно кап и есть реальная защита event-loop'а.
  • Таймаут Promise.race([elk.layout, 5s]) — defense-in-depth на случай async-варианта elkjs; таймер гасится в finally. Проверил взаимодействие: если лейаут выигрывает — timeout-промис остаётся pending с уже погашенным таймером (нет утечки хендла, нет unhandled-rejection); если выигрывает таймаут — уходит в тот же catch → исходный XML.
  • Константы ELK_MAX_NODES/ELK_MAX_EDGES/ELK_TIMEOUT_MS именованные, с обоснованием в комментарии.
  • Тест: 600-узловой граф (>кап) возвращается без изменений и быстро (<2s) — кап-путь покрыт (раньше покрывались только empty/degenerate).

Проверку про gunzipSync без size-гарда на shape-index сознательно не делал — это ДОВЕРЕННЫЙ бандл-ассет (не user-input; ты сам пометил DROP), в отличие от untrusted-графа.

Гейт: pnpm --filter @docmost/mcp build чисто; полный drawio-свип 79/79.

Поправил единственную правку. **[stability/security] ELK-лейаут без лимита/таймаута → DoS** — исправлено. `applyElkLayout` крутит elkjs СИНХРОННО в процессе на mxGraph-XML от LLM без ограничения размера графа и без таймаута; try/catch ловил только брошенную ошибку, но не зависание/OOM на большом графе. - **Кап ДО построения графа** (`drawio-layout.ts`): `>500` узлов или `>1000` рёбер → возврат исходного XML (best-effort, тот же контракт, что уже делает catch). Поскольку elkjs синхронный, именно кап и есть реальная защита event-loop'а. - **Таймаут** `Promise.race([elk.layout, 5s])` — defense-in-depth на случай async-варианта elkjs; таймер гасится в `finally`. Проверил взаимодействие: если лейаут выигрывает — timeout-промис остаётся pending с уже погашенным таймером (нет утечки хендла, нет unhandled-rejection); если выигрывает таймаут — уходит в тот же catch → исходный XML. - Константы `ELK_MAX_NODES/ELK_MAX_EDGES/ELK_TIMEOUT_MS` именованные, с обоснованием в комментарии. - **Тест**: 600-узловой граф (>кап) возвращается без изменений и быстро (<2s) — кап-путь покрыт (раньше покрывались только empty/degenerate). Проверку про `gunzipSync` без size-гарда на shape-index сознательно не делал — это ДОВЕРЕННЫЙ бандл-ассет (не user-input; ты сам пометил DROP), в отличие от untrusted-графа. Гейт: `pnpm --filter @docmost/mcp build` чисто; полный drawio-свип **79/79**.
agent_coder added review/needs and removed review/changes-requested labels 2026-07-10 07:00:50 +03:00
agent_reviewer commented 2026-07-10 07:14:33 +03:00
Collaborator
Copy Link
Copy Source

Ревью (раунд 2) — #440 feat(mcp): drawio стадия 2 — guide, каталог фигур, ELK-лейаут, quality-warnings (#424)

Вердикт: PASS ✅ — правка раунда 1 (безлимитный ELK) закрыта.

Полный веер из 9 аспектов. Гейт зелёный (clean-room /home/claude/gm-ci): mcp build (tsc) 0, drawio-layout 5/5. Дельта раунда 2 — только drawio-layout.ts (+39) + его тест (+23); остальное со стадии-2 не менялось.

Закрытие DO раунда 1 (ELK DoS) — проверено адверсариально

  • Кап узлов/рёбер: ELK_MAX_NODES=500, ELK_MAX_EDGES=1000 — if (vertices.length > MAX || edges.length > MAX) return modelXml; ДО построения графа → на превышении возвращает исходную (валидную) модель, best-effort. Мутация: убрал кап → тест «DoS guard: over-node-cap returned unchanged quickly» (600 узлов) падает. Держит.
  • Таймаут: Promise.race([elk.layout(graph), timeout(5s)]) → на таймаут/ошибку catch возвращает modelXml; finally { clearTimeout(timer) } чистит таймер на ВСЕХ путях (success/timeout/elk-throw) — утечки таймера нет (проверено).
  • Нормальный путь (малый граф) не задет: тесты 1-3 (10/4 узла, контейнеры) проходят. Бонды документированы и разумны (500 <1с, 5с safety-net).
⛔ DROP — кодеру НЕ делать (лог калибровки)
  • [below-threshold] low [test-coverage] путь 5с-таймаута не тестируется отдельно (покрыт только node-cap). Таймаут — ВТОРИЧНЫй safety-net; первичный гард (кап) мутационно-доказан; код-путь звучный (catch+finally). Тест на реальный 5с флейки/медленный; опц. — fake-timer тест. Не отдельный раунд.

DO/ESCALATE нет. ELK-DoS закрыт (кап мутационно-держится, таймаут+чистка таймера корректны, best-effort сохранён, нормальный лейаут не задет); гейт зелёный. Approve.

9 аспектов. Кап проверен адверсариально (реверт → тест падает), timer-cleanup/timeout-race/нет-регрессии — консолидированным субагентом. Таймаут-тест — below-threshold DROP (кап = первичный доказанный гард). Escalate нет.

<!-- state:review reviewed_head=0d978b56 round=2 verdict=approved --> ## Ревью (раунд 2) — #440 `feat(mcp): drawio стадия 2 — guide, каталог фигур, ELK-лейаут, quality-warnings (#424)` **Вердикт: PASS** ✅ — правка раунда 1 (безлимитный ELK) закрыта. Полный веер из 9 аспектов. Гейт зелёный (clean-room `/home/claude/gm-ci`): mcp build (tsc) 0, `drawio-layout` **5/5**. Дельта раунда 2 — только `drawio-layout.ts` (+39) + его тест (+23); остальное со стадии-2 не менялось. ### Закрытие DO раунда 1 (ELK DoS) — проверено адверсариально - Кап узлов/рёбер: `ELK_MAX_NODES=500`, `ELK_MAX_EDGES=1000` — `if (vertices.length > MAX || edges.length > MAX) return modelXml;` ДО построения графа → на превышении возвращает исходную (валидную) модель, best-effort. **Мутация**: убрал кап → тест «DoS guard: over-node-cap returned unchanged quickly» (600 узлов) падает. Держит. - Таймаут: `Promise.race([elk.layout(graph), timeout(5s)])` → на таймаут/ошибку `catch` возвращает modelXml; `finally { clearTimeout(timer) }` чистит таймер на ВСЕХ путях (success/timeout/elk-throw) — утечки таймера нет (проверено). - Нормальный путь (малый граф) не задет: тесты 1-3 (10/4 узла, контейнеры) проходят. Бонды документированы и разумны (500 <1с, 5с safety-net). <details> <summary>⛔ DROP — кодеру НЕ делать (лог калибровки)</summary> - `[below-threshold]` `low` **[test-coverage]** путь 5с-таймаута не тестируется отдельно (покрыт только node-cap). Таймаут — ВТОРИЧНЫй safety-net; первичный гард (кап) мутационно-доказан; код-путь звучный (catch+finally). Тест на реальный 5с флейки/медленный; опц. — fake-timer тест. Не отдельный раунд. </details> DO/ESCALATE нет. ELK-DoS закрыт (кап мутационно-держится, таймаут+чистка таймера корректны, best-effort сохранён, нормальный лейаут не задет); гейт зелёный. Approve. <sub>9 аспектов. Кап проверен адверсариально (реверт → тест падает), timer-cleanup/timeout-race/нет-регрессии — консолидированным субагентом. Таймаут-тест — below-threshold DROP (кап = первичный доказанный гард). Escalate нет.</sub>
agent_reviewer added review/approved and removed review/needs labels 2026-07-10 07:14:34 +03:00
This pull request has changes conflicting with the target branch.
  • apps/server/src/core/ai-chat/tools/ai-chat-tools.service.ts
  • apps/server/src/core/ai-chat/tools/docmost-client.loader.ts
  • packages/mcp/src/index.ts
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/424-drawio-rules:feat/424-drawio-rules
git checkout feat/424-drawio-rules
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
epic
Large multi-phase effort spanning many changes
 needs-human
эскалация: нужно решение человека
 review/approved
в последнем ревью нет открытых blocking-находок
 review/changes-requested
последнее ревью оставило открытые blocking-находки
 review/needs
head не ревьюился (head != reviewed_head)
 возможно, баг
Похоже на баг, но требует подтверждения/воспроизведения; чиним, если повторится
No Label review/approved
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent_coder agent_qa agent_reviewer agent_vscode vvzvlad
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vvzvlad/gitmost#440
Delete Branch "feat/424-drawio-rules"

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?