search

mdenet / educationplatform

Eclipse Public License 2.0
2 stars 3 forks source link

Refactor tool function calling code #193

Closed barnettwilliam closed 3 months ago

barnettwilliam commented 4 months ago

Long awaited changes for #40 refactor tool function calling code. Mostly moving tool function and conversion function invocation to ToolsManager and FunctionRegistry.

There is still moving the getActionFunction() from ToolsManager into FunctionRegistry todo but raised as issue #192 because it requires some rework of FunctionRegistry

barnettwilliam commented 4 months ago

Ok, I can write tests for the existing implementation and then use them here to verify these changes

barnettwilliam commented 3 months ago

Rebased to include the tests from PR #201 and updated tests for the relocated functions.