fix(slack): emit message_sent hook on outbound delivery (mirror Telegram) by rishitamrakar · Pull Request #89943 · openclaw/openclaw

rishitamrakar · 2026-06-03T17:12:32Z

Summary

Emit terminal Slack message_sent plugin hooks and internal message:sent hooks across actual agent reply delivery paths.

The final implementation covers normal replies, media, blocks, native streaming, stream fallback, finalized previews, native slash responses, failures, and TTS supplemental media. It preserves the finalized thread session key and logical Slack target, reports the delivered Slack message ID, and emits one terminal outcome per logical payload.

Fixes #89942.

Implementation

Add a shared Slack sent-hook emitter for canonical event/context construction.
Emit after successful delivery and on terminal failure for normal, media, block, preview, streaming, fallback, slash, and TTS paths.
Aggregate multi-send and streamed outcomes so partial transport activity does not create duplicate or contradictory terminal hooks.
Preserve physical DM delivery while reporting the logical user target to lifecycle hooks.
Use finalized thread session keys for cross-hook correlation.
Clear fallback-owned ChatStreamer buffer text, then finalize any already-visible native stream without resending the fallback payload.
Add regression coverage for delivery cardinality, correlation, fallback, previews, slash responses, TTS, and Slack SDK return shapes.

Verification

node scripts/run-vitest.mjs extensions/slack/src/monitor/replies.test.ts extensions/slack/src/monitor/slash.test.ts extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts extensions/slack/src/streaming.test.ts extensions/slack/src/outbound-delivery.test.ts
- 5 files, 172 tests passed.
pnpm tsgo:extensions
- Passed.
pnpm tsgo:extensions:test
- Passed.
pnpm test:extensions:package-boundary:compile
- 114/114 plugins compiled.
.agents/skills/autoreview/scripts/autoreview --mode branch --base origin/main --engine codex --thinking high --stream-engine-output
- Clean; no actionable findings.
Direct dependency inspection:
- Verified installed @slack/web-api 7.16.0 ChatStreamer source and types for retained failed-flush buffers, buffer-free finalization retries, and stop() response message IDs.

Real Behavior Proof

Behavior addressed: Slack agent replies did not emit the documented terminal message_sent lifecycle hook. This change emits one terminal outcome per logical reply across normal, streaming, preview, fallback, slash, failure, and TTS delivery paths.
Real environment tested: Source-built OpenClaw gateway on macOS and Node, connected through Socket Mode to a real Slack workspace with the stock Slack plugin and a local lifecycle probe plugin.
Exact steps or command run after this patch: Build the branch from source, run the built gateway against an isolated OpenClaw state directory, register the probe for message_sent, then trigger real Slack mentions with native streaming, native-progress tool delivery, and streaming disabled.
Evidence after fix: Copied live output (redacted runtime log) from the source-built gateway:
```
[mst-probe] agent_end fired sessionKey=<redacted> (turn counter reset)
[mst-probe] message_sent FIRED #1-this-turn channelId=slack to=channel:<redacted> success=true messageId=<slack-ts-redacted> event.sessionKey=<redacted> ctx.sessionKey=<redacted>
```
Equivalent captures were recorded for native streaming, native-progress delivery, and streaming disabled. Each showed #1-this-turn; the native-progress path therefore did not double-emit.
Observed result after fix: Native streaming, native-progress delivery, and non-streaming delivery each emitted exactly one successful terminal hook after the reply appeared in Slack.
What was not tested: Live failure injection, slash responses, TTS supplemental delivery, and the rare failed native-stream finalization fallback were not exercised against the workspace. These paths are covered by focused regression tests, including a test against the pinned @slack/web-api 7.16.0 ChatStreamer implementation.

Credits

Original report, implementation, and live Slack proof by @rishitamrakar. Maintainer fixups expanded coverage to all Slack agent reply paths and reconciled terminal delivery outcomes.

clawsweeper · 2026-06-03T17:14:16Z

Codex review: needs changes before merge. Reviewed June 13, 2026, 8:45 AM ET / 12:45 UTC.

Summary
The PR adds Slack message_sent and internal message:sent emission across normal reply, preview, streaming, fallback, slash, failure, and TTS delivery paths, plus regression coverage and stream message-id propagation.

PR surface: Source +491, Tests +1053. Total +1544 across 11 files.

Reproducibility: yes. source-level reproduction is high confidence: docs promise message_sent, current main's Slack reply paths send without that hook, and the PR body/comments include real Slack workspace proof for the main fixed paths.

Review metrics: 1 noteworthy metric.

Plugin hook surface: 1 Slack observation hook newly emitted. Existing plugins can begin receiving lifecycle events for Slack deliveries, so maintainers should review the compatibility and event-shape impact before merge.

Merge readiness
Overall: 🦐 gold shrimp
Proof: 🦞 diamond lobster
Patch quality: 🦐 gold shrimp
Result: needs maintainer review before merge.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

