Repository navigation
Conversation
Approvals, command auth, arg menus, session reads and patches, skill command listing and remote media now use the async replacements that 2026.11.1 ships, ahead of the January 2027 removal. Buttons render from MessagePresentation instead of the InteractiveReply bridge, including presentation-only host replies. Zod 4 deprecations cleaned up.
Tools, outbound sends, commands and pairing notices reused nothing and opened a connection per call. They now borrow the running monitor's client for the same account, token and server, and fall back to a short-lived client otherwise. The registry lives on a process-wide key because each plugin entry is bundled separately.
Presence, reasoning, addressing, model menus, reply actions, history and inbound media now live in their own files. Code moved verbatim; the monitorInlineProvider closure is untouched and phase 2 rewrites it.
morajabi
added a commit
that referenced
this pull request
Nov 8, 2026
…itor sends From the 1.0 review (draft #158): typed command actions keep their kind and run as system commands instead of agent turns, disabled buttons stay inert, presentation fallback text survives authored text, cancelled callers can't borrow the shared client, and the monitor's 15 direct sends share one sender that waits an explicit server flood hint (up to 60s, twice) with a stable randomId and never retries uncertain sends.
morajabi
added a commit
that referenced
this pull request
Nov 8, 2026
…itor sends From the 1.0 review (draft #158): typed command actions keep their kind and run as system commands instead of agent turns, disabled buttons stay inert, presentation fallback text survives authored text, cancelled callers can't borrow the shared client, and the monitor's 15 direct sends share one sender that waits an explicit server flood hint (up to 60s, twice) with a stable randomId and never retries uncertain sends.
morajabi
force-pushed
the
claude/openclaw-1.0
branch
from
November 8, 2026 12:46
620ca31 to
9cde5a6
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Review follow-up stacked on #157. The phase-1 migration is a useful foundation; this fixes concrete presentation and delivery gaps found by comparing OpenClaw 2026.11.1/current source, native Discord/Slack/Telegram adapters, Hermes, bridge v2, and current Inline primitives.
The internal rendered-button data now carries explicit command-kind metadata. Inline protocol, wire fields, database, and plugin configuration schema are unchanged. No version bump or release.
The source-backed review and remaining 1.0 plan explains the larger work still needed: inbound receipt ownership and metadata recovery, published output-settings APIs, composer integration, Detailed progress, grant-backed approvals, Auto-thread session continuity, and real-host bot hand-backs. The monitor retry helper is not a durable outbox.
Validation:
bun run checkpasses type checking, lint, 48 test files / 770 tests with coverage, packed-artifact validation, and the runtime/declaration build. New regressions exercise typed-button dispatch through the monitor, disabled controls, URL fallback, cancellation, flood waits, stable send identity, and uncertain outcomes. A real gateway/live bots were not used.Base is the existing 1.0 branch so the diff contains only this review's fixes.