@harbour-enterprises/superdoc 1.17.0-next.6 → 1.17.0-next.7

package/dist/chunks/{SuperConverter-Ddyn6710.cjs → SuperConverter-CEFypeJ2.cjs}

@@ -33838,7 +33838,7 @@ var SuperConverter = class SuperConverter {
 	static getStoredSuperdocVersion(docx) {
 		return SuperConverter.getStoredCustomProperty(docx, "SuperdocVersion");
 	}
-	static setStoredSuperdocVersion(docx = this.convertedXml, version = "1.17.0-next.6") {
+	static setStoredSuperdocVersion(docx = this.convertedXml, version = "1.17.0-next.7") {
 		return SuperConverter.setStoredCustomProperty(docx, "SuperdocVersion", version, false);
 	}
 	static generateWordTimestamp() {

package/dist/chunks/{SuperConverter-DQyZgbId.es.js → SuperConverter-RKhfX479.es.js}

@@ -33824,7 +33824,7 @@ var SuperConverter = class SuperConverter {
 	static getStoredSuperdocVersion(docx) {
 		return SuperConverter.getStoredCustomProperty(docx, "SuperdocVersion");
 	}
-	static setStoredSuperdocVersion(docx = this.convertedXml, version = "1.17.0-next.6") {
+	static setStoredSuperdocVersion(docx = this.convertedXml, version = "1.17.0-next.7") {
 		return SuperConverter.setStoredCustomProperty(docx, "SuperdocVersion", version, false);
 	}
 	static generateWordTimestamp() {

package/dist/chunks/{src-B578ierl.cjs → src-BacOu6cf.cjs}

@@ -1,5 +1,5 @@
 const require_rolldown_runtime = require("./rolldown-runtime-Dp2H1eGw.cjs");
-const require_SuperConverter = require("./SuperConverter-Ddyn6710.cjs");
+const require_SuperConverter = require("./SuperConverter-CEFypeJ2.cjs");
 const require_jszip = require("./jszip-DCT9QYaK.cjs");
 const require_uuid = require("./uuid-CHj_rjgt.cjs");
 const require_constants = require("./constants-CpniKo9Z.cjs");
@@ -261,7 +261,7 @@ var DEFAULT_ENDPOINT = "https://ingest.superdoc.dev/v1/collect";
 const COMMUNITY_LICENSE_KEY = "community-and-eval-agplv3";
 function getSuperdocVersion() {
 	try {
-		return "1.17.0-next.6";
+		return "1.17.0-next.7";
 	} catch {
 		return "unknown";
 	}
@@ -20734,7 +20734,7 @@ const canUseDOM = () => {
 		return false;
 	}
 };
-var summaryVersion = "1.17.0-next.6";
+var summaryVersion = "1.17.0-next.7";
 var nodeKeys = [
 	"group",
 	"content",
@@ -21321,7 +21321,7 @@ const OPERATION_DEFINITIONS = {
 	},
 	"format.apply": {
 		memberPath: "format.apply",
-		description: "Apply explicit mark changes (bold, italic, underline, strike) to the target range using boolean patch semantics.",
+		description: "Apply explicit inline style changes (bold, italic, underline, strike) to the target range using boolean patch semantics.",
 		requiresDocumentContext: true,
 		metadata: mutationOperation({
 			idempotency: "conditional",
@@ -21502,7 +21502,11 @@ const OPERATION_DEFINITIONS = {
 			supportsDryRun: false,
 			supportsTrackedMode: false,
 			possibleFailureCodes: ["INVALID_TARGET", "NO_OP"],
-			throws: [...T_NOT_FOUND_COMMAND, "INVALID_TARGET"]
+			throws: [
+				...T_NOT_FOUND_COMMAND,
+				"INVALID_TARGET",
+				"INVALID_INPUT"
+			]
 		}),
 		referenceDocPath: "comments/patch.mdx",
 		referenceGroup: "comments"
@@ -21559,8 +21563,8 @@ const OPERATION_DEFINITIONS = {
 		referenceDocPath: "track-changes/get.mdx",
 		referenceGroup: "trackChanges"
 	},
-	"review.decide": {
-		memberPath: "review.decide",
+	"trackChanges.decide": {
+		memberPath: "trackChanges.decide",
 		description: "Accept or reject a tracked change (by ID or scope: all).",
 		requiresDocumentContext: true,
 		metadata: mutationOperation({
@@ -21568,10 +21572,14 @@ const OPERATION_DEFINITIONS = {
 			supportsDryRun: false,
 			supportsTrackedMode: false,
 			possibleFailureCodes: ["NO_OP"],
-			throws: T_NOT_FOUND_COMMAND
+			throws: [
+				...T_NOT_FOUND_COMMAND,
+				"INVALID_INPUT",
+				"INVALID_TARGET"
+			]
 		}),
-		referenceDocPath: "review/decide.mdx",
-		referenceGroup: "review"
+		referenceDocPath: "track-changes/decide.mdx",
+		referenceGroup: "trackChanges"
 	},
 	"query.match": {
 		memberPath: "query.match",
@@ -22337,7 +22345,7 @@ var operationSchemas = {
 	"format.apply": {
 		input: objectSchema({
 			target: textAddressSchema,
-			marks: (() => {
+			inline: (() => {
 				return {
 					type: "object",
 					properties: Object.fromEntries(MARK_KEYS.map((key$1) => [key$1, { type: "boolean" }])),
@@ -22345,7 +22353,7 @@ var operationSchemas = {
 					minProperties: 1
 				};
 			})()
-		}, ["target", "marks"]),
+		}, ["target", "inline"]),
 		output: textMutationResultSchemaFor("format.apply"),
 		success: textMutationSuccessSchema,
 		failure: textMutationFailureSchemaFor("format.apply")
@@ -22505,7 +22513,7 @@ var operationSchemas = {
 		input: objectSchema({ id: { type: "string" } }, ["id"]),
 		output: trackChangeInfoSchema
 	},
-	"review.decide": {
+	"trackChanges.decide": {
 		input: {
 			type: "object",
 			properties: {
@@ -22515,9 +22523,9 @@ var operationSchemas = {
 			required: ["decision", "target"],
 			additionalProperties: false
 		},
-		output: receiptResultSchemaFor("review.decide"),
+		output: receiptResultSchemaFor("trackChanges.decide"),
 		success: receiptSuccessSchema,
-		failure: receiptFailureResultSchemaFor("review.decide")
+		failure: receiptFailureResultSchemaFor("trackChanges.decide")
 	},
 	"query.match": {
 		input: objectSchema({
@@ -22679,14 +22687,9 @@ var GROUP_METADATA = {
 	},
 	trackChanges: {
 		title: "Track Changes",
-		description: "Tracked-change inspection operations.",
+		description: "Tracked-change inspection and review operations.",
 		pagePath: "track-changes/index.mdx"
 	},
-	review: {
-		title: "Review",
-		description: "Accept or reject tracked changes.",
-		pagePath: "review/index.mdx"
-	},
 	query: {
 		title: "Query",
 		description: "Deterministic selector-based queries for mutation targeting.",
@@ -22763,7 +22766,8 @@ function validateCreateCommentInput(input) {
 		if (hasTarget) throw new DocumentApiValidationError("INVALID_TARGET", "Cannot combine parentCommentId with target. Replies do not take a target.", { fields: ["parentCommentId", "target"] });
 		return;
 	}
-	if (hasTarget && !isTextAddress(target)) throw new DocumentApiValidationError("INVALID_TARGET", "target must be a text address object.", {
+	if (!hasTarget) throw new DocumentApiValidationError("INVALID_TARGET", "comments.create requires a target for root comments.", { field: "target" });
+	if (!isTextAddress(target)) throw new DocumentApiValidationError("INVALID_TARGET", "target must be a text address object.", {
 		field: "target",
 		value: target
 	});
@@ -22784,6 +22788,15 @@ function validatePatchCommentInput(input) {
 		field: "commentId",
 		value: commentId
 	});
+	const mutationFields = [
+		"text",
+		"target",
+		"status",
+		"isInternal"
+	];
+	const providedFields = mutationFields.filter((f) => input[f] !== void 0);
+	if (providedFields.length === 0) throw new DocumentApiValidationError("INVALID_INPUT", "comments.patch requires exactly one mutation field (text, target, status, or isInternal).", { allowedFields: [...mutationFields] });
+	if (providedFields.length > 1) throw new DocumentApiValidationError("INVALID_INPUT", `comments.patch accepts exactly one mutation field per call, got ${providedFields.length}: ${providedFields.join(", ")}.`, { providedFields: [...providedFields] });
 	const { status } = input;
 	if (status !== void 0 && status !== "resolved") throw new DocumentApiValidationError("INVALID_TARGET", `status must be "resolved", got "${String(status)}".`, {
 		field: "status",
@@ -22804,21 +22817,20 @@ function executeCommentsCreate(adapter, input, options) {
 }
 function executeCommentsPatch(adapter, input, options) {
 	validatePatchCommentInput(input);
-	let lastReceipt = { success: true };
-	if (input.text !== void 0) lastReceipt = adapter.edit({
+	if (input.text !== void 0) return adapter.edit({
 		commentId: input.commentId,
 		text: input.text
 	}, options);
-	if (input.target !== void 0) lastReceipt = adapter.move({
+	if (input.target !== void 0) return adapter.move({
 		commentId: input.commentId,
 		target: input.target
 	}, options);
-	if (input.status === "resolved") lastReceipt = adapter.resolve({ commentId: input.commentId }, options);
-	if (input.isInternal !== void 0) lastReceipt = adapter.setInternal({
+	if (input.status === "resolved") return adapter.resolve({ commentId: input.commentId }, options);
+	if (input.isInternal !== void 0) return adapter.setInternal({
 		commentId: input.commentId,
 		isInternal: input.isInternal
 	}, options);
-	return lastReceipt;
+	return { success: true };
 }
 function executeCommentsDelete(adapter, input, options) {
 	return adapter.remove({ commentId: input.commentId }, options);
@@ -22884,31 +22896,31 @@ function normalizeMutationOptions(options) {
 function executeWrite(adapter, request, options) {
 	return adapter.write(request, normalizeMutationOptions(options));
 }
-var STYLE_APPLY_INPUT_ALLOWED_KEYS = new Set(["target", "marks"]);
+var STYLE_APPLY_INPUT_ALLOWED_KEYS = new Set(["target", "inline"]);
 function validateStyleApplyInput(input) {
 	if (!isRecord(input)) throw new DocumentApiValidationError("INVALID_INPUT", "format.apply input must be a non-null object.");
 	assertNoUnknownFields(input, STYLE_APPLY_INPUT_ALLOWED_KEYS, "format.apply");
-	const { target, marks } = input;
+	const { target, inline } = input;
 	if (target === void 0) throw new DocumentApiValidationError("INVALID_TARGET", "format.apply requires a target.");
 	if (!isTextAddress(target)) throw new DocumentApiValidationError("INVALID_TARGET", "target must be a text address object.", {
 		field: "target",
 		value: target
 	});
-	if (marks === void 0 || marks === null) throw new DocumentApiValidationError("INVALID_INPUT", "format.apply requires a marks object.");
-	if (!isRecord(marks)) throw new DocumentApiValidationError("INVALID_INPUT", "marks must be a non-null object.", {
-		field: "marks",
-		value: marks
+	if (inline === void 0 || inline === null) throw new DocumentApiValidationError("INVALID_INPUT", "format.apply requires an inline object.");
+	if (!isRecord(inline)) throw new DocumentApiValidationError("INVALID_INPUT", "inline must be a non-null object.", {
+		field: "inline",
+		value: inline
 	});
-	const markKeys$1 = Object.keys(marks);
-	if (markKeys$1.length === 0) throw new DocumentApiValidationError("INVALID_INPUT", "marks must include at least one known key.");
-	for (const key$1 of markKeys$1) {
-		if (!MARK_KEY_SET.has(key$1)) throw new DocumentApiValidationError("INVALID_INPUT", `Unknown mark key "${key$1}". Known keys: bold, italic, underline, strike.`, {
-			field: "marks",
+	const inlineKeys = Object.keys(inline);
+	if (inlineKeys.length === 0) throw new DocumentApiValidationError("INVALID_INPUT", "inline must include at least one known key.");
+	for (const key$1 of inlineKeys) {
+		if (!MARK_KEY_SET.has(key$1)) throw new DocumentApiValidationError("INVALID_INPUT", `Unknown inline style key "${key$1}". Known keys: bold, italic, underline, strike.`, {
+			field: "inline",
 			key: key$1
 		});
-		const value = marks[key$1];
-		if (typeof value !== "boolean") throw new DocumentApiValidationError("INVALID_INPUT", `Mark "${key$1}" must be a boolean, got ${typeof value}.`, {
-			field: "marks",
+		const value = inline[key$1];
+		if (typeof value !== "boolean") throw new DocumentApiValidationError("INVALID_INPUT", `Inline style "${key$1}" must be a boolean, got ${typeof value}.`, {
+			field: "inline",
 			key: key$1,
 			value
 		});
@@ -23085,14 +23097,32 @@ function executeTrackChangesList(adapter, input) {
 function executeTrackChangesGet(adapter, input) {
 	return adapter.get(input);
 }
-function executeReviewDecide(adapter, input, options) {
-	const isAll = "scope" in input.target && input.target.scope === "all";
+function executeTrackChangesDecide(adapter, rawInput, options) {
+	const raw = rawInput;
+	if (typeof raw !== "object" || raw == null) throw new DocumentApiValidationError("INVALID_INPUT", "trackChanges.decide input must be a non-null object.", { value: raw });
+	const input = raw;
+	if (input.decision !== "accept" && input.decision !== "reject") throw new DocumentApiValidationError("INVALID_INPUT", `trackChanges.decide decision must be "accept" or "reject", got "${String(input.decision)}".`, {
+		field: "decision",
+		value: input.decision
+	});
+	if (typeof input.target !== "object" || input.target == null) throw new DocumentApiValidationError("INVALID_TARGET", "trackChanges.decide target must be an object with { id: string } or { scope: \"all\" }.", {
+		field: "target",
+		value: input.target
+	});
+	const target = input.target;
+	const isAll = target.scope === "all";
+	if (!isAll) {
+		if (typeof target.id !== "string" || target.id.length === 0) throw new DocumentApiValidationError("INVALID_TARGET", "trackChanges.decide target must have { id: string } or { scope: \"all\" }.", {
+			field: "target",
+			value: input.target
+		});
+	}
 	if (input.decision === "accept") {
 		if (isAll) return adapter.acceptAll({}, options);
-		return adapter.accept({ id: input.target.id }, options);
+		return adapter.accept({ id: target.id }, options);
 	}
 	if (isAll) return adapter.rejectAll({}, options);
-	return adapter.reject({ id: input.target.id }, options);
+	return adapter.reject({ id: target.id }, options);
 }
 function buildDispatchTable(api) {
 	return {
@@ -23122,7 +23152,7 @@ function buildDispatchTable(api) {
 		"comments.list": (input) => api.comments.list(input),
 		"trackChanges.list": (input) => api.trackChanges.list(input),
 		"trackChanges.get": (input) => api.trackChanges.get(input),
-		"review.decide": (input, options) => api.review.decide(input, options),
+		"trackChanges.decide": (input, options) => api.trackChanges.decide(input, options),
 		"query.match": (input) => api.query.match(input),
 		"mutations.preview": (input) => api.mutations.preview(input),
 		"mutations.apply": (input) => api.mutations.apply(input),
@@ -23177,25 +23207,25 @@ function createDocumentApi(adapters) {
 			bold(input, options) {
 				return executeStyleApply$1(adapters.format, {
 					...input,
-					marks: { bold: true }
+					inline: { bold: true }
 				}, options);
 			},
 			italic(input, options) {
 				return executeStyleApply$1(adapters.format, {
 					...input,
-					marks: { italic: true }
+					inline: { italic: true }
 				}, options);
 			},
 			underline(input, options) {
 				return executeStyleApply$1(adapters.format, {
 					...input,
-					marks: { underline: true }
+					inline: { underline: true }
 				}, options);
 			},
 			strikethrough(input, options) {
 				return executeStyleApply$1(adapters.format, {
 					...input,
-					marks: { strike: true }
+					inline: { strike: true }
 				}, options);
 			},
 			apply(input, options) {
@@ -23208,11 +23238,11 @@ function createDocumentApi(adapters) {
 			},
 			get(input) {
 				return executeTrackChangesGet(adapters.trackChanges, input);
+			},
+			decide(input, options) {
+				return executeTrackChangesDecide(adapters.trackChanges, input, options);
 			}
 		},
-		review: { decide(input, options) {
-			return executeReviewDecide(adapters.trackChanges, input, options);
-		} },
 		create: {
 			paragraph(input, options) {
 				return executeCreateParagraph(adapters.create, input, options);
@@ -23289,7 +23319,7 @@ var REQUIRED_COMMANDS = {
 		"setCommentInternal"
 	],
 	"comments.delete": ["removeComment"],
-	"review.decide": [
+	"trackChanges.decide": [
 		"acceptTrackedChangeById",
 		"rejectTrackedChangeById",
 		"acceptAllTrackedChanges",
@@ -25122,22 +25152,22 @@ function executeStyleApply(editor, tr, target, step, mapping) {
 	const markEntries = [
 		[
 			"bold",
-			step.args.marks.bold,
+			step.args.inline.bold,
 			schema.marks.bold
 		],
 		[
 			"italic",
-			step.args.marks.italic,
+			step.args.inline.italic,
 			schema.marks.italic
 		],
 		[
 			"underline",
-			step.args.marks.underline,
+			step.args.inline.underline,
 			schema.marks.underline
 		],
 		[
 			"strike",
-			step.args.marks.strike,
+			step.args.inline.strike,
 			schema.marks.strike
 		]
 	];
@@ -25217,22 +25247,22 @@ function executeSpanStyleApply(editor, tr, target, step, mapping) {
 	const markEntries = [
 		[
 			"bold",
-			step.args.marks.bold,
+			step.args.inline.bold,
 			schema.marks.bold
 		],
 		[
 			"italic",
-			step.args.marks.italic,
+			step.args.inline.italic,
 			schema.marks.italic
 		],
 		[
 			"underline",
-			step.args.marks.underline,
+			step.args.inline.underline,
 			schema.marks.underline
 		],
 		[
 			"strike",
-			step.args.marks.strike,
+			step.args.inline.strike,
 			schema.marks.strike
 		]
 	];
@@ -25822,7 +25852,7 @@ function styleApplyWrapper(editor, input, options) {
 			message: "format.apply requires a non-collapsed target range."
 		}
 	};
-	const markKeys$1 = Object.keys(input.marks).filter((k$1) => input.marks[k$1] !== void 0);
+	const markKeys$1 = Object.keys(input.inline).filter((k$1) => input.inline[k$1] !== void 0);
 	for (const key$1 of markKeys$1) {
 		const schemaName = MARK_KEY_TO_SCHEMA_NAME[key$1];
 		if (schemaName) requireSchemaMark(editor, schemaName, "format.apply");
@@ -25843,7 +25873,7 @@ function styleApplyWrapper(editor, input, options) {
 				id: stepId,
 				op: "format.apply",
 				where: STUB_WHERE,
-				args: { marks: input.marks }
+				args: { inline: input.inline }
 			},
 			targets: [{
 				kind: "range",
@@ -30048,7 +30078,7 @@ var Editor = class Editor extends EventEmitter$1 {
 		return migrations.length > 0;
 	}
 	processCollaborationMigrations() {
-		console.debug("[checkVersionMigrations] Current editor version", "1.17.0-next.6");
+		console.debug("[checkVersionMigrations] Current editor version", "1.17.0-next.7");
 		if (!this.options.ydoc) return;
 		let docVersion = this.options.ydoc.getMap("meta").get("version");
 		if (!docVersion) docVersion = "initial";