fix(doctor): make restart follow-up actionable by giodl73-repo · Pull Request #87361 · openclaw/openclaw

giodl73-repo · 2026-05-27T17:11:57Z

Behavior addressed

Post-restart doctor guidance no longer presents openclaw doctor --non-interactive as an immediate command that can be run from every delivery surface. The generated restart sentinel hint now tells users to run the profile-aware command from a terminal or approvals-capable OpenClaw surface.

Real environment tested

WSL Ubuntu local checkout on branch fix-69755-doctor-hint-actionable, based on origin/main at 11dfef201f.

Exact steps or command run after this patch

node scripts/run-vitest.mjs src/infra/restart-sentinel.test.ts src/gateway/server-methods/update.test.ts src/gateway/server-restart-sentinel.test.ts src/agents/tools/gateway-tool.test.ts src/auto-reply/reply/commands-session-restart.test.ts src/agents/openclaw-gateway-tool.test.ts --reporter=dot
./node_modules/.bin/oxfmt --check src/infra/restart-sentinel.ts src/gateway/server-methods/config-write-flow.ts src/infra/update-restart-sentinel-payload.ts src/gateway/server-restart-sentinel.ts src/agents/tools/gateway-tool.ts src/auto-reply/reply/commands-session.ts src/infra/restart-sentinel.test.ts src/gateway/server-methods/update.test.ts src/gateway/server-restart-sentinel.test.ts src/agents/tools/gateway-tool.test.ts src/auto-reply/reply/commands-session-restart.test.ts src/agents/openclaw-gateway-tool.test.ts
./node_modules/.bin/oxlint --tsconfig config/tsconfig/oxlint.core.json src/infra/restart-sentinel.ts src/infra/restart-sentinel.test.ts src/agents/openclaw-gateway-tool.test.ts src/agents/tools/gateway-tool.test.ts src/auto-reply/reply/commands-session-restart.test.ts
git diff --check

Real restart-sentinel round trip using the patched source module:

export OPENCLAW_STATE_DIR=$(mktemp -d)
export OPENCLAW_PROFILE=isolated
./node_modules/.bin/tsx --eval 'import { consumeRestartSentinel, formatDoctorNonInteractiveHint, formatRestartSentinelMessage, writeRestartSentinel } from "./src/infra/restart-sentinel.ts"; void (async () => { const payload = { kind: "restart", status: "ok", ts: Date.now(), sessionKey: "agent:main:redacted:dm:redacted", message: "/restart", doctorHint: formatDoctorNonInteractiveHint(), stats: { mode: "gateway.restart", reason: "/restart" } }; const sentinelPath = await writeRestartSentinel(payload); const sentinel = await consumeRestartSentinel(); console.log("sentinelPath=" + sentinelPath.replace(process.env.OPENCLAW_STATE_DIR, "$OPENCLAW_STATE_DIR")); console.log("--- delivered restart notice ---"); console.log(formatRestartSentinelMessage(sentinel.payload)); })();'

Evidence after fix

formatDoctorNonInteractiveHint() now returns:

Recommended follow-up: run openclaw doctor --non-interactive in a terminal or approvals-capable OpenClaw surface.

With OPENCLAW_PROFILE=isolated, it preserves profile-aware command formatting:

Recommended follow-up: run openclaw --profile isolated doctor --non-interactive in a terminal or approvals-capable OpenClaw surface.

Redacted real restart-sentinel output from the command above:

sentinelPath=$OPENCLAW_STATE_DIR/restart-sentinel.json
--- delivered restart notice ---
Gateway restart restart ok (gateway.restart)
/restart
Recommended follow-up: run openclaw --profile isolated doctor --non-interactive in a terminal or approvals-capable OpenClaw surface.

Focused restart/update delivery tests passed: 9 files, 153 tests.

GitHub CI for this PR is green, including check-guards, check-lint, check-prod-types, check-test-types, checks-node-agentic-commands-doctor, checks-node-agentic-commands-doctor-shared, checks-node-agentic-gateway-core, checks-node-auto-reply-reply-commands, and the changed-path node shards.

Observed result after fix

The restart sentinel still carries a single doctorHint string, so no public config, plugin, or payload-shape surface changed. The hint text now distinguishes recommended follow-up from actionability in constrained heartbeat/outbound contexts.

What was not tested

I did not exercise a live macOS heartbeat session or live approval-timeout flow.

node scripts/crabbox-wrapper.mjs run --shell -- "pnpm check:changed" could not run because the local crabbox binary failed its own --version/--help sanity check before invoking the command. Running pnpm check:changed directly also stopped before checks due this WSL worktree using a symlinked node_modules and pnpm requiring an interactive purge confirmation.

