Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

fix(agent): Put most capable Claude at top - #3658

Merged
charlesvien merged 1 commit into
mainfrom
posthog-code/invert-claude-model-picker-order
Jul 22, 2026
Merged

fix(agent): Put most capable Claude at top#3658
charlesvien merged 1 commit into
mainfrom
posthog-code/invert-claude-model-picker-order

Conversation

@Twixes

@Twixes Twixes commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Human tldr: Agent picker was showing Haiku first, then Sonnet, then Fable - not the most intuitively as a hierarchy.


Problem

The Claude model picker listed families most-capable-first in the DOM. But the picker dropdown opens upward (side="top") from the composer, so the first list item renders at the far top of the popup and the last item sits nearest the trigger where the eye lands — leaving the strongest model (Fable) visually stranded away from where you look. Raised in a Slack thread (link).

Changes

Inverted MODEL_FAMILY_ORDER to least-capable-first (haiku → sonnet → opus → fable). Because the menu opens upward, this renders Fable nearest the trigger — the visual top of the menu. Updated the picker-order unit tests to match.

How did you test this?

pnpm --filter @posthog/agent test — the compareModelsForPicker and applyAvailableModelsAllowlist suites pass with the new ordering (641 tests passed).

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog from a Slack thread

Invert MODEL_FAMILY_ORDER so the family order is least-capable first. The picker
opens upward (side="top") from the composer, so items later in the list render
nearest the trigger and read as the top of the menu — this lands the most-capable
family (Fable) on top. Update the picker-order tests accordingly.

Generated-By: PostHog Code
Task-Id: ab97f429-92f1-4d95-8a32-ad67d14df18d
@trunk-io

trunk-io Bot commented Jul 21, 2026

Copy link
Copy Markdown

😎 Merged manually by @charlesvien - details.

@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 3c1407b.

@Twixes
Twixes marked this pull request as ready for review July 21, 2026 19:46
@Twixes
Twixes requested a review from a team July 21, 2026 19:46
@Twixes Twixes changed the title fix(agent): put Fable on top of the model picker fix(agent): Put most capable Claude at top Jul 21, 2026
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(agent): put Fable on top of the mode..." | Re-trigger Greptile

@charlesvien
charlesvien merged commit e179286 into main Jul 22, 2026
34 checks passed
@charlesvien
charlesvien deleted the posthog-code/invert-claude-model-picker-order branch July 22, 2026 06:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants