Skip to content

fix(slack): confirm final delivery before completing progress#3

Open
simonvanlaak wants to merge 48 commits into
betafrom
fix/slack-final-delivery-progress
Open

fix(slack): confirm final delivery before completing progress#3
simonvanlaak wants to merge 48 commits into
betafrom
fix/slack-final-delivery-progress

Conversation

@simonvanlaak

Copy link
Copy Markdown
Collaborator

Summary\n- keep Slack progress in sending/error state until final delivery is actually observed\n- recover failed native stream finalization by sending the real final payload instead of buffered thinking/progress text\n- preserve draft preview progress behavior and add regression coverage\n\n## Tests\n- pnpm test extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts extensions/slack/src/monitor/message-handler/dispatch.progress-plan.test.ts

Simon van Laak and others added 30 commits April 24, 2026 18:20
Normalize WhatsApp onboarding allowlist entries to digit-only WhatsApp IDs and reject invalid owner-phone inputs during prompt validation.

(cherry picked from commit 68a500c)
* fix(telegram): reuse preview for long text finals

* test(qa): cover long telegram finals

* fix(qa): satisfy extension lint

* fix(qa): keep telegram long final fixture to two chunks

* test(telegram): cover three chunk finals

* fix(telegram): force long final preview boundary

(cherry picked from commit e03fe1e)
Bind the default loopback gateway listener only to `127.0.0.1` on Windows so libuv dual-stack `::1` behavior cannot wedge localhost HTTP requests.

Also keeps non-Windows dual-loopback behavior covered, replaces the redundant Windows passthrough test with guard coverage, and adds the required changelog entry.

Fixes openclaw#69674.

Tests:
- pnpm exec oxfmt --check --threads=1 CHANGELOG.md src/gateway/net.ts src/gateway/net.test.ts
- pnpm test src/gateway/net.test.ts
- pnpm check:changed
- GitHub required checks: green

Thanks @SARAMALI15792.

Co-authored-by: saram ali <140950904+SARAMALI15792@users.noreply.github.com>
Co-authored-by: Brad Groux <3053586+BradGroux@users.noreply.github.com>
(cherry picked from commit 978bc53)
…isted] (openclaw#74161)

Summary:
- The PR updates agents skill prompt guidance to require exact `<location>` paths for single- and multi-skill selection, adds prompt assertions, and records the fix in the changelog.
- Reproducibility: yes. Static source reproduction is enough: current main lacks the exact-`<location>` guard  ... illsSection()`, while the PR diff adds it to both selection branches and asserts the resulting prompt text.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix: enforce exact skill paths for all skill matches

Validation:
- ClawSweeper review passed for head 743c984.
- Required merge gates passed before the squash merge.

Prepared head SHA: 743c984
Review: openclaw#74161 (comment)

Co-authored-by: tianguicheng <tianguicheng@xiaomi.com>
Co-authored-by: sallyom <somalley@redhat.com>
(cherry picked from commit c739088)
Accept drive-absolute Windows sandbox Docker bind sources in config and runtime validation while keeping blocked-path and allowed-root comparisons case-insensitive for Windows drive paths.

Also remove a stale WhatsApp setup import that blocked extension lint after the rebase.

Co-authored-by: 6607changchun <84566142+6607changchun@users.noreply.github.com>
Co-authored-by: Brad Groux <3053586+BradGroux@users.noreply.github.com>
(cherry picked from commit d02fbc6)
steipete and others added 17 commits May 5, 2026 08:37
OpenClaw 2026.5.4

# -----BEGIN SSH SIGNATURE-----
# U1NIU0lHAAAAAQAAADMAAAALc3NoLWVkMjU1MTkAAAAgj1exoJyvxMic+cdy5NV+9ddyyh
# QWs4TuZ6d9sQgcVb0AAAADZ2l0AAAAAAAAAAZzaGE1MTIAAABTAAAAC3NzaC1lZDI1NTE5
# AAAAQPf5yFEhmI4eXCFEPPcJKyQ8SvRdOLHzr9teYA8Fp57EbNT0OzAEQe+9yxRQK8RUES
# pQ8HdinAXaAbVf1iIFxAk=
# -----END SSH SIGNATURE-----
# gpg verification failed.
This reverts commit 4174b73, reversing
changes made to 5fdd961.
Replace the beta tree with upstream v2026.5.7 plus preserved 4ok Slack/runtime patches.\n\nThis keeps the PR mergeable into beta without force-pushing beta history.\n\nPreserves:\n- Slack DM native task-card streams and assistant-thread reuse\n- env token fallback for unresolved Slack env refs\n- progress/status behavior during native plans\n- same-bot allowBots handling\n- beta Docker image tags\n\nBrings in upstream diagnostics-otel run/model/tool/tool.loop telemetry.

Co-authored-by: Shadow <shadow@openclaw.ai>
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.

10 participants