Add multi-AI provider profile switching for environment variable configuration by Copilot · Pull Request #72 · shuyu-labs/WebCode

Copilot · 2026-03-03T14:10:18Z

Original prompt

This section details on the original issue you should resolve

<issue_title>环境变量配置可以添加多种AI的，可以自由切换</issue_title>
<issue_description>
如果某个AI限额了可以根据配置的环境变量快速切换AI</issue_description>

<agent_instructions>完成这个需求。可以有多套环境变量进行切换</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes 环境变量配置可以添加多种AI的，可以自由切换 #70

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

xuzeyu91 · 2026-03-05T04:43:55Z

@copilot 评审一下

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Copilot · 2026-03-05T04:48:06Z

@copilot 评审一下

代码评审完成，发现两个问题并已修复（commit 696f9ce）：

问题1：删除无确认步骤
原代码点击删除图标会立即删除方案，没有二次确认。现已改为两步操作：点击删除 → 行内展示确认/取消按钮（红色警告样式），防止误删。

问题2：编辑已有方案时无法重命名
原代码方案名称输入框仅在新建时显示，编辑时用 _editingProfile.ProfileName（旧名称）保存。现已将名称输入框同时显示于编辑模式，并统一使用 _newProfileName 保存，支持编辑时重命名。

UI 改进效果（删除确认行）：

┌─────────────────────────────────────────────────────┐
│ ⚠ 确认删除此配置方案？   [确认]  [取消]              │  ← 红色背景确认行
└─────────────────────────────────────────────────────┘

方案名称输入框（新建和编辑均显示）：

┌─────────────────────────────────────────────────────┐
│ 配置方案                                              │
│ [OpenAI (可修改方案名)                             ] │
└─────────────────────────────────────────────────────┘

Copilot

Pull request overview

This PR adds support for switching between multiple AI provider environment-variable configurations per CLI tool by introducing named “profiles”, with UI controls to manage/activate them and localization updates to support the new UX.

Changes:

Added CliToolEnvProfile model + repository to persist per-tool named env-var profiles and activate/deactivate them.
Updated CliToolEnvironmentService to resolve env vars by priority: active profile → DB default → appsettings, and exposed profile CRUD APIs.
Updated EnvironmentVariableConfigModal to manage profiles (create/edit/activate/deactivate/delete) and added new localization keys.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
WebCodeCli.Domain/Domain/Model/CliToolEnvProfile.cs	New SqlSugar entity for storing per-tool env-var profiles.
WebCodeCli.Domain/Repositories/Base/CliToolEnv/ICliToolEnvProfileRepository.cs	Repository interface for profile queries and activation/deactivation.
WebCodeCli.Domain/Repositories/Base/CliToolEnv/CliToolEnvProfileRepository.cs	Repository implementation for profile persistence and activation logic.
WebCodeCli.Domain/Domain/Service/CliToolEnvironmentService.cs	Adds profile APIs and updates env-var resolution priority.
WebCodeCli/Components/EnvironmentVariableConfigModal.razor	UI for profile management and contextual editing of default vs profile configs.
WebCodeCli/Resources/Localization/zh-CN.json	Adds profile-management localization keys (source-of-truth).
WebCodeCli/Resources/Localization/en-US.json	Adds profile-management localization keys (source-of-truth).
WebCodeCli/Resources/Localization/ja-JP.json	Adds profile-management localization keys (source-of-truth).
WebCodeCli/Resources/Localization/ko-KR.json	Adds profile-management localization keys (source-of-truth).
WebCodeCli/wwwroot/Resources/Localization/zh-CN.json	Same localization keys under wwwroot (appears to be build output).
WebCodeCli/wwwroot/Resources/Localization/en-US.json	Same localization keys under wwwroot (appears to be build output).
WebCodeCli/wwwroot/Resources/Localization/ja-JP.json	Same localization keys under wwwroot (appears to be build output).
WebCodeCli/wwwroot/Resources/Localization/ko-KR.json	Same localization keys under wwwroot (appears to be build output).

Copilot · 2026-03-05T04:55:28Z

+    "saveProfile": "프로파일 저장",
+    "cancelEdit": "편집 취소"
  },
  "fileSearch": {


These localization files under wwwroot/Resources/Localization look like build outputs. The project copies Resources/Localization/*.json into wwwroot/Resources/Localization at build time (see WebCodeCli.csproj CopyLocalizationFiles), so committing edits here will either be overwritten or create drift. Suggest removing wwwroot/Resources/Localization/*.json changes from the PR and keeping WebCodeCli/Resources/Localization/*.json as the single source of truth.

Suggested change

"saveProfile": "프로파일 저장",

"cancelEdit": "편집 취소"

},

"fileSearch": {

"saveProfile": "프로파일 저장"

},

"fileSearch": {

"fileSearch": {

Copilot · 2026-03-05T04:55:28Z

+        _isLoading = true;
+        StateHasChanged();
+        try
+        {
+            await CliToolEnvironmentService.ActivateProfileAsync(_selectedTool.Id, profile.Id);


DeactivateProfiles() refreshes _profiles but doesn’t reload the default env vars into the editor. After deactivation the effective config falls back to default, but the editor may remain empty until the modal is reopened. Suggest calling GetEnvironmentVariablesAsync (or ResetToDefaultAsync depending on intent) to repopulate _envVars after successful deactivation.

Copilot · 2026-03-05T04:55:28Z

+                _logger.LogInformation("从激活方案 [{ProfileName}] 加载工具 {ToolId} 的环境变量配置", activeProfile.ProfileName, toolId);
+                var profileVars = JsonSerializer.Deserialize<Dictionary<string, string>>(activeProfile.EnvVarsJson) ?? new();
+                return profileVars
+                    .Where(kvp => !string.IsNullOrWhiteSpace(kvp.Value))
+                    .ToDictionary(kvp => kvp.Key, kvp => kvp.Value);


If activeProfile.EnvVarsJson is invalid JSON, JsonSerializer.Deserialize will throw and the outer catch returns an empty dictionary, skipping the intended fallback to DB default/appsettings. Suggest catching JsonException around the profile deserialization (log + ignore profile) and then continuing to the DB/appsettings resolution path.

Suggested change

_logger.LogInformation("从激活方案 [{ProfileName}] 加载工具 {ToolId} 的环境变量配置", activeProfile.ProfileName, toolId);

var profileVars = JsonSerializer.Deserialize<Dictionary<string, string>>(activeProfile.EnvVarsJson) ?? new();

return profileVars

.Where(kvp => !string.IsNullOrWhiteSpace(kvp.Value))

.ToDictionary(kvp => kvp.Key, kvp => kvp.Value);

try

{

_logger.LogInformation("从激活方案 [{ProfileName}] 加载工具 {ToolId} 的环境变量配置", activeProfile.ProfileName, toolId);

var profileVars = JsonSerializer.Deserialize<Dictionary<string, string>>(activeProfile.EnvVarsJson) ?? new();

return profileVars

.Where(kvp => !string.IsNullOrWhiteSpace(kvp.Value))

.ToDictionary(kvp => kvp.Key, kvp => kvp.Value);

}

catch (JsonException jsonEx)

{

_logger.LogWarning(jsonEx,

"激活方案 [{ProfileName}] 的环境变量 JSON 无效，忽略该方案并回退到数据库和配置文件。ToolId: {ToolId}",

activeProfile.ProfileName,

toolId);

}

Copilot · 2026-03-05T04:55:29Z

+                                                                    class="px-2 py-1 text-xs font-semibold text-green-700 bg-green-50 border border-green-200 rounded-md hover:bg-green-100 transition-all"
+                                                                    title="@T("envConfig.activateProfile")">
+                                                                @T("envConfig.activateProfile")
+                                                            </button>
+                                                        }


Delete action doesn’t prompt for confirmation even though envConfig.deleteProfileConfirm is added to localization. Consider asking for confirmation (e.g., via JS confirm or an existing modal pattern) before calling DeleteProfile(profile) to avoid accidental profile deletion.

Copilot · 2026-03-05T04:55:29Z

+    "saveProfile": "Save Profile",
+    "cancelEdit": "Cancel Edit"


These localization files under wwwroot/Resources/Localization look like build outputs. The project copies Resources/Localization/*.json into wwwroot/Resources/Localization at build time (see WebCodeCli.csproj CopyLocalizationFiles), so committing edits here will either be overwritten or create drift. Suggest removing wwwroot/Resources/Localization/*.json changes from the PR and keeping WebCodeCli/Resources/Localization/*.json as the single source of truth.

Suggested change

"saveProfile": "Save Profile",

"cancelEdit": "Cancel Edit"

"saveProfile": "Save Profile"

Copilot · 2026-03-05T04:55:30Z

+    /// 删除指定方案
+    /// </summary>
+    public async Task<bool> DeleteProfileAsync(int profileId)
+    {
+        try
+        {
+            return await DeleteAsync(x => x.Id == profileId);
+        }
+        catch (Exception ex)
+        {
+            _logger.LogError(ex, "删除方案 {ProfileId} 失败", profileId);


DeleteProfileAsync(int profileId) deletes by ID only. Since higher-level APIs are tool-scoped, consider deleting with an additional ToolId predicate (or first loading and validating ownership) to avoid cross-tool deletion if a wrong ID is ever passed.

Suggested change

/// 删除指定方案

/// </summary>

public async Task<bool> DeleteProfileAsync(int profileId)

{

try

{

return await DeleteAsync(x => x.Id == profileId);

}

catch (Exception ex)

{

_logger.LogError(ex, "删除方案 {ProfileId} 失败", profileId);

/// 删除指定工具下的指定方案

/// </summary>

public async Task<bool> DeleteProfileAsync(string toolId, int profileId)

{

try

{

return await DeleteAsync(x => x.Id == profileId && x.ToolId == toolId);

}

catch (Exception ex)

{

_logger.LogError(ex, "删除工具 {ToolId} 的方案 {ProfileId} 失败", toolId, profileId);

Copilot · 2026-03-05T04:55:30Z

+        _newProfileName = profile.ProfileName;
+
+        try
+        {
+            if (!string.IsNullOrWhiteSpace(profile.EnvVarsJson))


CancelProfileEdit() clears _envVars, so after canceling (or after saving a profile, since Save() calls CancelProfileEdit()), the UI falls back to “editing default config” but shows an empty env-var list even when a default config exists. Suggest reloading the default env vars when exiting profile edit (and only clearing when a profile is active, matching ShowAsync).

Copilot · 2026-03-05T04:55:30Z

+    "saveProfile": "プロファイルを保存",
+    "cancelEdit": "編集をキャンセル"


These localization files under wwwroot/Resources/Localization look like build outputs. The project copies Resources/Localization/*.json into wwwroot/Resources/Localization at build time (see WebCodeCli.csproj CopyLocalizationFiles), so committing edits here will either be overwritten or create drift. Suggest removing wwwroot/Resources/Localization/*.json changes from the PR and keeping WebCodeCli/Resources/Localization/*.json as the single source of truth.

Suggested change

"saveProfile": "プロファイルを保存",

"cancelEdit": "編集をキャンセル"

"saveProfile": "プロファイルを保存"

Copilot · 2026-03-05T04:55:31Z

+        {
+            await CliToolEnvironmentService.DeleteProfileAsync(_selectedTool.Id, profile.Id);
+            _profiles = await CliToolEnvironmentService.GetProfilesAsync(_selectedTool.Id);


The fallback profile name "未命名方案" is hard-coded and not localized. Suggest either requiring a non-empty profile name (validation + disable save) or using a localized string for the default name so all languages behave consistently.

Copilot · 2026-03-05T04:55:31Z

+    {
+        try
+        {
+            var result = await _profileRepository.DeleteProfileAsync(profileId);


DeleteProfileAsync(string toolId, int profileId) ignores toolId and deletes purely by profileId. To prevent accidental cross-tool deletion (and to make the toolId parameter meaningful), verify the profile belongs to toolId before deleting or push the (toolId, profileId) constraint into the repository delete query.

Suggested change

var result = await _profileRepository.DeleteProfileAsync(profileId);

var result = await _profileRepository.DeleteProfileAsync(toolId, profileId);

xuzeyu91 · 2026-03-11T12:25:05Z

@copilot 还有问题吗可否合入

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Copilot · 2026-03-11T12:36:17Z

@copilot 还有问题吗可否合入

代码层面的阻塞我这边已经处理完了，最新提交是 8d7b3de。

当前状态：

本地 dotnet build WebCodeCli/WebCodeCli.csproj 通过
CodeQL 扫描结果：0 alerts
之前评审里提到的几个实质问题已修复：激活/取消激活后的编辑区状态同步、无效 JSON 回退、按 toolId 约束删除、激活过程加事务与目标校验、wwwroot 本地化生成文件不再作为源修改

是否可合入：

从代码角度看可以了
建议等当前分支最新 CI 绿灯后再点合入。我查到最近列表里旧提交有成功记录，但最新 head 还没看到对应完成状态。

界面截图参考：
https://github.com/user-attachments/assets/f1f4300f-4be6-4ff1-a2b9-a3898d1b2afc

Initial plan

aa72cf1

Copilot AI assigned Copilot and xuzeyu91 Mar 3, 2026

Copilot started work on behalf of xuzeyu91 March 3, 2026 14:10 View session

Add multi-AI environment variable profile switching

48c36d0

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Add support for multiple AI environment variable configurations~~ Add multi-AI provider profile switching for environment variable configuration Mar 3, 2026

Copilot finished work on behalf of xuzeyu91 March 3, 2026 14:27

Copilot started work on behalf of xuzeyu91 March 5, 2026 04:44 View session

xuzeyu91 marked this pull request as ready for review March 5, 2026 04:47

Copilot AI review requested due to automatic review settings March 5, 2026 04:47

Copilot started reviewing on behalf of xuzeyu91 March 5, 2026 04:47 View session

Fix delete confirmation and enable profile rename during edit

696f9ce

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Copilot finished work on behalf of xuzeyu91 March 5, 2026 04:48

Copilot AI reviewed Mar 5, 2026

View reviewed changes

Copilot started work on behalf of xuzeyu91 March 11, 2026 12:25 View session

Copilot AI and others added 2 commits March 11, 2026 12:30

Fix profile state sync and repository safety

03a873e

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Polish final profile review fixes

8d7b3de

Co-authored-by: xuzeyu91 <26290929+xuzeyu91@users.noreply.github.com>

Copilot finished work on behalf of xuzeyu91 March 11, 2026 12:37

xuzeyu91 merged commit 7a2d5a8 into main Mar 11, 2026

	"saveProfile": "Save Profile",
	"cancelEdit": "Cancel Edit"
	"saveProfile": "Save Profile"

		"saveProfile": "プロファイルを保存",
		"cancelEdit": "編集をキャンセル"

	"saveProfile": "プロファイルを保存",
	"cancelEdit": "編集をキャンセル"
	"saveProfile": "プロファイルを保存"

	var result = await _profileRepository.DeleteProfileAsync(profileId);
	var result = await _profileRepository.DeleteProfileAsync(toolId, profileId);

Conversation

Copilot AI commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Comments on the Issue (you are @copilot in this section)

Uh oh!

xuzeyu91 commented Mar 5, 2026

Uh oh!

Copilot AI commented Mar 5, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

xuzeyu91 commented Mar 11, 2026

Uh oh!

Copilot AI commented Mar 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Mar 3, 2026 •

edited

Loading