Skip to content

llm -c should re-add Toolbox tools with serialized configuration #1092

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

Open
simonw opened this issue May 26, 2025 · 2 comments
Open

llm -c should re-add Toolbox tools with serialized configuration #1092

simonw opened this issue May 26, 2025 · 2 comments

Comments

@simonw
Copy link
Owner

simonw commented May 26, 2025

Spotted working on https://github.com/simonw/llm-tools-datasette

llm -T 'Datasette("https://datasette.io/content")' 'show tables' --td
llm -c 'Do something neat with the views' # Does not work

This relates to:

Need to log that initial setting in order to be able to reuse it on subsequent llm -c calls.

@simonw simonw changed the title llm -c does not carry over tools that use classes llm -c does not carry over toolbox tools May 26, 2025
@simonw
Copy link
Owner Author

simonw commented May 26, 2025

I'm dropping this out of the stable release, see rationale here:

I'll record the information but the actual code that enforces serializability and re-inflates objects feels unnecessary for this first release - I'll have llm -c just NOT support class-based tools, and I'll document that somewhere.

simonw added a commit that referenced this issue May 27, 2025
simonw added a commit that referenced this issue May 27, 2025
@simonw simonw changed the title llm -c does not carry over toolbox tools llm -c should work with configured toolbox tools May 27, 2025
@simonw
Copy link
Owner Author

simonw commented May 27, 2025

Update this when I implement this feature:

Toolboxes are not currently supported with the `llm -c` option, but they work well with `llm chat`. Try chatting with the Datasette content database like this:

@simonw simonw changed the title llm -c should work with configured toolbox tools llm -c should re-add Toolbox tools with serialized configuration Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant