masci / banks

95%

push

github

fix: resolve tool callables from an explicit registry instead of importlib (#75)

The `tool` filter now registers each callable by name in
`CompletionExtension._callable_registry` at render time. `_get_tool_callable`
looks up callables exclusively from this registry — `importlib.import_module`
is no longer used — so only developer-supplied functions are ever invocable.

`Tool.import_path` is also excluded from the payload sent to the LLM, as the
field carries no meaning for the model and should not be exposed.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

102 of 108 branches covered (94.44%)

Branch coverage included in aggregate %.

17 of 17 new or added lines in 2 files covered. (100.0%)

846 of 892 relevant lines covered (94.84%)

0.95 hits per line