clawsweeper · 2026-05-27T17:13:32Z

Codex review: needs maintainer review before merge. Reviewed May 27, 2026, 5:00 PM ET / 21:00 UTC.

Summary
The branch changes formatDoctorNonInteractiveHint to route users toward a terminal or approvals-capable OpenClaw surface and updates restart/gateway/session tests for the new hint.

PR surface: Source +3, Tests +21. Total +24 across 5 files.

Reproducibility: yes. from source inspection: current main still formats the restart doctor hint as an immediate Run: command, and the related issue documents the approval-constrained failure mode. I did not run a live macOS heartbeat repro in this read-only review.

Review metrics: none identified.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🐚 platinum hermit
Patch quality: 🦞 diamond lobster
Result: ready for maintainer review.

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

Rank-up moves:

none

Risk before merge

The PR body did not exercise a live macOS heartbeat or approval-timeout flow; confidence comes from the shared helper, focused sibling-surface tests, and a source-module restart-sentinel round trip.

Maintainer options:

Decide the mitigation before merge
Land the narrow helper and test update after protected-label maintainer review, then let the closing reference resolve Post-restart suggestion to run openclaw doctor --non-interactive is not always actionable in heartbeat / approval-constrained contexts #69755.
Pause or close
Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge
Manual review is appropriate because the PR has a protected maintainer label and no narrow automated repair remains after proof was added.

Security
Cleared: The diff only changes a restart guidance string and tests; it does not touch dependencies, workflows, credentials, permissions, downloads, or package execution surfaces.

Review details

Best possible solution:

Land the narrow helper and test update after protected-label maintainer review, then let the closing reference resolve #69755.

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

Yes from source inspection: current main still formats the restart doctor hint as an immediate Run: command, and the related issue documents the approval-constrained failure mode. I did not run a live macOS heartbeat repro in this read-only review.

Is this the best way to solve the issue?

Yes: updating the central helper is the narrowest maintainable fix because update, config-write, gateway-tool, and auto-reply restart paths already share that formatter and the PR preserves the existing payload shape.

AGENTS.md: found and applied where relevant.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 7f7eca1ad2ba.

Label changes

Label changes:

add P2: The PR addresses a real post-restart operational follow-up bug, but the blast radius is limited to user-facing guidance text.
add proof: sufficient: Contributor real behavior proof is sufficient. The PR body now includes redacted terminal output from a patched restart-sentinel write/consume round trip showing the new profile-aware hint after the fix.
add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🦞 diamond lobster.
add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (terminal): The PR body now includes redacted terminal output from a patched restart-sentinel write/consume round trip showing the new profile-aware hint after the fix.
remove rating: 🦪 silver shellfish: Current PR rating is rating: 🐚 platinum hermit, so this older rating label is no longer current.
remove status: 📣 needs proof: Current PR status label is status: 👀 ready for maintainer look.

Label justifications:

P2: The PR addresses a real post-restart operational follow-up bug, but the blast radius is limited to user-facing guidance text.
rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🦞 diamond lobster.
status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (terminal): The PR body now includes redacted terminal output from a patched restart-sentinel write/consume round trip showing the new profile-aware hint after the fix.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body now includes redacted terminal output from a patched restart-sentinel write/consume round trip showing the new profile-aware hint after the fix.

Evidence reviewed

PR surface:

Source +3, Tests +21. Total +24 across 5 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	1	4	1	+3
Tests	4	29	8	+21
Docs	0	0	0	0
Config	0	0	0	0
Generated	0	0	0	0
Other	0	0	0	0
Total	5	33	9	+24

What I checked:

Repository policy read: Root AGENTS.md and scoped src/agents / src/agents/tools policy were read fully; the protected-label, real-proof, and scoped agent-test guidance affected this review. (AGENTS.md:17, 7f7eca1ad2ba)
Current main still has the reported hint: On current main, the central formatter still returns Run: ${formatCliCommand("openclaw doctor --non-interactive", env)}, so the central problem is not already implemented on main. (src/infra/restart-sentinel.ts:76, 7f7eca1ad2ba)
Shipped release still has the reported hint: The latest release tag v2026.5.26 points at 10ad3aa16068baa84a1bd9ac4f7d42ae725cedb7 and contains the same immediate Run: doctor hint. (src/infra/restart-sentinel.ts:76, 10ad3aa16068)
Patch is narrow and central: The PR commit changes only the shared restart-sentinel hint helper plus expectations in five focused tests; it does not change sentinel payload shape, config, dependencies, or workflows. (src/infra/restart-sentinel.ts:73, 14159d24c726)
Shared callers use the central formatter: Update, config-write, gateway-tool restart, and auto-reply restart paths all populate doctorHint through formatDoctorNonInteractiveHint, so changing the helper reaches the sibling restart surfaces inspected. (src/infra/update-restart-sentinel-payload.ts:43, 7f7eca1ad2ba)
Real behavior proof supplied: The updated PR body includes a redacted terminal round trip that writes and consumes a restart sentinel with OPENCLAW_PROFILE=isolated and prints the new profile-aware follow-up message. (14159d24c726)

