search

vercel / modelfusion

The TypeScript library for building AI applications.
https://modelfusion.dev
MIT License
1.11k stars 80 forks source link

Update tool name pattern to accept `.` for allowing namespace on tools #227

Closed Necmttn closed 9 months ago

Necmttn commented 9 months ago

So we can do for example:

bank.balance()
wallet.balance()
vercel[bot] commented 9 months ago

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

Name Status Preview Comments Updated (UTC)
modelfusion ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 2, 2024 3:40am
lgrammel commented 9 months ago

The check was orginally in there for openai functions. With tools, it has been removed. I'll see if I can fully remove it.

image

lgrammel commented 9 months ago

Removed the check: https://github.com/lgrammel/modelfusion/commit/d47068793c300c02a9ca2815c0ecd60b721419ff

(will be part of the next release)

Necmttn commented 9 months ago

hah weird. Openai actually accepts . in the action definitions in the playground

CleanShot 2567-01-02 at 16 49 12@2x
lgrammel commented 9 months ago

Change included in: https://github.com/lgrammel/modelfusion/releases/tag/v0.112.0