SkillRegistry.validate_plan() and validate_skill() called the non-existent
ToolRegistry.get_tool() method, causing AttributeError at runtime. The correct
method is ToolRegistry.get(), which is already used by SkillRegistry.refresh().
Changes:
- Replace self._tool_registry.get_tool(entry.name) with self._tool_registry.get(entry.name)
in validate_plan()
- Replace self._tool_registry.get_tool(ref) with self._tool_registry.get(ref)
in validate_skill()
- Update Behave BDD tests to use mock.get instead of mock.get_tool
- Add new BDD scenarios covering validate_skill() happy path, not-found path,
unregistered include path, and no-tool-registry path
- Remove stale get_tool mock setup from test helpers
ISSUES CLOSED: #2914
freemo