[P2] Fix the consistent-return failure in extensions/slack/src/monitor/message-handler/dispatch.ts.
Rerun the focused Slack hook tests and extension lint/typecheck after the return-shape change.

Risk before merge

[P1] Existing Slack plugins that already listen for message_sent will start observing Slack reply events, which is intended but compatibility-sensitive for downstream analytics or automation.
[P1] The patch changes several Slack delivery paths where duplicate, missing, or mis-correlated sent hooks could affect plugin behavior even when user-visible Slack messages still deliver.
[P1] Current head has a deterministic lint failure in dispatch.ts that must be fixed before merge.

Maintainer options:

Fix lint and accept the Slack hook surface (recommended)
Repair the deliverNormally return-shape lint failure, then merge once maintainers accept that Slack plugins will begin receiving documented message_sent events.
Ask for more live edge-path proof
Maintainers can request live Slack proof for slash, failure, or TTS supplement paths if those edge cases are considered merge-critical beyond the existing focused tests.
Pause for lifecycle API direction
If maintainers want sent-hook semantics centralized rather than adapter-owned, pause this branch and route the broader lifecycle design through the existing hook coverage issues.

Copy recommended automerge instruction

@clawsweeper automerge

Special instructions:
Fix the `consistent-return` lint failure in `extensions/slack/src/monitor/message-handler/dispatch.ts` by making `deliverNormally` return an explicit `string | undefined` on every path or by refactoring the thread-id result into a closed local result shape; preserve the current Slack `message_sent` hook behavior and regression tests.

Next step before merge

[P2] A narrow mechanical repair can fix the lint-only blocker without changing the reviewed Slack hook semantics.

Security
Cleared: The diff stays within Slack delivery logic and tests, with no dependency, workflow, package, secret-handling, or code-execution surface added.

Review findings

[P2] Make deliverNormally returns consistent — extensions/slack/src/monitor/message-handler/dispatch.ts:941-974

Review details

Best possible solution:

Repair the lint-only return-shape defect, then land the Slack-owned hook emission after maintainer acceptance of the compatibility-sensitive lifecycle event expansion.

Do we have a high-confidence way to reproduce the issue?

Yes, source-level reproduction is high confidence: docs promise message_sent, current main's Slack reply paths send without that hook, and the PR body/comments include real Slack workspace proof for the main fixed paths.

Is this the best way to solve the issue?

Yes, after the lint fix: emitting from Slack-owned delivery boundaries mirrors Telegram and avoids a broad core refactor, while broader lifecycle hook coverage remains tracked separately.

Full review comments:

[P2] Make deliverNormally returns consistent — extensions/slack/src/monitor/message-handler/dispatch.ts:941-974
deliverNormally now returns a thread id on the successful delivery paths, but it still has no-value returns on other branches, so oxlint flags typescript(consistent-return) and the Slack bundled lint check fails. Return an explicit undefined from every non-delivery branch or refactor this into a closed result shape so the PR can pass lint without changing hook semantics.
Confidence: 0.91

Overall correctness: patch is incorrect
Overall confidence: 0.88

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 4c23d1d5978e.

Label changes

Label changes:

add proof: sufficient: Contributor real behavior proof is sufficient. The PR body and follow-up comments describe a source-built gateway tested against a real Slack workspace with lifecycle probe logs showing exactly one successful message_sent for native streaming, native-progress, and non-streaming deliveries.
add rating: 🦐 gold shrimp: Overall readiness is 🦐 gold shrimp; proof is 🦞 diamond lobster and patch quality is 🦐 gold shrimp.
add status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Sufficient (logs): The PR body and follow-up comments describe a source-built gateway tested against a real Slack workspace with lifecycle probe logs showing exactly one successful message_sent for native streaming, native-progress, and non-streaming deliveries.
remove rating: 🐚 platinum hermit: Current PR rating is rating: 🦐 gold shrimp, so this older rating label is no longer current.
remove status: 👀 ready for maintainer look: Current PR status label is status: ⏳ waiting on author.

Label justifications:

P2: The PR fixes a documented Slack plugin hook gap with limited blast radius but needs a normal merge-blocking lint repair.
merge-risk: 🚨 compatibility: Merging changes the observable hook stream for existing Slack plugin consumers by adding message_sent events.
merge-risk: 🚨 message-delivery: The patch touches Slack reply, streaming, fallback, slash, and failure delivery paths where hook duplication or suppression would affect delivery observers.
rating: 🦐 gold shrimp: Overall readiness is 🦐 gold shrimp; proof is 🦞 diamond lobster and patch quality is 🦐 gold shrimp.
status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Sufficient (logs): The PR body and follow-up comments describe a source-built gateway tested against a real Slack workspace with lifecycle probe logs showing exactly one successful message_sent for native streaming, native-progress, and non-streaming deliveries.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body and follow-up comments describe a source-built gateway tested against a real Slack workspace with lifecycle probe logs showing exactly one successful message_sent for native streaming, native-progress, and non-streaming deliveries.

