Skip to content

Validate VLAs and templates during creation/definition - #87

Merged
bzp99 merged 7 commits into
yassine-refactorfrom
feat/vla-evaluation-validation
Sep 14, 2026
Merged

bzp99 merged 7 commits into
yassine-refactorfrom
feat/vla-evaluation-validation

Conversation

@MYRhouma

@MYRhouma MYRhouma commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

This validates rendered evaluation logic before it is added to a VLA.

The VLA Manager distinguishes invalid template input, rendering problems, invalid engine syntax, unavailable evaluation services, and valid requirements. Invalid requirements are blocked with feedback that explains what needs correction.

The implementation adds a processing validation endpoint, a VLA Manager template-validation endpoint, and frontend feedback in the requirement dialog.

Related work

The validation contract is designed to work alongside the processing endpoint work in #77 without merging the older branch wholesale.

Checks

  • DVA Processing: 12 tests passed
  • VLA Manager API: 44 tests passed
  • VLA Manager API lint passed
  • VLA Manager frontend tests: 4 passed
  • VLA Manager production build passed

CC @bzp99

@MYRhouma
MYRhouma marked this pull request as ready for review September 3, 2026 12:59
Copilot AI lite review requested due to automatic review settings September 3, 2026 12:59

This comment was marked as outdated.

@bzp99 bzp99 changed the title feat(vla): validate evaluation logic before use Validate VLAs and templates during creation/definition Sep 14, 2026
@bzp99
bzp99 force-pushed the feat/vla-evaluation-validation branch from f1b374c to d04dec8 Compare September 14, 2026 17:51
@bzp99
bzp99 merged commit d04dec8 into yassine-refactor Sep 14, 2026
9 checks passed
@bzp99
bzp99 deleted the feat/vla-evaluation-validation branch September 14, 2026 17:55
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.

3 participants