refactor(mcp)!: BREAKING — все имена MCP-тулов snake_case → camelCase (#412) #463
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vvzvlad/gitmost#463
Reference in New Issue
Block a user
Delete Branch "refactor/412-camelcase-tool-names"
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?
Summary
Второй линк breaking-окна #416 (#411→ЭТОТ→#413→#415). BREAKING. closes #412
Один логический тул жил под двумя именами: внешний MCP snake_case (
edit_page_text), in-app camelCase (editPageText) — дублирование доков, путаница при переносе промптов/скиллов, помеха шарингу спек. Решение владельца: единый camelCase везде, включая внешний MCP. После этогоmcpName === inAppKeyдля каждой shared-спеки.tool-specs.ts:mcpNameвыведен из ключа спеки для всех 43 shared-спек (раньше divergent snake). Проверено —mcpNameчитается только структурно (цикл регистрации, генератор<tool_inventory>,TOOL_FAMILY), ничего не полагалось на старое snake-значение. +5 inline-регистраций (tableGet/updateComment/deleteComment/docmostTransform;searchбез изменений). Рантайм: 47 тулов, все camelCase, ноль подчёркиваний.mcpName === toSnake(inAppKey)→mcpName === inAppKey;toSnakeудалён;tool-specs.test/tool-inventory.testобновлены (скрейпер расширен до[a-zA-Z0-9_]).ai-chat.prompt/guard уже на in-app camelCase-ключах — без изменений.comment-signalEXCLUDED_TOOLS схлопнут с дублей snake+camel.assertUnambiguousMatch(op: "patch_node"|"delete_node")вprosemirror-markdown/node-ops(op интерполируется в model-facing ошибку) →"patchNode"|"deleteNode".PageService.updatePageContentи т.п.) НЕ тронуты — переименованы только ИМЕНА ТУЛОВ.⚠️ Breaking (в CHANGELOG таблица 46 строк + миграция)
Все внешние MCP-клиенты ломаются разом: permission-allowlists (
mcp__gitmost-*__get_node→__getNode), сохранённые промпты/скиллы,.mcp.json, метрики per-tool-label. Мигрировать в lockstep с деплоем. Релизится вместе с #411 — чтобы внешние конфиги ломались один раз.How verified
Стенд (symlink
@docmost/mcp+@docmost/prosemirror-markdownна пакеты ветки, восстановлены):node --test→ 677/677;tsc -p apps/serverчисто;jest ai-chat-tools.service + shared-tool-specs.contract + tool-tiers + ai-chat.prompt + comment-signal-inapp→ 323.Внутренний цикл: 1 проход (APPROVE WITH SUGGESTIONS). Ревьюер подтвердил живой энумерацией 47 camelCase-тулов и исчерпывающим grep'ом отсутствие утечек; конвенция перевёрнута и энфорсится; структурные зависимости от
mcpNameцелы. Единственная находка — устаревшая формулировка «snake_case» в двух ВНУТРЕННИХ комментарияхai-chat-tools.service.ts— поправил (не модель-обращённое).Порядок мержа (стопка breaking-окна)
Стоит на develop; #413 (markdown-default) стоит на ЭТОЙ ветке, #415 на #413. Собираю стопкой без ожидания мержей; порядок опишу в #416. Мержить: #411(влит) → этот → #413 → #415.
Checklist
tools/listвнешнего MCP не содержит ни одного snake-имени (47 camelCase)mcpName === inAppKeyдля каждой shared-спеки (контракт зелёный)Ревью — #463
refactor(mcp)!: BREAKING — все имена MCP-тулов snake_case → camelCase (#412)Вердикт: PASS ✅
Полный веер из 9 аспектов (на develop
e19275e9, #462 влит; 55 файлов, +799/−727 — механический rename). Второй линк breaking-окна #416 — осознанный BREAKING (решение владельца, #412), с migration-таблицей. Гейт зелёный, инвариантmcpName===inAppKeyмутационно-пришпилен.Полнота rename + name-keyed гейты (клик)
Единый camelCase везде →
mcpName === inAppKey === registryKeyдля всех 43 shared-спек + 5 inline (tableGet/updateComment/deleteComment/docmostTransform;searchбез изменений — уже одно слово).mcpNameвыведен из ключа,toSnakeудалён.SERVER_INSTRUCTIONS— 0 snake-токенов (кроме тега<tool_inventory>). tool-specsmcpName— все camelCase; ROUTING_PROSE/TOOL_FAMILY(70+ ключей)/INLINE_MCP_INVENTORY — camelCase; model-facing ошибки (node-ops assertUnambiguousMatchop) →patchNode/deleteNode; ничего в рантайме не хардкодило snake-имя (grep чист — читалось только структурно).COMMENT_SIGNAL_EXCLUDED_TOOLSпереименован в camelCase, ни один тул не выпал/не добавился в exclusion;INLINE_TOOL_TIERScamelCase без тихой смены тира;inAppOnly/mcpOnlyсохранены (importPageMarkdown всё ещё inAppOnly). Расширения возможностей нет./^[a-z][a-zA-Z0-9]*$/(ловит будущий underscore). Адверсариально: вернул один mcpName в snake → контракт-тест «mcpName and inAppKey are the same camelCase identifier» ПАДАЕТ → load-bearing.Объективный гейт (клин-рум /home/claude/gm-ci)
[]. Адверсариальный snake-mcpName краснит контракт-тест; restore зелёный.⛔ DROP — кодеру НЕ делать (лог калибровки)
[below-threshold]low[documentation] 2 стейл-коммента с old-именами:ai-chat-tools.service.ts:785(«patch_node») и:953(«docmost_transform») — ВНУТРЕННИЕ комменты, не model-facing/не client-facing, поведение не задето (заявка PR «рантайм zero-underscore» относится к именам тулов — выполнена, гейт подтвердил). Рекомендую подмести для полноты, но не блок.DO/ESCALATE нет. Rename полон (runtime 0-underscore объективно), инвариант mcpName===inAppKey тестируется+адверсариально держится, name-keyed гейты (exclusion/tiers) переименованы без смены членства, in-app не задет, доки×2 + CHANGELOG migration полны. BREAKING осознанный (#412), в lockstep с #411. Approve.
9 аспектов (coherence+regressions+security+stability, testcov+conv+docs+simp+arch — все LGTM). Полнота runtime подтверждена (SERVER_INSTRUCTIONS snake=[]); mcpName===inAppKey адверсариально пришпилен; EXCLUDED_TOOLS/tiers без смены членства; README×2+CHANGELOG полны. 2 стейл-коммента — below-threshold DROP. Escalate нет.