fix: accidental regression on enabled_skills being empty = all
This commit is contained in:
+1
-7
@@ -129,13 +129,7 @@ async fn run(
|
||||
}
|
||||
};
|
||||
|
||||
let node_has_enabled_skills = node
|
||||
.enabled_skills
|
||||
.as_deref()
|
||||
.map(|s| !s.is_empty())
|
||||
.unwrap_or(false);
|
||||
|
||||
if policy.skills_enabled && node_has_enabled_skills {
|
||||
if policy.skills_enabled {
|
||||
let mut tools = role.enabled_tools().map(|v| v.to_vec()).unwrap_or_default();
|
||||
for decl in skill_function_declarations() {
|
||||
if !tools.contains(&decl.name) {
|
||||
|
||||
@@ -259,7 +259,9 @@ impl GraphValidator {
|
||||
continue;
|
||||
}
|
||||
if let Err(e) = paths::validate_skill_name(name) {
|
||||
result.error(ValidationError::new(format!(
|
||||
result.error(ValidationError::with_node(
|
||||
node_id,
|
||||
format!(
|
||||
"llm node 'enabled_skills' contains an invalid skill name: '{name}': {e}"
|
||||
)));
|
||||
continue;
|
||||
|
||||
Reference in New Issue
Block a user