-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
tools.exec.approvalRunningNoticeMs is documented but rejected by the 2026.3.24 config schema #57270
Copy link
Copy link
Open
BingqingLyu/openclaw
#1652Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.staleMarked as stale due to inactivityMarked as stale due to inactivity
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.staleMarked as stale due to inactivityMarked as stale due to inactivity
Type
Fields
Priority
None yet
Summary
The OpenClaw docs describe
tools.exec.approvalRunningNoticeMsas a supported config key for approval-gated exec follow-up notices, but the local2026.3.24 (cff6dc9)build rejects that field as an unrecognized key undertools.exec.This looks like a docs/schema mismatch rather than an operator mistake.
Version
2026.3.24 (cff6dc9)Docs claim
The exec tool docs currently state:
Relevant doc path:
docs/tools/exec.mdReproduction
tools.exec:{ "tools": { "exec": { "host": "gateway", "security": "allowlist", "ask": "on-miss", "approvalRunningNoticeMs": 3000 } } }The command fails during config load before it can return the
tools.execobject.Actual result
The runtime rejects the config with:
This is a hard config failure, not a warning.
Expected result
One of these should be true:
tools.exec.approvalRunningNoticeMs, orWhy this matters
This is easy to trip over because the documented knob is directly relevant when testing approval-gated exec follow-up behavior. Operators trying to tune the running-notice delay will hit a hard config failure even though they are following the docs.
Suspected root cause
The docs and runtime schema appear to have drifted. The key may have moved, been removed, or only exist in another schema surface, but the current
tools.execconfig validator on2026.3.24does not accept it.Suggested fix
Either:
tools.exec.approvalRunningNoticeMs, orCurrent operator workaround
Do not set
tools.exec.approvalRunningNoticeMson2026.3.24. The approval flow can still be tested and used successfully, but this specific knob must be treated as unsupported on this build until the docs and schema match again.Local verification already performed
approvalRunningNoticeMsspecifically fails at config-parse time when placed undertools.execExtra note
This issue is independent of the earlier approval follow-up heartbeat fix. The approval path itself can work correctly while this specific docs-described knob still fails schema validation.