github / github/app

When reviewing a diff an commenting, there ought to be a model selector.

Open
#3,751 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
No language data
Stars
2.1k
Forks
157
PR merge metrics
No merged PRs in 30d

Description

When reviewing a diff an commenting, there ought to be a model selector.

Often I find that, when I am submitting feedback on some code, it would be handy to switch to Auto model selection, or to a cheaper model, and need to un-maximize, switch or check what model is in use, then switch back. Many times I will also need to switch to a "smarter" model too, which runs into the same issue.

Even worse, if something is already running in the session, then I can't reliably switch models for just that one piece of feedback.

The constant back and forth this causes, and the waiting it imposes on me as a reviewer, mean that I can either stick with one smart model, stick with one cheap model, or suffer from constant delays while I wait for the agent to complete it's current task. None of those options are ideal.

Having the ability to choose which model will handle a given message, especially on the diff viewer, would solve this by allowing to instruct the use of a specific model for a given message.

The overall session ought to have a "default" model for handling messages that arrive without a specified override, as well as handling messages from other sessions.


Field Value
App version 1.1.17
OS macOS 26.6.2
Theme GitHub
Path /chat
Tenure Week 11

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in the /chat diff-review flow and trace the message composer and existing model-selection controls. Clarify how a per-message override should coexist with the session default, including messages sent while another task is running. Done means reviewers can choose a model for an individual message without leaving the diff view, while unqualified messages use the session default.

Written by the indexing model from the issue text.

Assessment

Domain
desktop
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.