Skip to content

New frontend websocket config #86

Open
amaloney1 wants to merge 2 commits intofeature/pipeline-structfrom
frontend/websocket-package
Open

New frontend websocket config #86
amaloney1 wants to merge 2 commits intofeature/pipeline-structfrom
frontend/websocket-package

Conversation

@amaloney1
Copy link
Copy Markdown
Contributor

Pull Request Template

What?

Removed old frontend websocket configs (per-node) and replaced it with the name style pipeline (single, ordered pipeline). Enforced some order based on workflow document.


How?

Websocket config:

  • Built a PipelinePayload from the React Flow graph using topological sorting for execution order.
  • Added defaults for preprocessing and window configs
  • Moved node config handling so that filter/window nodes push config into their node data rather than dispatching global events (pipeline is therefore built directly from graph state)
  • Replaced per-config websocket messages with a single pipeline payload send (on pipeline changes and websocket open)

Workflow enforcement:

  • Added validatePipeline to compute errors and warnings before sending the pipeline
  • Enforced structural constraints including: only 1 source and window node, window must connect to source directly, output nodes are terminal, etc.
  • added console warnings for ordering issues (e.g. filter before window)

Testing

Ran the local frontend to briefly test the order changes made. Feature/pipeline/backend tested the compatibility of the websocket change with the backend workflow.


@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 4, 2026

❌ PR approval conditions NOT satisfied. At least 1 team lead OR 2 team members must approve before merging.
[Last checked: 2026-04-04T21:28:48.074Z]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant