Skip to content

Update Models Dialog to Single List with Recently Used - #563

Merged
adamdotdevin merged 9 commits into
anomalyco:devfrom
timoclsn:update-models-dialog
Jul 1, 2025
Merged

adamdotdevin merged 9 commits into
anomalyco:devfrom
timoclsn:update-models-dialog

Conversation

@timoclsn

@timoclsn timoclsn commented Jun 30, 2025 •

Copy link
Copy Markdown
Contributor

This PR reworks the models dialog as suggested in #541:

  • No pages anymore, just a single list.
  • Models are sorted by recent usage, so the most used models are at the top.
  • Unused models should be sorted by release date.
  • Increased the displayed models to 10.
  • Dialog width is dynamically calculated by the longest model/provider name to prevent wrapping.
  • Model name is in normal text color, provider in muted text color.

Bildschirmfoto 2025-06-30 um 21 21 12

@timoclsn

timoclsn commented Jun 30, 2025 •

Copy link
Copy Markdown
Contributor Author

@adamdotdevin this is a prototype for the reworked models dialog. I think it's good enough to play with it and get a feel for whether it's the right direction.

What I didn't get working is the second sort by release date. How do I get this info in the Go part? Do you have to update that in the opencode-sdk-go so that I can use it? Right now it's sorted alphabetically by model name.

@timoclsn timoclsn mentioned this pull request Jun 30, 2025
@adamdotdevin

Copy link
Copy Markdown
Member

@timoclsn i just pushed a commit (ef73926) to the dev branch that includes ReleaseDate on the models; can you rebase and proceed? also, not sure how i feel about auto-selected second model, i get the idea, but i think it's maybe a little jarring.

@timoclsn
timoclsn force-pushed the update-models-dialog branch from e4f982c to 7660239 Compare June 30, 2025 20:56
@timoclsn
timoclsn marked this pull request as ready for review June 30, 2025 21:16
@timoclsn

Copy link
Copy Markdown
Contributor Author

@adamdotdevin I updated the PR to sort by the new ReleaseDate, and it now selects the first item. I agree that in practice it's a bit weird because you expect the first item to be selected even if you never want to select it in this case. 👍

@timoclsn

Copy link
Copy Markdown
Contributor Author

But I'm not sure I like the sorting. Seems a bit "wild" and hard to find stuff when you don't have recents, don't you think? 🤔

@adamdotdevin

Copy link
Copy Markdown
Member

But I'm not sure I like the sorting. Seems a bit "wild" and hard to find stuff when you don't have recents, don't you think? 🤔

will check first thing tomorrow morning!

@adamdotdevin

Copy link
Copy Markdown
Member

great work man, personally love the sorting; in reality, only the most recent models are very effective in these agentic coding assistant roles, so i think the cream rises with release date sorting. going to go ahead and merge, this is so much better than the current state.

@adamdotdevin
adamdotdevin merged commit 8f3d7b4 into anomalyco:dev Jul 1, 2025
@timoclsn
timoclsn deleted the update-models-dialog branch July 2, 2025 07:08
bussard76 pushed a commit to bussard76/openwork that referenced this pull request May 12, 2026
Root cause: On macOS, GUI applications don't inherit shell profile modifications (.zshrc, .bashrc). When OpenWork spawns the opencode engine, the PATH doesn't include paths like
  /opt/homebrew/bin where Homebrew-installed tools like npx reside.

  The fix (packages/desktop/src-tauri/src/paths.rs): Added a common_tool_paths() function that returns typical locations where user-installed tools are found:
  - macOS: /opt/homebrew/bin, /usr/local/bin, ~/.nvm/current/bin, ~/.volta/bin, ~/.bun/bin, ~/.cargo/bin, etc.
  - Linux: Similar paths adapted for Linux conventions
  - Windows: volta, pnpm, cargo, npm global paths

  These paths are now prepended to the PATH environment variable when spawning processes.

  To test the fix: Build and run the app, then try enabling your Playwright MCP server - it should now find npx correctly without needing to specify the full path.
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
meSingh pushed a commit to meSingh/opencode that referenced this pull request Sep 6, 2026
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.

2 participants