Skip to content

fix(code-mode): sharpen exec tool description so models stop wasting turns rediscovering constraints#84368

Merged
clawsweeper[bot] merged 4 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84269
May 20, 2026
Merged

fix(code-mode): sharpen exec tool description so models stop wasting turns rediscovering constraints#84368
clawsweeper[bot] merged 4 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84269

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Makes #84269 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

  • Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Co-author credit kept:

fish notes: model gpt-5.5, reasoning high; reviewed against ae25826.

@clawsweeper clawsweeper Bot added agents Agent runtime and tooling size: XS clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. clawsweeper Tracked by ClawSweeper automation labels May 20, 2026
@clawsweeper clawsweeper Bot mentioned this pull request May 20, 2026
Closed
@openclaw-barnacle openclaw-barnacle Bot removed the proof: supplied External PR includes structured after-fix real behavior proof. label May 20, 2026
@clawsweeper

clawsweeper Bot commented May 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Codex review: passed.

Workflow note: Future ClawSweeper reviews update this same comment in place.

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.

Summary
The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.

Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mode runtime and reference docs already enforce. The linked source PR also includes a live before/after recitation path showing the model receives the changed tool description.

PR rating
Overall: 🦞 diamond lobster
Proof: 🦞 diamond lobster
Patch quality: 🦞 diamond lobster
Summary: Strong live-output proof and a narrow implementation make this an above-average, low-risk PR.

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.

Real behavior proof
Sufficient (live_output): The source PR body provides after-fix live output showing the updated exec description reaches the model, plus final-head schema readback and remote merge-ref validation.

Next step before merge
No repair lane is needed because there are no review findings and the PR is already in the automerge path with sufficient proof.

Security
Cleared: The diff changes model-visible description strings, a regression test, and changelog text only; it adds no dependency, workflow, secret, permission, or new execution path.

Review details

Best possible solution:

Land the schema-text, regression-test, and changelog update if the exact repaired head remains green, preserving the id-based catalog bridge wording.

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

Yes at source level: current main's exec schema omits constraints that the current code-mode runtime and reference docs already enforce. The linked source PR also includes a live before/after recitation path showing the model receives the changed tool description.

Is this the best way to solve the issue?

Yes. A description-only schema update with focused assertions is the narrowest maintainable fix because the runtime behavior and catalog bridge already exist.

Label justifications:

  • P3: This is a small model-visible guidance and regression-test update for an opt-in agent code-mode surface.
  • rating: 🦞 diamond lobster: Current PR rating is 🦞 diamond lobster because proof is 🦞 diamond lobster, patch quality is 🦞 diamond lobster, and Strong live-output proof and a narrow implementation make this an above-average, low-risk PR.
  • status: 🚀 automerge armed: This PR is in ClawSweeper's automerge lane. Sufficient (live_output): The source PR body provides after-fix live output showing the updated exec description reaches the model, plus final-head schema readback and remote merge-ref validation.
  • proof: sufficient: Contributor real behavior proof is sufficient. The source PR body provides after-fix live output showing the updated exec description reaches the model, plus final-head schema readback and remote merge-ref validation.

What I checked:

  • Current main omits the constraints: On current main, the exec tool description only tells models to use ALL_TOOLS and tools.search/describe/call, without saying that module access is disabled, that only JavaScript/TypeScript are accepted, or that describe/call should use catalog ids. (src/agents/code-mode.ts:829, ff5354ee4fbd)
  • PR head adds model-visible guidance: The PR head expands the exec description and parameter descriptions with the JavaScript/TypeScript, no Node module, and id-based catalog bridge constraints. (src/agents/code-mode.ts:829, 8ff85071ce23)
  • Regression test covers schema text: The added test asserts that the model-visible exec schema continues to mention Node.js module unavailability, tools.search, tools.describe(entry.id), tools.call(entry.id, args), ALL_TOOLS, and the JavaScript/TypeScript language enum. (src/agents/code-mode.test.ts:248, 8ff85071ce23)
  • Runtime contract matches the new text: The current runtime rejects non-JavaScript/TypeScript language values, rejects import/require/module access, and dispatches search/describe/call through string ids. (src/agents/code-mode.ts:299, ff5354ee4fbd)
  • Docs contract matches the new text: The code-mode reference documents exec input as JavaScript/TypeScript, rejects import/require/module-loader patterns, exposes ALL_TOOLS and tools, and defines describe/call as id-based helpers. Public docs: docs/reference/code-mode.md. (docs/reference/code-mode.md:274, ff5354ee4fbd)
  • PR surface is limited: The diff from current main to the repaired head changes only CHANGELOG.md, src/agents/code-mode.ts, and src/agents/code-mode.test.ts, with no workflow, dependency, lockfile, secret, or runtime execution-path change. (8ff85071ce23)

Likely related people:

  • fuller-stack-dev: git blame and git log point the current code-mode exec schema, runtime, tests, and docs to the code-mode introduction commit authored by FullerStackDev. (role: introduced behavior; confidence: high; commits: 896fd13b1c61; files: src/agents/code-mode.ts, src/agents/code-mode.worker.ts, src/agents/code-mode.test.ts)
  • obviyus: git show lists Ayaan Zaidi as committer for the code-mode introduction commit that added the affected files to main. (role: merger/committer; confidence: medium; commits: 896fd13b1c61; files: src/agents/code-mode.ts, src/agents/code-mode.worker.ts, src/agents/code-mode.test.ts)

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

@clawsweeper clawsweeper Bot added rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels May 20, 2026
@clawsweeper

clawsweeper Bot commented May 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

ClawSweeper PR egg