Likely related people:

steipete: Peter Steinberger is the blamed author/committer for the current restart-sentinel helper and the current restart caller wiring after the latest main-side packaging refresh. (role: recent area contributor; confidence: high; commits: 659b5dce79ee; files: src/infra/restart-sentinel.ts, src/infra/update-restart-sentinel-payload.ts, src/gateway/server-methods/config-write-flow.ts)
steipete: The earlier command formatting behavior used by the doctor hint was normalized in commit 6d5195c890699c49e41e4bd4b632ed7efe95d6df. (role: command-hint refactor owner; confidence: high; commits: 6d5195c89069; files: src/cli/command-format.ts, src/infra/restart-sentinel.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.

clawsweeper · 2026-05-27T17:17:49Z

ClawSweeper PR egg

✨ Hatched: 🥚 common Brave Branchling

Hatch command

Comment @clawsweeper hatch when this PR is hatchable.

Hatchability rules:

Merged PRs are hatchable.
Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

Rarity: 🥚 common.
Trait: sparkles near resolved comments.
Image traits: location proof lagoon; accessory tiny test log scroll; palette cobalt, lime, and pearl; mood sleepy but ready; pose sitting proudly on a smooth stone; shell brushed metal shell; lighting tiny status-light glow; background little resolved-comment flags.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Brave Branchling 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.
Hatchability usually comes from sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness. A merged PR is already final, so merge makes the egg hatchable independently.
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.

giodl73-repo · 2026-05-27T20:51:31Z

@clawsweeper re-review

clawsweeper · 2026-05-27T20:51:34Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

State: Complete
Detail: The targeted re-review finished, the durable review comment was updated, and the synced verdict was routed.
Run: https://github.com/openclaw/clawsweeper/actions/runs/26538052141
Updated: 2026-05-27T21:01:03.494Z

steipete

Maintainer review approved.

Behavior addressed: post-restart doctor guidance is no longer presented as an immediately actionable command from every delivery surface; it now points users to a terminal or approvals-capable OpenClaw surface while preserving profile-aware command formatting.

Real environment tested: GitHub CI for PR head 14159d2; maintainer code-path review from a clean local main checkout.

Exact steps or command run after this patch:

git status -sb
git pull --ff-only
gh pr view 87361 --repo openclaw/openclaw --json number,title,state,isDraft,mergeable,reviewDecision,headRefOid,baseRefOid,author,assignees,maintainerCanModify,url,body,additions,deletions,changedFiles
gh pr checks 87361 --repo openclaw/openclaw --json name,state,bucket,workflow,link
gh pr diff 87361 --repo openclaw/openclaw --patch
gh issue view 69755 --repo openclaw/openclaw
rg -n "formatDoctorNonInteractiveHint|doctorHint|restart-sentinel|formatRestartSentinelMessage" src/infra src/gateway src/agents src/auto-reply
read restart sentinel formatter, gateway restart tool, slash restart command, config write sentinel, update sentinel payload, and adjacent tests

Evidence after fix: diff changes the shared formatDoctorNonInteractiveHint() string only, and all restart/config/update writers still populate the same single doctorHint field consumed by formatRestartSentinelMessage(). CI reports 104 checks with no fail/pending/cancel blockers; relevant green lanes include check-guards, check-lint, check-prod-types, check-test-types, checks-node-agentic-commands-doctor, checks-node-agentic-commands-doctor-shared, checks-node-agentic-gateway-core, and checks-node-auto-reply-reply-commands.

Observed result after fix: example restart sentinel output changes from Run: openclaw --profile isolated doctor --non-interactive to Recommended follow-up: run openclaw --profile isolated doctor --non-interactive in a terminal or approvals-capable OpenClaw surface.

What was not tested: I did not run a live macOS heartbeat session or live approval-timeout flow locally; contributor noted the same gap in the PR body.

steipete · 2026-05-27T22:23:58Z

Landed on main.

Gate: GitHub CI for head 14159d2 reported 104 checks with no fail/pending/cancel blockers; maintainer review covered restart-sentinel formatter and restart/config/update doctorHint writers.
Merge commit: f3e2851
Linked issue: Post-restart suggestion to run openclaw doctor --non-interactive is not always actionable in heartbeat / approval-constrained contexts #69755 is closed with proof.

Thanks @giodl73-repo!

…026.5.28) (#759) 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.27` → `2026.5.28` | --- ### Release Notes <details> <summary>openclaw/openclaw (ghcr.io/openclaw/openclaw)</summary> ### [`v2026.5.28`](https://github.com/openclaw/openclaw/blob/HEAD/CHANGELOG.md#2026528) [Compare Source](openclaw/openclaw@v2026.5.27...v2026.5.28) ##### Highlights - Agent and Codex runtime recovery is steadier: subagents keep cwd/workspace separation, hook context stays prompt-local, session locks release on timeout abort while live OpenClaw locks survive cleanup, stale restart continuations are avoided, and Codex app-server/helper failures no longer tear down shared runtime state. ([#87218](openclaw/openclaw#87218), [#86875](openclaw/openclaw#86875), [#87409](openclaw/openclaw#87409), [#87399](openclaw/openclaw#87399), [#87375](openclaw/openclaw#87375), [#88129](openclaw/openclaw#88129)) - Channel delivery and session identity got safer across outbound plugin hooks, Matrix room ids, iMessage reactions/approvals, Slack final replies, Discord recovered tool warnings, runtime-config message actions, WhatsApp profile auth roots, Telegram polling, and Microsoft Teams service URL trust checks. ([#73706](openclaw/openclaw#73706), [#75670](openclaw/openclaw#75670), [#87366](openclaw/openclaw#87366), [#87451](openclaw/openclaw#87451), [#87334](openclaw/openclaw#87334), [#84535](openclaw/openclaw#84535), [#82492](openclaw/openclaw#82492), [#83304](openclaw/openclaw#83304), [#87160](openclaw/openclaw#87160)) - Mobile and chat surfaces got a broader refresh: the iOS Pro UI, hosted push relay default, realtime Talk tab playback, Gateway chat transport, onboarding, Talk permissions, WebChat reconnect delivery, and session picker behavior now preserve more state across reconnects and empty searches. ([#87367](openclaw/openclaw#87367), [#87531](openclaw/openclaw#87531), [#87682](openclaw/openclaw#87682), [#88096](openclaw/openclaw#88096), [#88105](openclaw/openclaw#88105)) Thanks [@ngutman](https://github.com/ngutman) and [@BunsDev](https://github.com/BunsDev). - Browser, channel, and automation inputs are stricter: Browser tool timeouts, viewport/tab indices, Gateway ports, cron retry handling, Discord component ids, schema array refs, Telegram callback pages, and channel progress callbacks now reject malformed values earlier and preserve the intended delivery context. ([#82887](openclaw/openclaw#82887)) - Provider, media, and document coverage expands with Claude Opus 4.8, Fal Krea image schemas, NVIDIA featured models, MiniMax streaming music responses, encrypted PDF extraction, voice model catalogs, GitHub Copilot agent runtime support, and a Codex Supervisor plugin path for delegated Codex workflows. ([#87845](openclaw/openclaw#87845), [#87890](openclaw/openclaw#87890), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764), [#87751](openclaw/openclaw#87751), [#87794](openclaw/openclaw#87794)) - CLI, auth, doctor, and provider paths fail faster and recover more clearly: malformed numeric/version options are rejected, workspace dotenv provider credentials are ignored, heartbeat defaults, OAuth/token lifetimes, and local service startup requests are bounded, agent auth health labels are clearer, legacy `api_key` auth profiles migrate to canonical form, and restart guidance is actionable. ([#87398](openclaw/openclaw#87398), [#86281](openclaw/openclaw#86281), [#87361](openclaw/openclaw#87361), [#88133](openclaw/openclaw#88133), [#83655](openclaw/openclaw#83655), [#87559](openclaw/openclaw#87559), [#88088](openclaw/openclaw#88088), [#85924](openclaw/openclaw#85924)) Thanks [@vincentkoc](https://github.com/vincentkoc) and [@giodl73-repo](https://github.com/giodl73-repo). - Plugin and Gateway hot paths do less repeated work while preserving cache correctness for install records, config JSON parsing, tool search catalogs, session stores, manifest model rows, auto-enabled plugin config, browser tokens, viewer assets, and release-split external plugin packages. ([#86699](openclaw/openclaw#86699)) - Release, QA, and E2E validation now bound more log, artifact, harness, and cross-OS waits so failing lanes produce proof instead of hanging or false-greening. ##### Changes - Status: show active subagent details in status output. - Diffs: split the default language pack and expand default Diffs language coverage while keeping the host floor aligned. ([#87370](openclaw/openclaw#87370), [#87372](openclaw/openclaw#87372)) Thanks [@RomneyDa](https://github.com/RomneyDa). - ClawHub: add plugin display names plus skill verification and trust surfaces. ([#87354](openclaw/openclaw#87354), [#86699](openclaw/openclaw#86699)) Thanks [@thewilloftheshadow](https://github.com/thewilloftheshadow) and [@Patrick-Erichsen](https://github.com/Patrick-Erichsen). - iOS: refresh the dev app with Pro Command, Chat, Agents, Settings, hosted push relay defaults, and realtime Talk playback wired to gateway sessions, diagnostics, chat, and realtime Talk. ([#87367](openclaw/openclaw#87367), [#88096](openclaw/openclaw#88096), [#88105](openclaw/openclaw#88105)) Thanks [@Solvely-Colin](https://github.com/Solvely-Colin) and [@ngutman](https://github.com/ngutman). - Docs: clarify Codex computer-use setup, paste-token stdin auth setup, macOS gateway sleep troubleshooting, native Codex hook relay recovery, container model auth, install deployment cards, device-token admin gating, CLI setup flow compatibility, Notte cloud browser CDP setup, and backport targets. ([#87313](openclaw/openclaw#87313), [#63050](openclaw/openclaw#63050), [#87685](openclaw/openclaw#87685)) Thanks [@bdjben](https://github.com/bdjben), [@liaoandi](https://github.com/liaoandi), and [@thewilloftheshadow](https://github.com/thewilloftheshadow). - PDF/tools: use ClawPDF for PDF extraction, support encrypted PDF extraction, and surface MCP structured content in agent tool results. ([#87670](openclaw/openclaw#87670), [#87751](openclaw/openclaw#87751)) - Providers: add Claude Opus 4.8 support, Fal Krea image model schemas, NVIDIA featured model catalogs, MiniMax streaming music responses, and provider-backed voice model catalogs. ([#87845](openclaw/openclaw#87845), [#87890](openclaw/openclaw#87890), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764), [#87794](openclaw/openclaw#87794)) Thanks [@eleqtrizit](https://github.com/eleqtrizit) and [@vincentkoc](https://github.com/vincentkoc). - Codex/GitHub: add the GitHub Copilot agent runtime and the Codex Supervisor plugin package. - Plugins: externalize GitHub Copilot and Tokenjuice as official install-on-demand plugins with npm and ClawHub publish metadata. - Workboard: add agent coordination tools for tracking and handing off active agent work. - Discord: show commentary in progress drafts so live Discord runs expose useful in-progress context. ([#85200](openclaw/openclaw#85200)) - Plugin SDK: add a reply payload sending hook for plugins that need to deliver channel-owned replies and flatten package types for SDK declarations. ([#82823](openclaw/openclaw#82823), [#87165](openclaw/openclaw#87165)) Thanks [@piersonr](https://github.com/piersonr) and [@RomneyDa](https://github.com/RomneyDa). - Policy: add policy comparison, ingress-channel conformance, and sandbox-posture conformance checks. ([#85572](openclaw/openclaw#85572), [#85744](openclaw/openclaw#85744), [#86768](openclaw/openclaw#86768)) ##### Fixes - Agents: fall back to local config pruning when the optional `agents delete` Gateway probe cannot authenticate, so offline installs can still delete agents without removing shared workspaces. - Tighten phone-control mutation authorization \[AI]. ([#87150](openclaw/openclaw#87150)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Clarify directive persistence authorization policy \[AI]. ([#86369](openclaw/openclaw#86369)) Thanks [@pgondhi987](https://github.com/pgondhi987). - Agents/Codex: keep spawned agent cwd/workspace state separated, forward ACP spawn attachments, keep hook context prompt-local, release session locks on timeout abort and runtime teardown without deleting live OpenClaw-owned locks during cleanup, avoid session event queue self-wait, clean up exec abort listeners, stream assistant deltas incrementally, recover raw missing-thread compaction failures, preserve rotated compaction session identity, keep compaction-timeout snapshots continuable, preserve shared app-server state across startup or helper failures, keep native hook relay alive across restarts and prune stale bridge files, close native hook relay replacement races, keep Claude live tool progress visible for watchdog recovery, suppress abandoned requester completion handoff, route workspace memory through tools, resolve Codex runtime models first, report quarantined dynamic tools, format `skills` command output, bind node auto-review to prepared plans, retry Claude CLI transcript probes, and bound compaction/steering retries. ([#87218](openclaw/openclaw#87218), [#86875](openclaw/openclaw#86875), [#86123](openclaw/openclaw#86123), [#88129](openclaw/openclaw#88129), [#87399](openclaw/openclaw#87399), [#87375](openclaw/openclaw#87375), [#72574](openclaw/openclaw#72574), [#87383](openclaw/openclaw#87383), [#87400](openclaw/openclaw#87400), [#83022](openclaw/openclaw#83022), [#87671](openclaw/openclaw#87671), [#87738](openclaw/openclaw#87738), [#87747](openclaw/openclaw#87747), [#87706](openclaw/openclaw#87706), [#87546](openclaw/openclaw#87546), [#87541](openclaw/openclaw#87541), [#81048](openclaw/openclaw#81048)) Thanks [@mbelinky](https://github.com/mbelinky), [@Alix-007](https://github.com/Alix-007), [@luoyanglang](https://github.com/luoyanglang), [@yetval](https://github.com/yetval), [@sjf](https://github.com/sjf), [@joshavant](https://github.com/joshavant), [@benjamin1492](https://github.com/benjamin1492), [@c19354837](https://github.com/c19354837), [@fuller-stack-dev](https://github.com/fuller-stack-dev), [@pfrederiksen](https://github.com/pfrederiksen), and [@dodge1218](https://github.com/dodge1218). - Codex Supervisor: keep real-home app-server MCP session listing on the loaded state path, bound stored history scans, and close WebSocket probes cleanly. - Channels: thread canonical session keys into outbound hooks, preserve Matrix room-id case, keep fallback tool warnings mention-inert, retain delivered Slack final replies during late cleanup, continue iMessage polling after denied reactions, suppress duplicate native exec approvals, resolve Gateway message actions against the active runtime config, preserve Telegram SecretRef prompt config and polling keepalives, preserve WhatsApp profile auth roots, QR display, document filenames, and plugin hook config, suppress Discord recovered tool warnings, preserve the Discord voice outbound helper, cap Discord/Signal/Zalo channel request and container timeouts, and block untrusted Teams service URLs while keeping TeamsSDK patterns aligned. ([#73706](openclaw/openclaw#73706), [#75670](openclaw/openclaw#75670), [#87366](openclaw/openclaw#87366), [#87451](openclaw/openclaw#87451), [#87465](openclaw/openclaw#87465), [#87334](openclaw/openclaw#87334), [#84535](openclaw/openclaw#84535), [#76262](openclaw/openclaw#76262), [#83304](openclaw/openclaw#83304), [#82492](openclaw/openclaw#82492), [#87581](openclaw/openclaw#87581), [#77114](openclaw/openclaw#77114), [#86426](openclaw/openclaw#86426), [#85529](openclaw/openclaw#85529), [#87160](openclaw/openclaw#87160)) Thanks [@zeroaltitude](https://github.com/zeroaltitude), [@lukeboyett](https://github.com/lukeboyett), [@jarvis-mns1](https://github.com/jarvis-mns1), [@xiaotian](https://github.com/xiaotian), [@funmerlin](https://github.com/funmerlin), [@joshavant](https://github.com/joshavant), [@eleqtrizit](https://github.com/eleqtrizit), [@heyitsaamir](https://github.com/heyitsaamir), [@amittell](https://github.com/amittell), [@lidge-jun](https://github.com/lidge-jun), [@liorb-mountapps](https://github.com/liorb-mountapps), [@masatohoshino](https://github.com/masatohoshino), [@bladin](https://github.com/bladin), and [@giodl73-repo](https://github.com/giodl73-repo). - CLI/auth/doctor/providers: reject malformed numeric/timeout/subcommand-version inputs, ignore workspace dotenv provider credentials, wait for respawn child shutdown, bound heartbeat defaults plus Codex, GitHub Copilot, OpenAI, Anthropic, Google, Feishu, LM Studio, MiniMax, Xiaomi TTS, and local-provider OAuth/token/model requests, harden Codex auth probes, label auth health by agent, preserve explicit agentRuntime pins during Codex model migration, warm provider auth off the main thread, honor Codex response timeouts, stop migrating current Claude Haiku 4.5 profiles to Sonnet, bound local service startup, resolve GPT-5.5 without cached catalog, migrate legacy memory auto-provider config, rewrite non-canonical `api_key` auth profiles, and make doctor restart follow-ups actionable. ([#87398](openclaw/openclaw#87398), [#86281](openclaw/openclaw#86281), [#87361](openclaw/openclaw#87361), [#88133](openclaw/openclaw#88133), [#83655](openclaw/openclaw#83655), [#87559](openclaw/openclaw#87559), [#87719](openclaw/openclaw#87719), [#88088](openclaw/openclaw#88088), [#85924](openclaw/openclaw#85924), [#84362](openclaw/openclaw#84362)) Thanks [@Patrick-Erichsen](https://github.com/Patrick-Erichsen), [@samzong](https://github.com/samzong), [@giodl73-repo](https://github.com/giodl73-repo), [@alkor2000](https://github.com/alkor2000), [@mmaps](https://github.com/mmaps), [@nxmxbbd](https://github.com/nxmxbbd), and [@vincentkoc](https://github.com/vincentkoc). - Gateway/security/session state: expire browser tokens after auth rotation, scope assistant idempotency dedupe, drain probe client closes, avoid stale restart continuation reuse, preserve retry-after fallbacks and stale rate-limit cooldown probes, bound webchat image and artifact transcript scans, include seconds in inbound metadata timestamps, clear completed session active runs, clear stale chat stream buffers, and evict current plugin-state namespaces at row caps. ([#87810](openclaw/openclaw#87810), [#87833](openclaw/openclaw#87833), [#75089](openclaw/openclaw#75089)) Thanks [@joshavant](https://github.com/joshavant) and [@litang9](https://github.com/litang9). - Config/parsing/network: reject partial numeric parsing, parse provider/Discord retry headers and dates strictly, honor IPv6 and bare IPv6 `no_proxy` entries, preserve empty plugin allowlists, canonicalize secret target array indexes, and reject malformed media content lengths, inspected TCP ports, marketplace content lengths, cron epochs, sandbox stat fields, unsafe duration values, empty config path segments, noncanonical schema array refs, unsafe Telegram callback pages, and invalid Teams attachment-fetch DNS targets. ([#87883](openclaw/openclaw#87883)) Thanks [@zhangguiping-xydt](https://github.com/zhangguiping-xydt). - Browser/input hardening: reject invalid tab indexes, excessive viewport resizes, explicit zero CDP ports, malformed geolocation options, unsafe screenshot or permission-grant timeouts, loose response-body limits, invalid cookie expiries, and non-finite Browser tool delays/timeouts. - Cron/automation: retry recurring jobs after transient model rate limits before waiting for the next scheduled slot, and preflight model fallbacks before skipping scheduled work. ([#82887](openclaw/openclaw#82887)) Thanks [@chen-zhang-cs-code](https://github.com/chen-zhang-cs-code). - Auto-reply/directives: respect provider and relayed channel metadata during directive persistence so channel-originated decisions keep their intended context. ([#87683](openclaw/openclaw#87683)) - WhatsApp: resolve the auth directory from the active profile so profile-scoped WhatsApp installs do not drift to the wrong credential root. ([#82492](openclaw/openclaw#82492)) Thanks [@lidge-jun](https://github.com/lidge-jun). - Gateway/session state: clear completed session active runs, avoid cold-loading providers for MCP inventory, cache single-session child indexes, cap handshake timers, and bound preauth, auth-guard, media, transcript, readiness, and port options. - Channels/replies: preserve channel-owned progress callbacks when verbose output is off, keep group-room progress suppression intact, prefer external session delivery context, escape Discord component id delimiters, force final TUI chat repaints, show Slack reasoning previews, and normalize Discord/Matrix/Mattermost channel numeric options. ([#87476](openclaw/openclaw#87476), [#87423](openclaw/openclaw#87423)) - Agents/tool args: harden smart-quoted argument repair for edit arrays and exact escaped arguments so model-produced tool calls recover without corrupting valid input. ([#86611](openclaw/openclaw#86611)) Thanks [@ferminquant](https://github.com/ferminquant). - Providers/agents: preserve seeded Anthropic signatures, preserve signed thinking payloads, concatenate signature-delta chunks, preserve DeepSeek `reasoning_content` replay across tier suffixes, apply OpenRouter strict9 ids to Mistral routes, promote Ollama plain-text tool calls, load NVIDIA featured model catalogs, stream MiniMax music generation responses, and recover empty preflight compaction. ([#87593](openclaw/openclaw#87593), [#87493](openclaw/openclaw#87493), [#80775](openclaw/openclaw#80775), [#84764](openclaw/openclaw#84764)) Thanks [@Pluviobyte](https://github.com/Pluviobyte) and [@eleqtrizit](https://github.com/eleqtrizit). - Media/images: skip CLI image cache refs when resolving generated images, allow trusted generated HTML attachments, and bound generated video downloads so stale refs and slow providers fail cleanly. ([#87523](openclaw/openclaw#87523), [#87982](openclaw/openclaw#87982)) - File transfer: handle late tar stdin pipe errors after archive validation or unpacking has already settled. - Performance: trust install-record caches between reloads, prefer native JSON parsing, reuse unchanged tool-search catalogs, reuse gateway session and plugin metadata paths, skip unchanged store serialization, patch single-entry session writes, add precomputed session patch writers, reduce store clone allocations, cache manifest model catalog rows and auto-enabled plugin config, avoid full session snapshots for entry reads, defer configured Slack full startup, prefer bundled plugin dist entries, and slim current metadata identity caches. ([#87760](openclaw/openclaw#87760)) - Docker/release/QA: package runtime workspace templates, stream cross-OS served artifacts, preserve sparse Crabbox run artifacts, isolate npm plugin installs per package, reject incompatible package plugin API installs, drop the leftover root Sharp dependency from package manifests after the Rastermill migration, bound OpenClaw instance logs, plugin gauntlet relay logs, MCP channel buffers, kitchen-sink scans, agent-turn assertions, QA-Lab credential broker calls, QA Matrix substrate requests, and release scenario logs, and keep release/google live guards current. ([#87647](openclaw/openclaw#87647), [#87477](openclaw/openclaw#87477)) Thanks [@rohitjavvadi](https://github.com/rohitjavvadi) and [@vincentkoc](https://github.com/vincentkoc). - Release/CI: bound manual git fetches, ClawHub verifier responses, ClawHub owner metadata, dependency-guard error bodies, Parallels limits, startup/test/memory budget parsing, and diffs viewer build warnings so release lanes fail with useful proof instead of hanging. ([#87839](openclaw/openclaw#87839)) </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/759

fix(doctor): make restart follow-up actionable

14159d2

openclaw-barnacle Bot added agents Agent runtime and tooling size: XS maintainer Maintainer-authored PR labels May 27, 2026

clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. labels May 27, 2026

steipete self-assigned this May 27, 2026

steipete approved these changes May 27, 2026

View reviewed changes

steipete merged commit f3e2851 into main May 27, 2026
177 of 188 checks passed

steipete deleted the fix-69755-doctor-hint-actionable branch May 27, 2026 22:23

steipete mentioned this pull request May 27, 2026

Post-restart suggestion to run openclaw doctor --non-interactive is not always actionable in heartbeat / approval-constrained contexts #69755

Closed

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

📡 Upstream Digest — 2026-05-27 23:10 UTC curtismercier/openclaw-mods#960

Open

steipete pushed a commit that referenced this pull request May 27, 2026

fix(doctor): make restart follow-up actionable (#87361)

46c5d74

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 28, 2026

fix(doctor): make restart follow-up actionable (openclaw#87361)

b4956b6

SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026

fix(doctor): make restart follow-up actionable (openclaw#87361)

811a839

sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026

fix(doctor): make restart follow-up actionable (openclaw#87361)

e91d41a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(doctor): make restart follow-up actionable#87361

fix(doctor): make restart follow-up actionable#87361
steipete merged 1 commit into
mainfrom
fix-69755-doctor-hint-actionable

giodl73-repo commented May 27, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 27, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 27, 2026 •

edited

Loading

Uh oh!

giodl73-repo commented May 27, 2026

Uh oh!

clawsweeper Bot commented May 27, 2026 •

edited

Loading

Uh oh!

steipete left a comment

Uh oh!

Uh oh!

steipete commented May 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

giodl73-repo commented May 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Behavior addressed

Real environment tested

Exact steps or command run after this patch

Evidence after fix

Observed result after fix

What was not tested

Uh oh!

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

Uh oh!

Uh oh!

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

Uh oh!

Hatch command

Uh oh!

giodl73-repo commented May 27, 2026

Uh oh!

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

Uh oh!

Uh oh!

steipete left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

steipete commented May 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

giodl73-repo commented May 27, 2026 •

edited

Loading

clawsweeper Bot commented May 27, 2026 •

edited

Loading

clawsweeper Bot commented May 27, 2026 •

edited

Loading

clawsweeper Bot commented May 27, 2026 •

edited

Loading