Added LLM-based intent validation in ModelBuilder

[AumPatel1]

In ModelBuilder added a _validate_intent() method that performs basic checks and LLM-based validation ,to check that intent passed is not too long, empty and valid

[greptile-apps]

Greptile Overview

Greptile Summary

Added intent validation to ModelBuilder.build() that performs basic checks (non-empty, length constraints) and LLM-based semantic validation before starting the model build process.

Key changes:

• New _validate_intent() method with two-stage validation: basic checks followed by LLM quality assessment
• LLM validation uses the tool provider to check if intent is clear, actionable, and well-defined for ML
• Graceful degradation: if LLM validation fails due to parsing/API errors, it logs a warning but allows the build to proceed (basic validation already passed)
• Validation is called early in build() method (line 179) after parameter validation but before expensive agent operations

The implementation follows good error handling patterns with explicit ValueError re-raising for actual validation failures while catching and logging other exceptions for graceful degradation.

Confidence Score: 4/5

• Safe to merge with one minor type annotation issue that should be fixed
• The implementation is well-structured with proper error handling and graceful degradation. There's one type annotation inconsistency (feedback field marked as optional in comment but required in code) that should be corrected, but it won't cause runtime failures since the LLM will provide the field. The validation logic is sound and the placement is appropriate.
• plexe/model_builder.py - fix type annotation inconsistency on line 92

Important Files Changed

File Analysis

Filename	Score	Overview
plexe/model_builder.py	4/5	Added LLM-based intent validation with basic checks and graceful degradation; validation runs before model build starts

[greptile-apps]

Additional Comments (1)

1. plexe/model_builder.py, line 92 (link)

   syntax: comment says "Optional feedback" but feedback: str is required - should be feedback: Optional[str] = None or remove "Optional" from comment

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[marcellodebernardi]

Closing this PR as stale. Unfortunately it was opened at a time when we were not actively maintaining this project, and now relates to code that no longer exists.

However, I do see value in the notion of validating whether the input intent sufficiently specifies the problem. If you'd like to reopen and update this PR for the current version of the project, feel free to. Thanks for being interesting in contributing!