@tiwater/office-mcp 0.15.2 → 0.15.4

package/_shared/tool-runtime.mjs

@@ -8,8 +8,9 @@ import { spawn } from 'node:child_process';
 const sharedDir = path.dirname(fileURLToPath(import.meta.url));
 export const repoRoot = path.resolve(sharedDir, '..', '..');
 
-export function createToolResult(payload) {
+export function createToolResult(payload, { isError = false } = {}) {
   return {
+    ...(isError ? { isError: true } : {}),
     structuredContent: payload,
     content: [
       {

package/office/README.md

@@ -6,6 +6,9 @@ The server owns technical observation, native conversion, fixed-action edits,
 package validation, and native WPS rendering. Callers own document selections,
 values, business mappings, workflow decisions, and delivery status.
 
+Every inspect/export result binds its observation artifact to the exact source
+file path, SHA-256, and byte count used by that invocation.
+
 Every filesystem argument is identified in the published MCP input schema by
 `x-tiwater-file-role`. `read` marks an existing provider input; `write` marks a
 new provider artifact. Strings without this metadata are document values or
@@ -27,6 +30,9 @@ Each mutation tool fixes its provider operation type. Callers submit only the
 coordinates and values for that action, so they cannot provide an arbitrary
 operation discriminator or a multi-action plan language. A call may batch
 multiple changes only when every change has the same action kind.
+When a completed mutation receipt reports `summary.pass=false`, the MCP result
+also sets the standard `isError` field; consumers do not need a private failure
+interpretation.
 
 Structural worksheet row deletion is exposed as `xlsx_delete_rows`. Each change
 contains only `sheet`, `startRow`, and `count`; unsupported dependent workbook

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.15.2"
+    "version": "0.15.4"
   },
   "tools": [
     {

package/office/index.mjs

@@ -185,7 +185,7 @@ function fixedEditOutput(tool) {
 }
 
 function artifactOutput(tool) {
-  return z.object({ tool: z.literal(tool), runtime: runtimeIdentity, artifact }).strict();
+  return z.object({ tool: z.literal(tool), runtime: runtimeIdentity, source: artifact, artifact }).strict();
 }
 
 const tools = [
@@ -382,25 +382,30 @@ function buildServer() {
         ...(tool.outputSchema ? { outputSchema: tool.outputSchema } : {}),
         ...(tool.annotations ? { annotations: tool.annotations } : {}),
       },
-      async args => createToolResult(await tool.handler(args)),
+      async args => {
+        const payload = await tool.handler(args);
+        return createToolResult(payload, { isError: payload?.summary?.pass === false });
+      },
     );
   }
   return server;
 }
 
 async function docxInspect(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const result = await runJsonCandidateChain(docxCandidates, ['inspect', input, '--json']);
   return {
     tool: 'docx_inspect',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }
 
 async function docxInspectTables(args) {
-  const result = await runJsonCandidateChain(docxCandidates, ['inspect-tables', requireString(args.input, 'input'), '--json']);
-  return { tool: 'docx_inspect_tables', runtime: commandRuntime(result), artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json) };
+  const input = path.resolve(requireString(args.input, 'input'));
+  const result = await runJsonCandidateChain(docxCandidates, ['inspect-tables', input, '--json']);
+  return { tool: 'docx_inspect_tables', runtime: commandRuntime(result), source: await fileArtifact(input), artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json) };
 }
 
 async function docxObservation(tool, args) {
@@ -475,11 +480,12 @@ async function docxCompare(args) {
 }
 
 async function docxExportJson(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const result = await runJsonCandidateChain(docxCandidates, ['export-json', input]);
   return {
     tool: 'docx_export_json',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }
@@ -567,17 +573,18 @@ function nativeRenderBackend(sourceFormat) {
 }
 
 async function xlsxInspect(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const result = await runJsonCandidateChain(xlsxCandidates, ['inspect', input, '--json']);
   return {
     tool: 'xlsx_inspect',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }
 
 async function xlsxExportJson(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const cmdArgs = ['export-json', input];
   if (args.resolveMergedCells) {
     cmdArgs.push('--resolve-merged-cells');
@@ -586,6 +593,7 @@ async function xlsxExportJson(args) {
   return {
     tool: 'xlsx_export_json',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }
@@ -597,21 +605,23 @@ async function xlsxValidate(args) {
 }
 
 async function pptxInspect(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const result = await runJsonCandidateChain(pptxCandidates, ['inspect', input, '--json']);
   return {
     tool: 'pptx_inspect',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }
 
 async function pptxExportJson(args) {
-  const input = requireString(args.input, 'input');
+  const input = path.resolve(requireString(args.input, 'input'));
   const result = await runJsonCandidateChain(pptxCandidates, ['export-json', input]);
   return {
     tool: 'pptx_export_json',
     runtime: commandRuntime(result),
+    source: await fileArtifact(input),
     artifact: await writeJsonArtifact(requireString(args.output, 'output'), result.json),
   };
 }

package/package.json

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