Skip to content

Add document resource to n8n node and sendSigningNotifications to mcp#1035

Merged
SachaProbo merged 1 commit intomainfrom
SachaProbo/n8n-signature
Apr 13, 2026
Merged

Add document resource to n8n node and sendSigningNotifications to mcp#1035
SachaProbo merged 1 commit intomainfrom
SachaProbo/n8n-signature

Conversation

@SachaProbo
Copy link
Copy Markdown
Contributor

@SachaProbo SachaProbo commented Apr 13, 2026

Summary by cubic

Adds a new document resource to the Probo n8n node with 21 operations for documents, versions, and signatures, and adds sendSigningNotifications to MCP and the node to trigger pending signature reminders.

  • New Features

    • Document resource: create/get/list/update/delete; archive/unarchive.
    • Versions: draft create/update/delete; publish major/minor; get/list versions.
    • Approvals and signatures: request/void approvals; request/cancel signatures; get/list signatures; send signing notifications (MCP tool and node action).
  • Bug Fixes

    • update/updateVersion: move optional fields into updateFields so they’re only sent when set, avoiding unintended overwrites.
    • updateVersion: set valid enum defaults for classification and documentType to satisfy n8n linting.

Written for commit d40b114. Summary will update on new commits.

@SachaProbo SachaProbo requested a review from gearnode April 13, 2026 15:06
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 26 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="packages/n8n-node/nodes/Probo/actions/document/update.operation.ts">

<violation number="1" location="packages/n8n-node/nodes/Probo/actions/document/update.operation.ts:43">
P1: `trustCenterVisibility` defaults to `'NONE'`, so update requests always overwrite visibility even when the user did not intend to change that field.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Copy link
Copy Markdown
Contributor

@gearnode gearnode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SachaProbo SachaProbo force-pushed the SachaProbo/n8n-signature branch from 38c589d to 0e1f449 Compare April 13, 2026 15:27
Add a complete document resource to the n8n node with 21 operations
covering documents, versions, and signatures — matching the MCP
specification. Also add the sendSigningNotifications tool to the MCP
API for triggering pending signature reminders.

n8n operations: create, get, getAll, update, delete, archive,
unarchive, getVersion, getAllVersions, createDraftVersion,
updateVersion, deleteDraftVersion, publishMajorVersion,
publishMinorVersion, requestApproval, voidApproval, getSignature,
getAllSignatures, requestSignature, cancelSignature,
sendSigningNotifications.

Signed-off-by: Sacha Al Himdani <sacha@getprobo.com>
@SachaProbo SachaProbo force-pushed the SachaProbo/n8n-signature branch from 0e1f449 to d40b114 Compare April 13, 2026 15:31
@SachaProbo SachaProbo merged commit d40b114 into main Apr 13, 2026
17 checks passed
@SachaProbo SachaProbo deleted the SachaProbo/n8n-signature branch April 13, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants