Back

ollama #14096

app/ui: fix browser tools not registering for gptoss models with custom names

by larryk78 · Feb 06, 2026 at 19:32 UTC · scan-a9014560e8454271

View on Github · Download JSON
High Risk (60%)

Get this automatically on every PR

Install the Axiomo GitHub App to get Signals as check runs and PR comments on every pull request.

Install App

Risk Assessment

Risk level: High (60%)

Risk Drivers

  • new_contributor: First contribution from larryk78
  • cross_concern_spread: Touches 3 functional domains: api, frontend, testing
  • api_surface_change: API surface changed in 1 file(s)

Intent

5/5 criteria met

Fix browser tools not functioning for gptoss models with custom names

Acceptance Criteria

  • ✓
    Add isGptoss check to tools capability in server/images.go

    Added isGptoss check in server/images.go

  • ✓
    Modify supportsBrowserTools() to check model family string

    Changed supportsBrowserTools() in app/ui/ui.go

  • ✓
    Implement supportsWebSearch() in frontend

    Added supportsWebSearch() in app/ui/app/src/api.ts

  • ✓
    Preserve model families array when mapping models

    Preserved families array in app/ui/app/src/api.ts

  • ✓
    Tests for supportsBrowserTools() added

    Tests in app/ui/ui_test.go

Confidence: 90.0% Source: pr description AI: openai

Contributors

larryk78 PR Author 1 commit ? New Contributor
Account Age: 5087 days
Prior PRs: 1

First-time contributor to this repository. unfamiliar with 6 files being modified.

Evidence

Evidence Completeness: 0.0%
Missing: ci_passing, tests_passing, lint_passing, security_scan_clean, coverage_maintained, build_successful

Supply Chain

None Risk
Modifies dependencies
Modifies lockfile
Modifies CI config
Modifies build scripts

Focus Files

Review 1 high-priority file(s)

app/ui/ui_test.go +121

121 lines changed; Source code

high
app/ui/app/src/api.ts +30

Source code

medium
server/images_test.go +22

Source code

medium
app/ui/ui.go +8

Source code

medium
app/ui/app/src/components/ChatForm.tsx +6

Source code

medium
server/images.go +6

Source code

medium

Triage

41

minutes to review

high

effort level

none

staleness risk

Allocate focused review time

Recommendation

NEEDS DISCUSSION 22.0% readiness

Insufficient evidence (CI/tests) to evaluate

Next Steps

Question

Why is ci_passing missing? Consider adding this check.

Question

Why is tests_passing missing? Consider adding this check.

Nitpick

First contribution - consider welcoming and providing extra context