Skip to content

custom provider form minor improvements#6966

Merged
zanesq merged 1 commit intomainfrom
zane/custom-provider-form
Feb 5, 2026
Merged

custom provider form minor improvements#6966
zanesq merged 1 commit intomainfrom
zane/custom-provider-form

Conversation

@zanesq
Copy link
Collaborator

@zanesq zanesq commented Feb 4, 2026

Summary

Minor custom provider experience improvements

  • fix checkboxes not visible
  • we needed to use the native checkbox we don't actually use the radix checkbox anywhere in the codebase so its not styled properly (can come back to that later)
  • noticed when editing the previous model from a different provider was still hanging around
  • moved api key to an authentication section to help visually distinguish it
  • changed it to default unchecked with hide and show field since most local providers don't need an api key, added some helpful text about that
  • improved model default selection in the switch modal while I was in there
image image

initialProvider || currentProvider || null
);
const [model, setModel] = useState<string>(currentModel || '');
// Only use currentModel if we're not switching to a different provider
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thought this logic was worthy of a comment but can remove if people disagree

Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

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

nice - and this is far from "minor" but these are really annoying things!

@zanesq zanesq enabled auto-merge February 5, 2026 00:03
@zanesq zanesq added this pull request to the merge queue Feb 5, 2026
Merged via the queue into main with commit 5fc21ce Feb 5, 2026
16 of 17 checks passed
@zanesq zanesq deleted the zane/custom-provider-form branch February 5, 2026 00:08
tlongwell-block added a commit that referenced this pull request Feb 5, 2026
* origin/main:
  fix: detect context length errors in GCP Vertex AI provider (#6976)
  Added the ability to escape Jinja variables in recipes to include them in prompts (#6975)
  Bug Fix: bump pctx (#6967)
  fix(acp): fixtures now raise content mismatch errors (#6912)
  custom provider form minor improvements (#6966)
  Fix 'Edit In Place' feature appending instead of editing messages (#6955)
  docs: change RPI slash commands (#6963)
kuccello pushed a commit to kuccello/goose that referenced this pull request Feb 7, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 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