fix(models): retry timeout errors by default

This commit is contained in:
William Valentin
2026-02-15 22:06:30 -08:00
parent 2cdfb66071
commit 7877a1bcc9
4 changed files with 15 additions and 5 deletions
@@ -17,6 +17,7 @@ Scope: Production-risk-first audit of bugs, code improvements, and feature oppor
- ✅ F-014 addressed: `ModelRouter.setOnTierChange` now preserves existing listeners instead of replacing them, removing destructive listener-setter behavior.
- ✅ F-002 addressed: `config.patch` now supports durable persistence via atomic write + backup when daemon has a concrete config path, and response includes `persisted`/`persistError` so UI can distinguish runtime-only vs disk-persisted updates.
- ◑ F-003 partially addressed: tool execution now has an `AbortSignal` contract and executor triggers abort on timeout; host `shell.exec` and sandbox docker exec now respond to cancellation. Additional high-risk tools still need explicit cancellation coverage for full closure.
- ✅ F-015 addressed: retry defaults no longer classify timeout-style failures as non-retryable, improving resilience for transient timeout conditions.
## Executive Summary
+12
View File
@@ -2565,6 +2565,18 @@
"docs/plans/analysis/2026-02-16-codebase-audit-report.md"
],
"test_status": "pnpm test:run src/tools/executor.test.ts src/tools/builtin/shell.test.ts + pnpm typecheck passing"
},
"audit-followup-retry-timeout-defaults": {
"status": "completed",
"date": "2026-02-16",
"updated": "2026-02-16",
"summary": "Adjusted retry defaults so timeout-style errors are treated as retryable by default, improving resilience for transient provider/network timeouts.",
"files_modified": [
"src/models/retry.ts",
"src/models/retry.test.ts",
"docs/plans/analysis/2026-02-16-codebase-audit-report.md"
],
"test_status": "pnpm test:run src/models/retry.test.ts + pnpm typecheck passing"
}
},
"overall_progress": {