Add Pydantic-based config validation and type correction#310

Open

MaStr wants to merge 19 commits intomainfrom

claude/pydantic-config-conversion-Qc7UW

Commits on Apr 2, 2026

Add Pydantic config validation to eliminate type coercion bugs

claude
authored and
MaStr
committed
Enhance Pydantic config: history_days parsing, inverter alias, loglevel validation

claude
authored and
MaStr
committed
Add missing cache_ttl to InverterConfig, fix mqtt inverter passthrough

claude
authored and
MaStr
committed
Add Pydantic config migration docs for AI-assisted development

claude
authored and
MaStr
committed
Address PR review: exclude_none, required pvinstallations, cache_ttl default

MaStr
and
claude
committed
Address PR review round 2: nested HA config parsing, validator mutation

MaStr
and
claude
committed
Address PR review round 3: Optional vat/fees/markup, nested dict coercion, lint

MaStr
and
claude
committed
Fix docs: correct UtilityConfig types, provider list, and parsing description

MaStr
and
claude
committed
Address PR review round 4: empty YAML guard, rename misleading test class

MaStr
and
claude
committed
Address PR review round 5: mutable defaults, float(None) guard, defensive validation

MaStr
and
claude
committed
Address PR review round 6: ValidationError wrapping, yaml.safe_dump in tests

MaStr
and
claude
committed
Address PR review round 7: YAML error handling, inverter type validation, name required

MaStr
and
claude
committed
Address PR review round 8: int power rates, sanitize error messages, fix comment

MaStr
and
claude
committed
Fix docs: align power rate types with InverterConfig (float → int)

MaStr
and
claude
committed
Fix test assertions: max_grid_charge_rate is int, not float

MaStr
and
claude
committed
Address PR review round 11: empty-string validation, capacity > 0, annual_consumption None

MaStr
and
claude
committed
Address PR review round 12: PV name whitespace validation, provider required fields

MaStr
and
claude
committed
Make timezone required in BatcontrolConfig (no default)

MaStr
and
claude
committed
Housekeeping: update stale comments after rebase on main

MaStr
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add Pydantic-based config validation and type correction#310

Add Pydantic-based config validation and type correction#310
MaStr wants to merge 19 commits intomainfrom
claude/pydantic-config-conversion-Qc7UW

Commits on Apr 2, 2026

Add Pydantic config validation to eliminate type coercion bugs

Enhance Pydantic config: history_days parsing, inverter alias, loglevel validation

Add missing cache_ttl to InverterConfig, fix mqtt inverter passthrough

Add Pydantic config migration docs for AI-assisted development

Address PR review: exclude_none, required pvinstallations, cache_ttl default

Address PR review round 2: nested HA config parsing, validator mutation

Address PR review round 3: Optional vat/fees/markup, nested dict coercion, lint

Fix docs: correct UtilityConfig types, provider list, and parsing description

Address PR review round 4: empty YAML guard, rename misleading test class

Address PR review round 5: mutable defaults, float(None) guard, defensive validation

Address PR review round 6: ValidationError wrapping, yaml.safe_dump in tests

Address PR review round 7: YAML error handling, inverter type validation, name required

Address PR review round 8: int power rates, sanitize error messages, fix comment

Fix docs: align power rate types with InverterConfig (float → int)

Fix test assertions: max_grid_charge_rate is int, not float

Address PR review round 11: empty-string validation, capacity > 0, annual_consumption None

Address PR review round 12: PV name whitespace validation, provider required fields

Make timezone required in BatcontrolConfig (no default)

Housekeeping: update stale comments after rebase on main