eidolon-ai / eidolon

An Open Source Agent Service Framework
https://www.eidolonai.com/
Apache License 2.0
187 stars 18 forks source link

Ll/json schema fix #411

Closed LukeLalor closed 1 month ago

LukeLalor commented 1 month ago

I would like to do a refactor of the function handler since the lambda constructors are exceptionally confusing and the json schema back/forth creates issues like this.

I will create a second pr for that though, and this is just to unblock main

The issue was caused by nested json schema defs with the same name stomping on each other when we construct a json schema from a base model constructed from json schema 🙈