Feature: Plan Generation Validation Fix
  Regression test for bug where _validate always passed for code longer than 10 characters.

  Scenario: Validation properly fails when LLM response contains FAIL and code is long
    Given a PlanGenerationGraph instance
    And generated code longer than 10 characters
    And the LLM validation response is "FAIL: issues found"
    When the validation node runs
    Then the validation status should be "FAIL"
    And the bug where length over 10 characters forced PASS should be fixed

  Scenario: Validation properly fails when LLM response contains REJECTED
    Given a PlanGenerationGraph instance
    And generated code longer than 10 characters
    And the LLM validation response is "REJECTED: unsafe patterns detected"
    When the validation node runs
    Then the validation status should be "FAIL"
    And the validation should respect LLM rejection regardless of code length

  Scenario: Validation properly passes when LLM response contains PASS
    Given a PlanGenerationGraph instance
    And generated code longer than 10 characters
    And the LLM validation response is "PASS: all checks successful"
    When the validation node runs
    Then the validation status should be "PASS"