fix(config): preserve empty plugin allowlist by zhangguiping-xydt · Pull Request #87883 · openclaw/openclaw

zhangguiping-xydt · 2026-05-29T04:23:33Z

Summary

Problem: openclaw doctor --fix could turn an explicit empty plugins.allow into a restrictive allowlist when auto-enabling configured bundled plugins under plugins.bundledDiscovery: "compat".
Solution: Preserve empty plugins.allow as nonrestrictive while still extending non-empty restrictive allowlists.
What changed: Updated plugin auto-enable allowlist materialization and added regression coverage for explicit empty allowlists.
What did NOT change: Existing restrictive non-empty plugins.allow behavior remains unchanged.

Real behavior proof

Behavior or issue addressed: Doctor/plugin auto-enable should not convert plugins.allow: [] into a restrictive list that blocks bundled plugins under compat discovery.
Real environment tested: Local OpenClaw source checkout in /media/vdc/code/ai/aispace/openclaw-worktrees/issue-87869, using node --import tsx against the real OpenClaw applyPluginAutoEnable implementation.

Exact steps or command run after this patch:

node --import tsx --input-type=module <<'NODE'
import { applyPluginAutoEnable } from './src/config/plugin-auto-enable.apply.ts';
const result = applyPluginAutoEnable({
  config: {
    channels: { slack: { botToken: 'x' } },
    plugins: { allow: [], bundledDiscovery: 'compat' },
  },
  env: {},
});
console.log('openclaw config materialization output');
console.log('after allow:', JSON.stringify(result.config.plugins?.allow));
console.log('after slack enabled:', result.config.channels?.slack?.enabled === true);
console.log('after changes:', result.changes.join(' | '));
NODE

Evidence after fix:

openclaw config materialization output
after allow: []
after slack enabled: true
after changes: Slack configured, enabled automatically.

Observed result after fix: The explicit empty allowlist remains [], while the configured bundled Slack channel still auto-enables.
What was not tested: Did not run a full live gateway because this fix is in pure config materialization; the proof calls the real OpenClaw config function directly and the regression test covers the same invariant.

Regression Test Plan

Coverage level: Unit test
Target test file: src/config/plugin-auto-enable.core.test.ts
Scenario locked in: plugins.allow: [] with plugins.bundledDiscovery: "compat" remains empty after auto-enable.
Why this is the smallest reliable guardrail: The regression is caused by config materialization, so this test directly covers the root invariant without requiring external services.

Root Cause

Root cause: materializePluginAutoEnableCandidatesInternal called ensurePluginAllowlisted whenever plugins.allow was an array, including the explicit empty array case, which changed a nonrestrictive allowlist into a restrictive one.
Missing detection / guardrail: Existing tests covered unset allowlists and non-empty restrictive allowlists, but not explicit empty allowlists under compat bundled discovery.

clawsweeper · 2026-05-29T04:25:34Z

Codex review: passed. Reviewed May 29, 2026, 9:37 AM ET / 13:37 UTC.

Summary
The PR changes plugin auto-enable materialization so an explicit empty plugins.allow stays empty while non-empty restrictive allowlists are still extended, and adds a regression test.

PR surface: Source +3, Tests +17. Total +20 across 2 files.

Reproducibility: yes. Source inspection of current main shows an empty array reaches ensurePluginAllowlisted, and the linked report gives a concrete doctor --fix config path that matches that code.

Review metrics: 1 noteworthy metric.

Config materialization behavior: 1 changed, 0 added, 0 removed config surfaces. plugins.allow auto-enable semantics affect upgrade behavior and openclaw doctor --fix output for existing configs.

Merge readiness
Overall: 🦞 diamond lobster
Proof: 🦞 diamond lobster
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.

Risk before merge

[P1] This deliberately changes an upgrade-sensitive config materialization path, so maintainers should keep the intended contract explicit: empty plugins.allow remains nonrestrictive, while non-empty plugins.allow remains restrictive and is still extended for auto-enabled plugins.

Maintainer options:

Land the narrow compatibility repair (recommended)
Accept the intentional compatibility change because it restores the existing empty-allowlist contract and keeps non-empty restrictive allowlists unchanged.
Pause for broader config-owner confirmation
If maintainers want plugins.allow: [] to become restrictive, pause this PR and settle that policy with docs, migration, and loader behavior changes instead of merging a local repair.

