dexter icon indicating copy to clipboard operation
dexter copied to clipboard

Fix `deepMerge` for array of tool objects

Open rileytomasek opened this issue 1 year ago • 4 comments

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

Resolves #40

rileytomasek avatar Jul 17 '24 12:07 rileytomasek

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

vercel[bot] avatar Jul 17 '24 12:07 vercel[bot]

@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.

rileytomasek avatar Jul 17 '24 12:07 rileytomasek

@charliehelps this is a test

stevepeak avatar Mar 03 '25 22:03 stevepeak

Hi @stevepeak, thanks for the test! I'm here and listening. (Message from charlie)

charliecreates[bot] avatar Mar 03 '25 22:03 charliecreates[bot]