feat(core): Use active version instead of current version (no-changelog)

[dariacodes]

Summary

This PR decouples the active version from the current version of the workflow:

• Add a foreign (nullable) key activeVersionId to the workflow entity
  • It does not currently replace the active flag, but we will tackle this as tech debt later.
• Automatic executions now always run the populated active version of the workflow. Manual executions still use the data that is sent.
• Update public API endpoints with keeping the current behavior
  • Update activeVersion with the current version when updating and activating workflows
  • Return activeVersion for GET endpoints (it'll be the same as the current version until user starts using V2 endpoints)
  • Remove excludePinnedData from activate endpoint (it was unused and using "GET" type was semantically incorrect)
• Update the logic to always first add a record to workflow history and only then update the workflow entity, so that activeVersion is set once along with the workflow entity update

Active Version Usage

These are now using the active workflow version:

1. Workflow Activation [active-workflow-manager.ts - add]
   • When activating workflows, the system now uses activeVersion nodes and connections instead of current version.
2. Webhook Cleanup [active-workflow-manager.ts - clearWebhooks]
   • When deactivating workflows and removing webhooks, the system now uses activeVersion to properly identify and clean up the correct webhooks.
3. Activation Error Handling [active-workflow-manager.ts - activateWorkflow]
   • When workflow activation fails and throws an error workflow needs to execute, it now uses activeVersion instead of current version.
4. Production Webhook Execution [live-webhooks.ts - executeWebhook]
   • Production webhooks now use activeVersion instead of current version.

What I tested and how

• For public endpoints, I created workflows with an HTTP Request node that sends a payload to activate or update another workflow.
• For internal endpoints, I manually saved inactive and active workflows and checked which version was executed.
• I verified that the current behavior remains unchanged and that the active version from the DB is automatically executed, while for manual executions, the current version is executed. Execution data table had the right data.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-4277/feature-be-decouple-current-workflow-version-from-active-version

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.
• PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

[codecov]

Codecov Report

❌ Patch coverage is 75.12690% with 49 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...s/common/1763047800000-AddActiveVersionIdColumn.ts	21.42%	11 Missing ⚠️
...es/@n8n/db/src/repositories/workflow.repository.ts	16.66%	9 Missing and 1 partial ⚠️
packages/cli/src/active-workflow-manager.ts	56.25%	7 Missing ⚠️
packages/cli/src/workflows/workflow.service.ts	88.46%	3 Missing ⚠️
...editor-ui/src/app/composables/useWorkflowSaving.ts	0.00%	3 Missing ⚠️
...db/src/repositories/workflow-history.repository.ts	0.00%	2 Missing ⚠️
.../src/modules/chat-hub/chat-hub-workflow.service.ts	0.00%	2 Missing ⚠️
...lic-api/v1/handlers/workflows/workflows.handler.ts	91.66%	2 Missing ⚠️
packages/cli/src/webhooks/live-webhooks.ts	33.33%	2 Missing ⚠️
...ditor-ui/src/app/composables/useWorkflowHelpers.ts	33.33%	2 Missing ⚠️
... and 4 more

📢 Thoughts on this report? Let us know!

[cubic-dev-ai]

No issues found across 15 files

[currents-bot]

⏳

E2E Tests: n8n tests timed out after 30m

🟢 83 · 🔴 0 · ⚪️ 2 · 🟣 1

View Run Details

Run Details

• Project: n8n

• Groups: 2

• Framework: Playwright

• Run Status: Timed out

• Commit: ba03d02

• Spec files: 95

• Overall tests: 595

• Duration: 30m

• Parallelization: 1

Groups

GroupId	Results	Spec Files Progress
ui	🟢 83 · 🔴 0 · ⚪️ 2 · 🟣 1	8 / 88
ui:isolated	🟢 0 · 🔴 0 · ⚪️ 0	0 / 7

---

This message was posted automatically by currents.dev | Integration Settings

[tomi]

Just one nit comment. Feel free to merge without addressing