Next step before merge

[P2] No repair lane is needed because the PR already contains the focused fix and regression coverage; normal checks and automerge gates can evaluate the exact head.

Security
Cleared: The diff touches only config materialization and a regression test; it does not add dependencies, scripts, workflow permissions, secrets handling, or new code-execution paths.

Review details

Best possible solution:

Land the focused config-materialization fix after normal checks, preserving the empty-allowlist nonrestrictive contract and existing restrictive allowlist extension behavior.

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

Yes. Source inspection of current main shows an empty array reaches ensurePluginAllowlisted, and the linked report gives a concrete doctor --fix config path that matches that code.

Is this the best way to solve the issue?

Yes. The patch keeps the rule local to auto-enable materialization, aligns it with adjacent empty-allowlist handling, and preserves the existing behavior for non-empty restrictive allowlists.

AGENTS.md: found and applied where relevant.

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

Label changes

Label justifications:

P1: The PR fixes a reported config-breaking doctor --fix path that can disable bundled plugin/provider workflows for users on the current release.
merge-risk: 🚨 compatibility: The diff changes upgrade-sensitive plugins.allow materialization behavior, even though the change is a narrow compatibility repair.
rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🦞 diamond lobster and patch quality is 🦞 diamond lobster.
status: 🚀 automerge armed: This PR is in ClawSweeper's automerge lane. Sufficient (terminal): The PR body includes after-fix terminal output from the real config function showing the empty allowlist preserved while Slack auto-enables.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes after-fix terminal output from the real config function showing the empty allowlist preserved while Slack auto-enables.

Evidence reviewed

PR surface:

Source +3, Tests +17. Total +20 across 2 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	1	5	2	+3
Tests	1	17	0	+17
Docs	0	0	0	0
Config	0	0	0	0
Generated	0	0	0	0
Other	0	0	0	0
Total	2	22	2	+20

What I checked:

Current-main behavior: Current main treats any array-valued plugins.allow as restrictive during plugin auto-enable and always calls ensurePluginAllowlisted, so allow: [] is populated when a candidate is auto-enabled. (src/config/plugin-auto-enable.shared.ts:1043, a19225343b45)
PR implementation: The PR commit changes the auto-enable decision to only extend plugins.allow when it is a non-empty restrictive allowlist. (src/config/plugin-auto-enable.shared.ts:1044, c06837f5dd73)
Regression coverage: The new test covers plugins.allow: [] with plugins.bundledDiscovery: "compat", verifying Slack still auto-enables while the allowlist remains empty. (src/config/plugin-auto-enable.core.test.ts:292, c06837f5dd73)
Existing empty-allowlist contract: Adjacent plugin code already treats empty allowlists as nonrestrictive by requiring allow.length > 0 before blocking or adding explicit enabled entries. (src/plugins/enable.ts:25, a19225343b45)
Feature history: Blame shows the current auto-enable allowlist materialization block was introduced in commit 6a2ccbc, which also introduced the current test file in this checkout history. (src/config/plugin-auto-enable.shared.ts:1043, 6a2ccbc9294a)
PR discussion and proof: The PR body supplies after-fix terminal output from the real applyPluginAutoEnable implementation showing after allow: [], after slack enabled: true, and the expected auto-enable change text. (c06837f5dd73)

Likely related people:

