Conversation
Agent-Logs-Url: https://github.com/dfeen87/HLV-RAPS/sessions/76397be0-fc07-4325-a52e-47c4f90dc2cd Co-authored-by: dfeen87 <158860247+dfeen87@users.noreply.github.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
Normalizes the “prediction mismatch” failure naming to a single canonical spelling so supervisor failure reasons (enum + emitted string) remain consistent across runtime code paths, improving log/metric/API reason stability.
Changes:
- Renamed
RedundantSupervisor::FailureMode::MISMATED_PREDICTIONtoMISMATCHED_PREDICTION. - Updated the supervisor failure-reason string mapping to emit
"MISMATCHED_PREDICTION"for that failure mode.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
include/supervisor/redundant_supervisor.hpp |
Renames the FailureMode enum value to the canonical MISMATCHED_PREDICTION. |
src/supervisor/supervisor_failure_strings.hpp |
Updates stringification to match the canonical failure-mode name. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| CRITICAL_NO_ROLLBACK, | ||
| PRIMARY_CHANNEL_LOCKUP, | ||
| MISMATED_PREDICTION | ||
| MISMATCHED_PREDICTION |
There was a problem hiding this comment.
Renaming an enum constant in a public header is a source-breaking API change for downstream users that may still reference the misspelled name. Consider keeping MISMATED_PREDICTION as a deprecated alias to MISMATCHED_PREDICTION (same underlying value) so existing code continues to compile while the canonical spelling is used for logging/metrics.
| MISMATCHED_PREDICTION | |
| MISMATCHED_PREDICTION, | |
| [[deprecated("Use MISMATCHED_PREDICTION")]] MISMATED_PREDICTION = | |
| MISMATCHED_PREDICTION |
This addresses consistency bugs in supervisor failure signaling where prediction-mismatch was spelled differently across active components, creating divergent reason strings and enum semantics. The change aligns naming to a single canonical value to keep logs/metrics/API-facing failure reasons consistent.
Failure mode normalization
RedundantSupervisor::FailureMode::MISMATED_PREDICTIONtoMISMATCHED_PREDICTIONin:include/supervisor/redundant_supervisor.hppReason string alignment
src/supervisor/supervisor_failure_strings.hppMISMATCHED_PREDICTIONconsistently.Scope