From 909c54033cca63e977c808b447cc510eb94ce2f9 Mon Sep 17 00:00:00 2001
From: Sacha Al Himdani <sacha@probo.com>
Date: Tue, 7 Jul 2026 15:43:35 +0200
Subject: [PATCH] Expand n8n document update fields

Guard the content field against empty strings so adding it in the UI
without a value no longer sends content: "" to the API, which is not
valid ProseMirror content. This matches the existing defaultApproverIds
guard.

Also expose title, classification, and document type on the update
operation. Like content, these edit the current draft version, creating
one from the latest published version when none exists.

Signed-off-by: Sacha Al Himdani <sacha@probo.com>
---
 .../actions/document/update.operation.ts      | 60 ++++++++++++++++---
 1 file changed, 52 insertions(+), 8 deletions(-)

diff --git a/packages/n8n-node/nodes/Probo/actions/document/update.operation.ts b/packages/n8n-node/nodes/Probo/actions/document/update.operation.ts
index c0f05b1ef..8d5a7d0e9 100644
--- a/packages/n8n-node/nodes/Probo/actions/document/update.operation.ts
+++ b/packages/n8n-node/nodes/Probo/actions/document/update.operation.ts
@@ -43,6 +43,19 @@ export const description: INodeProperties[] = [
 			},
 		},
 		options: [
+			{
+				displayName: 'Classification',
+				name: 'classification',
+				type: 'options',
+				options: [
+					{ name: 'Confidential', value: 'CONFIDENTIAL' },
+					{ name: 'Internal', value: 'INTERNAL' },
+					{ name: 'Public', value: 'PUBLIC' },
+					{ name: 'Secret', value: 'SECRET' },
+				],
+				default: 'INTERNAL',
+				description: 'The classification of the document. Updating it edits the current draft version, creating one from the latest published version if none exists.',
+			},
 			{
 				displayName: 'Content',
 				name: 'content',
@@ -53,6 +66,38 @@ export const description: INodeProperties[] = [
 				default: '',
 				description: 'The body of the document as a ProseMirror document JSON string. Updating it edits the current draft version, creating one from the latest published version if none exists.',
 			},
+			{
+				displayName: 'Default Approver IDs',
+				name: 'defaultApproverIds',
+				type: 'string',
+				default: '',
+				description: 'Comma-separated list of default approver profile IDs',
+			},
+			{
+				displayName: 'Document Type',
+				name: 'documentType',
+				type: 'options',
+				options: [
+					{ name: 'Governance', value: 'GOVERNANCE' },
+					{ name: 'Other', value: 'OTHER' },
+					{ name: 'Plan', value: 'PLAN' },
+					{ name: 'Policy', value: 'POLICY' },
+					{ name: 'Procedure', value: 'PROCEDURE' },
+					{ name: 'Record', value: 'RECORD' },
+					{ name: 'Register', value: 'REGISTER' },
+					{ name: 'Report', value: 'REPORT' },
+					{ name: 'Template', value: 'TEMPLATE' },
+				],
+				default: 'POLICY',
+				description: 'The type of the document. Updating it edits the current draft version, creating one from the latest published version if none exists.',
+			},
+			{
+				displayName: 'Title',
+				name: 'title',
+				type: 'string',
+				default: '',
+				description: 'The title of the document. Updating it edits the current draft version, creating one from the latest published version if none exists.',
+			},
 			{
 				displayName: 'Trust Center Visibility',
 				name: 'trustCenterVisibility',
@@ -65,13 +110,6 @@ export const description: INodeProperties[] = [
 				default: 'NONE',
 				description: 'The trust center visibility of the document',
 			},
-			{
-				displayName: 'Default Approver IDs',
-				name: 'defaultApproverIds',
-				type: 'string',
-				default: '',
-				description: 'Comma-separated list of default approver profile IDs',
-			},
 		],
 	},
 ];
@@ -82,7 +120,10 @@ export async function execute(
 ): Promise<INodeExecutionData> {
 	const documentId = this.getNodeParameter('documentId', itemIndex) as string;
 	const updateFields = this.getNodeParameter('updateFields', itemIndex, {}) as {
+		title?: string;
 		content?: string;
+		classification?: string;
+		documentType?: string;
 		trustCenterVisibility?: string;
 		defaultApproverIds?: string;
 	};
@@ -119,7 +160,10 @@ export async function execute(
 	`;
 
 	const input: Record<string, unknown> = { id: documentId };
-	if (updateFields.content !== undefined) input.content = updateFields.content;
+	if (updateFields.title !== undefined && updateFields.title !== '') input.title = updateFields.title;
+	if (updateFields.content !== undefined && updateFields.content !== '') input.content = updateFields.content;
+	if (updateFields.classification !== undefined) input.classification = updateFields.classification;
+	if (updateFields.documentType !== undefined) input.documentType = updateFields.documentType;
 	if (updateFields.trustCenterVisibility !== undefined) input.trustCenterVisibility = updateFields.trustCenterVisibility;
 	if (updateFields.defaultApproverIds !== undefined && updateFields.defaultApproverIds !== '') {
 		input.defaultApproverIds = updateFields.defaultApproverIds.split(',').map(id => id.trim()).filter(Boolean);