openclaw - 💡(How to fix) Fix Wait for compaction before failing active requester steering [2 pull requests]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Fix Action

Fixed

RAW_BUFFERClick to expand / collapse

Follow-up from #85489.

Active requester steering currently treats compacting from queueEmbeddedPiMessageWithOutcomeAsync as a wake failure. That means completion delivery can fall through to requester-agent/direct fallback even though the active run may become steerable again as soon as compaction finishes.

Desired behavior: compacting should not be a terminal steering failure. Wait until compaction completes, then retry steering the same wake, bounded by the existing delivery timeout/cancellation path. Other wake failures such as no_active_run, not_streaming, delivery-mode mismatch, and runtime rejection can keep their current fallback behavior.

Likely surface:

  • src/agents/pi-embedded-runner/runs.ts
  • src/agents/subagent-announce-delivery.ts
  • src/agents/subagent-announce-delivery.test.ts

Proof expectation: regression test where an active requester is compacting, compaction ends within the delivery timeout, and generated-media completion stays on the active steering path instead of direct fallback.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING