Skip to content

Conversation

mark-espinoza
Copy link
Contributor

@mark-espinoza mark-espinoza commented Oct 6, 2025

Description

  • Fixed Model selection Drop Down Issue with Ask Sage Models
  • Updated Models list
  • Added Ask Sage Documentation page.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

Models and drop down has been fixed - before it was rendering a 'title' erro.
image

Added Ask Sage Docs and it's also pointing towards Continue
image

image

Tests

N/A

@mark-espinoza mark-espinoza requested a review from a team as a code owner October 6, 2025 01:19
@mark-espinoza mark-espinoza requested review from sestinj and removed request for a team October 6, 2025 01:19
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Oct 6, 2025
@mark-espinoza mark-espinoza changed the title LLM Provider - Ask Sage: Updated Model list, updated docs and fixed drop down menu option fix: LLM Provider - Ask Sage: Updated Model list, updated docs and fixed drop down menu option Oct 6, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 7 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="gui/src/pages/AddNewModel/configs/providers.ts">

<violation number="1" location="gui/src/pages/AddNewModel/configs/providers.ts:955">
With the default Ask Sage base URL moved from provider params into the form field, the Connect button stays disabled because `watch(&#39;apiBase&#39;)` reports an empty string until the user types. Move the default back into `params.apiBase` (or initialize it via `useForm({defaultValues})`) so the pre-filled value is treated as present.</violation>
</file>


Since this is your first cubic review, here's how it works:

  • cubic automatically reviews your code and comments on bugs and improvements
  • Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
  • Ask questions if you need clarification on any suggestion

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant