@memtensor/memos-local-openclaw-plugin 1.0.0 → 1.0.1

package/src/viewer/html.ts

@@ -512,6 +512,12 @@ input,textarea,select{font-family:inherit;font-size:inherit}
 .toggle-slider::before{content:'';position:absolute;height:14px;width:14px;left:3px;bottom:3px;background:#fff;border-radius:50%;transition:.2s}
 .toggle-switch input:checked+.toggle-slider{background:var(--pri)}
 .toggle-switch input:checked+.toggle-slider::before{transform:translateX(16px)}
+.test-conn-row{display:flex;align-items:center;gap:10px;margin-top:12px;padding-top:10px;border-top:1px dashed var(--border)}
+.test-conn-row .btn{font-size:11px;padding:5px 14px;border:1px solid var(--border);border-radius:6px}
+.test-result{font-size:12px;line-height:1.5;word-break:break-word}
+.test-result.ok{color:#22c55e}
+.test-result.fail{color:var(--rose)}
+.test-result.loading{color:var(--text-muted)}
 .settings-actions{display:flex;gap:12px;justify-content:flex-end;align-items:center;margin-top:16px;padding-top:16px;border-top:1px solid var(--border)}
 .settings-actions .btn{min-width:110px;padding:10px 20px;font-size:13px}
 .settings-actions .btn-primary{background:rgba(99,102,241,.08);color:var(--pri);border:1px solid rgba(99,102,241,.25);font-weight:600}
@@ -939,9 +945,12 @@ input,textarea,select{font-family:inherit;font-size:inherit}
         <div class="settings-grid">
           <div class="settings-field">
             <label data-i18n="settings.provider">Provider</label>
-            <select id="cfgEmbProvider">
+            <select id="cfgEmbProvider" onchange="onProviderChange('embedding')">
               <option value="openai_compatible">OpenAI Compatible</option>
               <option value="openai">OpenAI</option>
+              <option value="siliconflow">SiliconFlow (\u7845\u57FA\u6D41\u52A8)</option>
+              <option value="zhipu">Zhipu AI (\u667A\u8C31)</option>
+              <option value="bailian">Alibaba Bailian (\u767E\u70BC)</option>
               <option value="gemini">Gemini</option>
               <option value="azure_openai">Azure OpenAI</option>
               <option value="cohere">Cohere</option>
@@ -963,6 +972,10 @@ input,textarea,select{font-family:inherit;font-size:inherit}
             <input type="password" id="cfgEmbApiKey" placeholder="\u2022\u2022\u2022\u2022\u2022\u2022\u2022\u2022">
           </div>
         </div>
+        <div class="test-conn-row">
+          <button class="btn btn-sm btn-ghost" onclick="testModel('embedding')" id="testEmbBtn" data-i18n="settings.test">Test Connection</button>
+          <span class="test-result" id="testEmbResult"></span>
+        </div>
       </div>
 
       <div class="settings-section">
@@ -970,9 +983,14 @@ input,textarea,select{font-family:inherit;font-size:inherit}
         <div class="settings-grid">
           <div class="settings-field">
             <label data-i18n="settings.provider">Provider</label>
-            <select id="cfgSumProvider">
+            <select id="cfgSumProvider" onchange="onProviderChange('summarizer')">
               <option value="openai_compatible">OpenAI Compatible</option>
               <option value="openai">OpenAI</option>
+              <option value="siliconflow">SiliconFlow (\u7845\u57FA\u6D41\u52A8)</option>
+              <option value="zhipu">Zhipu AI (\u667A\u8C31)</option>
+              <option value="deepseek">DeepSeek</option>
+              <option value="bailian">Alibaba Bailian (\u767E\u70BC)</option>
+              <option value="moonshot">Moonshot (Kimi)</option>
               <option value="anthropic">Anthropic</option>
               <option value="gemini">Gemini</option>
               <option value="azure_openai">Azure OpenAI</option>
@@ -996,6 +1014,10 @@ input,textarea,select{font-family:inherit;font-size:inherit}
             <input type="number" id="cfgSumTemp" step="0.1" min="0" max="2" placeholder="0">
           </div>
         </div>
+        <div class="test-conn-row">
+          <button class="btn btn-sm btn-ghost" onclick="testModel('summarizer')" id="testSumBtn" data-i18n="settings.test">Test Connection</button>
+          <span class="test-result" id="testSumResult"></span>
+        </div>
       </div>
       </div>
 
@@ -1025,10 +1047,15 @@ input,textarea,select{font-family:inherit;font-size:inherit}
           <div class="settings-grid">
             <div class="settings-field">
               <label data-i18n="settings.provider">Provider</label>
-              <select id="cfgSkillProvider">
+              <select id="cfgSkillProvider" onchange="onProviderChange('skill')">
                 <option value="">— <span data-i18n="settings.skill.usemain">Use main summarizer</span> —</option>
                 <option value="openai_compatible">OpenAI Compatible</option>
                 <option value="openai">OpenAI</option>
+                <option value="siliconflow">SiliconFlow (\u7845\u57FA\u6D41\u52A8)</option>
+                <option value="zhipu">Zhipu AI (\u667A\u8C31)</option>
+                <option value="deepseek">DeepSeek</option>
+                <option value="bailian">Alibaba Bailian (\u767E\u70BC)</option>
+                <option value="moonshot">Moonshot (Kimi)</option>
                 <option value="anthropic">Anthropic</option>
                 <option value="gemini">Gemini</option>
                 <option value="azure_openai">Azure OpenAI</option>
@@ -1048,6 +1075,10 @@ input,textarea,select{font-family:inherit;font-size:inherit}
               <input type="password" id="cfgSkillApiKey" placeholder="\u2022\u2022\u2022\u2022\u2022\u2022\u2022\u2022">
             </div>
           </div>
+          <div class="test-conn-row">
+            <button class="btn btn-sm btn-ghost" onclick="testModel('skill')" id="testSkillBtn" data-i18n="settings.test">Test Connection</button>
+            <span class="test-result" id="testSkillResult"></span>
+          </div>
         </div>
       </div>
 
@@ -1460,11 +1491,24 @@ const I18N={
     'settings.telemetry.hint':'Anonymous usage analytics to help improve the plugin. Only sends tool names, latencies, and version info. No memory content, queries, or personal data is ever sent.',
     'settings.viewerport':'Viewer Port',
     'settings.viewerport.hint':'Requires restart to take effect',
+    'settings.test':'Test Connection',
+    'settings.test.loading':'Testing...',
+    'settings.test.ok':'Connected',
+    'settings.test.fail':'Failed',
     'settings.save':'Save Settings',
     'settings.reset':'Reset',
     'settings.saved':'Saved',
     'settings.restart.hint':'Some changes require restarting the OpenClaw gateway to take effect.',
     'settings.save.fail':'Failed to save settings',
+    'settings.save.emb.required':'Embedding model is required. Please configure an embedding model before saving.',
+    'settings.save.emb.fail':'Embedding model test failed, cannot save',
+    'settings.save.sum.fail':'Summarizer model test failed, cannot save',
+    'settings.save.skill.fail':'Skill model test failed, cannot save',
+    'settings.save.sum.fallback':'Summarizer model is not configured — will use OpenClaw native model as fallback.',
+    'settings.save.skill.fallback':'Skill dedicated model is not configured — will use OpenClaw native model as fallback.',
+    'settings.save.fallback.model':'Fallback model: ',
+    'settings.save.fallback.none':'Not available (no OpenClaw native model found)',
+    'settings.save.fallback.confirm':'Continue to save?',
     'migrate.title':'Import OpenClaw Memory',
     'migrate.desc':'Migrate your existing OpenClaw built-in memories and conversation history into this plugin. The import process uses smart deduplication to avoid duplicates.',
     'migrate.modes.title':'Three ways to use:',
@@ -1753,11 +1797,24 @@ const I18N={
     'settings.telemetry.hint':'匿名使用统计，帮助改进插件。仅发送工具名称、响应时间和版本信息，不会发送任何记忆内容、搜索查询或个人数据。',
     'settings.viewerport':'Viewer 端口',
     'settings.viewerport.hint':'修改后需重启网关生效',
+    'settings.test':'测试连接',
+    'settings.test.loading':'测试中...',
+    'settings.test.ok':'连接成功',
+    'settings.test.fail':'连接失败',
     'settings.save':'保存设置',
     'settings.reset':'重置',
     'settings.saved':'已保存',
     'settings.restart.hint':'部分设置修改后需要重启 OpenClaw 网关才能生效。',
     'settings.save.fail':'保存设置失败',
+    'settings.save.emb.required':'嵌入模型为必填项，请先配置嵌入模型再保存。',
+    'settings.save.emb.fail':'嵌入模型测试失败，无法保存',
+    'settings.save.sum.fail':'摘要模型测试失败，无法保存',
+    'settings.save.skill.fail':'技能模型测试失败，无法保存',
+    'settings.save.sum.fallback':'摘要模型未配置 — 将使用 OpenClaw 原生模型作为降级方案。',
+    'settings.save.skill.fallback':'技能专用模型未配置 — 将使用 OpenClaw 原生模型作为降级方案。',
+    'settings.save.fallback.model':'降级模型：',
+    'settings.save.fallback.none':'不可用（未检测到 OpenClaw 原生模型）',
+    'settings.save.fallback.confirm':'是否继续保存？',
     'migrate.title':'导入 OpenClaw 记忆',
     'migrate.desc':'将 OpenClaw 内置的记忆数据和对话历史迁移到本插件中。导入过程使用智能去重，避免重复导入。',
     'migrate.modes.title':'三种使用方式：',
@@ -2285,7 +2342,6 @@ async function loadTasks(){
         '</div>'+
         '<div class="card-actions" onclick="event.stopPropagation()">'+
           '<button class="btn btn-sm btn-ghost" onclick="openTaskDetail(\\''+task.id+'\\')">'+t('card.expand')+'</button>'+
-          '<button class="btn btn-sm btn-ghost" onclick="editTaskInline(\\''+task.id+'\\')">'+t('card.edit')+'</button>'+
           (task.status==='completed'&&(!task.skillStatus||task.skillStatus==='not_generated'||task.skillStatus==='skipped')?'<button class="btn btn-sm btn-ghost" onclick="retrySkillGen(\\''+task.id+'\\')">'+t('task.retrySkill.short')+'</button>':'')+
           '<button class="btn btn-sm btn-ghost" style="color:var(--accent)" onclick="deleteTask(\\''+task.id+'\\')">'+t('task.delete')+'</button>'+
         '</div>'+
@@ -2458,32 +2514,6 @@ async function deleteTask(taskId){
   }catch(e){ alert(t('task.delete.error')+e.message); }
 }
 
-async function editTaskInline(){
-  if(!_currentTaskData) return;
-  var task=_currentTaskData;
-  var titleEl=document.getElementById('taskDetailTitle');
-  var summaryEl=document.getElementById('taskDetailSummary');
-  var actionsEl=document.getElementById('taskDetailActions');
-
-  titleEl.innerHTML='<input id="editTaskTitle" class="filter-input" style="width:100%;font-size:16px;font-weight:600" value="'+esc(task.title||'')+'"/>';
-  summaryEl.innerHTML='<textarea id="editTaskSummary" class="filter-input" style="width:100%;min-height:80px;font-size:13px;resize:vertical">'+esc(task.summary||'')+'</textarea>';
-  actionsEl.innerHTML=
-    '<button class="btn btn-primary" onclick="saveTaskEdit()" style="font-size:12px">'+t('task.save')+'</button>'+
-    '<button class="btn btn-ghost" onclick="openTaskDetail(\\''+esc(task.id)+'\\')" style="font-size:12px">'+t('task.cancel')+'</button>';
-}
-
-async function saveTaskEdit(){
-  if(!_currentTaskId) return;
-  var title=document.getElementById('editTaskTitle').value.trim();
-  var summary=document.getElementById('editTaskSummary').value.trim();
-  try{
-    const r=await fetch('/api/task/'+_currentTaskId,{method:'PUT',headers:{'Content-Type':'application/json'},body:JSON.stringify({title:title,summary:summary})});
-    const d=await r.json();
-    if(!r.ok) throw new Error(d.error||'unknown');
-    openTaskDetail(_currentTaskId);
-    loadTasks();
-  }catch(e){ alert(t('task.save.error')+e.message); }
-}
 
 /* ─── Skills View Logic ─── */
 let skillsStatusFilter='';
@@ -2544,7 +2574,6 @@ async function loadSkills(){
         '</div>'+
         '<div class="card-actions" onclick="event.stopPropagation()">'+
           '<button class="btn btn-sm btn-ghost" onclick="openSkillDetail(\\''+skill.id+'\\')">'+t('card.expand')+'</button>'+
-          '<button class="btn btn-sm btn-ghost" onclick="editSkillInline(\\''+skill.id+'\\')">'+t('card.edit')+'</button>'+
           (skill.visibility==='public'?'<button class="btn btn-sm btn-ghost" onclick="toggleSkillPublic(\\''+skill.id+'\\',false)">\\u{1F512} '+t('skills.setPrivate')+'</button>':'<button class="btn btn-sm btn-ghost" onclick="toggleSkillPublic(\\''+skill.id+'\\',true)">\\u{1F310} '+t('skills.setPublic')+'</button>')+
           '<button class="btn btn-sm btn-ghost" style="color:var(--accent)" onclick="deleteSkill(\\''+skill.id+'\\')">'+t('skill.delete')+'</button>'+
         '</div>'+
@@ -2691,11 +2720,11 @@ async function toggleSkillVisibility(){
   const newVis=btn.dataset.vis==='public'?'private':'public';
   try{
     const r=await fetch('/api/skill/'+currentSkillId+'/visibility',{method:'PUT',headers:{'Content-Type':'application/json'},body:JSON.stringify({visibility:newVis})});
-    if(!r.ok) throw new Error('Failed: '+r.status);
+    if(!r.ok){var errBody='';try{var ej=await r.json();errBody=ej.error||JSON.stringify(ej);}catch(x){errBody=await r.text();}throw new Error(r.status+': '+errBody);}
     openSkillDetail(currentSkillId);
     loadSkills();
   }catch(e){
-    alert('Error: '+e.message);
+    toast('Error: '+e.message,'error');
   }
 }
 
@@ -2703,7 +2732,7 @@ async function toggleSkillPublic(id,setPublic){
   const newVis=setPublic?'public':'private';
   try{
     const r=await fetch('/api/skill/'+id+'/visibility',{method:'PUT',headers:{'Content-Type':'application/json'},body:JSON.stringify({visibility:newVis})});
-    if(!r.ok) throw new Error('Failed: '+r.status);
+    if(!r.ok){var errBody='';try{var ej=await r.json();errBody=ej.error||JSON.stringify(ej);}catch(x){errBody=await r.text();}throw new Error(r.status+': '+errBody);}
     toast(setPublic?t('toast.setPublic'):t('toast.setPrivate'),'success');
     loadSkills();
   }catch(e){
@@ -2751,7 +2780,37 @@ async function loadConfig(){
   }
 }
 
+var _providerDefaults={
+  siliconflow:{endpoint:'https://api.siliconflow.cn/v1',embModel:'BAAI/bge-m3',chatModel:'Qwen/Qwen2.5-7B-Instruct'},
+  openai:{endpoint:'https://api.openai.com/v1',embModel:'text-embedding-3-small',chatModel:'gpt-4o-mini'},
+  anthropic:{endpoint:'https://api.anthropic.com/v1/messages',chatModel:'claude-3-haiku-20240307'},
+  cohere:{endpoint:'https://api.cohere.com/v2',embModel:'embed-english-v3.0'},
+  mistral:{endpoint:'https://api.mistral.ai/v1',embModel:'mistral-embed'},
+  voyage:{endpoint:'https://api.voyageai.com/v1',embModel:'voyage-3'},
+  gemini:{endpoint:'',embModel:'text-embedding-004',chatModel:'gemini-2.0-flash'},
+  zhipu:{endpoint:'https://open.bigmodel.cn/api/paas/v4',embModel:'embedding-3',chatModel:'glm-4-flash'},
+  deepseek:{endpoint:'https://api.deepseek.com/v1',chatModel:'deepseek-chat'},
+  bailian:{endpoint:'https://dashscope.aliyuncs.com/compatible-mode/v1',embModel:'text-embedding-v3',chatModel:'qwen-max'},
+  moonshot:{endpoint:'https://api.moonshot.cn/v1',chatModel:'moonshot-v1-8k'}
+};
+function onProviderChange(section){
+  var map={embedding:['cfgEmbEndpoint','cfgEmbModel','emb'],summarizer:['cfgSumEndpoint','cfgSumModel','chat'],skill:['cfgSkillEndpoint','cfgSkillModel','chat']};
+  var m=map[section];if(!m)return;
+  var sel=document.getElementById(section==='embedding'?'cfgEmbProvider':section==='summarizer'?'cfgSumProvider':'cfgSkillProvider');
+  var pv=sel.value;
+  var def=_providerDefaults[pv];
+  if(!def)return;
+  var epEl=document.getElementById(m[0]);
+  var mdEl=document.getElementById(m[1]);
+  if(def.endpoint&&!epEl.value.trim()) epEl.value=def.endpoint;
+  if(m[2]==='emb'&&def.embModel&&!mdEl.value.trim()) mdEl.value=def.embModel;
+  if(m[2]==='chat'&&def.chatModel&&!mdEl.value.trim()) mdEl.value=def.chatModel;
+}
+
 async function saveConfig(){
+  var saveBtn=document.querySelector('.settings-actions .btn-primary');
+  saveBtn.disabled=true;saveBtn.textContent=t('settings.test.loading');
+
   const cfg={};
   const embP=document.getElementById('cfgEmbProvider').value;
   if(embP){
@@ -2761,11 +2820,15 @@ async function saveConfig(){
     const k=document.getElementById('cfgEmbApiKey').value.trim();if(k) cfg.embedding.apiKey=k;
   }
   const sumP=document.getElementById('cfgSumProvider').value;
-  if(sumP){
+  const sumModel=document.getElementById('cfgSumModel').value.trim();
+  const sumEndpoint=document.getElementById('cfgSumEndpoint').value.trim();
+  const sumApiKey=document.getElementById('cfgSumApiKey').value.trim();
+  var hasSumConfig=!!(sumModel||sumEndpoint||sumApiKey);
+  if(hasSumConfig&&sumP){
     cfg.summarizer={provider:sumP};
-    const v=document.getElementById('cfgSumModel').value.trim();if(v) cfg.summarizer.model=v;
-    const e=document.getElementById('cfgSumEndpoint').value.trim();if(e) cfg.summarizer.endpoint=e;
-    const k=document.getElementById('cfgSumApiKey').value.trim();if(k) cfg.summarizer.apiKey=k;
+    if(sumModel) cfg.summarizer.model=sumModel;
+    if(sumEndpoint) cfg.summarizer.endpoint=sumEndpoint;
+    if(sumApiKey) cfg.summarizer.apiKey=sumApiKey;
     const tp=document.getElementById('cfgSumTemp').value.trim();if(tp!=='') cfg.summarizer.temperature=Number(tp);
   }
   cfg.skillEvolution={
@@ -2776,29 +2839,118 @@ async function saveConfig(){
   const mk=document.getElementById('cfgSkillMinChunks').value.trim();if(mk) cfg.skillEvolution.minChunksForEval=Number(mk);
 
   const skP=document.getElementById('cfgSkillProvider').value;
-  if(skP){
+  const skModel=document.getElementById('cfgSkillModel').value.trim();
+  const skEndpoint=document.getElementById('cfgSkillEndpoint').value.trim();
+  const skApiKey=document.getElementById('cfgSkillApiKey').value.trim();
+  var hasSkillConfig=!!(skP&&(skModel||skEndpoint||skApiKey));
+  if(hasSkillConfig){
     cfg.skillEvolution.summarizer={provider:skP};
-    const sv=document.getElementById('cfgSkillModel').value.trim();if(sv) cfg.skillEvolution.summarizer.model=sv;
-    const se=document.getElementById('cfgSkillEndpoint').value.trim();if(se) cfg.skillEvolution.summarizer.endpoint=se;
-    const sk=document.getElementById('cfgSkillApiKey').value.trim();if(sk) cfg.skillEvolution.summarizer.apiKey=sk;
+    if(skModel) cfg.skillEvolution.summarizer.model=skModel;
+    if(skEndpoint) cfg.skillEvolution.summarizer.endpoint=skEndpoint;
+    if(skApiKey) cfg.skillEvolution.summarizer.apiKey=skApiKey;
   }
 
   const vp=document.getElementById('cfgViewerPort').value.trim();
   if(vp) cfg.viewerPort=Number(vp);
+  cfg.telemetry={enabled:document.getElementById('cfgTelemetryEnabled').checked};
 
-  cfg.telemetry={
-    enabled:document.getElementById('cfgTelemetryEnabled').checked
-  };
+  function done(){saveBtn.disabled=false;saveBtn.textContent=t('settings.save');}
+
+  // 1) Embedding model is required
+  if(!embP||embP===''){done();toast(t('settings.save.emb.required'),'error');return;}
 
+  // 2) Test embedding
+  try{
+    var er=await fetch('/api/test-model',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({type:'embedding',provider:cfg.embedding.provider,model:cfg.embedding.model||'',endpoint:cfg.embedding.endpoint||'',apiKey:cfg.embedding.apiKey||''})});
+    var ed=await er.json();
+    if(!ed.ok){done();toast(t('settings.save.emb.fail')+': '+ed.error,'error');document.getElementById('testEmbResult').className='test-result fail';document.getElementById('testEmbResult').innerHTML='\\u274C '+ed.error;return;}
+    document.getElementById('testEmbResult').className='test-result ok';document.getElementById('testEmbResult').innerHTML='\\u2705 '+t('settings.test.ok');
+  }catch(e){done();toast(t('settings.save.emb.fail')+': '+e.message,'error');return;}
+
+  // 3) Test summarizer if user filled it
+  if(hasSumConfig&&cfg.summarizer){
+    try{
+      var sr=await fetch('/api/test-model',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({type:'summarizer',provider:cfg.summarizer.provider,model:cfg.summarizer.model||'',endpoint:cfg.summarizer.endpoint||'',apiKey:cfg.summarizer.apiKey||''})});
+      var sd=await sr.json();
+      if(!sd.ok){done();toast(t('settings.save.sum.fail')+': '+sd.error,'error');document.getElementById('testSumResult').className='test-result fail';document.getElementById('testSumResult').innerHTML='\\u274C '+sd.error;return;}
+      document.getElementById('testSumResult').className='test-result ok';document.getElementById('testSumResult').innerHTML='\\u2705 '+t('settings.test.ok');
+    }catch(e){done();toast(t('settings.save.sum.fail')+': '+e.message,'error');return;}
+  }
+
+  // 4) Test skill model if user filled it
+  if(hasSkillConfig&&cfg.skillEvolution.summarizer){
+    try{
+      var kr=await fetch('/api/test-model',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({type:'summarizer',provider:cfg.skillEvolution.summarizer.provider,model:cfg.skillEvolution.summarizer.model||'',endpoint:cfg.skillEvolution.summarizer.endpoint||'',apiKey:cfg.skillEvolution.summarizer.apiKey||''})});
+      var kd=await kr.json();
+      if(!kd.ok){done();toast(t('settings.save.skill.fail')+': '+kd.error,'error');document.getElementById('testSkillResult').className='test-result fail';document.getElementById('testSkillResult').innerHTML='\\u274C '+kd.error;return;}
+      document.getElementById('testSkillResult').className='test-result ok';document.getElementById('testSkillResult').innerHTML='\\u2705 '+t('settings.test.ok');
+    }catch(e){done();toast(t('settings.save.skill.fail')+': '+e.message,'error');return;}
+  }
+
+  // 5) If summarizer or skill model not configured, check OpenClaw fallback and confirm
+  if(!hasSumConfig||!hasSkillConfig){
+    try{
+      var fr=await fetch('/api/fallback-model');
+      var fb=await fr.json();
+      var msgs=[];
+      if(!hasSumConfig){msgs.push(t('settings.save.sum.fallback'));}
+      if(!hasSkillConfig){msgs.push(t('settings.save.skill.fallback'));}
+      var fbInfo=fb.available?(fb.model+' ('+fb.baseUrl+')'):t('settings.save.fallback.none');
+      var confirmMsg=msgs.join('\\n')+'\\n\\n'+t('settings.save.fallback.model')+fbInfo+'\\n\\n'+t('settings.save.fallback.confirm');
+      if(!confirm(confirmMsg)){done();return;}
+    }catch(e){}
+  }
+
+  // 6) All tests passed, save
   try{
     const r=await fetch('/api/config',{method:'PUT',headers:{'Content-Type':'application/json'},body:JSON.stringify(cfg)});
     if(!r.ok) throw new Error(await r.text());
     const el=document.getElementById('settingsSaved');
     el.classList.add('show');
     setTimeout(()=>el.classList.remove('show'),2500);
+    toast(t('settings.saved'),'success');
   }catch(e){
-    showToast(t('settings.save.fail')+': '+e.message,'error');
+    toast(t('settings.save.fail')+': '+e.message,'error');
+  }finally{done();}
+}
+
+async function testModel(type){
+  var ids={embedding:['Emb','cfgEmbProvider','cfgEmbModel','cfgEmbEndpoint','cfgEmbApiKey'],summarizer:['Sum','cfgSumProvider','cfgSumModel','cfgSumEndpoint','cfgSumApiKey'],skill:['Skill','cfgSkillProvider','cfgSkillModel','cfgSkillEndpoint','cfgSkillApiKey']};
+  var c=ids[type];if(!c)return;
+  var resultEl=document.getElementById('test'+c[0]+'Result');
+  var btn=document.getElementById('test'+c[0]+'Btn');
+  var provider=document.getElementById(c[1]).value;
+  var model=document.getElementById(c[2]).value.trim();
+  var endpoint=document.getElementById(c[3]).value.trim();
+  var apiKey=document.getElementById(c[4]).value.trim();
+  if(!provider||(provider!=='local'&&!model)){
+    resultEl.className='test-result fail';
+    resultEl.innerHTML='\\u274C '+t('settings.test.fail')+'<div style="margin-top:4px;font-size:11px;color:var(--text-muted)">Provider and Model are required</div>';
+    return;
   }
+  if(provider!=='local'&&!apiKey){
+    resultEl.className='test-result fail';
+    resultEl.innerHTML='\\u274C '+t('settings.test.fail')+'<div style="margin-top:4px;font-size:11px;color:var(--text-muted)">API Key is required</div>';
+    return;
+  }
+  resultEl.className='test-result loading';resultEl.textContent=t('settings.test.loading');
+  btn.disabled=true;
+  try{
+    var body={type:type,provider:provider,model:model,endpoint:endpoint,apiKey:apiKey};
+    var r=await fetch('/api/test-model',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify(body)});
+    var d=await r.json();
+    if(d.ok){
+      resultEl.className='test-result ok';
+      resultEl.innerHTML='\\u2705 '+t('settings.test.ok')+'<div style="margin-top:4px;font-size:11px;color:var(--text-muted)">'+esc(d.detail||'')+'</div>';
+    }else{
+      var errMsg=d.error||'Unknown error';
+      resultEl.className='test-result fail';
+      resultEl.innerHTML='\\u274C '+t('settings.test.fail')+'<div style="margin-top:6px;font-size:11px;padding:8px 10px;background:rgba(239,68,68,.06);border:1px solid rgba(239,68,68,.15);border-radius:6px;white-space:pre-wrap;word-break:break-all;max-height:120px;overflow-y:auto;font-family:SF Mono,Monaco,Consolas,monospace">'+esc(errMsg)+'</div>';
+    }
+  }catch(e){
+    resultEl.className='test-result fail';
+    resultEl.innerHTML='\\u274C '+t('settings.test.fail')+'<div style="margin-top:6px;font-size:11px;padding:8px 10px;background:rgba(239,68,68,.06);border:1px solid rgba(239,68,68,.15);border-radius:6px;white-space:pre-wrap;word-break:break-all">'+esc(e.message)+'</div>';
+  }finally{btn.disabled=false;}
 }
 
 function renderSkillMarkdown(md){
@@ -2844,28 +2996,6 @@ async function deleteSkill(skillId){
   }catch(e){ alert(t('skill.delete.error')+e.message); }
 }
 
-function editSkillInline(){
-  var skill=window._currentSkillData;
-  if(!skill) return;
-  var descEl=document.getElementById('skillDetailDesc');
-  var actionsEl=document.getElementById('skillDetailActions');
-  descEl.innerHTML='<textarea id="editSkillDesc" class="filter-input" style="width:100%;min-height:60px;font-size:13px;resize:vertical">'+esc(skill.description||'')+'</textarea>';
-  actionsEl.innerHTML=
-    '<button class="btn btn-primary" onclick="saveSkillEdit()" style="font-size:12px">'+t('skill.save')+'</button>'+
-    '<button class="btn btn-ghost" onclick="openSkillDetail(\\''+esc(skill.id)+'\\')" style="font-size:12px">'+t('skill.cancel')+'</button>';
-}
-
-async function saveSkillEdit(){
-  if(!currentSkillId) return;
-  var desc=document.getElementById('editSkillDesc').value.trim();
-  try{
-    const r=await fetch('/api/skill/'+currentSkillId,{method:'PUT',headers:{'Content-Type':'application/json'},body:JSON.stringify({description:desc})});
-    const d=await r.json();
-    if(!r.ok) throw new Error(d.error||'unknown');
-    openSkillDetail(currentSkillId);
-    loadSkills();
-  }catch(e){ alert(t('skill.save.error')+e.message); }
-}
 
 function formatDuration(ms){
   const s=Math.floor(ms/1000);