Skip to content

v0.3.19: openai oss models, invite & search modal fixes#881

Merged
waleedlatif1 merged 2 commits intomainfrom
staging
Aug 5, 2025
Merged

v0.3.19: openai oss models, invite & search modal fixes#881
waleedlatif1 merged 2 commits intomainfrom
staging

Conversation

@waleedlatif1
Copy link
Collaborator

* feat(models): add openai oss models

* fix tool usage control for groq
* fix(invite): fixed invite modal

* fix(search-modal): fix search modal keyboard nav
@vercel
Copy link

vercel bot commented Aug 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sim (staging) 🛑 Canceled (Inspect) Aug 5, 2025 7:25pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Skipped (Inspect) Aug 5, 2025 7:25pm

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR introduces OpenAI OSS (Open Source Software) models to the Groq provider and fixes two critical UI issues in the search and invite modals. The changes include:

Model Provider Updates: Added 6 new OpenAI OSS models to the Groq provider configuration, including gpt-oss-120b (now the default), gpt-oss-20b, gemma2-9b-it, llama-3.1-8b-instant, llama-3.3-70b-versatile, and llama-guard-4-12b. The update also includes comprehensive pricing updates across all Groq models, removes the qwen-qwq-32b model, and renames llama-4-scout to llama-4-maverick. This expansion significantly increases the model options available to users, particularly cost-effective alternatives like gpt-oss-20b at $0.01/$0.25 per 1M tokens.

Search Modal Navigation Fix: Completely refactored the search modal's keyboard navigation system by extracting complex navigation logic into a dedicated useSearchNavigation hook. The new implementation introduces a section-based navigation approach that understands different content layouts - grid sections (blocks, tools, templates) now support intuitive left/right arrow navigation for horizontal movement, while list sections (workspaces, workflows, pages, docs) maintain traditional up/down navigation. The system includes sophisticated position memory when switching between sections and proper scroll management.

Invite Modal Repair: Fixed broken invite modal functionality by refactoring form submission handling, adding proper form refs, removing the problematic sentEmails state tracking, and improving UI consistency with tooltips and better layout. The submit mechanism was changed from direct form submission to manual submission via form refs to resolve event propagation issues in the modal context.

These changes integrate well with the existing codebase architecture, maintaining the established patterns for model provider configuration, custom hook patterns for complex UI state management, and modal component structures while significantly improving user experience and expanding platform capabilities.

Confidence score: 4/5

  • This PR is safe to merge with minimal risk as the changes are well-structured feature additions and targeted bug fixes
  • Score reflects solid implementation patterns, proper TypeScript usage, and focused problem-solving without introducing breaking changes
  • Pay close attention to apps/sim/providers/models.ts for potential model availability issues and apps/sim/app/workspace/[workspaceId]/w/components/search-modal/hooks/use-search-navigation.ts for complex navigation logic

4 files reviewed, 2 comments

Edit Code Review Bot Settings | Greptile

@waleedlatif1 waleedlatif1 changed the title v0.3.19: openai oss models, invite & search model fixes v0.3.19: openai oss models, invite & search modal fixes Aug 5, 2025
@waleedlatif1 waleedlatif1 merged commit aeef2b7 into main Aug 5, 2025
8 checks passed
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.

1 participant