Cleanup model blend cursorless grammar

C-Loftus / talon-ai-tools

Query LLMs and AI tools with voice commands

MIT License

39 stars 14 forks source link

Cleanup model blend cursorless grammar #81

Closed jaresty closed 3 weeks ago

jaresty commented 3 weeks ago

Model blend to didn't specify a source; now it specifies the source as clipboard
Model blend was only implicitly dropping the 'this'

pokey commented 3 weeks ago

This seems consistent. I think for convenience it would also be good to have:

"model paste" or "model paste that"
"model paste to "

Maybe we already have these. But they would be pretty consistent with "paste that" and "paste to "

jaresty commented 3 weeks ago

I like those. Updated.

jaresty commented 3 weeks ago

I'm ok with either of these. Can you discuss at the meetup and make a call which way to go? I'll be afk today but I'm ok if you want to make whatever change you agree to and merge it.

jaresty commented 3 weeks ago

If you would prefer I change it, just let me know what to do and I'll put it together tonight if I can find some time. Please feel free to edit it though. The current cursorless blend command is missing a clipboard option so wanted to fix that up.

jaresty commented 3 weeks ago

I think this aligns with the current grammar. I wasn't sure if I should be adding something for 'paste'.

C-Loftus commented 3 weeks ago

Looks good thanks!