Skip to content

Proposal: Map llama-skein reasoning capability in model discovery #34

Closed

Description

@androidand
Proposal

Why

llama-skein now advertises a reasoning boolean per model in /v1/models
(companion change advertise-reasoning-capability). opencode's
discoverOpenAICompatibleModels defaulted discovered local models to
capabilities.reasoning=false, so reasoning models (streaming reasoning_content
first) rendered as a frozen "not responding" during the think phase.

What

  • In discoverOpenAICompatibleModels, read item.reasoning from the /v1/models
    response and set capabilities.reasoning from it. A hand-configured provider
    model capability still wins over discovery (existing-first merge preserved).

Non-goals

  • Changing reasoning rendering itself (already gated on capabilities.reasoning).
  • Client regen: discovery reads the raw /models JSON, so no generated-client
    change is required for this mapping.

Disposition (2026-09-18)

Shipped. Shipped: provider.ts maps reasoning on discovery.

Tasks

  • 1. discoverOpenAICompatibleModels: map item.reasoning →
    capabilities.reasoning (existing config wins). provider.ts.
    Validation: cd packages/opencode && bun run typecheck
  • 2. Repo validation: typecheck opencode + tui.

Unchecked items above: see Disposition in proposal.md (2026-09-18).

Plan changes

1 done

Activity

  1. androidand commented on Sep 17, 2026

    @androidand
    OwnerAuthor

    Shipped; archived 2026-09-18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions