shouldSkipHeartbeatPendingFinalDelivery was using the default 300-char
threshold regardless of per-agent heartbeat config. Replace with inline
logic that resolves ackMaxChars from cfg.agents[agentId].heartbeat ->
cfg.agents.defaults.heartbeat -> DEFAULT_HEARTBEAT_ACK_MAX_CHARS.
Also fix: store the stripped text (remainder after HEARTBEAT_OK) rather
than the raw payload text. Previously pendingFinalDeliveryText would
have contained the HEARTBEAT_OK prefix, causing heartbeat-runner to
re-deliver it verbatim on retry.
Resolves clawsweeper P2 review finding on #79270.
Summary:
- The branch adds an opt-in Feishu top-level group-send fallback for withdrawn or missing normal quoted thread replies, plus regression coverage, a changelog entry, and CI/lint typing and baseline refreshes.
- Reproducibility: yes. at source level. Current main hard-errors withdrawn/not-found Feishu reply targets when `replyInThread` is true, and the existing regression test asserts that no top-level create fallback occurs.
Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(feishu): fall back from missing thread replies
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): address review for automerge-openclaw-openclaw-8030…
- PR branch already contained follow-up commit before automerge: fix(clawsweeper): reconcile automerge-openclaw-openclaw-80306 with ma…
- PR branch already contained follow-up commit before automerge: fix(ci): satisfy stricter lint and test types
- PR branch already contained follow-up commit before automerge: fix(ci): align Node 24 test typing
Validation:
- ClawSweeper review passed for head 93146f9d13.
- Required merge gates passed before the squash merge.
Prepared head SHA: 93146f9d13
Review: https://github.com/openclaw/openclaw/pull/80306#issuecomment-4415604729
Co-authored-by: Peter Steinberger <steipete@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Route group/channel task, subagent, and media completions through one requester-session delivery policy, including plugin legacy session keys.
Also keeps current Zalo lifecycle test typing green on CI after the latest main changes.
Co-authored-by: Merlin <merlin@funcracker.net>
Wait for Pi embedded abort cleanup to settle before releasing the session write lock.
This prevents a follow-up turn from racing prompt teardown from the previous run.
Co-authored-by: samzong <samzong.lu@gmail.com>
