polymech/zeroclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(skills): normalize path separators to forward slashes for XML output

Browse Source 
On Windows, file paths use backslashes (\) but the test expected forward
slashes (/). The render_skill_location function now normalizes all path
separators to forward slashes for consistent XML output across platforms.

This fixes the failing test:
prompt_skills_compact_mode_omits_instructions_and_tools

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
killf 2026-03-04 09:20:47 +08:00
parent 535f23b5c3
commit c697497e34
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/skills/mod.rs
View File

@ -764,12 +764,16 @@ fn resolve_skill_location(skill: &Skill, workspace_dir: &Path) -> PathBuf {
fn render_skill_location(skill: &Skill, workspace_dir: &Path, prefer_relative: bool) -> String {
let location = resolve_skill_location(skill, workspace_dir);
if prefer_relative {
if let Ok(relative) = location.strip_prefix(workspace_dir) {
return relative.display().to_string();
let path_str = if prefer_relative {
match location.strip_prefix(workspace_dir) {
Ok(relative) => relative.display().to_string(),
Err(_) => location.display().to_string(),
}
}
location.display().to_string()
} else {
location.display().to_string()
};
// Normalize path separators to forward slashes for XML output (portable across Windows/Unix)
path_str.replace('\\', "/")
}
/// Build the "Available Skills" system prompt section with full skill instructions.