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
|
if policy.skills_enabled {
|
||||||
.enabled_skills
|
|
||||||
.as_deref()
|
|
||||||
.map(|s| !s.is_empty())
|
|
||||||
.unwrap_or(false);
|
|
||||||
|
|
||||||
if policy.skills_enabled && node_has_enabled_skills {
|
|
||||||
let mut tools = role.enabled_tools().map(|v| v.to_vec()).unwrap_or_default();
|
let mut tools = role.enabled_tools().map(|v| v.to_vec()).unwrap_or_default();
|
||||||
for decl in skill_function_declarations() {
|
for decl in skill_function_declarations() {
|
||||||
if !tools.contains(&decl.name) {
|
if !tools.contains(&decl.name) {
|
||||||
|
|||||||
@@ -259,7 +259,9 @@ impl GraphValidator {
|
|||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if let Err(e) = paths::validate_skill_name(name) {
|
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}"
|
"llm node 'enabled_skills' contains an invalid skill name: '{name}': {e}"
|
||||||
)));
|
)));
|
||||||
continue;
|
continue;
|
||||||
|
|||||||
Reference in New Issue
Block a user