@attentionform/astra-mcp-server 0.2.0 → 0.2.1

package/README.md

@@ -387,6 +387,8 @@ Preflight read-only OOXML schema audit for a session before committing changes.
 
 **Returns:** `{ ok: true, sessionId, revision, dirty, validation: { valid, errorCount, errors } }`
 
+Validation advice: when `validate_document` reports an invalid document, its `normalizeHint` suggests attempting `normalize_document` and validating again. This is advice, not a repair forecast or a guarantee of validity; validation does not run repair rules to generate the hint. Normalization changes the open session; use `save_document` to persist the result, optionally to a new output path.
+
 ---
 
 ### `get_stats`

package/dist/index.js

@@ -34015,19 +34015,6 @@ async function repairDocxBuffer(input, options = {}) {
   }
   return { buffer: normalized.length > 0 ? pack(current) : input, normalized, remaining, rolledBack };
 }
-async function forecastNormalize(input) {
-  try {
-    const outcome = await repairDocxBuffer(input);
-    if (outcome.normalized.length === 0) return void 0;
-    return { changes: outcome.normalized.length, remaining: outcome.remaining };
-  } catch {
-    return void 0;
-  }
-}
-function normalizeHintSentence(forecast, errorCount, how) {
-  const left = forecast.remaining === 0 ? "and the document then validates" : forecast.remaining > 0 ? `and leaves ${forecast.remaining} of ${errorCount} errors, which need Word` : "and the rest need Word";
-  return `${how} can repair some of these errors (${forecast.changes} change${forecast.changes === 1 ? "" : "s"}) ${left}.`;
-}
 
 // ../engine/src/export/html.ts
 function asFootnoteZipCapable(parser2) {
@@ -42092,6 +42079,35 @@ var LEAN6_HANDLERS = {
 
 // src/tools/export.ts
 var import_zod6 = require("zod");
+
+// src/lib/fork-numbering.ts
+function adoptForkNumbering(fork, ast) {
+  const source = ast.numbering?.source;
+  const cloneDefinitions = (definitions) => Object.fromEntries(
+    Object.entries(definitions).map(([id, element]) => [id, element.cloneNode(true)])
+  );
+  fork.adoptAstForNumbering({
+    numbering: {
+      abstractNum: cloneDefinitions(ast.numbering?.abstractNum ?? {}),
+      concreteNum: cloneDefinitions(ast.numbering?.concreteNum ?? {}),
+      // Mirrors the NumberingSource clone in engine ast.ts (cloneAST); keep both
+      // in step when NumberingSource gains fields.
+      ...source ? {
+        source: {
+          ...source,
+          rootAttributes: source.rootAttributes.map((attribute) => ({ ...attribute })),
+          leading: [...source.leading],
+          trailing: [...source.trailing]
+        }
+      } : {}
+    },
+    relationships: Object.fromEntries(
+      Object.entries(ast.relationships ?? {}).map(([id, relationship]) => [id, { ...relationship }])
+    )
+  });
+}
+
+// src/tools/export.ts
 var fs15 = __toESM(require("fs"));
 var fsp8 = __toESM(require("fs/promises"));
 var path16 = __toESM(require("path"));
@@ -42255,16 +42271,7 @@ function serializeSessionCopy(parser2, ast) {
   }
   const fork = new DocxParser(zip.toBuffer());
   try {
-    const live = ast;
-    if (live && typeof live === "object") {
-      fork.adoptAstForNumbering({
-        numbering: {
-          abstractNum: { ...live.numbering?.abstractNum ?? {} },
-          concreteNum: { ...live.numbering?.concreteNum ?? {} }
-        },
-        relationships: { ...live.relationships ?? {} }
-      });
-    }
+    adoptForkNumbering(fork, ast);
   } catch {
   }
   return fork.rezip();
@@ -42658,16 +42665,7 @@ function serializeSessionCopy2(parser2, ast) {
   }
   const fork = new DocxParser(zip.toBuffer());
   try {
-    const live = ast;
-    if (live && typeof live === "object") {
-      fork.adoptAstForNumbering({
-        numbering: {
-          abstractNum: { ...live.numbering?.abstractNum ?? {} },
-          concreteNum: { ...live.numbering?.concreteNum ?? {} }
-        },
-        relationships: { ...live.relationships ?? {} }
-      });
-    }
+    adoptForkNumbering(fork, ast);
   } catch {
   }
   return fork.rezip();
@@ -43025,16 +43023,7 @@ function serializeSessionFork(parser2, ast) {
   try {
     const fork = new DocxParser(staleBytes);
     try {
-      const live = ast;
-      if (live && typeof live === "object") {
-        fork.adoptAstForNumbering({
-          numbering: {
-            abstractNum: { ...live.numbering?.abstractNum ?? {} },
-            concreteNum: { ...live.numbering?.concreteNum ?? {} }
-          },
-          relationships: { ...live.relationships ?? {} }
-        });
-      }
+      adoptForkNumbering(fork, ast);
     } catch {
     }
     return fork.rezip();
@@ -43061,15 +43050,9 @@ async function handleValidateDocument(store, input) {
   }
   try {
     return await store.withSession(parsed.data.sessionId, async (handle) => {
-      const { validation, buf } = await forkValidation(handle);
+      const { validation } = await forkValidation(handle);
       const pendingNormalizations = readPendingNormalizations(handle);
-      let normalizeHint;
-      if (!validation.valid) {
-        const forecast = await forecastNormalize(buf);
-        if (forecast) {
-          normalizeHint = normalizeHintSentence(forecast, validation.errorCount, "Run normalize_document: it");
-        }
-      }
+      const normalizeHint = validation.valid ? void 0 : "Call `normalize_document` to attempt the supported repairs, then call `validate_document` again.";
       return {
         ok: true,
         sessionId: handle.sessionId,
@@ -43421,7 +43404,7 @@ var import_zod9 = require("zod");
 // package.json
 var package_default = {
   name: "@attentionform/astra-mcp-server",
-  version: "0.2.0",
+  version: "0.2.1",
   description: "Official Model Context Protocol (MCP) server for ASTra document IDE",
   main: "dist/index.js",
   types: "dist/index.d.ts",
@@ -43462,8 +43445,8 @@ var package_default = {
     "adm-zip-gate": "node ../../scripts/check-adm-zip-imports.mjs"
   },
   dependencies: {
-    "@attentionform/astra-engine": "^0.2.0",
-    "@attentionform/ooxml-validator": "^0.2.0",
+    "@attentionform/astra-engine": "^0.2.1",
+    "@attentionform/ooxml-validator": "^0.2.1",
     "@modelcontextprotocol/sdk": "^1.32.2",
     "@xmldom/xmldom": "0.8.15",
     "adm-zip": "0.6.1",

package/dist/lib/fork-numbering.d.ts

@@ -0,0 +1,10 @@
+import { type DocumentAST, type DocxParser } from '@attentionform/astra-engine';
+/**
+ * Give a serialization fork the live numbering, including its original part
+ * metadata. The engine uses the source hashes to retain untouched bytes and
+ * the root attributes/extra children when definitions really have changed.
+ * Clone every numbering element and source record so fork serialization cannot
+ * change the session. Pass only numbering/relationships on purpose: the bare
+ * rezip path reads only those fields and must not receive other live AST data.
+ */
+export declare function adoptForkNumbering(fork: DocxParser, ast: DocumentAST): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@attentionform/astra-mcp-server",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Official Model Context Protocol (MCP) server for ASTra document IDE",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -41,8 +41,8 @@
     "adm-zip-gate": "node ../../scripts/check-adm-zip-imports.mjs"
   },
   "dependencies": {
-    "@attentionform/astra-engine": "^0.2.0",
-    "@attentionform/ooxml-validator": "^0.2.0",
+    "@attentionform/astra-engine": "^0.2.1",
+    "@attentionform/ooxml-validator": "^0.2.1",
     "@modelcontextprotocol/sdk": "^1.32.2",
     "@xmldom/xmldom": "0.8.15",
     "adm-zip": "0.6.1",