search

dexaai / dexter

LLM tools used in production at Dexa
https://dexter.dexa.ai
MIT License
69 stars 4 forks source link

Fix `deepMerge` for array of tool objects #42

Open rileytomasek opened 1 month ago

rileytomasek commented 1 month ago

This makes it so that the deepMerge function can handle merging arrays of tool objects.

Resolves #40

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dexter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2024 0:50am
rileytomasek commented 1 month ago

@colelawrence this fixes the most basic case that is meant more as a demonstration.

This will merge the tool objects based on order, which almost definitely isn't what we want. We probably want a smarter merge by function name for cases where they aren't in the same order.