Expose messageId in message CLI JSON output by 100menotu001 · Pull Request #84191 · openclaw/openclaw

100menotu001 · 2026-05-19T15:02:05Z

Summary

Adds a top-level messageId to openclaw message send --json output when the provider receipt contains one.

Motivation

The raw provider payload remains useful, but automation should not need provider-specific knowledge to find the id of the message it just sent.

Change Type

Bug fix
CLI output improvement

Scope

Extracts messageId from direct and nested send receipt shapes.
Adds top-level messageId only when present.
Preserves the existing raw payload object unchanged.
Adds a focused CLI JSON test for nested receipt output.

Linked Issue/PR

Closes #84186.

Real Behavior Proof

Behavior or issue addressed: openclaw message send --json must expose a top-level messageId when the provider receipt includes one while preserving the original provider payload.

Real environment tested: Redacted OpenClaw development checkout on this PR branch, using the real message CLI and Discord provider send path with IDs removed from public proof.

Exact steps or command run after this patch: Ran openclaw message send --channel discord --target channel:<redacted> --message <redacted> --json and transformed only the returned IDs to REDACTED_ID before publishing the output.

Evidence after fix: Redacted real CLI output:

{
  "action": "send",
  "channel": "discord",
  "topLevelMessageId": "REDACTED_ID",
  "payloadResultMessageId": "REDACTED_ID",
  "payloadPreserved": true
}

Observed result after fix: The JSON output included the new top-level message id, retained the nested provider message id, and preserved the existing provider payload object.

Not tested: No additional provider families were exercised; this proof covers the Discord CLI send path and the formatter path covered by the regression test.

Root Cause

buildMessageCliJson returned the raw provider payload but did not normalize common receipt identifiers into a stable CLI field.

Regression Test Plan

pnpm exec vitest run src/commands/message.test.ts

Result: 1 file passed, 6 tests passed.

User-visible / Behavior Changes

openclaw message send --json may now include messageId at the top level when available. Existing consumers can continue reading payload.

Diagram

N/A. This is a CLI JSON shape improvement.

Security Impact

No secrets, new permissions, or external calls. The id is already present in the existing payload.

Repro + Verification

Before this change, callers had to inspect nested provider payload fields. The new test verifies a nested payload.result.messageId becomes top-level messageId, and the redacted real CLI send confirms the field is present on actual Discord message delivery while preserving the original payload.

Evidence

Focused regression command passed.
Redacted real CLI send output above confirms top-level messageId and preserved payload.

Human Verification

Real provider send path was exercised with IDs redacted from public proof.

Compatibility / Migration

Backward compatible. This only adds a field when it can be derived.

Risks

Low. Existing output fields are preserved.

clawsweeper · 2026-05-19T15:03:19Z

Codex review: passed.

Workflow note: Future ClawSweeper reviews update this same comment in place.

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.

Summary
The PR promotes direct or nested send receipt IDs into openclaw message send --json, adds a focused command test, and adds a changelog entry.

Reproducibility: yes. at source level. Current main serializes only the raw payload while send receipts can carry payload.result.messageId; I did not execute the CLI in this read-only review.

PR rating
Overall: 🐚 platinum hermit
Proof: 🦞 diamond lobster
Patch quality: 🐚 platinum hermit
Summary: Focused additive patch with sufficient live proof and targeted regression coverage; remaining confidence depends on exact-head CI.

Rank-up moves:

Let exact-head CI or a maintainer-run focused command test pass before merge.

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.

Real behavior proof
Sufficient (live_output): The PR body includes redacted live Discord CLI output demonstrating the derived top-level id and preserved nested payload after the patch.

Risk before merge

This read-only review did not execute the focused Vitest command; exact-head CI should remain the merge gate.
The live proof covers the Discord CLI send path plus the mocked nested formatter path, not every provider receipt shape.

Maintainer options:

Decide the mitigation before merge
Land the additive top-level messageId field with the focused regression test after exact-head CI, while preserving the raw provider payload for existing consumers.
Pause or close
Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge
No repair lane is needed because there are no blocking findings; the existing automerge/maintainer CI path is the right next action.

Security
Cleared: The diff only changes CLI JSON shaping, a focused test, and changelog text; it does not touch secrets, permissions, dependencies, CI, install scripts, or supply-chain surfaces.

Review details

Best possible solution:

Land the additive top-level messageId field with the focused regression test after exact-head CI, while preserving the raw provider payload for existing consumers.

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

Yes, at source level. Current main serializes only the raw payload while send receipts can carry payload.result.messageId; I did not execute the CLI in this read-only review.

Is this the best way to solve the issue?

Yes. Promoting direct/nested messageId from existing receipts in the CLI formatter is narrow and backwards-compatible because the raw payload remains unchanged.

Label justifications:

