Skip to content

Tools improvements#181

Merged
virrius merged 3 commits intomainfrom
tools-improvements
Mar 12, 2026
Merged

Tools improvements#181
virrius merged 3 commits intomainfrom
tools-improvements

Conversation

@virrius
Copy link
Member

@virrius virrius commented Mar 7, 2026

Tool build logic has been reorganized

All transformations, imports, and configuration assembly have been moved to the first stage at the model validator level. This allows us to fail fast if there are issues and centrally validate the input configuration before the agent is built at runtime.

The factory has been simplified.

The agent, tools, and other components now use a unified ToolDefinition data model and are no longer aware of the multi-level configuration or other related complexities.

The number of dictionaries / non-validated non-model data structures in the agent and tool logic stages has been reduced.

Search has been moved to the tool configuration level, and the related parts in the documentation have been fixed.

Additional tests have been added for these cases.


Упорядочена логика сборки тулов

  • все преобразования, импорты и составление конфига вынесены на первый этап на уровень валидаторов модели. Что позволяет нам быстро падать в случае проблем и централизованно валидировать входную конфигурацию прежде чем агент будет собран в рантайме
  • Упрощена фабрика
  • Агент, тулы и далее используют единообразную модель данных ToolDefinition, ничего не знают про многоуровневый конфиг и любые прочие проблемы
  • Сокращено количество словарей/невалидированных немодельных данных на этапах логики агентов и тулов Search\

перенесён на уровень конфигурации тулов, пофикшены эти моменты в документации
Добавлены дополнительные тесты на эти кейсы

image

@virrius virrius self-assigned this Mar 7, 2026
EvilFreelancer
EvilFreelancer previously approved these changes Mar 11, 2026
@virrius virrius merged commit cead7be into main Mar 12, 2026
2 checks passed
@virrius virrius deleted the tools-improvements branch March 12, 2026 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants