diff --git a/scripts/dev-api.js b/scripts/dev-api.js index f4b6a94..b8e8475 100644 --- a/scripts/dev-api.js +++ b/scripts/dev-api.js @@ -3574,6 +3574,44 @@ export function mergeHermesMemoryConfig(config = {}, form = {}) { return next } +function normalizeHermesMultilineList(value) { + if (Array.isArray(value)) { + return value.map(item => String(item ?? '').trim()).filter(Boolean) + } + return String(value ?? '') + .split(/\r?\n/) + .map(item => item.trim()) + .filter(Boolean) +} + +export function buildHermesSkillsConfigValues(config = {}) { + const root = config && typeof config === 'object' && !Array.isArray(config) ? config : {} + const skills = root.skills && typeof root.skills === 'object' && !Array.isArray(root.skills) + ? root.skills + : {} + const externalDirs = Array.isArray(skills.external_dirs) + ? skills.external_dirs.map(item => String(item ?? '').trim()).filter(Boolean).join('\n') + : '' + return { + creationNudgeInterval: parseHermesInteger(skills.creation_nudge_interval, 'skills.creation_nudge_interval', 15, 0, 10000, false), + externalDirs, + } +} + +export function mergeHermesSkillsConfig(config = {}, form = {}) { + const next = mergeConfigsPreservingFields({}, config && typeof config === 'object' && !Array.isArray(config) ? config : {}) + const currentValues = buildHermesSkillsConfigValues(next) + const skills = next.skills && typeof next.skills === 'object' && !Array.isArray(next.skills) + ? mergeConfigsPreservingFields(next.skills, {}) + : {} + skills.creation_nudge_interval = parseHermesInteger(Object.hasOwn(form, 'creationNudgeInterval') ? form.creationNudgeInterval : currentValues.creationNudgeInterval, 'skills.creation_nudge_interval', 15, 0, 10000, true) + const externalDirs = normalizeHermesMultilineList(Object.hasOwn(form, 'externalDirs') ? form.externalDirs : currentValues.externalDirs) + if (externalDirs.length) skills.external_dirs = externalDirs + else delete skills.external_dirs + next.skills = skills + return next +} + export function buildHermesStreamingConfigValues(config = {}) { const root = config && typeof config === 'object' && !Array.isArray(config) ? config : {} const streaming = hermesStreamingConfigSource(root) @@ -10038,6 +10076,27 @@ const handlers = { } }, + hermes_skills_config_read() { + const { configPath, exists, config } = readHermesConfigYamlObject() + return { + exists, + configPath, + values: buildHermesSkillsConfigValues(config), + } + }, + + hermes_skills_config_save({ form } = {}) { + const { configPath, config } = readHermesConfigYamlObject() + const next = mergeHermesSkillsConfig(config, form || {}) + const backup = writeHermesConfigYamlObject(configPath, next) + return { + ok: true, + configPath, + backup, + values: buildHermesSkillsConfigValues(next), + } + }, + hermes_streaming_config_read() { const { configPath, exists, config } = readHermesConfigYamlObject() return { diff --git a/src-tauri/src/commands/hermes.rs b/src-tauri/src/commands/hermes.rs index 893afc5..33f77ee 100644 --- a/src-tauri/src/commands/hermes.rs +++ b/src-tauri/src/commands/hermes.rs @@ -2269,6 +2269,21 @@ fn yaml_string_field(map: &serde_yaml::Mapping, key: &str) -> Option { .map(|v| v.to_string()) } +fn yaml_string_sequence_field(map: &serde_yaml::Mapping, key: &str) -> Vec { + yaml_get(map, key) + .and_then(|value| value.as_sequence()) + .map(|items| { + items + .iter() + .filter_map(|item| item.as_str()) + .map(str::trim) + .filter(|item| !item.is_empty()) + .map(ToString::to_string) + .collect::>() + }) + .unwrap_or_default() +} + fn yaml_scalar_string_field(map: &serde_yaml::Mapping, key: &str) -> Option { let value = yaml_get(map, key)?; if let Some(value) = value.as_str() { @@ -3671,6 +3686,71 @@ fn merge_hermes_memory_config(config: &mut serde_yaml::Value, form: &Value) -> R Ok(()) } +fn normalize_hermes_multiline_list(raw: Option) -> Vec { + raw.unwrap_or_default() + .lines() + .map(str::trim) + .filter(|item| !item.is_empty()) + .map(ToString::to_string) + .collect() +} + +fn build_hermes_skills_config_values(config: &serde_yaml::Value) -> Value { + let root = config.as_mapping(); + let skills = root.and_then(|map| yaml_get_mapping(map, "skills")); + let creation_nudge_interval = skills + .map(|map| bounded_hermes_i64(yaml_i64_field(map, "creation_nudge_interval"), 15, 0, 10000)) + .unwrap_or(15); + let external_dirs = skills + .map(|map| yaml_string_sequence_field(map, "external_dirs").join("\n")) + .unwrap_or_default(); + + serde_json::json!({ + "creationNudgeInterval": creation_nudge_interval, + "externalDirs": external_dirs, + }) +} + +fn merge_hermes_skills_config(config: &mut serde_yaml::Value, form: &Value) -> Result<(), String> { + let current = build_hermes_skills_config_values(config); + let creation_nudge_interval = validate_hermes_i64( + if form.get("creationNudgeInterval").is_some() { + form_i64(form, "creationNudgeInterval") + } else { + Some(current["creationNudgeInterval"].as_i64().unwrap_or(15)) + }, + "skills.creation_nudge_interval", + 15, + 0, + 10000, + )?; + let external_dirs = normalize_hermes_multiline_list( + form_string(form, "externalDirs") + .or_else(|| current["externalDirs"].as_str().map(ToString::to_string)), + ); + + let root = ensure_yaml_object(config)?; + let skills = yaml_child_object(root, "skills")?; + skills.insert( + yaml_key("creation_nudge_interval"), + serde_yaml::Value::Number(creation_nudge_interval.into()), + ); + if external_dirs.is_empty() { + skills.remove(yaml_key("external_dirs")); + } else { + skills.insert( + yaml_key("external_dirs"), + serde_yaml::Value::Sequence( + external_dirs + .into_iter() + .map(serde_yaml::Value::String) + .collect(), + ), + ); + } + Ok(()) +} + fn normalize_hermes_streaming_transport( value: Option, strict: bool, @@ -5141,6 +5221,30 @@ pub fn hermes_memory_config_save(form: Value) -> Result { })) } +#[tauri::command] +pub fn hermes_skills_config_read() -> Result { + let (config_path, exists, config) = read_hermes_channel_yaml_config()?; + ensure_yaml_object(&mut config.clone())?; + Ok(serde_json::json!({ + "exists": exists, + "configPath": config_path.to_string_lossy(), + "values": build_hermes_skills_config_values(&config), + })) +} + +#[tauri::command] +pub fn hermes_skills_config_save(form: Value) -> Result { + let (config_path, _exists, mut config) = read_hermes_channel_yaml_config()?; + merge_hermes_skills_config(&mut config, &form)?; + let backup = write_hermes_yaml_config(&config_path, &config)?; + Ok(serde_json::json!({ + "ok": true, + "configPath": config_path.to_string_lossy(), + "backup": backup, + "values": build_hermes_skills_config_values(&config), + })) +} + #[tauri::command] pub fn hermes_streaming_config_read() -> Result { let (config_path, exists, config) = read_hermes_channel_yaml_config()?; @@ -10937,6 +11041,103 @@ streaming: } } +#[cfg(test)] +mod hermes_skills_config_tests { + use super::{build_hermes_skills_config_values, merge_hermes_skills_config}; + use serde_json::json; + + #[test] + fn skills_values_have_upstream_defaults() { + let config: serde_yaml::Value = serde_yaml::from_str("{}").unwrap(); + let values = build_hermes_skills_config_values(&config); + assert_eq!(values["creationNudgeInterval"], 15); + assert_eq!(values["externalDirs"], ""); + } + + #[test] + fn skills_values_read_yaml_fields() { + let config: serde_yaml::Value = serde_yaml::from_str( + r#" +skills: + creation_nudge_interval: 30 + external_dirs: + - ~/.agents/skills + - /home/shared/team-skills +"#, + ) + .unwrap(); + + let values = build_hermes_skills_config_values(&config); + assert_eq!(values["creationNudgeInterval"], 30); + assert_eq!( + values["externalDirs"], + "~/.agents/skills\n/home/shared/team-skills" + ); + } + + #[test] + fn merge_skills_config_preserves_unrelated_yaml() { + let mut config: serde_yaml::Value = serde_yaml::from_str( + r#" +model: + provider: anthropic +skills: + creation_nudge_interval: 15 + disabled: + - legacy-skill + custom_flag: keep-skills +memory: + memory_enabled: true +"#, + ) + .unwrap(); + + merge_hermes_skills_config( + &mut config, + &json!({ + "creationNudgeInterval": "0", + "externalDirs": " ~/.agents/skills \n\n /home/shared/team-skills ", + }), + ) + .unwrap(); + + assert_eq!(config["model"]["provider"].as_str(), Some("anthropic")); + assert_eq!(config["memory"]["memory_enabled"].as_bool(), Some(true)); + assert_eq!( + config["skills"]["creation_nudge_interval"].as_i64(), + Some(0) + ); + assert_eq!( + config["skills"]["external_dirs"][0].as_str(), + Some("~/.agents/skills") + ); + assert_eq!( + config["skills"]["external_dirs"][1].as_str(), + Some("/home/shared/team-skills") + ); + assert_eq!( + config["skills"]["disabled"][0].as_str(), + Some("legacy-skill") + ); + assert_eq!( + config["skills"]["custom_flag"].as_str(), + Some("keep-skills") + ); + } + + #[test] + fn merge_skills_config_rejects_invalid_values() { + let mut config = serde_yaml::Value::Mapping(serde_yaml::Mapping::new()); + let err = merge_hermes_skills_config(&mut config, &json!({ "creationNudgeInterval": -1 })) + .unwrap_err(); + assert!(err.contains("skills.creation_nudge_interval")); + let err = + merge_hermes_skills_config(&mut config, &json!({ "creationNudgeInterval": 10001 })) + .unwrap_err(); + assert!(err.contains("skills.creation_nudge_interval")); + } +} + #[cfg(test)] mod hermes_channel_tests { use super::{ diff --git a/src-tauri/src/lib.rs b/src-tauri/src/lib.rs index 92ce6cc..e88cea6 100644 --- a/src-tauri/src/lib.rs +++ b/src-tauri/src/lib.rs @@ -265,6 +265,8 @@ pub fn run() { hermes::hermes_tool_loop_guardrails_config_save, hermes::hermes_memory_config_read, hermes::hermes_memory_config_save, + hermes::hermes_skills_config_read, + hermes::hermes_skills_config_save, hermes::hermes_streaming_config_read, hermes::hermes_streaming_config_save, hermes::hermes_execution_limits_config_read, diff --git a/src/engines/hermes/pages/config.js b/src/engines/hermes/pages/config.js index ec5c22a..83c0227 100644 --- a/src/engines/hermes/pages/config.js +++ b/src/engines/hermes/pages/config.js @@ -43,6 +43,11 @@ const MEMORY_DEFAULTS = { flushMinTurns: 6, } +const SKILLS_DEFAULTS = { + creationNudgeInterval: 15, + externalDirs: '', +} + const STREAMING_DEFAULTS = { enabled: false, transport: 'edit', @@ -92,6 +97,7 @@ export function render() { let compressionValues = { ...COMPRESSION_DEFAULTS } let toolGuardrailsValues = { ...TOOL_GUARDRAILS_DEFAULTS } let memoryValues = { ...MEMORY_DEFAULTS } + let skillsValues = { ...SKILLS_DEFAULTS } let streamingValues = { ...STREAMING_DEFAULTS } let executionLimitsValues = { ...EXECUTION_LIMITS_DEFAULTS } let terminalValues = { ...TERMINAL_DEFAULTS } @@ -100,6 +106,7 @@ export function render() { let compressionLoading = true let toolGuardrailsLoading = true let memoryLoading = true + let skillsLoading = true let streamingLoading = true let executionLimitsLoading = true let terminalLoading = true @@ -108,6 +115,7 @@ export function render() { let compressionSaving = false let toolGuardrailsSaving = false let memorySaving = false + let skillsSaving = false let streamingSaving = false let executionLimitsSaving = false let terminalSaving = false @@ -116,6 +124,7 @@ export function render() { let compressionError = null let toolGuardrailsError = null let memoryError = null + let skillsError = null let streamingError = null let executionLimitsError = null let terminalError = null @@ -129,7 +138,7 @@ export function render() { } function isBusy() { - return loading || runtimeLoading || compressionLoading || toolGuardrailsLoading || memoryLoading || streamingLoading || executionLimitsLoading || terminalLoading || saving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || streamingSaving || executionLimitsSaving || terminalSaving + return loading || runtimeLoading || compressionLoading || toolGuardrailsLoading || memoryLoading || skillsLoading || streamingLoading || executionLimitsLoading || terminalLoading || saving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || skillsSaving || streamingSaving || executionLimitsSaving || terminalSaving } function option(labelKey, value, selected) { @@ -146,7 +155,7 @@ export function render() { } function renderRuntimePanel() { - const disabled = loading || saving || runtimeLoading || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || streamingSaving || executionLimitsSaving || terminalSaving + const disabled = loading || saving || runtimeLoading || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || skillsSaving || streamingSaving || executionLimitsSaving || terminalSaving return `
@@ -194,7 +203,7 @@ export function render() { } function renderCompressionPanel() { - const disabled = loading || saving || compressionLoading || compressionSaving || runtimeSaving || toolGuardrailsSaving || memorySaving || streamingSaving || executionLimitsSaving || terminalSaving + const disabled = loading || saving || compressionLoading || compressionSaving || runtimeSaving || toolGuardrailsSaving || memorySaving || skillsSaving || streamingSaving || executionLimitsSaving || terminalSaving return `
@@ -244,7 +253,7 @@ export function render() { } function renderToolGuardrailsPanel() { - const disabled = loading || saving || toolGuardrailsLoading || toolGuardrailsSaving || runtimeSaving || compressionSaving || memorySaving || streamingSaving || executionLimitsSaving || terminalSaving + const disabled = loading || saving || toolGuardrailsLoading || toolGuardrailsSaving || runtimeSaving || compressionSaving || memorySaving || skillsSaving || streamingSaving || executionLimitsSaving || terminalSaving return `
@@ -306,7 +315,7 @@ export function render() { } function renderMemoryPanel() { - const disabled = loading || saving || memoryLoading || memorySaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || streamingSaving || executionLimitsSaving || terminalSaving + const disabled = loading || saving || memoryLoading || memorySaving || skillsSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || streamingSaving || executionLimitsSaving || terminalSaving return `
@@ -355,8 +364,40 @@ export function render() { ` } + function renderSkillsConfigPanel() { + const disabled = loading || saving || skillsLoading || skillsSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || streamingSaving || executionLimitsSaving || terminalSaving + return ` +
+
+
+
${t('engine.hermesSkillsConfigTitle')}
+
${t('engine.hermesSkillsConfigDesc')}
+
+
+ ${skillsSaving ? t('engine.hermesConfigStatusSaving') : skillsLoading ? t('engine.hermesConfigStatusLoading') : t('engine.hermesSkillsConfigStatusReady')} + +
+
+
+ ${renderError(skillsError)} +
+ + +
+
${t('engine.hermesSkillsConfigFootnote')}
+
+
+ ` + } + function renderStreamingPanel() { - const disabled = loading || saving || streamingLoading || streamingSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || executionLimitsSaving || terminalSaving + const disabled = loading || saving || streamingLoading || streamingSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || skillsSaving || executionLimitsSaving || terminalSaving return `
@@ -408,7 +449,7 @@ export function render() { } function renderExecutionLimitsPanel() { - const disabled = loading || saving || executionLimitsLoading || executionLimitsSaving || terminalSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || streamingSaving + const disabled = loading || saving || executionLimitsLoading || executionLimitsSaving || terminalSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || skillsSaving || streamingSaving return `
@@ -480,7 +521,7 @@ export function render() { } function renderTerminalPanel() { - const disabled = loading || saving || terminalLoading || terminalSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || streamingSaving || executionLimitsSaving + const disabled = loading || saving || terminalLoading || terminalSaving || runtimeSaving || compressionSaving || toolGuardrailsSaving || memorySaving || skillsSaving || streamingSaving || executionLimitsSaving return `
@@ -571,6 +612,7 @@ export function render() { ${renderCompressionPanel()} ${renderToolGuardrailsPanel()} ${renderMemoryPanel()} + ${renderSkillsConfigPanel()}
@@ -594,6 +636,7 @@ export function render() { el.querySelector('#hm-compression-save')?.addEventListener('click', saveCompression) el.querySelector('#hm-tool-guardrails-save')?.addEventListener('click', saveToolGuardrails) el.querySelector('#hm-memory-save')?.addEventListener('click', saveMemory) + el.querySelector('#hm-skills-config-save')?.addEventListener('click', saveSkillsConfig) el.querySelector('#hm-streaming-save')?.addEventListener('click', saveStreaming) el.querySelector('#hm-execution-limits-save')?.addEventListener('click', saveExecutionLimits) el.querySelector('#hm-terminal-save')?.addEventListener('click', saveTerminal) @@ -624,6 +667,11 @@ export function render() { memoryValues = { ...MEMORY_DEFAULTS, ...(data?.values || {}) } } + async function loadSkillsConfig() { + const data = await api.hermesSkillsConfigRead() + skillsValues = { ...SKILLS_DEFAULTS, ...(data?.values || {}) } + } + async function loadStreaming() { const data = await api.hermesStreamingConfigRead() streamingValues = { ...STREAMING_DEFAULTS, ...(data?.values || {}) } @@ -645,6 +693,7 @@ export function render() { compressionLoading = true toolGuardrailsLoading = true memoryLoading = true + skillsLoading = true streamingLoading = true executionLimitsLoading = true terminalLoading = true @@ -653,6 +702,7 @@ export function render() { compressionError = null toolGuardrailsError = null memoryError = null + skillsError = null streamingError = null executionLimitsError = null terminalError = null @@ -720,6 +770,14 @@ export function render() { memoryLoading = false draw() } + try { + await loadSkillsConfig() + } catch (err) { + skillsError = humanizeError(err, t('engine.hermesSkillsConfigLoadFailed') || 'Load skills config failed') + } finally { + skillsLoading = false + draw() + } } async function refreshRawAfterStructuredSave() { @@ -753,6 +811,9 @@ export function render() { try { await loadMemory() } catch {} + try { + await loadSkillsConfig() + } catch {} try { await loadStreaming() } catch {} @@ -892,6 +953,32 @@ export function render() { } } + async function saveSkillsConfig() { + const form = { + creationNudgeInterval: el.querySelector('#hm-skills-creation-nudge-interval')?.value || '15', + externalDirs: el.querySelector('#hm-skills-external-dirs')?.value || '', + } + skillsSaving = true + skillsError = null + draw() + try { + const result = await api.hermesSkillsConfigSave(form) + skillsValues = { ...SKILLS_DEFAULTS, ...(result?.values || form) } + await refreshRawAfterStructuredSave() + const backup = result?.backup || '' + toast({ + message: t('engine.hermesSkillsConfigSaveSuccess'), + hint: backup ? t('engine.hermesConfigBackupHint', { path: backup }) : '', + }, 'success') + } catch (err) { + skillsError = humanizeError(err, t('engine.hermesSkillsConfigSaveFailed') || 'Save skills config failed') + toast(skillsError, 'error') + } finally { + skillsSaving = false + draw() + } + } + async function saveStreaming() { const form = { enabled: !!el.querySelector('#hm-streaming-enabled')?.checked, diff --git a/src/lib/tauri-api.js b/src/lib/tauri-api.js index f631645..382f2ac 100644 --- a/src/lib/tauri-api.js +++ b/src/lib/tauri-api.js @@ -517,6 +517,8 @@ export const api = { hermesToolLoopGuardrailsConfigSave: (form) => invoke('hermes_tool_loop_guardrails_config_save', { form }), hermesMemoryConfigRead: () => invoke('hermes_memory_config_read'), hermesMemoryConfigSave: (form) => invoke('hermes_memory_config_save', { form }), + hermesSkillsConfigRead: () => invoke('hermes_skills_config_read'), + hermesSkillsConfigSave: (form) => invoke('hermes_skills_config_save', { form }), hermesStreamingConfigRead: () => invoke('hermes_streaming_config_read'), hermesStreamingConfigSave: (form) => invoke('hermes_streaming_config_save', { form }), hermesExecutionLimitsConfigRead: () => invoke('hermes_execution_limits_config_read'), diff --git a/src/locales/modules/engine.js b/src/locales/modules/engine.js index 4d8bb20..46dd430 100644 --- a/src/locales/modules/engine.js +++ b/src/locales/modules/engine.js @@ -610,6 +610,16 @@ export default { hermesMemoryConfigNudgeInterval: _('整理提醒间隔', 'Review nudge interval', '整理提醒間隔'), hermesMemoryConfigFlushMinTurns: _('退出/重置前最少轮数', 'Minimum turns before flush', '退出/重置前最少輪數'), hermesMemoryConfigFootnote: _('提醒间隔按用户消息轮数计算,设为 0 可关闭提醒。flush 最小轮数会影响退出、重置和压缩前是否先写入记忆。外部记忆 provider 等高级字段会保留在 raw YAML 中。', 'The nudge interval is counted in user turns. Set it to 0 to disable nudges. flush minimum turns controls whether memory is written before exit, reset, or compression. Advanced fields such as external memory provider are preserved in raw YAML.', '提醒間隔依使用者訊息輪數計算,設為 0 可關閉提醒。flush 最小輪數會影響退出、重置和壓縮前是否先寫入記憶。外部記憶 provider 等進階欄位會保留在 raw YAML 中。'), + hermesSkillsConfigTitle: _('技能沉淀', 'Skill capture', '技能沉澱'), + hermesSkillsConfigDesc: _('控制 Hermes 何时提醒把可复用经验沉淀为 Skills,并指定可共享的外部技能目录。适合长跑任务和团队复用。', 'Control when Hermes nudges users to turn reusable experience into Skills and which external skill directories are shared. Useful for long-running work and team reuse.', '控制 Hermes 何時提醒把可重複使用經驗沉澱為 Skills,並指定可共享的外部技能目錄。適合長跑任務和團隊複用。'), + hermesSkillsConfigStatusReady: _('结构化配置', 'structured settings', '結構化設定'), + hermesSkillsConfigSave: _('保存技能配置', 'Save skill settings', '儲存技能設定'), + hermesSkillsConfigSaveSuccess: _('技能配置已保存,建议重启 Hermes Gateway 生效', 'Skill settings saved. Restart Hermes Gateway to take effect.', '技能設定已儲存,建議重啟 Hermes Gateway 生效'), + hermesSkillsConfigLoadFailed: _('加载技能配置失败', 'Load skill settings failed', '載入技能設定失敗'), + hermesSkillsConfigSaveFailed: _('保存技能配置失败', 'Save skill settings failed', '儲存技能設定失敗'), + hermesSkillsConfigCreationNudgeInterval: _('创建提醒间隔', 'Creation nudge interval', '建立提醒間隔'), + hermesSkillsConfigExternalDirs: _('外部技能目录(每行一个)', 'External skill directories, one per line', '外部技能目錄(每行一個)'), + hermesSkillsConfigFootnote: _('提醒间隔按用户消息轮数计算,设为 0 可关闭创建提醒。disabled、custom flag 等高级字段会保留在 raw YAML 中。', 'The nudge interval is counted in user turns. Set it to 0 to disable creation nudges. Advanced fields such as disabled skills and custom flags are preserved in raw YAML.', '提醒間隔依使用者訊息輪數計算,設為 0 可關閉建立提醒。disabled、custom flag 等進階欄位會保留在 raw YAML 中。'), // Batch 1 §E: 会话导出 sessionsExport: _('导出', 'Export', '匯出'), sessionsExportSuccess: _('已导出', 'Exported', '已匯出'), diff --git a/tests/hermes-config-page-ui.test.js b/tests/hermes-config-page-ui.test.js index 82b444f..0f010a5 100644 --- a/tests/hermes-config-page-ui.test.js +++ b/tests/hermes-config-page-ui.test.js @@ -39,6 +39,16 @@ test('Hermes 配置页会暴露记忆结构化配置字段', () => { } }) +test('Hermes 配置页会暴露 Skills 结构化配置字段', () => { + for (const id of [ + 'hm-skills-config-save', + 'hm-skills-creation-nudge-interval', + 'hm-skills-external-dirs', + ]) { + assert.match(source, new RegExp(`id="${id}"`), `缺少 ${id}`) + } +}) + test('Hermes 配置页会暴露网关流式结构化配置字段', () => { for (const id of [ 'hm-streaming-save', @@ -97,6 +107,7 @@ test('Hermes 配置页新增结构化配置不会暴露翻译 key', () => { const keys = new Set(extractEngineKeys().filter(key => ( key.includes('ToolGuardrails') || key.includes('MemoryConfig') || + key.includes('SkillsConfig') || key.includes('StreamingConfig') || key.includes('ExecutionLimits') || key.includes('TerminalConfig') diff --git a/tests/hermes-skills-config.test.js b/tests/hermes-skills-config.test.js new file mode 100644 index 0000000..ab1e447 --- /dev/null +++ b/tests/hermes-skills-config.test.js @@ -0,0 +1,61 @@ +import test from 'node:test' +import assert from 'node:assert/strict' + +import { + buildHermesSkillsConfigValues, + mergeHermesSkillsConfig, +} from '../scripts/dev-api.js' + +test('Hermes Skills 配置读取会提供上游默认值', () => { + const values = buildHermesSkillsConfigValues({}) + + assert.deepEqual(values, { + creationNudgeInterval: 15, + externalDirs: '', + }) +}) + +test('Hermes Skills 配置读取会回显创建提醒和外部目录', () => { + const values = buildHermesSkillsConfigValues({ + skills: { + creation_nudge_interval: 30, + external_dirs: ['~/.agents/skills', '/home/shared/team-skills'], + }, + }) + + assert.equal(values.creationNudgeInterval, 30) + assert.equal(values.externalDirs, '~/.agents/skills\n/home/shared/team-skills') +}) + +test('Hermes Skills 配置保存会保留未知字段并写入上游结构', () => { + const next = mergeHermesSkillsConfig({ + model: { provider: 'anthropic' }, + skills: { + creation_nudge_interval: 15, + disabled: ['legacy-skill'], + custom_flag: 'keep-skills', + }, + memory: { memory_enabled: true }, + }, { + creationNudgeInterval: '0', + externalDirs: ' ~/.agents/skills \n\n /home/shared/team-skills ', + }) + + assert.deepEqual(next.model, { provider: 'anthropic' }) + assert.deepEqual(next.memory, { memory_enabled: true }) + assert.equal(next.skills.creation_nudge_interval, 0) + assert.deepEqual(next.skills.external_dirs, ['~/.agents/skills', '/home/shared/team-skills']) + assert.deepEqual(next.skills.disabled, ['legacy-skill']) + assert.equal(next.skills.custom_flag, 'keep-skills') +}) + +test('Hermes Skills 配置保存会拒绝非法提醒间隔', () => { + assert.throws( + () => mergeHermesSkillsConfig({}, { creationNudgeInterval: '-1' }), + /skills\.creation_nudge_interval/, + ) + assert.throws( + () => mergeHermesSkillsConfig({}, { creationNudgeInterval: '10001' }), + /skills\.creation_nudge_interval/, + ) +})