✨ Hatched: 🥚 common Pearl Test Hopper

       /\  .---.  /\         
      /  \/     \/  \        
     /   ( -   - )   \       
    |       ._.       |      
    |   /|  ===  |\   |      
     \  \|______/|/  /       
      '._  `--'  _.'         
         '-.__.-'            
       _/|_|  |_|\_          
      /__|      |__\         
       .-----------.         
      '-------------'

Rarity: 🥚 common.
Trait: watches the merge queue.
Image traits: location proof lagoon; accessory review stamp; palette moonlit blue and soft silver; mood bright-eyed; pose guarding a tiny green check; shell smooth pearl shell; lighting tiny status-light glow; background small review tokens.
How to hatch it: once this PR reaches status: 👀 ready for maintainer look or status: 🚀 automerge armed, the PR author or a maintainer can comment @clawsweeper hatch to turn this ASCII egg into its generated creature image.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Pearl Test Hopper 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.
  • Hatchable usually means sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness.
  • 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.

@clawsweeper

clawsweeper Bot commented May 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=8ff85071ce232f542834019efb44eef8c8091e91)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-20T01:25:05Z
Merge commit: 0e2a06ae10e6

What merged:

  • The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
  • Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ... also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:

  • PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
  • PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

The automerge loop is complete.

Automerge progress:

  • 2026-05-20 00:15:12 UTC review passed ae25826f0f54 (structured ClawSweeper verdict: pass (sha=ae25826f0f542494012f1d68ab48f12c38aec...)
  • 2026-05-20 00:19:05 UTC review queued ae25826f0f54 (queued)
  • 2026-05-20 00:34:59 UTC review queued 8ff85071ce23 (after repair)
  • 2026-05-20 00:40:31 UTC review passed 8ff85071ce23 (structured ClawSweeper verdict: pass (sha=8ff85071ce232f542834019efb44eef8c8091...)
  • 2026-05-20 01:25:07 UTC merged 8ff85071ce23 (merged by ClawSweeper automerge)

Kaspre and others added 4 commits May 20, 2026 00:27
@Kaspre @clawsweeper
fix(code-mode): sharpen exec tool description so models stop wasting …
41f7537 
…turns rediscovering constraints

When tools.codeMode.enabled is true, the exec tool's description tells the
model to use tools.search/describe/call to call enabled tools, but does not
state any of the runtime's actual constraints. Observed traces show models
waste turns rediscovering them: trying language: "bash", writing require()
for I/O, and probing scope by trial and error.

This change tells the model up front:
- Node modules and require/import are NOT available
- language accepts only "javascript" or "typescript"
- the tools.search → describe → call bridge is the path to any shell,
  file, network, or external action
- which symbols are in scope (tools, ALL_TOOLS) versus what is not

No runtime behavior changes — only the tool-card description strings.
Snapshot tests do not reference either old string (verified via grep).
@Kaspre @clawsweeper
test(code-mode): cover exec tool guidance
c93886d
@clawsweeper
fix(code-mode): sharpen exec tool description so models stop wasting …
bd3dddc 
…turns rediscovering constraints
@clawsweeper
fix(code-mode): sharpen exec tool description so models stop wasting …
8ff8507 
…turns rediscovering constraints
@clawsweeper clawsweeper Bot force-pushed the clawsweeper/automerge-openclaw-openclaw-84269 branch from ae25826 to 8ff8507 Compare May 20, 2026 00:34
@clawsweeper clawsweeper Bot merged commit 0e2a06a into main May 20, 2026
98 checks passed
@clawsweeper clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-84269 branch May 20, 2026 01:25
@Haderach-Ram Haderach-Ram mentioned this pull request May 20, 2026
Open
@github-actions github-actions Bot mentioned this pull request May 20, 2026
Open
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
e715387 
…turns rediscovering constraints (#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff85071ce232f542834019efb44eef8c8091e91.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff85071ce232f542834019efb44eef8c8091e91
Review: openclaw/openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
3ec6370 
…turns rediscovering constraints (#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff85071ce232f542834019efb44eef8c8091e91.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff85071ce232f542834019efb44eef8c8091e91
Review: openclaw/openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
a0be1fb 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
9c0676d 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
3d460a3 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
6d69e60 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
galiniliev pushed a commit to galiniliev/openclaw that referenced this pull request May 25, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
01d434d 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
5aac975 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
1525757 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @Kaspre
@Takhoffman @SebTardif
fix(code-mode): sharpen exec tool description so models stop wasting …
a14b6a6 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
80e29d0 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
f9469e9 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
@clawsweeper @Kaspre @Takhoffman
fix(code-mode): sharpen exec tool description so models stop wasting …
155b494 
…turns rediscovering constraints (openclaw#84368)

Summary:
- The PR updates the code-mode exec tool description, adds regression coverage for the model-visible constraints, and records the fix in the changelog.
- Reproducibility: yes. at source level: current main's exec schema omits constraints that the current code-mo ...  also includes a live before/after recitation path showing the model receives the changed tool description.

Automerge notes:
- PR branch already contained follow-up commit before automerge: test(code-mode): cover exec tool guidance
- PR branch already contained follow-up commit before automerge: fix(code-mode): sharpen exec tool description so models stop wasting …

Validation:
- ClawSweeper review passed for head 8ff8507.
- Required merge gates passed before the squash merge.

Prepared head SHA: 8ff8507
Review: openclaw#84368 (comment)

Co-authored-by: Kaspre <kaspre@gmail.com>
Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

agents Agent runtime and tooling clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge clawsweeper Tracked by ClawSweeper automation P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. size: XS status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Kaspre