Skip to content

Commit 4929ea8

Browse files
committed
fix: correct logging inconsistencies in CLI validator
- Remove leading spaces from log messages for consistency - Change 'detailed_analysis' from 'LangGraph execution' to 'CLI tool execution' - Ensure all logging follows consistent patterns with other agents
1 parent 30bbc33 commit 4929ea8

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

agents/validate/ansible_lint_validator.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ async def validate_playbook(
195195
"validator_pattern": "cli_tool_wrapper",
196196
"json_extracted": True,
197197
"tool_names_called": ["ansible_lint_tool"],
198-
"detailed_analysis": "LangGraph execution",
198+
"detailed_analysis": "CLI tool execution",
199199
"actual_tool_calls": True
200200
},
201201
"session_info": {
@@ -212,7 +212,7 @@ async def validate_playbook(
212212
"elapsed_time": round(total_time, 3)
213213
}
214214

215-
logger.info(f" Ansible-lint CLI validation completed successfully in {total_time:.3f}s")
215+
logger.info(f"Ansible-lint CLI validation completed successfully in {total_time:.3f}s")
216216
return result
217217

218218
except Exception as e:
@@ -331,7 +331,7 @@ async def validate_syntax(
331331
"method_used": "cli_validation"
332332
}
333333

334-
logger.info(f" Ansible-lint CLI syntax validation completed: {'valid' if syntax_result['syntax_valid'] else 'invalid'}")
334+
logger.info(f"Ansible-lint CLI syntax validation completed: {'valid' if syntax_result['syntax_valid'] else 'invalid'}")
335335
return syntax_result
336336

337337
except Exception as e:
@@ -415,11 +415,11 @@ async def health_check(self) -> bool:
415415
result = await self.validate_playbook(test_playbook, "basic", "health-check")
416416

417417
# If we get here, the validation worked
418-
logger.info(" Ansible-lint CLI health check completed successfully")
418+
logger.info("Ansible-lint CLI health check completed successfully")
419419
return True
420420

421421
except Exception as e:
422-
logger.error(f" Ansible-lint CLI health check failed: {e}")
422+
logger.error(f"Ansible-lint CLI health check failed: {e}")
423423
return False
424424

425425
def get_status(self) -> Dict[str, Any]:

0 commit comments

Comments
 (0)