P2: This is a normal-priority CLI automation improvement with limited blast radius and an additive output shape.
rating: 🐚 platinum hermit: Current PR rating is 🐚 platinum hermit because proof is 🦞 diamond lobster, patch quality is 🐚 platinum hermit, and Focused additive patch with sufficient live proof and targeted regression coverage; remaining confidence depends on exact-head CI.
status: 🚀 automerge armed: This PR is in ClawSweeper's automerge lane. Sufficient (live_output): The PR body includes redacted live Discord CLI output demonstrating the derived top-level id and preserved nested payload after the patch.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes redacted live Discord CLI output demonstrating the derived top-level id and preserved nested payload after the patch.

What I checked:

Current JSON output lacks stable id: On current main, buildMessageCliJson serializes action, channel, dryRun, handledBy, and the raw payload, but no top-level messageId. (src/commands/message.ts:20, 88d8d6af9336)
Receipt id is available in send results: The send result contract allows a nested result.messageId, and sendMessage returns the last delivery result under result, which is the shape current JSON consumers must inspect. (src/infra/outbound/message.ts:97, 88d8d6af9336)
Runner preserves provider payload: handleSendAction returns send.payload as the CLI result payload, so promoting a stable field in the CLI formatter does not require changing provider delivery behavior. (src/infra/outbound/message-action-runner.ts:1018, 88d8d6af9336)
PR implementation is additive: The provided PR diff for head 9eba815fcc722653331a30df3a61ee42ab438dbc adds extractMessageId for direct payload.messageId and nested payload.result.messageId, while leaving the raw payload in place. (src/commands/message.ts:20, 9eba815fcc72)
Regression test covers nested receipt shape: The provided PR diff adds a command test where payload.result.messageId becomes top-level messageId and the original payload remains unchanged. (src/commands/message.test.ts:300, 9eba815fcc72)
Real behavior proof supplied: The PR body reports a real Discord openclaw message send --json run with IDs redacted, showing a derived top-level id, the nested provider id, and preserved payload. (9eba815fcc72)

Likely related people:

steipete: Peter Steinberger introduced the original message tool/CLI, later unified the message CLI/tools path, and split JSON payload output from logging in the affected command surface. (role: introduced and refactored behavior; confidence: high; commits: db222070143a, 721183e259f6, 4ee41cc6f31e; files: src/commands/message.ts, src/infra/outbound/message.ts, src/runtime.ts)
Takhoffman: Tak Hoffman recently changed src/commands/message.ts and src/commands/message.test.ts for message channel selection behavior in the same command path. (role: recent area contributor; confidence: medium; commits: 897a6a6c5b82; files: src/commands/message.ts, src/commands/message.test.ts)
Kevin Boyle: Kevin Boyle recently touched the message CLI command path for transcript mirroring, which is adjacent to the same send-result flow. (role: adjacent contributor; confidence: low; commits: 99deba798c91; files: src/commands/message.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 88d8d6af9336.

Takhoffman · 2026-05-19T15:45:09Z

@clawsweeper automerge

clawsweeper · 2026-05-19T15:47:24Z

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=9eba815fcc722653331a30df3a61ee42ab438dbc)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-19T23:30:25Z
Merge commit: 1bb0ebab0b06

What merged:

The PR promotes direct or nested send receipt IDs into openclaw message send --json, adds a focused command test, and adds a changelog entry.
Reproducibility: yes. at source level. Current main serializes only the raw payload while send receipts can carry payload.result.messageId; I did not execute the CLI in this read-only review.

Automerge notes:

PR branch already contained follow-up commit before automerge: test(cli): fix message json payload type
PR branch already contained follow-up commit before automerge: chore: retrigger PR checks
PR branch already contained follow-up commit before automerge: Expose messageId in message CLI JSON output

The automerge loop is complete.

Automerge progress:

2026-05-19 20:37:32 UTC repair completed (no branch change) in 25s Run: https://github.com/openclaw/clawsweeper/actions/runs/26123609383 source PR Expose messageId in message CLI JSON output #84191 is paused by clawsweeper:human-review; refusing to mutate the PR branch

2026-05-19 20:37:33 UTC repair finished (blocked) in 27s Run: https://github.com/openclaw/clawsweeper/actions/runs/26123609383 source PR Expose messageId in message CLI JSON output #84191 is paused by clawsweeper:human-review; refusing to mutate the PR branch

