Summary
When evaluation exceeds the configured timeout, the evaluator should return partial results collected so far instead of returning a fatal LibmagicError::Timeout error.
Context
Per Core Flow 12 (Evaluation Timeout Handling), timeout should be a graceful degradation, not a hard failure. The current implementation at src/evaluator/mod.rs:402 returns an error, discarding any partial matches found before the timeout.
Current behavior: Err(LibmagicError::Timeout { timeout_ms }) - all progress lost
Expected behavior: Ok(EvaluationResult { metadata: { timed_out: true }, ... }) - partial results preserved
Acceptance Criteria
Files to Modify
src/evaluator/mod.rs - Change timeout handling to return partial resultssrc/output/json.rs - Include timeout metadata in JSON outputsrc/main.rs - CLI warning message on timeout
References
Summary
When evaluation exceeds the configured timeout, the evaluator should return partial results collected so far instead of returning a fatal
LibmagicError::Timeouterror.Context
Per Core Flow 12 (Evaluation Timeout Handling), timeout should be a graceful degradation, not a hard failure. The current implementation at
src/evaluator/mod.rs:402returns an error, discarding any partial matches found before the timeout.Current behavior:
Err(LibmagicError::Timeout { timeout_ms })- all progress lostExpected behavior:
Ok(EvaluationResult { metadata: { timed_out: true }, ... })- partial results preservedAcceptance Criteria
EvaluationMetadata.timed_outis set totruewhen timeout occurs"timed_out": trueand"partial_results": truein metadataFiles to Modify
src/evaluator/mod.rs- Change timeout handling to return partial resultssrc/output/json.rs- Include timeout metadata in JSON outputsrc/main.rs- CLI warning message on timeoutReferences