feat: New agent harness selector - #3821
Merged
Merged
Conversation
|
😎 This pull request was merged. |
|
React Doctor found 3 issues in 2 files · 3 warnings. 3 warnings
Reviewed by React Doctor for commit |
Contributor
|
✅ Visual changes approved by @charlesvien — baseline updated in 6 changed, 2 removed. |
2 tasks
charlesvien
marked this pull request as ready for review
July 27, 2026 06:03
charlesvien
changed the title
charlesvien
force-pushed
the
feat/shared-reasoning-selector
branch
from
July 27, 2026 07:03
a9daa86 to
dabdf26
Compare
charlesvien
force-pushed
the
feat/shared-reasoning-selector
branch
2 times, most recently
from
July 28, 2026 23:58
56da163 to
8033e36
Compare
11 updated Run: 37e0c3d2-7ca6-4506-879a-3a56e04d83e0 Co-authored-by: charlesvien <5378415+charlesvien@users.noreply.github.com>
6 updated Run: 46032938-8fca-486a-ae87-3e28a4198338 Co-authored-by: charlesvien <5378415+charlesvien@users.noreply.github.com>
charlesvien
force-pushed
the
feat/shared-reasoning-selector
branch
from
July 30, 2026 05:07
8033e36 to
1a06d38
Compare
6 updated, 2 removed Run: 91fcb151-8635-4f2d-9c3d-a1823d89c17c Co-authored-by: charlesvien <5378415+charlesvien@users.noreply.github.com>
Gilbert09
added a commit
that referenced
this pull request
Jul 30, 2026
… (port #3821) Ports the desktop "new agent harness selector" (#3821) to apps/mobile. Replaces the separate model and reasoning pills with a single merged pill that opens a bottom sheet: a Faster→Smarter preset scale (model + effort together), a fast-mode toggle (flag-gated), a 200k/1M context-window control, and an Advanced section for picking harness/model/reasoning individually with reset. Sends context_window and fast_mode on cloud task creation and sandbox warming, persists the selection, and bumps the preferences store version once so stale installs land on valid defaults. Generated-By: PostHog Code Task-Id: 2af7b99c-c87f-415b-81ef-95a656535821
2 tasks
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Model and reasoning selection was split across per-surface dropdowns with drifting labels, five duplicated effort-type unions and no way to set context window or fast mode.
https://screen.studio/share/tvIw08EP
Changes
One merged control. Model and reasoning are a single pill ("Claude Opus 5 High") on the new-task composer, channel composer and session view. It opens on a Faster ($) / Smarter ($ $$) slider with five preset notches that set model plus effort together (Claude: Sonnet 5 medium/high, Opus 5 medium/xhigh, Fable 5 max; GPT: 5.6 Terra low, then 5.6 Sol low/medium/high/xhigh). The drag is continuous, updates the pill live and snaps on release. A lightning toggle flips fast mode, with a themed hover tooltip showing "Fast Mode (ON/OFF)" and an adapter-specific "Learn more" docs link; the pill gets a yellow inset ring and lightning icon while it's on.
Advanced view. The menu morphs into per-setting submenus: Harness, Model (picker-ordered, restricted models gated), Reasoning (Default badge, "?" docs link on Ultracode), Context Window (200k/1M) and "Reset to default" (middle notch, default context, fast off). A custom off-preset combo opens straight on Advanced until reset.
Behavior.
ultracodeeffort ships on extended Claude models; context window live-toggles the 1M beta; fast mode includes Opus 5 and sits behindposthog-desktop-fast-mode(default off).context_window/fast_mode(env-forwarded to the sandbox). Needs feat(tasks): Accept context window and fast mode on runs posthog#73803 deployed first.Consolidation.
EffortLevel, labels and docs URLs live once in@posthog/shared; loops, analytics, settings stores, mobile and the agent package alias them. Loops, settings default effort, autoresearch stage pickers and agent-application config all render the same shared dropdown in list form.Slider (default view, shown with the fast-mode flag enabled):
Advanced (Reasoning submenu open):
How did you test this?
pnpm typecheckacross all 24 packages; full agent/core/ui/workspace-server suites (three consecutive green UI runs after de-flaking the menu tests); drove the real app over CDP to verify the pill, slider, fast toggle, advanced submenus, model ordering and the post-migration defaults (screenshots above).Automatic notifications