Skip to content

refactor: introduce tool metadata factory interface #249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

chr-hertel
Copy link
Member

@chr-hertel chr-hertel commented Mar 12, 2025

Preparation for different tool metadata sources, e.g. config or mcp based

@chr-hertel chr-hertel force-pushed the refactor-toolbox-construct branch from eccbc3b to 9adbbb9 Compare March 13, 2025 20:48
chr-hertel added a commit that referenced this pull request Mar 13, 2025
Base automatically changed from refactor-toolbox-construct to main March 13, 2025 20:50
@chr-hertel chr-hertel force-pushed the refactor-tool-metadata-factory branch 3 times, most recently from b558cc9 to 85aac43 Compare March 13, 2025 21:10
@chr-hertel chr-hertel force-pushed the refactor-tool-metadata-factory branch from 85aac43 to 1136710 Compare March 13, 2025 21:15
@chr-hertel
Copy link
Member Author

Refactored again to keep it simple, could use a ChainFactory if we want to combine multiple factories instead of injecting a collection of factories.

@chr-hertel chr-hertel requested a review from OskarStark March 13, 2025 22:05
@chr-hertel chr-hertel merged commit 6a5dbe7 into main Mar 13, 2025
7 checks passed
@chr-hertel chr-hertel deleted the refactor-tool-metadata-factory branch March 13, 2025 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants