enhancement: add model provider options

[urmauur]

Describe Your Changes

Fixes Issues

• Closes #
• Closes #

Self Checklist

• Added relevant comments, esp in complex areas
• Updated docs (for bug fixes / features)
• Created issues for follow-up changes or refactoring needed

[Copilot]

Pull request overview

Adds a UI entry point in the Settings sidebar to create new (OpenAI-compatible) model providers, so users can add providers without navigating to the dedicated Model Providers page.

Changes:

• Adds an “Add Provider” action to the Settings menu’s Model Providers section via AddProviderDialog.
• Introduces createProvider logic in SettingsMenu to create and persist a new provider, then navigate to its settings page.
• Defaults the providers section expansion state to expanded.

Comments suppressed due to low confidence (1)

web-app/src/containers/SettingsMenu.tsx:46

• setTimeout(..., 0) around the navigation looks unnecessary here because addProvider (Zustand set) is synchronous. Prefer navigating immediately after addProvider (or, if a deferral is required, use a documented approach like queueMicrotask/requestAnimationFrame) to avoid leaving an untracked timer running if the component unmounts quickly.

      addProvider(newProvider)
      setTimeout(() => {
        navigate({ to: route.settings.providers, params: { providerName: name } })
      }, 0)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Barecheck - Code coverage report

Total: 22.14%

Your code coverage diff: 0.02% ▴

Uncovered files and lines

File	Lines
web-app/src/containers/SettingsMenu.tsx	31-47, 158-159, 171-177, 179-180, 182, 184, 191-199, 201-206, 208-223, 226-231, 233-234, 319

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

web-app/src/containers/tests/SettingsMenu.test.tsx:130

• The test selects the chevron via getAllByRole('button')[0], but the menu now renders multiple buttons (e.g., the new “Add Provider” button). This makes the test dependent on DOM ordering. Consider scoping the query to the Model Providers row/link and selecting the nested button from there so the test remains stable if more buttons are added later.

    // Click the chevron to collapse
    const chevronButtons = screen.getAllByRole('button')
    const chevron = chevronButtons[0]
    await user.click(chevron)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.