Problem
The duration parser silently ignores trailing numbers without units.
Location: src/core/schedule.rs:127-154
Input like "5m30" would parse as 5 minutes, silently ignoring the trailing 30.
Recommendation
Check if current_num is non-empty after the loop:
if !current_num.is_empty() {
return Err(ScheduleError::InvalidInterval(format!(
"number without unit: {}",
s
)));
}
Priority
Priority Fix - Silent parsing errors lead to unexpected behavior.
Created from PR review: #28 (comment)
Problem
The duration parser silently ignores trailing numbers without units.
Location:
src/core/schedule.rs:127-154Input like
"5m30"would parse as 5 minutes, silently ignoring the trailing30.Recommendation
Check if
current_numis non-empty after the loop:Priority
Priority Fix - Silent parsing errors lead to unexpected behavior.
Created from PR review: #28 (comment)