Repository navigation
Description
Task Summary
Switching between a workflow's two views -- the operator canvas (/user/workflow/:id) and the Form View (/user/workflow/:id/form) -- is a full browser page load in both directions (window.location.href in MenuComponent.openFormViewPage and WorkflowFormComponent.openCanvasPage). The two views are views of one open workflow, so everything that makes the workflow live is thrown away on the way out and rebuilt on the other side:
- both
ngOnDestroys callclearWorkflow()(which destroys the Yjs shared document and leaves its co-editing room),computingUnitStatusService.disconnect(),resetExecutionAndWorkers(), and clear the console and results; - the arriving view then re-bootstraps Angular (
APP_INITIALIZERblocks on/api/config), re-parses the whole bundle, re-fetches operator metadata that aprovidedIn: "root"singleton had already cached for the page, fetches the workflow, opens a new shared document and room, reconnects the computing unit, and waits for the backend to report the execution state again.
The visible cost is seconds of blank page on every switch, and a run that was already going takes that long to say so again. Nothing about it is necessary: WorkflowWebsocketService, ComputingUnitStatusService, WorkflowActionService and the joint graph it owns are all root-provided and survive a route change.
Routing between the two was tried during the Form View work and reverted; the code comments record three symptoms, two of which have identified causes:
- A ghost coeditor of yourself. The arriving view built a second shared document for the same workflow, so the page joined the room it was already in and saw its own other client.
- Undraggable operators. A JointJS paper binds to the joint graph, which is root-provided and outlives the component that created it, and no paper is ever disposed -- neither
WorkflowEditorComponent's norMiniMapComponent's. Every mount left another paper listening to that graph from a detached DOM node, and whichever paper answered a pointer event decided whether an operator could be dragged. Harmless while every mount followed a page load. - Broken runs. No independent cause found; likely a consequence of the first two, but this is the part that most needs checking in a browser.
Proposed: route in both directions, have the departing view hand the session to the arriving one instead of tearing it down, have the arriving view attach to what it was given instead of rebuilding it, and dispose both papers on destroy. Leaving the workspace altogether keeps today's teardown, including on unload.
Follow-up to the Form View feature (parent issue #8011); the switch itself landed in #8456.
Task Type
- Performance
- Refactor / Cleanup
Activity
github-actions commentedon Sep 17, 2026
👋 Thanks for opening this issue, @yangzhang75!
It looks like the issue description doesn't quite follow our template yet:
Filling out the template helps reviewers understand and triage your contribution faster. Please edit the description to complete it. This message will disappear automatically once the template is followed.
You can find the template prompts by editing the description, or see CONTRIBUTING.md for the full contribution flow.
docs(frontend): say only what held up about the undisposed papers
docs(frontend): say only what held up about the undisposed papers
perf(workflow): switch between a workflow's two views without reloadi…