action: from_yaml() returns confusing error for empty YAML #10546

New issue

Open

opened 2026-04-18 17:07:36 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-18 17:07:36 +00:00

Owner

Metadata

Commit: Current HEAD
Branch: main
File: src/cleveragents/action/schema.py
Lines: 365-368

Background and Context

When yaml.safe_load() returns None (from an empty YAML file like "---" or just whitespace), the error message says "got NoneType" which is confusing to users. A clearer message would explicitly state "empty YAML file" to help users understand what went wrong.

Expected Behavior

When an empty YAML file is provided, the error message should clearly indicate that the YAML file is empty and cannot be parsed as an Action schema.

Acceptance Criteria

Empty YAML files are detected and handled explicitly
Error message clearly states "empty YAML file"
Error message is helpful and actionable
Existing tests pass with improved error message
New tests verify error message for empty YAML

Subtasks

Add check for None result from yaml.safe_load()
Raise clear error with message "empty YAML file" or similar
Add test cases for empty YAML inputs ("---", whitespace, etc.)
Verify error message is clear and helpful

Definition of Done

Empty YAML files produce clear error messages
Error message explicitly mentions "empty YAML file"
All tests pass
Code review approved

Automated by CleverAgents Bot
Agent: new-issue-creator

## Metadata - **Commit**: Current HEAD - **Branch**: main - **File**: src/cleveragents/action/schema.py - **Lines**: 365-368 ## Background and Context When yaml.safe_load() returns None (from an empty YAML file like "---" or just whitespace), the error message says "got NoneType" which is confusing to users. A clearer message would explicitly state "empty YAML file" to help users understand what went wrong. ## Expected Behavior When an empty YAML file is provided, the error message should clearly indicate that the YAML file is empty and cannot be parsed as an Action schema. ## Acceptance Criteria - [ ] Empty YAML files are detected and handled explicitly - [ ] Error message clearly states "empty YAML file" - [ ] Error message is helpful and actionable - [ ] Existing tests pass with improved error message - [ ] New tests verify error message for empty YAML ## Subtasks - [ ] Add check for None result from yaml.safe_load() - [ ] Raise clear error with message "empty YAML file" or similar - [ ] Add test cases for empty YAML inputs ("---", whitespace, etc.) - [ ] Verify error message is clear and helpful ## Definition of Done - Empty YAML files produce clear error messages - Error message explicitly mentions "empty YAML file" - All tests pass - Code review approved --- **Automated by CleverAgents Bot** Agent: new-issue-creator

Rows
Columns