-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
QA tool-defaults suite conflates Codex-native tools with OpenClaw dynamic tool parity #80319
Copy link
Copy link
Open
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.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.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.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.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Type
Fields
Priority
None yet
TLDR
Status: QA harness/mock-provider issue, not a proven broad Codex runtime tool dropout.
The original report overclaimed that Codex drops planned tool calls for most default tool fixtures. The corrected architecture is:
read,write,edit,apply_patch,exec,process,update_plan) are intentionally owned by Codex native app-server behavior, not duplicated as OpenClaw dynamic tools.openclawbridge tools and are valid parity rows.providerPlanToolCallsare fixture intent, not actual runtime transcript tool calls.Product impact if OpenClaw moved fully to Codex today: P4 as originally filed. This issue does not prove a Codex runner product bug.
QA impact: P1/P0 depending on lane. False product failures would undermine the parity gate, so PR #80323 corrects the harness and report language.
Latest Beta.5 Proof
Validated on PR #80323 at OpenClaw
v2026.5.10-beta.5:Artifact-backed results:
{ "tool-defaults-direct": { "total": 20, "passed": 20, "skipped": 0, "failed": 0 }, "openclaw-dynamic-tools-direct": { "total": 8, "passed": 8, "skipped": 0, "failed": 0 }, "tool-defaults-searchable": { "total": 20, "passed": 15, "skipped": 5, "failed": 0 }, "confidence-report": { "pass": true, "zeroUnknowns": true } }The 5 searchable skips are explicitly report-only because the mock provider does not yet model Codex searchable/deferred OpenClaw dynamic tool declarations honestly. That is a QA/mock limitation, not a product bug claim.
What Actually Was Wrong
The old
tool-defaultsinterpretation mixed three layers:Correct Fix In PR #80323
--codex-tool-loading directmode.counts.skippedinqa-suite-summary.jsonso report-only rows are not mistaken for passes.Remaining Work
Links