Fix: Bug: Empty space shown for Model field when saved provider no longer exists

[hbhalodia]

What?

Closes #551

Why?

• PR removes the extra empty space, when provider and model are selected, and then provider is removed, it shows blank space in model selection area in settings.

How?

• PR checks for current providers fetched and what is stored in settings. If current providers does not have that data, the field would not render.

Use of AI Tools

AI assistance: Yes
Tool(s): GitHub Copilot, Claude
Model(s): Claude Opus 4.6
Used for: Used for issue creation and PR description, plus review the approach.

Testing Instructions

1. Open Developer Settings for any feature.
2. Select a provider from the Provider dropdown.
3. Select a model from the Model dropdown.
4. Remove the selected provider (e.g., deactivate/delete the provider integration).
5. Return to Developer Settings for the same feature.
6. You would see there is no extra spacing.

Screenshots or screencast

Before	After

Changelog Entry

Fixed - Empty space shown for Model field when saved provider no longer exists in developer settings.

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: hbhalodia <hbhalodia@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>
Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.20%. Comparing base (94f5d40) to head (80b31a0).

Additional details and impacted files

@@            Coverage Diff             @@
##             develop     #552   +/-   ##
==========================================
  Coverage      71.20%   71.20%           
  Complexity      1150     1150           
==========================================
  Files             67       67           
  Lines           5563     5563           
==========================================
  Hits            3961     3961           
  Misses          1602     1602           

Flag	Coverage Δ
unit	71.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[dkotter]

Thanks for the fixing this, tested and works as expected.

The one thing worth discussing is right now this properly hides the Model dropdown in the situation where someone removes a Connector. But when we make the AI request, we still try and use that Connector even though it doesn't exist.

While you could argue that's expected, the fact that we show the Provider dropdown as being set to Default is confusing here.

I wonder if we should output an error message above that dropdown letting the user know their previous chosen provider no longer exists and they need to update that our the feature won't work.

Or maybe add a new option to the dropdown in this scenario that has the previously set provider name and some helper text, like OpenAI - note this provider no longer exists and should be reset

[jeffpaul]

I wonder if we should output an error message above that dropdown letting the user know their previous chosen provider no longer exists and they need to update that our the feature won't work.

I'd prefer this path