Evidence reviewed

PR surface:

Source +491, Tests +1053. Total +1544 across 11 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	7	565	74	+491
Tests	4	1084	31	+1053
Docs	0	0	0	0
Config	0	0	0	0
Generated	0	0	0	0
Other	0	0	0	0
Total	11	1649	105	+1544

Acceptance criteria:

[P2] node scripts/run-vitest.mjs extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts extensions/slack/src/monitor/replies.test.ts extensions/slack/src/monitor/slash.test.ts extensions/slack/src/streaming.test.ts extensions/slack/src/outbound-delivery.test.ts.
[P1] pnpm tsgo:extensions.
[P1] extension lint or the failing check-lint lane equivalent.

What I checked:

Repository policy applied: The root and scoped extension policies were read and applied, including plugin ownership boundaries, full-path review expectations, dependency-contract proof, compatibility review, and read-only review constraints. (AGENTS.md:1, 4c23d1d5978e)
Documented hook contract: The docs describe message_sent as the outbound delivery success/failure hook and state hook handler failures are observation-only rather than delivery-affecting. Public docs: docs/plugins/hooks.md. (docs/plugins/hooks.md:127, 4c23d1d5978e)
Current Slack gap: Current main sends Slack monitor replies via sendMessageSlack without emitting the documented sent hook on those delivery paths. (extensions/slack/src/monitor/replies.ts:73, 4c23d1d5978e)
Sibling Telegram implementation: Telegram already builds a canonical sent context and emits both plugin message_sent and internal message:sent hooks from its adapter-owned delivery boundary. (extensions/telegram/src/bot/delivery.replies.ts:619, 4c23d1d5978e)
PR adds Slack hook helper: The PR adds a Slack helper that builds canonical sent context with session, message id, and group fields, then gates plugin hook emission through the global hook runner. (extensions/slack/src/message-sent-hook.ts:38, 7619a2dee566)
PR wires normal and slash delivery: The PR threads hook target, account, session key, and group context through normal reply and slash reply delivery, including success/failure emission helpers. (extensions/slack/src/monitor/replies.ts:45, 7619a2dee566)

Likely related people:

steipete: Recent history and this PR's fixup commits touch Slack replies, dispatch, streaming, slash hook coverage, and sent-hook correlation. (role: recent area contributor and PR fixup author; confidence: high; commits: 7c4601ec73fc, fa2b2ffab46f, fb9a21ae8fca; files: extensions/slack/src/monitor/replies.ts, extensions/slack/src/monitor/message-handler/dispatch.ts, extensions/slack/src/streaming.ts)
mvanhorn: Earlier Slack streaming fallback commits are central to the native-stream and fallback delivery paths touched by this PR. (role: stream fallback contributor; confidence: medium; commits: 676ed34cbd0b, e55b93263212; files: extensions/slack/src/monitor/message-handler/dispatch.ts, extensions/slack/src/streaming.ts)
martingarramon: Recent history includes routing stream fallback through the chunked sender, which is one of the delivery paths now emitting sent hooks. (role: adjacent Slack delivery contributor; confidence: medium; commits: 150053bc8699; files: extensions/slack/src/monitor/replies.ts, extensions/slack/src/monitor/message-handler/dispatch.ts)
zeroaltitude: Prior outbound hook work added session-key propagation to sent hooks, which is the shared invariant this PR mirrors for Slack. (role: adjacent hook owner; confidence: medium; commits: 05db9117750e; files: src/infra/outbound/deliver.ts, src/hooks/message-hook-mappers.ts)

What the crustacean ranks mean

🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

Co-authored-by: Rishi Tamrakar <[email protected]>

steipete · 2026-06-13T13:10:31Z

Land-ready verification for head 974768b75c4758ba768ecf8f98bee33e57aeea72.

Implemented terminal Slack message_sent / internal message:sent delivery outcomes across normal replies, media, blocks, native streaming, stream fallback, finalized previews, slash responses, failures, and TTS supplemental media. Correlation uses the finalized thread session key and logical target; ambiguous media/file and combined-fallback IDs are omitted rather than reporting incorrect Slack identifiers.

Local proof:

node scripts/run-vitest.mjs extensions/slack/src/monitor/replies.test.ts extensions/slack/src/monitor/slash.test.ts extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts extensions/slack/src/streaming.test.ts extensions/slack/src/outbound-delivery.test.ts
- 5 files, 172 tests passed.
pnpm tsgo:extensions
- Passed.
pnpm tsgo:extensions:test
- Passed.
pnpm test:extensions:package-boundary:compile
- 114/114 plugins compiled.
node scripts/run-vitest.mjs src/plugin-sdk/provider-catalog-shared.test.ts
- 1 file, 14 tests passed; this confirms the first CI failure in that unrelated shard was transient.
Final branch autoreview:
- .agents/skills/autoreview/scripts/autoreview --mode branch --base origin/main --engine codex --thinking high --stream-engine-output
- Clean, no actionable findings.

