Skip to content

fix(app): show reasoning selector in v2 prompt#29050

Open
Ogyeet10 wants to merge 1 commit into
anomalyco:devfrom
Ogyeet10:fix/app-v2-reasoning-selector
Open

fix(app): show reasoning selector in v2 prompt#29050
Ogyeet10 wants to merge 1 commit into
anomalyco:devfrom
Ogyeet10:fix/app-v2-reasoning-selector

Conversation

@Ogyeet10
Copy link
Copy Markdown

@Ogyeet10 Ogyeet10 commented May 24, 2026

Issue for this PR

Closes #29051

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The v2 prompt input used by non-prod desktop builds rendered the model picker but not the existing model variant selector. That made models like GPT-5.5 show as selected without a visible control for reasoning level.

This adds the same variant selector to the v2 prompt controls and reuses the existing variant list, selection, and cycle command behavior.

How did you verify your code works?

  • bun typecheck from packages/app
  • bun --cwd packages/desktop build
  • bun turbo typecheck from the push hook

Screenshots / recordings

Not included. Verified locally in the desktop dev app.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@Ogyeet10 Ogyeet10 requested a review from adamdotdevin as a code owner May 24, 2026 01:11
@github-actions github-actions Bot added needs:issue needs:compliance This means the issue will auto-close after 2 hours. labels May 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions github-actions Bot removed needs:compliance This means the issue will auto-close after 2 hours. needs:issue labels May 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

V2 prompt input hides model reasoning selector

1 participant