document-operations 1.1.6 → 1.1.7

package/dist/index.cjs

@@ -178,6 +178,7 @@ const convertDocumentOperation = defineOperation({
 	async run({ source, targetFormat, output, fonts, onSubstitutionDiagnostics, images }, context) {
 		const signal = context?.signal;
 		const { bytes, format } = await resolveDocumentInput(source, { signal });
+		const wantsFontSubstitutionDiagnostics = onSubstitutionDiagnostics === true;
 		const fontSubstitutions = [];
 		const conversionSignal = signal ?? new AbortController().signal;
 		const result = await converter.convert({
@@ -194,7 +195,7 @@ const convertDocumentOperation = defineOperation({
 				italic: font.italic,
 				bytes: (0, documents_js.base64ToBytes)(font.bytesBase64)
 			})),
-			onFontSubstitution: onSubstitutionDiagnostics === true ? (substitution) => fontSubstitutions.push(substitution) : void 0,
+			onFontSubstitution: wantsFontSubstitutionDiagnostics ? (substitution) => fontSubstitutions.push(substitution) : void 0,
 			images: images === void 0 ? void 0 : (destination) => {
 				const base64 = images[destination];
 				return base64 === void 0 ? void 0 : { bytes: (0, documents_js.base64ToBytes)(base64) };
@@ -205,7 +206,7 @@ const convertDocumentOperation = defineOperation({
 			targetFormat: result.document.format,
 			output: resolvedOutput,
 			diagnostics: [...result.diagnostics],
-			...onSubstitutionDiagnostics === true ? { fontSubstitutions: [...fontSubstitutions] } : {}
+			...wantsFontSubstitutionDiagnostics ? { fontSubstitutions: [...fontSubstitutions] } : {}
 		};
 	}
 });
@@ -563,7 +564,7 @@ const fromPackageOperation = defineOperation({
 			result = (0, documents_js.documentFromJson)(parsed);
 		} catch (error) {
 			if (error instanceof documents_js.UnrecognizedDocumentSchemaError) throw new Error("'source' has no recognised $schema -- only a file carrying a real DocumentTree (e.g. written by a caller's own --dump-package-equivalent step) can be read back by this operation", { cause: error });
-			if (error instanceof zod.z.ZodError) throw new Error(`'source' failed ${(0, documents_js.documentSchemaKindOf)(parsed) ?? "document schema"} validation: ${error.message}`, { cause: error });
+			if (error instanceof zod.z.ZodError) throw new Error(`'source' failed ${(0, documents_js.documentSchemaKindOf)(parsed)} validation: ${error.message}`, { cause: error });
 			throw error;
 		}
 		if (result.kind !== "DocumentTree") throw new Error(`'source' is a ${result.kind}, not a DocumentTree -- only a file carrying a real DocumentTree (e.g. written by a caller's own --dump-package-equivalent step) can be read back by this operation`);
@@ -781,20 +782,10 @@ const odbRenderReportOperation = defineOperation({
 		const pkg = (0, documents_js.decodeOdbPackage)(inputBytes);
 		const content = (0, documents_js.readOdbReportContent)(pkg, { report });
 		const mathDiagnostics = [];
-		const recordMathDiagnostic = (diagnostic, diagnosticContext) => {
-			mathDiagnostics.push({
-				kind: diagnostic.kind,
-				detail: diagnostic.detail,
-				sourcePath: diagnosticContext.sourcePath
-			});
-		};
 		const fontSubstitutions = [];
 		const charSubstitutions = [];
 		let bytes;
-		if (targetFormat === "docx") bytes = (0, documents_js.odbReportToDocx)(content, {
-			signal,
-			onMathDiagnostic: recordMathDiagnostic
-		});
+		if (targetFormat === "docx") bytes = (0, documents_js.odbReportToDocx)(content, { signal });
 		else if (targetFormat === "odt") bytes = (0, documents_js.odbReportToOdt)(content, { signal });
 		else bytes = (0, documents_js.odbReportToPdf)(content, {
 			signal,
@@ -804,15 +795,13 @@ const odbRenderReportOperation = defineOperation({
 				italic: font.italic,
 				bytes: (0, documents_js.base64ToBytes)(font.bytesBase64)
 			})),
-			onFontSubstitution: (substitution) => fontSubstitutions.push(substitution),
 			onSubstitution: (substitution, substitutionContext) => {
 				charSubstitutions.push({
 					from: substitution.from,
 					to: substitution.to,
 					pageIndex: substitutionContext.pageIndex
 				});
-			},
-			onMathDiagnostic: recordMathDiagnostic
+			}
 		});
 		return {
 			...await resolveDocumentOutput(bytes, output ?? {}),

package/dist/index.d.cts

@@ -418,8 +418,8 @@ declare const ConvertDocumentOutputSchema: z.ZodObject<{
   }, z.core.$strip>]>;
   diagnostics: z.ZodArray<z.ZodObject<{
     severity: z.ZodEnum<{
-      warning: "warning";
       info: "info";
+      warning: "warning";
     }>;
     code: z.ZodString;
     message: z.ZodString;
@@ -573,8 +573,8 @@ declare const convertDocumentOperation: {
     }, z.core.$strip>]>;
     diagnostics: z.ZodArray<z.ZodObject<{
       severity: z.ZodEnum<{
-        warning: "warning";
         info: "info";
+        warning: "warning";
       }>;
       code: z.ZodString;
       message: z.ZodString;
@@ -621,7 +621,7 @@ declare const convertDocumentOperation: {
       large?: true | undefined;
     };
     diagnostics: {
-      severity: "warning" | "info";
+      severity: "info" | "warning";
       code: string;
       message: string;
       pageIndex?: number | undefined;

package/dist/index.d.ts

@@ -418,8 +418,8 @@ declare const ConvertDocumentOutputSchema: z.ZodObject<{
   }, z.core.$strip>]>;
   diagnostics: z.ZodArray<z.ZodObject<{
     severity: z.ZodEnum<{
-      warning: "warning";
       info: "info";
+      warning: "warning";
     }>;
     code: z.ZodString;
     message: z.ZodString;
@@ -573,8 +573,8 @@ declare const convertDocumentOperation: {
     }, z.core.$strip>]>;
     diagnostics: z.ZodArray<z.ZodObject<{
       severity: z.ZodEnum<{
-        warning: "warning";
         info: "info";
+        warning: "warning";
       }>;
       code: z.ZodString;
       message: z.ZodString;
@@ -621,7 +621,7 @@ declare const convertDocumentOperation: {
       large?: true | undefined;
     };
     diagnostics: {
-      severity: "warning" | "info";
+      severity: "info" | "warning";
       code: string;
       message: string;
       pageIndex?: number | undefined;

package/dist/index.js

@@ -177,6 +177,7 @@ const convertDocumentOperation = defineOperation({
 	async run({ source, targetFormat, output, fonts, onSubstitutionDiagnostics, images }, context) {
 		const signal = context?.signal;
 		const { bytes, format } = await resolveDocumentInput(source, { signal });
+		const wantsFontSubstitutionDiagnostics = onSubstitutionDiagnostics === true;
 		const fontSubstitutions = [];
 		const conversionSignal = signal ?? new AbortController().signal;
 		const result = await converter.convert({
@@ -193,7 +194,7 @@ const convertDocumentOperation = defineOperation({
 				italic: font.italic,
 				bytes: base64ToBytes(font.bytesBase64)
 			})),
-			onFontSubstitution: onSubstitutionDiagnostics === true ? (substitution) => fontSubstitutions.push(substitution) : void 0,
+			onFontSubstitution: wantsFontSubstitutionDiagnostics ? (substitution) => fontSubstitutions.push(substitution) : void 0,
 			images: images === void 0 ? void 0 : (destination) => {
 				const base64 = images[destination];
 				return base64 === void 0 ? void 0 : { bytes: base64ToBytes(base64) };
@@ -204,7 +205,7 @@ const convertDocumentOperation = defineOperation({
 			targetFormat: result.document.format,
 			output: resolvedOutput,
 			diagnostics: [...result.diagnostics],
-			...onSubstitutionDiagnostics === true ? { fontSubstitutions: [...fontSubstitutions] } : {}
+			...wantsFontSubstitutionDiagnostics ? { fontSubstitutions: [...fontSubstitutions] } : {}
 		};
 	}
 });
@@ -562,7 +563,7 @@ const fromPackageOperation = defineOperation({
 			result = documentFromJson(parsed);
 		} catch (error) {
 			if (error instanceof UnrecognizedDocumentSchemaError) throw new Error("'source' has no recognised $schema -- only a file carrying a real DocumentTree (e.g. written by a caller's own --dump-package-equivalent step) can be read back by this operation", { cause: error });
-			if (error instanceof z.ZodError) throw new Error(`'source' failed ${documentSchemaKindOf(parsed) ?? "document schema"} validation: ${error.message}`, { cause: error });
+			if (error instanceof z.ZodError) throw new Error(`'source' failed ${documentSchemaKindOf(parsed)} validation: ${error.message}`, { cause: error });
 			throw error;
 		}
 		if (result.kind !== "DocumentTree") throw new Error(`'source' is a ${result.kind}, not a DocumentTree -- only a file carrying a real DocumentTree (e.g. written by a caller's own --dump-package-equivalent step) can be read back by this operation`);
@@ -780,20 +781,10 @@ const odbRenderReportOperation = defineOperation({
 		const pkg = decodeOdbPackage(inputBytes);
 		const content = readOdbReportContent(pkg, { report });
 		const mathDiagnostics = [];
-		const recordMathDiagnostic = (diagnostic, diagnosticContext) => {
-			mathDiagnostics.push({
-				kind: diagnostic.kind,
-				detail: diagnostic.detail,
-				sourcePath: diagnosticContext.sourcePath
-			});
-		};
 		const fontSubstitutions = [];
 		const charSubstitutions = [];
 		let bytes;
-		if (targetFormat === "docx") bytes = odbReportToDocx(content, {
-			signal,
-			onMathDiagnostic: recordMathDiagnostic
-		});
+		if (targetFormat === "docx") bytes = odbReportToDocx(content, { signal });
 		else if (targetFormat === "odt") bytes = odbReportToOdt(content, { signal });
 		else bytes = odbReportToPdf(content, {
 			signal,
@@ -803,15 +794,13 @@ const odbRenderReportOperation = defineOperation({
 				italic: font.italic,
 				bytes: base64ToBytes(font.bytesBase64)
 			})),
-			onFontSubstitution: (substitution) => fontSubstitutions.push(substitution),
 			onSubstitution: (substitution, substitutionContext) => {
 				charSubstitutions.push({
 					from: substitution.from,
 					to: substitution.to,
 					pageIndex: substitutionContext.pageIndex
 				});
-			},
-			onMathDiagnostic: recordMathDiagnostic
+			}
 		});
 		return {
 			...await resolveDocumentOutput(bytes, output ?? {}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "document-operations",
-  "version": "1.1.6",
+  "version": "1.1.7",
   "description": "The canonical set of document-conversion, editing, inspection, and .odb operations documents.js exposes -- one Zod input/output schema and one transport-agnostic run() function per operation, shared by document-mcp, document-cli, and a REST API server rather than each defining its own copy.",
   "type": "module",
   "repository": {
@@ -82,10 +82,10 @@
   "license": "MIT",
   "packageManager": "pnpm@11.6.0",
   "dependencies": {
-    "document-compute.js": "1.5.9",
-    "document-outline.js": "3.7.3",
-    "document-schema.js": "7.11.1",
-    "documents.js": "7.20.6",
+    "document-compute.js": "1.5.10",
+    "document-outline.js": "3.8.0",
+    "document-schema.js": "7.11.2",
+    "documents.js": "7.20.7",
     "zod": "4.4.3"
   },
   "devDependencies": {
@@ -98,7 +98,7 @@
     "@vitest/coverage-v8": "4.1.11",
     "eslint": "10.8.1",
     "husky": "9.1.7",
-    "odf.js": "7.25.2",
+    "odf.js": "7.25.3",
     "publint": "0.3.22",
     "semantic-release": "25.0.9",
     "tsdown": "0.22.14",