Peter Steinberger: git blame and git log --follow point to commit 6a2ccbc as introducing the current plugin-auto-enable implementation and tests in this checkout history. (role: recent area contributor; confidence: high; commits: 6a2ccbc9294a; files: src/config/plugin-auto-enable.shared.ts, src/config/plugin-auto-enable.core.test.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.

Takhoffman · 2026-05-29T11:51:34Z

@clawsweeper visualize

clawsweeper · 2026-05-29T11:51:37Z

🦞👀
ClawSweeper visual brief is being prepared.

I queued a read-only visual pass. It will create or update one marker-backed visual brief comment and will not trigger close, merge, repair, label, or branch changes.

Lens: auto

clawsweeper · 2026-05-29T11:53:10Z

Source: #87883 (comment)
Visual model: gpt-5.5, reasoning low.

Visual brief

Lens: state
Advisory only: maintainers remain the final judges.

Config state under bundledDiscovery: "compat"

Before this PR
--------------
plugins.allow: []                 🧠 meaning: nonrestrictive / allow bundled compat behavior
        |
        | doctor/plugin auto-enable sees Slack configured
        v
ensurePluginAllowlisted("slack")
        |
        v
plugins.allow: ["slack"]          🐛 meaning changed to restrictive
        |
        v
Bundled plugins outside list can be blocked ❌

Proposed behavior
-----------------
plugins.allow: []                 ✅ preserved as explicit empty / nonrestrictive
        |
        | doctor/plugin auto-enable sees Slack configured
        v
Slack enabled automatically ✅
        |
        v
plugins.allow: []                 ✅ still empty

Allowlist handling matrix

Input plugins.allow        Meaning in patch              Auto-enable behavior
-------------------        ----------------              --------------------
undefined                  no allowlist present           enable configured plugin; leave unset ✅
[]                         explicit nonrestrictive        enable configured plugin; keep [] ✅
["some-plugin"]            restrictive allowlist          add missing auto-enabled plugin ✅

Code path touched

materializePluginAutoEnableCandidatesInternal
        |
        +-- old guard:
        |      Array.isArray(allow)
        |      -> empty [] was treated like restrictive list 🐛
        |
        +-- new guard:
               Array.isArray(allow) && allow.length > 0
               -> only non-empty allowlists are extended ✅

Proof supplied

Unit regression:
  src/config/plugin-auto-enable.core.test.ts
  - config.plugins.allow = []
  - config.plugins.bundledDiscovery = "compat"
  - Slack bot token configured
  - expects Slack enabled ✅
  - expects plugins.allow remains [] ✅

Runtime-style proof in PR body:
  applyPluginAutoEnable(...)
  after allow: []
  after slack enabled: true
  changes: Slack configured, enabled automatically. ✅

Maintainer judgment points

🧑‍⚖️ Scope:
  XS patch, 2 files, 22 additions / 2 deletions.

🧑‍⚖️ Compatibility:
  Non-empty restrictive allowlists still get extended.
  Empty allowlists are now preserved as nonrestrictive.

⚠️ Remaining boundary:
  Proof is config-materialization level, not a full live gateway restart.
  That appears aligned with the claimed root cause, but maintainer judgment remains needed.

Legend: ✅ expected/preserved/proven; ❌ broken/failing path; ⚠️ unresolved concern/tradeoff; 🐛 confirmed bug path; 🧠 runtime/config state; 🧑‍⚖️ maintainer judgment point.

Maintainer ruling

Benefit: Preserves plugins.allow: [] semantics while still auto-enabling configured bundled plugins under compat discovery.
Risk: Low apparent code risk; main residual risk is whether live gateway loading has any separate allowlist gate not covered by this unit-level proof.
Proof needed: Existing supplied proof covers the config materialization invariant; optional live gateway confirmation would only broaden confidence.
Recommended next action: Maintainer review of whether empty plugins.allow is intended to remain nonrestrictive across all plugin loading paths.
Question presented: Should plugins.allow: [] continue to mean nonrestrictive when plugins.bundledDiscovery: "compat" is active?

Takhoffman · 2026-05-29T12:42:45Z

@clawsweeper automerge

clawsweeper · 2026-05-29T12:42:48Z

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=c06837f5dd73626b95008752462337ec76ce226e)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-29T13:37:51Z
Merge commit: b3c7ef6e62f5

What merged:

The PR changes plugin auto-enable materialization so an explicit empty plugins.allow stays empty while non-empty restrictive allowlists are still extended, and adds a regression test.
PR surface: Source +3, Tests +17. Total +20 across 2 files.
Reproducibility: yes. Source inspection of current main shows an empty array reaches ensurePluginAllowlisted, and the linked report gives a concrete doctor --fix config path that matches that code.

Automerge notes:

No ClawSweeper repair was needed after automerge opt-in.

The automerge loop is complete.

Automerge progress:

2026-05-29 13:31:49 UTC review queued c06837f5dd73 (queued)

