fix(editor): Use direct store reference for connections in initializeWorkspace (backport to release-candidate/2.14.x)

[n8n-assistant]

Description

Backport of #27552 to release-candidate/2.14.x.

Checklist for the author (@r00gm) to go through.

• Review the backport changes
• Fix possible conflicts
• Merge to target branch

After this PR has been merged, it will be picked up in the next patch release for release track.

Original description

Summary

initializeWorkspace set nodes via the direct initializedDocumentStore reference but set connections via the computed workflowDocumentStore?.value?. After resetWorkspace() clears workflowId to '', the computed evaluates to undefined and setConnections is silently skipped — so no connections are loaded.

This broke the workflow preview (used on n8n.io) for all workflows, most visibly for AI agent nodes whose shape depends on sub-node connections (model, memory, tool). Without connections, the agent renders as a plain square instead of a wide rectangle with input slots.

The fix uses initializedDocumentStore consistently for both setNodes and setConnections, matching the direct reference already returned by initState.

Regression introduced in: #27280

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-4999

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.

[cubic-dev-ai]

No issues found across 2 files

Architecture diagram

sequenceDiagram
    participant UI as Editor UI
    participant Ops as useCanvasOperations
    participant WfStore as Workflows Store
    participant DocStore as Workflow Document Store

    Note over UI, DocStore: Workspace Initialization (e.g. Loading a Workflow or Preview)

    UI->>Ops: initializeWorkspace(workflowData)
    
    Ops->>WfStore: Check current workflow ID
    Note right of WfStore: If resetWorkspace() was called, ID is ""

    Ops->>DocStore: useWorkflowDocumentStore(workflowData.id)
    DocStore-->>Ops: initializedDocumentStore (Direct Reference)

    Ops->>DocStore: setNodes(workflowData.nodes)

    alt Connection Handling
        Note over Ops, DocStore: Fix: Avoid dependency on reactive 'workflowId' computed property
        Ops->>DocStore: CHANGED: setConnections(workflowData.connections)
        Note right of Ops: Now uses the 'initializedDocumentStore' reference directly
    end

    Ops->>WfStore: Update global workflow metadata (ID, name, etc.)
    
    Note over Ops, DocStore: UI components (like AI Agents) now render with correct input slots
    
    Ops-->>UI: return { workflowDocumentStore }

Loading

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[n8n-assistant]

Got released with n8n@2.14.2