2 Commits

Author	SHA1	Message	Date
freemo	9f62f9c9c8	fix(skills): replace get_tool() calls with get() in SkillRegistry validation methods ... 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	2026-04-05 08:59:59 +00:00
freemo	a808c395f9	test(coverage): add Behave BDD tests to improve unit test coverage across 53 source modules ... Add 53 new .feature files and corresponding step definition files targeting uncovered lines identified in build/coverage.xml. Fix AmbiguousStep conflicts in 7 pre-existing step files by disambiguating step text. New tests cover: ACP clients/facade, actor CLI/config, application container, ACMS service/strategies, async worker, automation profile CLI, autonomy guardrail, bridge, change model, config CLI/service, context service, cross-plan correction, database models, decision service, decomposition clustering/service, discovery handler, langchain chat provider, langgraph nodes, materializers, multi-project service, plan apply/CLI/lifecycle/model/ preflight/resume/service, PostgreSQL analyzer, project CLI/context CLI, provider registry, reactive application/route, repositories, resolver handler, resource registry service, resume model, retry patterns, sandbox protocol, server CLI, skill CLI/service, skills registry, subplan execution/service, system CLI, UKO loader, UoW, and YAML template engine. Closes #645	2026-03-09 13:01:58 -04:00