Skip to content

fix(cron): treat transient tool error payloads as recoverable#1515

Merged
alexey-pelykh merged 2 commits intomainfrom
cherry-pick/d509a81a1-cron-transient-tool-error
Mar 16, 2026
Merged

fix(cron): treat transient tool error payloads as recoverable#1515
alexey-pelykh merged 2 commits intomainfrom
cherry-pick/d509a81a1-cron-transient-tool-error

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick of openclaw/openclaw@d509a81a1.

Conflict resolution: Deleted test file src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts — it tests via runEmbeddedPiAgent (gutted in fork). Production fix in run.ts applied cleanly.

Part of #681.

Co-authored-by: Sid-Qin [email protected]

Sid-Qin and others added 2 commits March 16, 2026 09:01
…aw#29527) thanks @Sid-Qin

Verified:
- pnpm install --frozen-lockfile
- pnpm check
- pnpm test -- --run src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts

Co-authored-by: Sid-Qin <[email protected]>
Co-authored-by: Tak Hoffman <[email protected]>
(cherry picked from commit d509a81)
Upstream uses runResult.meta?.error (Pi-embedded style), fork has
top-level runResult.error on AgentDeliveryResult.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@alexey-pelykh alexey-pelykh merged commit 0875cf5 into main Mar 16, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/d509a81a1-cron-transient-tool-error branch March 16, 2026 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants