feat(web-ui): add template editor to claude config panel #167
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -558,6 +558,7 @@ export function createCodexConfigMethods(options = {}) { | |
| template = `${template.trimEnd()}\n\n${appendBlock}\n`; | ||
| } | ||
| this.configTemplateContent = template; | ||
| this.configTemplateContext = 'codex'; | ||
| this.showConfigTemplateModal = true; | ||
| } catch (e) { | ||
| this.showMessage('加载模板失败', 'error'); | ||
|
|
@@ -807,9 +808,16 @@ export function createCodexConfigMethods(options = {}) { | |
| const performApply = async () => { | ||
| this.configTemplateApplying = true; | ||
| try { | ||
| let res; | ||
| if (this.configTemplateContext === 'claude') { | ||
| res = await api('apply-claude-settings-raw', { | ||
| content: this.configTemplateContent | ||
| }); | ||
| } else { | ||
| res = await api('apply-config-template', { | ||
| template: this.configTemplateContent | ||
| }); | ||
| } | ||
|
Comment on lines
+811
to
+820
Contributor
There was a problem hiding this comment. Claude apply path is context-aware, but confirmation preview gating is not. At Line 812 you route apply correctly, but Suggested frontend-safe fix (bypass codex diff gate for Claude context) async applyConfigTemplate() {
if (this.configTemplateApplying) {
return;
}
if (!this.configTemplateContent || !this.configTemplateContent.trim()) {
this.showMessage('模板不能为空', 'error');
return;
}
+ const isClaudeTemplate = this.configTemplateContext === 'claude';
// Default to two-step confirmation when the setting is unset.
// (The normalize helper lives in session-actions; keep a safe fallback here.)
- const shouldUseTwoStepConfirm = normalizeConfigTemplateDiffConfirmEnabled(this.configTemplateDiffConfirmEnabled);
+ const shouldUseTwoStepConfirm = !isClaudeTemplate
+ && normalizeConfigTemplateDiffConfirmEnabled(this.configTemplateDiffConfirmEnabled);
const performApply = async () => {
this.configTemplateApplying = true;
try {
let res;
if (this.configTemplateContext === 'claude') {
res = await api('apply-claude-settings-raw', {
content: this.configTemplateContent
});
} else {
res = await api('apply-config-template', {
template: this.configTemplateContent
});
}🤖 Prompt for AI Agents |
||
| if (res.error) { | ||
| this.showMessage(res.error, 'error'); | ||
| return; | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -171,3 +171,29 @@ | |
| </div> | ||
| </div> | ||
| </div> | ||
|
|
||
| <!-- Codex 轮询池控制模态框 --> | ||
| <div v-if="showCodexBridgePoolModal" class="modal-overlay" @click.self="showCodexBridgePoolModal = false"> | ||
| <div class="modal modal-bridge-pool" role="dialog" aria-modal="true" aria-labelledby="codex-bridge-pool-modal-title"> | ||
| <div class="modal-title" id="codex-bridge-pool-modal-title"> | ||
| <svg class="modal-title-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="18" height="18"><circle cx="6" cy="6" r="2"/><circle cx="18" cy="6" r="2"/><circle cx="12" cy="18" r="2"/><path d="M6 8v4h6v4"/><path d="M18 8v4h-6v4"/></svg> | ||
| 轮询池设置 | ||
| </div> | ||
| <div class="bridge-pool-modal-hint">勾选参与负载均衡的提供商</div> | ||
| <div v-if="localBridgeCandidateProviders().length === 0" class="bridge-pool-empty"> | ||
| <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.5" width="16" height="16"><path d="M13 16h-1v-4h-1m1-4h.01M21 12a9 9 0 1 1-18 0 9 9 0 0 1 18 0z"/></svg> | ||
| <span>暂无可用上游 provider，请先添加直连 provider</span> | ||
| </div> | ||
| <div v-else class="bridge-pool-list"> | ||
| <label v-for="cp in localBridgeCandidateProviders()" :key="cp.name" class="bridge-pool-item"> | ||
| <span class="bridge-pool-item-name">{{ cp.name }}</span> | ||
| <span class="bridge-pool-item-status" :class="{ active: !isLocalBridgeExcluded(cp.name) }">{{ isLocalBridgeExcluded(cp.name) ? '未启用' : '已启用' }}</span> | ||
| <input type="checkbox" :checked="!isLocalBridgeExcluded(cp.name)" @change="toggleLocalBridgeExcluded(cp.name)" /> | ||
| </label> | ||
| </div> | ||
| <div class="btn-group"> | ||
| <button class="btn btn-confirm" @click="showCodexBridgePoolModal = false">{{ t('common.close') }}</button> | ||
| </div> | ||
| </div> | ||
| </div> | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -82,6 +82,7 @@ | |
| :placeholder="t('claude.model.placeholder')" | ||
| > | ||
| <div class="config-template-hint">{{ t('claude.model.hint') }}</div> | ||
| <button class="btn-tool btn-template-editor" @click="openClaudeConfigTemplateEditor" :disabled="loading || !!initError">{{ t('config.template.openEditor') }}</button> | ||
| </div> | ||
|
|
||
| <div class="selector-section"> | ||
|
|
@@ -97,20 +98,6 @@ | |
| </div> | ||
|
|
||
| <div class="card-list"> | ||
| <div v-for="(config, name) in claudeConfigs" :key="name" :class="['card', { active: currentClaudeConfig === name }]" @click="applyClaudeConfig(name)" @keydown.enter.self.prevent="applyClaudeConfig(name)" @keydown.space.self.prevent="applyClaudeConfig(name)" tabindex="0" role="button" :aria-current="currentClaudeConfig === name ? 'true' : null"> | ||
| <div class="card-leading"> | ||
| <div class="card-icon">{{ name.charAt(0).toUpperCase() }}</div> | ||
|
|
@@ -141,26 +128,5 @@ | |
| </div> | ||
| </div> | ||
|
|
||
| </template> | ||
| </div> | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -124,6 +124,10 @@ | |
| <div class="card-leading"> | ||
| <div class="card-icon">{{ provider.name.charAt(0).toUpperCase() }}<span v-if="isTransformProvider(provider)" class="card-icon-dot" title="通过内建转换适配"></span></div> | ||
| <div class="card-content"> | ||
| <div v-if="provider.name === 'local'" class="bridge-pool-summary"> | ||
| <svg class="bridge-pool-summary-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="12" height="12"><circle cx="6" cy="6" r="2"/><circle cx="18" cy="6" r="2"/><circle cx="12" cy="18" r="2"/><path d="M6 8v4h6v4"/><path d="M18 8v4h-6v4"/></svg> | ||
| <span class="bridge-pool-summary-text">已启用 {{ localBridgeCandidateProviders().filter(cp => !isLocalBridgeExcluded(cp.name)).length }} / {{ localBridgeCandidateProviders().length }}</span> | ||
| </div> | ||
| <div class="card-title"> | ||
| <span>{{ provider.name }}</span> | ||
| <span v-if="provider.readOnly" class="provider-readonly-badge">{{ t('config.badge.system') }}</span> | ||
|
|
@@ -136,6 +140,9 @@ | |
| <span v-if="speedResults[provider.name]" :class="['latency', speedResults[provider.name].ok ? 'ok' : 'error']">{{ formatLatency(speedResults[provider.name]) }}</span> | ||
| <span :class="['pill', providerPillConfigured(provider) ? 'configured' : 'empty']">{{ providerPillText(provider) }}</span> | ||
| <div class="card-actions" @click.stop> | ||
| <button v-if="provider.name === 'local'" class="card-action-btn bridge-pool-trigger" @click="showCodexBridgePoolModal = true" :aria-label="'轮询池设置'" :title="'轮询池设置'"> | ||
| <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><circle cx="6" cy="6" r="2"/><circle cx="18" cy="6" r="2"/><circle cx="12" cy="18" r="2"/><path d="M6 8v4h6v4"/><path d="M18 8v4h-6v4"/></svg> | ||
| </button> | ||
| <button class="card-action-btn" :class="{ loading: speedLoading[provider.name] }" :disabled="!!speedLoading[provider.name]" @click="runSpeedTest(provider.name, { silent: true })" :aria-label="t('config.availabilityTestAria', { name: provider.name })" :title="t('config.availabilityTest')"> | ||
| <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M13 2L3 14h9l-1 8 10-12h-9l1-8z"/></svg> | ||
| </button> | ||
|
|
@@ -166,26 +173,5 @@ | |
| </div> | ||
| </div> | ||
| </div> | ||
| </template> | ||
| </div> | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,6 +3,43 @@ | |
| ============================================ */ | ||
|
|
||
|
|
||
| /* ---- 摘要状态（在卡片标题上方） ---- */ | ||
| .bridge-pool-summary { | ||
| display: flex; | ||
| align-items: center; | ||
| gap: 5px; | ||
| margin-bottom: 4px; | ||
| } | ||
|
|
||
| .bridge-pool-summary-icon { | ||
| display: inline-flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| width: 18px; | ||
| height: 18px; | ||
| border-radius: 5px; | ||
| background: var(--color-brand-light); | ||
| color: var(--color-brand-dark); | ||
| flex-shrink: 0; | ||
| } | ||
|
|
||
| .bridge-pool-summary-text { | ||
| font-size: 11px; | ||
| font-weight: 600; | ||
| color: var(--color-text-muted); | ||
| letter-spacing: -0.01em; | ||
| } | ||
|
|
||
| /* ---- 轮询池触发按钮 ---- */ | ||
| .bridge-pool-trigger { | ||
| color: var(--color-brand); | ||
| background: rgba(200, 121, 99, 0.08); | ||
| } | ||
|
|
||
| .bridge-pool-trigger:hover { | ||
| background: rgba(200, 121, 99, 0.16); | ||
| } | ||
|
|
||
| .bridge-pool-panel { | ||
| margin-top: 18px; | ||
| padding: 18px 20px 16px; | ||
|
|
@@ -195,3 +232,35 @@ | |
| padding: 10px 12px 10px 14px; | ||
| } | ||
| } | ||
|
|
||
| /* ---- 模态框中的轮询池 ---- */ | ||
| .modal-bridge-pool { | ||
| max-width: 380px; | ||
| } | ||
|
|
||
| .modal-title-icon { | ||
| display: inline-flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| width: 24px; | ||
| height: 24px; | ||
| border-radius: 6px; | ||
| background: var(--color-brand-light); | ||
| color: var(--color-brand-dark); | ||
| margin-right: 8px; | ||
| vertical-align: middle; | ||
| } | ||
|
|
||
| .bridge-pool-modal-hint { | ||
| font-size: 12px; | ||
| color: var(--color-text-muted); | ||
| margin-bottom: 14px; | ||
| padding-bottom: 12px; | ||
| border-bottom: 1px solid var(--color-border-soft); | ||
| } | ||
|
|
||
| .modal-bridge-pool .bridge-pool-list { | ||
| max-height: 320px; | ||
| overflow-y: auto; | ||
| margin-bottom: 16px; | ||
| } | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Local-bridge URL detection is too strict and misses valid forms.
The current check only matches
/bridge/claude-local/. URLs without trailing slash (e.g.,/bridge/claude-local) won’t match.Proposed fix
🤖 Prompt for AI Agents