Dependency proof:

Inspected the pinned @slack/web-api 7.16.0 ChatStreamer implementation and types.
Added a test against the real installed ChatStreamer proving failed-flush buffer text is cleared before a buffer-free finalization retry.

Real behavior proof:

Contributor’s source-built gateway test against a real Slack workspace produced exactly one successful hook for native streaming, native-progress, and non-streaming replies, with Slack timestamp and session-key correlation where a single message timestamp exists.
Live failure injection, slash, TTS, and rare finalization fallback were not exercised against the workspace; focused regression tests cover those paths.

CI proof:

CI run 27467453130 passed after rerunning the transient provider-catalog shard.
Real behavior proof run 27467629916 passed.
No known proof gaps beyond the live scenarios listed above.

Emit terminal Slack message_sent and message:sent hooks across normal, streaming, preview, fallback, slash, failure, and TTS reply paths with canonical session/target correlation and one outcome per logical payload. Fixes openclaw#89942 Co-authored-by: Rishi Tamrakar <[email protected]>

…26.6.8) (#1144) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/openclaw/openclaw](https://openclaw.ai) ([source](https://github.com/openclaw/openclaw)) | patch | `2026.6.6` → `2026.6.8` | --- ### Release Notes <details> <summary>openclaw/openclaw (ghcr.io/openclaw/openclaw)</summary> ### [`v2026.6.8`](https://github.com/openclaw/openclaw/blob/HEAD/CHANGELOG.md#202668) [Compare Source](openclaw/openclaw@v2026.6.6...v2026.6.8) ##### Highlights - Telegram and WhatsApp channel delivery are richer and less brittle: Telegram can send structured rich text with tables, lists, expandable blockquotes, prompt-preserving CLI backend delivery, retired native draft migration, and safer rich-media boundaries, while WhatsApp now honors configured ACP bindings. ([#92679](openclaw/openclaw#92679), [#84082](openclaw/openclaw#84082), [#89421](openclaw/openclaw#89421), [#92513](openclaw/openclaw#92513)) Thanks [@obviyus](https://github.com/obviyus), [@jzakirov](https://github.com/jzakirov), [@spacegeologist](https://github.com/spacegeologist), and [@TurboTheTurtle](https://github.com/TurboTheTurtle). - Agent and Gateway recovery is sharper across account-scoped DM sends, generated media completions, restart shutdown aborts, yielded subagent pauses, yielded cron media, heartbeat dedupe, session identity prompts, and unknown OpenAI agent selector rejection. ([#92788](openclaw/openclaw#92788), [#91246](openclaw/openclaw#91246), [#91357](openclaw/openclaw#91357), [#92631](openclaw/openclaw#92631), [#92146](openclaw/openclaw#92146), [#91287](openclaw/openclaw#91287), [#92468](openclaw/openclaw#92468), [#92510](openclaw/openclaw#92510)) Thanks [@yetval](https://github.com/yetval), [@TurboTheTurtle](https://github.com/TurboTheTurtle), [@ooiuuii](https://github.com/ooiuuii), [@openperf](https://github.com/openperf), [@IWhatsskill](https://github.com/IWhatsskill), [@ZengWen-DT](https://github.com/ZengWen-DT), and [@zhangguiping-xydt](https://github.com/zhangguiping-xydt). - Provider/model handling expands and tightens with GLM-5.2, Claude Haiku 4.5 catalog rows, OpenRouter and Google Vertex provider-prefix normalization, managed SecretRef auth, bounded model browse discovery, storeless OpenAI Responses replay gating, and Claude 4.5 Copilot tool-streaming safety. ([#92796](openclaw/openclaw#92796), [#90116](openclaw/openclaw#90116), [#92627](openclaw/openclaw#92627), [#91218](openclaw/openclaw#91218), [#90686](openclaw/openclaw#90686), [#92247](openclaw/openclaw#92247), [#90706](openclaw/openclaw#90706), [#75393](openclaw/openclaw#75393)) Thanks [@arkyu2077](https://github.com/arkyu2077), [@liuhao1024](https://github.com/liuhao1024), [@bymle](https://github.com/bymle), [@rohitjavvadi](https://github.com/rohitjavvadi), [@samson910022](https://github.com/samson910022), [@snowzlm](https://github.com/snowzlm), and [@Kailigithub](https://github.com/Kailigithub). - `/usage` and reply payload hooks now have a native full footer renderer, default template, fixed-decimal formatting, credential-aware limits, better partial-count handling, and warnings for broken templates instead of silent bad output. ([#92657](openclaw/openclaw#92657), [#89835](openclaw/openclaw#89835), [#89629](openclaw/openclaw#89629)) Thanks [@Marvinthebored](https://github.com/Marvinthebored). - UI and mobile flows are steadier: workspace files can collapse and start collapsed, WebChat backscroll survives streaming, the sidebar session picker remains interactive above the desktop workbench, reset soft args survive UI dispatch, stale dashboard session parent lineage is preserved, and iOS reconnects stale foreground gateways. ([#92779](openclaw/openclaw#92779), [#92622](openclaw/openclaw#92622), [#92705](openclaw/openclaw#92705), [#91353](openclaw/openclaw#91353), [#90658](openclaw/openclaw#90658), [#92552](openclaw/openclaw#92552)) Thanks [@shakkernerd](https://github.com/shakkernerd), [@TurboTheTurtle](https://github.com/TurboTheTurtle), [@NianJiuZst](https://github.com/NianJiuZst), [@zhouhe-xydt](https://github.com/zhouhe-xydt), [@luoyanglang](https://github.com/luoyanglang), and [@Solvely-Colin](https://github.com/Solvely-Colin). - Memory, state, and diagnostics recover cleaner: oversized OpenAI embedding batches split before 431s, QMD memory search stays available in transient mode, SQLite avoids WAL on NFS state volumes, stuck-session recovery scheduling no longer resets warning backoff, and Infinity chunk limits stay genuinely unbounded. ([#92650](openclaw/openclaw#92650), [#92618](openclaw/openclaw#92618), [#92639](openclaw/openclaw#92639), [#91247](openclaw/openclaw#91247), [#92752](openclaw/openclaw#92752), [#92735](openclaw/openclaw#92735)) Thanks [@mushuiyu886](https://github.com/mushuiyu886), [@TurboTheTurtle](https://github.com/TurboTheTurtle), [@849261680](https://github.com/849261680), [@gnanam1990](https://github.com/gnanam1990), and [@yhterrance](https://github.com/yhterrance). ##### Changes - Providers/models: add GLM-5.2 support and Claude Haiku 4.5 catalog entries while keeping provider-qualified model IDs normalized across OpenRouter and Google Vertex paths. ([#92796](openclaw/openclaw#92796), [#90116](openclaw/openclaw#90116), [#92627](openclaw/openclaw#92627), [#91218](openclaw/openclaw#91218)) Thanks [@arkyu2077](https://github.com/arkyu2077), [@liuhao1024](https://github.com/liuhao1024), and [@bymle](https://github.com/bymle). - Channel plugins: ship Telegram rich-message delivery and WhatsApp ACP binding support, including rich prompt handoff to CLI backends and transport fixtures for richer drafts. ([#92679](openclaw/openclaw#92679), [#92513](openclaw/openclaw#92513)) Thanks [@obviyus](https://github.com/obviyus) and [@TurboTheTurtle](https://github.com/TurboTheTurtle). - Agent commands: support `/btw` in CLI-backed sessions and keep CLI usage-error exits classified as usage failures instead of successful runs. ([#92669](openclaw/openclaw#92669), [#92162](openclaw/openclaw#92162)) Thanks [@joshavant](https://github.com/joshavant) and [@Pandah97](https://github.com/Pandah97). - Usage hooks: add built-in full footer rendering, default footer templates, per-turn usage state, credential-aware limits, and fixed-decimal formatting for usage-bar templates. ([#92657](openclaw/openclaw#92657), [#89835](openclaw/openclaw#89835), [#89629](openclaw/openclaw#89629)) Thanks [@Marvinthebored](https://github.com/Marvinthebored). - Docs and operator guidance: document node config examples, clarify before-install hook scope, correct agent default concurrency comments, refresh ZAI provider docs, and update channel/group docs for current Telegram and WhatsApp behavior. ([#92677](openclaw/openclaw#92677), [#92766](openclaw/openclaw#92766), [#92695](openclaw/openclaw#92695)) Thanks [@liuhao1024](https://github.com/liuhao1024), [@sallyom](https://github.com/sallyom), and [@ArielSmoliar](https://github.com/ArielSmoliar). ##### Fixes - Onboarding/skills: show the Homebrew install recommendation only on macOS and Linux, so FreeBSD and other unsupported platforms no longer get a misleading brew prompt. Fixes [#68893](openclaw/openclaw#68893); carries forward [#68894](openclaw/openclaw#68894), [#68910](openclaw/openclaw#68910), [#68941](openclaw/openclaw#68941), [#68943](openclaw/openclaw#68943), [#69002](openclaw/openclaw#69002), and [#69545](openclaw/openclaw#69545). Thanks [@yurivict](https://github.com/yurivict), [@Sanjays2402](https://github.com/Sanjays2402), [@Eruditi](https://github.com/Eruditi), [@JustInCache](https://github.com/JustInCache), [@nnish16](https://github.com/nnish16), and [@Mlightsnow](https://github.com/Mlightsnow). - Channels and delivery: preserve account-scoped DM channel send policy, rich Telegram final replies, rich Telegram tables and lists, Telegram thread-create CLI remapping, Slack outbound `message_sent` hooks, contributed message-tool schema optionality, same-channel generated media completions, and channel chunking around surrogate pairs and Infinity limits. ([#92788](openclaw/openclaw#92788), [#92679](openclaw/openclaw#92679), [#89421](openclaw/openclaw#89421), [#89943](openclaw/openclaw#89943), [#91137](openclaw/openclaw#91137), [#91246](openclaw/openclaw#91246), [#92735](openclaw/openclaw#92735)) Thanks [@yetval](https://github.com/yetval), [@obviyus](https://github.com/obviyus), [@spacegeologist](https://github.com/spacegeologist), [@rishitamrakar](https://github.com/rishitamrakar), [@lundog](https://github.com/lundog), [@TurboTheTurtle](https://github.com/TurboTheTurtle), and [@yhterrance](https://github.com/yhterrance). - Auto-reply/groups: keep ordinary group text replies on automatic final-reply delivery while allowing `message(action=send)` for files, images, and other attachments to the same group or topic. Carries forward [#43276](openclaw/openclaw#43276); refs [#48004](openclaw/openclaw#48004). Thanks [@NayukiChiba](https://github.com/NayukiChiba) and [@ShakaRover](https://github.com/ShakaRover). - Auto-reply/skills: preserve multiline payloads for `/skill` and direct skill slash commands while keeping command-head normalization for aliases, colon syntax, and bot mentions. Fixes [#79155](openclaw/openclaw#79155); carries forward [#81305](openclaw/openclaw#81305). Thanks [@web3blind](https://github.com/web3blind). - iMessage: normalize leading NUL sent-message echo prefixes while preserving interior NUL bytes and the leading attributedBody marker handling from [#73942](openclaw/openclaw#73942). Carries forward [#63581](openclaw/openclaw#63581). Thanks [@drvoss](https://github.com/drvoss). - Discord: give generated auto-thread titles a 60-second timeout and 4,096-token reasoning-model output budget, clamped to the selected model output cap. ([#64734](openclaw/openclaw#64734)) Thanks [@hanamizuki](https://github.com/hanamizuki). - Agent, cron, and Gateway runtime: mark active main sessions before restart shutdown aborts, pause yielded subagent runs whose terminal also signals abort, preserve yielded media completions, de-duplicate main-session heartbeat events, expose session identity in runtime prompts, reject unknown OpenAI agent selectors, keep generated media completions and slash-command block replies in WebChat, preserve fresh post-compaction usage while clearing stale usage snapshots, and require admin privileges for HTTP session/model override surfaces. ([#91357](openclaw/openclaw#91357), [#92631](openclaw/openclaw#92631), [#92146](openclaw/openclaw#92146), [#91287](openclaw/openclaw#91287), [#92468](openclaw/openclaw#92468), [#92510](openclaw/openclaw#92510), [#91246](openclaw/openclaw#91246), [#50795](openclaw/openclaw#50795), [#50845](openclaw/openclaw#50845), [#82874](openclaw/openclaw#82874), [#92651](openclaw/openclaw#92651), [#92646](openclaw/openclaw#92646)) Thanks [@ooiuuii](https://github.com/ooiuuii), [@openperf](https://github.com/openperf), [@IWhatsskill](https://github.com/IWhatsskill), [@ZengWen-DT](https://github.com/ZengWen-DT), [@zhangguiping-xydt](https://github.com/zhangguiping-xydt), [@Hollychou924](https://github.com/Hollychou924), [@leno23](https://github.com/leno23), and [@TurboTheTurtle](https://github.com/TurboTheTurtle). - Agents/exec: default empty-success background completion notices on only for real chat channels, preserving explicit opt-outs and keeping generic providers silent while carrying forward the narrow UX intent from [#39726](openclaw/openclaw#39726) and [#46926](openclaw/openclaw#46926). Thanks [@Sapientropic](https://github.com/Sapientropic) and [@wenkang-xie](https://github.com/wenkang-xie). - Providers and model replay: preserve storeless OpenAI Responses replay compatibility, avoid eager tool streaming for Claude 4.5 in Copilot, honor profile auth for SecretRef model entries, bound model browsing, strip provider prefixes where runtimes need bare IDs, and surface nested embedding fetch failures. ([#90706](openclaw/openclaw#90706), [#75393](openclaw/openclaw#75393), [#90686](openclaw/openclaw#90686), [#92247](openclaw/openclaw#92247), [#92627](openclaw/openclaw#92627), [#91218](openclaw/openclaw#91218), [#92628](openclaw/openclaw#92628)) Thanks [@snowzlm](https://github.com/snowzlm), [@Kailigithub](https://github.com/Kailigithub), [@rohitjavvadi](https://github.com/rohitjavvadi), [@samson910022](https://github.com/samson910022), [@liuhao1024](https://github.com/liuhao1024), [@bymle](https://github.com/bymle), and [@mushuiyu886](https://github.com/mushuiyu886). - Memory, state, diagnostics, and config: split header-too-large embedding batches, keep QMD memory search enabled in transient mode, avoid SQLite WAL on NFS volumes, preserve recovery scheduling outside stuck-session warning backoff, and keep shell environment fallbacks contained in config write tests. ([#92650](openclaw/openclaw#92650), [#92618](openclaw/openclaw#92618), [#92639](openclaw/openclaw#92639), [#91247](openclaw/openclaw#91247), [#92752](openclaw/openclaw#92752)) Thanks [@mushuiyu886](https://github.com/mushuiyu886), [@TurboTheTurtle](https://github.com/TurboTheTurtle), [@849261680](https://github.com/849261680), and [@gnanam1990](https://github.com/gnanam1990). - Workspace setup state: store setup completion outside the workspace dot directory using an OpenClaw-named root file, migrate valid legacy state forward, and avoid clobbering generic root `workspace-state.json` files for TigerFS-style dot-path compatibility. This Clownfish replacement carries forward the focused [#53326](openclaw/openclaw#53326) fix idea because the original branch was closed and uneditable. ([#53326](openclaw/openclaw#53326), [#44783](openclaw/openclaw#44783), [#39446](openclaw/openclaw#39446)) Thanks [@1qh](https://github.com/1qh). - UI/mobile/TUI: preserve dashboard session parent lineage, WebChat backscroll, reset soft command args, sidebar session picker interactivity, collapsed workspace files, resolved `/model` confirmation refs, and stale foreground iOS Gateway reconnects. ([#90658](openclaw/openclaw#90658), [#92622](openclaw/openclaw#92622), [#91353](openclaw/openclaw#91353), [#92705](openclaw/openclaw#92705), [#92779](openclaw/openclaw#92779), [#92773](openclaw/openclaw#92773), [#92552](openclaw/openclaw#92552)) Thanks [@luoyanglang](https://github.com/luoyanglang), [@TurboTheTurtle](https://github.com/TurboTheTurtle), [@zhouhe-xydt](https://github.com/zhouhe-xydt), [@NianJiuZst](https://github.com/NianJiuZst), [@shakkernerd](https://github.com/shakkernerd), [@NarahariRaghava](https://github.com/NarahariRaghava), and [@Solvely-Colin](https://github.com/Solvely-Colin). - TUI: reload the active session after external `/new` or `/reset` session-change events so stale transcript and stream state clear promptly. Fixes [#38966](openclaw/openclaw#38966); carries forward [#40472](openclaw/openclaw#40472). Thanks [@yizhanzjz](https://github.com/yizhanzjz) and [@wsyjh8](https://github.com/wsyjh8). - Control UI: preserve Gateway Access tokens during same-normalized WebSocket URL edits and reload gateway-scoped tokens when switching endpoints. Fixes [#41545](openclaw/openclaw#41545); repairs [#42001](openclaw/openclaw#42001) with additional source PRs [#41546](openclaw/openclaw#41546), [#41552](openclaw/openclaw#41552), and [#41718](openclaw/openclaw#41718). Thanks [@wsyjh8](https://github.com/wsyjh8), [@llagy0020](https://github.com/llagy0020), [@llagy007](https://github.com/llagy007), [@pingfanfan](https://github.com/pingfanfan), and [@zheliu2](https://github.com/zheliu2). - Gateway CLI: tolerate a single transient clean WebSocket close before `hello-ok` so one-shot RPC calls reconnect instead of failing noisily, while repeated clean pre-hello closes still surface. Carries forward source PRs [#54475](openclaw/openclaw#54475) and [#54774](openclaw/openclaw#54774); [#85253](openclaw/openclaw#85253) covered adjacent connect assembly diagnostics. Thanks [@ruanrrn](https://github.com/ruanrrn). - Gateway/Linux: keep root-owned systemd user service lifecycle commands on root's user manager when a stale `SUDO_USER` remains in a root shell with root's user bus environment. Fixes [#81410](openclaw/openclaw#81410). Thanks [@Ericksza](https://github.com/Ericksza) and [@ChuckClose-tech](https://github.com/ChuckClose-tech). - Release and test reliability: extend slow Gateway/full-suite watchdogs, split local full-suite shards when throttled, stabilize plugin auth marker fixtures, avoid brittle provider-ref error text, and keep QA Lab bootstrap selection assertions aligned with flow-only scenarios. ([#92652](openclaw/openclaw#92652)) - macOS Peekaboo bridge: update the embedded Peekaboo package to 3.5.2 and route bundled-skill CLI commands through the OpenClaw app bridge so they inherit its Screen Recording and Accessibility grants. - Agent routing: route subagent RPC callbacks addressed to an agent-shaped `--to` target to the correct session key instead of falling back to the main session, so WeChat (and other channel) session-key callbacks reach the intended subagent session. ([#90231](openclaw/openclaw#90231)) Thanks [@zhangguiping-xydt](https://github.com/zhangguiping-xydt). - Cron: preserve model, fallback, thinking, timeout, light-context, unsafe-content, and tool allow-list overrides on implicit text payloads by promoting them to agent turns, while explicit system events still prune those fields. Fixes [#28905](openclaw/openclaw#28905); carries forward [#64060](openclaw/openclaw#64060) and [#73946](openclaw/openclaw#73946). Thanks [@liaoandi](https://github.com/liaoandi). - QQBot delivery: keep markdown table chunks self-contained across message boundaries by preserving table state across block deliveries, flushing unfinished table-row fragments as plain text, and detecting short pipe-terminated rows by column count so split rows are not sent as malformed markdown. ([#92428](openclaw/openclaw#92428)) Thanks [@sliverp](https://github.com/sliverp). </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.erwanleboucher.dev/eleboucher/homelab/pulls/1144

Emit terminal Slack message_sent and message:sent hooks across normal, streaming, preview, fallback, slash, failure, and TTS reply paths with canonical session/target correlation and one outcome per logical payload. Fixes openclaw#89942 Co-authored-by: Rishi Tamrakar <[email protected]>

openclaw-barnacle Bot added channel: slack Channel integration: slack size: M triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels Jun 3, 2026

clawsweeper Bot mentioned this pull request Jun 3, 2026

fix(slack): emit message_sent hook on outbound delivery (mirror Telegram) #89942

Closed

openclaw-barnacle Bot removed the triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. label Jun 3, 2026

rishitamrakar force-pushed the fix/slack-message-sent-hook branch from 9b33fa7 to 50e39ff Compare June 3, 2026 19:04

rishitamrakar force-pushed the fix/slack-message-sent-hook branch from 50e39ff to b00c635 Compare June 3, 2026 19:10

steipete force-pushed the fix/slack-message-sent-hook branch from abdaf29 to 7619a2d Compare June 13, 2026 12:29

clawsweeper Bot added the proof: sufficient ClawSweeper judged the real behavior proof convincing. label Jun 13, 2026

steipete and others added 2 commits June 13, 2026 13:43

fix(slack): preserve sent hook message ids

f76a56c

Co-authored-by: Rishi Tamrakar <[email protected]>

fix(slack): finalize streams after fallback

974768b

Co-authored-by: Rishi Tamrakar <[email protected]>

steipete merged commit dc493bc into openclaw:main Jun 13, 2026
334 of 349 checks passed

github-actions Bot mentioned this pull request Jun 13, 2026

📡 Upstream Digest — 2026-06-13 13:29 UTC curtismercier/openclaw-mods#1066

Open

bek91 mentioned this pull request Jun 13, 2026

Align message_sent hooks with documented Slack delivery correlation #78046

Closed

clawsweeper Bot mentioned this pull request Jun 14, 2026

feat(slack): run message_sending hook on agent replies + ship slack-addressee-guard plugin #73247

Closed

steipete mentioned this pull request Jun 14, 2026

fix(slack): require delivery receipts for replies #80749

Closed

25 tasks

clawsweeper Bot mentioned this pull request Jun 19, 2026

fix(slack): tag gateway failure echoes #77045

Closed

This was referenced Jun 29, 2026

feat(plugin-sdk): generic outbound delivery-report seam; WhatsApp native auto-reply reports provider-accepted sends #97728

Open

[Slack] Agent lacks visibility into own channel messages from DM sessions #7359

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

fix(slack): emit message_sent hook on outbound delivery (mirror Telegram)#89943

fix(slack): emit message_sent hook on outbound delivery (mirror Telegram)#89943
steipete merged 9 commits into
openclaw:mainfrom
rishitamrakar:fix/slack-message-sent-hook

rishitamrakar commented Jun 3, 2026 •

edited by steipete

Loading

Uh oh!

clawsweeper Bot commented Jun 3, 2026 •

edited

Loading

Uh oh!

steipete commented Jun 13, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Uh oh!

Conversation

rishitamrakar commented Jun 3, 2026 • edited by steipete Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Implementation

Verification

Real Behavior Proof

Credits

Uh oh!

clawsweeper Bot commented Jun 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

steipete commented Jun 13, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

rishitamrakar commented Jun 3, 2026 •

edited by steipete

Loading

clawsweeper Bot commented Jun 3, 2026 •

edited

Loading