Skip to content

feat: add pre-run pipeline validation warnings#324

Open
xxivani wants to merge 1 commit intoc2siorg:mainfrom
xxivani:feat/pipeline-validation-warnings
Open

feat: add pre-run pipeline validation warnings#324
xxivani wants to merge 1 commit intoc2siorg:mainfrom
xxivani:feat/pipeline-validation-warnings

Conversation

@xxivani
Copy link
Copy Markdown
Contributor

@xxivani xxivani commented Apr 23, 2026

Description

Adds a pre-run pipeline validation pass that checks for known operator incompatibilities before sending the pipeline to the backend. A new validatePipeline utility walks the pipeline steps tracking channel state (colour vs grayscale) and flags operators that will likely fail given their input. Warnings are shown in a PipelineWarningsModal with a Cancel / Run Anyway choice so the check is never a hard blocker.
In reference to pr #198

Fixes #197

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

How Has This Been Tested?

Tested the following cases manually: grayscale operator into a colour-only block, colour image into a grayscale-only block, missing Read Image block, and a valid pipeline with no warnings (runs immediately without showing the modal).

  • Existing tests pass
  • New tests added
  • Manual testing

Screenshots (if applicable)

Screenshot 2026-03-12 at 9 17 03 PM

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review
  • I have added/updated documentation as needed
  • My changes generate no new warnings
  • Tests pass locally

@kx7m2qd
Copy link
Copy Markdown

kx7m2qd commented May 1, 2026

Reviewed the validatePipeline logic the channel state tracking approach (colour vs grayscale) looks clean. One thought: what happens if the pipeline has zero steps? Does validatePipeline return empty warnings and skip the modal entirely, or does it still show it? Also noticed the branch is out of date with main — might be worth a rebase before merge to avoid surprises.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] - Pipeline errors only surface at runtime: add pre-run validation warnings

2 participants