Skip to content

fix(ollama): default unknown capabilities to tools#84075

Merged
clawsweeper[bot] merged 3 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84055
May 19, 2026
Merged

fix(ollama): default unknown capabilities to tools#84075
clawsweeper[bot] merged 3 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84055

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

Makes #84055 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.
Known failing checks:

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 2752771.

@clawsweeper clawsweeper Bot added 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. extensions: ollama P2 Normal backlog priority with limited blast radius. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. clawsweeper Tracked by ClawSweeper automation labels May 19, 2026
@clawsweeper clawsweeper Bot mentioned this pull request May 19, 2026
Closed
@openclaw-barnacle openclaw-barnacle Bot removed the proof: supplied External PR includes structured after-fix real behavior proof. label May 19, 2026
@clawsweeper

clawsweeper Bot commented May 19, 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 branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.

Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without an explicit compat.supportsTools, and the source PR shows after-fix live provider output with supportsTools: true. I did not run local tests in this read-only review.

PR rating
Overall: 🐚 platinum hermit
Proof: 🐚 platinum hermit
Patch quality: 🐚 platinum hermit
Summary: Good focused PR with sufficient live-output proof and no blocking findings; the compatibility default should stay visible to maintainers before merge.

Rank-up moves:

  • none
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.

PR egg
✨ Hatched: 🥚 common Moonlit Diff Drake

       _..------.._          
    .-'  .-.  .-.  '-.       
   /    ( * )( * )    \      
  |        .--.        |     
  |   <\   ====   />   |     
   \    '.______.'    /      
    '-._   ____   _.-'       
        `-.____.-'           
       __/|_||_|\__          
      /__.'    '.__\         
       .-----------.         
      '-------------'

Rarity: 🥚 common.
Trait: sleeps inside passing CI.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Moonlit Diff Drake 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.
  • How to hatch it: reach status: 👀 ready for maintainer look or status: 🚀 automerge armed; that 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.

Real behavior proof
Sufficient (live_output): The credited source PR includes after-patch live CLI/provider output from a local Ollama-compatible endpoint without capabilities, directly showing the changed provider metadata.

Risk before merge
Why this matters: - Merging this PR intentionally treats unknown-capabilities Ollama models as tool-capable; an Ollama-compatible server that omits capabilities but rejects tool schemas would need an explicit compat.supportsTools: false model entry.

  • Live API status showed the replacement PR in an unstable/pending check state during review, so merge should remain behind exact-head required-check gates.

Maintainer options:

  1. Merge the narrow Ollama fallback after checks settle (recommended)
    Accept the tool-capable default for unknown native Ollama models because explicit capability lists still win and the documented per-model compat.supportsTools: false opt-out remains available.
  2. Require a strict fallback policy first
    If maintainers do not want unknown Ollama-compatible endpoints treated as tool-capable by default, ask for an explicit provider policy/config design with tests for both compatibility and strict behavior.
  3. Defer to the broader Ollama work
    Pause this PR if maintainers want this fallback default reviewed together with the broader open Ollama compatibility PR at fix(ollama): propagate supportsTools, disable idle watchdog, fix thinking and keep_alive #81915.

Next step before merge
No repair PR is indicated; keep this in the automerge/check gate until exact-head checks settle and maintainers accept the Ollama compatibility default.

Security
Cleared: No concrete security or supply-chain issue found; the workflow edit narrows an existing issue-labeler job to issue events and the provider change does not touch secrets, dependencies, or code download paths.

Review details

Best possible solution:

Land the narrow provider metadata fix if maintainers accept the documented opt-out for incompatible local models, and keep broader Ollama timeout, thinking, and keep-alive behavior in separate review.

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

Yes for the metadata gap: current main builds unknown-capabilities Ollama models without an explicit compat.supportsTools, and the source PR shows after-fix live provider output with supportsTools: true. I did not run local tests in this read-only review.

Is this the best way to solve the issue?

Yes, with the compatibility caveat: buildOllamaModelDefinition is the narrow provider-owned fallback point, and the existing per-model compat.supportsTools: false contract is the right escape hatch for incompatible servers.

Label justifications:

  • P2: This is a focused Ollama provider compatibility fix with limited blast radius and colocated regression assertions.
  • merge-risk: 🚨 compatibility: The PR changes the default metadata for unknown-capabilities Ollama-compatible models and could enable tool schemas against a server that does not support them.

What I checked:

  • Current-main fallback metadata gap: On current main, buildOllamaModelDefinition returns compat with only supportsUsageInStreaming: true when capabilities === undefined, so the unknown-capabilities path does not explicitly advertise tool support. (extensions/ollama/src/provider-models.ts:257, 1c1c75df7278)
  • PR provider change: The PR changes only the unknown-capabilities branch to include supportsTools: true; explicit capability lists still use capabilities.includes("tools"). (extensions/ollama/src/provider-models.ts:257, 27527716c09e)
  • Regression assertions: The branch adds assertions that an enriched no-capabilities Ollama model and a direct no-capabilities model definition both report compat.supportsTools === true. (extensions/ollama/src/provider-models.test.ts:44, 27527716c09e)
  • Runtime tool default: The shared runtime helper already treats missing compat.supportsTools as tool-capable and disables tools only when the field is explicitly false, so this PR makes the Ollama metadata explicit rather than removing the opt-out path. (src/agents/model-tool-support.ts:1, 1c1c75df7278)
  • Existing opt-out contract: Current docs tell users to set compat.supportsTools: false for Ollama models or servers that reliably fail on tool schemas, preserving a documented compatibility escape hatch. Public docs: docs/providers/ollama.md. (docs/providers/ollama.md:650, 1c1c75df7278)
  • Workflow repair is narrow: The workflow edit adds a job-level if: github.event_name == 'issues' to label-issues, which prevents that issue-only job from running on PR events after it failed in the source automerge loop. (.github/workflows/labeler.yml:763, 27527716c09e)

Likely related people:

  • steipete: Blame and recent history show the current Ollama provider-model files and labeler workflow sections were carried into current main through recent commits on these paths. (role: recent area contributor; confidence: high; commits: b4fdd1470b70, 50a2481652b6; files: extensions/ollama/src/provider-models.ts, extensions/ollama/src/provider-models.test.ts, .github/workflows/labeler.yml)
  • ImLukeF: Recent Ollama provider-model history includes the /api/show metadata cache work adjacent to the fallback path this PR changes. (role: adjacent feature contributor; confidence: medium; commits: 3da58226bfe6; files: extensions/ollama/src/provider-models.ts, extensions/ollama/src/provider-models.test.ts)
  • BruceMacD: The model capability detection history includes the /api/show capability work that established the capability-driven model definition path this PR adjusts. (role: feature contributor; confidence: medium; commits: ac3f55504cbe; files: extensions/ollama/src/provider-models.ts, extensions/ollama/src/provider-models.test.ts)

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

@clawsweeper clawsweeper Bot merged commit 5e0850f into main May 19, 2026
172 of 185 checks passed
@clawsweeper clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-84055 branch May 19, 2026 09:51
@clawsweeper

clawsweeper Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor Author

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=27527716c09e44ecb1b0c9fa2cf0854222d2be0f)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-19T09:51:26Z
Merge commit: 5e0850fc548c

What merged:

  • The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
  • Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with supportsTools: true. I did not run local tests in this read-only review.

Automerge notes:

  • PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

The automerge loop is complete.

Automerge progress:

  • 2026-05-19 09:51:12 UTC review passed 27527716c09e (structured ClawSweeper verdict: pass (sha=27527716c09e44ecb1b0c9fa2cf0854222d2b...)
  • 2026-05-19 09:51:28 UTC merged 27527716c09e (merged by ClawSweeper automerge)

@clawsweeper clawsweeper Bot mentioned this pull request May 19, 2026
Closed
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (#84075)
242ea3f 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

Validation:
- ClawSweeper review passed for head 27527716c09e44ecb1b0c9fa2cf0854222d2be0f.
- Required merge gates passed before the squash merge.

Prepared head SHA: 27527716c09e44ecb1b0c9fa2cf0854222d2be0f
Review: openclaw/openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (#84075)
a632ac2 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

Validation:
- ClawSweeper review passed for head 27527716c09e44ecb1b0c9fa2cf0854222d2be0f.
- Required merge gates passed before the squash merge.

Prepared head SHA: 27527716c09e44ecb1b0c9fa2cf0854222d2be0f
Review: openclaw/openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
a66b853 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
145b0f2 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 24, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
aed6dd5 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (openclaw#84075)
fe2960b 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
galiniliev pushed a commit to galiniliev/openclaw that referenced this pull request May 25, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (openclaw#84075)
3875ba1 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
0e4cb26 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
b69408b 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
@clawsweeper @dutifulbob
@osolmaz @SebTardif
fix(ollama): default unknown capabilities to tools (openclaw#84075)
7d860d3 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (openclaw#84075)
2bb07e7 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (openclaw#84075)
016ea5a 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@users.noreply.github.com>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
@clawsweeper @dutifulbob @osolmaz
fix(ollama): default unknown capabilities to tools (openclaw#84075)
d85e151 
Summary:
- The branch makes unknown-capabilities Ollama model definitions explicitly tool-capable, adds regression assertions and changelog text, and guards the issue-labeler job to run only on issue events.
- Reproducibility: yes. for the metadata gap: current main builds unknown-capabilities Ollama models without a ... er-fix live provider output with `supportsTools: true`. I did not run local tests in this read-only review.

Automerge notes:
- PR branch already contained follow-up commit before automerge: fix(ollama): default unknown capabilities to tools

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

Prepared head SHA: 2752771
Review: openclaw#84075 (comment)

Co-authored-by: Bob <dutifulbob@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: osolmaz
Co-authored-by: osolmaz <2453968+osolmaz@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

clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge clawsweeper Tracked by ClawSweeper automation extensions: ollama merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. P2 Normal backlog priority with limited blast radius. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary 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

@dutifulbob