@tiwater/office-mcp 0.21.5 → 0.21.6

package/_shared/large-json-result.mjs

@@ -1,6 +1,6 @@
 import { createHash } from 'node:crypto';
 import { createReadStream } from 'node:fs';
-import { mkdir, stat, writeFile } from 'node:fs/promises';
+import { mkdir, readFile, stat, writeFile } from 'node:fs/promises';
 import path from 'node:path';
 
 import { requireString } from './tool-runtime.mjs';
@@ -31,7 +31,7 @@ export async function deliverLargeJsonResult({ tool, args, runtime, payload, sou
     runtime,
     sources: await Promise.all(sourcePaths.map(fileArtifact)),
     returnContent: channels.returnContent,
-    artifact: channels.output === null ? null : await writeJsonArtifact(channels.output, payload),
+    artifact: channels.output === null ? null : await writeIdempotentJsonArtifact(channels.output, payload),
     receipt: {
       contentBytes,
       contentReturned,
@@ -47,6 +47,28 @@ export async function writeJsonArtifact(output, payload) {
   const bytes = Buffer.from(`${JSON.stringify(payload, null, 2)}\n`, 'utf8');
   await mkdir(path.dirname(fullPath), { recursive: true });
   await writeFile(fullPath, bytes, { flag: 'wx' });
+  return describeJsonArtifact(fullPath, bytes);
+}
+
+export async function writeIdempotentJsonArtifact(output, payload) {
+  const fullPath = path.resolve(output);
+  const bytes = Buffer.from(`${JSON.stringify(payload, null, 2)}\n`, 'utf8');
+  await mkdir(path.dirname(fullPath), { recursive: true });
+  try {
+    await writeFile(fullPath, bytes, { flag: 'wx' });
+  } catch (error) {
+    if (error?.code !== 'EEXIST') throw error;
+    const existing = await readFile(fullPath);
+    if (!existing.equals(bytes)) {
+      throw Object.assign(new Error(`artifact-path-already-exists-with-different-content:${fullPath}`), {
+        code: 'EEXIST',
+      });
+    }
+  }
+  return describeJsonArtifact(fullPath, bytes);
+}
+
+function describeJsonArtifact(fullPath, bytes) {
   return {
     path: fullPath,
     sha256: createHash('sha256').update(bytes).digest('hex'),

package/office/contracts/tiwater-office-provider-contract-manifest-v1.json

@@ -2,7 +2,7 @@
   "schema": "tiwater.office-provider-contract-manifest/v1",
   "provider": {
     "id": "@tiwater/office-mcp",
-    "version": "0.21.5"
+    "version": "0.21.6"
   },
   "tools": [
     {

package/office/index.mjs

@@ -19,6 +19,7 @@ import {
   fileArtifact,
   resultChannels,
   returnedContentBudgetBytes,
+  writeIdempotentJsonArtifact,
   writeJsonArtifact,
 } from '../_shared/large-json-result.mjs';
 import { withOutputWriteLock } from '../_shared/output-write-lock.mjs';
@@ -746,7 +747,7 @@ async function docxInspect(args) {
     returnContent: delivery.returnContent,
     artifact: delivery.output === null
       ? null
-      : await writeJsonArtifact(delivery.output, result.json),
+      : await writeIdempotentJsonArtifact(delivery.output, result.json),
     ...(delivery.returnContent ? compactDocxInspection(result.json) : {}),
   };
 }
@@ -770,7 +771,7 @@ async function docxObservation(tool, args) {
       const offset = Math.min(args.offset ?? 0, totalCount);
       const withArtifact = delivery.output === null ? retained : {
         ...retained,
-        artifact: await writeJsonArtifact(delivery.output, result.json),
+        artifact: await writeIdempotentJsonArtifact(delivery.output, result.json),
       };
       if (!delivery.returnContent) {
         return {
@@ -805,7 +806,7 @@ async function docxObservation(tool, args) {
       const totalCount = payload.tables.length;
       const withArtifact = delivery.output === null ? retained : {
         ...retained,
-        artifact: await writeJsonArtifact(delivery.output, result.json),
+        artifact: await writeIdempotentJsonArtifact(delivery.output, result.json),
       };
       if (!delivery.returnContent) {
         return {
@@ -872,7 +873,7 @@ async function docxObservation(tool, args) {
       };
       const withArtifact = delivery.output === null ? retained : {
         ...retained,
-        artifact: await writeJsonArtifact(delivery.output, detailPage),
+        artifact: await writeIdempotentJsonArtifact(delivery.output, detailPage),
       };
       if (!delivery.returnContent) {
         return {
@@ -964,7 +965,7 @@ async function docxReadObject(args) {
       returnContent: delivery.returnContent,
       artifact: delivery.output === null
         ? null
-        : await writeJsonArtifact(delivery.output, result.json),
+        : await writeIdempotentJsonArtifact(delivery.output, result.json),
       receipt: {
         schema: 'tiwater.docx-read-object-receipt/v1',
         observationCount: observations.length,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiwater/office-mcp",
-  "version": "0.21.5",
+  "version": "0.21.6",
   "description": "Published MCP server for Tiwater Office document capabilities",
   "type": "module",
   "license": "MIT",