Improve custom provider creation experience by katzdave · Pull Request #7541 · block/goose

katzdave · 2026-02-26T18:47:04Z

Rather than showing your existing provider, the custom provider panel now shows this (i'm using random characters as an API key) immediately after custom provider creation.

Some of the providers seem to fetch the models no problem with an invalid key, but others do not; I think this warning is a decent signal that something might be wrong without the noisy full error. And then if you make a conversation, you get the error about invalid API key.

katzdave · 2026-02-26T18:47:51Z

ui/desktop/src/components/settings/providers/ProviderSettingsPage.tsx


  // This function will be passed to ProviderGrid for manual refreshes after config changes
-  const refreshProviders = useCallback(() => {
+  const refreshProviders = useCallback(async () => {


This is the biggest issue, not blocking here.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c4e4e68840

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

chatgpt-codex-connector · 2026-02-26T18:52:49Z

ui/desktop/src/components/settings/models/subcomponents/SwitchModelModal.tsx

    (async () => {
      try {
-        const providersResponse = await getProviders(false);
+        const providersResponse = await getProviders(!!initialProvider);


Avoid forcing provider refresh for preselected provider flows

Using getProviders(!!initialProvider) forces a network refresh whenever the modal opens with initialProvider set (for example, onboarding launch and configure flows). ConfigContext.getProviders(true) returns an empty array on fetch failures, so a transient /config/providers error now leaves this modal with no provider/model options even when cached providers were already available; the previous getProviders(false) path would still show cached data.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-02-26T18:52:49Z

ui/desktop/src/components/settings/providers/ProviderGrid.tsx

      const { createCustomProvider } = await import('../../../api');
-      await createCustomProvider({ body: data, throwOnError: true });
+      const result = await createCustomProvider({ body: data, throwOnError: true });
+      const providerId = result.data?.replace('Custom provider added - ID: ', '') || null;


Stop parsing provider ID from human-readable response text

This extracts the new provider ID by stripping a hard-coded phrase from a free-form success message. Because the API response type is just string, any wording change or backend/UI version skew will produce a bad providerId, and the follow-up model picker opens without the intended provider preselected. Returning a structured payload (or resolving the new provider from refreshed provider data) avoids this brittle coupling.

Useful? React with 👍 / 👎.

…d-output * origin/main: Improve custom provider creation experience (#7541) fix(scheduler): schedules added via CLI showing up in UI (#7594)

* origin/main: (27 commits) feat: return structured {stdout, stderr} from shell tool with output schema (#7604) Improve custom provider creation experience (#7541) fix(scheduler): schedules added via CLI showing up in UI (#7594) chore: openai reasoning model cleanup (#7529) chore(deps): bump hono from 4.12.1 to 4.12.3 in /evals/open-model-gym/mcp-harness (#7585) chore(deps): bump minimatch from 10.1.1 to 10.2.3 in /evals/open-model-gym/suite (#7498) chore(deps): bump swiper from 11.2.10 to 12.1.2 in /documentation (#7368) Better network failure error & antrhopic retry (#7595) feat: make the text bar persistent and add a queue for messages (#7560) fix: outdated clippy command in goosehints (#7590) chore(deps): bump hono from 4.11.7 to 4.12.1 in /evals/open-model-gym/mcp-harness (#7417) chore(deps-dev): bump ajv from 6.12.6 to 6.14.0 in /ui/desktop (#7437) chore(deps): bump ajv from 8.17.1 to 8.18.0 in /evals/open-model-gym/mcp-harness (#7491) chore(deps): bump hono from 4.12.0 to 4.12.2 in /ui/desktop (#7515) chore(deps-dev): bump rollup from 4.57.1 to 4.59.0 in /ui/desktop (#7522) chore(deps): bump minimatch in /ui/desktop (#7572) fix: validate configure probe for streaming providers (#7564) Dockerfile: add missing build/runtime dependencies (#7546) fix(claude-code): Permission routing for smart-approve (#7501) Add base_path field to custom provider config (#7558) ...

* main: (74 commits) feat: return structured {stdout, stderr} from shell tool with output schema (#7604) Improve custom provider creation experience (#7541) fix(scheduler): schedules added via CLI showing up in UI (#7594) chore: openai reasoning model cleanup (#7529) chore(deps): bump hono from 4.12.1 to 4.12.3 in /evals/open-model-gym/mcp-harness (#7585) chore(deps): bump minimatch from 10.1.1 to 10.2.3 in /evals/open-model-gym/suite (#7498) chore(deps): bump swiper from 11.2.10 to 12.1.2 in /documentation (#7368) Better network failure error & antrhopic retry (#7595) feat: make the text bar persistent and add a queue for messages (#7560) fix: outdated clippy command in goosehints (#7590) chore(deps): bump hono from 4.11.7 to 4.12.1 in /evals/open-model-gym/mcp-harness (#7417) chore(deps-dev): bump ajv from 6.12.6 to 6.14.0 in /ui/desktop (#7437) chore(deps): bump ajv from 8.17.1 to 8.18.0 in /evals/open-model-gym/mcp-harness (#7491) chore(deps): bump hono from 4.12.0 to 4.12.2 in /ui/desktop (#7515) chore(deps-dev): bump rollup from 4.57.1 to 4.59.0 in /ui/desktop (#7522) chore(deps): bump minimatch in /ui/desktop (#7572) fix: validate configure probe for streaming providers (#7564) Dockerfile: add missing build/runtime dependencies (#7546) fix(claude-code): Permission routing for smart-approve (#7501) Add base_path field to custom provider config (#7558) ...

* origin/main: feat: return structured {stdout, stderr} from shell tool with output schema (#7604) Improve custom provider creation experience (#7541) fix(scheduler): schedules added via CLI showing up in UI (#7594) chore: openai reasoning model cleanup (#7529) chore(deps): bump hono from 4.12.1 to 4.12.3 in /evals/open-model-gym/mcp-harness (#7585) chore(deps): bump minimatch from 10.1.1 to 10.2.3 in /evals/open-model-gym/suite (#7498) chore(deps): bump swiper from 11.2.10 to 12.1.2 in /documentation (#7368) Better network failure error & antrhopic retry (#7595) feat: make the text bar persistent and add a queue for messages (#7560)

* origin/main: (107 commits) Merge platform/builtin extensions (#7630) Clean up stale references to removed components (#7644) fix: scope empty session reuse to current window to prevent session mixing (#7602) fix: prevent abort in local inference (#7633) Revert git patch for llama-cpp-2 (#7642) docs: update recipe usage step to reflect auto-submit behavior (#7639) docs: add guide for customizing the sidebar (#7638) docs: update Claude Code approve behavior and model list in cli-providers guide (#7448) fix: restore provider and extensions for LRU-evicted sessions (#7616) Restore goosed logging (#7622) feat: return structured {stdout, stderr} from shell tool with output schema (#7604) Improve custom provider creation experience (#7541) fix(scheduler): schedules added via CLI showing up in UI (#7594) chore: openai reasoning model cleanup (#7529) chore(deps): bump hono from 4.12.1 to 4.12.3 in /evals/open-model-gym/mcp-harness (#7585) chore(deps): bump minimatch from 10.1.1 to 10.2.3 in /evals/open-model-gym/suite (#7498) chore(deps): bump swiper from 11.2.10 to 12.1.2 in /documentation (#7368) Better network failure error & antrhopic retry (#7595) feat: make the text bar persistent and add a queue for messages (#7560) fix: outdated clippy command in goosehints (#7590) ... # Conflicts: # Cargo.lock # Cargo.toml # crates/goose-server/src/commands/agent.rs # crates/goose-server/src/main.rs # crates/goose-server/src/routes/reply.rs

custom provider autoswap

c4e4e68

katzdave requested a review from lifeizhou-ap February 26, 2026 18:47

katzdave commented Feb 26, 2026

View reviewed changes

chatgpt-codex-connector bot reviewed Feb 26, 2026

View reviewed changes

review comment

733468d

lifeizhou-ap approved these changes Mar 2, 2026

View reviewed changes

lifeizhou-ap added this pull request to the merge queue Mar 2, 2026

Merged via the queue into main with commit 858c1b6 Mar 2, 2026
26 of 27 checks passed

lifeizhou-ap deleted the dkatz/custom-ui branch March 2, 2026 18:29

github-actions bot mentioned this pull request Mar 2, 2026

chore(release): release version 1.27.0 (minor) #7611

Merged

craigwalkeruk pushed a commit to craigwalkeruk/custom-goose that referenced this pull request Mar 5, 2026

Improve custom provider creation experience (block#7541)

1b6966f

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve custom provider creation experience#7541

Improve custom provider creation experience#7541
lifeizhou-ap merged 2 commits intomainfrom
dkatz/custom-ui

katzdave commented Feb 26, 2026

Uh oh!

katzdave Feb 26, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Feb 26, 2026

Uh oh!

chatgpt-codex-connector bot Feb 26, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

katzdave commented Feb 26, 2026

Uh oh!

katzdave Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants