refactor(mcp): дедупликация конвертер-смежных хелперов (node-ops форк, footnote-*, parse-node-arg)

Аудит при подготовке #413 нашёл дрейфующие дубли между packages/mcp и
packages/prosemirror-markdown. Четыре дедупа (поведение тулов не меняется):

1. node-ops: форк ~960 строк сведён в ОДНУ копию в prosemirror-markdown (живая
   mcp-версия — строгое надмножество замороженного #293-seed'а пакета; сверено по
   git-истории, новая пакет-копия байт-в-байт == прежней mcp-копии). Barrel-экспорт
   полной поверхности; mcp/client.ts/page-search.ts/transforms.ts/collaboration.ts
   импортируют из пакета; тесты переехали. node-ops тянет stripInlineMarkdown ->
   пакет-локальная text-normalize.ts несёт только этот примитив (mcp-версия —
   домен #408; заголовок документирует дубликацию + источник истины).
2. footnote-lex/footnote-analyze (vestigial legacy [^id]: диагностика): сведены к
   одному fence-aware предупреждению 'reference-style footnotes -> use ^[...]'
   (полезно для класса #410); footnote-lex удалён.
3. footnote-authoring -> примитивы (footnoteContentKey/makeFootnoteDefinition/
   generateFootnoteId) перенесены в пакетный footnote.ts, одна реализация конвенции.
4. parse-node-arg -> перенесён в prosemirror-markdown (не mcp: сервер CommonJS не
   импортирует ESM-only @docmost/mcp, но нативно импортирует пакет), обе копии
   удалены, консьюмеры перенаправлены.

canonicalizeFootnotes/ENFORCEMENT RULE #228 и comment-anchor/json-edit/text-normalize
(mcp) не тронуты. API-поверхность node-ops оставлена чистой для #409/#413.

closes #414

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

apps/server/src/core/ai-chat/tools/ai-chat-tools.service.ts

@@ -17,7 +17,7 @@ import {
   resolveCurrentPageResult,
   type SelectionContext,
 } from './current-page.util';
-import { parseNodeArg } from './parse-node-arg';
+import { parseNodeArg } from '@docmost/prosemirror-markdown';
 import { modelFriendlyInput } from './model-friendly-input';
 import { SandboxStore } from '../../../integrations/sandbox/sandbox.store';
 import {

apps/server/src/core/ai-chat/tools/parse-node-arg.spec.ts

@@ -1,10 +1,10 @@
-import { parseNodeArg } from './parse-node-arg';
+import { parseNodeArg } from '@docmost/prosemirror-markdown';
 
 /**
- * Unit tests for the in-app `parseNodeArg` helper. It mirrors the standalone
- * MCP helper (packages/mcp/src/lib/parse-node-arg.ts) and is used by the
- * patchNode / insertNode / updatePageJson tool adapters. Behavior must be
- * byte-identical: object passthrough, valid-string parse, invalid-string throw.
+ * Unit tests for the shared `parseNodeArg` helper (#414: now the single copy in
+ * `@docmost/prosemirror-markdown`, imported by both the server tool adapters and
+ * `@docmost/mcp`). Used by the patchNode / insertNode / updatePageJson adapters.
+ * Behavior: object passthrough, valid-string parse, invalid-string throw.
  */
 describe('parseNodeArg', () => {
   it('passes an object through unchanged', () => {

apps/server/src/core/ai-chat/tools/parse-node-arg.ts

@@ -1,26 +0,0 @@
-// The model sometimes serializes a ProseMirror node arg as a JSON string
-// instead of an object. Normalize: parse a string to an object (throwing on
-// invalid JSON), pass an object through unchanged. Shared by patchNode /
-// insertNode (and the analogous updatePageJson content parsing).
-//
-// This is behaviorally identical to `packages/mcp/src/lib/parse-node-arg.ts`
-// (the function logic, default/explicit throw messages and branch order match;
-// only comments and quote style differ). We cannot import that helper here:
-// `@docmost/mcp` is ESM-only and this server
-// compiles with module:commonjs, so it is loaded at runtime via the
-// `new Function('import()')` trick (see docmost-client.loader.ts). Sharing
-// runtime code across that ESM/CJS boundary by a normal import is impossible,
-// hence the mirrored copy.
-export function parseNodeArg(
-  node: unknown,
-  errMsg = 'node was a string but not valid JSON',
-): unknown {
-  if (typeof node === 'string') {
-    try {
-      return JSON.parse(node);
-    } catch {
-      throw new Error(errMsg);
-    }
-  }
-  return node;
-}