[mponcet]

I don't want pi to execute (risky) bash commands 😅

[pablopunk]

great one! given pi.nvim has no interaction other than reading and writing files, I don't think we should allow users to disable read/write tools, so the table should not consider those and always pass them down

Args	Send
tools = {}	--tools read,edit
tools = { "read" }	--tools read,edit
tools = { "bash" }	--tools read,edit,bash

do you agree?

also a unit test would help here

[mponcet]

Yes it makes sense. I added a unit test to validate this behavior ( tools = { "bash" } should give --tools read,edit,bash)

Also added some basic validation/type check of opts.tools .

[pablopunk]

thank uuuuuuuu