refactor(automation-runtime): classify skip timeout via ledger reason - #1427
Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
c1efdfe to
74ccba2
Compare
Route skipped_terminal_failure_class through AutomationSkipReasonV1 instead of an inline English string match. Delete the duplicate path. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
74ccba2 to
56c451a
Compare
Performance Comparison
|
df0c327
into
codex/tracedecay-total-redesign-plan-reopened
Summary
STOP-REFETCH rebase onto tip
d2c9294089(after #1415 Kiro advisory land). Single commit unchanged in intent.Migrate
skipped_terminal_failure_classfrom an inline"session_evidence_timed_out"string match toAutomationSkipReasonV1::from_ledger_reason, then keep only the typedSessionEvidenceTimedOut→Timeoutmapping. Deletes the internal dual-path literal beside the canonical ledger parser.Principle
migrate-callers-then-delete-legacy-apis — no shim left.
Test plan
commitlintfrom tip base