2026-05-19 21:34:14 UTC review passed c4237a0128ea (structured ClawSweeper verdict: pass (sha=c4237a0128ea2188bc6fa5ea31ff09b6c28b2...)

2026-05-19 21:34:35 UTC repair queued c4237a0128ea (autonomous) Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390

2026-05-19 23:10:00 UTC repair started (running) in 1s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 automerge-openclaw-openclaw-84191

2026-05-19 23:10:11 UTC validation plan (passed) in 11s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 pnpm check:changed; pnpm lint; pnpm check:test-types

2026-05-19 23:10:23 UTC Codex write preflight (passed) in 24s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 danger-full-access

2026-05-19 23:14:40 UTC Codex edit 1 b1ca69e96403 (complete) in 4m 41s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 exit 0

2026-05-19 23:18:02 UTC validation and review 1 9eba815fcc72 (complete) in 8m 3s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 already-current

2026-05-19 23:18:08 UTC repair completed 9eba815fcc72 (branch updated) in 8m 8s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 initial automerge rebase is delegated to Codex repair

2026-05-19 23:18:08 UTC review queued 9eba815fcc72 (after repair)

2026-05-19 23:28:10 UTC automerge wait 9eba815fcc72 (waiting) in 18m 10s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 waiting for exact-head ClawSweeper review pass

2026-05-19 23:22:37 UTC review queued 9eba815fcc72 (queued)

2026-05-19 23:28:11 UTC repair finished 9eba815fcc72 (pushed) in 18m 11s Run: https://github.com/openclaw/clawsweeper/actions/runs/26126712390 repair_contributor_branch

2026-05-19 23:30:08 UTC review passed 9eba815fcc72 (structured ClawSweeper verdict: pass (sha=9eba815fcc722653331a30df3a61ee42ab438...)

2026-05-19 23:30:27 UTC merged 9eba815fcc72 (merged by ClawSweeper automerge)

clawsweeper · 2026-05-19T16:32:50Z

🦞✅
ClawSweeper is pausing this repair loop for human review.

Source: clawsweeper[bot]
Reason: No ClawSweeper repair lane is needed because the patch has no blocking code findings; remaining work is ordinary maintainer/CI review.; Cleared: The diff only changes CLI JSON shaping and a focused test; it does not touch secrets, permissions, dependencies, CI, install scripts, or other supply-chain surfaces. (sha=9ff66c4de289f3c6fe44d07c281f6a340574cf19)

Why human review is needed:
This item has security-sensitive risk. ClawSweeper is pausing instead of making an autonomous change that could affect trust, credentials, permissions, or exposure.

Recommended next action:
Have a maintainer review the security-sensitive detail and provide an explicit safe path before asking ClawSweeper to continue.

I added clawsweeper:human-review and left the final call with a maintainer.

clawsweeper · 2026-05-19T23:30:10Z

ClawSweeper PR egg

✨ Hatched: 💎 rare Cosmic Patch Peep

       _..------.._          
    .-'  .-.  .-.  '-.       
   /    ( * )( * )    \      
  |        .--.        |     
  |   <\   ====   />   |     
   \    '.______.'    /      
    '-._   ____   _.-'       
        `-.____.-'           
       __/|_||_|\__          
      /__.'    '.__\         
       `-----------'         
 =====================

Rarity: 💎 rare.
Trait: sparkles near resolved comments.
Image traits: location diff observatory; accessory commit compass; palette pearl, teal, and neon green; mood determined; pose waving from a small platform; shell soft velvet shell; lighting calm overcast light; background tiny artifact crates.
How to hatch it: once this PR reaches status: 👀 ready for maintainer look or status: 🚀 automerge armed, the PR author or a maintainer can comment @clawsweeper hatch to turn this ASCII egg into its generated creature image.
Share on X: post this hatch
Copy: My PR egg hatched a 💎 rare Cosmic Patch Peep in ClawSweeper.

What is this egg doing here?

Eggs appear after the PR passes real-behavior proof. It is here for vibes, not verdicts: it does not change labels, ratings, merge decisions, or automation.
The shell reacts to review momentum: open follow-up work warms it up, re-review makes it wobble, and a clean final review lets it hatch.
Hatchable usually means sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness.
The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

100menotu001 · 2026-05-20T02:03:38Z

@clawsweeper re-review

clawsweeper · 2026-05-20T02:05:26Z

🦞🧹
ClawSweeper could not start a re-review for this item.

Reason: re-review requires an open issue or PR.

Summary: - The PR promotes direct or nested send receipt IDs into `openclaw message send --json`, adds a focused command test, and adds a changelog entry. - Reproducibility: yes. at source level. Current main serializes only the raw payload while send receipts can carry `payload.result.messageId`; I did not execute the CLI in this read-only review. Automerge notes: - PR branch already contained follow-up commit before automerge: test(cli): fix message json payload type - PR branch already contained follow-up commit before automerge: chore: retrigger PR checks - PR branch already contained follow-up commit before automerge: Expose messageId in message CLI JSON output Validation: - ClawSweeper review passed for head 9eba815fcc722653331a30df3a61ee42ab438dbc. - Required merge gates passed before the squash merge. Prepared head SHA: 9eba815fcc722653331a30df3a61ee42ab438dbc Review: openclaw/openclaw#84191 (comment) Co-authored-by: OpenClaw Contributor <100menotu001@users.noreply.github.com> Co-authored-by: Craig <froelich@craigs.mac.studio.froho> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

…026.5.20) (#615) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/openclaw/openclaw](https://openclaw.ai) ([source](https://github.com/openclaw/openclaw)) | patch | `2026.5.19` → `2026.5.20` | --- > ⚠️ **Warning** > > Some dependencies could not be looked up. Check the [Dependency Dashboard](issues/567) for more information. --- ### Release Notes <details> <summary>openclaw/openclaw (ghcr.io/openclaw/openclaw)</summary> ### [`v2026.5.20`](https://github.com/openclaw/openclaw/blob/HEAD/CHANGELOG.md#2026520) [Compare Source](openclaw/openclaw@v2026.5.19...v2026.5.20) ##### Changes - Exec approvals: remove the old `cat SKILL.md && printf ... && <skill-wrapper>` allowlist compatibility path so skill files must be loaded with the read tool and only the real skill executable is auto-allowed. - Discord: let voice sessions follow configured Discord users into voice channels, with allowed-channel checks, multi-user handoff, bounded reconciliation, and DAVE recovery preservation. ([#84264](openclaw/openclaw#84264)) Thanks [@fuller-stack-dev](https://github.com/fuller-stack-dev). - Discord/voice: include bounded `IDENTITY.md`, `USER.md`, and `SOUL.md` profile context in realtime voice session instructions by default, with `voice.realtime.bootstrapContextFiles: []` available to disable it. ([#84499](openclaw/openclaw#84499)) Thanks [@fuller-stack-dev](https://github.com/fuller-stack-dev). - Dependencies: bump the bundled Codex harness to `@openai/codex` `0.132.0` and refresh the app-server model-list docs for the new catalog. - CLI/policy: add the bundled Policy plugin for policy-backed channel conformance checks, doctor lint findings, and opt-in workspace repair. ([#80407](openclaw/openclaw#80407)) Thanks [@giodl73-repo](https://github.com/giodl73-repo). - Agents/config: allow `agents.list[].experimental.localModelLean` so lean local-model mode can be enabled for one configured agent instead of globally. - Providers/xAI: add device-code OAuth login so remote and headless setups can authorize xAI without a localhost browser callback. ([#84005](openclaw/openclaw#84005)) Thanks [@fuller-stack-dev](https://github.com/fuller-stack-dev). - Providers/OpenRouter: honor provider-level `params.provider` routing policy for OpenRouter requests, with model and agent params overriding the defaults. Thanks [@amknight](https://github.com/amknight). ##### Fixes - CLI/tasks: include stale-running task maintenance decisions in `openclaw tasks maintenance --json` so retained and reconcile candidates explain backing-session, cron, CLI, and wedged-subagent state. ([#84691](openclaw/openclaw#84691)) Thanks [@efpiva](https://github.com/efpiva). - Codex app-server: keep system-prompt reports working when bootstrap hooks provide workspace files with only a path and content, so hook-supplied SOUL/IDENTITY/TOOLS/USER context still reports injected characters correctly. ([#84736](openclaw/openclaw#84736)) Thanks [@JARVIS-Glasses](https://github.com/JARVIS-Glasses). - Providers/MiniMax music: stop advertising `durationSeconds` control and remove prompt-injected duration hints, so `music_generate` reports MiniMax duration as an unsupported override instead of suggesting MiniMax can enforce track length. Fixes [#84508](openclaw/openclaw#84508). Thanks [@neeravmakwana](https://github.com/neeravmakwana). - Doctor: warn when sandbox tool policy hides configured MCP server tools before provider requests. ([#84699](openclaw/openclaw#84699)) Thanks [@nxmxbbd](https://github.com/nxmxbbd). - WhatsApp: update Baileys to `7.0.0-rc12`. - Build: suppress per-locale `rolldown-plugin-dts:fake-js` CommonJS dts warnings emitted while bundling the intentionally-inlined `zod/v4/locales/*.d.cts` files, so `pnpm build` output stays readable after the 0.25.1 plugin bump. Thanks [@romneyda](https://github.com/romneyda). - CLI/nodes: route lazy plugin-registration logs to stderr for JSON-mode `openclaw nodes` commands so stdout stays parseable. ([#84684](openclaw/openclaw#84684)) Thanks [@TurboTheTurtle](https://github.com/TurboTheTurtle). - Approvals: route manual `/approve` decisions through the trusted approval runtime so active exec and plugin approvals no longer look unknown or expired. - Mac app: update the About settings copyright year to 2026. ([#84385](openclaw/openclaw#84385)) Thanks [@pejmanjohn](https://github.com/pejmanjohn). - Dependencies: update `@openclaw/fs-safe` to `0.2.7` so OpenClaw's default Python-helper-off policy keeps best-effort Node write fallbacks for private stores, secret writes, run logs, and media attachments on Linux/macOS. - Infra/secrets: restore the fail-closed contract for `tryReadSecretFileSync` so credential loaders that pass `rejectSymlink: true` (Telegram, LINE, Zalo, IRC, Nextcloud Talk tokens) refuse symlinked credential files instead of silently accepting them, and the infra-state CI shard's secret-file symlink test passes again. Thanks [@romneyda](https://github.com/romneyda). - Browser: honor the configured image sanitization limit for screenshots and labeled snapshots so browser-captured images follow the same resize policy as other image results. ([#84595](openclaw/openclaw#84595)) - Doctor: remove unrecognized `models.providers.*.models[*].compat.thinkingFormat` values during `doctor --fix` so stale provider model config can validate after upgrade. Fixes [#77803](openclaw/openclaw#77803). - Doctor: warn when `openclaw.json` stores plaintext secret-bearing config fields, including model provider API keys and sensitive provider headers. ([#84718](openclaw/openclaw#84718)) Thanks [@lukaIvanic](https://github.com/lukaIvanic). - Status: show the configured default, session-selected model, reason, clear hint, and docs link when a session remains pinned to a model that differs from `agents.defaults.model.primary`. - WebChat: clear stale typing indicators when session change events mark the active chat run complete. - Mac app: keep local packaging signed with a stable app identity for permission testing and fix Control UI production builds under current Vite/Highlight.js exports. - macOS app: update the embedded Peekaboo bridge to 3.2.1 so OpenClaw-hosted UI automation works with current Peekaboo CLI capture flows. - Cron: deliver preferred final assistant output for successful scheduled runs when trailing plain tool warnings remain in diagnostics instead of marking the run failed. - fix(mattermost): fail closed on missing channel type \[AI]. ([#84091](openclaw/openclaw#84091)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Recheck rebuilt system.run argv \[AI]. ([#84090](openclaw/openclaw#84090)) Thanks [@pgondhi987](https://github.com/pgondhi987). - CLI: keep the private QA subcommand out of exported command descriptors unless `OPENCLAW_ENABLE_PRIVATE_QA_CLI=1`, so root help and subcommand markers match runtime registration. ([#84519](openclaw/openclaw#84519)) - CLI/cron: bound `openclaw cron show` job lookup pagination so non-advancing or unbounded `cron.list` responses fail instead of hanging the command. Fixes [#83856](openclaw/openclaw#83856). ([#83989](openclaw/openclaw#83989)) - Agents/messages: stop message-tool-only turns after a successful source-channel `message` send while keeping transcript mirrors under the session write lock. ([#84289](openclaw/openclaw#84289)) - Agents: filter silent heartbeat response-tool transcript artifacts out of embedded context snapshots so later user turns are not polluted by heartbeat no-op messages. ([#83477](openclaw/openclaw#83477)) Thanks [@fuller-stack-dev](https://github.com/fuller-stack-dev). - Agents/OpenAI: log repeated strict tool-schema downgrade diagnostics once per provider/model/tool signature, reducing duplicate debug noise while preserving `strict=false` fallback behavior. Fixes [#82930](openclaw/openclaw#82930). ([#82933](openclaw/openclaw#82933)) Thanks [@galiniliev](https://github.com/galiniliev). - Agents/code mode: spell out the `exec` tool's JavaScript/TypeScript, no Node module, and catalog-bridge constraints in model-visible schema text so agents can use enabled tools without trial-and-error. ([#84269](openclaw/openclaw#84269)) Thanks [@Kaspre](https://github.com/Kaspre). - Codex: give `image_generate` dynamic-tool calls a 120s default watchdog when no per-call or configured image timeout is set, so image generation no longer falls back to the generic 30s bridge timeout. ([#84254](openclaw/openclaw#84254)) Thanks [@moritzmmayerhofer](https://github.com/moritzmmayerhofer). - Codex: avoid duplicate dynamic tool terminal diagnostics while large diagnostic backlogs drain without blocking tool responses. ([#82937](openclaw/openclaw#82937)) Thanks [@galiniliev](https://github.com/galiniliev). - CLI/message: include a stable top-level `messageId` in `openclaw message --json` output when channel sends return one. ([#84191](openclaw/openclaw#84191)) Thanks [@100menotu001](https://github.com/100menotu001). - Cron: preserve legacy top-level array `jobs.json` stores when loading or adding scheduled jobs so old cron jobs are no longer treated as an empty store during upgrade. Fixes [#60799](openclaw/openclaw#60799). ([#84433](openclaw/openclaw#84433)) Thanks [@IWhatsskill](https://github.com/IWhatsskill). - Gateway/agents: use an agent's `identity.name` in Gateway agent summaries when `agents.list[].name` is unset, so configured agent labels remain visible in clients. ([#84355](openclaw/openclaw#84355); refs [#57835](openclaw/openclaw#57835)) Thanks [@luoyanglang](https://github.com/luoyanglang). - Channels/replies: keep normal `/verbose` failed-tool progress compact in message-tool replies and prevent late text-only tool output from appearing after the final answer. ([#84303](openclaw/openclaw#84303)) Thanks [@VACInc](https://github.com/VACInc). - Plugins/hooks: apply a default 30-second timeout to `before_compaction` and `after_compaction` hooks so a hung plugin handler no longer blocks compaction completion. ([#84153](openclaw/openclaw#84153)) - Discord: preserve disabled presentation buttons when adapting and rendering Discord message controls. ([#84188](openclaw/openclaw#84188)) Thanks [@100menotu001](https://github.com/100menotu001). - Twitch: add a test-only client-manager registry reset helper so non-isolated Twitch tests can clear cached managers between cases. Fixes [#83887](openclaw/openclaw#83887). ([#84244](openclaw/openclaw#84244)) Thanks [@hclsys](https://github.com/hclsys). - Cron: run main-session scheduled work on a cron-owned wake lane while preserving reply delivery context, so background cron turns no longer block human main-session chat. Fixes [#82766](openclaw/openclaw#82766). ([#82767](openclaw/openclaw#82767)) Thanks [@galiniliev](https://github.com/galiniliev). - Cron: use structured embedded-run denial metadata for isolated scheduled tasks so blocked exec requests fail the job without treating ordinary assistant prose as a denial. ([#84067](openclaw/openclaw#84067)) Thanks [@abnershang](https://github.com/abnershang). - Cron: keep recovered tool warnings diagnostic for successful scheduled runs so final cron output is delivered instead of being replaced by a post-processing warning. ([#84045](openclaw/openclaw#84045)) Thanks [@abnershang](https://github.com/abnershang). - Plugins/perf: thread explicit plugin discovery results through `loadBundledCapabilityRuntimeRegistry`, `resolveBundledPluginSources`, and `listChannelCatalogEntries` so callers that already hold a discovery result skip redundant filesystem walks. Thanks [@SebTardif](https://github.com/SebTardif). - harden update restart script creation \[AI]. ([#84088](openclaw/openclaw#84088)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Docker: keep the bundled Codex plugin in official release image keep lists so the default OpenAI agent harness remains available after Docker pruning. Fixes [#83613](openclaw/openclaw#83613). ([#83626](openclaw/openclaw#83626)) Thanks [@YuanHanzhong](https://github.com/YuanHanzhong). - CLI/channels: preserve the first line of `openclaw channels logs` output when the rolling tail window starts exactly on a line boundary, mirroring the already-fixed `readLogSlice` behavior in `src/logging/log-tail.ts`. - Control UI: treat terminal session status as authoritative over stale active-run flags so completed terminal runs stop showing abort/live UI. ([#84057](openclaw/openclaw#84057)) - CLI: preserve embedded equals signs in inline root option values instead of truncating after the second separator. ([#83995](openclaw/openclaw#83995)) Thanks [@ThiagoCAltoe](https://github.com/ThiagoCAltoe). - Matrix/config: accept `messages.queue.byChannel.matrix` queue overrides and keep queue provider schema/type keys aligned for Matrix, Google Chat, and Mattermost. Thanks [@bdjben](https://github.com/bdjben). - CLI: format `openclaw acp client` failures through the shared error formatter so object-shaped errors stay readable instead of printing `[object Object]`. Fixes [#83904](openclaw/openclaw#83904). ([#84080](openclaw/openclaw#84080)) - Providers/Ollama: default unknown-capabilities models to tool-capable so discovered native Ollama models can use tools when `/api/show` omits capabilities. ([#84055](openclaw/openclaw#84055)) Thanks [@dutifulbob](https://github.com/dutifulbob). - Installer/Windows: launch `install.ps1` onboarding as an attached child process so fresh native Windows installs do not freeze visibly at `Starting setup...` or corrupt the wizard's terminal rendering. - CLI/update: keep restart health checks working across one-version CLI/Gateway protocol skew and use the managed Gateway service Node for all follow-up commands even when the package root is unchanged, so `openclaw update` no longer silently switches the gateway to a different Node binary when multiple Node installations are present. Thanks [@amknight](https://github.com/amknight). - CLI/gateway: include the running Gateway version in `gateway status` JSON output, preserving existing server metadata while falling back to status RPC data for read probes. Fixes [#56222](openclaw/openclaw#56222). Thanks [@galiniliev](https://github.com/galiniliev). - Memory/search: close local embedding providers when active-memory searches time out so pending local model loads and embedding contexts are aborted and released. ([#83858](openclaw/openclaw#83858)) Thanks [@brokemac79](https://github.com/brokemac79). - CLI/nodes: request pending node surface approval scopes before `openclaw nodes approve` so exec-capable node approval can use admin-scoped Gateway credentials instead of failing with `missing scope: operator.admin`. ([#84392](openclaw/openclaw#84392)) Thanks [@joshavant](https://github.com/joshavant). - Gateway: reject slow node event sends before outbound buffers grow unbounded and log the rejected payload diagnostic. ([#84387](openclaw/openclaw#84387)) Thanks [@samzong](https://github.com/samzong). - Agents: include bounded trajectory queued-writer diagnostics in `pi-trajectory-flush` timeout warnings so flush stalls show pending writes, queued bytes, and append state. Fixes [#82961](openclaw/openclaw#82961). ([#82962](openclaw/openclaw#82962)) Thanks [@galiniliev](https://github.com/galiniliev). - Agents/subagents: recover stale completion announces by retrying unsupported transcript-wait wakes without transcript waiting and forcing a message-tool handoff when the requester run is already stale. Fixes [#83699](openclaw/openclaw#83699). ([#83700](openclaw/openclaw#83700)) Thanks [@galiniliev](https://github.com/galiniliev). - Agents/subagents: constrain wildcard subagent target allowlists to configured agents while preserving explicitly listed compatibility targets. Fixes [#84040](openclaw/openclaw#84040). ([#84357](openclaw/openclaw#84357)) Thanks [@joshavant](https://github.com/joshavant). - Providers/Anthropic: route Anthropic model refs selected with Claude CLI auth through the Claude CLI runtime so shorthand refs such as `anthropic/opus-4.7` no longer fall back to embedded Anthropic billing. Fixes [#84222](openclaw/openclaw#84222). ([#84374](openclaw/openclaw#84374)) Thanks [@joshavant](https://github.com/joshavant). - Agents: honor explicit `models.providers.<id>.timeoutSeconds` values above the default idle watchdog for cloud and self-hosted providers, so long first-token waits no longer fall back at \~120s when the provider timeout is higher. ([#83979](openclaw/openclaw#83979)) Thanks [@yujiawei](https://github.com/yujiawei). - Agents/Codex: keep encrypted Responses reasoning replay provenance-bound so stale mirrored Codex transcripts drop invalid encrypted content before request assembly while preserving matching same-session replay. Fixes [#83836](openclaw/openclaw#83836). ([#84367](openclaw/openclaw#84367)) Thanks [@joshavant](https://github.com/joshavant). - Agents/subagents: skip stale embedded-run wake probes for dormant completion requesters, so late subagent completions go straight to requester-agent/direct handoff instead of producing `reason=no_active_run` queue noise. ([#82964](openclaw/openclaw#82964)) Thanks [@galiniliev](https://github.com/galiniliev). - CLI: retry config snapshot reads after a transient failure so one rejected read no longer poisons later commands in the same process. ([#83931](openclaw/openclaw#83931)) Thanks [@honor2030](https://github.com/honor2030). - Media: decode URL path basenames before using them as remote media fallback filenames, so files like `My%20Report.pdf` are surfaced as `My Report.pdf`. Fixes [#84050](openclaw/openclaw#84050). ([#84052](openclaw/openclaw#84052)) Thanks [@jbetala7](https://github.com/jbetala7). - WhatsApp: clarify inbound group diagnostics so observed but unregistered groups point to `channels.whatsapp.groups` without changing routing or sender authorization. ([#83846](openclaw/openclaw#83846)) Thanks [@neeravmakwana](https://github.com/neeravmakwana). - WhatsApp: drain pending outbound deliveries on a 30s periodic timer in addition to the reconnect handler, so messages enqueued while the provider is already connected no longer wait for the next reconnect to send. ([#79083](openclaw/openclaw#79083)) Thanks [@Oviemudiaga](https://github.com/Oviemudiaga). - CLI/TUI: include gateway plugin slash commands in TUI autocomplete, so connected sessions can suggest plugin-owned commands exposed by the running Gateway. ([#83640](openclaw/openclaw#83640)) Thanks [@se7en-agent](https://github.com/se7en-agent). - Gateway/mobile: restore QR setup-code handoff of bounded operator tokens for iOS and Android onboarding while keeping admin and pairing scopes out of bootstrap. ([#83684](openclaw/openclaw#83684)) Thanks [@ngutman](https://github.com/ngutman). - iOS: repair Release archive compilation for the TestFlight build. ([#84255](openclaw/openclaw#84255)) Thanks [@ngutman](https://github.com/ngutman). - Agents/compaction: bound plugin-owned CLI transcript compaction with the host safety timeout so a hung context engine can no longer stall post-turn cleanup. ([#84083](openclaw/openclaw#84083)) Thanks [@100yenadmin](https://github.com/100yenadmin). - Control UI/usage: truncate long context skill, tool, and file names in the usage panel while keeping the full name available on hover. ([#42197](openclaw/openclaw#42197)) Thanks [@Rain120](https://github.com/Rain120). - Codex: respect explicit `models auth order set` and `config.auth.order` precedence over stale `lastGood` in `/codex account`, and show `no working credential` when every explicit-order profile is ineligible instead of marking a lower-ranked profile as active. Fixes [#84386](openclaw/openclaw#84386). ([#84412](openclaw/openclaw#84412)) Thanks [@openperf](https://github.com/openperf). - Agents: honor `messages.suppressToolErrors` for mutating tool failures so configured chat surfaces do not receive separate warning payloads. ([#81561](openclaw/openclaw#81561)) Thanks [@moeedahmed](https://github.com/moeedahmed). - Agents/fallback: surface billing guidance for mixed rate-limit plus billing fallback exhaustion instead of generic failure copy. Fixes [#79396](openclaw/openclaw#79396). ([#79489](openclaw/openclaw#79489)) Thanks [@aayushprsingh](https://github.com/aayushprsingh). </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/615

Summary: - The PR promotes direct or nested send receipt IDs into `openclaw message send --json`, adds a focused command test, and adds a changelog entry. - Reproducibility: yes. at source level. Current main serializes only the raw payload while send receipts can carry `payload.result.messageId`; I did not execute the CLI in this read-only review. Automerge notes: - PR branch already contained follow-up commit before automerge: test(cli): fix message json payload type - PR branch already contained follow-up commit before automerge: chore: retrigger PR checks - PR branch already contained follow-up commit before automerge: Expose messageId in message CLI JSON output Validation: - ClawSweeper review passed for head 9eba815. - Required merge gates passed before the squash merge. Prepared head SHA: 9eba815 Review: openclaw#84191 (comment) Co-authored-by: OpenClaw Contributor <100menotu001@users.noreply.github.com> Co-authored-by: Craig <froelich@craigs.mac.studio.froho> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

openclaw-barnacle Bot added commands Command implementations size: S triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels May 19, 2026

clawsweeper Bot mentioned this pull request May 19, 2026

message send --json should expose a top-level messageId #84186

Closed

clawsweeper Bot added rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. labels May 19, 2026

openclaw-barnacle Bot added proof: supplied External PR includes structured after-fix real behavior proof. and removed triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels May 19, 2026

clawsweeper Bot added the clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge label May 19, 2026

clawsweeper Bot added the clawsweeper:human-review Needs maintainer review before ClawSweeper can continue label May 19, 2026

100menotu001 and others added 4 commits May 19, 2026 23:11

fix(cli): expose message id in json output

19f7cf9

test(cli): fix message json payload type

fab9a97

chore: retrigger PR checks

b1ca69e

Expose messageId in message CLI JSON output

9eba815

clawsweeper Bot force-pushed the codex/message-cli-json-message-id branch from d96e335 to 9eba815 Compare May 19, 2026 23:18

clawsweeper Bot merged commit 1bb0eba into openclaw:main May 19, 2026
97 checks passed

Haderach-Ram mentioned this pull request May 20, 2026

Ecosystem Digest — 2026-05-20 Haderach-Ram/openclaw-radar#13

Open

github-actions Bot mentioned this pull request May 20, 2026

📡 Upstream Digest — 2026-05-20 02:34 UTC curtismercier/openclaw-mods#901

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Expose messageId in message CLI JSON output#84191

Expose messageId in message CLI JSON output#84191
clawsweeper[bot] merged 4 commits into
openclaw:mainfrom
100menotu001:codex/message-cli-json-message-id

100menotu001 commented May 19, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 19, 2026 •

edited

Loading

Uh oh!

Takhoffman commented May 19, 2026

Uh oh!

clawsweeper Bot commented May 19, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

Uh oh!

100menotu001 commented May 20, 2026

Uh oh!

clawsweeper Bot commented May 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

100menotu001 commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Motivation

Change Type

Scope

Linked Issue/PR

Real Behavior Proof

Root Cause

Regression Test Plan

User-visible / Behavior Changes

Diagram

Security Impact

Repro + Verification

Evidence

Human Verification

Compatibility / Migration

Risks

Uh oh!

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

Uh oh!

Uh oh!

Takhoffman commented May 19, 2026

Uh oh!

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

Uh oh!

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

Uh oh!

100menotu001 commented May 20, 2026

Uh oh!

clawsweeper Bot commented May 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

100menotu001 commented May 19, 2026 •

edited

Loading

clawsweeper Bot commented May 19, 2026 •

edited

Loading

clawsweeper Bot commented May 19, 2026 •

edited

Loading