czlonkowski a78d7404c0 fix: filter out description field in workflow updates (issue #431) ...

Root Cause:
- n8n recently added a `description` field to workflows
- The field is returned by GET /workflows/:id endpoint
- But PUT /workflows/:id rejects it with "additional properties" error
- Our cleanWorkflowForUpdate() wasn't filtering it out

The Fix:
1. Added `description` to the list of removed fields in cleanWorkflowForUpdate()
2. Added `description` field to Workflow type definition
3. Added test case to verify description field is excluded

Code Review Improvements:
- Removed diagnostic logging code (was only needed for debugging)
- Improved comment accuracy in type definition
- Added comprehensive test coverage for description field

Testing:
- Build succeeds
- New test passes: "should exclude description field for n8n API compatibility"
- All unit tests pass
- Integration tests confirmed fix works with real n8n API

Related: #431

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en

2025-11-20 18:30:36 +01:00

68 MiB

Raw Permalink History

The file is too large to be shown. View Raw