Add --json mode to binaries#49
Conversation
| // Call the appropriate tool | ||
| let response_content = call_tool(name, args, &files, model).await?; | ||
| let response_content = | ||
| call_tool(name, args, &files, model, &mut result).await?; |
There was a problem hiding this comment.
I don't like how I need to pass in this mutable list reference, but I don't see a better way
There was a problem hiding this comment.
Yeah, I think this is what makes the most sense
AstraBert
left a comment
AstraBert
left a comment
There was a problem hiding this comment.
Looks good!
At this point, I think it could be great to add a semtools skills with examples of the things an LLM can do with semtools. I think it can be very simple (like this one I wrote), but would be helpful in giving the LLM an idea of how to use jq with JSON mode :))
| // Call the appropriate tool | ||
| let response_content = call_tool(name, args, &files, model).await?; | ||
| let response_content = | ||
| call_tool(name, args, &files, model, &mut result).await?; |
There was a problem hiding this comment.
Yeah, I think this is what makes the most sense
|
@AstraBert an example skill would be great. However, I had pretty bad luck the last time I tried to add a skill for semtools. It felt like I couldn't reliably get it to activate the skill and it would default to grepping for things etc. Part of me wonders if this type of knowledge just always needs to be in the prompt? Lemme know if you try it out though |
2 participants
Adds a
--jsonmode to all binaries, to facilitate usage with thejqtool in bash. This is a pretty common tool for coding agents, so I can see this helping a lotThis PR is also way bigger than I expected. I should have added structured outputs instead of strings from the beginning 😢
Fixes #29
Fixes #28