fix(extension): show Base URL before API Key for custom provider

[achuvyas-kv]

Problem

In the extension popup, when Custom (OpenAI-compatible) or Azure is selected, API Key appeared before Base URL. That order is awkward for custom endpoints — users typically set the server URL first, then optionally add a key.

Solution

Reordered the model selection fields so Base URL comes before API Key for providers that require a base URL.

Changes

Extension UI

• runners/extension/popup.html — moved Base URL field above API Key
• runners/extension/popup.js — updated layout comment to match new field order

Issue

N/A

Summary by CodeRabbit

• UI Improvements
  • Reordered the Base URL field in the provider configuration for a clearer setup flow.
• Release
  • Updated the extension version to 0.10.1.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 55a30e6d-6ef0-47c4-ba1e-acb2e64d9f5e

📥 Commits

Reviewing files that changed from the base of the PR and between fbc8592 and 487f5b3.

📒 Files selected for processing (3)

• runners/extension/manifest.json
• runners/extension/popup.html
• runners/extension/popup.js

---

Walkthrough

The extension version changes to 0.10.1. The hidden Base URL field moves before the API Key field in the provider configuration UI, and an accompanying comment is updated.

Changes

Extension update

Layer / File(s)	Summary
Provider configuration row ordering runners/extension/popup.html, runners/extension/popup.js	Moves the hidden Base URL field before API Key and updates the documented row order.
Extension version increment runners/extension/manifest.json	Updates the manifest version from 0.10.0 to 0.10.1.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

• KeyValueSoftwareSystems/agent-opfor#205: Related provider configuration layout unification.

Suggested reviewers: jithin23-kv

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: reordering Base URL before API Key for custom provider setup.
Description check	✅ Passed	The description covers Problem, Solution, Changes, and Issue, but it omits the How to test section from the template.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/extension-custom-provider-order

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}