2026-05-29 13:37:31 UTC review passed c06837f5dd73 (structured ClawSweeper verdict: pass (sha=c06837f5dd73626b95008752462337ec76ce2...)

2026-05-29 12:46:38 UTC repair queued c06837f5dd73 (autonomous) Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006

2026-05-29 12:48:00 UTC repair started (running) in 0s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 automerge-openclaw-openclaw-87883

2026-05-29 12:48:18 UTC validation plan (passed) in 18s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 pnpm check:changed; pnpm lint; pnpm check:test-types

2026-05-29 12:48:35 UTC Codex write preflight (passed) in 35s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 danger-full-access

2026-05-29 12:49:03 UTC Codex edit 1 c06837f5dd73 (running) in 1m 3s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 repairing branch

2026-05-29 13:09:07 UTC repair completed (no branch change) in 21m 7s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 Codex fix worker timed out after 1200000ms

2026-05-29 13:09:08 UTC repair finished (blocked) in 21m 8s Run: https://github.com/openclaw/clawsweeper/actions/runs/26638066006 Codex fix worker timed out after 1200000ms

2026-05-29 13:37:54 UTC merged c06837f5dd73 (merged by ClawSweeper automerge)

Summary: - The PR changes plugin auto-enable materialization so an explicit empty `plugins.allow` stays empty while non-empty restrictive allowlists are still extended, and adds a regression test. - PR surface: Source +3, Tests +17. Total +20 across 2 files. - Reproducibility: yes. Source inspection of current main shows an empty array reaches `ensurePluginAllowlisted`, and the linked report gives a concrete `doctor --fix` config path that matches that code. Automerge notes: - No ClawSweeper repair was needed after automerge opt-in. Validation: - ClawSweeper review passed for head c06837f. - Required merge gates passed before the squash merge. Prepared head SHA: c06837f Review: openclaw#87883 (comment) Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

…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

Summary: - The PR changes plugin auto-enable materialization so an explicit empty `plugins.allow` stays empty while non-empty restrictive allowlists are still extended, and adds a regression test. - PR surface: Source +3, Tests +17. Total +20 across 2 files. - Reproducibility: yes. Source inspection of current main shows an empty array reaches `ensurePluginAllowlisted`, and the linked report gives a concrete `doctor --fix` config path that matches that code. Automerge notes: - No ClawSweeper repair was needed after automerge opt-in. Validation: - ClawSweeper review passed for head c06837f. - Required merge gates passed before the squash merge. Prepared head SHA: c06837f Review: openclaw#87883 (comment) Co-authored-by: 张贵萍0668001030 <zhang.guiping@xydigit.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

fix(config): preserve empty plugin allowlist

c06837f

openclaw-barnacle Bot added size: XS proof: supplied External PR includes structured after-fix real behavior proof. labels May 29, 2026

clawsweeper Bot merged commit b3c7ef6 into openclaw:main May 29, 2026
175 of 187 checks passed

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

📡 Upstream Digest — 2026-05-29 14:44 UTC curtismercier/openclaw-mods#971

Open

Alix-007 mentioned this pull request Jun 2, 2026

fix(doctor): keep empty plugin allowlists open #87885

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(config): preserve empty plugin allowlist#87883

fix(config): preserve empty plugin allowlist#87883
clawsweeper[bot] merged 1 commit into
openclaw:mainfrom
zhangguiping-xydt:feat/issue-87869

zhangguiping-xydt commented May 29, 2026

Uh oh!

clawsweeper Bot commented May 29, 2026 •

edited

Loading

Uh oh!

Takhoffman commented May 29, 2026

Uh oh!

clawsweeper Bot commented May 29, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 29, 2026

Uh oh!

Takhoffman commented May 29, 2026

Uh oh!

clawsweeper Bot commented May 29, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

zhangguiping-xydt commented May 29, 2026

Summary

Real behavior proof

Regression Test Plan

Root Cause

Uh oh!

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

Uh oh!

Uh oh!

Takhoffman commented May 29, 2026

Uh oh!

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

Uh oh!

Uh oh!

clawsweeper Bot commented May 29, 2026

Visual brief

Maintainer ruling

Uh oh!

Takhoffman commented May 29, 2026

Uh oh!

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

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

clawsweeper Bot commented May 29, 2026 •

edited

Loading

clawsweeper Bot commented May 29, 2026 •

edited

Loading

clawsweeper Bot commented May 29, 2026 •

edited

Loading